sunholo 0.144.8__tar.gz → 0.144.9__tar.gz

{sunholo-0.144.8/src/sunholo.egg-info → sunholo-0.144.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sunholo
-Version: 0.144.8
+Version: 0.144.9
 Summary: AI DevOps - a package to help deploy GenAI to the Cloud.
 Author-email: Holosun ApS <multivac@sunholo.com>
 License: Apache License, Version 2.0

{sunholo-0.144.8 → sunholo-0.144.9}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "sunholo"
-version = "0.144.8"
+version = "0.144.9"
 description = "AI DevOps - a package to help deploy GenAI to the Cloud."
 readme = "README.md"
 requires-python = ">=3.11"

{sunholo-0.144.8 → sunholo-0.144.9}/src/sunholo/agents/fastapi/vac_routes.py

@@ -353,7 +353,6 @@ class VACRoutesFastAPI:
         additional_routes: Optional[List[Dict]] = None,
         mcp_servers: Optional[List[Dict[str, Any]]] = None,
         add_langfuse_eval: bool = True,
-        enable_mcp_server: bool = False,
         enable_a2a_agent: bool = False,
         a2a_vac_names: Optional[List[str]] = None
     ):
@@ -371,9 +370,6 @@ class VACRoutesFastAPI:
             mcp_servers: List of external MCP server configurations to connect to:
                        [{"name": "server-name", "command": "python", "args": ["server.py"]}]
             add_langfuse_eval: Whether to enable Langfuse evaluation and tracing
-            enable_mcp_server: Whether to enable the MCP server at /mcp endpoint for 
-                             Claude Desktop/Code integration. When True, automatically
-                             includes built-in VAC tools and supports custom tool registration.
             enable_a2a_agent: Whether to enable A2A (Agent-to-Agent) protocol endpoints
             a2a_vac_names: List of VAC names available for A2A agent interactions
         
@@ -398,7 +394,7 @@ class VACRoutesFastAPI:
         
         ## MCP Server Integration
         
-        When enable_mcp_server=True, the following happens:
+        When VACMCPServer is available, the following happens automatically:
         1. MCP server is mounted at /mcp endpoint
         2. Built-in VAC tools are automatically registered:
            - vac_stream, vac_query, list_available_vacs, get_vac_info
@@ -452,13 +448,13 @@ class VACRoutesFastAPI:
         self.mcp_client_manager = MCPClientManager() if MCPClientManager else None
         self._mcp_initialized = False
         
-        # MCP server initialization
-        self.enable_mcp_server = enable_mcp_server
+        # MCP server initialization - automatically enabled if VACMCPServer is available
         self.vac_mcp_server = None
         self._custom_mcp_tools = []
         self._custom_mcp_resources = []
         
-        if self.enable_mcp_server and VACMCPServer:
+        # Enable MCP server if VACMCPServer is available
+        if VACMCPServer:
             self.vac_mcp_server = VACMCPServer(
                 server_name="sunholo-vac-fastapi-server",
                 include_vac_tools=True
@@ -497,7 +493,7 @@ class VACRoutesFastAPI:
             stream_interpreter: Streaming interpreter function
             vac_interpreter: Non-streaming interpreter function  
             app_lifespan: Optional app lifespan context manager
-            **kwargs: Additional arguments passed to VACRoutesFastAPI (except enable_mcp_server)
+            **kwargs: Additional arguments passed to VACRoutesFastAPI
             
         Returns:
             Tuple of (FastAPI app, VACRoutesFastAPI instance)
@@ -569,7 +565,6 @@ class VACRoutesFastAPI:
                 app,
                 stream_interpreter=stream_interpreter,
                 vac_interpreter=vac_interpreter,
-                enable_mcp_server=False,  # Don't enable again since we manually mounted
                 **kwargs
             )
             
@@ -589,7 +584,6 @@ class VACRoutesFastAPI:
                 app,
                 stream_interpreter=stream_interpreter,
                 vac_interpreter=vac_interpreter,
-                enable_mcp_server=False,
                 **kwargs
             )
         
@@ -694,7 +688,7 @@ class VACRoutesFastAPI:
             self.app.post("/mcp/resources/read")(self.handle_mcp_read_resource)
         
         # MCP server endpoint - mount the FastMCP app
-        if self.enable_mcp_server and self.vac_mcp_server:
+        if self.vac_mcp_server:
             try:
                 mcp_app = self.vac_mcp_server.get_http_app()
                 
@@ -713,8 +707,7 @@ class VACRoutesFastAPI:
                         "The FastAPI app must be created with the MCP lifespan.\n\n"
                         "Quick fix: Use the helper method:\n"
                         "  app, vac_routes = VACRoutesFastAPI.create_app_with_mcp(\n"
