Content
72%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 efficient, well-structured overview that establishes a clear evaluation philosophy and defers detail appropriately to four real reference files. The main weakness is workflow clarity: the inspection procedure is described in implicit prose rather than explicit, checkpointed steps.
Suggestions
Turn the 'Inspecting' paragraph into a short numbered sequence with explicit checkpoints (e.g., 1. Read the test name and state the behavior it claims; 2. Check the assertions pin it; 3. Ask what change would make it fail; 4. Check error paths and boundaries).
Include one inline weak-vs-strong assertion example in the body so the core heuristic is usable without opening references/assertion-quality.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and each section contributes skill-specific framing ('A test suite is only worth the failures it would catch', the two failure modes, consequence-oriented reporting). Minor over-explanation: the causes of flakiness (timing guesses, shared state, real clocks) restate what Claude already knows. | 4 / 5 |
Actionability | For an instruction-only skill, the guidance is concrete and executable: start from the test's name, check 'whether removing the feature would make it fail', check error paths, and the weak-vs-strong report phrasing example ('an empty cart throws in checkout and nothing would catch it'). Not a 5 because no inline example of a weak vs. strong assertion is given — that detail is deferred entirely to references. | 4 / 5 |
Workflow Clarity | The 'Inspecting' section does give a rough sequence (name → do assertions pin the behavior → would removal fail → error paths and boundaries), but it is prose rather than checkpointed steps, and the actual staged procedure (the 2-minute pass and deep pass) lives in references/inspection-checklist.md rather than the body. Sequence present but checkpoints implicit. | 3 / 5 |
Progressive Disclosure | The body is a concise overview with a dedicated Reference section pointing to four real, existing files in references/ (inspection-checklist.md, red-flags.md, assertion-quality.md, example-report.md), each clearly signaled with a one-line description. References are one level deep and easy to navigate, matching the top anchor. | 5 / 5 |
Total | 16 / 20 Passed |