code-reviewlisted

# code-review — voidcorp craftsman edition A review without a framework is "the first ten things I noticed." This skill provides the framework: six dimensions in order, blocker vs nit, evidence block in the PR body, two modes, explicit composition with Claude Code's native `/code-review` and specialized agents. The skill orchestrates; it does not duplicate the deep-pass agents. **Attribution**: see `.source` in this directory. --- ## Six dimensions, in order The order matters. A correctness issue blocks regardless of beautiful structure. A perf issue does not block if the first four dimensions are clean. 1. **Correctness** — does it do what it claims? edge cases? error paths? composes with `tdd` (cycle evidence) and `systematic-debugging` (root cause for fixes). 2. **Tests** — failing test before the code? real code over mocks? names describe behavior? composes with `testing`. 3. **Security** — input validated at trust boundaries? secrets handled? SQL safe? LLM input untrusted? composes with `security-guidance` (which routes deep audits to `harness:security-audit`). 4. **Structure** — boundaries respected (no domain importing infrastructure)? service/repository split? function lengths? composes with `hexagonal-architecture`, `domain-driven-design`, `refactoring`, and the `doctrine-critic` agent. 5. **Readability** — names? exhaustive switches? `any` slips? `as` casts? composes with `typescript-strict`. 6. **Performance** — obvious O(n²) inside loops? leaky reactive subs