meta-reviewing-cli-reviewing
CLI Code Review Patterns
Quick Guide: When reviewing CLI code, verify SIGINT handling, p.isCancel() checks, exit code constants, parseAsync() usage, and user feedback (spinners, clear errors). Check config hierarchy, help text quality, and dry-run support. Distinguish severity (Must Fix vs Should Fix vs Nice to Have) and explain WHY each issue matters.
<critical_requirements>
CRITICAL: Before Reviewing CLI Code
All code must follow project conventions in CLAUDE.md (kebab-case, named exports, import ordering,
import type, named constants)
(You MUST verify SIGINT (Ctrl+C) handling exists in CLI entry point)
(You MUST verify p.isCancel() is called after EVERY @clack/prompts call)
(You MUST verify exit codes use named constants - flag ANY magic numbers in process.exit())
(You MUST verify parseAsync() is used for async actions, not parse())