APEX/BrowserUse_and_ComputerUse_skills
5
0
Fork 0
Code Issues 4 Pull requests 1 Projects 1 Releases Packages Wiki Activity Actions

fix(delegate): move _saved_tool_names save/restore to _run_single_child scope

Browse source 
Fixes #1802

The v0.3.0 refactor split child agent construction (_build_child_agent)
and execution (_run_single_child) into separate functions. This created
a scope bug where _saved_tool_names was defined in _build_child_agent
but referenced in _run_single_child's finally block, causing a NameError
on every delegate_task call.

Solution: Move the save/restore logic entirely into _run_single_child,
keeping the save and restore in the same scope as the try/finally block.
This is cleaner than passing the variable through and removes the dead
save from _build_child_agent.
This commit is contained in:
Bartok9 2026-03-18 03:32:26 -04:00 • committed by Test
parent 6fc4e36625
commit ba7248c669
1 changed files with 7 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

11
tools/delegate_tool.py
View file

@ -173,10 +173,6 @@ def _build_child_agent(
from run_agent import AIAgent from run_agent import AIAgent
import model_tools import model_tools
# Save the parent's resolved tool names before the child agent can
# overwrite the process-global via get_tool_definitions().
_saved_tool_names = list(model_tools._last_resolved_tool_names)
# When no explicit toolsets given, inherit from parent's enabled toolsets # When no explicit toolsets given, inherit from parent's enabled toolsets
# so disabled tools (e.g. web) don't leak to subagents. # so disabled tools (e.g. web) don't leak to subagents.
if toolsets: if toolsets:
@ -263,6 +259,13 @@ def _run_single_child(
# Get the progress callback from the child agent # Get the progress callback from the child agent
child_progress_cb = getattr(child, 'tool_progress_callback', None) child_progress_cb = getattr(child, 'tool_progress_callback', None)
# Save the parent's resolved tool names before the child agent can
# overwrite the process-global via get_tool_definitions().
# This must be in _run_single_child (not _build_child_agent) so the
# save/restore happens in the same scope as the try/finally.
import model_tools
_saved_tool_names = list(model_tools._last_resolved_tool_names)
try: try:
result = child.run_conversation(user_message=goal) result = child.run_conversation(user_message=goal)