Skip to content

Conversation

@jacekradko
Copy link
Member

@jacekradko jacekradko commented Jan 21, 2026

Summary

⚠️ DO NOT MERGE - For review/discussion only

This PR removes the headless build variant from @clerk/clerk-js and introduces a new ui prop to control whether @clerk/ui is loaded.

Background

After the UI split to @clerk/ui, the browser builds are now identical in size:

  • clerk.browser.js = 242K
  • clerk.headless.browser.js = 242K

The headless build is redundant. A new ui prop provides cleaner control over UI loading.

New ui Prop API

The ui prop replaces clerkJSVariant: 'headless':

// Skip loading @clerk/ui entirely
<ClerkProvider ui={false} />

// Default behavior (load UI normally)
<ClerkProvider />

Changes

Build changes:

  • Add react-native export condition in package.json for Expo/RN
  • Rename clerkHeadless build → clerkNative (no chunk splitting for RN)
  • Remove clerkHeadlessBrowser build (identical to regular browser)
  • Update Expo to import from @clerk/clerk-js instead of /headless
  • Delete headless source files and export directory

API changes:

  • Remove clerkJSVariant prop (was only used for 'headless')
  • Add new ui prop to skip loading @clerk/ui when set to false
  • Add shouldLoadClerkUi() helper in shared package
  • Update all SDK packages: Next.js, Astro, React, Vue, Nuxt, React Router, TanStack Start, Express

Breaking Changes

  • @clerk/clerk-js/headless import path removed
  • clerkJSVariant prop removed - use ui={false} instead
  • Expo/React Native users should import from @clerk/clerk-js directly

Test plan

  • Verify Expo builds correctly with the new import
  • Verify clerk.native.js is built without chunk splitting
  • Verify web apps work without the headless build variant
  • Verify ui={false} skips loading clerk-ui

Summary by CodeRabbit

  • New Features

    • Flexible ui option to enable/disable Clerk UI and specify UI URL/version
    • Environment variables to control UI behavior (disable, URL, version)
    • React Native "native" build variant
  • Deprecations

    • Headless variant removed
    • clerkJSVariant replaced by ui across frameworks
  • Tests

    • Added integration and unit tests validating UI-disabled and ui-config behaviors

✏️ Tip: You can customize this high-level summary in your review settings.

nikosdouvlis and others added 8 commits January 9, 2026 13:15
why:
when using clerkJSVariant='headless', applications only need control components and don't require the full UI bundle. loading the unnecessary @clerk/ui script adds overhead without providing value.

what changed:
- clerk-script.tsx: conditionally render clerk-ui script tag only when clerkJSVariant !== 'headless'
- integration template: read NEXT_PUBLIC_CLERK_JS_VARIANT env var and pass to ClerkProvider

users can now set NEXT_PUBLIC_CLERK_JS_VARIANT='headless' to skip loading the ~100KB @clerk/ui bundle when using only control components.
why:
when using clerkJSVariant='headless', applications only need control components and don't require the full UI bundle. loading the unnecessary @clerk/ui script adds overhead without providing value.

what changed:
- build-clerk-hotload-script: skip generating clerk-ui script tag when clerkJsVariant === 'headless'
- create-clerk-instance: getClerkUiEntryChunk returns undefined for headless variant to skip client-side hot-loading

users can now set clerkJSVariant='headless' to skip loading the ~100KB @clerk/ui bundle when using only control components.
why:
when using clerkJSVariant='headless', applications only need control components and don't require the full UI bundle. loading the unnecessary @clerk/ui script adds overhead without providing value.

what changed:
isomorphicClerk's getClerkUiEntryChunk method now returns undefined when clerkJSVariant === 'headless', skipping the loadClerkUiScript call entirely.

users can now set clerkJSVariant='headless' to skip loading the ~100KB @clerk/ui bundle when using only control components.
why:
when using clerkJSVariant='headless', applications only need control components and don't require the full UI bundle. loading the unnecessary @clerk/ui script adds overhead without providing value.

what changed:
clerkPlugin now checks if clerkJSVariant === 'headless' and skips the loadClerkUiScript call, resolving the clerkUiCtorPromise to undefined instead.

users can now set clerkJSVariant='headless' to skip loading the ~100KB @clerk/ui bundle when using only control components.
why:
verify that the headless variant correctly skips clerk-ui script injection across the full integration stack (env var → prop → script rendering).

what changed:
created headless-variant.test.ts that sets CLERK_JS_VARIANT='headless' and asserts clerk-ui script is absent while clerk-js script is present.
…ition

The headless variant is no longer needed now that UI components have been
moved to @clerk/ui. The browser builds are now identical in size.