-                        "      stream_interpreter=your_interpreter,\n"
-                        "      enable_mcp_server=True\n"
+                        "      stream_interpreter=your_interpreter\n"
                         "  )\n\n"
                         "Or manually configure the lifespan - see documentation for details."
                     )
@@ -727,8 +720,8 @@ class VACRoutesFastAPI:
                 log.error(f"Failed to mount MCP server: {e}")
                 raise RuntimeError(f"MCP server initialization failed: {e}") from e
         
-        # MCP debug endpoint - add after MCP server mounting
-        if self.enable_mcp_server:
+        # MCP debug endpoint - add if we have a MCP server instance
+        if self.vac_mcp_server:
             self.app.get("/debug/mcp")(self.handle_mcp_debug)
         
         # A2A agent endpoints
@@ -1620,7 +1613,7 @@ class VACRoutesFastAPI:
         pending_tools = len(self._custom_mcp_tools) if hasattr(self, '_custom_mcp_tools') else 0
         
         return {
-            "mcp_enabled": self.enable_mcp_server or has_mcp,
+            "mcp_enabled": has_mcp,
             "has_mcp_server": has_mcp,
             "mcp_tools_count": len(mcp_tools),
             "mcp_tools": [tool["name"] for tool in mcp_tools] if mcp_tools else [],

{sunholo-0.144.8 → sunholo-0.144.9/src/sunholo.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sunholo
-Version: 0.144.8
+Version: 0.144.9
 Summary: AI DevOps - a package to help deploy GenAI to the Cloud.
 Author-email: Holosun ApS <multivac@sunholo.com>
 License: Apache License, Version 2.0

{sunholo-0.144.8 → sunholo-0.144.9}/tests/test_vac_routes_mcp.py

@@ -214,7 +214,6 @@ def mcp_server_app():
         app,
         stream_interpreter=mock_async_stream_interpreter,
         vac_interpreter=mock_async_vac_interpreter,
-        enable_mcp_server=False,  # Disable initially
         add_langfuse_eval=False
     )
     
@@ -253,7 +252,6 @@ def mcp_client_app():
         stream_interpreter=mock_async_stream_interpreter,
         vac_interpreter=mock_async_vac_interpreter,
         mcp_servers=mcp_servers,
-        enable_mcp_server=False,
         add_langfuse_eval=False
     )
     # Replace with mock MCP client manager
@@ -320,7 +318,6 @@ class TestMCPServer:
         vac_routes = VACRoutesFastAPI(
             app,
             stream_interpreter=mock_async_stream_interpreter,
-            enable_mcp_server=False,
             add_langfuse_eval=False
         )
         client = TestClient(app)
@@ -331,6 +328,64 @@ class TestMCPServer:
 
 
 
+class TestMCPDebugEndpoint:
+    """Test MCP debug endpoint functionality."""
+    
+    def test_debug_endpoint_with_mcp_server(self):
+        """Test /debug/mcp endpoint when MCP server is enabled."""
+        app, vac_routes = VACRoutesFastAPI.create_app_with_mcp(
+            title="Test App",
+            stream_interpreter=mock_async_stream_interpreter
+        )
+        
+        # Add a mock tool to test
+        @vac_routes.add_mcp_tool
+        async def test_tool(input: str) -> str:
+            """Test tool for MCP."""
+            return f"Test: {input}"
+        
+        client = TestClient(app)
+        response = client.get("/debug/mcp")
+        
+        assert response.status_code == 200
+        data = response.json()
+        
+        # Check basic structure
+        assert "mcp_enabled" in data
+        assert "has_mcp_server" in data
+        assert "mcp_tools_count" in data
+        assert "mcp_tools" in data
+        
+        # Check values
+        assert data["mcp_enabled"] is True
+        assert data["has_mcp_server"] is True
+        assert data["mcp_tools_count"] >= 5  # Built-in tools + test_tool
+        
+        # Check that test_tool is in the list
+        tool_names = [tool["name"] for tool in data["mcp_tools"]]
+        assert "test_tool" in tool_names
+        
+        # Check built-in tools are present
+        assert "vac_stream" in tool_names
+        assert "vac_query" in tool_names
+        assert "list_available_vacs" in tool_names
+        assert "get_vac_info" in tool_names
+    
+    def test_debug_endpoint_without_mcp_server(self):
+        """Test /debug/mcp endpoint when MCP server is not present."""
+        app = FastAPI()
+        vac_routes = VACRoutesFastAPI(
+            app,
+            stream_interpreter=mock_async_stream_interpreter,
+            add_langfuse_eval=False
+        )
+        
+        # Should not have debug endpoint if no MCP server
+        client = TestClient(app)
+        response = client.get("/debug/mcp")
+        assert response.status_code == 404
+
+
 class TestMCPErrorHandling:
     """Test MCP error handling."""