Skip to content

fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 - #8924

Closed
tombryden wants to merge 1 commit into
clerk:mainfrom
tombryden:main
Closed

fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56#8924
tombryden wants to merge 1 commit into
clerk:mainfrom
tombryden:main

Conversation

@tombryden

@tombrydentombryden commented Jun 18, 2026•
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown

Description

Adds internal to import ClerkExpo to fix ios development build failure with expo sdk 56. Error: ambiguous implicit access level for import of 'ClerkExpo'; it is imported as 'internal' elsewhere

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • Chores
    • Internal module import optimization for improved code organization.

Note: This release contains no user-facing changes.

@changeset-bot

changeset-botBot commented Jun 18, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: fe16a3b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel

vercelBot commented Jun 18, 2026

Copy link
Copy Markdown

@tombryden is attempting to deploy a commit to the Clerk Production Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitaiBot commented Jun 18, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

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: 11c105b8-475f-466c-ab98-0ca29d63f458

📥 Commits

Reviewing files that changed from the base of the PR and between 88525a6 and fe16a3b.

📒 Files selected for processing (1)
  • packages/expo/ios/ClerkNativeBridge.swift
📝 Walkthrough

Walkthrough

In ClerkNativeBridge.swift, the import ClerkExpo declaration is changed to internal import ClerkExpo, restricting the module's visibility to the internal access level.

Changes

ClerkExpo Module Import Visibility

Layer / File(s)Summary
ClerkExpo internal import
packages/expo/ios/ClerkNativeBridge.swift		The import ClerkExpo statement is updated to internal import ClerkExpo to explicitly set the module's access level to internal.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A single word slipped into place,
internal now guards the space.
The bridge stands firm, the module neat,
One quiet change — a tidy feat.
🐇 hops away satisfied ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely describes the main change: adding an internal import for ClerkExpo to fix an iOS build issue with Expo SDK 56.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@wobsoriano

wobsoriano commented Jun 19, 2026•
edited
Loading

Copy link
Copy Markdown
Member

Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?

@tombryden

tombryden commented Jun 19, 2026•
edited
Loading

Copy link
Copy Markdown
Author

Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?

Hello! Can see #8927 addresses the issue.

This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks

@wobsoriano

wobsoriano commented Jun 19, 2026•
edited
Loading

Copy link
Copy Markdown
Member

Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?

Hello! Can see #8927 addresses the issue.

This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks

Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927

Feel free to test and let us know

@tombryden

tombryden commented Jun 19, 2026

Copy link
Copy Markdown
Author

Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?

Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks

Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927

Feel free to test and let us know

Great, gave it a quick test and all works as expected. Thanks! 👍

@mikepitre

mikepitre commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

@tombryden this is now available in 3.5.0

@tombryden

tombryden commented Jun 19, 2026

Copy link
Copy Markdown
Author

@tombryden this is now available in 3.5.0

Perfect, thanks for the fix 🥇

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tombryden@wobsoriano@mikepitre
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 by tombryden · Pull Request #8924 · clerk/javascript · GitHub
Skip to content
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 - #8924
Closed
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Closed
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56#8924
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Conversation
@tombryden
@tombrydentombryden
commented
Jun 18, 2026•

edited by coderabbitai
Bot
Loading

Copy link


Copy Markdown

Description
Adds internal to import ClerkExpo to fix ios development build failure with expo sdk 56. Error: ambiguous implicit access level for import of 'ClerkExpo'; it is imported as 'internal' elsewhere
Checklist
  • pnpm test runs as expected.
  • pnpm build runs as expected.
  •  (If applicable) JSDoc comments have been added or updated for any package exports
  •  (If applicable) Documentation has been updated
Type of change
  •  🐛 Bug fix
  •  🌟 New feature
  •  🔨 Breaking change
  •  📖 Refactoring / dependency upgrade / documentation
  •  other:
Summary by CodeRabbit
  • Chores
    • Internal module import optimization for improved code organization.
Note: This release contains no user-facing changes.
@changeset-bot
changeset-botBot
commented
Jun 18, 2026

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: fe16a3b
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@vercel
vercelBot
commented
Jun 18, 2026

