Content
92%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.
An exemplarily lean process skill: the red-green-refactor loop is clearly sequenced with explicit validation checkpoints, a feedback loop, and a completion checklist, while test-writing detail is correctly pushed to one well-signaled reference file. The only gap is the absence of a concrete example test command in the body itself.
Suggestions
In step 2, name one concrete way to determine the test command (e.g., 'use the project's test script or README, e.g. npm test / pytest') so the step is executable in unfamiliar projects.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~30-line body never explains concepts Claude already knows; every line instructs ("Write one test for one behavior... use the real code, not mocks"; "Add nothing the test does not need"). Lean and efficient with every token earning its place; not 4 because there is no over-explanation left to trim. | 5 / 5 |
Actionability | Steps are specific and directive ("The test must fail because the behavior is missing, not because of a typo or a missing import"), and test-writing detail is concretely delegated to the reference file. However, "Run the project's test command" gives no concrete command or way to find one — mostly executable guidance with a minor gap; matches 4, not 5 (nothing is copy-paste ready). | 4 / 5 |
Workflow Clarity | The five-step Red/Green/Refactor sequence has explicit validation checkpoints ("Watch it fail", "Watch it pass", "Run the suite again"), a feedback loop for error recovery ("If it passes, it tests nothing new: change the test"), and a final checklist ("Before you report the work as done"). Matches the 5 anchor exactly; not 4 because checkpoints and recovery are explicit rather than implicit. | 5 / 5 |
Progressive Disclosure | The body is a concise, well-sectioned overview with a single clearly signaled, one-level-deep reference — "When writing or changing a test, read references/writing-good-tests.md" — and the bundle confirms that file exists (198 lines) with no nested references inside it. Not 4: navigation is unambiguous and nothing that belongs in the reference is inlined. | 5 / 5 |
Total | 19 / 20 Passed |