arda/skills
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #393 from mattpocock/tdd-reference-only-seams

Browse source 
tdd: reshape into reference-only with pre-agreed seams
This commit is contained in:
Matt Pocock 2026-07-01 12:51:40 +01:00 • committed by GitHub
commit 21f59763be
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 23 additions and 101 deletions
Download patch file Download diff file Expand all files Collapse all files

7
.changeset/tdd-reference-only-seams.md Normal file
View file

@ -0,0 +1,7 @@
---
"mattpocock-skills": patch
---
Reshape the `tdd` skill into reference-only. The red → green → refactor loop is anchored by leading words the model already holds, so the step-by-step Workflow was largely restating the loop and duplicating the horizontal-slicing anti-pattern. Dropped the Workflow and per-cycle checklist; folded their one durable idea — vertical slices / tracer bullets — into the Anti-patterns section and a short Rules-of-the-loop list. Introduced **seam** as the leading word for where tests go, collapsing the old Philosophy "public interfaces" prose and the Planning "confirm interface / behaviors" handshake into one rule: test only at pre-agreed seams, confirmed with the user before any test is written.
Also dropped the refactor stage — TDD is now red → green, not red → green → refactor. Refactoring belongs to the review stage, not the implementation loop, so the refactor rule and `refactoring.md` were removed (its home is the `review` skill).