Content
82%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 tight, example-driven reference: every section carries executable code or precise API syntax, assumes GSAP-adjacent competence without padding, and is easy to navigate. The main weaknesses are mild redundancy between the Best practices/Do Not sections and the earlier body, and topic-style organization instead of an explicit build-a-timeline workflow.
Suggestions
Consolidate the "Official GSAP Best practices" and "Do Not" sections: several bullets restate earlier sections verbatim (the defaults example, position parameter, and labels guidance each appear twice) — keep only net-new rules such as the ScrollTrigger placement guidance.
Consider framing the sections as an explicit ordered workflow for building a timeline (create → set defaults → place tweens with position parameters → label → nest → control) so the reference reads as a process rather than isolated topics.
If the skill grows further, move the full position-parameter syntax list and the playback-control method reference into a references/ file, keeping SKILL.md as the navigable overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and code-dense with no concept explanations (no "GSAP is a popular library..." padding), and every section teaches GSAP-specific syntax. It falls short of anchor 5 because the "Official GSAP Best practices" and "Do Not" sections restate earlier sections — e.g., the defaults example `defaults: { duration: 0.5, ease: "power2.out" }` appears verbatim in both "Timeline Defaults" and "Do Not", and the position-parameter and labels bullets are repeated — so a handful of tokens could be trimmed. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready JavaScript throughout: timeline creation, position-parameter examples with inline comments (`tl.to(".b", { y: 50 }, "+=0.5"); // 0.5s after last end`), defaults, labels, nesting, and precise control methods (`tl.time(2)`, `tl.progress(0.5)`, `tl.kill()`). The examples cover the common cases for each feature, matching anchor 5 with no gaps. | 5 / 5 |
Workflow Clarity | Sections progress in the order a developer actually builds a timeline — create, position, defaults, constructor options, labels, nesting, playback control — so the sequence is clear. It sits at anchor 4 rather than 5 because the content is organized as reference topics rather than an explicit ordered workflow, and there are no verification/debugging checkpoints (acceptable here since animation code involves no destructive or batch operations, so the workflow-clarity cap of 3 does not apply). | 4 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent) and the skill is self-contained; the body is well-organized with clear section headers and a cross-link to sibling skills ("For single tweens and eases use gsap-core; for scroll-driven timelines use gsap-scrolltrigger"). At ~95 lines it exceeds the under-50-lines simple-skill case, and denser API reference material (the full position-parameter syntax, playback-control method list) could eventually move to a references/ file, which places it at anchor 4 rather than 5. | 4 / 5 |
Total | 17 / 20 Passed |