Content
71%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.
A well-structured, lean index-style skill with a strong routing table, concrete script invocation, and real bundle support for motion-blur and the animation-map audit script. The main weakness is that most routing-table targets (rules, blueprints, transitions, techniques, adapters) are not present in the provided bundle, and the default compose path lacks an inline minimal example.
Suggestions
Ship the routing-table targets (rules-index.md, blueprints/, transitions/, techniques.md, adapters/) in the skill bundle, or note in SKILL.md where they live — as provided, 18+ referenced paths are dangling.
Add a short inline minimal example of the default path (2-4 rules glued on one paused GSAP timeline) so the primary workflow is executable without first loading rules/<name>.md.
Fold the 'Picking a runtime' bullets into the routing table (or vice versa) and drop the opening recap that restates the frontmatter description to trim redundant tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely lean — a routing table, terse decision rules, and one script block — and the constraint section is genuinely non-obvious domain contract rather than concepts Claude already knows. Minor trimmable padding exists: the opening paragraph restates the frontmatter description verbatim in structure, and 'Picking a runtime' overlaps the adapter rows of the routing table. Fits 'efficient; minor instances of over-explanation' rather than the every-token-earns-its-place anchor. | 4 / 5 |
Actionability | The script section gives a copy-paste-ready bash command with flags and output path, the routing table gives exact file targets for each need, and the blueprint section states concrete load conditions. The primary compose path ('glue them together with a single paused GSAP timeline') never shows an inline minimal example — it delegates to rules/<name>.md — which is the 'minor gap' separating it from fully copy-paste-ready coverage of the common case. | 4 / 5 |
Workflow Clarity | A clear default-first sequence is present: compose from 2-4 atomic rules, escalate to a blueprint only under two explicit conditions, then audit the result — 'Use it to audit choreography (dead zones, stagger consistency, lifecycle warnings) after authoring' is an explicit post-authoring validation checkpoint. Not anchor 5 because the audit-to-fix feedback loop is only implied and lint/check validation lives in a sibling skill ('hyperframes-cli') rather than being sequenced here. | 4 / 5 |
Progressive Disclosure | The SKILL.md is structured exactly as an index should be (overview, routing table, one-level-deep references), and references/motion-blur.md and scripts/animation-map.mjs are real, substantial files. However, scored against the actual bundle structure, the majority of referenced paths — rules-index.md, rules/<name>.md, blueprints-index.md, blueprints/<id>.md, transitions/overview.md, transitions/catalog.md, techniques.md, and all 13 adapters/*.md — have no corresponding files in the provided bundle, making most navigation targets unverifiable/dangling. That is more than the 'minor organization gaps' of anchor 4, while the signaling itself is far better than anchor 3's buried references; the missing files are the deciding factor. | 3 / 5 |
Total | 15 / 20 Passed |