plan-review-loop
Use when a completed /plan needs explicit multi-reviewer approval and refinement before implementation begins.
Version
1.0.0
Maturity
draft
Repository
agent-skills
License
GNU GPL v3
Skill metadata
SKILL.md
Plan review loop
Use this skill to run structured plan reviews after /plan completes, with customizable reviewer personas and explicit approval gates.
Use this skill when
- A
/planis complete and you want reviewers (Jason and Freddy by default) to analyze it before implementation begins - A plan was revised after reviewer feedback and needs another structured approval round
- You want documented approval gates with explicit verdict tokens before committing to an implementation
- You need feedback from implementation-risk and architectural-risk perspectives on the same plan
Do not use this skill when
- You do not yet have a completed
/planto review (use/planfirst) - You need ad-hoc human review outside the structured skill workflow (do that separately)
- The decision is already made and the plan is final (no review needed)
Routing boundary
| Situation | Use this skill? | Route instead |
|---|---|---|
| A finished plan needs structured multi-reviewer approval before implementation | Yes | — |
| You want stress-testing of a design and CONTEXT.md updated | No | grill-with-docs |
| You want fast informal critique without a full review loop | No | grill-me |
| The plan is already approved and implementation has started | No | proceed with implementation |
Inputs to gather
Required
- A completed
/planoutput from the current session
Optional
- Custom reviewer personas (default: Jason and Freddy) defined in
references/personas/ - Prior reviewer feedback to incorporate if re-invoking for another round
First move
- After
/plancompletes, invoke the skill explicitly:Use the /plan-review-loop skill to review and refine the current plan - Reviewers analyze the plan according to their personas and rubrics
- Each reviewer returns a verdict token:
[PLAN-APPROVED]or[PLAN-REVISE-NEEDED] - If all return
[PLAN-APPROVED]in the same round → plan is ready - If any returns
[PLAN-REVISE-NEEDED]→ update your plan and invoke the skill again - If you reach 3 rounds without unanimous approval → the skill exits and you decide the next step
Workflow
- Invoke: After
/planfinishes, explicitly request the review loop skill - Review Round: Each configured reviewer (Jason and Freddy by default) analyzes the plan
- Collect verdicts: Reviewers return tokens; parse the explicit
[PLAN-APPROVED]or[PLAN-REVISE-NEEDED] - Decision:
- All reviewers return
[PLAN-APPROVED]in the same round → ✅ plan is approved, proceed to implementation - At least one returns
[PLAN-REVISE-NEEDED]→ ❌ plan needs revision; update it and request another review - Round 3 completes without unanimous approval → ⏱️ max rounds reached; you decide next steps
- All reviewers return
- Customize (optional): Edit or create
references/personas/*.mdto replace or add reviewers
Outputs
- Approval verdict: unanimous
[PLAN-APPROVED]in the same round (plan is ready to implement) or[PLAN-REVISE-NEEDED]with reviewer notes (plan needs revision) - Round number when approval was reached or a statement that the 3-round maximum was hit without consensus
Guardrails
- Do not modify the plan yourself between rounds; the skill parses reviewer tokens to determine completion status
- Do not exceed 3 review rounds without explicit decision outside this skill
- Verdict tokens are case-sensitive and exact:
[PLAN-APPROVED]and[PLAN-REVISE-NEEDED]only - Missing, malformed, or ambiguous tokens (multiple tokens in one response) block approval
- All configured reviewers must return a verdict in the same round for unanimous approval
Validation
The skill validates successfully when:
All persona files in
references/personas/conform to the schema documented inreferences/personas/README.mdVerdict tokens (
[PLAN-APPROVED]/[PLAN-REVISE-NEEDED]) are preserved unchanged and exactly matchedThe round logic enforces a 3-round maximum and unanimous same-round approval requirement
Smoke test:
- should trigger: "Use Jason and Freddy to review this finished /plan before implementation."
- should not trigger: "Stress-test this design and update CONTEXT.md as we go." (→
grill-with-docs)
Examples
Approve path (happy path):
You: Use the /plan-review-loop skill to review and refine the current plan Jason: [PLAN-APPROVED] The tasks are well-structured and validation is solid. Freddy: [PLAN-APPROVED] Architecture is clear and risk mitigation is covered. Result: ✅ Plan approved! Ready to proceed.Revise path (iteration):
You: Use the /plan-review-loop skill to review and refine the current plan Jason: [PLAN-REVISE-NEEDED] Missing rollback procedure in task 5. Freddy: [PLAN-APPROVED] Architecture looks good. Result: ❌ Not approved (Jason needs revision). Update the plan and request another round.Customization:
Create or edit references/personas/alice.md to add a custom security reviewer. Then invoke: Use the /plan-review-loop skill with Alice, Jason, and Freddy as reviewers.
Reference files
references/personas/README.md— persona schema, YAML frontmatter, and customization guidereferences/review-contract.md— detailed verdict tokens, round rules, and approval criteria../plan-review/references/personas/jason.md— Jason persona (implementation/execution focus)../plan-review/references/personas/freddy.md— Freddy persona (architecture/risk focus)assets/smoke-test-prompts.md— manual validation scenarios (5 test cases: approve, revise, mixed, max-rounds, persona-swap)