What problem does it solve?
This Skill provides an engineering-manager level review to lock architecture, data flow, diagrams, edge cases, and test coverage before implementation. It prevents costly rework by surfacing architectural mistakes, missing tests, performance pitfalls, and scope creep early in the cycle.
Core Features & Use Cases
- Interactive plan walkthroughs: Step-by-step review of design docs, branch context, and proposed implementation with concrete recommendations.
- Scope and complexity checks: Step 0 evaluates minimal changes, files touched, and suggests scope reduction when designs are overbuilt.
- Diagrams, tests, and rollout guidance: Generates ASCII diagrams, test coverage suggestions, performance checks, and deployment/rollback recommendations.
- Repo ownership and process checks: Honors repo mode, searches for existing design docs, and writes a Plan Status Footer to the plan file.
- Use Case: Before shipping a payments feature, run this Skill to validate API boundaries, database migrations, failure modes, test plans, and rollout strategy so implementation is safe to start.
Quick Start
Run /plan-eng-review on your current branch and provide a link or summary of the design doc to get an architecture audit, diagrams, and a prioritized test checklist.