fix: remove underscore prefix from tool method names

FunctionTool derives the tool name from the function's __name__.
Methods named _search_ards etc. registered as "_search_ards" in the
agent, causing "Tool '_search_ards' not found" errors.

Renamed all tool methods to their public names (search_ards,
search_agents, etc.) so FunctionTool picks up the correct names
without needing a post-hoc override.

Author: zeroasterisk

src/google/adk_community/tools/ardhf/ardhf_toolset.py

@@ -304,7 +304,7 @@ async def _do_search(
 
   # -- Tool implementations -----------------------------------------------
 
-  async def _search_ards(
+  async def search_ards(
       self,
       tool_context: ToolContext,
       query: str,
@@ -330,7 +330,7 @@ async def _search_ards(
         query, artifact_type=artifact_type, limit=limit
     )
 
-  async def _search_agents(
+  async def search_agents(
       self,
       tool_context: ToolContext,
       query: str,
@@ -349,7 +349,7 @@ async def _search_agents(
     """
     return await self._do_search(query, artifact_type="a2a", limit=limit)
 
-  async def _search_skills(
+  async def search_skills(
       self,
       tool_context: ToolContext,
       query: str,
@@ -370,7 +370,7 @@ async def _search_skills(
         query, artifact_type="skill", limit=limit
     )
 
-  async def _search_tools(
+  async def search_tools(
       self,
       tool_context: ToolContext,
       query: str,
@@ -389,7 +389,7 @@ async def _search_tools(
     """
     return await self._do_search(query, artifact_type="mcp", limit=limit)
 
-  async def _search_spaces(
+  async def search_spaces(
       self,
       tool_context: ToolContext,
       query: str,
@@ -410,7 +410,7 @@ async def _search_spaces(
         query, artifact_type="space", limit=limit
     )
 
-  async def _get_agent_card(
+  async def get_agent_card(
       self,
       tool_context: ToolContext,
       url: str,
@@ -448,7 +448,7 @@ async def _get_agent_card(
           "content_type": "text/markdown",
       }
 
-  async def _connect_agent(
+  async def connect_agent(
       self,
       tool_context: ToolContext,
       agent_card_url: str,
@@ -568,20 +568,16 @@ async def get_tools(
       readonly_context: ReadonlyContext | None = None,
   ) -> list[BaseTool]:
     """Return all search, inspect, and connect tools."""
-    tool_defs = [
-        (self._search_ards, "search_ards"),
-        (self._search_agents, "search_agents"),
-        (self._search_skills, "search_skills"),
-        (self._search_tools, "search_tools"),
-        (self._search_spaces, "search_spaces"),
-        (self._get_agent_card, "get_agent_card"),
-        (self._connect_agent, "connect_agent"),
+    tool_funcs = [
+        self.search_ards,
+        self.search_agents,
+        self.search_skills,
+        self.search_tools,
+        self.search_spaces,
+        self.get_agent_card,
+        self.connect_agent,
     ]
-    tools: list[BaseTool] = []
-    for func, name in tool_defs:
-      tool = FunctionTool(func)
-      tool.name = name
-      tools.append(tool)
+    tools: list[BaseTool] = [FunctionTool(func) for func in tool_funcs]
 
     if readonly_context is not None:
       tools = [

tests/unittests/tools/ardhf/test_ardhf_toolset.py

@@ -179,7 +179,7 @@ async def test_search_agents_delegates_with_a2a_type(self):
     ) as mock_search:
       mock_search.return_value = {"results": []}
       mock_context = AsyncMock()
-      await toolset._search_agents(
+      await toolset.search_agents(
           mock_context, query="test", limit=5
       )
 
@@ -197,7 +197,7 @@ async def test_search_skills_delegates_with_skill_type(self):
     ) as mock_search:
       mock_search.return_value = {"results": []}
       mock_context = AsyncMock()
-      await toolset._search_skills(
+      await toolset.search_skills(
           mock_context, query="test", limit=5
       )
 
@@ -215,7 +215,7 @@ async def test_search_tools_delegates_with_mcp_type(self):
     ) as mock_search:
       mock_search.return_value = {"results": []}
       mock_context = AsyncMock()
-      await toolset._search_tools(
+      await toolset.search_tools(
           mock_context, query="test", limit=5
       )
 
@@ -233,7 +233,7 @@ async def test_search_spaces_delegates_with_space_type(self):
     ) as mock_search:
       mock_search.return_value = {"results": []}
       mock_context = AsyncMock()
-      await toolset._search_spaces(
+      await toolset.search_spaces(
           mock_context, query="test", limit=5
       )
 
@@ -305,7 +305,7 @@ async def test_get_agent_card_rejects_file_url(self):
     toolset = AgentFinderToolset()
     mock_context = AsyncMock()
 
-    result = await toolset._get_agent_card(
+    result = await toolset.get_agent_card(
         mock_context, url="file:///etc/passwd"
     )
 
@@ -318,7 +318,7 @@ async def test_connect_agent_rejects_file_url(self):
     toolset = AgentFinderToolset()
     mock_context = AsyncMock()
 
-    result = await toolset._connect_agent(
+    result = await toolset.connect_agent(
         mock_context,
         agent_card_url="ftp://example.com/agent.json",
         message="hello",
@@ -342,7 +342,7 @@ async def test_returns_parsed_json_for_json_content(self):
         return_value=json_content,
     ):
       mock_context = AsyncMock()
-      result = await toolset._get_agent_card(
+      result = await toolset.get_agent_card(
           mock_context, url="https://example.com/tool.json"
       )
 
@@ -361,7 +361,7 @@ async def test_returns_markdown_for_non_json_content(self):
         return_value=md_content,
     ):
       mock_context = AsyncMock()
-      result = await toolset._get_agent_card(
+      result = await toolset.get_agent_card(
           mock_context,
           url="https://example.com/SKILL.md",
       )