Plan Engineering Review

Review the plan thoroughly before code changes. Do not implement. Preserve the same behavior as the original skill by delegating each concern to smaller local skills and then synthesizing the result.

Skill Chain

Run these local skills in order:

1. /plan-review-intake - find the plan, read .planning/* context, detect branch/base, and gather existing TODOs.
2. /interrogate-me - stress-test assumptions, dependencies, architecture, APIs, tests, deployment, and failure modes one question at a time.
3. /plan-review-scope - challenge scope, complexity, existing reuse, and "complete vs shortcut" tradeoffs.
4. /plan-review-architecture - review boundaries, data flow, dependency graph, security boundaries, rollout, and failure scenarios.
5. /plan-review-code-quality - review organization, DRY risks, error handling, diagrams, maintainability, and local conventions.
6. /plan-review-tests - produce the test diagram, identify coverage gaps, and write the QA handoff artifact.
7. /plan-review-performance - review N+1 risk, caching, memory, latency, and scaling.
8. /plan-review-wrapup - write the review artifact, TODO proposals, unresolved decisions, and completion summary.