feat(expo): Move Google Sign to a separate package#9015
Merged
Conversation
🦋 Changeset detectedLatest commit: 2dc5feb The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 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 |
Contributor
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughGoogle Sign-In is extracted into ChangesGoogle Sign-In package extraction
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant ExpoApp as Expo app
participant ExpoPlugin as withClerkExpoGoogleSignIn
participant InfoPlist as iOS Info.plist
participant GoogleHook as useSignInWithGoogle
ExpoApp->>ExpoPlugin: apply config plugin
ExpoPlugin->>InfoPlist: add configured Google URL scheme
ExpoApp->>GoogleHook: import Google Sign-In hook
GoogleHook-->>ExpoApp: expose authentication flow
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
wobsoriano
force-pushed
the
rob/mobile-560
branch
from
June 30, 2026 21:29
771cade to
91717af
Compare
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Member
Author
|
!snapshot |
@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: |
wobsoriano
marked this pull request as ready for review
July 1, 2026 19:09
Contributor
API Changes Report
Summary
No API Changes DetectedAll packages have stable APIs with no detected changes. Report generated by Break Check Last ran on |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (4)
packages/expo-google-signin/app.plugin.d.ts (1)
3-5: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueConsider a JSDoc comment for this public config-plugin export.
withClerkExpoGoogleSignInis the public entrypoint developers reference in theirapp.json/app.config.jsplugins array. Since Clerk Docs generates reference pages from JSDoc inpackages/**, a short JSDoc block describing usage/behavior (e.g., that it wires the iOS URL scheme) would help keep generated docs accurate if this symbol is surfaced.As per path instructions, "Pay special attention to changes that add, remove, or modify JSDoc comments on public APIs... that may appear in generated reference docs."
🤖 Prompt for 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. In `@packages/expo-google-signin/app.plugin.d.ts` around lines 3 - 5, Add a short JSDoc block to the public config-plugin export in withClerkExpoGoogleSignIn so generated reference docs have usage/behavior context. Update the declaration around the withClerkExpoGoogleSignIn symbol to describe what the plugin does (for example, wiring the iOS URL scheme) and keep the export signature unchanged.Source: Path instructions
packages/expo-google-signin/app.plugin.js (1)
9-11: 🩺 Stability & Availability | 🔵 Trivial | ⚡ Quick winSilent no-op when the iOS URL scheme env var is missing.
If
EXPO_PUBLIC_CLERK_GOOGLE_IOS_URL_SCHEMEisn't set, the plugin silently returns the config unmodified. For apps intending to use native Google Sign-In on iOS, this fails silently at build time rather than surfacing an actionable error, making the eventual runtime sign-in failure harder to diagnose.As per coding guidelines, "Provide meaningful error messages to developers" and "Include error recovery suggestions where applicable."
💡 Proposed fix to warn when the scheme is missing
if (!iosUrlScheme) { + console.warn( + '[`@clerk/expo-google-signin`] EXPO_PUBLIC_CLERK_GOOGLE_IOS_URL_SCHEME is not set. ' + + 'Native Google Sign-In will not work on iOS until this is configured.', + ); return config; }🤖 Prompt for 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. In `@packages/expo-google-signin/app.plugin.js` around lines 9 - 11, The iOS URL scheme check in the plugin currently returns the config unchanged when EXPO_PUBLIC_CLERK_GOOGLE_IOS_URL_SCHEME is missing, which hides a misconfiguration. Update the app.plugin.js logic around the iosUrlScheme guard to surface an actionable build-time error or warning with a clear message and recovery hint instead of silently no-oping, so developers can fix the missing native Google Sign-In setup before runtime. Reference the existing plugin flow and the iosUrlScheme handling to keep the behavior explicit and easy to locate.Source: Coding guidelines
packages/expo-google-signin/vitest.setup.mts (1)
18-18: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueEmpty
beforeAllis a no-op.
beforeAll(() => {});does nothing; can be removed unless intended as a placeholder for future setup.🧹 Proposed removal
- -beforeAll(() => {});🤖 Prompt for 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. In `@packages/expo-google-signin/vitest.setup.mts` at line 18, The empty beforeAll in vitest.setup.mts is a no-op and should be removed. Delete the beforeAll(() => {}) call unless you plan to add real setup logic there later, keeping the file clean and avoiding unnecessary test hooks.packages/expo-google-signin/src/useSignInWithGoogle.ts (1)
24-25: 📐 Maintainability & Code Quality | 🔵 TrivialDocs-team review may be needed for the removed migration note.
This public JSDoc paragraph was deleted, so please confirm the generated reference still points users to the standalone package/plugin migration steps somewhere else. As per path instructions, public/reference-facing JSDoc changes should be checked for generated docs drift.
🤖 Prompt for 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. In `@packages/expo-google-signin/src/useSignInWithGoogle.ts` around lines 24 - 25, The public JSDoc in useSignInWithGoogle was shortened by removing a migration note, so verify that the standalone package/plugin migration guidance is still available in the generated reference. Update the surrounding docs comment in useSignInWithGoogle (and any related public docs entry points) so users are still directed to the correct migration steps, or restore an equivalent reference elsewhere in the reference-facing JSDoc.Source: Path instructions
🤖 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 `@packages/expo-google-signin/package.json`:
- Around line 51-63: The devDependency versions in the package.json for
expo-google-signin are misaligned with the Expo SDK being targeted, specifically
the react-native version is too new for SDK 54. Update the expo-related
devDependencies in this package to match the Expo 54-compatible versions, and
keep the peerDependencies range broad enough to support the intended SDKs; use
the existing package.json dependency block and peerDependencies block as the
place to align the versions.
In `@packages/expo-google-signin/src/useSignInWithGoogle.types.ts`:
- Around line 3-7: The StartGoogleAuthenticationFlowParams type currently
redeclares SignUpUnsafeMetadata as a plain Record, which bypasses the shared
ambient contract and any app-specific augmentation. Update
useSignInWithGoogle.types so StartGoogleAuthenticationFlowParams references the
shared SignUpUnsafeMetadata symbol directly instead of defining a local alias,
keeping the unsafeMetadata field aligned with the common contract.
---
Nitpick comments:
In `@packages/expo-google-signin/app.plugin.d.ts`:
- Around line 3-5: Add a short JSDoc block to the public config-plugin export in
withClerkExpoGoogleSignIn so generated reference docs have usage/behavior
context. Update the declaration around the withClerkExpoGoogleSignIn symbol to
describe what the plugin does (for example, wiring the iOS URL scheme) and keep
the export signature unchanged.
In `@packages/expo-google-signin/app.plugin.js`:
- Around line 9-11: The iOS URL scheme check in the plugin currently returns the
config unchanged when EXPO_PUBLIC_CLERK_GOOGLE_IOS_URL_SCHEME is missing, which
hides a misconfiguration. Update the app.plugin.js logic around the iosUrlScheme
guard to surface an actionable build-time error or warning with a clear message
and recovery hint instead of silently no-oping, so developers can fix the
missing native Google Sign-In setup before runtime. Reference the existing
plugin flow and the iosUrlScheme handling to keep the behavior explicit and easy
to locate.
In `@packages/expo-google-signin/src/useSignInWithGoogle.ts`:
- Around line 24-25: The public JSDoc in useSignInWithGoogle was shortened by
removing a migration note, so verify that the standalone package/plugin
migration guidance is still available in the generated reference. Update the
surrounding docs comment in useSignInWithGoogle (and any related public docs
entry points) so users are still directed to the correct migration steps, or
restore an equivalent reference elsewhere in the reference-facing JSDoc.
In `@packages/expo-google-signin/vitest.setup.mts`:
- Line 18: The empty beforeAll in vitest.setup.mts is a no-op and should be
removed. Delete the beforeAll(() => {}) call unless you plan to add real setup
logic there later, keeping the file clean and avoiding unnecessary test hooks.
🪄 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), Repository UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: 2eb2524a-7c13-4f0f-9303-ae45c2177857
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (35)
.changeset/expo-google-signin-package.mdpackages/expo-google-signin/android/build.gradlepackages/expo-google-signin/android/src/main/AndroidManifest.xmlpackages/expo-google-signin/android/src/main/java/expo/modules/clerk/googlesignin/ClerkGoogleSignInModule.ktpackages/expo-google-signin/app.plugin.d.tspackages/expo-google-signin/app.plugin.jspackages/expo-google-signin/expo-module.config.jsonpackages/expo-google-signin/ios/ClerkGoogleSignIn.podspecpackages/expo-google-signin/ios/ClerkGoogleSignInModule.swiftpackages/expo-google-signin/package.jsonpackages/expo-google-signin/src/__tests__/useSignInWithGoogle.test.tspackages/expo-google-signin/src/global.d.tspackages/expo-google-signin/src/google-one-tap/ClerkGoogleOneTapSignIn.tspackages/expo-google-signin/src/google-one-tap/index.tspackages/expo-google-signin/src/google-one-tap/types.tspackages/expo-google-signin/src/index.tspackages/expo-google-signin/src/specs/NativeClerkGoogleSignIn.android.tspackages/expo-google-signin/src/specs/NativeClerkGoogleSignIn.tspackages/expo-google-signin/src/useSignInWithGoogle.android.tspackages/expo-google-signin/src/useSignInWithGoogle.ios.tspackages/expo-google-signin/src/useSignInWithGoogle.shared.tspackages/expo-google-signin/src/useSignInWithGoogle.tspackages/expo-google-signin/src/useSignInWithGoogle.types.tspackages/expo-google-signin/tsconfig.declarations.jsonpackages/expo-google-signin/tsconfig.jsonpackages/expo-google-signin/tsup.config.tspackages/expo-google-signin/vitest.config.mtspackages/expo-google-signin/vitest.setup.mtspackages/expo/android/build.gradlepackages/expo/app.plugin.jspackages/expo/expo-module.config.jsonpackages/expo/package.jsonpackages/expo/src/google/index.tspackages/expo/src/plugin/withClerkExpo.tspackages/expo/src/types/index.ts
💤 Files with no reviewable changes (3)
- packages/expo/android/build.gradle
- packages/expo/src/types/index.ts
- packages/expo/app.plugin.js
mikepitre
reviewed
Jul 1, 2026
mikepitre
reviewed
Jul 1, 2026
mikepitre
reviewed
Jul 1, 2026
Member
Author
|
!snapshot |
Member
Author
|
!snapshot |
Contributor
|
Hey @wobsoriano - the snapshot version command generated the following package versions:
Tip: Use the snippet copy button below to quickly install the required packages. npm i @clerk/astro@3.4.20-snapshot.v20260721140731 --save-exact
npm i @clerk/backend@3.12.0-snapshot.v20260721140731 --save-exact
npm i @clerk/chrome-extension@3.1.55-snapshot.v20260721140731 --save-exact
npm i @clerk/clerk-js@6.25.6-snapshot.v20260721140731 --save-exact
npm i @clerk/electron@0.0.17-snapshot.v20260721140731 --save-exact
npm i @clerk/electron-passkeys@0.0.4-snapshot.v20260721140731 --save-exact
npm i @clerk/eslint-plugin@0.2.1-snapshot.v20260721140731 --save-exact
npm i @clerk/expo@4.0.0-snapshot.v20260721140731 --save-exact
npm i @clerk/expo-google-signin@1.0.0-snapshot.v20260721140731 --save-exact
npm i @clerk/expo-passkeys@2.0.0-snapshot.v20260721140731 --save-exact
npm i @clerk/express@2.1.44-snapshot.v20260721140731 --save-exact
npm i @clerk/fastify@3.1.54-snapshot.v20260721140731 --save-exact
npm i @clerk/headless@0.0.14-snapshot.v20260721140731 --save-exact
npm i @clerk/hono@0.1.54-snapshot.v20260721140731 --save-exact
npm i @clerk/localizations@4.13.6-snapshot.v20260721140731 --save-exact
npm i @clerk/msw@0.0.50-snapshot.v20260721140731 --save-exact
npm i @clerk/nextjs@7.5.21-snapshot.v20260721140731 --save-exact
npm i @clerk/nuxt@2.6.20-snapshot.v20260721140731 --save-exact
npm i @clerk/react@6.12.6-snapshot.v20260721140731 --save-exact
npm i @clerk/react-router@3.5.13-snapshot.v20260721140731 --save-exact
npm i @clerk/shared@4.25.6-snapshot.v20260721140731 --save-exact
npm i @clerk/swingset@0.0.21-snapshot.v20260721140731 --save-exact
npm i @clerk/tanstack-react-start@1.4.21-snapshot.v20260721140731 --save-exact
npm i @clerk/testing@2.2.11-snapshot.v20260721140731 --save-exact
npm i @clerk/ui@1.25.6-snapshot.v20260721140731 --save-exact
npm i @clerk/upgrade@2.0.5-snapshot.v20260721140731 --save-exact
npm i @clerk/vue@2.4.18-snapshot.v20260721140731 --save-exact |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Moves the native Google Sign-In implementation out of
@clerk/expoand into a new optional package,@clerk/expo-google-signin.The Expo SDK currently includes the native Google Sign-In module even for apps that never call
useSignInWithGoogle(). That means every Expo app using@clerk/expocan pick up the extra native iOS/Android Google Sign-In dependencies during prebuild/autolinking. This split keeps the native Google module opt-in while preserving the existing JS-facing API through a re-export.This is intended for the next major version because apps using native Google Sign-In will need to install the new package and add its Expo config plugin.
Migration
Apps that use native Google Sign-In now need to install the new package:
and add its config plugin:
{ "expo": { "plugins": [ "@clerk/expo", "@clerk/expo-google-signin" ] } }The existing import path can continue to work:
but the native module is no longer included unless
@clerk/expo-google-signinis installed and prebuilt into the app.Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change
Summary by CodeRabbit
@clerk/expo-google-signinpackage with a matching Expo config plugin.useSignInWithGoogleand related flow types via the new package.@clerk/expo; install@clerk/expo-google-signin, add its config plugin, and rebuild.