dev-review

1# Dev Review
2 
3Run final pre-push review for configured AI docs features. Before changing docs or code, propose the concrete review plan and wait for user approval unless the user already approved the exact phase plan.
4 
5## Phase Contract
6 
71. Run `npx ai-devkit@latest lint` before phase work.
82. If working on a named feature, run `npx ai-devkit@latest lint --feature <name>`.
93. Check `git status -sb` and `git diff --stat`.
104. Read feature docs and relevant changed files before findings.
115. Apply the `verify` skill before claiming readiness. Run `dev-smoke` when the change touches user-facing flows.
126. If parent `dev-lifecycle` established usable task tracing, emit review phase, progress, blocker/finding, next-step, and final evidence/readiness events per `task`.
13 
14## Code Review
15 
16Use for Phase 9. Take a holistic review stance: findings first, ordered by severity, grounded in file/line references.
17 
181. Gather context: feature description, modified files, design docs, risky areas, tests already run.
192. Verify design alignment by summarizing architectural intent and checking implementation matches.
203. For each modified file, grep exported names to trace callers and dependents. Read relevant signatures, call sites, and type definitions.
214. Check consistency against 1-2 similar modules.
225. Search for existing utilities the new code could reuse or now duplicates. Flag near-matches honestly; do not force a wrong abstraction.
23…