Changes:
- Add `react-native` export condition in package.json for Expo/RN
- Rename `clerkHeadless` build to `clerkNative` (no chunk splitting)
- Remove `clerkHeadlessBrowser` build (identical to regular browser)
- Update Expo to import from `@clerk/clerk-js` instead of `/headless`
- Deprecate `clerkJSVariant` option (now ignored)
- Delete headless source files and export directory

BREAKING CHANGE: `@clerk/clerk-js/headless` import path removed.
Expo/React Native users should import from `@clerk/clerk-js` directly.
@changeset-bot
Copy link

changeset-bot bot commented Jan 21, 2026

🦋 Changeset detected

Latest commit: b85a057

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When 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

@vercel
Copy link

vercel bot commented Jan 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment Jan 21, 2026 3:29pm

Request Review

….com:clerk/javascript into jrad/remove-headless-variant
clerkJSVariant is still used to control whether @clerk/ui loads.
The option just no longer affects the clerk-js URL since the
separate headless build has been removed.
Replace the clerkJSVariant: 'headless' pattern with a cleaner ui prop API:
- ui: false - Skip loading @clerk/ui (for custom UIs)
- ui: { version?, url? } - Load UI with specific version/URL
- ui: undefined (default) - Load UI normally

Also adds shouldLoadClerkUi() helper function to shared package.

Breaking change: clerkJSVariant is removed in favor of ui prop.
@jacekradko jacekradko changed the title [DO NOT MERGE] feat(clerk-js): remove headless variant feat(clerk-js): remove headless variant, add ui prop Jan 21, 2026
@jacekradko jacekradko marked this pull request as ready for review January 21, 2026 15:09
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 21, 2026

📝 Walkthrough

Walkthrough

Replaces the clerkJSVariant option with a unified ui option (false | undefined | { url?: string; version?: string }) and introduces environment variables CLERK_UI_DISABLED, CLERK_UI_URL, and CLERK_UI_VERSION. Adds a shouldLoadClerkUi helper to gate UI loading. Removes headless build artifacts and adds a native (React Native) variant in clerk-js. Propagation and handling of the new UI config are updated across integrations (Astro, Next.js, Express, React, Vue, Nuxt, React Router, TanStack, Expo, Chrome extension, and related tooling).

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 38.89% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: removing the headless variant and introducing a ui prop. It is concise, specific, and accurately reflects the core refactoring objective.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In @.changeset/light-eagles-stay.md:
- Around line 1-2: The changeset is empty and must document the breaking changes
introduced: explicitly list affected package names and semver bumps (use major
bumps for breaking changes), describe the removals of clerkJSVariant and the old
import path `@clerk/clerk-js/headless`, and note the new ui prop and its migration
steps; update the changeset body to include a short summary, per-package entries
(e.g., package-name: major), and a brief migration note explaining how to
replace/remove clerkJSVariant, how to update import paths to the new
entrypoints, and how to adopt the new ui prop so consumers can upgrade safely.

In `@packages/expo/src/provider/singleton/createClerkInstance.ts`:
- Around line 1-2: The import path for FapiRequestInit/FapiResponse is invalid;
instead derive request/response types from the public Clerk API: create a
Handler type from Parameters<Clerk["__internal_onBeforeRequest"]>[0], then
extract Req = Parameters<Handler>[0] and use Req as the type for the requestInit
parameter, and similarly extract the response type from
Clerk["__internal_onAfterResponse"] to replace FapiResponse usage; update
function signatures in createClerkInstance.ts to use these derived types
(referencing Clerk, __internal_onBeforeRequest, __internal_onAfterResponse,
Handler, and Req).
🧹 Nitpick comments (1)
packages/react-router/src/client/ReactRouterClerkProvider.tsx (1)

102-109: Type safety concern with as any casts.

The as any casts on lines 102 and 108 bypass TypeScript's type checking. While this may be a workaround for complex generic constraints with TUi, it could mask type mismatches at runtime.

Consider whether these casts can be replaced with more precise type assertions or if the underlying types can be aligned to avoid the need for any.

Comment on lines +1 to +2
---
---
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Empty changeset file - missing package entries and description.

This changeset is empty but the PR introduces breaking changes (removal of clerkJSVariant, @clerk/clerk-js/headless import path, new ui prop). The changeset should document affected packages and their semver bumps.

🤖 Prompt for AI Agents
In @.changeset/light-eagles-stay.md around lines 1 - 2, The changeset is empty
and must document the breaking changes introduced: explicitly list affected
package names and semver bumps (use major bumps for breaking changes), describe
the removals of clerkJSVariant and the old import path `@clerk/clerk-js/headless`,
and note the new ui prop and its migration steps; update the changeset body to
include a short summary, per-package entries (e.g., package-name: major), and a
brief migration note explaining how to replace/remove clerkJSVariant, how to
update import paths to the new entrypoints, and how to adopt the new ui prop so
consumers can upgrade safely.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants