Content
57%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 thorough, code-rich TDD skill with a clear stepped workflow and useful runner/matrix guidance; its main weaknesses are verbosity in best-practice/mistake sections that restate known concepts, a missing validate-fix-retry feedback loop in the workflow, and no progressive disclosure of the large mock/pattern reference material into separate files.
Suggestions
Move the Supabase/Redis/OpenAI mock recipes, full E2E specs, and coverage-threshold config into reference files (e.g. references/mocking.md, references/e2e-patterns.md) and link to them from SKILL.md.
Add an explicit feedback loop after Step 3 and Step 5: on failure, read the error, fix the smallest cause, re-run `<test>`, and only proceed once GREEN — then run `<coverage>` as the final gate.
Cut the 'Best Practices', 'Common Testing Mistakes', and 'Success Metrics' sections to a few non-obvious items, removing guidance Claude already knows (one-assert-per-test, arrange-act-assert, test edge cases).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely useful and code-forward, but it pads sections Claude already knows (a 10-item 'Best Practices' list, 'Common Testing Mistakes', generic 'Success Metrics', boilerplate one-assert-per-test advice) that restate familiar testing concepts rather than adding net-new guidance. | 3 / 5 |
Actionability | It provides mostly executable, copy-paste-ready patterns for unit/integration/E2E/Bun tests plus a concrete runner-detection matrix and mock snippets; minor gaps include placeholder test bodies ('// Test implementation') and the unresolved `<coverage>`/`<test>` tokens. | 4 / 5 |
Workflow Clarity | The 7-step TDD sequence is clearly laid out with an explicit RED gate in Step 3 ('Tests should fail') and coverage verification in Step 7, but there is no validate→fix→retry feedback loop after a failing run and no explicit checkpoint before declaring GREEN, which the feedback-loops note expects for batch/test operations. | 3 / 5 |
Progressive Disclosure | Structure is reasonable with sectioned headers, but the entire skill is a single ~467-line SKILL.md with no bundle files; large reference-style blocks (mock recipes, full E2E specs, coverage config) that would suit separate reference files are inlined, and there are no one-level-deep reference links to offload detail. | 3 / 5 |
Total | 13 / 20 Passed |