Copy link


Copy Markdown

@tombryden is attempting to deploy a commit to the Clerk Production Team on Vercel.
A member of the Team first needs to authorize it.
@coderabbitai
coderabbitaiBot
commented
Jun 18, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Review Change Stack
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: 11c105b8-475f-466c-ab98-0ca29d63f458
📥 Commits
Reviewing files that changed from the base of the PR and between 88525a6 and fe16a3b.
📒 Files selected for processing (1)
  • packages/expo/ios/ClerkNativeBridge.swift
📝 Walkthrough
Walkthrough
In ClerkNativeBridge.swift, the import ClerkExpo declaration is changed to internal import ClerkExpo, restricting the module's visibility to the internal access level.
Changes
ClerkExpo Module Import Visibility
Layer / File(s)Summary
ClerkExpo internal import
packages/expo/ios/ClerkNativeBridge.swift		The import ClerkExpo statement is updated to internal import ClerkExpo to explicitly set the module's access level to internal.
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
A single word slipped into place,
internal now guards the space.

The bridge stands firm, the module neat,

One quiet change — a tidy feat.

🐇 hops away satisfied ✨
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely describes the main change: adding an internal import for ClerkExpo to fix an iOS build issue with Expo SDK 56.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
❤️ Share
Comment @coderabbitai help to get the list of available commands and usage tips.
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
@tombryden
tombryden
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.

This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
Great, gave it a quick test and all works as expected. Thanks! 👍
@mikepitre
mikepitre
commented
Jun 19, 2026

Copy link


Copy Markdown

Contributor
@tombryden this is now available in 3.5.0
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
@tombryden this is now available in 3.5.0
Perfect, thanks for the fix 🥇
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@tombryden@wobsoriano@mikepitre

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 by tombryden · Pull Request #8924 · clerk/javascript · GitHub
Skip to content
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 - #8924
Closed
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Closed
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56#8924
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Conversation
@tombryden
@tombrydentombryden
commented
Jun 18, 2026•

edited by coderabbitai
Bot
Loading

Copy link


Copy Markdown

Description
Adds internal to import ClerkExpo to fix ios development build failure with expo sdk 56. Error: ambiguous implicit access level for import of 'ClerkExpo'; it is imported as 'internal' elsewhere
Checklist
  • pnpm test runs as expected.
  • pnpm build runs as expected.
  •  (If applicable) JSDoc comments have been added or updated for any package exports
  •  (If applicable) Documentation has been updated
Type of change
  •  🐛 Bug fix
  •  🌟 New feature
  •  🔨 Breaking change
  •  📖 Refactoring / dependency upgrade / documentation
  •  other:
Summary by CodeRabbit
  • Chores
    • Internal module import optimization for improved code organization.
Note: This release contains no user-facing changes.
@changeset-bot
changeset-botBot
commented
Jun 18, 2026

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: fe16a3b
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@vercel
vercelBot
commented
Jun 18, 2026

Copy link


Copy Markdown

@tombryden is attempting to deploy a commit to the Clerk Production Team on Vercel.
A member of the Team first needs to authorize it.
@coderabbitai
coderabbitaiBot
commented
Jun 18, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Review Change Stack
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: 11c105b8-475f-466c-ab98-0ca29d63f458
📥 Commits
Reviewing files that changed from the base of the PR and between 88525a6 and fe16a3b.
📒 Files selected for processing (1)
  • packages/expo/ios/ClerkNativeBridge.swift
📝 Walkthrough
Walkthrough
In ClerkNativeBridge.swift, the import ClerkExpo declaration is changed to internal import ClerkExpo, restricting the module's visibility to the internal access level.
Changes
ClerkExpo Module Import Visibility
Layer / File(s)Summary
ClerkExpo internal import
packages/expo/ios/ClerkNativeBridge.swift		The import ClerkExpo statement is updated to internal import ClerkExpo to explicitly set the module's access level to internal.
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
A single word slipped into place,
internal now guards the space.

