Content
71%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 highly actionable, well-structured doctrine skill with executable commands, exact file paths, and good/bad examples, plus a real workflow and verification checklist. Its weaknesses are length and repetition (~725 lines where tighter editing could cut a third) and a complete absence of progressive disclosure — lane templates and examples are prime candidates for reference files that would shrink the always-loaded body.
Suggestions
Move the eight 'Lane Templates' sections into a references/ file (e.g. references/lane-templates.md) and keep a one-line lane map in SKILL.md pointing to it — this alone would cut the body by roughly a third.
Deduplicate repeated rules (placeholder comments, 'ComponentPreview' demo checks, banned marketing adjectives) by consolidating them into the single 'Anti-Slop Rules' or 'Code Example Rules' section and removing the duplicates from 'Voice' and 'Demos and Previews'.
Add explicit error-recovery feedback loops to the Workflow or Verification Checklist — e.g. 'if build:source fails, fix MDX syntax and re-run before proceeding' — so validation acts as a checkpoint sequence rather than a passive checklist.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense rule content rather than padded prose, but at ~725 lines it includes notable redundancy: placeholder-comment bans appear in both 'Voice' and 'Code Example Rules', 'ComponentPreview' rules appear in both 'Demos and Previews' and 'Anti-Slop Rules', banned openings repeat across sections, and the goal-template section repeats three near-identical bash blocks. Not a 2 because there is little explanation of concepts Claude already knows — nearly every line is project-specific doctrine — but it could be tightened considerably. | 3 / 5 |
Actionability | Fully executable guidance throughout: copy-paste commands ('node .agents/skills/autogoal/scripts/create-goal-scratchpad.mjs --template docs ...', 'pnpm --filter www build:source', 'pnpm --filter www check:docs'), exact file paths ('content/docs/meta.json', 'apps/www/src/registry/registry-kits.ts', 'registry-kits.ts'), explicit MDX component usage rules, and concrete bad/good writing examples. This covers the common cases for a style/workflow skill. | 5 / 5 |
Workflow Clarity | An explicit 11-step 'Workflow' sequence plus a thorough 'Verification Checklist' with concrete validation commands (MDX parse check, docs parity check, route check). Not a 5 because error-recovery feedback loops are not spelled out — the checklist says what to verify but not what to do when a check fails, and validation lives in a separate section from the workflow steps rather than being interleaved. | 4 / 5 |
Progressive Disclosure | The skill is a single monolithic file with no bundle files (no references/, scripts/, or assets/) and no one-level-deep pointers — everything, including eight detailed lane templates and the shadcn corpus list, is inlined in SKILL.md. It is well-structured with clear headings, which keeps it above a 2 (navigation is possible), but content that clearly belongs in separate reference files (lane templates, writing voice examples, verification checklist) is inlined. | 3 / 5 |
Total | 15 / 20 Passed |