agents/prompts/pm.md

@@ -131,6 +131,10 @@ Return ONLY valid JSON (no markdown, no explanation):
 }
 ```
 
+Valid values for `status`: `"done"`, `"blocked"`.
+
+If status is "blocked", include `"blocked_reason": "..."` and `"analysis": "..."` explaining why the task cannot be planned.
+
 ## Blocked Protocol
 
 If you cannot plan the pipeline (task is completely ambiguous, no information to work with, or explicitly outside the system scope), return this JSON **instead of** the normal output:

agents/runner.py

@@ -1800,7 +1800,7 @@ def run_pipeline(
                     pass
         else:
             # Review mode: wait for manual approval
-            models.update_task(conn, task_id, status="review", execution_mode="review")
+            models.update_task(conn, task_id, status="review")
 
         # Run post-pipeline hooks (failures don't affect pipeline status)
         try:

core/context_builder.py

@@ -332,6 +332,7 @@ def format_prompt(context: dict, role: str, prompt_template: str | None = None)
         sections.append("")
 
     # Revision context: director's comment + agent's previous output
+    task = context.get("task")
     if task and task.get("revise_comment"):
         sections.append("## Director's revision request:")
         sections.append(task["revise_comment"])