refactor: rename server/tools/admin.py to stats.py

 only contained read-only reporting tools (,
), not actual admin operations. The name  better
reflects its read-only introspection purpose and disambiguates from
's  tool.

Author: GoodbyePlanet

.claude/CLAUDE.md

@@ -57,7 +57,7 @@ All server code lives under `server/`. The system has two main phases: **ingesti
 - `store/qdrant.py` runs hybrid search via Qdrant `FusionQuery(fusion=RRF)` — combines dense and BM25 sparse results
   using Reciprocal Rank Fusion
 - `store/commit_store.py` handles commit search (dense-only)
-- `tools/` contains the MCP tool implementations: `search.py`, `index.py`, `history.py`, `admin.py`
+- `tools/` contains the MCP tool implementations: `search.py`, `index.py`, `history.py`, `stats.py`
 
 ### MCP interface (`server/main.py`)
 

server/main.py

@@ -92,15 +92,15 @@ async def combined(scope_app: Starlette) -> AsyncIterator[None]:
 def main() -> None:
     from server.tools.search import register_search_tools
     from server.tools.index import register_index_tools
-    from server.tools.admin import register_admin_tools
+    from server.tools.stats import register_stats_tools
     from server.tools.history import register_history_tools
     from server.prompts.service import register_service_prompts
     from server.prompts.system import register_system_prompts
     from server.routes.reindex import register_http_routes
 
     register_search_tools(mcp)
     register_index_tools(mcp)
-    register_admin_tools(mcp)
+    register_stats_tools(mcp)
     register_history_tools(mcp)
     register_service_prompts(mcp)
     register_system_prompts(mcp)

server/tools/admin.py server/tools/stats.pyserver/tools/admin.py renamed to server/tools/stats.py

@@ -11,7 +11,7 @@
 logger = logging.getLogger(__name__)
 
 
-def register_admin_tools(mcp: FastMCP) -> None:
+def register_stats_tools(mcp: FastMCP) -> None:
 
     @mcp.tool()
     async def list_indexed_services() -> str:

tests/tools/test_admin.py tests/tools/test_stats.pytests/tools/test_admin.py renamed to tests/tools/test_stats.py

@@ -3,20 +3,20 @@
 from unittest.mock import AsyncMock, patch
 
 from server.config import ServiceConfig
-from server.tools.admin import register_admin_tools
+from server.tools.stats import register_stats_tools
 from tests.tools.conftest import get_tool
 
 
 def _tool(name: str):
-    return get_tool(register_admin_tools, name)
+    return get_tool(register_stats_tools, name)
 
 
 async def test_list_indexed_services_reports_when_empty() -> None:
     list_indexed_services = _tool("list_indexed_services")
     store = AsyncMock()
     store.get_service_stats.return_value = []
 
-    with patch("server.tools.admin.get_store", return_value=store):
+    with patch("server.tools.stats.get_store", return_value=store):
         result = await list_indexed_services()
 
     assert "No services indexed yet" in result
@@ -42,7 +42,7 @@ async def test_list_indexed_services_formats_stats_sorted_by_name() -> None:
         },
     ]
 
-    with patch("server.tools.admin.get_store", return_value=store):
+    with patch("server.tools.stats.get_store", return_value=store):
         result = await list_indexed_services()
 
     assert result.index("alpha") < result.index("zebra")
@@ -55,8 +55,8 @@ async def test_index_stats_reports_qdrant_error() -> None:
     store.collection_info.side_effect = RuntimeError("connection refused")
 
     with (
-        patch("server.tools.admin.get_store", return_value=store),
-        patch("server.tools.admin.settings") as mock_settings,
+        patch("server.tools.stats.get_store", return_value=store),
+        patch("server.tools.stats.settings") as mock_settings,
     ):
         mock_settings.load_services.return_value = []
         result = await index_stats()
@@ -77,8 +77,8 @@ async def test_index_stats_includes_provider_endpoint() -> None:
     svc = ServiceConfig(name="orders", github_repo="org/orders", exclude=[])
 
     with (
-        patch("server.tools.admin.get_store", return_value=store),
-        patch("server.tools.admin.settings") as mock_settings,
+        patch("server.tools.stats.get_store", return_value=store),
+        patch("server.tools.stats.settings") as mock_settings,
     ):
         mock_settings.load_services.return_value = [svc]
         mock_settings.embeddings_provider = "voyage"