fix(react): gate remote Clerk UI loader behind __BUILD_DISABLE_RHC__ by jacekradko · Pull Request #8773 · clerk/javascript

jacekradko · 2026-06-08T01:36:13Z

@clerk/chrome-extension was getting rejected from the Chrome Web Store under MV3 because the bundle ships a remote script loader for @clerk/ui's ui.browser.js. The extension already bundles the UI via @clerk/ui/no-rhc so the loader never runs, but getClerkUIEntryChunk wasn't gating it behind __BUILD_DISABLE_RHC__ the way getClerkJsEntryChunk already does for clerk-js. Gating it lets the loader (and the createElement('script') it uses) tree-shake out of the extension build.

The search-for-rhc.mjs postbuild check should have caught this since it already greps for /npm/@clerk/ui, but its directory mode was missing -r, so grep never looked inside dist and always came back clean. That's fixed here too, which is why it goes in the same PR as the gate rather than on its own. I rebuilt react and the extension to confirm the RHC strings are gone and the check passes, then pointed the fixed checker at a planted URL to make sure it still fails when there's something to find.

Closes#8770

Summary by CodeRabbit

Release Notes

Bug Fixes
- Chrome extension builds now exclude remote UI script loaders.
- No-remote-code React builds now properly handle remote UI script loading.
Chores
- Improved recursive search in build configuration utilities.

getClerkUIEntryChunk did not gate loadClerkUIScript behind __BUILD_DISABLE_RHC__, so no-remote-code builds (e.g. @clerk/chrome-extension) shipped the @clerk/ui ui.browser.js CDN loader as unreachable code. The Chrome Web Store rejects this under Manifest V3 even though it never runs. Gate it like getClerkJsEntryChunk so it dead-code-eliminates out of the bundle. Also fix scripts/search-for-rhc.mjs directory mode, which was missing -r and so never descended into dist, silently passing. This made the chrome-extension postbuild and ui check:no-rhc guards no-ops. Closes#8770

changeset-bot · 2026-06-08T01:36:17Z

🦋 Changeset detected

Latest commit: fb6ad51

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

This PR includes changesets to release 6 packages

Name	Type
@clerk/chrome-extension	Patch
@clerk/react	Patch
@clerk/expo	Patch
@clerk/nextjs	Patch
@clerk/react-router	Patch
@clerk/tanstack-react-start	Patch

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 · 2026-06-08T01:36:20Z

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

Project	Deployment	Actions	Updated (UTC)
clerk-js-sandbox	Ready	Preview, Comment	Jun 9, 2026 3:20am

coderabbitai · 2026-06-08T01:36:45Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository YAML (base), Repository UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: cca227c4-2999-4f5f-89bc-dc4dce38e9be

📥 Commits

Reviewing files that changed from the base of the PR and between c8ff729 and fb6ad51.

📒 Files selected for processing (1)

packages/react/src/isomorphicClerk.ts

🚧 Files skipped from review as they are similar to previous changes (1)

packages/react/src/isomorphicClerk.ts

📝 Walkthrough

Walkthrough

This PR gates the Clerk UI remote script loader behind the __BUILD_DISABLE_RHC__ build flag in the React package, documents the resulting Chrome extension cleanup, and updates the RHC search utility to traverse directories recursively.

Changes

Remote Code Gate for Clerk UI

Layer / File(s)	Summary
Build-time gate for Clerk UI remote loader `packages/react/src/isomorphicClerk.ts`, `.changeset/react-gate-ui-loader-norhc.md`	`getClerkUIEntryChunk` is wrapped with a `!__BUILD_DISABLE_RHC__` guard so it returns `undefined` when the flag is enabled, preventing the remote UI CDN loader from being bundled; unchanged behavior when the flag is not set.
Chrome extension cleanup documentation and RHC search improvement `.changeset/chrome-extension-strip-ui-rhc-loader.md`, `scripts/search-for-rhc.mjs`	Adds a changeset documenting removal of the remote `@clerk/ui` loader from the Chrome extension bundle and updates `asyncSearchRHC` to use `grep -r` for directory searches so nested files are checked.

🎯 2 (Simple) | ⏱️ ~10 minutes

A gate to block the script so tight,
No remote code will bundle in flight!
Chrome's manifest won't fuss and fight,
Extensions pass inspection just right. 🐰✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: gating the remote Clerk UI loader behind a build-time flag to enable tree-shaking in builds that disable remote-hosted code.
Linked Issues check	✅ Passed	The PR fully addresses issue `#8770` by implementing a build-time gate for the remote UI script loader, enabling tree-shaking for no-RHC builds and fixing the postbuild verification script.
Out of Scope Changes check	✅ Passed	All changes are directly related to the stated objective: gating the remote UI loader and fixing the recursive directory search in the verification script.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

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

