code-reviewlisted

# Code Review Skill Systematic code review following a fixed sequence: scope the change, assess blast radius, scan for security issues, verify tests, detect breaking changes, check performance, and score quality. ## Review Workflow 1. **Scope** -- Identify what changed (files, modules, services). Classify as feature, bugfix, refactor, config, or docs. 2. **Blast radius** -- Trace dependencies to determine how far the change reaches. See [blast-radius.md](blast-radius.md). 3. **Security scan** -- Run pattern-based checks for common vulnerabilities. See [security-scan.md](security-scan.md). 4. **Test coverage** -- Verify new/changed code has tests. Flag untested branches and edge cases. 5. **Breaking changes** -- Detect API, schema, config, and dependency changes that break consumers. See [breaking-changes.md](breaking-changes.md). 6. **Performance** -- Flag N+1 queries, unbounded loops, missing pagination, large allocations, blocking I/O on hot paths. 7. **Quality** -- Score SOLID adherence, complexity, and code smells. See [quality-checks.md](quality-checks.md). Run the full [checklist](checklist.md) to ensure nothing is missed. ## Output Format Organize findings into four categories with severity: | Category | Severity | Meaning | |--------------|--------------|----------------------------------------------| | MUST FIX | CRITICAL/HIGH| Bugs, security holes, data loss risks | | SHOULD FIX | HIGH/MEDIUM | Desig