Skip to content

Commit 0d68872

Browse files
Revert "Fix subagent tool inheritance when parent has default enabled_toolsets" (#7)
This reverts commit 55a7752.
1 parent 55a7752 commit 0d68872

1 file changed

Lines changed: 0 additions & 16 deletions

File tree

tools/delegate_tool.py

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -892,22 +892,6 @@ def _build_child_agent(
892892
# Note: enabled_toolsets=None means "all tools enabled" (the default),
893893
# so we must derive effective toolsets from the parent's loaded tools.
894894
parent_enabled = getattr(parent_agent, "enabled_toolsets", None)
895-
896-
# If enabled_toolsets is a minimal/default set (like ["hermes-acp"]) that doesn't
897-
# actually cover the parent's loaded tools, derive from valid_tool_names instead.
898-
# This ensures subagents get all tools the parent has, not just the default set.
899-
if parent_enabled is not None and parent_agent and hasattr(parent_agent, "valid_tool_names"):
900-
import model_tools
901-
# Check if enabled_toolsets actually covers the loaded tools
902-
loaded_toolsets = {
903-
ts
904-
for name in parent_agent.valid_tool_names
905-
if (ts := model_tools.get_toolset_for_tool(name)) is not None
906-
}
907-
# If enabled_toolsets is missing tools the parent actually has, derive from loaded tools
908-
if loaded_toolsets and not loaded_toolsets.issubset(set(parent_enabled)):
909-
parent_enabled = None # Force derivation from valid_tool_names
910-
911895
if parent_enabled is not None:
912896
parent_toolsets = set(parent_enabled)
913897
elif parent_agent and hasattr(parent_agent, "valid_tool_names"):

0 commit comments

Comments
 (0)