Merge branch 'main' into carp/references-theming-architecture

Author: alexcarpenter

.changeset/clear-tracked-test-intervals.md

@@ -0,0 +1,2 @@
+---
+---

.changeset/orange-pandas-burn.md

@@ -0,0 +1,5 @@
+---
+"@clerk/shared": patch
+---
+
+Remove internal tag from publicly re-exported error helpers

.github/workflows/lock-threads.yml

@@ -8,7 +8,6 @@ on:
     - cron: '0 0 * * *'
 
 permissions:
-  contents: write # only for delete-branch option
   issues: write
   pull-requests: write
 

.github/workflows/validate-renovate-config.yml

@@ -16,14 +16,5 @@ jobs:
       - name: Checkout repo
         uses: actions/checkout@v4
 
-      - name: Setup
-        id: config
-        uses: ./.github/actions/init
-        with:
-          cache-enabled: true
-          turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }}
-          turbo-team: ${{ vars.TURBO_TEAM }}
-          turbo-token: ${{ secrets.TURBO_TOKEN }}
-
       - name: Validate Renovate Config
-        run: npx --yes --package renovate@latest renovate-config-validator
+        run: npx --yes --package renovate@43.150.0 renovate-config-validator

packages/clerk-js/vitest.setup.mts

@@ -10,8 +10,11 @@ configure({ asyncUtilTimeout: 5000 });
 
 // Track all timers created during tests to clean them up
 const activeTimers = new Set<ReturnType<typeof setTimeout>>();
+const activeIntervals = new Set<ReturnType<typeof setInterval>>();
 const originalSetTimeout = global.setTimeout;
 const originalClearTimeout = global.clearTimeout;
+const originalSetInterval = global.setInterval;
+const originalClearInterval = global.clearInterval;
 
 // Wrap setTimeout to track all timers
 global.setTimeout = ((callback: any, delay?: any, ...args: any[]) => {
@@ -28,15 +31,34 @@ global.clearTimeout = ((timerId?: ReturnType<typeof setTimeout>) => {
   }
 }) as typeof clearTimeout;
 
+// Wrap setInterval the same way so libraries like @formkit/auto-animate
+// (which polls via setInterval and calls requestAnimationFrame inside it)
+// cannot leak callbacks past the test environment teardown.
+global.setInterval = ((callback: any, delay?: any, ...args: any[]) => {
+  const intervalId = originalSetInterval(callback, delay, ...args);
+  activeIntervals.add(intervalId);
+  return intervalId;
+}) as typeof setInterval;
+
+global.clearInterval = ((intervalId?: ReturnType<typeof setInterval>) => {
+  if (intervalId) {
+    activeIntervals.delete(intervalId);
+    originalClearInterval(intervalId);
+  }
+}) as typeof clearInterval;
+
 beforeEach(() => {
   activeTimers.clear();
+  activeIntervals.clear();
 });
 
 afterEach(() => {
   cleanup();
-  // Clear all tracked timers to prevent post-test execution
+  // Clear all tracked timers/intervals to prevent post-test execution
   activeTimers.forEach(timerId => originalClearTimeout(timerId));
   activeTimers.clear();
+  activeIntervals.forEach(intervalId => originalClearInterval(intervalId));
+  activeIntervals.clear();
 });
 
 // Store the original method

packages/shared/src/errors/helpers.ts

@@ -80,35 +80,27 @@ export function isNetworkError(e: any): boolean {
 
 /**
  * Checks if the provided error is either a ClerkAPIResponseError, a ClerkRuntimeError, or a MetamaskError.
- *
- * @internal
  */
 export function isKnownError(error: any): error is ClerkAPIResponseError | ClerkRuntimeError | MetamaskError {
   return isClerkAPIResponseError(error) || isMetamaskError(error) || isClerkRuntimeError(error);
 }
 
 /**
  * Checks if the provided error is a Clerk runtime error indicating a reverification was cancelled.
- *
- * @internal
  */
 export function isReverificationCancelledError(err: any) {
   return isClerkRuntimeError(err) && err.code === 'reverification_cancelled';
 }
 
 /**
  * Checks if the provided error is a Metamask error.
- *
- * @internal
  */
 export function isMetamaskError(err: any): err is MetamaskError {
   return 'code' in err && [4001, 32602, 32603].includes(err.code) && 'message' in err;
 }
 
 /**
  * Checks if the provided error is clerk api response error indicating a user is locked.
- *
- * @internal
  */
 export function isUserLockedError(err: any) {
   return isClerkAPIResponseError(err) && err.errors?.[0]?.code === 'user_locked';
@@ -134,8 +126,6 @@ export function isPasswordCompromisedError(err: any) {
 
 /**
  * Checks if the provided error is an EmailLinkError.
- *
- * @internal
  */
 export function isEmailLinkError(err: Error): err is EmailLinkError {
   return err.name === 'EmailLinkError';