Content
67%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.
The body is actionable and well-sectioned, with executable examples for every major pattern (useGSAP, context+revert, contextSafe, SSR). Its main weaknesses are redundancy — the 'Do Not' section largely duplicates 'Best practices' and itself — and small completeness gaps in the primary code snippet.
Suggestions
Merge the 'Do Not' section into 'Best practices' (or deduplicate it: items 1 and 2 state the same selector-without-scope rule, and items 3 and 4 restate Best practices bullets) to reclaim tokens.
Add `import gsap from "gsap";` (and the useRef import) to the first useGSAP snippet so the primary example is fully copy-paste ready.
Compress the 'Refs for Targets' prose paragraph into a short bullet rule matching the style of the other sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient and teaches GSAP-React-specific knowledge (scope, contextSafe, revertOnUpdate) rather than general concepts, matching anchor 3's 'mostly efficient but could be tightened'. It is not anchor 4 because the redundancy is more than minor: 'Do Not' items 1 and 2 restate the same selector-without-scope rule, and 'Do Not' items 3 and 4 duplicate the 'Best practices' bullets, while the 'Refs for Targets' section is prose that could be a single rule line. | 3 / 5 |
Actionability | Concrete, near copy-paste-ready code throughout: install commands, registerPlugin call, useGSAP with scope, gsap.context() with ctx.revert() cleanup, and a complete contextSafe example with listener removal, matching anchor 4. It falls short of anchor 5 because the primary useGSAP snippet omits the `import gsap` statement, so the first example is not fully self-contained. | 4 / 5 |
Workflow Clarity | The decision flow is clear — prefer useGSAP when @gsap/react is available, fall back to gsap.context() + ctx.revert() in useEffect, keep everything client-side for SSR — matching anchor 4's clear sequence. It is not anchor 5 because the flow is spread across parallel sections rather than one ordered sequence, and there are no explicit checkpoints (acceptable here since the skill involves no destructive or batch operations, so the validation cap does not apply). | 4 / 5 |
Progressive Disclosure | With no bundle files present, the single-file structure is well-organized with clear section headers and a one-level-deep external link ('Learn More' to gsap.com) plus pointers to sibling skills, matching anchor 4. It is not anchor 5 because at ~130 lines some deeper material (the contextSafe walkthrough, revertOnUpdate/dependency details) is inlined and could be split into reference files, and there are no well-signaled internal references to structure navigation. | 4 / 5 |
Total | 15 / 20 Passed |