Content
85%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.
The body is a highly actionable, well-sequenced operational guide with exact CLI commands, concrete code patterns, and strong validation/confirmation checkpoints for destructive operations. Its one material flaw is that its progressive-disclosure layer is broken in practice — all seven referenced detail files are missing from the bundle, leaving the detailed-reference links pointing nowhere.
Suggestions
Ship the referenced rule and reference files (rules/styling.md, rules/forms.md, rules/composition.md, rules/icons.md, rules/base-vs-radix.md, cli.md, customization.md) in the skill bundle — every detailed-reference link in the body currently points to a file that does not exist, so progressive disclosure cannot function.
Deduplicate the two composition.md-linked Critical Rules sections ("Component Structure" and "Use Components, Not Custom Markup") into one section, and trim Key Patterns entries that restate Critical Rules bullets verbatim (gap-* vs space-y-*, size-* vs w-*/h-*, icon sizing).
Either reference assets/shadcn.png (e.g. for visual identification of the current style) in the body or remove the unused asset from the bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with shadcn-specific rules Claude would not know (FieldGroup/InputGroup conventions, data-icon, preset subcommands), so nearly every token earns its place — but there is mild duplication: the Key Patterns code block restates Critical Rules bullets ("gap-*", "size-*", icon sizing appear in both), and two separate rule sections both link to composition.md. This fits "efficient; minor instances of over-explanation that could be trimmed" rather than the lean 5 anchor, and is well above the noticeably-verbose 2/3 anchors. | 4 / 5 |
Actionability | Guidance is fully executable throughout: exact CLI invocations with flags ("npx shadcn@latest add <component> --diff <file>", "apply <code> --only theme,font"), copy-paste TSX snippets for the common cases, a component-selection table, and a Quick Reference command block. Specific commands and examples cover the common workflows, matching the top anchor. | 5 / 5 |
Workflow Clarity | The 9-step Workflow is clearly sequenced with explicit validation checkpoints and feedback loops: check installed components before add, use --dry-run/--diff before updating, "always read the added files and verify they are correct" and "Fix all issues before moving on" (a validate-fix-retry loop), plus confirm-first rules for destructive operations ("Never use --overwrite without the user's explicit approval", asking before preset switches). This matches the anchor with explicit validation steps and error-recovery loops. | 5 / 5 |
Progressive Disclosure | The in-body structure is well designed — a Detailed References section with one-level-deep, clearly signaled links, each with a scope description — but none of the referenced files exist in the bundle: rules/styling.md, rules/forms.md, rules/composition.md, rules/icons.md, rules/base-vs-radix.md, cli.md, and customization.md are all absent (the bundle contains only two unreferenced PNGs in assets/). Since every reference target is broken, the disclosure structure does not actually function, which is a larger-than-minor organization gap (below the 4 anchor) though the SKILL.md itself is more structured than the 2 anchor's inlined monolith. | 3 / 5 |
Total | 17 / 20 Passed |