Merge branch 'main' into chore/typecheck

clerk · Jan 7, 2025 · 74e01bd · 74e01bd
2 parents 214780d + e21d544
commit 74e01bd
Show file tree

Hide file tree

Showing 241 changed files with 4,129 additions and 1,704 deletions.
diff --git a/.changeset/cold-tomatoes-float.md b/.changeset/cold-tomatoes-float.md
diff --git a/.changeset/rotten-berries-jog.md b/.changeset/rotten-berries-jog.md
@@ -0,0 +1,5 @@
+---
+'@clerk/clerk-js': patch
+---
+
+Fix username field error message rendering within sign-up continue flow.
diff --git a/.changeset/short-news-enjoy.md b/.changeset/short-news-enjoy.md
@@ -0,0 +1,5 @@
+---
+'@clerk/clerk-js': patch
+---
+
+Improve accessibility of the Keyless prompt.
diff --git a/.changeset/tame-wolves-sit.md b/.changeset/tame-wolves-sit.md
@@ -0,0 +1,5 @@
+---
+"@clerk/vue": patch
+---
+
+Fix an issue where `getToken()` from `useAuth()` composable returns an empty value
diff --git a/.changeset/breezy-foxes-lay.md → .changeset/tiny-cups-change.md b/.changeset/breezy-foxes-lay.md → .changeset/tiny-cups-change.md
diff --git a/.changeset/wild-planes-swim.md b/.changeset/wild-planes-swim.md
diff --git a/integration/constants.ts b/integration/constants.ts
@@ -56,10 +56,6 @@ export const constants = {
    * The version of the dependency to use, controlled programmatically.
    */
   E2E_REACT_DOM_VERSION: process.env.E2E_REACT_DOM_VERSION,
-  /**
-   * The version of the dependency to use, controlled programmatically.
-   */
-  E2E_REMIX_VERSION: process.env.E2E_REMIX_VERSION,
   /**
    * The version of the dependency to use, controlled programmatically.
    */

diff --git a/integration/presets/astro.ts b/integration/presets/astro.ts
@@ -1,5 +1,6 @@
 import { applicationConfig } from '../models/applicationConfig';
 import { templates } from '../templates';
+import { linkPackage } from './utils';
 
 const astroNode = applicationConfig()
   .setName('astro-node')
@@ -9,9 +10,9 @@ const astroNode = applicationConfig()
   .addScript('dev', 'pnpm dev')
   .addScript('build', 'pnpm build')
   .addScript('serve', 'pnpm preview')
-  .addDependency('@clerk/astro', '*')
-  .addDependency('@clerk/types', '*')
-  .addDependency('@clerk/localizations', '*');
+  .addDependency('@clerk/astro', linkPackage('astro'))
+  .addDependency('@clerk/types', linkPackage('types'))
+  .addDependency('@clerk/localizations', linkPackage('localizations'));
 
 const astroStatic = astroNode.clone().setName('astro-hybrid').useTemplate(templates['astro-hybrid']);
 

diff --git a/integration/presets/elements.ts b/integration/presets/elements.ts
@@ -1,6 +1,7 @@
 import { constants } from '../constants';
 import { applicationConfig } from '../models/applicationConfig.js';
 import { templates } from '../templates/index.js';
+import { linkPackage } from './utils';
 
 const nextAppRouter = applicationConfig()
   .setName('elements-next')
@@ -13,8 +14,8 @@ const nextAppRouter = applicationConfig()
   .addDependency('next', constants.E2E_NEXTJS_VERSION)
   .addDependency('react', constants.E2E_REACT_VERSION)
   .addDependency('react-dom', constants.E2E_REACT_DOM_VERSION)
-  .addDependency('@clerk/nextjs', constants.E2E_CLERK_VERSION || '*')
-  .addDependency('@clerk/elements', constants.E2E_CLERK_VERSION || '*');
+  .addDependency('@clerk/nextjs', constants.E2E_CLERK_VERSION || linkPackage('nextjs'))
+  .addDependency('@clerk/elements', constants.E2E_CLERK_VERSION || linkPackage('elements'));
 
 export const elements = {
   nextAppRouter,

diff --git a/integration/presets/expo.ts b/integration/presets/expo.ts
@@ -1,5 +1,6 @@
 import { applicationConfig } from '../models/applicationConfig';
 import { templates } from '../templates';
+import { linkPackage } from './utils';
 
 const expoWeb = applicationConfig()
   .setName('expo-web')
@@ -9,7 +10,7 @@ const expoWeb = applicationConfig()
   .addScript('dev', 'pnpm dev')
   .addScript('build', 'pnpm build')
   .addScript('serve', 'pnpm start')
-  .addDependency('@clerk/clerk-expo', '*');
+  .addDependency('@clerk/clerk-expo', linkPackage('expo'));
 
 export const expo = {
   expoWeb,

diff --git a/integration/presets/express.ts b/integration/presets/express.ts
@@ -1,6 +1,7 @@
 import { constants } from '../constants';
 import { applicationConfig } from '../models/applicationConfig';
 import { templates } from '../templates';
+import { linkPackage } from './utils';
 
 const vite = applicationConfig()
   .setName('express-vite')
@@ -10,7 +11,7 @@ const vite = applicationConfig()
   .addScript('dev', 'pnpm dev')
   .addScript('build', 'pnpm build')
   .addScript('serve', 'pnpm start')
-  .addDependency('@clerk/clerk-sdk-node', constants.E2E_CLERK_VERSION || '*');
+  .addDependency('@clerk/clerk-sdk-node', constants.E2E_CLERK_VERSION || linkPackage('sdk-node'));
 
 export const express = {
   vite,

diff --git a/integration/presets/index.ts b/integration/presets/index.ts
@@ -8,7 +8,6 @@ import { next } from './next';
 import { nuxt } from './nuxt';
 import { react } from './react';
 import { reactRouter } from './react-router';
-import { remix } from './remix';
 import { tanstack } from './tanstack';
 
 export const appConfigs = {
@@ -17,7 +16,6 @@ export const appConfigs = {
   longRunningApps: createLongRunningApps(),
   next,
   react,
-  remix,
   elements,
   expo,
   astro,

diff --git a/integration/presets/longRunningApps.ts b/integration/presets/longRunningApps.ts
@@ -9,7 +9,6 @@ import { next } from './next';
 import { nuxt } from './nuxt';
 import { react } from './react';
 import { reactRouter } from './react-router';
-import { remix } from './remix';
 import { tanstack } from './tanstack';
 import { vue } from './vue';
 
@@ -24,7 +23,6 @@ export const createLongRunningApps = () => {
     { id: 'react.vite.withEmailCodes', config: react.vite, env: envs.withEmailCodes },
     { id: 'react.vite.withEmailCodes_persist_client', config: react.vite, env: envs.withEmailCodes_destroy_client },
     { id: 'react.vite.withEmailLinks', config: react.vite, env: envs.withEmailLinks },
-    { id: 'remix.node.withEmailCodes', config: remix.remixNode, env: envs.withEmailCodes },
     { id: 'next.appRouter.withEmailCodes', config: next.appRouter, env: envs.withEmailCodes },
     {
       id: 'next.appRouter.withEmailCodes_persist_client',

diff --git a/integration/presets/next.ts b/integration/presets/next.ts
@@ -1,6 +1,7 @@
 import { constants } from '../constants';
 import { applicationConfig } from '../models/applicationConfig.js';
 import { templates } from '../templates/index.js';
+import { linkPackage } from './utils';
 
 const appRouter = applicationConfig()
   .setName('next-app-router')
@@ -13,9 +14,9 @@ const appRouter = applicationConfig()
   .addDependency('next', constants.E2E_NEXTJS_VERSION)
   .addDependency('react', constants.E2E_REACT_VERSION)
   .addDependency('react-dom', constants.E2E_REACT_DOM_VERSION)
-  .addDependency('@clerk/nextjs', constants.E2E_CLERK_VERSION || '*')
-  .addDependency('@clerk/shared', '*')
-  .addDependency('@clerk/types', '*');
+  .addDependency('@clerk/nextjs', constants.E2E_CLERK_VERSION || linkPackage('nextjs'))
+  .addDependency('@clerk/shared', linkPackage('shared'))
+  .addDependency('@clerk/types', linkPackage('types'));
 
 const appRouterTurbo = appRouter.clone().setName('next-app-router-turbopack').addScript('dev', 'pnpm dev');
 

diff --git a/integration/presets/nuxt.ts b/integration/presets/nuxt.ts
@@ -1,5 +1,6 @@
 import { applicationConfig } from '../models/applicationConfig';
 import { templates } from '../templates';
+import { linkPackage } from './utils';
 
 const nuxtNode = applicationConfig()
   .setName('nuxt-node')
@@ -10,7 +11,7 @@ const nuxtNode = applicationConfig()
   .addScript('dev', 'pnpm dev')
   .addScript('build', 'pnpm build')
   .addScript('serve', 'pnpm preview')
-  .addDependency('@clerk/nuxt', '*');
+  .addDependency('@clerk/nuxt', linkPackage('nuxt'));
 
 export const nuxt = {
   node: nuxtNode,

diff --git a/integration/presets/react-router.ts b/integration/presets/react-router.ts
@@ -1,6 +1,7 @@
 import { constants } from '../constants';
 import { applicationConfig } from '../models/applicationConfig.js';
 import { templates } from '../templates/index.js';
+import { linkPackage } from './utils';
 
 const reactRouterNode = applicationConfig()
   .setName('react-router-node')
@@ -10,8 +11,19 @@ const reactRouterNode = applicationConfig()
   .addScript('dev', 'pnpm dev')
   .addScript('build', 'pnpm build')
   .addScript('serve', 'pnpm start')
-  .addDependency('@clerk/react-router', constants.E2E_CLERK_VERSION || '*');
+  .addDependency('@clerk/react-router', constants.E2E_CLERK_VERSION || linkPackage('react-router'));
+
+const reactRouterLibrary = applicationConfig()
+  .setName('react-router-library')
+  .useTemplate(templates['react-router-library'])
+  .setEnvFormatter('public', key => `VITE_${key}`)
+  .addScript('setup', 'pnpm install')
+  .addScript('dev', 'pnpm dev')
+  .addScript('build', 'pnpm build')
+  .addScript('serve', 'pnpm preview')
+  .addDependency('@clerk/react-router', constants.E2E_CLERK_VERSION || linkPackage('react-router'));
 
 export const reactRouter = {
   reactRouterNode,
+  reactRouterLibrary,
 } as const;
diff --git a/integration/presets/react.ts b/integration/presets/react.ts
@@ -1,6 +1,7 @@
 import { constants } from '../constants';
 import { applicationConfig } from '../models/applicationConfig';
 import { templates } from '../templates';
+import { linkPackage } from './utils';
 
 const cra = applicationConfig()
   .setName('react-cra')
@@ -10,8 +11,8 @@ const cra = applicationConfig()
   .addScript('dev', 'pnpm start')
   .addScript('build', 'pnpm build')
   .addScript('serve', 'pnpm start')
-  .addDependency('@clerk/clerk-react', constants.E2E_CLERK_VERSION || '*')
-  .addDependency('@clerk/themes', constants.E2E_CLERK_VERSION || '*');
+  .addDependency('@clerk/clerk-react', constants.E2E_CLERK_VERSION || linkPackage('react'))
+  .addDependency('@clerk/themes', constants.E2E_CLERK_VERSION || linkPackage('themes'));
 
 const vite = cra
   .clone()

diff --git a/integration/presets/remix.ts b/integration/presets/remix.ts
diff --git a/integration/presets/tanstack.ts b/integration/presets/tanstack.ts
@@ -1,5 +1,6 @@
 import { applicationConfig } from '../models/applicationConfig.js';
 import { templates } from '../templates/index.js';
+import { linkPackage } from './utils';
 
 const router = applicationConfig()
   .setName('tanstack-router')
@@ -9,7 +10,7 @@ const router = applicationConfig()
   .addScript('dev', 'pnpm dev')
   .addScript('build', 'pnpm build')
   .addScript('serve', 'pnpm start')
-  .addDependency('@clerk/tanstack-start', '*');
+  .addDependency('@clerk/tanstack-start', linkPackage('tanstack-start'));
 
 const start = applicationConfig()
   .setName('tanstack-start')
@@ -19,7 +20,7 @@ const start = applicationConfig()
   .addScript('dev', 'pnpm dev')
   .addScript('build', 'pnpm build')
   .addScript('serve', 'pnpm start')
-  .addDependency('@clerk/tanstack-start', '*');
+  .addDependency('@clerk/tanstack-start', linkPackage('tanstack-start'));
 
 export const tanstack = {
   start,

diff --git a/integration/presets/utils.ts b/integration/presets/utils.ts
@@ -0,0 +1,8 @@
+import path from 'node:path';
+
+export function linkPackage(pkg: string) {
+  // eslint-disable-next-line turbo/no-undeclared-variables
+  if (process.env.CI === 'true') return '*';
+
+  return `link:${path.resolve(process.cwd(), `packages/${pkg}`)}`;
+}
diff --git a/integration/presets/vue.ts b/integration/presets/vue.ts
@@ -1,5 +1,6 @@
 import { applicationConfig } from '../models/applicationConfig';
 import { templates } from '../templates';
+import { linkPackage } from './utils';
 
 const vite = applicationConfig()
   .setName('vue-vite')
@@ -9,7 +10,7 @@ const vite = applicationConfig()
   .addScript('dev', 'pnpm dev')
   .addScript('build', 'pnpm build')
   .addScript('serve', 'pnpm preview')
-  .addDependency('@clerk/vue', '*');
+  .addDependency('@clerk/vue', linkPackage('vue'));
 
 export const vue = {
   vite,

diff --git a/integration/templates/index.ts b/integration/templates/index.ts
@@ -9,7 +9,6 @@ export const templates = {
   'react-cra': resolve(__dirname, './react-cra'),
   'react-vite': resolve(__dirname, './react-vite'),
   'express-vite': resolve(__dirname, './express-vite'),
-  'remix-node': resolve(__dirname, './remix-node'),
   'elements-next': resolve(__dirname, './elements-next'),
   'astro-node': resolve(__dirname, './astro-node'),
   'astro-hybrid': resolve(__dirname, './astro-hybrid'),
@@ -19,6 +18,7 @@ export const templates = {
   'vue-vite': resolve(__dirname, './vue-vite'),
   'nuxt-node': resolve(__dirname, './nuxt-node'),
   'react-router-node': resolve(__dirname, './react-router-node'),
+  'react-router-library': resolve(__dirname, './react-router-library'),
 } as const;
 
 if (new Set([...Object.values(templates)]).size !== Object.values(templates).length) {

diff --git a/integration/templates/next-app-router/src/app/create-organization/page.tsx b/integration/templates/next-app-router/src/app/create-organization/page.tsx
@@ -0,0 +1,9 @@
+import { CreateOrganization } from '@clerk/nextjs';
+
+export default function Page() {
+  return (
+    <div>
+      <CreateOrganization fallback={<>Loading create organization</>} />
+    </div>
+  );
+}
diff --git a/integration/templates/next-app-router/src/app/organization-list/page.tsx b/integration/templates/next-app-router/src/app/organization-list/page.tsx
@@ -0,0 +1,9 @@
+import { OrganizationList } from '@clerk/nextjs';
+
+export default function Page() {
+  return (
+    <div>
+      <OrganizationList fallback={<>Loading organization list</>} />
+    </div>
+  );
+}
diff --git a/integration/templates/next-app-router/src/app/organization-profile/page.tsx b/integration/templates/next-app-router/src/app/organization-profile/page.tsx
@@ -0,0 +1,12 @@
+import { OrganizationProfile } from '@clerk/nextjs';
+
+export default function Page() {
+  return (
+    <div>
+      <OrganizationProfile
+        routing='hash'
+        fallback={<>Loading organization profile</>}
+      />
+    </div>
+  );
+}
diff --git a/integration/templates/next-app-router/src/app/organization-switcher/page.tsx b/integration/templates/next-app-router/src/app/organization-switcher/page.tsx
@@ -0,0 +1,9 @@
+import { OrganizationList, OrganizationSwitcher } from '@clerk/nextjs';
+
+export default function Page() {
+  return (
+    <div>
+      <OrganizationSwitcher fallback={<>Loading organization switcher</>} />
+    </div>
+  );
+}
diff --git a/integration/templates/next-app-router/src/app/page.tsx b/integration/templates/next-app-router/src/app/page.tsx
@@ -5,7 +5,7 @@ import { ClientId } from './client-id';
 export default function Home() {
   return (
     <main>
-      <UserButton />
+      <UserButton fallback={<>Loading user button</>} />
       <ClientId />
       <SignedIn>SignedIn</SignedIn>
       <SignedOut>SignedOut</SignedOut>

diff --git a/integration/templates/next-app-router/src/app/sign-in/[[...catchall]]/page.tsx b/integration/templates/next-app-router/src/app/sign-in/[[...catchall]]/page.tsx
@@ -7,6 +7,7 @@ export default function Page() {
         routing={'path'}
         path={'/sign-in'}
         signUpUrl={'/sign-up'}
+        fallback={<>Loading sign in</>}
         __experimental={{
           combinedProps: {},
         }}

diff --git a/integration/templates/next-app-router/src/app/sign-up/[[...catchall]]/page.tsx b/integration/templates/next-app-router/src/app/sign-up/[[...catchall]]/page.tsx
@@ -7,6 +7,7 @@ export default function Page() {
         routing={'path'}
         path={'/sign-up'}
         signInUrl={'/sign-in'}
+        fallback={<>Loading sign up</>}
       />
     </div>
   );

diff --git a/integration/templates/next-app-router/src/app/switcher/page.tsx b/integration/templates/next-app-router/src/app/switcher/page.tsx
@@ -1,5 +1,10 @@
 import { OrganizationSwitcher } from '@clerk/nextjs';
 
 export default function Page() {
-  return <OrganizationSwitcher hidePersonal={true} />;
+  return (
+    <OrganizationSwitcher
+      hidePersonal={true}
+      fallback={<>Loading organization switcher</>}
+    />
+  );
 }