From 9f979151634d5bb3acded9649ee5153cae2bc650 Mon Sep 17 00:00:00 2001 From: teknium1 <127238744+teknium1@users.noreply.github.com> Date: Mon, 29 Jun 2026 02:11:33 -0700 Subject: [PATCH] fix(browser): route open-timeout base through _safe_command_timeout Wire the salvaged _safe_command_timeout() guard into the surviving open-timeout call site. _get_open_command_timeout() feeds the browser_navigate 'open' path; this closes the last call site that could observe a None timeout from a torn cache (#14331), since the original PR's max(_get_command_timeout(), 60) site no longer exists on main (now routed through _get_open_command_timeout). --- tools/browser_tool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/browser_tool.py b/tools/browser_tool.py index 36853214668..0c255bf6c81 100644 --- a/tools/browser_tool.py +++ b/tools/browser_tool.py @@ -278,7 +278,7 @@ def _safe_command_timeout() -> int: def _get_open_command_timeout(*, first_open: bool = False) -> int: """Timeout for agent-browser ``open`` (navigation / daemon cold start).""" - base = _get_command_timeout() + base = _safe_command_timeout() floor = MIN_FIRST_OPEN_TIMEOUT if first_open else MIN_OPEN_TIMEOUT return max(base, floor)