fix(web-shell): guard the resumed add step outside a document

yiliang114 · Sep 19, 2026 · Sep 17, 2026 · Sep 17, 2026 · Sep 17, 2026 · Sep 17, 2026
commit 74fb08e133d8ba0a506fd9e9447ac9572a363080
diff --git a/packages/web-shell/client/config/remote-workspace-add.test.ts b/packages/web-shell/client/config/remote-workspace-add.test.ts
@@ -141,4 +141,13 @@ describe('remote workspace add navigation', () => {
       `${testOrigin}/session/current?daemon=https%3A%2F%2Fremote.example`,
     );
   });
+
+  it('reports no step for an entry point evaluated outside a document', () => {
+    vi.stubGlobal('window', undefined);
+    try {
+      expect(getRemoteWorkspaceAddStep()).toBeUndefined();
+    } finally {
+      vi.unstubAllGlobals();
+    }
+  });
 });
diff --git a/packages/web-shell/client/config/remote-workspace-add.ts b/packages/web-shell/client/config/remote-workspace-add.ts
@@ -12,6 +12,10 @@ export type RemoteWorkspaceAddStep = 'connect' | 'browse';
 export function getRemoteWorkspaceAddStep():
   | RemoteWorkspaceAddStep
   | undefined {
+  // Parity with getAllowedDaemonOrigin in the sibling module: this is the only
+  // reader on a render path, and an entry point that evaluates outside a
+  // document has no URL to carry a step.
+  if (typeof window === 'undefined') return undefined;
   const step = new URLSearchParams(window.location.search).get(FLOW_PARAM);
   return step === 'connect' || step === 'browse' ? step : undefined;
 }