Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(clerk-js,clerk-react,chrome-extension): Build and consume packages without remotely hosted code #4551

Open
wants to merge 7 commits into
base: tm/ext-updates
Choose a base branch
from

Conversation

tmilewski
Copy link
Member

@tmilewski tmilewski commented Nov 13, 2024

Description

Removes remotely hosted code (RHC) for Browser Extensions in the following fashion:

clerk-js

Removes RHC via __BUILD_DISABLE_RHC__ and creates a new build under @clerk/clerk-js/no-rhc.

clerk-react

Introduces __BUILD_DISABLE_RHC__ into the clerk-react build which, if unset, defaults to false.

clerk-react is bundled into chrome-extension and in that process __BUILD_DISABLE_RHC__ is set to true which ultimately removes the RHC.

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:

@tmilewski tmilewski self-assigned this Nov 13, 2024
Copy link

changeset-bot bot commented Nov 13, 2024

🦋 Changeset detected

Latest commit: 9dab906

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

This PR includes changesets to release 7 packages
Name Type
@clerk/clerk-js Patch
@clerk/clerk-react Patch
@clerk/chrome-extension Major
@clerk/clerk-expo Patch
@clerk/nextjs Patch
@clerk/remix Patch
@clerk/tanstack-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

@tmilewski
Copy link
Member Author

!allow-major

@tmilewski
Copy link
Member Author

!snapshot

@clerk-cookie
Copy link
Collaborator

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

Package Version
@clerk/astro 1.4.11-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/backend 1.16.3-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/chrome-extension 2.0.0-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/clerk-js 5.34.0-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/elements 0.19.2-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/clerk-expo 2.3.3-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/expo-passkeys 0.0.4-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/express 1.3.13-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/fastify 2.0.15-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/localizations 3.6.3-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/nextjs 6.3.3-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/clerk-react 5.15.4-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/remix 4.2.51-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/clerk-sdk-node 5.0.64-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/shared 2.12.2-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/tanstack-start 0.4.27-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/testing 1.3.25-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/themes 2.1.44-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/types 4.33.0-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/ui 0.1.19-snapshot.v82229a2b5efc2b13289f104989759757ba878957
@clerk/vue 0.0.3-snapshot.v82229a2b5efc2b13289f104989759757ba878957

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/backend@1.16.3-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/chrome-extension

npm i @clerk/chrome-extension@2.0.0-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/clerk-js

npm i @clerk/clerk-js@5.34.0-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/elements

npm i @clerk/elements@0.19.2-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/clerk-expo

npm i @clerk/clerk-expo@2.3.3-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/expo-passkeys

npm i @clerk/expo-passkeys@0.0.4-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/express

npm i @clerk/express@1.3.13-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/fastify

npm i @clerk/fastify@2.0.15-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/localizations

npm i @clerk/localizations@3.6.3-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/nextjs

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

@clerk/clerk-react

npm i @clerk/clerk-react@5.15.4-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/remix

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

@clerk/clerk-sdk-node

npm i @clerk/clerk-sdk-node@5.0.64-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/shared

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

@clerk/tanstack-start

npm i @clerk/tanstack-start@0.4.27-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/testing

npm i @clerk/testing@1.3.25-snapshot.v82229a2b5efc2b13289f104989759757ba878957 --save-exact

@clerk/themes

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

@clerk/types

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

@clerk/ui

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

@clerk/vue

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

@tmilewski tmilewski marked this pull request as ready for review November 13, 2024 20:13
@tmilewski

This comment was marked as outdated.

@clerk-cookie

This comment was marked as outdated.

@@ -20,9 +23,3 @@ export * from './runtime';
export * from './url';
export * from './web3';
export * from './windowNavigate';
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removes a number of duplicated exports.

@tmilewski
Copy link
Member Author

!snapshot rhc-beta

@clerk-cookie
Copy link
Collaborator

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

Package Version
@clerk/astro 1.4.11-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/backend 1.16.3-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/chrome-extension 2.0.0-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/clerk-js 5.34.0-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/elements 0.19.2-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/clerk-expo 2.3.3-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/expo-passkeys 0.0.4-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/express 1.3.13-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/fastify 2.0.15-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/localizations 3.6.3-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/nextjs 6.3.3-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/clerk-react 5.15.4-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/remix 4.2.51-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/clerk-sdk-node 5.0.64-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/shared 2.12.2-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/tanstack-start 0.4.27-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/testing 1.3.25-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/themes 2.1.44-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/types 4.33.0-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/ui 0.1.19-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b
@clerk/vue 0.0.3-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b

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/backend@1.16.3-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/chrome-extension

npm i @clerk/chrome-extension@2.0.0-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/clerk-js

npm i @clerk/clerk-js@5.34.0-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/elements

npm i @clerk/elements@0.19.2-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/clerk-expo

npm i @clerk/clerk-expo@2.3.3-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/expo-passkeys

npm i @clerk/expo-passkeys@0.0.4-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/express

npm i @clerk/express@1.3.13-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/fastify

npm i @clerk/fastify@2.0.15-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/localizations

npm i @clerk/localizations@3.6.3-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/nextjs

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

@clerk/clerk-react

npm i @clerk/clerk-react@5.15.4-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/remix

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

@clerk/clerk-sdk-node

npm i @clerk/clerk-sdk-node@5.0.64-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/shared

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

@clerk/tanstack-start

npm i @clerk/tanstack-start@0.4.27-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/testing

npm i @clerk/testing@1.3.25-rhc-beta.v9dab906166f584529bf54880bc8daa910511316b --save-exact

@clerk/themes

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

@clerk/types

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

@clerk/ui

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

@clerk/vue

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

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.

3 participants