thkt/reviewing-testability
skills/reviewing-testability/SKILL.md
Testable code design review. Use when: テスト容易性, モック, mock-friendly, DI. Do NOT use for type safety (reviewing-type-safety), security (reviewing-security), or readability (reviewing-readability).
$ install --global
skillsauthnpx skillsauth add thkt/claude-config reviewing-testabilityInstall 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 20, 2026, 9:28 AM7.2s4 files scanned
SKILL.md
- name:
- reviewing-testability
- description:
- >
- Testable code design review. Use when:
- テスト容易性, モック, mock-friendly,
- allowed-tools:
- [Read, Grep, Glob, Task]
- agent:
- testability-reviewer
- context:
- fork
- user-invocable:
- false
Testability Review
Detection
| ID | Pattern | Fix |
| --- | ---------------------------- | ------------------------------- |
| TE1 | Direct import { db } usage | Inject dependency as parameter |
| TE1 | new Service() inside class | Constructor injection |
| TE2 | fetch() inside component | Extract to hook/service, inject |
| TE2 | Mixed side effects + logic | Separate pure/impure |
| TE3 | Deep mock chains | Simplify dependencies |
| TE4 | Global config access | Pass config as prop/parameter |
| TE4 | Date.now() in logic | Inject clock/time provider |
| TE5 | Tight coupling | Depend on abstractions (DIP) |
Criteria
Test setup < 10 lines. No deep mock chains. Dependencies explicit.
References
| Topic | File |
| ------- | -------------------------------------------------------- |
| DI | ${CLAUDE_SKILL_DIR}/references/dependency-injection.md |
| Pure | ${CLAUDE_SKILL_DIR}/references/pure-functions.md |
| Mocking | ${CLAUDE_SKILL_DIR}/references/mock-friendly.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