Content
65%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 tightly focused entry-point card with outstanding actionability — exact file paths, rewrite recipes, and markup rules — but it lacks any verification checkpoint and its references point entirely outside the shipped bundle. Tightening the repeated asset path listings and adding a 'verify the deck activates' step would lift it further.
Suggestions
Add a validation step after Recipe A/B, e.g. '4. Verify: open the project index.html and confirm base.css/animations.css/runtime.js resolve (no 404s) and keyboard navigation (arrows, T) works before continuing.'
List the four shared runtime files once and reference them from both recipes instead of enumerating them three times across steps 3A and 3B.
Ship or inline the referenced materials (the master SKILL.md rules, the template folder, and shared assets) so the card's deferred references resolve within the bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly operational but includes unnecessary explanation and repetition: 'because it sits three folders deep inside design-templates/html-ppt/templates/full-decks/' explains relative-URL resolution Claude already knows, and the four asset files are listed in Recipe A, then their rewritten targets listed again, then 'read the same four files' repeats them in Recipe B. This fits 'Mostly efficient but includes some unnecessary explanation or could be tightened'; not a 2 because there is no padding beyond these instances and every section is operationally motivated. | 3 / 5 |
Actionability | The guidance is fully executable: exact copy sources ('copy design-templates/html-ppt/assets/fonts.css, ... assets/base.css, ... assets/animations/animations.css, and ... assets/runtime.js into a project-local assets/'), the exact rewrite ('rewrite the four <link>/<script> tags in index.html from ../../../assets/... to the matching project-local paths'), the body class to keep ('.tpl-testing-safety-alert'), theme location ('assets/themes/*.css'), and speaker-notes markup ('<aside class="notes">'). Not a 4 because both recipes give copy-paste-ready paths covering the common cases with no gaps. | 5 / 5 |
Workflow Clarity | Six numbered steps are clearly sequenced ('Read the master skill first' → copy template → bring the runtime → pick a theme → replace demo content → speaker notes), but no validation checkpoint exists — nothing tells Claude to verify the deck loads or keyboard navigation works after the copy/rewrite. This matches 'Steps listed but validation gaps; checkpoints missing or implicit'; not a 4 because step 3 flags the 404 failure mode as a warning rather than an explicit verify step, and not a 2 because the sequence itself is complete and unambiguous. | 3 / 5 |
Progressive Disclosure | The card is well-structured with headers and clearly signals one-level-deep references ('[design-templates/html-ppt/SKILL.md](../html-ppt/SKILL.md)', 'the upstream [full-decks catalog](../html-ppt/references/full-decks.md)', 'design-templates/html-ppt/LICENSE'), but none of these referenced paths exist in the skill's bundle (no references/, scripts/, or assets/ directories are present), so navigation to the deferred material is broken in this bundle. This fits 'Some structure but could be better organized' territory on navigation; not a 4 because scored against the actual bundle structure, the references resolve to nothing. | 3 / 5 |
Total | 14 / 20 Passed |