pkg-pr-new · 2026-06-08T01:38:49Z

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@8773

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@8773

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@8773

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@8773

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@8773

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@8773

@clerk/express

npm i https://pkg.pr.new/@clerk/express@8773

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@8773

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@8773

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@8773

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@8773

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@8773

@clerk/react

npm i https://pkg.pr.new/@clerk/react@8773

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@8773

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@8773

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@8773

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@8773

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@8773

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@8773

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@8773

commit: fb6ad51

github-actions · 2026-06-08T01:39:13Z

Break Check: no API changes detected across the tracked packages.

_{Last ran on c8ff729. Pushes that change no tracked declarations (no API surface change vs. base) are skipped and don't update this comment.}

github-actions · 2026-06-09T03:25:47Z

API Changes Report

Generated by Break Check on 2026-06-09T03:25:44.791Z

Summary

Metric	Count
Packages analyzed	19
Packages with changes	0
🔴 Breaking changes	0
🟡 Non-breaking changes	0
🟢 Additions	0

Note
Break Check could not snapshot 3 subpaths; the diff below excludes them.
@clerk/astro ./env: Internal Error: Unable to determine module for: /home/runner/_work/javascript/javascript/packages/astro/env.d.ts You have encountered a software defect. Please consider reporting the issue to the maintainers of this application.
@clerk/shared ./cookie: Internal Error: Unable to follow symbol for "Cookies" You have encountered a software defect. Please consider reporting the issue to the maintainers of this application.
@clerk/testing ./cypress: Symbol not found for identifier: Cypress

No API Changes Detected

All packages have stable APIs with no detected changes.

Report generated by Break Check

_{Last ran on fb6ad51.}

github-actionsBot added the react label Jun 8, 2026

vercelBotdeployed to Preview June 8, 2026 01:36 View deployment

jacekradko requested a review from tmilewski June 9, 2026 03:19

Merge branch 'main' into jacek/chrome-ext-ui-rhc-gate
fb6ad51

vercelBotdeployed to Preview June 9, 2026 03:20 View deployment

tmilewski approved these changes Jun 9, 2026
View reviewed changes

wobsoriano approved these changes Jun 9, 2026
View reviewed changes

jacekradko merged commit 0c854c3 into mainJun 9, 2026
48 checks passed

jacekradko deleted the jacek/chrome-ext-ui-rhc-gate branch June 9, 2026 16:12

clerk-cookie mentioned this pull request Jun 9, 2026
ci(repo): Version packages #8735
Merged

coderabbitaiBot mentioned this pull request Jun 18, 2026
fix(react): Use globalThis instead of bare global in the browser Clerk loader #8909
Merged

coderabbitaiBot mentioned this pull request Aug 21, 2026
chore: merge deploy/auth-fix into develop deangilmoreremix/remix-new-editor#11
Closed

Navigation Menu

Uh oh!

fix(react): gate remote Clerk UI loader behind __BUILD_DISABLE_RHC__ - #8773

fix(react): gate remote Clerk UI loader behind __BUILD_DISABLE_RHC__#8773
jacekradko merged 2 commits into
mainfrom
jacek/chrome-ext-ui-rhc-gate

jacekradko commented Jun 8, 2026•
edited by coderabbitai Bot
Loading

Uh oh!

changeset-botBot commented Jun 8, 2026•
edited
Loading

Uh oh!

vercelBot commented Jun 8, 2026•
edited
Loading

Uh oh!

coderabbitaiBot commented Jun 8, 2026•
edited
Loading

Walkthrough

Changes

Uh oh!

pkg-pr-newBot commented Jun 8, 2026•
edited
Loading

Uh oh!

github-actionsBot commented Jun 8, 2026

Uh oh!

github-actionsBot commented Jun 9, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

jacekradko commented Jun 8, 2026• edited by coderabbitai BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Release Notes

Uh oh!

changeset-botBot commented Jun 8, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

vercelBot commented Jun 8, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitaiBot commented Jun 8, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

pkg-pr-newBot commented Jun 8, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actionsBot commented Jun 8, 2026

Uh oh!

github-actionsBot commented Jun 9, 2026

API Changes Report

Summary

No API Changes Detected

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

jacekradko commented Jun 8, 2026•
edited by coderabbitai Bot
Loading

changeset-botBot commented Jun 8, 2026•
edited
Loading

vercelBot commented Jun 8, 2026•
edited
Loading

coderabbitaiBot commented Jun 8, 2026•
edited
Loading

pkg-pr-newBot commented Jun 8, 2026•
edited
Loading