plan-eng-review

Reviews implementation plans for architecture, code quality, test coverage, and performance issues.

Updated Apr 5, 2026
One-click install
npx skills add https://github.com/Simon-YHKim/eject-button --skill plan-eng-review-simon-yhkim
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: plan-eng-review
Source: https://github.com/Simon-YHKim/eject-button/tree/main/.claude/skills/plan-eng-review
Command: npx skills add https://github.com/Simon-YHKim/eject-button --skill plan-eng-review-simon-yhkim

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve? Plans and design docs often hide architecture flaws, missing edge cases, and untested codepaths that only surface after implementation begins. This Skill performs an engineering-manager-style review of a plan before any code is written, catching scope creep, over-engineering, and coverage gaps early. ## Core Features & Use Cases - Scope Challenge (Step 0): Challenges whether existing code already solves sub-problems, flags plans touching 8+ files or 2+ new services, and verifies chosen patterns against current best practices via web search. - Four-Section Interactive Review: Walks through Architecture, Code Quality, Tests, and Performance one section at a time, presenting each issue individually with an opinionated recommendation and confidence score. - Test Coverage Diagrams: Traces every planned codepath and user flow into an ASCII coverage diagram, marking branches that need unit tests, E2E tests, or evals, with a mandatory regression-test rule. - Use Case: Before implementing a new feature, run the review on your plan doc to get findings like "[P1] (confidence: 9/10) auth.ts:47 — token check returns undefined when session expires" and decide each fix interactively. ## Quick Start Ask the assistant to run an engineering review of your current plan or design doc before you start coding.

Frequently Asked Questions about plan-eng-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review an architecture plan before writing code?

Run an engineering plan review that challenges scope first, then walks through architecture, code quality, tests, and performance section by section. Each issue is presented individually with a recommendation and confidence score so you decide the direction before implementation starts.

What does a plan engineering review check for?

It checks whether existing code already solves sub-problems, flags plans touching more than 8 files or 2 new services, verifies patterns against current best practices, traces every codepath for test coverage, and evaluates failure scenarios for new integration points.

How do I know if my plan has enough test coverage?

The review traces every planned function, conditional branch, error path, and user flow into an ASCII coverage diagram. Each branch is marked as tested, happy-path-only, or untested, and E2E-worthy flows and regression tests are flagged explicitly.

When should I run a plan review versus a code review?

Run a plan review after writing a design doc or plan but before any implementation, when changing direction is cheap. Code review happens after the diff exists; plan review catches architecture and scope problems that code review cannot fix economically.

Can the review work without a design doc?

Yes, it proceeds with a standard review if no design doc is found, but it first offers to run a prerequisite office-hours step that produces a structured problem statement and explored alternatives, giving the review sharper input.