Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The content is highly actionable with executable templates, specific commands, and clear debugging workflows with validation checkpoints. Its main weakness is progressive disclosure — everything lives in one ~200-line file with no bundle files to offload detail — and minor verbosity from duplicated guidance.
Suggestions
Split the inline unit/integration/MCP test templates into reference files (e.g. references/unit-template.js, references/integration-template.js) and link to them from SKILL.md to improve progressive disclosure.
De-duplicate the nock/jest.clearAllMocks() guidance, which appears in both the template blocks and the "Key conventions" bullets, to tighten conciseness.
Move the lengthy "Is It the Test or the Code?" decision tables into a reference file, keeping only the core decision rules inline in SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely efficient and avoids teaching Claude basic Jest/testing concepts, but it runs ~200 lines with substantial inline templates and tables that could be tightened; the duplicated nock/clearAllMocks guidance and lengthy template blocks push it toward the "mostly efficient but could be tightened" anchor. | 2 / 3 |
Actionability | Provides fully executable, copy-paste-ready unit and integration test templates, specific commands (`npm test --workspace=@app-connect/core`, `npx jest ... --verbose --no-coverage`), and concrete helper signatures like createMockUser(overrides) and the in-memory SQLite pattern. | 3 / 3 |
Workflow Clarity | "Examining Failing Tests" and "Is It the Test or the Code?" give a clear numbered sequence with validation checkpoints (run isolated, read failure message, --verbose, check env) plus explicit feedback loops and decision rules for error recovery. | 3 / 3 |
Progressive Disclosure | The body is a single monolithic file with all detail inline and no bundle references to offload material; the closing "Additional Reference" section lists real repo files but these are external source paths rather than skill-bundle references, so structure is present but not split into one-level-deep bundle files. | 2 / 3 |
Total | 10 / 12 Passed |