Content
78%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 SKILL.md body is an efficient, well-structured pointer that delegates detail to a single clearly-signaled reference file. Its only weakness is that the body itself carries no executable test-authoring guidance or validation steps, relying entirely on the reference.
Suggestions
Add a one-line inline summary of the core workflow (e.g. the analyze→plan→implement→verify loop) so the body is actionable without immediately opening the reference.
Surface the key execution command (e.g. `npm run test-unit -- <path>`) inline so the most common action is executable from SKILL.md alone.
Mention the critical import constraint (import `src/Neo.mjs` + `src/core/_export.mjs`) briefly in the body, since it is the most common failure mode noted in the reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The two-sentence body is lean and assumes Claude's competence; every token earns its place and nothing over-explains concepts Claude already knows. | 5 / 5 |
Actionability | The body gives a concrete pointer ('read and follow `.agents/skills/unit-test/references/unit-test.md`') but the executable test-authoring detail is delegated to the reference, leaving the body itself incomplete as standalone guidance. | 3 / 5 |
Workflow Clarity | A clear single action with a well-defined trigger and an explicit boundary clause ('Review-only placement/idiom audits stay under `pr-review`'), though validation checkpoints live only in the referenced file rather than the body. | 4 / 5 |
Progressive Disclosure | The body is a clean overview pointing to one real, one-level-deep bundle file (references/unit-test.md) via an exact, clearly signaled path; content is appropriately split and easy to navigate. | 5 / 5 |
Total | 17 / 20 Passed |