Content
80%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 tight, well-structured conventions skill: every line is a repo-specific rule, sections are cleanly organized, and the only external pointer is a single one-level reference. Its weaknesses are the absence of any executable code example illustrating the required test layout and an implicit rather than explicit red/green verification loop.
Suggestions
Add a minimal copy-paste-ready bun:test example (root describe with declarations inside, an it, and a beforeEach) to make the 'Test files' layout rules concrete and lift actionability to fully executable.
Make the red/green checkpoint explicit in the BDD loop, e.g. 'Write the scenario, run `bun test`, confirm it fails (red), implement until `bun test` passes (green), then refactor' — this adds the missing validation/feedback loop.
Clarify the 'See also: `repo-conventions`' pointer with a relative path or one-phrase scope note so navigation to the related skill is unambiguous.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~35 lines of dense, rule-style bullets with zero padding — it never explains what BDD/TDD are beyond a one-line framing ('Same red → green → refactor rhythm as TDD, oriented around user-facing outcomes') and every token carries a project-specific rule Claude could not infer. This matches the 'lean and efficient; assumes Claude's competence' anchor; a 4 would require noticeable over-explanation that is absent. | 5 / 5 |
Actionability | Guidance is concrete and executable in the instruction-only sense: exact APIs ('`describe` / `it` from `bun:test` (not `test`)', 'describe.each / it.each', 'beforeAll / beforeEach'), a concrete path convention ('test/support/ modules that export functions'), and a measurable policy ('a few happy paths plus one or two errors'). It falls short of the top anchor because there is no copy-paste-ready example (e.g., a minimal describe/it block showing the required root-describe layout), which is the one gap for an instruction-only skill. | 4 / 5 |
Workflow Clarity | The core sequence is named ('concrete examples of desired behaviour first, then implement until they pass... refactor is required after green') but the checkpoints are implicit — it never states how to verify green (e.g., run `bun test` and see failures before/after), matching 'sequence present but checkpoints missing or implicit'. Not a 4 because no explicit verification step or feedback loop (fail → fix → re-run) is spelled out; not a 2 because the rhythm and its ordering are clearly stated. | 3 / 5 |
Progressive Disclosure | The skill is under 50 lines with well-organized sections (Test files, Testing library code) and no content that warrants external files; the single 'See also: `repo-conventions`' pointer is one level deep and clearly signaled. Per the rubric's simple-skill guidance this matches the top anchor. | 5 / 5 |
Total | 17 / 20 Passed |