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, dense index-style skill body with a clean routing table, precise constraints, and a runnable audit script. Its main defects are a self-contradictory contract restatement in Critical Constraints, no inline example for the default composition workflow, no fix-and-re-audit feedback loop, and — most significantly — a routing table whose destinations are mostly absent from the provided bundle.
Suggestions
Ship (or remove) the routing-table destinations: `rules-index.md`, `blueprints-index.md`, `transitions/overview.md`, `transitions/catalog.md`, `techniques.md`, and the `adapters/*.md` files are referenced but missing from the bundle, so most routes dead-end — trim the table to what is actually shipped or bundle the files.
Add a short feedback loop after the animation-map audit step (e.g. "if the map flags dead zones or lifecycle warnings, fix the timeline and re-run") so validation has an explicit recovery path rather than ending at the report.
Delete or shrink the Critical Constraints parenthetical that lists ~8 core rules immediately after stating "Don't restate the full contract here" — trust the pointer to `hyperframes-core` and keep only the two animation-craft additions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and assumes Claude's competence — no explanations of what GSAP/Lottie/Three.js are, and the routing table and runtime picker are dense signal. The one trimmable spot: the Critical Constraints section says "Don't restate the full contract here" and then immediately summarizes ~8 core rules in a parenthetical ("single paused timeline, `data-duration` governs length, no `Math.random` / `Date.now` / `performance.now`, no `repeat: -1`..."). Minor over-explanation that could be trimmed (4 anchor), but the rest is lean enough that it is not the noticeably-verbose 3. | 4 / 5 |
Actionability | Mostly executable guidance: a copy-paste-ready bash command for the audit script, precise do/don't constraints ("never derive positions from `getBoundingClientRect()` at tween time"; "never `width` / `height` / `top` / `left` for layout changes"; "Spatial motion uses GSAP transform aliases only"), and a concrete default workflow ("Pick 2-4 rules from `rules-index.md`, glue them together with a single paused GSAP timeline, done"). The minor gap: no inline minimal example of the default compose-rules-into-a-timeline step — the "glue them together" step is left entirely to `hyperframes-core` and the rule files, so the most common case has no worked snippet (4 anchor, not 5). | 4 / 5 |
Workflow Clarity | Clear decision structure: an explicit default path (compose atomic rules), explicit deviation criteria ("Load a blueprint when — The scene matches an existing pre-designed multi-phase template..."), a routing table keyed by intent, and an explicit post-authoring validation step ("Use it to audit choreography (dead zones, stagger consistency, lifecycle warnings) after authoring"). Not 5 because there is no error-recovery feedback loop — what to do when the audit or `hyperframes lint/check` flags issues (fix and re-audit) is left implicit, and lint/check integration appears only in See Also; not 3 because checkpoints are present and the sequence is unambiguous. | 4 / 5 |
Progressive Disclosure | The SKILL.md itself is a model router — a well-signaled routing table, "Do not read it speculatively; load it when you've already decided", and bulk detail correctly kept out of the overview. However, scored against the actual bundle: of the ~20 referenced destinations (`rules-index.md`, `rules/<name>.md`, `blueprints-index.md`, `blueprints/<id>.md`, `transitions/overview.md`, `transitions/catalog.md`, `techniques.md`, and ~11 `adapters/*.md` files), only `references/motion-blur.md` (verified substantive, 133 lines, one level deep, no nested references) and `scripts/animation-map.mjs` actually exist. Most routes dead-end in the provided bundle, which is more than the "minor organization gaps" of the 4 anchor; the strong in-file structure keeps it above 3's "references not clearly signaled" framing only because the signaling itself is excellent — the delivery is not. | 3 / 5 |
Total | 15 / 20 Passed |