Content
63%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-structured reference-style body with genuinely executable commands and concrete conventions, weakened by duplicated trigger sections and ambiguous file-reference signaling. Consolidating the two 'when to use' sections and giving referenced files full, consistent paths would make it notably tighter and easier to navigate.
Suggestions
Merge 'Activation Conditions' and 'When to Use This Skill' into one section — they list nearly identical triggers and cost unnecessary tokens.
Give referenced files consistent, full paths (e.g., 'rules/test-mock-patterns.md') and replace the vague 'See rules/ directory' with the actual file list or fold it into the Quick Reference table.
Add a minimal inline example of a unit test and a mock so the core task is executable without leaving SKILL.md, plus a command for running a single test file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly tables and commands with no concept re-teaching, but 'Activation Conditions' (5 bullets) and 'When to Use This Skill' (3 bullets) duplicate each other, and the opening line 'Write reliable tests with Bun units and Rstest integrations' merely restates the frontmatter. This matches 'mostly efficient but could be tightened' rather than the minor-trim level 4. | 3 / 5 |
Actionability | Concrete, executable guidance throughout: the commands block ('bun run test:unit', 'bun run test:coverage', etc.), explicit conventions ('.test.ts = Bun unit, .test.tsx / .integration.test.ts = Rstest integration'), and the specific path 'test-utils/test-utils.tsx'. Not 5 because there is no inline example of actually writing or mocking a test — that is deferred entirely to referenced files not present in this bundle. | 4 / 5 |
Workflow Clarity | A clear derivable sequence exists (pick test type via the Quick Reference table, apply conventions, run the matching command) and no destructive/batch cap applies. Not 5 because 'Debugging test failures' is listed as an activation condition yet no debugging feedback loop is described, and no command for running a single test file is given. | 4 / 5 |
Progressive Disclosure | References are only one level deep and gathered in a navigation table, but the five referenced files ('test-unit-vs-integration.md', 'test-mock-patterns.md', etc.) are named without paths while the Rules section vaguely says 'See rules/ directory for detailed guidance' — inconsistent signaling, and no bundle directories exist here to verify the files. This matches 'references present but not clearly signaled' rather than the mostly-clear level 4. | 3 / 5 |
Total | 14 / 20 Passed |