docs(repo): add headless primitives reference to mosaic skill#9239
Conversation
Adds a how-to reference for @clerk/headless (consuming primitives, the useRender part-authoring contract, file layout, animation lifecycle, shared hooks/utils) and routes to it from the mosaic skill. Fixes stale renderElement references in the headless README (removed in favor of useRender).
🦋 Changeset detectedLatest commit: a7512c0 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.
|
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-google-signin
@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: |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds Mosaic documentation for Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.claude/skills/mosaic/references/headless.md:
- Around line 52-62: Align the documentation’s useRender contract across both
sites: in .claude/skills/mosaic/references/headless.md, scope native props and
render behavior to each element-rendering part and state that pass-through parts
such as Root and Portal have their own APIs; in packages/headless/README.md,
change the useRender description to say that every part rendering a DOM element
calls the hook.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: 29d85d06-ed0a-4ad5-9894-bafa2fa0a1b3
📒 Files selected for processing (4)
.changeset/mosaic-headless-reference-doc.md.claude/skills/mosaic/SKILL.md.claude/skills/mosaic/references/headless.mdpackages/headless/README.md
🔗 Linked repositories identified
CodeRabbit considers these linked repositories for cross-repo context during reviews:
clerk/clerk_go(manual)clerk/dashboard(manual)clerk/accounts(manual)clerk/backoffice(manual)clerk/clerk(manual)clerk/clerk-docs(manual)clerk/cloudflare-workers(manual)clerk/cli(auto-detected)clerk/clerk-ios(auto-detected)clerk/clerk-android(auto-detected)
Description
Adds a reference doc for working with the
@clerk/headlesspackage components, within themosaicskill, and fixes stale docs.What's here
references/headless.mdin the mosaic skill — a how-to for the shared headless conventions (the per-primitive API docs stay in eachsrc/primitives/<name>/README.md, which this points to):renderfunction-or-element override,data-*styling,makeCustomizablefrom@clerk/ui).useRendercontract + param table, universal rules ('use client',forwardRef,ownProps satisfies DefaultProps<Tag>,mergeProps(internal, consumer)ordering),stateAttributesMapping.root+mountedconvention, thedata-*animation lifecycle, shared hooks (useControllableState,useTransition, …) and utils (cssVars,resetLayoutStyles).SKILL.mdtable.renderElementreferences inpackages/headless/README.md(renderElementwas removed in feat(headless): remove renderElement, consolidate into useRender #9233 and consolidated intouseRender; the dist-path in the TS2742 note is nowuse-render).Changeset
Empty — docs/tooling only, and the README change is in the private
@clerk/headlesspackage. No user-facing published change.