trim comments to the load-bearing lines

Author: ayaangazali

swarms/agents/autonomous_loop.py

@@ -213,10 +213,8 @@ def _run_autonomous_loop(
 
             self._say_user(task)
 
-            # Add planning tools to tools_list_dictionary.
-            # "lazy" ships a small always-on core plus the search_tools
-            # meta-tool; the rest are fetched on demand, which keeps the
-            # resident schema block from being re-sent on every iteration.
+            # "lazy" ships a small core plus search_tools; the rest are
+            # fetched on demand, so the resident block stays flat.
             if self.agent.selected_tools == "lazy":
                 planning_tools = get_lazy_autonomous_tools()
                 logger.info(
@@ -364,9 +362,7 @@ def _run_autonomous_loop(
                 )
             )
 
-            # Filter tool handlers if selected_tools is not "all".
-            # "lazy" keeps every handler: the model can load any schema at
-            # run time, so the handler has to be there when it does.
+            # "lazy" keeps every handler: any schema can be loaded at run time.
             if (
                 self.agent.selected_tools != "all"
                 and self.agent.selected_tools != "lazy"

swarms/structs/autonomous_loop_utils.py

@@ -594,10 +594,8 @@ def get_autonomous_planning_tools() -> List[Dict[str, Any]]:
     ]
 
 
-# The always-resident set under selected_tools="lazy". Everything the loop
-# structurally depends on (planning, subtask bookkeeping, termination, and the
-# user-facing reply), plus the meta-tool used to fetch the rest. Keeping this
-# small is the point: the resident schema cost stays flat as tools are added.
+# Always resident under selected_tools="lazy": what the loop structurally
+# depends on, plus the meta-tool that fetches the rest.
 LAZY_CORE_TOOL_NAMES = (
     "create_plan",
     "subtask_done",
@@ -769,9 +767,7 @@ def search_tools_tool(agent: Any, query: str = "", **kwargs) -> str:
         agent.tools_list_dictionary = []
     agent.tools_list_dictionary.extend(matches)
 
-    # Rebuild so the newly added schemas reach the provider. Without this the
-    # client keeps the tool list it was constructed with and the model would
-    # be told about tools it cannot actually call.
+    # Rebuild, or the client keeps the tool list it was constructed with.
     try:
         agent.llm = agent.llm_handling()
     except Exception as e:

tests/structs/test_async_subagent.py

@@ -1283,11 +1283,6 @@ def run(self, task):
     print("=" * 60)
 
 
-# ============================================================================
-# Lazy autonomous tool loading (#1753)
-# ============================================================================
-
-
 class _ToolAgentStub:
     """Minimal stand-in exposing what search_tools_tool touches."""
 
@@ -1324,7 +1319,6 @@ def test_lazy_core_is_a_strict_subset_and_much_smaller():
 
     full_size = len(json.dumps(full))
     lazy_size = len(json.dumps(lazy))
-    # The whole point is a large reduction, not a marginal one.
     assert lazy_size < full_size * 0.5
 
 
@@ -1393,7 +1387,6 @@ def test_repeating_a_search_does_not_reload_or_dump_the_catalogue():
 
     assert len(agent.tools_list_dictionary) == count
     assert "No tool matched" in message
-    # The fallback lists names, which is cheap, rather than loading schemas.
     assert "think" in message