Update app.py

app.py

@@ -323,11 +323,17 @@ INTENT RULES:
 "curriculum" — on-topic. Follow: teach → re_teach → show_and_tell → assess.
 "chitchat" — casual. Respond warmly, bring up active topic.
 
-OUTPUT — return ONLY this JSON (stage_updates: EXACTLY 1 entry for the active objective only):
+STAGE VALUE RULES — CRITICAL:
+The fields teach / re_teach / show_and_tell / assess must ONLY ever be the exact string "complete" or "Not_Complete".
+NEVER put a sentence, description, or any other text in these fields.
+Set the current active stage ("{active_stage}") to "complete" if the student has completed it, else "Not_Complete".
+All other stages keep their previous value — use "Not_Complete" if unknown.
+
+OUTPUT — return ONLY this JSON (stage_updates: EXACTLY 1 entry):
 {{
   "intent": "<block|questions|curriculum|chitchat>",
   "response": "<reply, max 50 words>",
-  "stage_updates": [{{"topic":"{active_topic}","goal":"{active_goal}","teach":"<complete|Not_Complete>","re_teach":"<complete|Not_Complete>","show_and_tell":"<complete|Not_Complete>","assess":"<complete|Not_Complete>"}}],
+  "stage_updates": [{{"topic":"{active_topic}","goal":"{active_goal}","teach":"Not_Complete","re_teach":"Not_Complete","show_and_tell":"Not_Complete","assess":"Not_Complete"}}],
   "thought": "<one sentence>",
   "action": "<teach|re_teach|show_and_tell|assess|answer|redirect|discourage|end|chitchat>",
   "observation": "<one sentence>"
@@ -440,6 +446,10 @@ def _apply_state_updates(
     valid_statuses   = {"complete", "Not_Complete"}
 
     for upd in parsed.get("stage_updates", []):
+        # Sanitise: coerce any non-enum value to "Not_Complete"
+        for stage in ("teach", "re_teach", "show_and_tell", "assess"):
+            if upd.get(stage) not in valid_statuses:
+                upd[stage] = "Not_Complete"
         for item in current_learning:
             if item.get("topic") == upd.get("topic"):
                 for obj in item.get("learning_objectives", []):