fix(clerk-js): Remove beforeunload event listener from SafeLock (#7775) (#7818)

Co-authored-by: Ben Singer <benjaminsinger@ftmedia.com>
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Nikos Douvlis <nikosdouvlis@gmail.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

Author: 5 people

.changeset/remove-beforeunload-safelock.md

@@ -0,0 +1,5 @@
+---
+'@clerk/clerk-js': patch
+---
+
+Removed redundant `beforeunload` event listener from SafeLock that was disabling the browser's back-forward cache (bfcache), degrading navigation performance.

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

@@ -3,12 +3,6 @@ import Lock from 'browser-tabs-lock';
 export function SafeLock(key: string) {
   const lock = new Lock();
 
-  // TODO: Figure out how to fix this linting error
-  // eslint-disable-next-line @typescript-eslint/no-misused-promises
-  window.addEventListener('beforeunload', async () => {
-    await lock.releaseLock(key);
-  });
-
   const acquireLockAndRun = async (cb: () => Promise<unknown>) => {
     if ('locks' in navigator && isSecureContext) {
       const controller = new AbortController();