Content
100%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.
An exemplary skill body: dense with measured, non-obvious facts, fully executable commands and code, explicit validation and feedback loops in every workflow, and clean single-file organization with nothing warranting extraction. No content explains concepts Claude already knows beyond one-sentence rationale that guards against specific regressions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every token carries measured, non-obvious, repo-specific facts Claude cannot know — "costs roughly 4,700 additional modules (measured; re-measure with `--verbose`)", "takes `tsc --noEmit` from 12.6s to 8m07s — a 38x regression", "the registry contains one (`stt_deepgram_v2`)", "a few hundred tools are versioned". No padding, no generic concept explanations; rationale sentences exist only to prevent specific regressions (e.g., the .json-import cleanup trap). Fits the 5 anchor: lean, assumes competence, every token earns its place. | 5 / 5 |
Actionability | Fully executable, copy-paste-ready guidance throughout: exact commands (`bun run generate:tool-metadata`, `bun run check:tool-metadata`, `bun run check:tool-registry-boundary` with `--verbose`, `--check`, `--update-baseline`), exact module paths in the import table with sizes, and a concrete test-mocking snippet (`vi.mocked(getToolMetadata).mockImplementation(toolsMetadataMock.getToolMetadata)`). Matches the 5 anchor: specific examples cover the common cases. | 5 / 5 |
Workflow Clarity | Multi-step processes are clearly sequenced with explicit validation checkpoints and feedback loops: regeneration ends in "what CI runs; fails if stale"; guard failure has recovery guidance ("do not add the entry to an allowlist — there isn't one. Find the symbol the offending file actually needs and move it"); mock verification uses a sentinel check ("change a fixture value to a sentinel and confirm the test fails"); edge verification ends with "Compare the reachable module count before and after". Matches the 5 anchor: clear sequence, explicit validation, error-recovery loops. | 5 / 5 |
Progressive Disclosure | A single ~96-line file with no bundle files and well-organized sections (rule, import table, generated artifacts, testing, guard/baseline, verification, new-caller policy). All content is core operational guidance needed nearly every time the skill fires; nothing clearly belongs in a separate file, and splitting it would add indirection without token savings. Navigation via headers and the decision table is easy. Fits the 5 anchor's spirit (content appropriately placed) even though no external references are warranted. | 5 / 5 |
Total | 20 / 20 Passed |