Content
71%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.
A tight, opinionated methodology skill with concrete directives, a clear loop sequence, and a genuine seam-confirmation checkpoint. Its two weaknesses are the missing red-phase validation step and two dead references to unbundled files (tests.md, mocking.md) that break progressive disclosure.
Suggestions
Bundle tests.md and mocking.md alongside SKILL.md (or remove the links) — both references in 'What a good test is' point to files that do not exist in the skill directory.
Add an explicit red-phase verification step, e.g. "Run the test and confirm it fails for the expected reason before writing any implementation", plus what to do if it unexpectedly passes.
Trim the opening meta-narrative ("This skill is the reference that... consult them before and during the loop, not after") to a single sentence, and cut the restatement of basic test doctrine in 'What a good test is'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and sectioned, with only minor trimmable material: the opening meta-narrative ("This skill is the reference that makes that loop produce tests worth keeping... consult them before and during the loop, not after") and restated known wisdom ("Tests verify behavior through public interfaces, not implementation details"). Not 5 because a few sentences explain concepts Claude already knows; not 2-3 because there is no padded section and most tokens carry non-obvious judgment (the anti-pattern "tells", the pre-agreed-seams rule). | 4 / 5 |
Actionability | Concrete, executable guidance throughout: "write down the seams under test and confirm them with the user", a literal question to ask ("What's the public interface, and which seams should we test?"), a worked tautology example ("expect(add(a, b)).toBe(a + b)"), and an explicit tool invocation ("call the Skill tool with 'codebase-design'"). Not 5 because there is no inline example of a good test or a full red-green cycle — those are deferred to files that are not in the bundle, leaving minor gaps. | 4 / 5 |
Workflow Clarity | The loop is clearly sequenced ("Red before green. Write the failing test first, then only enough code to pass it", "One slice at a time") with one explicit checkpoint ("No test is written at an unconfirmed seam"). Not 5 because the core TDD validation — run the test and confirm it fails for the expected reason before implementing — is implied rather than stated, and there is no recovery guidance for when the failing test unexpectedly passes; not 3 because sequence and checkpoints are mostly present with only minor gaps. | 4 / 5 |
Progressive Disclosure | Sections are well organized and the two external references are clearly signaled and one level deep ("See [tests.md](tests.md) for examples and [mocking.md](mocking.md) for mocking guidelines"), but neither tests.md nor mocking.md exists in the bundle (no references/ directory), so the navigation chain is broken. Not 4 because references must resolve to real files; not 2 because the inline content is appropriately structured rather than bulk that belongs elsewhere. | 3 / 5 |
Total | 15 / 20 Passed |