feat(repo): Add mosaic migration skill#9106
Conversation
🦋 Changeset detectedLatest commit: 36130eb The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Repository UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThis PR adds the Mosaic Claude Code skill and related reference docs for styling, machine/controller/view contracts, isolated testing, migration, and parity auditing. It also updates the skills index, architecture reference, and adds a changeset. ChangesMosaic Skill Documentation
Estimated code review effort: 3 (Moderate) | ~25 minutes Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
Merge the mosaic-machine and mosaic-migration skills into one `mosaic` skill with per-layer references (machines, controllers, views), the migration playbook, and the parity-audit prompt. Update the skills README table and the mosaic-architecture migration pointer.
Add a three-layer testing reference (machine/controller/view) to the mosaic skill with real snippets from the delete-organization tests. Collapse the skill's machines.md into a pointer to the in-tree machine/README.md + ADOPTION.md so the machine runtime has a single source of truth. Add the testing row to the SKILL.md router and broaden the trigger to cover testing.
Add a styling how-to to the mosaic skill covering useRecipe with single-slot and multi-slot recipes, slot registration, the variant-vs-state distinction, compoundVariants/sx, and the useSlot/slot sugar, grounded in the real button and tabs components. Add the styling row to the SKILL.md router and broaden the trigger to cover styling.
|
here's what a 1-shot did when asking it to use the skill to create the members list: |
Description
Adds a new skill to help with mosaic component migrations to go from the current component to an abstracted controller/machine/view approach.
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change
Summary by CodeRabbit