Make MCP stdio clean for Glama

Author: PsYcGoD

src/sage/mcp/__init__.py

@@ -1,6 +1,25 @@
-"""MCP server integration for SAGE."""
+"""MCP server integration for SAGE.
 
-from .server import MCPServer
-from .tools import SAGE_TOOLS
+The package keeps imports lazy so ``python -m sage.mcp.server`` starts with a
+clean module state. Eagerly importing ``sage.mcp.server`` from here causes
+Python's runpy module to emit a RuntimeWarning before the MCP stdio server
+starts, which is noisy for strict MCP registry and Glama checks.
+"""
+
+from __future__ import annotations
+
+from typing import Any
 
 __all__ = ["MCPServer", "SAGE_TOOLS"]
+
+
+def __getattr__(name: str) -> Any:
+    if name == "MCPServer":
+        from .server import MCPServer
+
+        return MCPServer
+    if name == "SAGE_TOOLS":
+        from .tools import SAGE_TOOLS
+
+        return SAGE_TOOLS
+    raise AttributeError(name)

src/sage/mcp/server.py

@@ -44,6 +44,10 @@
 MIN_IDLE_TIMEOUT_SECONDS = 10
 
 
+def _verbose_stderr_enabled() -> bool:
+    return os.getenv("SAGE_MCP_VERBOSE", "").strip().lower() in {"1", "true", "yes", "on"}
+
+
 def _mcp_idle_timeout() -> int:
     """Return the MCP stdio idle timeout.
 
@@ -169,11 +173,12 @@ def _idle_watchdog(self) -> None:
             time.sleep(1)
             if time.monotonic() - self._last_activity < self.idle_timeout:
                 continue
-            print(
-                f"[SAGE MCP Server] idle for {self.idle_timeout}s; exiting",
-                file=sys.stderr,
-                flush=True,
-            )
+            if _verbose_stderr_enabled():
+                print(
+                    f"[SAGE MCP Server] idle for {self.idle_timeout}s; exiting",
+                    file=sys.stderr,
+                    flush=True,
+                )
             self._running = False
             self._remove_session()
             os._exit(0)
@@ -274,11 +279,12 @@ def run(self):
         self._cleanup_session_records()
         self._write_session()
         self._start_idle_watchdog()
-        print(
-            f"[SAGE MCP Server] stdio ready (idle timeout {self.idle_timeout}s)",
-            file=sys.stderr,
-            flush=True,
-        )
+        if _verbose_stderr_enabled():
+            print(
+                f"[SAGE MCP Server] stdio ready (idle timeout {self.idle_timeout}s)",
+                file=sys.stderr,
+                flush=True,
+            )
 
         try:
             for line in sys.stdin: