update after comment of enyst

Author: VascoSch92

openhands-sdk/openhands/sdk/subagent/registry.py

@@ -146,7 +146,7 @@ def _factory(llm: "LLM") -> "Agent":
         from openhands.sdk.tool.spec import Tool
 
         # Load LLM profile if agent_def.model is different from
-        # 'inherit' and it was given
+        # 'inherit' and empty string
         if agent_def.model and agent_def.model != "inherit":
             store = _get_profile_store()
             available_profiles = [name.removesuffix(".json") for name in store.list()]
@@ -159,12 +159,6 @@ def _factory(llm: "LLM") -> "Agent":
             profile_name = agent_def.model
             llm = store.load(profile_name)
 
-        # Ensure sub-agent LLM always has streaming disabled and
-        # reset metrics such that the sub-agent has its own
-        # Metrics object
-        llm = llm.model_copy(update={"stream": False})
-        llm.reset_metrics()
-
         # the system prompt of the subagent is added as a suffix of the
         # main system prompt
         agent_context = (

openhands-tools/openhands/tools/delegate/impl.py

@@ -122,8 +122,20 @@ def _spawn_agents(self, action: "DelegateAction") -> DelegateObservation:
             ]
 
             for agent_id, agent_type in zip(action.ids, resolved_agent_types):
+                sub_agent_llm = parent_llm.model_copy()
+                # resetting metrics such that the sub-agent has its own
+                # Metrics object
+                sub_agent_llm.reset_metrics()
+
                 factory = get_agent_factory(name=agent_type)
-                worker_agent = factory.factory_func(parent_llm)
+                worker_agent = factory.factory_func(sub_agent_llm)
+
+                # ensuring that the sub-agent LLM has stream deactivated
+                worker_agent = worker_agent.model_copy(
+                    update={
+                        "llm": worker_agent.llm.model_copy(update={"stream": False})
+                    }
+                )
 
                 # Use parent visualizer's create_sub_visualizer method if available
                 # This allows custom visualizers (e.g., TUI-based) to create

openhands-tools/openhands/tools/task/manager.py

@@ -238,9 +238,19 @@ def _get_sub_agent(self, subagent_type: str) -> Agent:
         """
         parent = self.parent_conversation
         parent_llm = parent.agent.llm
+        sub_agent_llm = parent_llm.model_copy()
+        # Reset metrics such that the sub-agent has its own
+        # Metrics object
+        sub_agent_llm.reset_metrics()
 
         factory = get_agent_factory(subagent_type)
-        return factory.factory_func(parent_llm)
+        sub_agent = factory.factory_func(sub_agent_llm)
+
+        # ensuring that the sub-agent LLM has stream deactivated
+        sub_agent = sub_agent.model_copy(
+            update={"llm": sub_agent.llm.model_copy(update={"stream": False})}
+        )
+        return sub_agent
 
     def _run_task(self, task: Task, prompt: str) -> Task:
         """Run a task synchronously."""