From e98e71401a28edcf80cde03e292350ba76d092ca Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 23 Jan 2026 05:59:29 +0000 Subject: [PATCH] fix: always skip browser opens in tests --- CHANGELOG.md | 2 +- src/commands/onboard-helpers.test.ts | 1 - src/commands/onboard-helpers.ts | 1 - 3 files changed, 1 insertion(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 47b6541a5..9e7b05521 100644 --- a/CHANGELOG.md +++ b/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 diff --git a/src/commands/onboard-helpers.test.ts b/src/commands/onboard-helpers.test.ts index 5305b24af..cdd38baf1 100644 --- a/src/commands/onboard-helpers.test.ts +++ b/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 = diff --git a/src/commands/onboard-helpers.ts b/src/commands/onboard-helpers.ts index cdc2b1e9e..dd8288a60 100644 --- a/src/commands/onboard-helpers.ts +++ b/src/commands/onboard-helpers.ts @@ -311,7 +311,6 @@ export async function detectBinary(name: string): Promise { } 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"; }