Skip to content

fix(honcho): auto-enable honcho toolset when configured (rebased) #14408

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Bartok9 wants to merge 1 commit into from
Closed

fix(honcho): auto-enable honcho toolset when configured (rebased) #14408

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions hermes_cli/tools_config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -607,6 +607,18 @@ def _get_platform_tools(
enabled_toolsets.add(pts)
# else: known but not in config = user disabled it

# Honcho toolset: auto-enable when configured, similar to plugins.
# Honcho is not in CONFIGURABLE_TOOLSETS (it is enabled via `hermes honcho setup`),
# so it would be silently dropped when platform_toolsets is explicitly configured.
if "honcho" not in enabled_toolsets:
try:
from honcho_integration.client import HonchoClientConfig
hcfg = HonchoClientConfig.from_global_config()
if hcfg.enabled and (hcfg.api_key or hcfg.base_url):
enabled_toolsets.add("honcho")
except Exception:
pass # honcho not installed or not configured — skip silently

# Preserve any explicit non-configurable toolset entries (for example,
# custom toolsets or MCP server names saved in platform_toolsets).
platform_default_keys = {p["default_toolset"] for p in PLATFORMS.values()}
Expand Down
Loading