Content
70%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 strongly actionable, well-sequenced TDD skill whose Red-Green-Refactor workflow with per-phase verification and recovery loops is exemplary. Its weaknesses are repetition of the same anti-rationalization message across three sections, a non-rendering graphviz diagram, and a broken link to a writing-good-tests.md file that is not in the bundle.
Suggestions
Create the referenced writing-good-tests.md (or remove the link) — the four bullets that follow the link are a natural seed for that file, which would also slim the body.
Merge the Red Flags list with the Common Rationalizations table, since nearly every red flag duplicates a row of the table, and drop or shrink the graphviz diagram to a one-line cycle description.
Replace fragmentary code examples (the `// ...` in submitForm, the `// YAGNI` stub) with complete, runnable versions to reach copy-paste-ready coverage.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly tight and imperative, but the same enforcement message is repeated across three sections ("Delete it. Start over" in The Iron Law, "Keep as reference, write tests first... Delete means delete" in Common Rationalizations, and "Keep as reference" or "adapt existing code" in Red Flags), and the ~20-line graphviz `dot` diagram renders as raw code in markdown, adding tokens without adding instruction. Not level 4 because the duplication and the decorative diagram are trimming candidates beyond 'minor'; not level 2 because every section still carries operational content rather than padding. | 3 / 5 |
Actionability | Concrete, executable guidance dominates: real commands (`npm test path/to/test.test.ts`), complete TypeScript test/implementation examples with Good/Bad contrasts, a worked bug-fix walkthrough, a verification checklist, and a troubleshooting table. Not level 5 because some examples are fragments — the GREEN example for retry is fully executable, but `submitForm` ends in `// ...` and the Bad GREEN example is a stub with `// YAGNI` — and the link to writing-good-tests.md leads nowhere. | 4 / 5 |
Workflow Clarity | The Red-Green-Refactor cycle is explicitly sequenced with a mandatory verification command after each phase ("Verify RED - Watch It Fail... MANDATORY. Never skip", "Verify GREEN... MANDATORY"), includes feedback loops for error recovery ("Test errors? Fix error, re-run until it fails correctly", "Test fails? Fix code, not test"), and closes with a per-item verification checklist plus a "When Stuck" recovery table. This matches the level-5 anchor: clear sequence, explicit validation steps, and error-recovery loops. | 5 / 5 |
Progressive Disclosure | The single external reference is well-signaled inline ("read [writing-good-tests.md](writing-good-tests.md) for the rules that keep tests honest") and one level deep, but the file does not exist — no references/ directory or writing-good-tests.md is present anywhere in the skill bundle, so the link is broken. The body is also ~315 lines with content (the Good Tests rules and the Common Rationalizations table) that reads like material meant for that missing file. Not level 4 because a dangling reference defeats navigation; not level 2 because section structure is clear and there is no monolithic wall of text. | 3 / 5 |
Total | 15 / 20 Passed |