← ClaudeAtlas

namht-plan-reviewlisted

Critique an implementation plan or user-story plan BEFORE building, through multiple lenses — Product (right thing? simpler? scope creep), Architecture/Eng (fits existing patterns, blast radius, risk), Risk/QA (what breaks, regression, edge cases), DevEx (complexity, maintainability) — grounded in the KB, ending with a verdict (proceed / revise) + concrete fixes. Use when the user says "/plan-review", "review this plan", "is this plan good", or after /namht-plan.
NamHT4Devlop/nam-claude-skill · ★ 0 · Code & Development · score 72
Install: claude install-skill NamHT4Devlop/nam-claude-skill
# namht-plan-review — multi-lens critique of a plan (catch bad plans early) A plan is the cheapest place to fix a mistake. Review the plan (from `/namht-plan`, `/namht-discover`, or pasted) before code. ## Input The plan / user stories (pasted, or a file under `namht-sessions/`). Identify the feature + the modules/entities it targets. Ground in KB (`13-business-rules`, `16-architecture-patterns`, `10-core-flows`) and by tracing callers (blast radius of the proposed changes). ## Lenses (score each: ✅ ok / ⚠️ concern / ❌ blocker) 1. **Product** — is this the *right* thing? Is there a simpler version that delivers most value? Scope creep? Does it match the success metric? Anything that should be cut or deferred? 2. **Architecture / Eng** — does it follow the documented pattern & layer rules (`16-architecture-patterns`)? Blast radius (who else is affected — grep callers)? Migrations / breaking changes handled? Any risky shortcut? 3. **Risk / QA** — what existing flows could break (regression)? Edge cases, error paths, concurrency, permissions, data states the plan ignores? Is it testable? 4. **DevEx / Maintainability** — complexity added, duplication, naming, "will the next dev understand this?", premature abstraction (YAGNI). 5. **Reuse & Rollout** (when the plan came from `/namht-build`) — check its **Reuse Report**: for every row marked `new`, does the justification name real searches, and can you find an existing candidate it missed (helper, service, D