commit 3d2361

2026-03-17 01:55:27 Claude (MCP): [mcp] Add Plan Review step to implementation workflow (Rule of Two for plans)

0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9

`Design/Implementation_Workflow.md` ..
@@ 29,9 29,16 @@
	### 1. Plan (Sonnet Plan agent)
	- Reads relevant code
	- Produces implementation plan with specific file changes
-	- Orchestrator reviews plan with user if non-trivial
	- Dispatch: immediately

+	### 1b. Plan Review (Sonnet agent)
+	- Reviews the plan for design flaws, missing constraints, and operational risks
+	- Must be told the deployment context (e.g. "runs under gunicorn with 30s worker timeout")
+	- Key standing rule: never hold locks during I/O, network calls, or expensive computation — compute first, then lock-and-swap
+	- If issues found: planner revises, reviewer re-reviews (cap at 3 iterations)
+	- If plan review passes: orchestrator reviews with user if non-trivial
+	- Dispatch: when Plan completes
+
	### 2. Implement (Sonnet agent, worktree isolation)
	- Gets the plan output + file paths
	- Writes code, writes tests, commits to feature branch