fix(browser): track explicit devtools_port from config (#1383)

#1369 unconditionally set hasExplicitDevtoolsPort=false for config-attach,
which broke users who explicitly set devtools_port in config. Now track
whether config.claude.devtools_port was actually provided.
This commit is contained in:
Kai (Tam Nhu) Tran
2026-05-23 21:55:02 -04:00
committed by GitHub
parent f985cec3ff
commit b63943eb45
+2 -1
View File
@@ -230,6 +230,7 @@ export function getEffectiveClaudeBrowserAttachConfig(
const override = getBrowserAttachOverride(env);
const configUserDataDir =
resolveBrowserUserDataDir(config.claude.user_data_dir) ?? getRecommendedBrowserUserDataDir();
const configHasExplicitPort = config.claude.devtools_port !== undefined;
const configPort = normalizeDevtoolsPort(config.claude.devtools_port);
const configEvalMode = config.claude.eval_mode ?? 'readonly';
const envEvalMode = parseBrowserEvalMode(env.CCS_BROWSER_EVAL_MODE);
@@ -253,7 +254,7 @@ export function getEffectiveClaudeBrowserAttachConfig(
overrideActive: false,
userDataDir: configUserDataDir,
devtoolsPort: configPort,
hasExplicitDevtoolsPort: false,
hasExplicitDevtoolsPort: configHasExplicitPort,
evalMode: effectiveEvalMode,
};
}