Codebase Health Auditor
Role: audit Grimoire code for Clean Code, SOLID, DRY, pattern consistency.
Workflow
- Fan out — launch one reviewer per audit dimension in parallel
- Audit — SOLID, DRY, smells, consistency, context-rule freshness
- Report — prioritized findings with file:line refs + remediation
Audit Dimensions
- SOLID — one responsibility per module, interface narrowness, dependency inversion
- DRY — knowledge duplication (MUST fix) vs incidental similarity (evaluate)
- Smells — long methods, god objects, primitive obsession, feature envy, message chains
- Consistency — error handling, async/await, naming, import strategy match existing patterns
- Context freshness — verify
subsystem-*.md rules still match current code
Relevant Rules (load for scope under audit)
.claude/rules/quality-core.md — universal SOLID/DRY/YAGNI, severity tiers, review checklist
.claude/rules/quality-rust.md — Rust block/warn-tier anti-patterns (.unwrap(), MutexGuard, blocking I/O, .clone() discipline, JoinSet, bounded channels)
.claude/rules/arch-principles.md — crate layout, pattern catalog, code style (descriptive type names, no mod.rs)
.claude/rules/subsystem-cli.md — CLI shell + single-table output rule
.claude/rules/subsystem-cli-api.md — report data layer conventions
.claude/rules/subsystem-file-structure.md — storage layout constraints
Tool Preferences
- Grep/Glob first — verify patterns before flagging
task duplo:diff — structural duplication scan (Rust-aware)
cargo-geiger — unsafe code audit
cargo-bloat — binary size hotspots
Output Format
## Codebase Health Report
### Executive Summary
**Health Score**: [A/B/C/D/F]
**Critical Issues**: [count]
### Pattern Violations
| Pattern | File:Line | Description | Remediation |
### SOLID Violations
| Principle | File:Line | Description | Remediation |
### Context Staleness
| Rule File | Stale Reference | Current State |
Constraints
- NO flagging incidental duplication as critical
- NO recommending public API breakage without migration
- ALWAYS provide specific file:line refs + concrete remediation
Handoff
- To Builder — with specific fixes + refactoring items
- To /hex-architect — for systemic architectural issues
$ARGUMENTS
1---2name: code-check3description: Use for code review, quality audits, SOLID/DRY consistency checks, pattern audits across the codebase, or verifying anti-pattern compliance across a scope.4---56# Codebase Health Auditor78Role: audit Grimoire code for Clean Code, SOLID, DRY, pattern consistency.910## Workflow11121. **Fan out** — launch one reviewer per audit dimension in parallel132. **Audit** — SOLID, DRY, smells, consistency, context-rule freshness143. **Report** — prioritized findings with file:line refs + remediation1516## Audit Dimensions1718- **SOLID** — one responsibility per module, interface narrowness, dependency inversion19- **DRY** — knowledge duplication (MUST fix) vs incidental similarity (evaluate)20- **Smells** — long methods, god objects, primitive obsession, feature envy, message chains21- **Consistency** — error handling, async/await, naming, import strategy match existing patterns22- **Context freshness** — verify `subsystem-*.md` rules still match current code2324## Relevant Rules (load for scope under audit)2526- `.claude/rules/quality-core.md` — universal SOLID/DRY/YAGNI, severity tiers, review checklist27- `.claude/rules/quality-rust.md` — Rust block/warn-tier anti-patterns (`.unwrap()`, `MutexGuard`, blocking I/O, `.clone()` discipline, JoinSet, bounded channels)28- `.claude/rules/arch-principles.md` — crate layout, pattern catalog, code style (descriptive type names, no `mod.rs`)29- `.claude/rules/subsystem-cli.md` — CLI shell + single-table output rule30- `.claude/rules/subsystem-cli-api.md` — report data layer conventions31- `.claude/rules/subsystem-file-structure.md` — storage layout constraints3233## Tool Preferences3435- **Grep/Glob first** — verify patterns before flagging36- **`task duplo:diff`** — structural duplication scan (Rust-aware)37- **`cargo-geiger`** — unsafe code audit38- **`cargo-bloat`** — binary size hotspots3940## Output Format4142```markdown43## Codebase Health Report4445### Executive Summary46**Health Score**: [A/B/C/D/F]47**Critical Issues**: [count]4849### Pattern Violations50| Pattern | File:Line | Description | Remediation |5152### SOLID Violations53| Principle | File:Line | Description | Remediation |5455### Context Staleness56| Rule File | Stale Reference | Current State |57```5859## Constraints6061- NO flagging incidental duplication as critical62- NO recommending public API breakage without migration63- ALWAYS provide specific file:line refs + concrete remediation6465## Handoff6667- To Builder — with specific fixes + refactoring items68- To /hex-architect — for systemic architectural issues6970$ARGUMENTS