Content
56%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 well-organized, example-driven reference whose biggest liability is that the run commands are wrong for the tool: it is a Vitest skill that instructs `bun test`, which invokes Bun's test runner instead of Vitest. Secondary issues are an inline assertion API Claude already knows and no offloading of reference material to bundle files.
Suggestions
Replace the `bun test ...` commands in Running Tests with the actual Vitest CLI (`npx vitest run`, `npx vitest --watch`, `npx vitest --ui`, `npx vitest --coverage`, `--testNamePattern` for filtering) so copy-paste actually runs Vitest.
Cut or move the Assertions section — Claude already knows the Jest-compatible matcher API — keeping only Vitest-specific notes like `toHaveBeenCalledOnce()` or `expect.extend()`.
Move the detailed mocking, timer, and coverage material into a references/ file (e.g. references/mocking.md) and link to it from a short overview section, and verify the See Also skills exist or drop them.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean code examples with no padded prose, but the Assertions section re-lists the standard Jest-compatible matcher API ("toBeTruthy()", "toBeGreaterThan(3)", "toContain(item)") that Claude already knows, and basics like the beforeEach/afterEach scaffolding in "Basic Structure" add little. It is above anchor 2 (no heavily padded explanation sections) but not the lean anchor 5 because several blocks could be cut outright. | 3 / 5 |
Actionability | The writing, mocking, and configuration examples are concrete and executable, but the entire "Running Tests" section uses `bun test` (Bun's own test runner) with Bun's flags like `--grep`, not Vitest's CLI (`npx vitest run`, `vitest --watch`, `vitest --ui`) — copy-pasting those commands would not run Vitest at all. Per the guideline to evaluate only what is written, this is a real gap in executable correctness rather than a minor omission, placing it at anchor 3 rather than 4. | 3 / 5 |
Workflow Clarity | The sections form a logical progression — install, configure, run, write tests, mock, measure coverage, integrate, best practices — and each stage gives concrete commands. No validation checkpoints are required since testing is non-destructive, and it is above anchor 3 because the sequence is coherent and each step is immediately actionable, though it stops short of anchor 5's explicit feedback loops. | 4 / 5 |
Progressive Disclosure | There are no bundle files (references/, scripts/, assets/ are absent) and the See Also links point to sibling skills that do not exist in this bundle, so everything lives inline in one ~240-line file. Section headers are clear, but the assertion catalog and detailed mocking/timers material are the kind of API reference the anchor examples say belongs in a separate file, matching anchor 3 (content that should be separate is inline) rather than the well-split anchor 4/5 or the header-less anchor 2. | 3 / 5 |
Total | 13 / 20 Passed |