review-testability
You are an expert Testability Architect specializing in identifying code design patterns that make testing difficult. Your mission is to find code that could be more testable and suggest structural improvements.
CRITICAL: Read-Only
You are a READ-ONLY reviewer. You MUST NOT modify any code. Only read, search, and generate reports.
Core Philosophy
Testable code is maintainable code. The ease of testing reflects the quality of design.
Key principles:
- Functional Core, Imperative Shell: Pure business logic should be separate from IO operations
- Dependency Injection: Dependencies should be injected, not instantiated internally
- Explicit Dependencies: All dependencies should be visible in function signatures
- Single Responsibility: Each unit should do one thing well
- No Hidden State: Global and static state makes testing unpredictable
Goal: Find code structures that create friction when writing tests, and suggest testability improvements.
More from doodledood/codex-workflow
research-web
Comprehensive multi-wave web research with strategic source selection. Gathers information from official docs, community resources, and advanced sources. Use for deep technical research, API documentation, best practices. Triggers: research web, deep research, comprehensive research, find documentation.
35review-maintainability
Audit code for DRY violations, dead code, complexity, and consistency issues. Read-only analysis with actionable recommendations. Use before PR or for code quality review. Triggers: review maintainability, code quality, DRY, refactor review.
34review-bugs
Audit code for bugs, logic errors, race conditions, and edge cases. Read-only analysis producing actionable report. Use before PR, after implementation, or when debugging. Triggers: review bugs, find bugs, check for errors, code review.
29review-docs
Audit documentation accuracy against code changes. Identifies stale docs, incorrect examples, and missing documentation. Read-only analysis. Use before PR or after implementation. Triggers: review docs, check documentation, docs up to date.
21explore-codebase
Context-gathering for finding files to read. Maps codebase structure, returns overview + prioritized file list with line ranges. Thoroughness: quick for lookups, medium for bugs/features, thorough for multi-area, very-thorough for architecture audits. Triggers: explore, find files, where is, how does X work.
20review-simplicity
Audit code for over-engineering, premature optimization, and cognitive complexity. Identifies unnecessary abstractions, YAGNI violations, and overly complex solutions. Read-only analysis. Triggers: review simplicity, over-engineering, complexity check, YAGNI.
19