pin down dependencies + fix mcp/agent template (#1429)

Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
Co-authored-by: Eitan Yarmush <eitan.yarmush@solo.io>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: 3 people

go/core/cli/internal/agent/frameworks/adk/python/templates/pyproject.toml.tmpl

@@ -4,5 +4,7 @@ version = "0.1"
 description = "{{.Name}} agent"
 readme = "README.md"
 dependencies = [
-  "google-adk>=1.8.0",
+  "google-adk==1.25.1",
 ]
+
+requires-python = ">=3.13"

go/core/cli/internal/mcp/frameworks/python/templates/.python-version.tmpl

@@ -1 +1 @@
-3.11 
+3.12

go/core/cli/internal/mcp/frameworks/python/templates/Dockerfile.tmpl

@@ -1,5 +1,5 @@
 # Multi-stage build for {{.ProjectName}} MCP server using uv
-FROM python:3.11-slim AS builder
+FROM python:3.12-slim AS builder
 
 # Install uv
 COPY --from=ghcr.io/astral-sh/uv:latest /uv /usr/local/bin/uv
@@ -26,7 +26,7 @@ COPY src/ ./src/
 COPY manifest.yaml ./
 
 # Production stage
-FROM python:3.11-slim
+FROM python:3.12-slim
 
 # Install uv in production
 COPY --from=ghcr.io/astral-sh/uv:latest /uv /usr/local/bin/uv

go/core/cli/internal/mcp/frameworks/python/templates/pyproject.toml.tmpl

@@ -12,12 +12,12 @@ description = "{{.ProjectName}} MCP server built with FastMCP"
     {name = "Unknown Author"}
 ]{{end}}{{end}}
 readme = "README.md"
-requires-python = ">=3.10"
+requires-python = ">=3.12"
 dependencies = [
-    "fastmcp>=0.2.0",
-    "pydantic>=2.0.0",
-    "pyyaml>=6.0",
-    "python-dotenv>=1.0.0",
+    "fastmcp==3.0.0",
+    "pydantic==2.12.5",
+    "pyyaml==6.0.3",
+    "python-dotenv==1.2.1",
 ]
 
 [build-system]
@@ -32,12 +32,12 @@ packages = ["src"]
 
 [tool.uv]
 dev-dependencies = [
-    "pytest>=7.0.0",
-    "pytest-asyncio>=0.21.0",
-    "black>=22.0.0",
-    "mypy>=1.0.0",
-    "ruff>=0.1.0",
-    "types-PyYAML>=6.0.0",
+    "pytest==9.0.2",
+    "pytest-asyncio==1.3.0",
+    "black==26.1.0",
+    "mypy==1.19.1",
+    "ruff==0.15.1",
+    "types-PyYAML==6.0.12.20250915",
 ]
 
 [tool.black]

go/core/cli/internal/mcp/frameworks/python/templates/src/core/server.py.tmpl

@@ -4,6 +4,7 @@ This server automatically discovers and loads tools from the tools directory.
 Each tool file should contain a function decorated with @mcp.tool().
 """
 
+import asyncio
 import importlib.util
 import logging
 import sys
@@ -75,14 +76,14 @@ class DynamicMCPServer:
         for tool_file in tool_files:
             try:
                 # Get the number of tools before importing
-                tools_before = len(self.mcp._tool_manager._tools)
+                tools_before = len(asyncio.run(self.mcp.list_tools()))
 
                 # Simply import the module - tools auto-register via @mcp.tool()
                 # decorator
                 tool_name = tool_file.stem
                 if self._import_tool_module(tool_file, tool_name):
                     # Check if any tools were actually registered
-                    tools_after = len(self.mcp._tool_manager._tools)
+                    tools_after = len(asyncio.run(self.mcp.list_tools()))
                     if tools_after > tools_before:
                         self.loaded_tools.append(tool_name)
                         loaded_count += 1
@@ -140,10 +141,21 @@ class DynamicMCPServer:
             return False
 
     def get_tools_sync(self) -> dict[str, Any]:
-        """Get tools synchronously for testing purposes."""
-        # This is a simplified version for testing - in real usage, use get_tools()
-        # async
-        return self.mcp._tool_manager._tools
+        """Get tools synchronously for testing purposes.
+
+        This prefers returning the internal tool mapping (name -> tool object)
+        to preserve compatibility with existing tests/code, and falls back to
+        using list_tools() if no such mapping is available.
+        """
+        # Prefer an internal mapping if FastMCP exposes one (e.g., for tests
+        # that rely on direct access to the underlying tool objects/callables).
+        internal_tools = getattr(self.mcp, "tools", None)
+        if isinstance(internal_tools, dict):
+            return internal_tools
+
+        # Fallback: build a mapping from the async list_tools() API.
+        tools_list = asyncio.run(self.mcp.list_tools())
+        return {t.name: t for t in tools_list}
 
     def run(self, transport_mode: str = "stdio", host: str = "localhost", port: int = 3000) -> None:
         """Run the FastMCP server.

go/core/cli/internal/mcp/frameworks/python/templates/src/main.py.tmpl

@@ -7,13 +7,16 @@ Each tool file should contain a function decorated with @mcp.tool().
 Usage Examples:
   # Stdio mode (default MCP transport)
   python src/main.py
-  
+
   # HTTP mode with MCP protocol over HTTP
   python src/main.py --transport http
-  
+
   # Custom host/port
   python src/main.py --transport http --host localhost --port 8080
-  
+
+  # Stateless HTTP mode
+  python src/main.py --transport http --stateless-http
+
   # Environment variable mode
   MCP_TRANSPORT_MODE=http python src/main.py
 """
@@ -52,6 +55,18 @@ def main() -> None:
         help="Port to bind to in HTTP mode (default: 3000)"
     )
 
+    stateless_http_default = os.getenv("MCP_STATELESS_HTTP", "false").lower() == "true"
+
+    parser.add_argument(
+        "--stateless-http",
+        action=argparse.BooleanOptionalAction,
+        default=stateless_http_default,
+        help=(
+            "Enable stateless HTTP mode (use --no-stateless-http to disable; "
+            "default is derived from MCP_STATELESS_HTTP env var)"
+        ),
+    )
+
     args = parser.parse_args()
 
     # Check environment variable for transport mode
@@ -78,8 +93,11 @@ def main() -> None:
 
         if transport_mode not in ["http", "stdio"]:
             raise ValueError(f"Invalid transport mode: {transport_mode}. Must be one of: http, or stdio")
-        
-        server.run(transport_mode=transport_mode, host=args.host, port=args.port)
+
+        server.run(transport_mode=transport_mode,
+                   host=args.host,
+                   port=args.port,
+                   stateless_http=args.stateless_http)
 
     except KeyboardInterrupt:
         print("\nShutting down server...")