coursecast 0.3.1 → 0.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coursecast",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "One prompt → a full interactive course — live widgets, quizzes, a mastery dashboard — rendered to self-contained HTML and deployed live.",
   "type": "module",
   "bin": {

package/src/engines/shared-prompts.mjs

@@ -141,10 +141,41 @@ export function validateLesson(obj) {
   if (obj.simulation) {
     if (!obj.simulation.html) e.push('"simulation" is missing "html"')
     if (!obj.simulation.js) e.push('"simulation" is missing "js"')
+    e.push(...validateSimulationWiring(obj.simulation))
   }
   return e
 }
 
+// The simulation's markup and its script are written as two separate strings, so a typo in
+// one doesn't show up until a learner clicks and the console throws. These checks catch the
+// two ways that actually happens, and because they run inside validate+repair the model gets
+// a chance to fix its own mistake before the lesson is ever rendered.
+function validateSimulationWiring({ html = '', js = '' }) {
+  const e = []
+  if (!html || !js) return e
+
+  // 1. inline handlers (onclick="doThing()") that name a function the script never defines
+  const handlers = new Set()
+  for (const m of String(html).matchAll(/\son[a-z]+\s*=\s*(["'])(.*?)\1/gi)) {
+    const fn = m[2].match(/^\s*([A-Za-z_$][\w$]*)\s*\(/)
+    if (fn) handlers.add(fn[1])
+  }
+  for (const fn of handlers) {
+    const defined = new RegExp(
+      `(function\\s+${fn}\\b|\\b(?:var|let|const)\\s+${fn}\\s*=|\\b${fn}\\s*=\\s*(?:function|\\()|window\\.${fn}\\s*=)`,
+    ).test(js)
+    if (!defined) e.push(`simulation.html calls "${fn}()" from an inline handler, but simulation.js never defines it`)
+  }
+
+  // 2. getElementById('x') where the markup has no element with that id
+  const ids = new Set([...String(html).matchAll(/\sid\s*=\s*(["'])(.*?)\1/gi)].map((m) => m[2].trim()))
+  for (const m of String(js).matchAll(/getElementById\(\s*(["'])(.*?)\1\s*\)/gi)) {
+    const want = m[2].trim()
+    if (want && !ids.has(want)) e.push(`simulation.js looks up id "${want}", which does not exist in simulation.html`)
+  }
+  return e.slice(0, 6)
+}
+
 export function validateSyllabus(obj) {
   const e = []
   if (!obj || typeof obj !== 'object') return ['the output was not a JSON object']