code-quality

<objective> Code Quality is the canonical 7-phase validation workflow shared with the `sniper` agent (this skill and `agents/sniper.md` must stay in sync): explore architecture, research documentation, analyze impact via Grep usages, detect duplication with jscpd (DRY), detect errors via linters, apply precision corrections informed by docs/impact/DRY findings, then re-verify until linters, tests, and duplication are all clean. It covers SOLID compliance, DRY thresholds, error priority (security/logic/performance/style), file-size limits, and architecture-pattern violations across all languages. It validates quality AFTER the fact -- it does not confirm the original problem was actually solved. Run the `verification` skill first to confirm functional resolution, then code-quality to confirm the fix is clean. </objective> # Code Quality Skill Canonical workflow definition: `agents/sniper.md` (this skill and the sniper agent share the same 7-phase workflow — update both together). ## 🚨 MANDATORY 7-PHASE WORKFLOW ``` PHASE 1: Exploration (explore-codebase) → BLOCKER PHASE 2: Documentation (research-expert) → BLOCKER PHASE 3: Impact Analysis (Grep usages) → BLOCKER PHASE 3.5: DRY Detection (jscpd duplication) → NON-BLOCKING PHASE 4: Error Detection (linters) PHASE 5: Precision Correction (with docs + impact + DRY) PHASE 6: Verification (re-run linters, tests, duplication) ``` **CRITICAL**: Phases 1-3 are BLOCKERS. Never skip them. **DRY**: Phase 3.5 is non-blocking but fin...