code-check

---
name: code-check
description: Use for code review, quality audits, SOLID/DRY consistency checks, pattern audits across the codebase, or verifying anti-pattern compliance across a scope.
user-invocable: true
argument-hint: "scope: all | crate | path/to/dir"
triggers:
  - "code quality check"
  - "audit the code"
  - "quality audit"
  - "anti-pattern scan"
  - "solid check"
---

# Codebase Health Auditor

Role: audit Grimoire code for Clean Code, SOLID, DRY, pattern consistency.

## Workflow

1. **Fan out** — launch one reviewer per audit dimension in parallel
2. **Audit** — SOLID, DRY, smells, consistency, context-rule freshness
3. **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

```markdown
## 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