Skip to content

Conversation

@chriscanin
Copy link
Member

@chriscanin chriscanin commented Feb 2, 2026

Description

This PR is to create a better export for the apple sign in and google sign in hook, so that the peer dependencies that are required by these hooks will only be required when using them. I have also updated the docs to reflect the need for these peer dependencies to be installed when using these hooks.

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

  • New Features

    • Google and Apple sign-in are available via dedicated package entry points, added to the published distribution.
  • Breaking Changes

    • The native sign-in hooks have been moved to their own entry points; existing imports from the main package must be updated to the new paths.

@changeset-bot
Copy link

changeset-bot bot commented Feb 2, 2026

🦋 Changeset detected

Latest commit: 91f48b0

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

This PR includes changesets to release 1 package
Name Type
@clerk/expo Major

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 Feb 2, 2026

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

Project Deployment Actions Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment Feb 3, 2026 0:36am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 2, 2026

📝 Walkthrough

Walkthrough

The change updates the Expo package public API by adding two new export entry points, ./google and ./apple, and including their dist folders in the published files. The Google and Apple sign-in hooks and related types are re-exported from their respective new index modules, and those two hooks are removed from the main hooks index.

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions updating useSignInWithApple import but the changeset shows this is a breaking change affecting both useSignInWithApple AND useSignInWithGoogle. The title only highlights the Apple hook, not the equally important Google hook relocation or the broader objective of optimizing dependencies. Update the title to reflect both hooks, e.g., 'fix(expo): Move sign-in hooks to dedicated entry points to optimize dependencies' or 'fix(expo): Relocate useSignInWithApple and useSignInWithGoogle to @clerk/expo/[apple|google]'.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ 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.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 2, 2026

Open in StackBlitz

@clerk/agent-toolkit

npm i https://pkg.pr.new/@clerk/agent-toolkit@7737

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7737

@clerk/expo

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

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 1702db0

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.

2 participants