Content
60%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 well-structured, opinionated process skill with a clear workflow and genuinely useful anti-pattern guidance (horizontal vs vertical slices). Two real weaknesses: the Philosophy section re-explains testing basics Claude already knows, and every external reference in the body is a broken link to files that don't exist in the bundle.
Suggestions
Create the referenced files (tests.md with good/bad test examples, mocking.md with the mocking guidelines, refactoring.md with refactor candidates) or remove the links and inline the essential content — as written, all three are dead references.
Cut the Philosophy section to the project-specific opinion (integration-style tests through public interfaces, no implementation coupling) and drop the general explanation of why implementation-coupled tests break under refactor.
Add error-recovery guidance to the workflow: what to do when a new test passes immediately (it isn't testing anything new) or when minimal code doesn't reach GREEN.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The Philosophy section spends roughly three paragraphs explaining good-vs-bad testing concepts Claude already knows (implementation coupling, tests surviving refactors, mocking internals) before reaching the skill-specific opinion. The rest of the body is lean and directive. Matches anchor 3 ('mostly efficient but includes some unnecessary explanation') rather than 4 because the padding is confined to a whole section, and rather than 2 because only that one section over-explains. | 3 / 5 |
Actionability | Concrete, instruction-level guidance throughout: planning checklists with user-approval gates, an exact question to ask ('What should the public interface look like?'), explicit RED→GREEN cycle rules, a per-cycle checklist, and a concrete pointer to run the /codebase-design skill. Minor gaps keep it below anchor 5 — no test-framework or command specifics and the RED→GREEN blocks are schematic rather than executable. Anchor 4 ('mostly executable guidance... minor gaps') fits; this is acceptable for an instruction-only skill per the scoring notes. | 4 / 5 |
Workflow Clarity | Clear four-phase sequence (Planning → Tracer Bullet → Incremental Loop → Refactor) with explicit checkpoints: each test must fail then pass, 'Never refactor while RED', and 'Run tests after each refactor step', plus checklists. Not anchor 5 because there is no error-recovery/feedback-loop guidance (e.g. what to do when the new test never goes RED, or when minimal code doesn't reach GREEN); stronger than anchor 3 because validation states are explicit rather than implicit. | 4 / 5 |
Progressive Disclosure | The body links to [tests.md](tests.md), [mocking.md](mocking.md), and [refactoring.md](refactoring.md), but none of these files exist in the bundle — there is no references/, scripts/, or assets/ directory. Scored against the actual bundle structure per the judging guidelines, the claimed one-level-deep split is undelivered and navigation dead-ends. Anchor 2 fits (structure present in intent but effectively non-functional); anchor 3 would require working but poorly signaled references. | 2 / 5 |
Total | 13 / 20 Passed |