Content
38%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The TDD guidance itself is tight and opinionated with a sensible structure, but the skill is undermined by an off-topic telemetry block consuming roughly half the body and by references to example files that are absent from the bundle. A reader following the body's own pointers hits dead ends.
Suggestions
Remove or relocate the 'Skillpack activation reporting' block; it is unrelated to TDD and consumes the largest share of the body's token budget.
Fix the dangling references: either ship tests.md and mocking.md in the bundle or move their essential content inline, since no references/ directory exists.
Add an explicit validation checkpoint to the loop, e.g., 'Run the test and confirm it fails for the expected reason before writing any implementation; confirm it passes before starting the next slice.'
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly half the body is a 'Skillpack activation reporting' telemetry block (UUID generation rules, curl/PowerShell snippets, identity-discovery instructions) that is unrelated to TDD, and lines like "TDD is the red → green loop" explain concepts Claude already knows. The TDD-specific sections themselves are lean, which keeps this above anchor 1's 'severely verbose' but firmly at anchor 2's 'several unnecessary... padded sections'. | 2 / 5 |
Actionability | The loop rules are reasonably concrete ("One seam, one test, one minimal implementation per cycle", "Red before green") but the promised examples — "See [tests.md](tests.md) for examples and [mocking.md](mocking.md) for mocking guidelines" — point to files that do not exist in the bundle, and no worked example is provided inline. This is anchor 3's 'concrete guidance but incomplete; missing key details', not 4, because key guidance is unreachable. | 3 / 5 |
Workflow Clarity | The red → green sequence is present across the 'Rules of the loop' section, but validation checkpoints are implicit: it never instructs running the test to confirm it fails before implementing, or confirming it passes after. This matches anchor 3 ('sequence present but checkpoints missing or implicit'); the sequencing is too coherent for anchor 2 and lacks explicit validation steps for anchor 4. | 3 / 5 |
Progressive Disclosure | The in-file structure is well-sectioned, but the only two references (tests.md, mocking.md) are dangling — no references/ directory or bundle files exist, so following them fails entirely. Per the guideline to score against the actual bundle structure, clearly-signaled references to nonexistent files are worse than anchor 3's 'references present but not clearly signaled', fitting anchor 2's unusable-reference condition. | 2 / 5 |
Total | 10 / 20 Passed |