Content
22%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill is essentially a thin table of contents that defers all substantive content to a referenced file that doesn't exist in the bundle. It lacks any concrete code examples, executable Bats test patterns, or specific commands, making it nearly useless as standalone guidance. The content reads more like a project brief than an actionable skill.
Suggestions
Add at least one complete, executable Bats test example (e.g., a simple @test block with setup/teardown) directly in the SKILL.md so Claude has immediate actionable guidance.
Replace the abstract instruction bullets with concrete steps including actual commands (e.g., `bats test/`, `setup()` and `teardown()` function templates, `run` and `assert_output` usage).
Include a minimal quick-start workflow with validation: create test file → write test → run `bats test.bats` → check output → fix failures.
Either provide the referenced `resources/implementation-playbook.md` bundle file or inline the most critical patterns from it into the SKILL.md body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 'Use this skill when' and 'Do not use this skill when' sections add moderate bloat—Claude doesn't need to be told when shell scripts aren't involved. The limitations section is boilerplate. However, the instructions section itself is reasonably brief. | 2 / 3 |
Actionability | There are no concrete code examples, no executable Bats test snippets, no specific commands, and no copy-paste ready content. The instructions are entirely abstract ('Write tests for exit codes, output, and side effects') with no demonstration of how to actually do any of it. | 1 / 3 |
Workflow Clarity | The five bullet points in Instructions are vague high-level steps with no validation checkpoints, no specific commands, and no feedback loops. For a testing skill that involves multi-step setup and execution, this is insufficient. | 1 / 3 |
Progressive Disclosure | The skill references `resources/implementation-playbook.md` for detailed patterns, which is a reasonable one-level-deep reference. However, no bundle files were provided, so the reference cannot be verified, and the SKILL.md itself contains almost no substantive content—it defers nearly everything to the external file without providing a useful quick-start overview. | 2 / 3 |
Total | 6 / 12 Passed |