chore: scope oauth retry fix

Author: jacekradko

.changeset/curly-cameras-laugh.md

@@ -2,4 +2,4 @@
 "@clerk/clerk-js": patch
 ---
 
-Fix Core 3 OAuth retry routing to the previously selected provider after an abandoned redirect or BFCache restore.
+Fix Core 3 OAuth retry routing to the previously selected provider after an abandoned redirect.

packages/clerk-js/src/utils/__tests__/runAsyncResourceTask.test.ts

@@ -62,30 +62,4 @@ describe('runAsyncTask', () => {
       status: 'idle',
     });
   });
-
-  it('resets fetch status when a pending task is restored from bfcache', async () => {
-    const emitSpy = vi.spyOn(eventBus, 'emit');
-    let resolveTask: () => void;
-    const task = vi.fn(
-      () =>
-        new Promise<void>(resolve => {
-          resolveTask = resolve;
-        }),
-    );
-
-    const pendingTask = runAsyncResourceTask(resource, task);
-    await Promise.resolve();
-
-    const pageShowEvent = new Event('pageshow') as PageTransitionEvent;
-    Object.defineProperty(pageShowEvent, 'persisted', { value: true });
-    window.dispatchEvent(pageShowEvent);
-
-    expect(emitSpy).toHaveBeenNthCalledWith(3, 'resource:fetch', {
-      resource,
-      status: 'idle',
-    });
-
-    resolveTask!();
-    await pendingTask;
-  });
 });

packages/clerk-js/src/utils/runAsyncResourceTask.ts

@@ -11,22 +11,6 @@ export async function runAsyncResourceTask<T>(
   resource: BaseResource,
   task: () => Promise<T>,
 ): Promise<{ result?: T; error: ClerkError | null }> {
-  const resetFetchStatus = () => {
-    eventBus.emit('resource:fetch', {
-      resource,
-      status: 'idle',
-    });
-  };
-  const resetFetchStatusOnPageShow = (event: PageTransitionEvent) => {
-    if (event.persisted) {
-      resetFetchStatus();
-    }
-  };
-
-  if (typeof window !== 'undefined' && typeof window.addEventListener === 'function') {
-    window.addEventListener('pageshow', resetFetchStatusOnPageShow);
-  }
-
   eventBus.emit('resource:error', { resource, error: null });
   eventBus.emit('resource:fetch', {
     resource,
@@ -41,9 +25,9 @@ export async function runAsyncResourceTask<T>(
     // TODO @userland-errors:
     return { error: err };
   } finally {
-    if (typeof window !== 'undefined' && typeof window.removeEventListener === 'function') {
-      window.removeEventListener('pageshow', resetFetchStatusOnPageShow);
-    }
-    resetFetchStatus();
+    eventBus.emit('resource:fetch', {
+      resource,
+      status: 'idle',
+    });
   }
 }