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 highly actionable, well-organized reference for ScrollTrigger with excellent executable code throughout, undermined by token-efficiency issues (guidance repeated 3-4x across the body, including a verbatim duplicate Do-Not bullet) and by inlining a large API-reference table and advanced sections that belong in separate reference files. Deduplicating the best-practices/Do-Not overlap and splitting the reference material would lift both weak dimensions.
Suggestions
Consolidate repeated guidance into a single canonical statement: refresh()-after-layout-change is stated in the Refresh and Cleanup section, twice in the best-practices list, and twice as near-verbatim Do-Not bullets (one of which is a duplicate to delete outright); the same applies to the markers-in-production and containerAnimation ease:'none' warnings.
Move the 20-row config-options table and the advanced static APIs (ScrollTrigger.batch(), ScrollTrigger.scrollerProxy()) into a references/ file (e.g. references/api.md), keeping SKILL.md a lean overview with one-level-deep pointers per the progressive-disclosure model.
Merge the 'Official GSAP best practices' and 'Do Not' sections into a single compact checklist, since they largely restate each other; this would cut roughly 30-40 lines without losing any unique guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The core sections (config table, batch, scrollerProxy, horizontal scroll) are dense and efficient, but guidance is repeated across sections: refresh-after-layout-change appears in 'Refresh and Cleanup', twice in the best-practices list, and as two near-verbatim duplicate bullets in 'Do Not' (lines 307 and 314); 'ease: none' for containerAnimation is stated 4+ times and markers-in-production 3 times. This is more than minor trimming, so it fits 'mostly efficient but includes some unnecessary explanation' rather than the level-4 'minor instances'. | 3 / 5 |
Actionability | Every section provides executable, copy-paste-ready JavaScript: gsap.registerPlugin, basic triggers with commented start/end/toggleActions, ScrollTrigger.create, batch with interval/batchMax, a full scrollerProxy example, scrub, pin, timeline, a dynamic horizontal-scroll example with function-based x/end and invalidateOnRefresh, and kill/getById cleanup. Concrete examples cover the common cases, matching the top anchor. | 5 / 5 |
Workflow Clarity | Content is sequenced logically (register plugin first, then patterns, then refresh/cleanup lifecycle), the containerAnimation pattern is laid out as an explicit numbered 1-4 sequence, and markers: true serves as a dev-time visual validation checkpoint. There is no destructive or batch file operation requiring a validation cap, but the reference nature leaves minor gaps (no explicit verify-setup or error-recovery steps), so level 4 fits better than 5. | 4 / 5 |
Progressive Disclosure | This is a single-file skill over 50 lines with a large config-options table plus advanced static API sections (batch, scrollerProxy) inlined — classic reference material that belongs in references/ files. Sections are well-organized with clear headers, matching 'some structure but content that should be separate is inline' rather than level 4's 'most content appropriately placed' since no reference files exist at all. | 3 / 5 |
Total | 15 / 20 Passed |