Content
35%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 content is essentially a table of contents with no actionable substance. It lacks any concrete code examples, specific Bats syntax, or executable commands, deferring everything to an external playbook. The instructions read as high-level bullet points that describe what to do without showing how, making the skill nearly useless without the referenced resource file.
Suggestions
Add at least one concrete, executable Bats test example (e.g., a minimal .bats file with @test, setup, teardown) directly in the SKILL.md so it provides immediate value without requiring the external resource.
Replace vague instruction bullets like 'Write tests for exit codes, output, and side effects' with specific Bats commands and assertions (e.g., `run my_script.sh; [ "$status" -eq 0 ]`).
Remove or significantly trim the 'Use this skill when' / 'Do not use this skill when' sections—these consume tokens without adding actionable guidance.
Add a quick-start workflow with explicit steps including how to install Bats, run a test file, and interpret results, with validation checkpoints.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 'Use this skill when' and 'Do not use this skill when' sections are somewhat unnecessary padding—Claude doesn't need to be told when not to use a skill in this level of detail. The introductory paragraph also restates the title. However, the instructions section itself is reasonably brief. | 2 / 3 |
Actionability | The instructions are entirely vague and abstract ('Write tests for exit codes, output, and side effects', 'Set up a test structure with helpers and fixtures') with zero concrete code, commands, or examples. Everything actionable is deferred to an external resource file. | 1 / 3 |
Workflow Clarity | There is a rough sequence implied (confirm environments → set up structure → write tests → add setup/teardown → run in CI), but steps lack specificity, have no validation checkpoints, and no feedback loops for error recovery. | 2 / 3 |
Progressive Disclosure | There is a reference to a detailed resource file (resources/implementation-playbook.md), which is good progressive disclosure. However, the SKILL.md itself provides almost no useful quick-start content—it's essentially an empty shell pointing elsewhere, making the overview layer too thin to be useful on its own. | 2 / 3 |
Total | 7 / 12 Passed |