The bridge stands firm, the module neat,

One quiet change — a tidy feat.

🐇 hops away satisfied ✨
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely describes the main change: adding an internal import for ClerkExpo to fix an iOS build issue with Expo SDK 56.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
❤️ Share
Comment @coderabbitai help to get the list of available commands and usage tips.
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
@tombryden
tombryden
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.

This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
Great, gave it a quick test and all works as expected. Thanks! 👍
@mikepitre
mikepitre
commented
Jun 19, 2026

Copy link


Copy Markdown

Contributor
@tombryden this is now available in 3.5.0
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
@tombryden this is now available in 3.5.0
Perfect, thanks for the fix 🥇
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@tombryden@wobsoriano@mikepitre

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 by tombryden · Pull Request #8924 · clerk/javascript · GitHub
Skip to content
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 - #8924
Closed
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Closed
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56#8924
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Conversation
@tombryden
@tombrydentombryden
commented
Jun 18, 2026•

edited by coderabbitai
Bot
Loading

Copy link


Copy Markdown

Description
Adds internal to import ClerkExpo to fix ios development build failure with expo sdk 56. Error: ambiguous implicit access level for import of 'ClerkExpo'; it is imported as 'internal' elsewhere
Checklist
  • pnpm test runs as expected.
  • pnpm build runs as expected.
  •  (If applicable) JSDoc comments have been added or updated for any package exports
  •  (If applicable) Documentation has been updated
Type of change
  •  🐛 Bug fix
  •  🌟 New feature
  •  🔨 Breaking change
  •  📖 Refactoring / dependency upgrade / documentation
  •  other:
Summary by CodeRabbit
  • Chores
    • Internal module import optimization for improved code organization.
Note: This release contains no user-facing changes.
@changeset-bot
changeset-botBot
commented
Jun 18, 2026

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: fe16a3b
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@vercel
vercelBot
commented
Jun 18, 2026

Copy link


Copy Markdown

@tombryden is attempting to deploy a commit to the Clerk Production Team on Vercel.
A member of the Team first needs to authorize it.
@coderabbitai
coderabbitaiBot
commented
Jun 18, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Review Change Stack
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: 11c105b8-475f-466c-ab98-0ca29d63f458
📥 Commits
Reviewing files that changed from the base of the PR and between 88525a6 and fe16a3b.
📒 Files selected for processing (1)
  • packages/expo/ios/ClerkNativeBridge.swift
📝 Walkthrough
Walkthrough
In ClerkNativeBridge.swift, the import ClerkExpo declaration is changed to internal import ClerkExpo, restricting the module's visibility to the internal access level.
Changes
ClerkExpo Module Import Visibility
Layer / File(s)Summary
ClerkExpo internal import
packages/expo/ios/ClerkNativeBridge.swift		The import ClerkExpo statement is updated to internal import ClerkExpo to explicitly set the module's access level to internal.
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
A single word slipped into place,
internal now guards the space.

The bridge stands firm, the module neat,

One quiet change — a tidy feat.

🐇 hops away satisfied ✨
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely describes the main change: adding an internal import for ClerkExpo to fix an iOS build issue with Expo SDK 56.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
❤️ Share
Comment @coderabbitai help to get the list of available commands and usage tips.
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
@tombryden
tombryden
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.

This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
Great, gave it a quick test and all works as expected. Thanks! 👍
@mikepitre
mikepitre
commented
Jun 19, 2026

Copy link


Copy Markdown

Contributor
@tombryden this is now available in 3.5.0
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
@tombryden this is now available in 3.5.0
Perfect, thanks for the fix 🥇
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@tombryden@wobsoriano@mikepitre

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 by tombryden · Pull Request #8924 · clerk/javascript · GitHub
Skip to content
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 - #8924
Closed
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Closed
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56#8924
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Conversation
@tombryden
@tombrydentombryden
commented
Jun 18, 2026•

edited by coderabbitai
Bot
Loading

Copy link


Copy Markdown

