From d0ac8d9fc71c138b5e941dac0bb355c6663c3e1b Mon Sep 17 00:00:00 2001 From: Test Date: Fri, 20 Mar 2026 22:27:13 -0700 Subject: [PATCH] chore: remove dead top-level toolsets config key The top-level 'toolsets' key in config.yaml was never read at runtime. Tool selection uses platform_toolsets (per-platform) or the --toolsets CLI flag. The key existed in load_cli_config() defaults and the example config as 'toolsets: [all]', misleading users into thinking it controlled tool availability. - Remove from load_cli_config() hardcoded defaults - Remove from hermes config show output - Replace in cli-config.yaml.example with deprecation note pointing to platform_toolsets and hermes tools --- cli-config.yaml.example | 54 +++++------------------------------------ cli.py | 2 +- hermes_cli/config.py | 1 - 3 files changed, 7 insertions(+), 50 deletions(-) diff --git a/cli-config.yaml.example b/cli-config.yaml.example index 4e05b397d..3d948e684 100644 --- a/cli-config.yaml.example +++ b/cli-config.yaml.example @@ -424,7 +424,7 @@ agent: # Toolsets # ============================================================================= # Control which tools the agent has access to. -# Use "all" to enable everything, or specify individual toolsets. +# Use `hermes tools` to interactively enable/disable tools per platform. # ============================================================================= # Platform Toolsets (per-platform tool configuration) @@ -533,53 +533,11 @@ platform_toolsets: # debugging - terminal + web + file (for troubleshooting) # safe - web + vision + moa (no terminal access) -# ----------------------------------------------------------------------------- -# OPTION 1: Enable all tools (default) -# ----------------------------------------------------------------------------- -toolsets: - - all - -# ----------------------------------------------------------------------------- -# OPTION 2: Minimal - just web search and terminal -# Great for: Simple coding tasks, quick lookups -# ----------------------------------------------------------------------------- -# toolsets: -# - web -# - terminal - -# ----------------------------------------------------------------------------- -# OPTION 3: Research mode - no execution capabilities -# Great for: Safe information gathering, research tasks -# ----------------------------------------------------------------------------- -# toolsets: -# - web -# - vision -# - skills - -# ----------------------------------------------------------------------------- -# OPTION 4: Full automation - browser + terminal -# Great for: Web scraping, automation tasks, testing -# ----------------------------------------------------------------------------- -# toolsets: -# - terminal -# - browser -# - web - -# ----------------------------------------------------------------------------- -# OPTION 5: Creative mode - vision + image generation -# Great for: Design work, image analysis, creative tasks -# ----------------------------------------------------------------------------- -# toolsets: -# - vision -# - image_gen -# - web - -# ----------------------------------------------------------------------------- -# OPTION 6: Safe mode - no terminal or browser -# Great for: Restricted environments, untrusted queries -# ----------------------------------------------------------------------------- -# toolsets: -# - safe +# NOTE: The top-level "toolsets" key is deprecated and ignored. +# Tool configuration is managed per-platform via platform_toolsets above. +# Use `hermes tools` to configure interactively, or edit platform_toolsets directly. +# +# CLI override: hermes chat --toolsets terminal,web,file # ============================================================================= # MCP (Model Context Protocol) Servers diff --git a/cli.py b/cli.py index 4d9ea6835..c3dcbcb4c 100755 --- a/cli.py +++ b/cli.py @@ -211,7 +211,7 @@ def load_cli_config() -> Dict[str, Any]: "hype": "YOOO LET'S GOOOO!!! I am SO PUMPED to help you today! Every question is AMAZING and we're gonna CRUSH IT together! This is gonna be LEGENDARY! ARE YOU READY?! LET'S DO THIS!", }, }, - "toolsets": ["all"], + "display": { "compact": False, "resume_display": "full", diff --git a/hermes_cli/config.py b/hermes_cli/config.py index 9be96e42f..b2fd27c15 100644 --- a/hermes_cli/config.py +++ b/hermes_cli/config.py @@ -1607,7 +1607,6 @@ def show_config(): print(color("◆ Model", Colors.CYAN, Colors.BOLD)) print(f" Model: {config.get('model', 'not set')}") print(f" Max turns: {config.get('agent', {}).get('max_turns', DEFAULT_CONFIG['agent']['max_turns'])}") - print(f" Toolsets: {', '.join(config.get('toolsets', ['all']))}") # Display print()