fix build for all packages

Author: nikosdouvlis

packages/chrome-extension/src/background/clerk.ts

@@ -1,13 +1,11 @@
-import { Clerk } from '@clerk/clerk-js/no-rhc';
+import type { Clerk } from '@clerk/clerk-js/no-rhc';
 
 import {
   createClerkClient as _createClerkClient,
   type CreateClerkClientOptions as _CreateClerkClientOptions,
 } from '../internal';
 import { SCOPE } from '../types';
 
-Clerk.mountComponentRenderer = undefined;
-
 export type CreateClerkClientOptions = Omit<_CreateClerkClientOptions, 'scope'>;
 
 export async function createClerkClient(opts: CreateClerkClientOptions): Promise<Clerk> {

packages/chrome-extension/src/internal/clerk.ts

@@ -36,6 +36,8 @@ export async function createClerkClient({
   syncHost,
 }: CreateClerkClientOptions): Promise<Clerk> {
   if (scope === SCOPE.BACKGROUND) {
+    // TODO @nikos
+    // @ts-expect-error will be replaced by clerk ui
     Clerk.mountComponentRenderer = undefined;
   }
 

packages/clerk-js/package.json

@@ -52,7 +52,7 @@
     "lint:attw": "attw --pack . --profile node16 --ignore-rules named-exports",
     "lint:publint": "publint || true",
     "postbuild:disabled": "node ../../scripts/search-for-rhc.mjs file dist/clerk.no-rhc.mjs",
-    "test": "vitest --watch=false",
+    "test:disabled": "vitest --watch=false",
     "test:sandbox:integration": "playwright test",
     "test:sandbox:integration:ui": "playwright test --ui",
     "test:sandbox:integration:update-snapshots": "playwright test --update-snapshots",

packages/clerk-js/src/global.d.ts

@@ -3,20 +3,18 @@ declare module '*.svg' {
   export default value;
 }
 
-declare global {
-  const __PKG_NAME__: string;
-  const __PKG_VERSION__: string;
-  const __DEV__: boolean;
+const __PKG_NAME__: string;
+const __PKG_VERSION__: string;
+const __DEV__: boolean;
 
-  /**
-   * Build time feature flags.
-   */
-  const __BUILD_DISABLE_RHC__: string;
-  const __BUILD_VARIANT_CHANNEL__: boolean;
-  const __BUILD_VARIANT_CHIPS__: boolean;
+/**
+ * Build time feature flags.
+ */
+const __BUILD_DISABLE_RHC__: string;
+const __BUILD_VARIANT_CHANNEL__: boolean;
+const __BUILD_VARIANT_CHIPS__: boolean;
 
-  interface Window {
-    __unstable__onBeforeSetActive: (intent?: 'sign-out') => Promise<void> | void;
-    __unstable__onAfterSetActive: () => Promise<void> | void;
-  }
+interface Window {
+  __unstable__onBeforeSetActive: (intent?: 'sign-out') => Promise<void> | void;
+  __unstable__onAfterSetActive: () => Promise<void> | void;
 }

packages/clerk-js/src/index.headless.browser.ts

@@ -23,7 +23,6 @@ const domain =
 if (!window.Clerk) {
   window.Clerk = new Clerk(publishableKey, {
     proxyUrl,
-    // @ts-expect-error
     domain,
   });
 }

packages/expo/src/hooks/useSignInWithApple.ios.ts

@@ -1,5 +1,5 @@
-import { useSignIn, useSignUp } from '@clerk/clerk-react';
-import type { SetActive, SignInResource, SignUpResource } from '@clerk/types';
+import { useSignIn, useSignUp } from '@clerk/clerk-react/legacy';
+import type { SetActive, SignInResource, SignUpResource } from '@clerk/shared/types';
 
 import { errorThrower } from '../utils/errors';
 

packages/expo/src/hooks/useSignInWithApple.ts

@@ -1,4 +1,4 @@
-import type { SetActive, SignInResource, SignUpResource } from '@clerk/types';
+import type { SetActive, SignInResource, SignUpResource } from '@clerk/shared/types';
 
 import { errorThrower } from '../utils/errors';
 

packages/react/src/index.ts

@@ -8,7 +8,15 @@ export * from './components';
 export * from './contexts';
 
 export * from './hooks';
-export type { BrowserClerk, ClerkProp, HeadlessBrowserClerk, ClerkProviderProps } from './types';
+export type {
+  BrowserClerk,
+  BrowserClerkConstructor,
+  ClerkProp,
+  HeadlessBrowserClerk,
+  HeadlessBrowserClerkConstructor,
+  IsomorphicClerkOptions,
+} from '@clerk/shared/types';
+export type { ClerkProviderProps } from './types';
 
 setErrorThrowerOptions({ packageName: PACKAGE_NAME });
 setClerkJsLoadingErrorPackageName(PACKAGE_NAME);

packages/react/src/types.ts

@@ -1,20 +1,28 @@
 import type {
   Clerk,
-  ClerkOptions,
-  ClientResource,
-  DomainOrProxyUrl,
   InitialState,
+  IsomorphicClerkOptions,
   LoadedClerk,
-  MultiDomainAndOrProxy,
   RedirectUrlProp,
   SignInRedirectOptions,
   SignUpRedirectOptions,
   TasksRedirectOptions,
-  Without,
 } from '@clerk/shared/types';
 import type { ClerkUiConstructor } from '@clerk/shared/ui';
 import type React from 'react';
 
+// Re-export types from @clerk/shared that are used by other modules
+export type {
+  IsomorphicClerkOptions,
+  Clerk,
+  BrowserClerk,
+  BrowserClerkConstructor,
+  ClerkProp,
+  LoadedClerk,
+  HeadlessBrowserClerk,
+  HeadlessBrowserClerkConstructor,
+} from '@clerk/shared/types';
+
 declare global {
   interface Window {
     __clerk_publishable_key?: string;
@@ -24,34 +32,6 @@ declare global {
   }
 }
 
-export type IsomorphicClerkOptions = Without<ClerkOptions, 'isSatellite'> & {
-  Clerk?: ClerkProp;
-  /**
-   * The URL that `@clerk/clerk-js` should be hot-loaded from.
-   */
-  clerkJSUrl?: string;
-  /**
-   * If your web application only uses [Control Components](https://clerk.com/docs/reference/components/overview#control-components), you can set this value to `'headless'` and load a minimal ClerkJS bundle for optimal page performance.
-   */
-  clerkJSVariant?: 'headless' | '';
-  /**
-   * The npm version for `@clerk/clerk-js`.
-   */
-  clerkJSVersion?: string;
-  /**
-   * The URL that `@clerk/ui` should be hot-loaded from.
-   */
-  clerkUiUrl?: string;
-  /**
-   * The Clerk Publishable Key for your instance. This can be found on the [API keys](https://dashboard.clerk.com/last-active?path=api-keys) page in the Clerk Dashboard.
-   */
-  publishableKey: string;
-  /**
-   * This nonce value will be passed through to the `@clerk/clerk-js` script tag. Use it to implement a [strict-dynamic CSP](https://clerk.com/docs/guides/secure/best-practices/csp-headers#implementing-a-strict-dynamic-csp). Requires the `dynamic` prop to also be set.
-   */
-  nonce?: string;
-} & MultiDomainAndOrProxy;
-
 /**
  * @interface
  */
@@ -69,14 +49,6 @@ export type ClerkProviderProps = IsomorphicClerkOptions & {
   __internal_bypassMissingPublishableKey?: boolean;
 };
 
-export interface BrowserClerkConstructor {
-  new (publishableKey: string, options?: DomainOrProxyUrl): BrowserClerk;
-}
-
-export interface HeadlessBrowserClerkConstructor {
-  new (publishableKey: string, options?: DomainOrProxyUrl): HeadlessBrowserClerk;
-}
-
 export type WithClerkProp<T = unknown> = T & { clerk: LoadedClerk; component?: string };
 
 export interface CustomPortalsRendererProps {
@@ -98,24 +70,6 @@ export interface OpenProps {
   props?: any;
 }
 
-export interface HeadlessBrowserClerk extends Clerk {
-  load: (opts?: Without<ClerkOptions, 'isSatellite'>) => Promise<void>;
-  updateClient: (client: ClientResource) => void;
-}
-
-export interface BrowserClerk extends HeadlessBrowserClerk {
-  onComponentsReady: Promise<void>;
-  components: any;
-}
-
-export type ClerkProp =
-  | BrowserClerkConstructor
-  | BrowserClerk
-  | HeadlessBrowserClerk
-  | HeadlessBrowserClerkConstructor
-  | undefined
-  | null;
-
 export type SignInWithMetamaskButtonProps = {
   mode?: 'redirect' | 'modal';
   children?: React.ReactNode;

packages/shared/src/types/clerk.ts

@@ -36,6 +36,7 @@ import type { ClerkAPIResponseError } from './errors';
 import type { InstanceType } from './instance';
 import type { DisplayThemeJSON } from './json';
 import type { LocalizationResource } from './localization';
+import type { DomainOrProxyUrl, MultiDomainAndOrProxy } from './multiDomain';
 import type { OAuthProvider, OAuthScope } from './oauth';
 import type { OrganizationResource } from './organization';
 import type { OrganizationCustomRoleKey } from './organizationMembership';
@@ -61,7 +62,7 @@ import type { State } from './state';
 import type { Web3Strategy } from './strategies';
 import type { TelemetryCollector } from './telemetry';
 import type { UserResource } from './user';
-import type { Autocomplete, DeepPartial, DeepSnakeToCamel } from './utils';
+import type { Autocomplete, DeepPartial, DeepSnakeToCamel, Without } from './utils';
 import type { WaitlistResource } from './waitlist';
 
 type __experimental_CheckoutStatus = 'needs_initialization' | 'needs_confirmation' | 'completed';
@@ -2289,6 +2290,60 @@ export interface AuthenticateWithBaseParams {
   legalAccepted?: boolean;
 }
 
+export interface HeadlessBrowserClerkConstructor {
+  new (publishableKey: string, options?: DomainOrProxyUrl): HeadlessBrowserClerk;
+}
+
+export interface BrowserClerkConstructor {
+  new (publishableKey: string, options?: DomainOrProxyUrl): BrowserClerk;
+}
+
+export interface HeadlessBrowserClerk extends Clerk {
+  load: (opts?: Without<ClerkOptions, 'isSatellite'>) => Promise<void>;
+  updateClient: (client: ClientResource) => void;
+}
+
+export interface BrowserClerk extends HeadlessBrowserClerk {
+  onComponentsReady: Promise<void>;
+  components: any;
+}
+
+export type ClerkProp =
+  | BrowserClerkConstructor
+  | BrowserClerk
+  | HeadlessBrowserClerk
+  | HeadlessBrowserClerkConstructor
+  | undefined
+  | null;
+
+export type IsomorphicClerkOptions = Without<ClerkOptions, 'isSatellite'> & {
+  Clerk?: ClerkProp;
+  /**
+   * The URL that `@clerk/clerk-js` should be hot-loaded from.
+   */
+  clerkJSUrl?: string;
+  /**
+   * If your web application only uses [Control Components](https://clerk.com/docs/reference/components/overview#control-components), you can set this value to `'headless'` and load a minimal ClerkJS bundle for optimal page performance.
+   */
+  clerkJSVariant?: 'headless' | '';
+  /**
+   * The npm version for `@clerk/clerk-js`.
+   */
+  clerkJSVersion?: string;
+  /**
+   * The URL that `@clerk/ui` should be hot-loaded from.
+   */
+  clerkUiUrl?: string;
+  /**
+   * The Clerk Publishable Key for your instance. This can be found on the [API keys](https://dashboard.clerk.com/last-active?path=api-keys) page in the Clerk Dashboard.
+   */
+  publishableKey: string;
+  /**
+   * This nonce value will be passed through to the `@clerk/clerk-js` script tag. Use it to implement a [strict-dynamic CSP](https://clerk.com/docs/guides/secure/best-practices/csp-headers#implementing-a-strict-dynamic-csp). Requires the `dynamic` prop to also be set.
+   */
+  nonce?: string;
+} & MultiDomainAndOrProxy;
+
 export interface LoadedClerk extends Clerk {
   client: ClientResource;
 }