From b7bcc8c09479357a29b2c8fb4318c40311ec3148 Mon Sep 17 00:00:00 2001 From: Alec Jiang Date: Mon, 16 Mar 2026 18:11:37 +0800 Subject: [PATCH] fix(baoyu-chrome-cdp): keep HTTP validation in findExistingChromeDebugPort findExistingChromeDebugPort callers (fetch_google_cookies_via_cdp) depend on /json/version being available. TCP-only check could misclassify non-CDP listeners as reusable, causing waitForChromeDebugPort to timeout instead of falling back to launching a new Chrome. Restore isDebugPortReady (HTTP) as the validator for this function; TCP-only check remains in discoverRunningChromeDebugPort for Chrome 146 approval mode. Co-Authored-By: Claude Opus 4.6 (1M context) --- packages/baoyu-chrome-cdp/src/index.ts | 1 - .../scripts/vendor/baoyu-chrome-cdp/src/index.ts | 1 - 2 files changed, 2 deletions(-) diff --git a/packages/baoyu-chrome-cdp/src/index.ts b/packages/baoyu-chrome-cdp/src/index.ts index b455324..ee4fd71 100644 --- a/packages/baoyu-chrome-cdp/src/index.ts +++ b/packages/baoyu-chrome-cdp/src/index.ts @@ -210,7 +210,6 @@ export async function findExistingChromeDebugPort(options: FindExistingChromeDeb const timeoutMs = options.timeoutMs ?? 3_000; const parsed = parseDevToolsActivePort(path.join(options.profileDir, "DevToolsActivePort")); - if (parsed && await isPortListening(parsed.port, timeoutMs)) return parsed.port; if (parsed && parsed.port > 0 && await isDebugPortReady(parsed.port, timeoutMs)) return parsed.port; if (process.platform === "win32") return null; diff --git a/skills/baoyu-danger-gemini-web/scripts/vendor/baoyu-chrome-cdp/src/index.ts b/skills/baoyu-danger-gemini-web/scripts/vendor/baoyu-chrome-cdp/src/index.ts index b455324..ee4fd71 100644 --- a/skills/baoyu-danger-gemini-web/scripts/vendor/baoyu-chrome-cdp/src/index.ts +++ b/skills/baoyu-danger-gemini-web/scripts/vendor/baoyu-chrome-cdp/src/index.ts @@ -210,7 +210,6 @@ export async function findExistingChromeDebugPort(options: FindExistingChromeDeb const timeoutMs = options.timeoutMs ?? 3_000; const parsed = parseDevToolsActivePort(path.join(options.profileDir, "DevToolsActivePort")); - if (parsed && await isPortListening(parsed.port, timeoutMs)) return parsed.port; if (parsed && parsed.port > 0 && await isDebugPortReady(parsed.port, timeoutMs)) return parsed.port; if (process.platform === "win32") return null;