Context
During testing of the skill invocation cleanup (PR #107), the review phase noted that the test skill's "test the test" requirement — "verify the test fails without your fix" — is impractical in speedrun mode because the fix is already applied. Reverting it mid-workflow risks losing work and creating git confusion.
A better approach: write the regression test before implementing the fix, as part of the fix phase. This way the test naturally fails (proving it catches the bug), then the fix makes it pass.
Proposal
Move regression test creation from the test skill to the fix skill:
- Fix skill Step 2.5 (new): Write a regression test that reproduces the bug. Verify it fails. This proves the test actually catches the problem.
- Fix skill Step 3: Implement the fix. The regression test should now pass.
- Test skill: Focus on running the full suite, adding edge case coverage, and integration testing. Remove or soften the "test the test" requirement since it was already satisfied during fix.
This is essentially test-driven bug fixing — write the failing test first, then make it pass.
Benefits
- "Test the test" happens naturally instead of requiring an awkward revert
- Works in both controller and speedrun modes
- Aligns with TDD practices
- The regression test serves as a concrete specification of what "fixed" means before any code is written
Scope
workflows/bugfix/.claude/skills/fix/SKILL.md — add regression test step
workflows/bugfix/.claude/skills/test/SKILL.md — adjust "test the test" guidance
workflows/bugfix/.claude/skills/speedrun/SKILL.md — no changes needed (phase order unchanged)
Context
During testing of the skill invocation cleanup (PR #107), the review phase noted that the test skill's "test the test" requirement — "verify the test fails without your fix" — is impractical in speedrun mode because the fix is already applied. Reverting it mid-workflow risks losing work and creating git confusion.
A better approach: write the regression test before implementing the fix, as part of the fix phase. This way the test naturally fails (proving it catches the bug), then the fix makes it pass.
Proposal
Move regression test creation from the
testskill to thefixskill:This is essentially test-driven bug fixing — write the failing test first, then make it pass.
Benefits
Scope
workflows/bugfix/.claude/skills/fix/SKILL.md— add regression test stepworkflows/bugfix/.claude/skills/test/SKILL.md— adjust "test the test" guidanceworkflows/bugfix/.claude/skills/speedrun/SKILL.md— no changes needed (phase order unchanged)