Fix default redirectMethod for Node.js environments (#1301)

BilalG1 · web-flow · commit df5c5884ae7f · 2026-03-31T16:00:22.000-07:00
## Summary - Fixes the default `redirectMethod` to check `typeof window !== "undefined"` before defaulting to `"window"` - In Node.js/CLI environments (where `window` is undefined), defaults to `"none"` instead, preventing runtime errors ## Context PR #1253 changed the default `redirectMethod` from `"none"` to `"window"`, but this breaks non-browser environments (Node.js scripts, CLI tools, backend services) that don't have a `window` object. ## Test plan - [ ] Verify existing e2e tests pass (they explicitly set `redirectMethod: "none"`) - [ ] Verify browser usage still defaults to `"window"` - [ ] Verify Node.js usage (e.g. `stack-cli`) defaults to `"none"`  ## Summary by CodeRabbit * **Bug Fixes** * Fixed redirect behavior so client-side redirects continue in browser environments, while non-browser or server-side executions now default to no automatic redirect, avoiding unexpected navigation. Existing platform-specific redirect handling remains unchanged.
diff --git a/packages/template/src/lib/stack-app/apps/implementations/client-app-impl.ts b/packages/template/src/lib/stack-app/apps/implementations/client-app-impl.ts
@@ -530,7 +530,7 @@ export class _StackClientAppImplIncomplete<HasTokenStore extends boolean, Projec
     }
 
     this._tokenStoreInit = resolvedOptions.tokenStore;
-    this._redirectMethod = resolvedOptions.redirectMethod || "window";
+    this._redirectMethod = resolvedOptions.redirectMethod || (isBrowserLike() ? "window" : "none");
     this._redirectMethod = resolvedOptions.redirectMethod || "nextjs"; // THIS_LINE_PLATFORM next
     this._urlOptions = resolvedOptions.urls ?? {};
     this._oauthScopesOnSignIn = resolvedOptions.oauthScopesOnSignIn ?? {};

Original file line number	Diff line number	Diff line change
`@@ -530,7 +530,7 @@ export class _StackClientAppImplIncomplete<HasTokenStore extends boolean, Projec`
`530`	`530`	`}`
`531`	`531`
`532`	`532`	`this._tokenStoreInit = resolvedOptions.tokenStore;`
`533`		`- this._redirectMethod = resolvedOptions.redirectMethod \|\| "window";`
	`533`	`+ this._redirectMethod = resolvedOptions.redirectMethod \|\| (isBrowserLike() ? "window" : "none");`
`534`	`534`	`this._redirectMethod = resolvedOptions.redirectMethod \|\| "nextjs"; // THIS_LINE_PLATFORM next`
`535`	`535`	`this._urlOptions = resolvedOptions.urls ?? {};`
`536`	`536`	`this._oauthScopesOnSignIn = resolvedOptions.oauthScopesOnSignIn ?? {};`