fix(clerk-expo): Export useLocalCredentials from a submodule

[panteliselef]

Description

Turns out that tree shaking does not work as expected, and when importing any hook into an app that does not have expo-local-authentication installed the app will break due to the missing dependency.

Before

import {  useLocalCredentials } from "@clerk/clerk-expo";

After

import { useLocalCredentials } from "@clerk/clerk-expo/local-credentials";

Updated Docs PR

Checklist

• npm test runs as expected.
• npm run 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:

[changeset-bot]

🦋 Changeset detected

Latest commit: ccc03ea

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

This PR includes changesets to release 1 package

Name	Type
@clerk/clerk-expo	Minor

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

[panteliselef]

!snapshot

[clerk-cookie]

Hey @panteliselef - the snapshot version command generated the following package versions:

Package	Version
@clerk/astro	1.1.0-snapshot.v5f4676a
@clerk/backend	1.7.0-snapshot.v5f4676a
@clerk/chrome-extension	1.2.2-snapshot.v5f4676a
@clerk/clerk-js	5.15.0-snapshot.v5f4676a
@clerk/elements	0.13.1-snapshot.v5f4676a
@clerk/clerk-expo	2.2.0-snapshot.v5f4676a
@clerk/express	0.0.29-snapshot.v5f4676a
@clerk/fastify	1.0.31-snapshot.v5f4676a
@clerk/localizations	2.5.9-snapshot.v5f4676a
@clerk/nextjs	5.3.2-snapshot.v5f4676a
@clerk/clerk-react	5.4.2-snapshot.v5f4676a
@clerk/remix	4.2.15-snapshot.v5f4676a
@clerk/clerk-sdk-node	5.0.28-snapshot.v5f4676a
@clerk/shared	2.5.2-snapshot.v5f4676a
@clerk/tanstack-start	0.2.2-snapshot.v5f4676a
@clerk/testing	1.2.11-snapshot.v5f4676a
@clerk/themes	2.1.21-snapshot.v5f4676a
@clerk/types	4.14.0-snapshot.v5f4676a

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/astro

npm i @clerk/[email protected] --save-exact

@clerk/backend

npm i @clerk/[email protected] --save-exact

@clerk/chrome-extension

npm i @clerk/[email protected] --save-exact

@clerk/clerk-js

npm i @clerk/[email protected] --save-exact

@clerk/elements

npm i @clerk/[email protected] --save-exact

@clerk/clerk-expo

npm i @clerk/[email protected] --save-exact

@clerk/express

npm i @clerk/[email protected] --save-exact

@clerk/fastify

npm i @clerk/[email protected] --save-exact

@clerk/localizations

npm i @clerk/[email protected] --save-exact

@clerk/nextjs

npm i @clerk/[email protected] --save-exact

@clerk/clerk-react

npm i @clerk/[email protected] --save-exact

@clerk/remix

npm i @clerk/[email protected] --save-exact

@clerk/clerk-sdk-node

npm i @clerk/[email protected] --save-exact

@clerk/shared

npm i @clerk/[email protected] --save-exact

@clerk/tanstack-start

npm i @clerk/[email protected] --save-exact

@clerk/testing

npm i @clerk/[email protected] --save-exact

@clerk/themes

npm i @clerk/[email protected] --save-exact

@clerk/types

npm i @clerk/[email protected] --save-exact

[anagstef]

❓ Is it ok to introduce a breaking change in a minor release?

[the-simian]

@anagstef , the breaking change was already released in 3663 as a minor release. This was presumably by accident, due to unexpected expectations of how tree shaking would work with metro. So, technically no, its not ok but its already done; but it wasn't intended to be breaking either.
This fix is the "correction" for that. If this 'fixes' the break, then the feature itself is indeed nonbreaking, so this would be a patch.

[the-simian]

To be clear: this PR would be a patch and the feature would be ( once corrected ) a minor since its not an intentional breaking change requiring a migration, and doesn't deprecate any surface of the api.

If you must install the additional dependency still, then it would be a major change, because that is intentional feature addition requiring a migration.

[panteliselef]

I'd have to agree that this is a breaking change, but it needs to be released as minor is order to fix a bug + introduces an API change.

[the-simian]

thanks for looking into this and responding so quickly to issue #3950, @panteliselef

[LauraBeatris]

@panteliselef Awesome work 🏅