plan-eng-review

Run this Skill on your plan and it performs a structured review across architecture, code quality, tests, and performance. Each issue is presented individually with lettered options and an opinionated recommendation, so decisions are made before implementation starts.

The review covers four sections: architecture (component boundaries, data flow, security), code quality (DRY violations, error handling, over-engineering), tests (coverage diagram for new codepaths), and performance (N+1 queries, caching, memory). It also produces scope and failure-mode analyses.

Yes, the Skill offers a SMALL CHANGE mode that compresses the review into one combined pass, picking the single most important issue per section. It still requires the scope challenge step and a test diagram, but batches questions into one round.

Yes, the test review builds a diagram of all new UX, data flows, codepaths, and branching logic, then verifies each item has a corresponding test. Failure modes without tests and error handling are flagged as critical gaps.

Deferred work is captured in a "NOT in scope" section with rationale, and each potential TODO is proposed individually with what, why, pros, cons, context, and dependencies. You choose per item whether to add it to TODOS.md, skip it, or build it now.