Content
42%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 lean, well-sectioned skeleton whose substance is entirely deferred to a playbook file that is missing from the bundle. What remains is abstract guidance with no executable Bats example, no commands, and no validation checkpoints, leaving the skill unable to act on its own instructions.
Suggestions
Fix the bundle: either add resources/implementation-playbook.md or remove the dangling reference and inline the essential patterns — as written, every 'detailed example' is unreachable.
Add at least one complete, copy-paste Bats test to Instructions (e.g., an @test block using `run`, `$status`, and `assert_output`) plus the commands to install and run the suite.
Make the workflow steps concrete with checkpoints: name the setup/teardown hooks (setup(), teardown() with temp dirs) and a verification step such as running the suite locally before wiring CI.
Tighten duplication: drop the intro sentence and the 'Use this skill when' list, both of which restate the frontmatter description.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short and assumes Claude knows what Bats is (no concept explainer), but the intro sentence "Comprehensive guidance for writing comprehensive unit tests..." duplicates the description nearly verbatim, the "Use this skill when" section repeats the description's triggers, and the Limitations section is generic boilerplate. Not 4 because there are several trimmable padded sections, not 2 because nothing over-explains known concepts. | 3 / 5 |
Actionability | Every instruction is an abstract directive — "Set up a test structure with helpers and fixtures", "Write tests for exit codes, output, and side effects" — with no concrete code, commands, or a single example Bats test (no @test block, no run/exit assertions, no `bats` invocation). Not 1 because the bullets do direct concrete activities rather than only describing; not 3 because even pseudocode-level specifics of a Bats test are absent, and the only detailed material is deferred to a file that does not exist in the bundle. | 2 / 5 |
Workflow Clarity | The Instructions list a recognizable sequence — "Confirm shell dialects" → "Set up a test structure" → "Write tests..." → "Add setup/teardown and run tests in CI" — but each step is a one-line abstraction with no validation checkpoints (e.g., how to verify the suite runs or that CI wiring works). Not 4 because checkpoints are entirely missing, not 2 because a rough, sensible sequence is present and the operations are not destructive. | 3 / 5 |
Progressive Disclosure | The one reference — "open `resources/implementation-playbook.md`" — is one level deep and clearly signaled in both Instructions and a Resources section, but no bundle files exist at all: there is no resources/ directory, so the reference dangles and every 'detailed example' it promises is unreachable. Not 2 because the structure and signaling are good; not 4/5 because navigation fails at the first hop and the body itself contains no compensating detail. | 3 / 5 |
Total | 11 / 20 Passed |