Write/Review Isolation — Two-Stage Review Protocol
Enforces a non-negotiable separation between the agent authoring code and the agent reviewing it. Loaded by agents that dispatch reviewers, author multi-stage pipelines, or implement tasks requiring formal verification against a specification.
The Two-Stage Protocol
STAGE 2 — CODE QUALITY REVIEW
- Agent:
pipeline-quality-reviewer. - Inputs: Same as Stage 1, plus the Stage 1
PASSverdict. - Goal: Evaluate maintainability, naming, separation of concerns, and edge-case handling.
- Exit Condition: Terminal
PASSonly when all critical and major issues are resolved.
The Review Loop
Red Flags — STOP
- "The output looks fine, I'll skip Stage 1." → STOP. Spec compliance is the primary defense against production regressions.
- "I'll mention this AC issue in the Stage 2 report." → STOP. Re-dispatch Stage 1 immediately.
- "I added a helper that wasn't in the spec, but it's clearly needed." → STOP. This is over-build. Re-architect the spec or remove the helper.
Rationalization Table
Reference Files
sk-pipeline-patterns/SKILL.md— Pattern 5 integration.pipeline-spec-reviewer.md— Spec reviewer blueprint.pipeline-quality-reviewer.md— Quality reviewer blueprint.sk-rationalization-resistance/SKILL.md— Resistance mechanism standards.