Dotclaude qual
Use when you want a deeper multi-angle quality review before shipping, especially after substantial or risky changes.
git clone https://github.com/JHostalek/dotclaude
T=$(mktemp -d) && git clone --depth=1 https://github.com/JHostalek/dotclaude "$T" && mkdir -p ~/.claude/skills && cp -r "$T/skills/qual" ~/.claude/skills/jhostalek-dotclaude-qual && rm -rf "$T"
skills/qual/SKILL.mdtarget_path = $ARGUMENTS
If target_path provided, analyze that path. Otherwise, files changed since the default branch. Full-codebase analysis requires explicit user request.
Architecture
You coordinate specialist teammates who analyze in independent context windows. Separation matters — your implementation bias doesn't leak into their findings, and their diverse perspectives catch what a single pass misses.
Two waves: detect then simplify. Wave 2 exists because LLMs reliably over-engineer fixes — even your own wave 1 fixes are suspect and need an independent simplification pass.
Wave 1: Detect
Spawn specialist teammates from
${CLAUDE_SKILL_DIR}/agents/| Teammate | Agent file | Focus |
|---|---|---|
| Skeptic | | Bugs, security, performance, correctness |
| Silent-failure hunter | | Hidden errors, swallowed exceptions |
| Pattern harmonizer | | Divergent implementations, inconsistent patterns |
| Comment auditor | | Misleading/stale/useless comments |
Wave 1 teammates are read-only — analysis tools only, no editing. Mixing analysis and editing in one pass leads to fixes that miss the bigger picture.
The Simplifier runs in wave 2, not here.
Actionability Gate
Every finding must pass all five criteria. This filter prevents the report from being dominated by noise — LLM analysis without it produces 60%+ false positives:
- Scoped — within the target path or recent changes
- Specific — points to a concrete code path, not a general observation
- Certain — "this WILL cause X", not "this could theoretically..."
- Non-intentional — not a deliberate pattern justified by comments, tests, or conventions
- Discrete — has a single, focused fix (not "rewrite this module")
Deduplicate across lenses: when multiple teammates flag the same underlying issue, merge into one finding citing all relevant lenses. Keep the highest severity.
Triage
| Severity | Boundary |
|---|---|
| CRITICAL | Security vulnerabilities, data loss risks, silent failures hiding bugs |
| HIGH | Logic errors, missing error handling on critical paths |
| MEDIUM | Pattern inconsistencies, genuine maintainability issues |
| LOW | Minor improvements |
| IGNORE | False positives, justified patterns, style-only concerns |
Conflict resolution: Safety > style. Cohesion > fragmentation. Direct code > abstraction. Pattern consistency > local optimization. Type annotations > JSDoc.
Approval Gate
Present findings grouped by severity. Each finding: severity, confidence, location, issue, recommended fix, source lens(es).
Stop. Wait for user to select which fixes to apply.
After approval: implement selected fixes, verify with quality gates. If a fix breaks something, revert rather than cascade.
Wave 2: Simplify
Spawn one Simplifier teammate (
simplifier.md