Content
36%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.
The body is well-organized and lean, but it is a hollow shell: instructions are vague high-level bullets with no executable code, no validation checkpoints, and its only route to concrete detail is a reference to a file that is not present in the bundle. The body also duplicates the frontmatter description in its intro and use/don't-use sections.
Suggestions
Add at least one copy-paste-ready example Bats test to the Instructions (e.g., a @test block using run, "$status", and "$output" assertions for an exit-code and an output case) so the guidance is executable rather than descriptive.
Fix the broken reference: either include resources/implementation-playbook.md in the bundle or inline the essential patterns, since the body currently defers all detail to a file that does not exist.
Strengthen the workflow with concrete commands and a validation checkpoint (e.g., how to install/run Bats, 'run bats and confirm all tests pass; on failure, fix the test or script and re-run') instead of abstract steps like 'Set up a test structure with helpers and fixtures'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short, but the intro sentence duplicates the frontmatter description ("Comprehensive guidance for writing comprehensive unit tests ... including test patterns, fixtures, and best practices") and the 'Use this skill when' / 'Do not use this skill when' sections largely restate the description's triggers. This is mostly efficient with some clear redundancy that could be trimmed — anchor 3, not the tight anchor 4. | 3 / 5 |
Actionability | The instructions are high-level hints only — "Confirm shell dialects and supported environments", "Set up a test structure with helpers and fixtures", "Write tests for exit codes, output, and side effects" — with no code, commands, or even a single example Bats test (@test block, run/assert patterns). This matches anchor 2: minimal concrete guidance with the specific execution steps missing; it is not anchor 1 because the bullets do name concrete testing concerns rather than pure abstraction. | 2 / 5 |
Workflow Clarity | A rough sequence exists (confirm dialects → set up structure → write tests → add setup/teardown and run in CI) but the steps are poorly defined and validation is entirely absent — there is no 'run the tests and check they pass' step, no failure handling, and the final instruction defers to a missing file. Anchor 2 (rough sequence, many gaps, validation absent) fits better than anchor 3, whose example at least lists concrete, executable steps. | 2 / 5 |
Progressive Disclosure | The body is well-sectioned with a clearly signaled, one-level-deep reference ("If detailed examples are required, open resources/implementation-playbook.md"), but that file does not exist in the bundle — the sole pointer to detailed content is broken, so navigation dead-ends. Good structure with a broken reference lands at anchor 3 rather than 4, and the simple-skill exception for a 5 does not apply because the body explicitly defers its detail to a missing external file. | 3 / 5 |
Total | 10 / 20 Passed |