Implement SILENT_MODE to prevent reviewer from prompting user during autonomous loops

- Updates the reviewer agent to skip `vscode_askQuestions` when `SILENT_MODE` is present in the prompt.
- Ensures the engineer agent uses this mode during its review-fix iterations to maintain an uninterrupted workflow.
- Maintains interactive prompting only when the reviewer is invoked directly by a user.

Author: niemyjski

.claude/agents/engineer.md

@@ -178,8 +178,8 @@ Report back with: PASS or FAIL, and describe what you see.
 ```
 iteration = 0
 while iteration < 3:
-    # 4a: Review
-    invoke @reviewer with: scope, 1-sentence summary, list of modified files
+    # 4a: Review (ALWAYS include "SILENT_MODE" in the prompt so reviewer doesn't ask user)
+    invoke @reviewer with: SILENT_MODE, scope, 1-sentence summary, list of modified files
 
     if 0 findings: DONE → move to Step 5
 

.claude/agents/reviewer.md

@@ -188,12 +188,11 @@ End your review with:
 [One sentence on overall quality and most important finding]
 ```
 
-# Final Ask (Required)
-
-If reviewer is invoked directly by a user, call `vscode_askQuestions` (askuserquestion) before ending and include a concise findings summary in the prompt:
+# Final Behavior
 
+**Default (direct invocation by user):** After outputting the Summary block, call `vscode_askQuestions` (askuserquestion) with a concise findings summary:
 - Blockers count + top blocker
 - Warnings count + top warning
-- Ask whether to run a deeper pass, hand off to engineer, or stop
+- Ask whether to hand off to engineer, run a deeper pass, or stop
 
-If reviewer is invoked as a subagent by engineer, do **not** prompt the user. Return findings only and let engineer continue automatically into a deeper pass/fix loop.
+**When prompt includes "SILENT_MODE":** Do NOT call `vscode_askQuestions`. Output the Summary block and stop. Return findings only — the calling agent handles next steps. This mode is used when the engineer invokes you as part of its autonomous review-fix loop.