CtrlK
BlogDocsLog inGet started
Tessl Logo

review-animations

Reviews animation and motion code against a high craft bar derived from Emil Kowalski's design engineering philosophy. Default to flagging; approval is earned.

59

Quality

69%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.agents/skills/review-animations/SKILL.md

The canonical home for this skill is review-animations in emilkowalski/skills

SKILL.md
Quality
Evals
Security

Quality

Content

85%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A tight, highly actionable rulebook with an unambiguous review workflow and a well-designed output contract — the content quality is strong. The one material defect is bundle structure: STANDARDS.md is referenced four times as the source of exact values but does not exist in the bundle, leaving the skill unable to deliver the precise citations it repeatedly promises.

Suggestions

Ship the referenced STANDARDS.md (easing curves, per-element duration budgets, spring configs, gesture tables) in the bundle, or inline the essential values into SKILL.md and remove the dead links.

Deduplicate the escalation-trigger list against the Ten Standards (e.g., ease-in, scale(0), layout properties, symmetric timing) by marking the standard numbers they escalate, trimming tokens without losing the flag-on-sight signal.

State a fallback for when a needed value is not in STANDARDS.md (e.g., 'if no exact value is catalogued, flag it as unspecified rather than approximating') so the missing-catalog case degrades gracefully.

DimensionReasoningScore

Conciseness

The body is dense and rule-driven with no padding or explanations of concepts Claude already knows — concrete values like `scale(0.9–0.97)`, sub-300ms budgets, and `@media (hover: hover) and (pointer: fine)` appear inline. It stays at 4 rather than 5 because the "Aggressive Escalation Triggers" list substantially re-states the Ten Non-Negotiable Standards (ease-in, scale(0), layout properties, hover gating, symmetric timing all appear twice), which could be tightened into one list with pointers.

4 / 5

Actionability

Guidance is copy-paste concrete throughout: the example findings table gives exact replacements (`transition: transform 200ms ease-out`, `transform: scale(0.95); opacity: 0`, `var(--radix-popover-content-transform-origin)`), and standards cite specific thresholds (sub-300ms, scale 0.9–0.97, 30–80ms stagger). It exceeds the 4 anchor because the example rows cover the common cases end-to-end, not just isolated snippets.

5 / 5

Workflow Clarity

This is a single-purpose review skill and the action is unambiguous: apply the ten standards, flag the escalation triggers, propose fixes via the ordered remedial hierarchy, then emit the two-part required output ending in an explicit Block/Approve decision with concrete criteria. The mandatory findings table plus explicit verdict checklist function as validation checkpoints for the review's own output, matching the simple-skill exception for a 5. Not below 5 because nothing in the sequence is ambiguous and the review task carries no destructive/batch risk requiring additional validation.

5 / 5

Progressive Disclosure

SKILL.md itself is well-structured with clear sections and appropriately defers detailed tables (easing curves, duration budgets, spring config) to a one-level-deep, clearly-signaled [STANDARDS.md](STANDARDS.md) — good design. However, the bundle contains no STANDARDS.md (no references/, scripts/, or assets/ directories exist), so all four links to the rule catalog are broken and the instruction "pull the exact one from STANDARDS.md" cannot be followed, defeating navigation to the detailed material.

3 / 5

Total

17

/

20

Passed

Description

53%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A concise, concrete single-purpose description with natural core keywords, but it lacks any explicit 'Use when...' trigger guidance and omits common synonyms (transitions, easing, micro-interactions) that would improve triggering and completeness. Adding a use-when clause listing natural trigger phrases would lift both completeness and trigger-term quality.

Suggestions

Add an explicit trigger clause, e.g. "Use when reviewing animation, transition, or motion code (CSS transitions/keyframes, Framer Motion, WAAPI) or when a diff changes easing, duration, or transform properties."

Include common user synonyms such as "transitions", "easing", "micro-interactions", and named libraries (Framer Motion) so the skill surfaces for the phrases users actually say.

Optionally enumerate 2-3 concrete checks (easing/duration standards, GPU-only properties, reduced-motion a11y) to raise specificity without adding fluff.

DimensionReasoningScore

Specificity

The description names the domain clearly ("Reviews animation and motion code against a high craft bar") and one concrete action (reviewing) plus a behavioral stance ("Default to flagging; approval is earned"), matching the anchor for 1-2 concrete actions without comprehensive coverage. It is not a 4 because no additional specific capabilities (e.g., checking easing, duration, performance, a11y) are enumerated, and not a 2 because the action and quality bar are concrete rather than generic.

3 / 5

Completeness

The "what" is clear and concrete (reviews animation and motion code against a high craft bar), but there is no "when" — no 'Use when...' clause or equivalent explicit trigger guidance, which per the judging guidelines caps this at 3. It is not a 2 because the "what" half is explicit and specific rather than vague.

3 / 5

Trigger Term Quality

Core natural terms are present — "animation", "motion code", "review" — which a user asking for an animation review would say. It stays at 3 rather than 4 because common variations users actually use are missing: "transition", "easing", "micro-interaction", "Framer Motion", "CSS animation". It is above 2 because the included keywords are natural, not jargon-only.

3 / 5

Distinctiveness Conflict Risk

The animation/motion niche with a craft/flagging framing is mostly distinct from other skills, with only minor overlap risk against general code-review skills that trigger on "review my code". It is not a 5 because the word "review" alone could pull general review skills into contention when the user means a general diff review.

4 / 5

Total

13

/

20

Passed

Validation

93%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 3 missing

Warning

Total

15

/

16

Passed

Repository
1weiho/open-slide
Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.