Content
71%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 extremely lean, well-sectioned body whose one instruction is concrete, but it under-delivers on the skill's stated purpose: the review itself has no actionable guidance, failure handling is undefined, and the bundle's reference file is orphaned while the script is referenced via a path that doesn't exist in the bundle. Conciseness is exemplary; actionability and navigation are the weak points.
Suggestions
Reference the script with its bundle-relative path (scripts/check.sh) instead of '.cursor/skills/acr__example__all-agents__review-change/scripts/check.sh', so the command is runnable as bundled.
Add a failure branch, e.g. 'If it exits non-zero, stop and report the failure output — do not proceed to a verdict,' so the non-zero-exit case has defined handling.
Link references/REFERENCE.md from the body (e.g. 'Use deterministic evidence — see references/REFERENCE.md') so the bundled guidance is discoverable, and briefly state what the review should examine and report so the body matches the description's promise.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is nine lines with zero padding: 'Process steps in order. Do not skip ahead.' and the output contract ('It emits {"ok":true} on stdout and exits non-zero on failure') each earn their place, and no concepts Claude already knows are re-explained. This matches the 'lean and efficient; every token earns its place' anchor exactly. | 5 / 5 |
Actionability | The single command is concrete and executable, but key details are missing: nothing says what to do when check.sh exits non-zero, and the body gives no actionable guidance for the review work itself (what to examine in the change or its evidence, what to report), even though the description promises 'Review a proposed code change and its evidence'. This is concrete-but-incomplete, fitting the score-3 anchor rather than the minor-gaps anchor at 4. | 3 / 5 |
Workflow Clarity | As a simple single-action skill the step is unambiguous — a specific script, its success output, and 'Finish here.' terminate the flow — which supports the simple-skill exception. It falls short of 5 because the failure branch is defined ('exits non-zero on failure') yet no recovery or reporting step follows it, a minor checkpoint gap matching the score-4 anchor. | 4 / 5 |
Progressive Disclosure | Scored against the actual bundle (references/REFERENCE.md, scripts/check.sh): the body references the script via '.cursor/skills/acr__example__all-agents__review-change/scripts/check.sh', a path that does not match the bundle-relative 'scripts/check.sh', and references/REFERENCE.md is never mentioned from the body at all, leaving its guidance undiscoverable. Sections are well organized, but references are present yet not correctly signaled — the score-3 anchor, not 4 (the mis-pathed script and orphaned reference are more than minor organization gaps). | 3 / 5 |
Total | 15 / 20 Passed |