review

# Code Review Reviews against the full Darkroom quality checklist defined in the reviewer agent. Focus areas: TypeScript strictness, React patterns, accessibility, performance, security, file structure. ## Current State - Branch: !`git branch --show-current 2>/dev/null || echo "unknown"` - Staged files: !`git diff --staged --stat 2>/dev/null || echo "nothing staged"` - Unstaged files: !`git diff --stat 2>/dev/null || echo "nothing unstaged"` ## Get Changes ```bash # Unstaged changes git diff # Staged changes git diff --staged # Specific file git diff path/to/file ``` ## Large diffs: walk a reading diff, not a summary Past ~200 changed lines, present the diff walkthrough as a **reading diff** — the real diff, abridged — never a prose-only summary (prose can lie by omission; a diff can't). Every line shown is verbatim from `git diff`; the only editing allowed is removal and compression: drop whole no-signal lines, fold 2+ contiguous same-polarity lines into a `...` row, or elide a noisy span inside a kept line. Never rewrite one. Drop import churn, lockfiles, generated files, and formatting-only hunks; show one instance of a mechanical rename and fold the rest with a count; keep every behavioral change. Close with an accounting line — `showing N of M changed lines — dropped: ...` — naming any new dependency, changed import target, or lockfile version/integrity bump even though import churn is dropped. The full diff remains the merge authority. Full protocol: the "Readi...