fix(nextjs): Check before using window identifier

clerk · May 23, 2024 · 539e534 · 539e534
1 parent b71dae6
commit 539e534
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/packages/nextjs/src/app-router/client/useAwaitablePush.ts b/packages/nextjs/src/app-router/client/useAwaitablePush.ts
@@ -13,7 +13,7 @@ export const useAwaitablePush = () => {
   const router = useRouter();
 
   return useInternalNavFun({
-    windowNav: window?.history.pushState.bind(window.history),
+    windowNav: typeof window !== 'undefined' ? window.history.pushState.bind(window.history) : undefined,
     routerNav: router.push.bind(router),
   });
 };
diff --git a/packages/nextjs/src/app-router/client/useAwaitableReplace.ts b/packages/nextjs/src/app-router/client/useAwaitableReplace.ts
@@ -13,7 +13,7 @@ export const useAwaitableReplace = () => {
   const router = useRouter();
 
   return useInternalNavFun({
-    windowNav: window?.history.replaceState.bind(window.history),
+    windowNav: typeof window !== 'undefined' ? window.history.replaceState.bind(window.history) : undefined,
     routerNav: router.replace.bind(router),
   });
 };
diff --git a/packages/nextjs/src/app-router/client/useInternalNavFun.ts b/packages/nextjs/src/app-router/client/useInternalNavFun.ts
@@ -14,14 +14,14 @@ declare global {
 }
 
 export const useInternalNavFun = (props: {
-  windowNav: typeof window.history.pushState | typeof window.history.replaceState;
+  windowNav: typeof window.history.pushState | typeof window.history.replaceState | undefined;
   routerNav: AppRouterInstance['push'] | AppRouterInstance['replace'];
 }) => {
   const { windowNav, routerNav } = props;
   const pathname = usePathname();
   const [isPending, startTransition] = useTransition();
 
-  if (typeof window !== 'undefined') {
+  if (windowNav) {
     window.__clerk_internal_navFun = (to, opts) => {
       return new Promise<void>(res => {
         if (!window.__clerk_internal_navPromisesBuffer) {