docs(skill): add web-animation-design skill (original-prose, inspire-only)#316
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…only) New .claude/skills/web-animation-design/SKILL.md — a front-end animation decision guide (easing families, sub-300ms duration bands, transform/opacity GPU rule, springs, prefers-reduced-motion + touch-hover a11y, before/after review-table format). Ties into docs/design.md §Motion + frontend/lib/useMotion.ts. License posture: the uploaded mcpmarket file was distilled from Emil Kowalski's paid "Animations on the Web" course with NO declared license. Shipped as ORIGINAL PROSE / INSPIRE-ONLY (zero upstream text copied; the principles are standard published front-end facts; Emil Kowalski + easings.net credited as inspiration) per the project's good-code-bad-code / 9arm-fallback precedent. The earlier curl|bash marketplace installer was declined as an RCE vector. Skill count 45 -> 46 in 4-doc lockstep (CLAUDE.md, AGENTS.md, SKILL.md, .claude/skills/README.md); THIRD_PARTY_NOTICES.md entry added. Docs/.claude only; no compute/schema/frontend code change; no CI code-path impact. https://claude.ai/code/session_0144kHrCYNaamMPH57b7xdM7
dackclup
force-pushed
the
claude/optimistic-brown-UUcXA
branch
from
d790abc to
e064138
Compare
This was referenced May 29, 2026
dackclup
added a commit
that referenced
this pull request
May 29, 2026
…ONS_LEARNED (#319) Follow-up to #318: its docs-reviewer gate (which returned after #318 merged) caught that the #316 skill-count 45→46 bump updated only 4 of the 7 doc homes. PHASE_STATUS.md §Current state, CONTEXT.md (×3 mentions), and .claude/agents/README.md stayed stale at 45. This PR brings all 7 homes to 46 and corrects the docs/LESSONS_LEARNED.md entry that itself under-counted them ("4 doc homes" → "7"; bare "README" → ".claude/skills/README.md"; top-level README.md has no skill count). Fittingly self-demonstrating: the first follow-up grep still missed two homes (different phrasings), needing broader passes — exactly the lesson. Doc-only — no compute / schema / scoring / valuation / frontend code change; ruff clean. https://claude.ai/code/session_0144kHrCYNaamMPH57b7xdM7 Co-authored-by: Claude <noreply@anthropic.com>
dackclup
pushed a commit
that referenced
this pull request
May 30, 2026
…iewer SHOULD-FIX) The Recently-merged PR #316 entry substituted '(skill count 45 → 46)' for the commit's own '(original-prose, inspire-only)'. Both are true; keep both for commit-log fidelity per the docs-reviewer re-check note. https://claude.ai/code/session_0144kHrCYNaamMPH57b7xdM7
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds a new top-level Claude Code skill
web-animation-designat.claude/skills/web-animation-design/SKILL.md— a front-end animation decision guide:transform+opacityonlyprefers-reduced-motion+ touch-hover (@media (hover: hover) and (pointer: fine)) accessibilityIt ties directly into the existing motion system —
docs/design.md§Motion (the 5 binding rules) andfrontend/lib/useMotion.ts(useCountUp/usePlayOnMount+ thegauge-sweepkeyframe) — so it's the rationale layer under the project's contract.License posture — ORIGINAL PROSE / INSPIRE-ONLY
The file the maintainer uploaded was an
mcpmarketbundle distilled from Emil Kowalski's paid "Animations on the Web" course with no declared license. To avoid redistributing commercial-course-derived expression to a public repo, this skill ships as original QuantRank prose — zero upstream text copied. The principles it covers are standard, widely-published front-end facts (MDN / web.dev / CSS-Tricks / easings.net); Emil Kowalski + easings.net are credited as inspiration in the skill's Attribution section.This matches the project's established precedent for undeclared-license sources (
good-code-bad-code-reviewreference-link,9arminspire-only fallback) — documented in a newTHIRD_PARTY_NOTICES.mdentry. (The earliercurl … | bashone-liner from the same marketplace was declined as an unverified remote-code-execution vector; only inspected, hand-written content was adopted.)Doc updates
CLAUDE.md§Layout +SKILL.mdlayout table +.claude/skills/README.mdTHIRD_PARTY_NOTICES.mdentry (inspire-only posture + action-items table)PHASE_STATUS_INFLIGHT.mdentry (lockstep)Scope / verification
.claude/+ docs only — no compute / schema / scoring / valuation / frontend-code change, no CI code-path impact (markdown skill).ruff check .clean. Branch rebuilt on top ofmain(post-#315-merge).https://claude.ai/code/session_0144kHrCYNaamMPH57b7xdM7
Generated by Claude Code