skills/use-context-reviewer-readability/SKILL.md
Code readability review. Do NOT use for security (use-context-reviewer-security), type errors (use-context-reviewer-strictness), error handling (use-context-reviewer-silence), or test design (use-context-reviewer-testability).
npx skillsauth add thkt/dotclaude use-context-reviewer-readabilityInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
Thresholds are based on cognitive limits (working memory, one-screen focus) and established metrics (McCabe complexity).
| Target | Recommended | Why | | --------------------- | ----------- | -------------------------------------------- | | Function lines | ≤30 | One-screen readability | | File lines | ≤400 | Module-level cognitive ceiling | | Nesting depth | ≤3 | Branch tracking within working memory | | Function arguments | ≤3 | Argument order memorization limit | | Cyclomatic complexity | ≤10 | McCabe 1976: testable without path explosion |
| ID | Pattern | Fix |
| --- | ----------------------------- | -------------------------------- |
| RD1 | processData() (vague) | validateUserEmail() |
| RD1 | Misleading identifiers | Names reveal intent |
| RD2 | Nesting > 3 levels | Guard clauses, extract functions |
| RD2 | Function > 30 lines | Decompose |
| RD3 | Comment: // increment i | Delete (self-evident) |
| RD3 | Comment: // TODO: fix later | Create issue or fix now |
| RD4 | Interface for single impl | Remove until 2nd impl |
| RD4 | Class for stateless logic | Pure function |
| RD5 | > 5 function parameters | Config object or decompose |
| Topic | File | | --------------- | -------------------------------------------------- | | Control Flow | ${CLAUDE_SKILL_DIR}/references/control-flow.md | | Comments | ${CLAUDE_SKILL_DIR}/references/comments-clarity.md | | AI Antipatterns | ${CLAUDE_SKILL_DIR}/references/ai-antipatterns.md | | Naming | ${CLAUDE_SKILL_DIR}/references/naming.md |
documentation
Generates and updates .claude/OUTCOME.md interactively. When the file is absent or empty (no Behavior / all sections TBD), collects content via AskUserQuestion and writes the stub; when present, shows the current state and applies updates.
development
Judge a SKILL.md against craft axes (single responsibility, description distinctiveness, imperative voice, verifiable completion, calibration, progressive disclosure) and apply the fixes the audit surfaces. Do not use for format-presence-only checks (use reviewer-prompt) or reproducibility loops (use /tuning).
tools
Internal helper for /think Step 11. Renders SOW.md + Spec.md as an integrated Astro view and returns a dev server URL.
development
Extract repository spec while detecting bugs, spec gaps, and consistency drift via dual-purpose documentation. OUTCOME.md-axis question-driven exploration with ephemeral output. Do NOT use for code review (use /audit or /polish), feature implementation (use /code), planning only (use /think), or single-bug fix (use /fix).