code-quality
Report-only review of changed production code for structural maintainability after implementation, especially after reactive edits. Checks minimal change, single responsibility and misleading names, duplication, coupling, and module boundaries. Richard Ejem adds clarity/contracts, type modeling, reuse/evolution, and nit findings. Excludes lint/typechecker and test concerns. Supports base, path, revision-range, uncommitted, focus/skip, and instruction scopes.
Pinned to revision 371dd1e3f703, so it is the text this page describes rather than whatever the author pushed since.
Files
- skills/code-quality/SKILL.md
- skills/code-quality/README.md
- skills/code-quality/eval.json
- skills/code-quality/passes/coupling.md
- skills/code-quality/passes/duplication.md
- skills/code-quality/passes/minimal-change.md
- skills/code-quality/passes/single-responsibility.md
- skills/code-quality/references/arguments.md
- skills/code-quality/references/large-scope.md
- skills/code-quality/references/models.md
- skills/code-quality/references/report-format.md
- skills/code-quality/references/review-rules.md
- skills/code-quality/references/richard-ejem/clarity-and-contracts.md
- skills/code-quality/references/richard-ejem/reuse-and-evolution.md
- skills/code-quality/references/richard-ejem/type-modeling.md
- skills/code-quality/references/verify-findings.md
Every link opens the file at its source, pinned to the revision this page describes.