v5.2.9 — ui-design-taste plugin + release hygiene fixes - #33
Merged
Conversation
Installs the ui-design-taste plugin (5 new skills: emil-design-eng, impeccable, taste, animate, improve-animations) for premium UI/animation work, wired into the frontend domain rule so they load automatically on UI/animation/design keywords. Also fixes three real regressions surfaced by the test suite while reviewing the plugin install: - improve-animations/SKILL.md contained the literal phrase "ignore previous instructions" as a documentation example, tripping kit's own prompt-injection scanner (and CI's security-scan job). Reworded. - manifest.json's lastAuditedAt had been stamped with a real timestamp instead of null, violating the distribution-hygiene contract. - lib/security-scanner.js's 100KB oversized-file ceiling flagged the plugin's legitimate vendored assets (font-index.json, live-browser.js) as HIGH severity, blocking a clean `kit scan`. Raised to 2MB. Synced the 39 -> 44 skill count across six docs that were missed (architecture.md, contributor-guide.md, faq.md, getting-started.md, docs/index.md, mkdocs.yml), and version-bumped to 5.2.9 via scripts/sync-version.js. 1037 tests passing (54 files). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Author
|
/review gemini |
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
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.
Summary
ui-design-tasteplugin viakit plugin install: 5 new skills (emil-design-eng,impeccable,taste,animate,improve-animations) for premium UI/animation work, registered inmanifest.json(39 → 44 skills) and wired into thefrontenddomain rule inloading-rules.json.improve-animations/SKILL.mddocumented its own injection-defense rule using the literal example phrase "ignore previous instructions", which trippedkit scan's scanner and CI'ssecurity-scanjob. Reworded without changing intent.manifest.json.lastAuditedAtregression: was stamped with a real timestamp instead of shipping asnull, violating the distribution-hygiene contract (tests/structural/distribution-hygiene.test.js).font-index.json~1MB,live-browser.js~511KB) exceeded the old 100KB oversized-file threshold inlib/security-scanner.js, keepingkit scanfrom exiting clean. Raised to 2MB — the file-type anomaly check (the one that actually signals something suspicious) is untouched.docs/architecture.mdincl. its ASCII diagram math,docs/contributor-guide.md,docs/faq.md,docs/getting-started.md,docs/index.md,mkdocs.yml), refreshed the "Latest" release blurbs inREADME.md/docs/index.md, and version-bumped to5.2.9viascripts/sync-version.js+package-lock.json.Test plan
npm test— 1037/1037 tests passing (54 files)npm pack --dry-run— tarball builds clean, 365 files, 1.4MB packed / 4.1MB unpacked, all plugin skill files includedmanifest.json's 44 skill entries match the 44 directories under.agent/skills/(no missing/extra)security-scanjob greps locally (ignore previous instructions,disregard all rules) — cleangit tag v5.2.9onmain, thennpm publish(requires maintainernpm login— not authenticated in this environment)🤖 Generated with Claude Code