Description
Adds internal to import ClerkExpo to fix ios development build failure with expo sdk 56. Error: ambiguous implicit access level for import of 'ClerkExpo'; it is imported as 'internal' elsewhere
Checklist
  • pnpm test runs as expected.
  • pnpm build runs as expected.
  •  (If applicable) JSDoc comments have been added or updated for any package exports
  •  (If applicable) Documentation has been updated
Type of change
  •  🐛 Bug fix
  •  🌟 New feature
  •  🔨 Breaking change
  •  📖 Refactoring / dependency upgrade / documentation
  •  other:
Summary by CodeRabbit
  • Chores
    • Internal module import optimization for improved code organization.
Note: This release contains no user-facing changes.
@changeset-bot
changeset-botBot
commented
Jun 18, 2026

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: fe16a3b
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@vercel
vercelBot
commented
Jun 18, 2026

Copy link


Copy Markdown

@tombryden is attempting to deploy a commit to the Clerk Production Team on Vercel.
A member of the Team first needs to authorize it.
@coderabbitai
coderabbitaiBot
commented
Jun 18, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Review Change Stack
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: 11c105b8-475f-466c-ab98-0ca29d63f458
📥 Commits
Reviewing files that changed from the base of the PR and between 88525a6 and fe16a3b.
📒 Files selected for processing (1)
  • packages/expo/ios/ClerkNativeBridge.swift
📝 Walkthrough
Walkthrough
In ClerkNativeBridge.swift, the import ClerkExpo declaration is changed to internal import ClerkExpo, restricting the module's visibility to the internal access level.
Changes
ClerkExpo Module Import Visibility
Layer / File(s)Summary
ClerkExpo internal import
packages/expo/ios/ClerkNativeBridge.swift		The import ClerkExpo statement is updated to internal import ClerkExpo to explicitly set the module's access level to internal.
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
A single word slipped into place,
internal now guards the space.

The bridge stands firm, the module neat,

One quiet change — a tidy feat.

🐇 hops away satisfied ✨
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely describes the main change: adding an internal import for ClerkExpo to fix an iOS build issue with Expo SDK 56.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
❤️ Share
Comment @coderabbitai help to get the list of available commands and usage tips.
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
@tombryden
tombryden
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.

This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
Great, gave it a quick test and all works as expected. Thanks! 👍
@mikepitre
mikepitre
commented
Jun 19, 2026

Copy link


Copy Markdown

Contributor
@tombryden this is now available in 3.5.0
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
@tombryden this is now available in 3.5.0
Perfect, thanks for the fix 🥇
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@tombryden@wobsoriano@mikepitre

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 by tombryden · Pull Request #8924 · clerk/javascript · GitHub
Skip to content
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 - #8924
Closed
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Closed
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56#8924
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Conversation
@tombryden
@tombrydentombryden
commented
Jun 18, 2026•

edited by coderabbitai
Bot
Loading

Copy link


Copy Markdown

Description
Adds internal to import ClerkExpo to fix ios development build failure with expo sdk 56. Error: ambiguous implicit access level for import of 'ClerkExpo'; it is imported as 'internal' elsewhere
Checklist
  • pnpm test runs as expected.
  • pnpm build runs as expected.
  •  (If applicable) JSDoc comments have been added or updated for any package exports
  •  (If applicable) Documentation has been updated
Type of change
  •  🐛 Bug fix
  •  🌟 New feature
  •  🔨 Breaking change
  •  📖 Refactoring / dependency upgrade / documentation
  •  other:
Summary by CodeRabbit
  • Chores
    • Internal module import optimization for improved code organization.
Note: This release contains no user-facing changes.
@changeset-bot
changeset-botBot
commented
Jun 18, 2026

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: fe16a3b
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@vercel
vercelBot
commented
Jun 18, 2026

Copy link


Copy Markdown

@tombryden is attempting to deploy a commit to the Clerk Production Team on Vercel.
A member of the Team first needs to authorize it.
@coderabbitai
coderabbitaiBot
commented
Jun 18, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Review Change Stack
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: 11c105b8-475f-466c-ab98-0ca29d63f458
📥 Commits
Reviewing files that changed from the base of the PR and between 88525a6 and fe16a3b.
📒 Files selected for processing (1)
  • packages/expo/ios/ClerkNativeBridge.swift
