From a6289927d39eb21df03a70f3d91e7eb80c54de7c Mon Sep 17 00:00:00 2001 From: beardthelion Date: Mon, 4 May 2026 23:02:24 -0500 Subject: [PATCH] docs(web_tools): correct web_extract summarizer timeout comment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The comment at tools/web_tools.py:700-702 stated the runtime default for auxiliary.web_extract.timeout is 360s. The actual runtime default is 30s (_DEFAULT_AUX_TIMEOUT in agent/auxiliary_client.py:3140), used by _get_task_timeout when no auxiliary.web_extract.timeout key is present in config.yaml. The 360s figure is the config template default written by hermes_cli/config.py:697 into freshly-generated config.yaml files. It only takes effect when that key exists in the user's config — not as a fallback. Users on configs that predate commit 20b4060d (Apr 5, 2026), or who removed the key, fall through to the 30s _DEFAULT_AUX_TIMEOUT runtime default. The comment was introduced in 20b4060d alongside the template-default bump from 30 to 360. The runtime default in auxiliary_client.py was not changed in that commit and has remained 30s since 839d9d74 (Mar 28, 2026). --- tools/web_tools.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tools/web_tools.py b/tools/web_tools.py index 352b4a55b1..e24ace2f87 100644 --- a/tools/web_tools.py +++ b/tools/web_tools.py @@ -698,8 +698,10 @@ Create a markdown summary that captures all key information in a well-organized, "temperature": 0.1, "max_tokens": max_tokens, # No explicit timeout — async_call_llm reads auxiliary.web_extract.timeout - # from config (default 360s / 6min). Users with slow local models can - # increase it in config.yaml. + # from config.yaml. Fresh configs ship with 360s; if the key is absent + # the runtime default is 30s (_DEFAULT_AUX_TIMEOUT in + # agent/auxiliary_client.py). Users with slow local models should set + # or increase auxiliary.web_extract.timeout in config.yaml. } if extra_body: call_kwargs["extra_body"] = extra_body