Content
75%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-organized, actionable best-practices reference with executable examples and concrete do/don't guidance throughout. Its main weaknesses are the redundant 'Best practices' summary section, advisory guidance lacking concrete APIs (kill/pause off-screen animations), and absence of a diagnostic sequence for performance tuning.
Suggestions
Remove or merge the 'Best practices' section (lines 67-72) into the preceding sections — it restates the stagger, quickTo, will-change, and cleanup guidance already given, trimming redundant tokens.
Make the off-screen cleanup guidance concrete: name the APIs (e.g. gsap.killTweensOf(), tween.pause() paired with an IntersectionObserver or ScrollTrigger onEnter/onLeave) so 'pause or kill off-screen animations' is executable.
Add a short diagnostic sequence for optimization tasks — e.g. confirm jank in DevTools (FPS meter) → switch layout properties to transforms → reduce simultaneous tweens → re-measure — to give the guidance an actionable order.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely lean with concrete, GSAP-specific guidance, but the 'Best practices' section (lines 67-72) restates stagger, quickTo, and cleanup guidance already covered above, and the 'Do Not' section partially overlaps as well. This is anchor 4's 'minor instances that could be trimmed' rather than anchor 5's every-token-earns-its-place, and not anchor 3 since there is no concept over-explanation or padding beyond the redundant summary sections. | 4 / 5 |
Actionability | Concrete property lists (x, y, scaleX, rotation, opacity), an executable copy-paste quickTo example, a will-change CSS snippet, and named commands (ScrollTrigger.refresh(), stagger) give mostly executable guidance. It falls short of anchor 5 because some guidance stays advisory without the 'how' — 'Pause or kill off-screen or inactive animations' gives no API (e.g. gsap.killTweensOf) or detection method, and stagger usage is named but not shown. | 4 / 5 |
Workflow Clarity | As a best-practices reference rather than a multi-step process, the content is well organized: a clear 'When to Use' entry point, topic sections, and consolidated do/don't lists, with no destructive or batch operations requiring validation. It does not reach 5 because there is no optimization sequence (e.g., measure/profile FPS first, then apply transforms, then re-measure) to guide an actual performance-tuning task. | 4 / 5 |
Progressive Disclosure | The skill is self-contained with clear section headers and a one-level, well-signaled pointer to sibling skills ('for ScrollTrigger performance see gsap-scrolltrigger'). It sits at anchor 4 rather than 5 because the body exceeds 50 lines with the ScrollTrigger section duplicating scope a sibling skill already covers, and the 'Best practices'/'Do Not' sections restate inline content instead of tightening the structure. | 4 / 5 |
Total | 16 / 20 Passed |