📝 Walkthrough
Walkthrough
In ClerkNativeBridge.swift, the import ClerkExpo declaration is changed to internal import ClerkExpo, restricting the module's visibility to the internal access level.
Changes
ClerkExpo Module Import Visibility
Layer / File(s)Summary
ClerkExpo internal import
packages/expo/ios/ClerkNativeBridge.swift		The import ClerkExpo statement is updated to internal import ClerkExpo to explicitly set the module's access level to internal.
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
A single word slipped into place,
internal now guards the space.

The bridge stands firm, the module neat,

One quiet change — a tidy feat.

🐇 hops away satisfied ✨
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely describes the main change: adding an internal import for ClerkExpo to fix an iOS build issue with Expo SDK 56.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
❤️ Share
Comment @coderabbitai help to get the list of available commands and usage tips.
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
@tombryden
tombryden
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.

This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
Great, gave it a quick test and all works as expected. Thanks! 👍
@mikepitre
mikepitre
commented
Jun 19, 2026

Copy link


Copy Markdown

Contributor
@tombryden this is now available in 3.5.0
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
@tombryden this is now available in 3.5.0
Perfect, thanks for the fix 🥇
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@tombryden@wobsoriano@mikepitre

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 by tombryden · Pull Request #8924 · clerk/javascript · GitHub
Skip to content
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 - #8924
Closed
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Closed
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56#8924
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Conversation
@tombryden
@tombrydentombryden
commented
Jun 18, 2026•

edited by coderabbitai
Bot
Loading

Copy link


Copy Markdown

Description
Adds internal to import ClerkExpo to fix ios development build failure with expo sdk 56. Error: ambiguous implicit access level for import of 'ClerkExpo'; it is imported as 'internal' elsewhere
Checklist
  • pnpm test runs as expected.
  • pnpm build runs as expected.
  •  (If applicable) JSDoc comments have been added or updated for any package exports
  •  (If applicable) Documentation has been updated
Type of change
  •  🐛 Bug fix
  •  🌟 New feature
  •  🔨 Breaking change
  •  📖 Refactoring / dependency upgrade / documentation
  •  other:
Summary by CodeRabbit
  • Chores
    • Internal module import optimization for improved code organization.
Note: This release contains no user-facing changes.
@changeset-bot
changeset-botBot
commented
Jun 18, 2026

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: fe16a3b
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@vercel
vercelBot
commented
Jun 18, 2026

Copy link


Copy Markdown

@tombryden is attempting to deploy a commit to the Clerk Production Team on Vercel.
A member of the Team first needs to authorize it.
@coderabbitai
coderabbitaiBot
commented
Jun 18, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Review Change Stack
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: 11c105b8-475f-466c-ab98-0ca29d63f458
📥 Commits
Reviewing files that changed from the base of the PR and between 88525a6 and fe16a3b.
📒 Files selected for processing (1)
  • packages/expo/ios/ClerkNativeBridge.swift
📝 Walkthrough
Walkthrough
In ClerkNativeBridge.swift, the import ClerkExpo declaration is changed to internal import ClerkExpo, restricting the module's visibility to the internal access level.
Changes
ClerkExpo Module Import Visibility
Layer / File(s)Summary
ClerkExpo internal import
packages/expo/ios/ClerkNativeBridge.swift		The import ClerkExpo statement is updated to internal import ClerkExpo to explicitly set the module's access level to internal.
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
A single word slipped into place,
internal now guards the space.

The bridge stands firm, the module neat,

One quiet change — a tidy feat.

🐇 hops away satisfied ✨
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely describes the main change: adding an internal import for ClerkExpo to fix an iOS build issue with Expo SDK 56.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
❤️ Share
Comment @coderabbitai help to get the list of available commands and usage tips.
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
@tombryden
tombryden
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.

This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
Great, gave it a quick test and all works as expected. Thanks! 👍
@mikepitre
mikepitre
commented
Jun 19, 2026

