Content
63%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-sectioned with runnable commands and a real references layer, but it is let down by an oversized inlined Slide System section and a significant bundle mismatch: many of the data, asset, and template paths it instructs Claude to read do not exist in the skill directory. The Chart.js example also contradicts the skill's own token-compliance rules by hardcoding hex values.
Suggestions
Move the Slide System details (CSV inventories, decision flow, Chart.js integration, pattern breaking) into a 'references/slide-system.md' file and keep only the quick-start commands and requirements in SKILL.md, cutting the body by roughly half.
Fix the bundle mismatch: either ship the referenced 'data/*.csv', 'assets/', and 'templates/design-tokens-starter.json' files or remove/repair those references, since the contextual decision workflow is unexecutable without them.
Rewrite the Chart.js example to use var(--color-primary) and token-derived colors so it complies with the skill's own 'WRONG - hardcoded' guidance, and add a brief feedback loop after the validation step (fix violations and re-run slide-token-validator.py).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient tables and code, but the ~130-line Slide System section inlines CSV inventories, an ASCII decision-flow diagram, and Chart.js/animation details that duplicate what the data files and reference implementation already encode, and the three-layer token architecture is explained both inline and again in 'references/token-architecture.md'. This fits 'Mostly efficient but includes some unnecessary explanation or could be tightened' rather than the minor-trim level of 4. | 3 / 5 |
Actionability | Commands are concrete and runnable — 'node scripts/generate-tokens.cjs --config tokens.json -o tokens.css', 'python scripts/search-slides.py "problem slide" --context --position 2 --total 9' — and the Chart.js snippet is copy-paste ready, all matching scripts that actually ship. It is not a 5 because the Contextual Decision Flow is descriptive ('Query slide-layout-logic.csv → layout') rather than executable, and the Chart.js example itself hardcodes '#FF6B6B' and 'rgba(255, 107, 107, 0.1)' instead of demonstrating the mandated var(--...) pattern. | 4 / 5 |
Workflow Clarity | The Contextual Decision Flow gives a clearly numbered 5-step sequence ending in an explicit validation step ('5. Validate with slide-token-validator.py'), and the Quick Start pairs generation with 'validate-tokens.cjs'. It falls short of 5 because there is no error-recovery feedback loop (what to do when validation fails) and the early steps are abstract ('Parse goal/context') rather than concrete commands; slide generation is not destructive, so the cap-at-3 rule does not apply. | 4 / 5 |
Progressive Disclosure | Scored against the actual bundle: the seven 'references/*.md' files exist and are clearly signaled via a topic table, but the body also points to files that are absent from the bundle — eight 'data/*.csv' files, 'assets/design-tokens.json', 'assets/css/slide-animations.css', 'assets/designs/slides/claudekit-pitch-251223.html', and template 'design-tokens-starter.json' — and the bulk of the Slide System content is inlined in SKILL.md where it belongs in a reference file. This matches 'Some structure but could be better organized... content that should be separate is inline' rather than the good-structure anchor at 4. | 3 / 5 |
Total | 14 / 20 Passed |