Content
22%Reviews 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 with no actionable content of its own. It defers all concrete guidance to a referenced playbook that isn't provided in the bundle, leaving the SKILL.md body without any executable examples, Bats syntax, or meaningful workflow steps. The content fails to serve as a useful quick-start reference and relies entirely on external files for substance.
Suggestions
Add a concrete quick-start example showing a minimal Bats test file with @test blocks, setup/teardown, and a run command so the skill is immediately actionable without opening referenced files.
Replace the abstract instruction bullets with specific, executable steps (e.g., 'mkdir -p test/fixtures && cat > test/example.bats <<EOF ... EOF && bats test/').
Add a validation checkpoint such as 'Run `bats --tap test/` and verify all tests pass before committing' to establish a feedback loop in the workflow.
Ensure referenced paths are consistent—the skill references both `resources/implementation-playbook.md` and `./sub-skills/implementation-playbook.md` which appear to be different paths for the same content.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 'Use this skill when' and 'Do not use this skill when' sections add moderate bloat—Claude can infer applicability from context. The instructions section is brief but the overall content has unnecessary framing without adding much actionable value. | 2 / 3 |
Actionability | The instructions are entirely abstract ('Confirm shell dialects', 'Set up a test structure', 'Write tests for exit codes'). There are zero concrete code examples, no Bats syntax, no executable commands, and no sample test files. Everything actionable is deferred to a referenced file. | 1 / 3 |
Workflow Clarity | The five bullet points in Instructions loosely suggest a sequence but lack specificity, validation checkpoints, or any feedback loops. There is no clear multi-step workflow with concrete commands or verification steps. | 1 / 3 |
Progressive Disclosure | The skill references `resources/implementation-playbook.md` and a sub-skill, which is appropriate progressive disclosure structure. However, no bundle files were provided to verify these references exist, the SKILL.md itself contains almost no substantive quick-start content, and the references are somewhat redundant (the same playbook is mentioned twice under different paths). | 2 / 3 |
Total | 6 / 12 Passed |