What problem does it solve?
This Skill eliminates the risk of merging feature branches with hidden code quality issues like duplicated logic, untestable business logic, low-value tests, and inconsistent patterns that accumulate technical debt and increase long-term maintenance costs.
Core Features & Use Cases
- DRY Violation Detection: Scans all changed files for duplicated logic, constants, and patterns, checking existing utilities in src/common/ and src/db/ before recommending new shared code.
- Testability Improvements: Identifies business logic embedded in UI components or entry points, recommending extraction into testable service or helper functions with optional database parameters for in-memory test injection.
- Test Quality Audits: Flags tests that only assert trivial outcomes, miss meaningful edge cases, or duplicate existing coverage, ensuring every test provides clear value.
- Complexity & Pattern Consistency Checks: Highlights overly nested conditionals, n+1 query patterns, and deviations from established repository conventions including the repository pattern, Zod validation, and co-located test directories.
- Use Case: Ideal for developers preparing to merge a feature branch, who want to catch quality issues early without manually reviewing every changed file.
Quick Start
Invoke the polish skill to review all changed files in your current branch and receive prioritized, actionable feedback on code quality issues before merging.