Copy link


Copy Markdown

Contributor
@tombryden this is now available in 3.5.0
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
@tombryden this is now available in 3.5.0
Perfect, thanks for the fix 🥇
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@tombryden@wobsoriano@mikepitre

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 by tombryden · Pull Request #8924 · clerk/javascript · GitHub
Skip to content
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56 - #8924
Closed
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Closed
fix(expo): internal import ClerkExpo so ios build works with Expo SDK 56#8924
tombryden wants to merge 1 commit into
clerk:mainfrom 
tombryden:main
Conversation
@tombryden
@tombrydentombryden
commented
Jun 18, 2026•

edited by coderabbitai
Bot
Loading

Copy link


Copy Markdown

Description
Adds internal to import ClerkExpo to fix ios development build failure with expo sdk 56. Error: ambiguous implicit access level for import of 'ClerkExpo'; it is imported as 'internal' elsewhere
Checklist
  • pnpm test runs as expected.
  • pnpm build runs as expected.
  •  (If applicable) JSDoc comments have been added or updated for any package exports
  •  (If applicable) Documentation has been updated
Type of change
  •  🐛 Bug fix
  •  🌟 New feature
  •  🔨 Breaking change
  •  📖 Refactoring / dependency upgrade / documentation
  •  other:
Summary by CodeRabbit
  • Chores
    • Internal module import optimization for improved code organization.
Note: This release contains no user-facing changes.
@changeset-bot
changeset-botBot
commented
Jun 18, 2026

Copy link


Copy Markdown

⚠️ No Changeset found
Latest commit: fe16a3b
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@vercel
vercelBot
commented
Jun 18, 2026

Copy link


Copy Markdown

@tombryden is attempting to deploy a commit to the Clerk Production Team on Vercel.
A member of the Team first needs to authorize it.
@coderabbitai
coderabbitaiBot
commented
Jun 18, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Review Change Stack
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: 11c105b8-475f-466c-ab98-0ca29d63f458
📥 Commits
Reviewing files that changed from the base of the PR and between 88525a6 and fe16a3b.
📒 Files selected for processing (1)
  • packages/expo/ios/ClerkNativeBridge.swift
📝 Walkthrough
Walkthrough
In ClerkNativeBridge.swift, the import ClerkExpo declaration is changed to internal import ClerkExpo, restricting the module's visibility to the internal access level.
Changes
ClerkExpo Module Import Visibility
Layer / File(s)Summary
ClerkExpo internal import
packages/expo/ios/ClerkNativeBridge.swift		The import ClerkExpo statement is updated to internal import ClerkExpo to explicitly set the module's access level to internal.
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
A single word slipped into place,
internal now guards the space.

The bridge stands firm, the module neat,

One quiet change — a tidy feat.

🐇 hops away satisfied ✨
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title clearly and concisely describes the main change: adding an internal import for ClerkExpo to fix an iOS build issue with Expo SDK 56.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
❤️ Share
Comment @coderabbitai help to get the list of available commands and usage tips.
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
@tombryden
tombryden
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
@wobsoriano
wobsoriano
commented
Jun 19, 2026•

edited

Loading

Copy link


Copy Markdown

Member
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.
This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
Hi! Can you try installing the latest version (3.4.7) and let me know if you're still seeing the error?
Hello! Can see #8927 addresses the issue.

This issue persisted when using 3.4.7 (I haven't patched with the above PR to test though - can only do this later today). Thanks
Here's the snapshot for the mentioned PR! npm i https://pkg.pr.new/@clerk/expo@8927
Feel free to test and let us know
Great, gave it a quick test and all works as expected. Thanks! 👍
@mikepitre
mikepitre
commented
Jun 19, 2026

Copy link


Copy Markdown

Contributor
@tombryden this is now available in 3.5.0
@tombryden
tombryden
commented
Jun 19, 2026

Copy link


Copy Markdown

Author
@tombryden this is now available in 3.5.0
Perfect, thanks for the fix 🥇
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@tombryden@wobsoriano@mikepitre