thkt/reviewing-readability
skills/reviewing-readability/SKILL.md
Code readability review. Use when: 可読性, 明確, 命名, 変数名, 関数名, ネスト, 関数設計, コメント, 複雑, Miller's Law, ミラーの法則, 認知負荷, AI-generated, 過剰設計. Do NOT use for security (reviewing-security), type errors (reviewing-type-safety), error handling (reviewing-silent-failures), or test design (reviewing-testability).
$ install --global
skillsauthnpx skillsauth add thkt/claude-config reviewing-readabilityInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
Security Scan Results
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
3
Scanners Passed
9
Scanners in report
Clean
TrivyContainer and dependency vulnerability scanner
95%
Clean
SemgrepStatic code analysis for vulnerabilities
95%
Clean
mcp-scan (Snyk)Model Context Protocol security validation
95%
Skipped
Snyk (dep)Open source security scanning
50%
Skipped
Socket.devSupply chain security analysis
50%
Skipped
VirusTotalMulti-engine malware detection
50%
Skipped
CrowdStrikeAdvanced threat intelligence
50%
Skipped
OSV-ScannerOpen Source Vulnerability database check
50%
Skipped
OWASP Dep-Check
50%
Last scanned: Apr 22, 2026, 7:18 AM127.4s5 files scanned
SKILL.md
- name:
- reviewing-readability
- description:
- >
- Code readability review. Use when:
- 可読性, 明確, 命名, 変数名, 関数名,
- allowed-tools:
- [Read, Grep, Glob, Task]
- agent:
- code-quality-reviewer
- context:
- fork
- user-invocable:
- false
Readability Review
Thresholds: see rules/development/THRESHOLDS.md.
Detection
| 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 |
References
| 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 |
Related Skills
thkt/use-workflow-plan-preview
tools
VerifiedTrustedCommunity
Internal helper for /think Step 11. Renders SOW.md + Spec.md as an integrated Astro view and returns a dev server URL.
9SKILL.mdUpdated Jun 1, 2026
thkt/probe
development
VerifiedTrustedCommunity
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).
9SKILL.mdUpdated Jun 1, 2026
thkt/audit-adr-gaps
development
VerifiedTrustedCommunity
Discover undocumented design decisions and challenge each candidate via critic-design before promotion. Rank by impact and reversibility, produce ADR promotion candidates. Treat each candidate as a position arguing for ADR status, not a fact to be filed. Pairs with audit-adr-drift, which scans existing ADRs for drift against code.
9SKILL.mdUpdated Jun 1, 2026
thkt/audit-adr-drift
development
VerifiedTrustedCommunity
Scan ADR Decision sections against current code and report drift with modification direction and priority. Do NOT use for repos without ADRs (use audit-adr-gaps instead).
9SKILL.mdUpdated Jun 1, 2026