Content
78%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.
Excellent actionability and lean, non-padded writing: complete executable components, clear decision tables, rules tied to anti-patterns with fixes. The weaknesses are duplication between examples and a monolithic single-file layout with zero progressive disclosure — no references/ files despite enough example material to justify them.
Suggestions
Split the 14+ code examples into one-level-deep reference files (e.g. references/drag-gestures.md, references/text-svg-animation.md, references/loaders.md) and keep SKILL.md as an overview with the decision tables, rules, and anti-patterns, linking to each.
De-duplicate repeated content: drop the standalone drag-to-dismiss example in favor of the end-to-end version, and factor the shared visibilitychange pause pattern (ShimmerSkeleton and PulseDot are near-identical) into one canonical snippet or a shared hook.
Add the missing trigger synonyms from the description's scope (swipe, reorder, spinner/loading state) to the 'When to Activate' list so routing matches how users actually phrase these requests.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and code-forward with essentially no explanation of concepts Claude already knows — no 'what is motion' padding, compact decision tables, and terse one-line principles. It is not a 5 because of verbatim duplication: the drag-to-dismiss sheet appears both standalone and again inside the end-to-end example, and the visibility-pause pattern is repeated nearly identically in both ShimmerSkeleton and PulseDot. | 4 / 5 |
Actionability | Nearly every pattern ships as a complete, copy-paste-ready component with imports, 'use client', token/spring usage, and cleanup — e.g. 'export function BottomSheet', 'export function CursorFollower', 'export function LoadingButton' — covering the common cases named in the Outputs section. The few JSX fragments (draggable card, swipe detection) are idiomatic usage snippets rather than pseudocode, so this sits at anchor 5 rather than 4. | 5 / 5 |
Workflow Clarity | Sequencing and routing are clear: 'Requires motion-foundations to be set up first', 'Use these when motion-patterns is not enough', a 'Choosing the right advanced API' decision table, and numbered Rules cross-referenced from an Anti-Patterns table with fixes. It is not 5 because there is no explicit multi-step workflow with validation checkpoints (acceptable for a non-destructive reference skill, but the anchor-5 feedback-loop structure is absent); not 3 because the ordering, prerequisites, and decision guidance present no real gaps. | 4 / 5 |
Progressive Disclosure | The file has good section structure and no broken or nested references, but there are no bundle files at all — all 14+ full code examples plus an end-to-end example are inlined in a single ~590-line SKILL.md. Content that would fit one-level-deep reference files (loader components, text/SVG examples, the end-to-end sheet) is inline, matching the anchor-3 pattern of 'content that should be separate is inline' despite decent organization. | 3 / 5 |
Total | 16 / 20 Passed |