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, highly actionable contract document with excellent progressive disclosure and a concrete validation checklist. Its main weakness is redundancy — several rules are stated two or three times across the pitfalls, lint-gotchas, and non-negotiable sections — which inflates token cost without adding information.
Suggestions
Deduplicate the three rules stated multiple times (the `gsap_css_transform_conflict` centering rule, the `.clip` autoAlpha/visibility ban, and root sizing `100%` vs `1920px`): state each once in the most relevant section and cross-reference it elsewhere with just the lint rule name.
Consolidate "Agent pitfalls", "First-pass lint gotchas", and "Non-negotiable rules" into one section — they overlap heavily and the three-way split is what drives the repetition; a single grouped list (caught-by-lint vs silent) would be shorter and easier to scan.
Tighten the long multi-clause parenthetical sentences (e.g. the media-nesting rule and the unique-id rule) by moving their edge-case detail into the already-linked references, keeping only the rule and its lint name inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, assumes competence (no basic HTML/GSAP explanations), and earns most of its tokens — but three rules are each stated twice within ~70 lines: the `gsap_css_transform_conflict` rule appears in "Agent pitfalls" and again in "First-pass lint gotchas"; the `.clip` autoAlpha/visibility ban appears three times (Agent pitfalls, lint gotchas, and non-negotiable rules); and root sizing (`100%` not `1920px`) appears in both "Root must be sized" and the lint gotchas list. Sentences like the media-nesting rule carry long multi-clause parentheticals that could be tightened. This is more than the 'minor instances' of anchor 4, though well short of padded concept explanations, so it lands at 3. | 3 / 5 |
Actionability | Mostly executable guidance: named lint rules with their exact fixes (`gsap.fromTo(el, { x: -40 }, { x: 0 })` instead of CSS `transform: translateX(-40px)`), exact CLI commands (`npx hyperframes timeline [--json]`, `npx hyperframes snapshot --at <midpoints>`), and specific attribute contracts (`window.__timelines["<id>"]` keyed to `data-composition-id`). Not a 5 because the complete copy-paste-ready composition skeleton lives in `references/minimal-composition.md` rather than inline — defensible, but the body itself has only one small code snippet. | 4 / 5 |
Workflow Clarity | The build → edit → validate flow is clear, and the final Validation section is an explicit checklist of concrete commands with an ordering gate ("`npx hyperframes render` only after the user approves") plus a trap explanation (a lint error silently disables the layout/contrast audits). It falls short of anchor 5 because no explicit validate → fix → re-run feedback loop is described — error recovery is implied by the lint-rule list rather than sequenced. | 4 / 5 |
Progressive Disclosure | The body is a genuine overview/build guide with a purpose-labeled references table ("Read it to…") covering all 10 files; every referenced path exists in `references/` and is substantive (verified, e.g. `minimal-composition.md` contains a complete runnable skeleton), all references are one level deep, and detail is correctly delegated per topic. Cross-links to sibling skills are clearly signaled. This matches the anchor-5 example structure well. | 5 / 5 |
Total | 16 / 20 Passed |