fixed info endpoints (#9)

Author: ayomaska18

src/adapter/routes.py

@@ -98,7 +98,7 @@ def _get_tool_categories(self) -> dict[str, list[dict[str, Any]]]:
         categories: dict[str, list[dict[str, Any]]] = {}
 
         for tool_name, tool in self.tools.items():
-            parts = tool_name.split("/", 1)
+            parts = tool_name.split("_", 1)
             category = parts[0] if len(parts) > 1 else "uncategorized"
             short_name = parts[1] if len(parts) > 1 else tool_name
 
@@ -151,9 +151,9 @@ async def get_category_tools(category: str):
                 ],
             }
 
-        @app.get(f"{self.route_prefix}/{{category}}/{{tool_name:path}}", tags=["Info"])
+        @app.get(f"{self.route_prefix}/{{category}}/{{tool_name}}", tags=["Info"])
         async def get_tool_info(category: str, tool_name: str):
-            full_name = f"{category}/{tool_name}"
+            full_name = f"{category}_{tool_name}"
             if full_name not in self.tools:
                 raise HTTPException(
                     status_code=404, detail=f"Tool '{full_name}' not found"

tests/adapter/test_routes.py

@@ -22,7 +22,7 @@ def _create_route_generator(self, tools: dict) -> RouteGenerator:
 
     def test_single_category_single_tool(self):
         tools = {
-            "calculator/add": self._create_mock_tool("calculator/add"),
+            "calculator_add": self._create_mock_tool("calculator_add"),
         }
         generator = self._create_route_generator(tools)
 
@@ -31,13 +31,13 @@ def test_single_category_single_tool(self):
         assert "calculator" in categories
         assert len(categories["calculator"]) == 1
         assert categories["calculator"][0]["name"] == "add"
-        assert categories["calculator"][0]["full_name"] == "calculator/add"
+        assert categories["calculator"][0]["full_name"] == "calculator_add"
 
     def test_single_category_multiple_tools(self):
         tools = {
-            "calculator/add": self._create_mock_tool("calculator/add"),
-            "calculator/subtract": self._create_mock_tool("calculator/subtract"),
-            "calculator/multiply": self._create_mock_tool("calculator/multiply"),
+            "calculator_add": self._create_mock_tool("calculator_add"),
+            "calculator_subtract": self._create_mock_tool("calculator_subtract"),
+            "calculator_multiply": self._create_mock_tool("calculator_multiply"),
         }
         generator = self._create_route_generator(tools)
 
@@ -48,9 +48,9 @@ def test_single_category_multiple_tools(self):
 
     def test_multiple_categories(self):
         tools = {
-            "calculator/add": self._create_mock_tool("calculator/add"),
-            "filesystem/read": self._create_mock_tool("filesystem/read"),
-            "shell/execute": self._create_mock_tool("shell/execute"),
+            "calculator_add": self._create_mock_tool("calculator_add"),
+            "filesystem_read": self._create_mock_tool("filesystem_read"),
+            "shell_execute": self._create_mock_tool("shell_execute"),
         }
         generator = self._create_route_generator(tools)
 
@@ -66,20 +66,20 @@ def test_multiple_categories(self):
 
     def test_uncategorized_tool(self):
         tools = {
-            "simple_tool": self._create_mock_tool("simple_tool"),
+            "simpletool": self._create_mock_tool("simpletool"),
         }
         generator = self._create_route_generator(tools)
 
         categories = generator._get_tool_categories()
 
         assert "uncategorized" in categories
         assert len(categories["uncategorized"]) == 1
-        assert categories["uncategorized"][0]["name"] == "simple_tool"
+        assert categories["uncategorized"][0]["name"] == "simpletool"
 
     def test_mixed_categorized_and_uncategorized(self):
         tools = {
-            "calculator/add": self._create_mock_tool("calculator/add"),
-            "simple_tool": self._create_mock_tool("simple_tool"),
+            "calculator_add": self._create_mock_tool("calculator_add"),
+            "simpletool": self._create_mock_tool("simpletool"),
         }
         generator = self._create_route_generator(tools)
 
@@ -91,17 +91,16 @@ def test_mixed_categorized_and_uncategorized(self):
 
     def test_nested_category_path(self):
         tools = {
-            "data/csv/read": self._create_mock_tool("data/csv/read"),
+            "data_csv_read": self._create_mock_tool("data_csv_read"),
         }
         generator = self._create_route_generator(tools)
 
         categories = generator._get_tool_categories()
 
         assert "data" in categories
         assert len(categories["data"]) == 1
-        # The sub_tool should be "csv/read" (everything after first /)
-        assert categories["data"][0]["name"] == "csv/read"
-        assert categories["data"][0]["full_name"] == "data/csv/read"
+        assert categories["data"][0]["name"] == "csv_read"
+        assert categories["data"][0]["full_name"] == "data_csv_read"
 
     def test_empty_tools(self):
         tools = {}
@@ -113,31 +112,31 @@ def test_empty_tools(self):
 
     def test_tool_endpoint_path(self):
         tools = {
-            "calculator/add": self._create_mock_tool("calculator/add"),
+            "calculator_add": self._create_mock_tool("calculator_add"),
         }
         generator = self._create_route_generator(tools)
 
         categories = generator._get_tool_categories()
 
-        expected_endpoint = "/tools/calculator/add"
+        expected_endpoint = "/tools/calculator_add"
         assert categories["calculator"][0]["endpoint"] == expected_endpoint
 
     def test_custom_route_prefix(self):
         mock_client = MagicMock()
         generator = RouteGenerator(client=mock_client, route_prefix="/api/v1/tools")
         generator.tools = {
-            "calculator/add": self._create_mock_tool("calculator/add"),
+            "calculator_add": self._create_mock_tool("calculator_add"),
         }
 
         categories = generator._get_tool_categories()
 
-        expected_endpoint = "/api/v1/tools/calculator/add"
+        expected_endpoint = "/api/v1/tools/calculator_add"
         assert categories["calculator"][0]["endpoint"] == expected_endpoint
 
     def test_tool_description_preserved(self):
         tools = {
-            "calculator/add": self._create_mock_tool(
-                "calculator/add", description="Add two numbers together"
+            "calculator_add": self._create_mock_tool(
+                "calculator_add", description="Add two numbers together"
             ),
         }
         generator = self._create_route_generator(tools)