fix: always skip browser opens in tests

CHANGELOG.md

@@ -44,7 +44,7 @@ Docs: https://docs.clawd.bot
 - Docs: fix gog auth services example to include docs scope. (#1454) Thanks @zerone0x.
 - Slack: read thread replies for message reads when threadId is provided (replies-only). (#1450) Thanks @rodrigouroz.
 - macOS: prefer linked channels in gateway summary to avoid false “not linked” status.
-- macOS/tests: fix gateway summary lookup after guard unwrap; prevent browser opens during tests unless explicitly enabled. (ECID-1483)
+- macOS/tests: fix gateway summary lookup after guard unwrap; prevent browser opens during tests. (ECID-1483)
 - Providers: improve GitHub Copilot integration (enterprise support, base URL, and auth flow alignment).
 
 ## 2026.1.21-2

src/commands/onboard-helpers.test.ts

@@ -27,7 +27,6 @@ afterEach(() => {
 
 describe("openUrl", () => {
   it("quotes URLs on win32 so '&' is not treated as cmd separator", async () => {
-    vi.stubEnv("CLAWDBOT_ALLOW_TEST_BROWSER_OPEN", "1");
     const platformSpy = vi.spyOn(process, "platform", "get").mockReturnValue("win32");
 
     const url =

src/commands/onboard-helpers.ts

@@ -311,7 +311,6 @@ export async function detectBinary(name: string): Promise<boolean> {
 }
 
 function shouldSkipBrowserOpenInTests(): boolean {
-  if (process.env.CLAWDBOT_ALLOW_TEST_BROWSER_OPEN) return false;
   if (process.env.VITEST) return true;
   return process.env.NODE_ENV === "test";
 }