feat(*): Wire clerkUIVersion through all framework packages#7740
feat(*): Wire clerkUIVersion through all framework packages#7740nikosdouvlis wants to merge 4 commits intomainfrom
Conversation
Why: The `clerkUIVersion` prop was typed in `@clerk/shared/types` but never wired through framework packages, making it effectively dead code. Users could pass the prop without TypeScript errors, but it would be silently ignored. This mirrors the existing `clerkJSVersion` functionality for `@clerk/ui`. What changed: - Added `NEXT_PUBLIC_CLERK_UI_VERSION` env var support to Next.js - Added `PUBLIC_CLERK_UI_VERSION` env var support to Astro - Added `VITE_CLERK_UI_VERSION` env var support to React Router and Tanstack Start - Added `CLERK_UI_VERSION` env var support to Express - Added `NUXT_PUBLIC_CLERK_UI_VERSION` env var support to Nuxt - Wired the prop through SSR state for React Router and Tanstack Start
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: 8685399 The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
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 |
📝 WalkthroughWalkthroughThis PR adds a new clerkUIVersion property and corresponding environment variables across multiple integrations (Astro, Express, Next.js, Nuxt, React Router, TanStack React Start). Environment variable names include PUBLIC_CLERK_UI_VERSION, NEXT_PUBLIC_CLERK_UI_VERSION, and CLERK_UI_VERSION. The value is threaded through params, env merging, runtime constants, and emitted clerk state without changing function signatures or control flow. 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Important Action Needed: IP Allowlist UpdateIf your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:
Failure to add the new IP will result in interrupted reviews. Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
packages/react-router/src/server/utils.ts (1)
80-103:⚠️ Potential issue | 🟠 MajorAdd tests for clerkUIVersion propagation before merge.
This PR introduces a new runtime configuration path (
__clerkUIVersionin the SSR clerk state), but no tests were added/updated to cover it. Please add a unit or integration test to validate that the env var is correctly read and serialized into the response state.packages/react-router/src/client/types.ts (1)
6-28:⚠️ Potential issue | 🟠 MajorAdd tests for clerkUIVersion env/config propagation.
No tests were added or updated for this multi-framework change (env parsing, runtime config, script URL generation). Please add coverage to ensure the new clerkUIVersion wiring is respected end‑to‑end before merge.
@clerk/agent-toolkit
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@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: |
Summary
Wire
clerkUIVersionoption through all framework packages to allow specifying a custom@clerk/uiversion.clerkUIVersionto Astro'sAstroClerkIntegrationParamstype🤖 Generated with Claude Code
Summary by CodeRabbit