[Bug] planning_enabled permanently strips the director's SwarmSpec schema

HierarchicalSwarm.run_director and the async streaming loop both set
self.director.tools_list_dictionary = None before running the planning
sub-step.

That assignment cannot do what it looks like it does.
setup_director_with_planning builds its own throwaway Agent and already
excludes base_model and tools_list_dictionary from the settings it
forwards, so planning has always run schema-free without any help. The
line only reaches the real director -- the one the swarm depends on for
structured SwarmSpec output on the very next call, and the one the
caller may own when a director is passed in explicitly.

Delete both assignments. Add a test that pins the director's schema
across a planning-enabled run_director.

Author: ayaangazali

swarms/structs/hiearchical_swarm.py

@@ -762,7 +762,6 @@ def run_director(
         """
         try:
             if self.planning_enabled is True:
-                self.director.tools_list_dictionary = None
                 out = self.setup_director_with_planning(
                     task=f"History: {self.conversation.get_str()} \n\n Task: {task}",
                     img=img,
@@ -2000,7 +1999,6 @@ async def arun_stream(
             # Optional planning sub-step (non-streaming — creates a
             # throwaway agent with modified tools)
             if self.planning_enabled:
-                self.director.tools_list_dictionary = None
                 plan_out = await asyncio.to_thread(
                     self.setup_director_with_planning,
                     task=director_task_str,

tests/structs/test_hierarchical_swarm.py

@@ -1056,5 +1056,41 @@ def test_reassignment_targets_nested_swarm_worker_by_name():
     assert "[RECOVERY NOTICE]" not in swarm.conversation.get_str()
 
 
+def test_planning_keeps_the_director_tool_schema(monkeypatch):
+    """The planning sub-step must not disarm the real director.
+
+    setup_director_with_planning builds its own throwaway agent, so
+    clearing the director's tools_list_dictionary bought nothing and
+    cost the SwarmSpec schema the very next call depends on.
+    """
+    schema = [{"type": "function", "function": {"name": "SwarmSpec"}}]
+    director = StubAgent(
+        "Director",
+        [
+            {
+                "plan": "Hand the task to the worker.",
+                "orders": [
+                    {"agent_name": "Worker", "task": "do the thing"}
+                ],
+            }
+        ],
+    )
+    director.tools_list_dictionary = schema
+
+    swarm = make_recovery_swarm(
+        director, [StubAgent("Worker", ["done"])]
+    )
+    swarm.planning_enabled = True
+    monkeypatch.setattr(
+        swarm,
+        "setup_director_with_planning",
+        lambda task, img=None: "a plan",
+    )
+
+    swarm.run_director(task="ship it")
+
+    assert director.tools_list_dictionary == schema
+
+
 if __name__ == "__main__":
     pytest.main([__file__, "-v"])