hermes-admin/hermes-agent
1
0
Fork 0
Code Issues Pull Requests Actions 4 Packages Projects Releases Wiki Activity

refactor(agent/web_search_registry): reuse the eligible set in the legacy walk; compact log calls

Browse Source
This commit is contained in:
Teknium
2026-09-02 19:04:51 -07:00
parent 8873f7abc7
commit aa0ebfc103
1 changed files with 5 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
agent/web_search_registry.py
View File

@@ -38,8 +38,7 @@ def _read_config_key(*path: str) -> Optional[str]:
try:
from hermes_cli.config import load_config_readonly
cfg = load_config_readonly()
cur = cfg
cur = load_config_readonly()
for segment in path:
if not isinstance(cur, dict):
return None
@@ -107,13 +106,10 @@ def _resolve(configured: Optional[str], *, capability: str) -> Optional[WebSearc
if provider is not None and _capable(provider):
return provider
if provider is None:
logger.debug(
"web backend '%s' configured but not registered; falling back", configured
)
logger.debug("web backend '%s' configured but not registered; falling back", configured)
else:
logger.debug(
"web backend '%s' configured but does not support '%s'; falling back",
configured, capability,
"web backend '%s' configured but does not support '%s'; falling back", configured, capability
)
# Fallbacks are availability-filtered so a registered-but-keyless provider
@@ -124,7 +120,7 @@ def _resolve(configured: Optional[str], *, capability: str) -> Optional[WebSearc
for legacy in _LEGACY_PREFERENCE:
provider = snapshot.get(legacy)
if provider is not None and _capable(provider) and _available(provider):
if provider is not None and provider in eligible:
return provider
# Keyless free tier (anonymous public MCP tiers) is last-resort only: it is
@@ -139,9 +135,7 @@ def _resolve(configured: Optional[str], *, capability: str) -> Optional[WebSearc
if provider.is_keyless_available():
return provider
except Exception as exc: # noqa: BLE001 — buggy provider skipped
logger.debug(
"provider %s.is_keyless_available() raised %s", name, exc
)
logger.debug("provider %s.is_keyless_available() raised %s", name, exc)
return None