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 a dense, actionable reference with a clear workflow and verification step, but it is undermined by dangling references — the entire progressive-disclosure layer (examples, API reference, templates, validation script) points to files that do not exist — and by padded, redundant sections (duplicate design philosophy, duplicated when-to-use, meta-commentary metadata).
Suggestions
Ship the referenced bundle files (./examples/, ./reference/, ./templates/, ./scripts/validate_motion_config.py, ./schema/motion-config.schema.json) or remove the links — as delivered, every on-demand reference is a dead end.
Merge 'Purpose' and 'Design Philosophy (Apple/Jon Ive Principles)' into one section and delete the trailing 'Metadata' section (research citations, token-efficiency claims) — none of it changes how the skill is executed and it duplicates frontmatter metadata.
Make the 'Common Patterns' snippets fully executable by adding the import line (e.g. import { motion } from 'motion/react') to each code block, and drop the 'When to Use' section that restates the frontmatter description.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The bulk is dense reference material (tables, decision tree, patterns), but several sections are padded or redundant: 'Purpose' and 'Design Philosophy (Apple/Jon Ive Principles)' state the same philosophy twice, 'When to Use' duplicates the frontmatter description, and the trailing 'Metadata' section (arXiv citations, '87% core reduction') is meta-commentary irrelevant to execution. This fits 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than anchor 2, since the core content is compact tables rather than prose re-explaining known concepts. | 3 / 5 |
Actionability | Strongly concrete: copy-paste patterns ('whileHover={{ y: -8, boxShadow: … }}'), a decision tree with exact values ('stiffness: 300-400, damping: 20', 'viewport={{once: true, amount: 0.3}}'), install commands, and an error table with specific fixes. Falls short of 5 because the 'Copy-Paste Ready' snippets omit import statements (imports appear only in the framework table) and the 'More patterns → ./examples/ directory' pointer targets files that do not exist in the bundle. | 4 / 5 |
Workflow Clarity | The 'Clarify → Plan → Implement → Verify' workflow is a clear sequence, and Step 4 is an explicit verification checkpoint with named tools (Chrome DevTools Performance tab, Lighthouse CLS audit, system reduced-motion test), complemented by the Error Handling table for recovery. Not 5 because checkpoints name tools rather than define validate→fix→retry loops, and the 'Implement in Phases' bullets remain high-level. | 4 / 5 |
Progressive Disclosure | The structure is well signaled — 'Examples Library (Progressive Loading)', 'Reference Documentation (Load On-Demand)', and Templates sections all point one level deep — but the actual bundle contains none of the referenced files: ./examples/*.md, ./reference/api-reference.md, ./reference/spring-physics.md, ./templates/*.tsx, ./scripts/validate_motion_config.py, and ./schema/motion-config.schema.json are all missing. The on-demand loading strategy the skill is built around breaks at runtime; the reasonably self-sufficient inline content keeps this at 3 rather than 2. | 3 / 5 |
Total | 14 / 20 Passed |