review-plan

name	review-plan
description	Review implementation plans for parallelization, TDD, types, libraries, and security before execution
disable-model-invocation	true

Review Plan

Review implementation plans created by superpowers:writing-plans before execution.

Arguments

• Path: Plan file to review (e.g., docs/plans/2025-01-15-auth-feature.md)

Hard gates (sequence)

Do not skip ahead; each step passes only when the condition is objectively satisfied (artifact path, tool success, or labeled capture—not “I read it mentally”).

1. Plan file reachable — Pass: Read (or equivalent) succeeds on Path; if not, stop and report the missing path. Pass: You can quote or point to where **Goal:**, **Architecture:**, and **Tech Stack:** appear, or you record “header field X absent” as a finding before Step 2.
2. Skills loaded before agents — Pass: For each row you will rely on in Step 2’s table, the corresponding skill is loaded via the Skill tool (or you record explicit N/A with reason, e.g. stack not present). Do not spawn Step 3 Task runs until this gate passes.
3. Five reviews captured — Pass: You have five labeled artifacts (one per agent): pasted outputs, task transcripts, or saved snippet files. Pass: Each of the five INVESTIGATE/CHECK/VERIFY prompts has a corresponding response block before Step 4.
4. Review file on disk before user prompt — Pass: The file at [plan-dir]/[plan-basename]-review.md exists; Pass: Read succeeds on that path. Only then run the “Next Steps” / options prompt in Step 5.

Step 1: Read and Parse Plan

Read the plan file and extract:

1. Header fields:

   • **Goal:** - Feature description
   • **Architecture:** - Approach summary
   • **Tech Stack:** - Technologies used

2. Verify via file patterns:

   • .py files → Python
   • .ts, .tsx files → TypeScript
   • .go files → Go
   • pytest commands → pytest
   • vitest, jest commands → JavaScript/TypeScript testing
   • go test commands → Go testing

Step 2: Load Skills

Use the Skill tool to load each applicable skill (e.g., Skill(skill: "beagle-python:python-code-review")).

Based on detected tech stack, load relevant skills:

Detected	Skill
Python	beagle-python:python-code-review
FastAPI	beagle-python:fastapi-code-review
SQLAlchemy	beagle-python:sqlalchemy-code-review
PostgreSQL	beagle-python:postgres-code-review
pytest	beagle-python:pytest-code-review
React Router	beagle-react:react-router-code-review
React Flow	beagle-react:react-flow-code-review
shadcn/ui	beagle-react:shadcn-code-review
vitest	beagle-react:vitest-testing
Go	beagle-go:go-code-review
BubbleTea	beagle-go:bubbletea-code-review

Step 3: Launch 5 Parallel Agents

Use the Task tool to spawn 5 agents simultaneously. Each receives:

• Full plan content
• Detected tech stack
• Relevant skill content from Step 2

Agent 1: Parallelization Analysis

Analyze whether this implementation plan can be executed by parallel subagents.

INVESTIGATE:
1. Which tasks can run in parallel (no dependencies between them)?
2. Which tasks must be sequential (Task B depends on Task A output)?
3. Are there any circular dependencies or blocking issues?
4. What is the critical path?

Return:
- Recommended batch structure for parallel execution
- Maximum concurrent agents
- Any blocking issues that prevent parallelization

Agent 2: TDD & Over-Engineering Check

Verify TDD discipline in this implementation plan.

CHECK each task for:
1. Tests written BEFORE implementation (RED phase)
2. Step to run test and verify it fails
3. Minimal implementation to make test pass (GREEN phase)
4. Tests focus on behavior, not implementation details

LOOK FOR over-engineering:
- Excessive mocking (testing implementation vs behavior)
- Too many abstraction layers
- Defensive code for impossible scenarios
- Premature optimization

Return: TDD adherence assessment and over-engineering concerns.

Agent 3: Type & API Verification

Verify types and APIs in the plan match the actual codebase.

SEARCH the codebase for:
1. All types referenced in the plan's code blocks
2. Existing type definitions
3. API endpoint contracts (request/response shapes)
4. Import paths

VERIFY:
1. All properties referenced exist in the types
2. Enum values match between plan and codebase
3. Import paths are correct
4. No type mismatches

Return: List of mismatches with file:line references.

Agent 4: Library Best Practices

Verify library usage in this plan follows best practices.

For each library referenced:
1. Are function signatures correct for current versions?
2. Are there deprecated APIs being used?
3. Does usage follow library documentation?
4. Are installation commands correct?

Check against loaded skills for technology-specific guidance.

Return: Incorrect API usage with recommendations.

Agent 5: Security & Edge Cases

Check for security gaps and missing error handling.

VERIFY:
1. Input validation at system boundaries
2. Error handling in API/DB operations
3. Auth/authz checks where needed
4. Edge cases are handled

Return: Security gaps and missing error handling.

Step 4: Synthesize Report

Gate: Hard gate 3 must pass (five labeled agent outputs present). After all agents complete, create consolidated report:

## Plan Review: [Feature Name from plan]

**Plan:** `[path to plan file]`
**Tech Stack:** [Detected technologies]

### Summary Table

| Criterion | Status | Notes |
|-----------|--------|-------|
| Parallelization | ✅ GOOD / ⚠️ ISSUES | [Brief note] |
| TDD Adherence | ✅ GOOD / ⚠️ ISSUES | [Brief note] |
| Type/API Match | ✅ GOOD / ⚠️ ISSUES | [Brief note] |
| Library Practices | ✅ GOOD / ⚠️ ISSUES | [Brief note] |
| Security/Edge Cases | ✅ GOOD / ⚠️ ISSUES | [Brief note] |

### Issues Found

#### Critical (Must Fix Before Execution)

1. [Task N, Step M] ISSUE_CODE
   - Issue: What's wrong
   - Why: Impact if not fixed
   - Fix: Specific change
   - Suggested edit:

[replacement content]

#### Major (Should Fix)

2. [Task N] ISSUE_CODE
- Issue: ...
- Why: ...
- Fix: ...

#### Minor (Nice to Have)

3. [Task N] ISSUE_CODE
- Issue: ...
- Fix: ...

### Verdict

**Ready to execute?** Yes | With fixes (1-N) | No

**Reasoning:** [1-2 sentence assessment]

Step 5: Save Review and Prompt

Gate: After writing the review file, satisfy Hard gate 4 (Read succeeds on the review path) before prompting the user.

Save review to same directory as plan:

• Plan: docs/plans/2025-01-15-feature.md
• Review: docs/plans/2025-01-15-feature-review.md

Review file header:

# Plan Review: [Feature Name]

> **To apply fixes:** Open new session, run:
> `Read this file, then apply the suggested fixes to [plan path]`

**Reviewed:** [Current date/time]
**Verdict:** [Yes | With fixes (1-N) | No]

---

Prompt user:

---

## Next Steps

**Review saved to:** `[review file path]`

**Options:**

1. **Apply fixes now** - Edit the plan file to address issues
2. **Save & fix later** - Open new session to apply fixes
3. **Proceed anyway** - Execute plan despite issues (not recommended for Critical)

Which option?

Rules

• Satisfy Hard gates 1–2 before Step 3; Hard gate 3 before Step 4; Hard gate 4 before the Step 5 options prompt
• Load skills BEFORE launching agents (Hard gate 2)
• All 5 agents run in parallel via Task tool
• Reference Task:Step for each issue
• Provide copyable suggested edits for Critical/Major issues
• Save review before prompting user (Hard gate 4)
• Never auto-execute plan; require user choice
• Number issues sequentially (1, 2, 3...)