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.
The body is well-organized with a clear mindset, explicit when/when-not sections, and strong anti-pattern guidance with WHY/BAD/GOOD structure. Its main weaknesses are operational: the workflow depends on files and commands (bun run generate/validate, scratchpad template, schema) that are missing from the bundle, which makes the later steps non-executable as written, and there is notable redundancy between the Mindset and Anti-Patterns sections.
Suggestions
Ship the referenced `templates/skill-output.yaml` and `schemas/design-reference.schema.json` (or remove the references) so that workflow steps 4, 5, and 8 resolve to real files.
Back `bun run generate` and `bun run validate` with actual scripts in the bundle, or replace them with concrete inline commands Claude can execute directly.
Consolidate the Mindset bullets with the overlapping anti-patterns (the invent/expand-tokens rule is stated three times), and add references/elements-of-design.md and references/principles-of-design.md to the References table so every bundle file is discoverable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient — tight sections, concrete commands, no explanation of concepts Claude already knows — but it could be tightened: the "never invent/expand tokens" theme is stated three times (Mindset bullet, anti-pattern 1, anti-pattern 2), and several BAD examples are just comments ("# assuming Tailwind defaults for breakpoints"). This fits the "mostly efficient but includes some unnecessary explanation or could be tightened" anchor rather than 4, where the redundancy would be only a minor instance. | 3 / 5 |
Actionability | The discovery grep commands are executable, but steps 4, 6, and 8 depend on tooling that does not exist: `bun run generate --template enforcement-skill` and `bun run validate --schema schemas/design-reference.schema.json` have no backing scripts (no scripts/, no package.json), and the referenced `templates/skill-output.yaml` and `schemas/design-reference.schema.json` are absent from the bundle. This is concrete-but-incomplete guidance with missing key details — the 3 anchor — rather than the 4 anchor's "minor gaps". | 3 / 5 |
Workflow Clarity | The seven-step summary is clearly sequenced with validation checkpoints: "Review populated scratchpad before generating final docs", the TBD/null grep gate before generating markdown, schema validation, and a final structured confirmation report. It does not reach 5 because there is no explicit fix-and-retry loop if validation fails, and the summary is labeled "the full 8-step generation process" while listing only seven items (workflow.md actually has eight steps). | 4 / 5 |
Progressive Disclosure | The SKILL.md is a genuine overview with a clear, one-level-deep References table, but the actual bundle structure does not match what is referenced: `templates/skill-output.yaml` and `schemas/design-reference.schema.json` (referenced six times across the body and References table) do not exist, and two bundle files (references/elements-of-design.md, references/principles-of-design.md) are orphaned — not reachable from any documented pointer. These are structural gaps beyond the 4 anchor's "minor organization gaps". | 3 / 5 |
Total | 13 / 20 Passed |