test-strategy-reviewer · diff

git:20260323.057cdcd to git:20260329.e81eb98

1 added, 0 removed. Audit C to C.

# test-strategy-reviewer
## Purpose During Development
Use this skill when Codex and similar AI coding agents need to assess whether a change has adequate verification, regression protection, and test scope.
This is a reusable development-time reasoning framework. It is not runtime code and must not be coupled into the product being reviewed.
## When To Apply
- any risky feature or refactor
- PR review where coverage is unclear
- bug fixes that need recurrence prevention
- contract, safety, or LLM behavior changes
## What To Inspect
- existing unit, integration, and end-to-end tests
- the diff’s risk profile
- code paths that can regress silently
- fixtures and golden data
- observed gaps between behavior and verification
## Rules To Enforce
- tests should reflect risk, not just code movement
- boundary behavior deserves explicit verification
- bug fixes should add recurrence protection when practical
- contract changes should verify both producer and consumer expectations
- performance and safety claims need dedicated validation, not implicit trust
## Common Anti-Patterns To Catch
- only testing happy paths for risky code
- no regression test for a claimed bug fix
- relying on manual verification for repeatable logic
- adding a contract without fixture or schema coverage
- treating unchanged tests as proof of low risk
## Expected Output From The Agent
- coverage gaps
- risk-based test recommendations
- minimal acceptable verification bar
- follow-up test debt notes
## Output Structure
- Summary
- Key Findings
- Risks
+ - Affected Files
- Coverage Gaps
- Suggested Improvements
- Suggested Tests
- Skills applied
## Example Usage
"Review this PR for missing regression coverage and verification blind spots."
"Use this skill to define the minimum test plan for a high-risk refactor."