Content
77%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.
The body is an unusually actionable TDD guide: concrete Good/Bad code, exact commands, worked examples, and a verification loop with explicit error-recovery guidance. Its weaknesses are redundancy — the rationalization rebuttals appear three times in different forms — and a dangling reference to a testing-anti-patterns.md file that is not in the bundle, which undermines the progressive-disclosure structure of an otherwise well-sectioned document.
Suggestions
Fix the broken reference: either add testing-anti-patterns.md to a references/ directory or remove the 'Testing Anti-Patterns' section's link, since the file does not exist in the bundle.
Consolidate the triple-repeated rationalization content ('Why Order Matters', 'Common Rationalizations', 'Red Flags') into one section — the same excuses are rebutted three times, costing tokens without adding guidance.
Consider moving the rationalization table and anti-patterns detail into the referenced file so SKILL.md stays a lean overview of the cycle, per the red flags list already being a summary.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The TDD cycle, code examples, and checklist are tight and actionable, but the same anti-rationalization content is repeated three times ('Why Order Matters' prose, the 'Common Rationalizations' table covering the identical excuses — 'I'll test after', 'deleting X hours is wasteful', 'tests after achieve the same goals', 'already manually tested' — and the 'Red Flags' list repeating them again), plus a token-heavy graphviz diagram that a four-item list would convey. Mostly efficient with clear tightening opportunities, so not anchor 2's 'several padded sections' nor anchor 4's 'minor instances'. | 3 / 5 |
Actionability | Copy-paste-ready TypeScript examples with explicit Good/Bad pairs for both the test and the implementation, exact commands ('npm test path/to/test.test.ts'), a fully worked bug-fix example showing expected FAIL/PASS output, a verification checklist, and a troubleshooting table. This covers the common cases the way the anchor-5 example does. | 5 / 5 |
Workflow Clarity | The red-green-refactor sequence is explicit with mandatory verification checkpoints at each phase, failure-mode feedback loops ('Test passes? You're testing existing behavior. Fix test.', 'Test errors? Fix error, re-run until it fails correctly', 'Test fails? Fix code, not test.'), a pre-completion checklist, and 'Confirm' criteria distinguishing expected failure from typos — matching the anchor-5 pattern of clear sequence plus validation plus error-recovery loops. | 5 / 5 |
Progressive Disclosure | The body is a single ~370-line monolith with clear section headers but no working bundle: the one reference — 'read [testing-anti-patterns.md](testing-anti-patterns.md)' — points to a file that does not exist (no references/ directory or sibling file is present), and large reusable content (the rationalization tables, anti-patterns) is inlined rather than split out. That sits at anchor 3 ('some structure, references present but problematic, content that should be separate is inline') — above anchor 2 because sections are well-organized and the reference is clearly signaled, below 4 because the sole reference is broken. | 3 / 5 |
Total | 16 / 20 Passed |