do not expose internals

Author: jacekradko

integration/tests/session-token-cache/multi-session.test.ts

@@ -80,12 +80,6 @@ testAgainstRunningApps({ withEnv: [appConfigs.envs.withSessionTasks] })(
       expect(page2User1SessionInfo.userId).toBe(user1SessionInfo.userId);
       expect(page2User1SessionInfo.sessionId).toBe(user1SessionInfo.sessionId);
 
-      const page2HasUser1Token = await page2.evaluate(() => {
-        const clerk = (window as any).Clerk;
-        return clerk?._sessionTokenCache?.size?.() > 0;
-      });
-      expect(page2HasUser1Token).toBe(true);
-
       // Use clerk.client.signIn.create() instead of navigating to /sign-in
       // because navigating replaces the session by default (transferable: true)
       const signInResult = await page2.evaluate(
@@ -159,20 +153,6 @@ testAgainstRunningApps({ withEnv: [appConfigs.envs.withSessionTasks] })(
       expect(page2MultiSessionInfo.allSessionIds).toContain(user2SessionInfo.sessionId);
       expect(page2MultiSessionInfo.activeSessionId).toBe(user2SessionInfo.sessionId);
 
-      const tab2CacheInfo = await page2.evaluate(() => {
-        const clerk = (window as any).Clerk;
-        const cache = clerk?._sessionTokenCache;
-        const cacheSize = cache?.size?.() || 0;
-
-        return {
-          cacheSize,
-          currentSessionId: clerk?.session?.id,
-        };
-      });
-
-      expect(tab2CacheInfo.currentSessionId).toBe(user2SessionInfo.sessionId);
-      expect(tab2CacheInfo.cacheSize).toBeGreaterThan(0);
-
       const tokenFetchRequests: Array<{ sessionId: string; url: string }> = [];
       await context.route('**/v1/client/sessions/*/tokens*', async route => {
         const url = route.request().url();
@@ -213,15 +193,13 @@ testAgainstRunningApps({ withEnv: [appConfigs.envs.withSessionTasks] })(
         const clerk = (window as any).Clerk;
         return {
           activeSessionId: clerk?.session?.id,
-          cacheSize: clerk?._sessionTokenCache?.size?.() || 0,
           userId: clerk?.user?.id,
         };
       });
 
       // Tab1 should STILL have user1 as the active session (independent per tab)
       expect(tab1FinalInfo.userId).toBe(user1SessionInfo.userId);
       expect(tab1FinalInfo.activeSessionId).toBe(user1SessionInfo.sessionId);
-      expect(tab1FinalInfo.cacheSize).toBeGreaterThan(0);
     });
   },
 );

integration/tests/session-token-cache/single-session.test.ts

@@ -52,28 +52,6 @@ testAgainstRunningApps({ withEnv: [appConfigs.envs.withEmailCodes] })(
       const u2 = createTestUtils({ app, page: page2 });
       await u2.po.expect.toBeSignedIn();
 
-      const clerkInfo1 = await page1.evaluate(() => {
-        const clerk = (window as any).Clerk;
-        return {
-          hasBroadcastChannel: !!clerk?._broadcastChannel,
-          loaded: clerk?.loaded,
-        };
-      });
-
-      const clerkInfo2 = await page2.evaluate(() => {
-        const clerk = (window as any).Clerk;
-        return {
-          hasBroadcastChannel: !!clerk?._broadcastChannel,
-          loaded: clerk?.loaded,
-        };
-      });
-
-      expect(clerkInfo1.loaded).toBe(true);
-      expect(clerkInfo2.loaded).toBe(true);
-
-      expect(clerkInfo1.hasBroadcastChannel).toBe(true);
-      expect(clerkInfo2.hasBroadcastChannel).toBe(true);
-
       const page1SessionInfo = await page1.evaluate(() => {
         const clerk = (window as any).Clerk;
         return {
@@ -104,14 +82,9 @@ testAgainstRunningApps({ withEnv: [appConfigs.envs.withEmailCodes] })(
 
       expect(page1Token).toBeTruthy();
 
-      // Wait for page2 to receive the broadcast and update its cache
-      await page2.waitForFunction(
-        () => {
-          const clerk = (window as any).Clerk;
-          return clerk?._sessionTokenCache?.size?.() > 0;
-        },
-        { timeout: 10000 },
-      );
+      // Wait for broadcast to propagate between tabs (broadcast is nearly instant, but we add buffer)
+      // eslint-disable-next-line playwright/no-wait-for-timeout
+      await page2.waitForTimeout(2000);
 
       const page2Result = await page2.evaluate(async () => {
         const clerk = (window as any).Clerk;

packages/clerk-js/src/core/clerk.ts

@@ -159,7 +159,6 @@ import { createCheckoutInstance } from './modules/checkout/instance';
 import { BaseResource, Client, Environment, Organization, Waitlist } from './resources/internal';
 import { getTaskEndpoint, navigateIfTaskExists, warnMissingPendingTaskHandlers } from './sessionTasks';
 import { State } from './state';
-import { SessionTokenCache, type TokenCache } from './tokenCache';
 import { warnings } from './warnings';
 
 type SetActiveHook = (intent?: 'sign-out') => void | Promise<void>;
@@ -266,18 +265,6 @@ export class Clerk implements ClerkInterface {
     return this.#publishableKey;
   }
 
-  get _broadcastChannel(): BroadcastChannel | null {
-    return this.#broadcastChannel;
-  }
-
-  get _sessionTokenCache(): TokenCache {
-    return SessionTokenCache;
-  }
-
-  get _options(): ClerkOptions {
-    return this.#options;
-  }
-
   get version(): string {
     return Clerk.version;
   }