chore: prevent logging to stdout, reserved for MCP JSON-RPC protocol

Author: KNedelec

packages/developer_mcp_server/src/developer_mcp_server/server.py

@@ -2,21 +2,19 @@
 
 import logging
 
+from gg_api_core import configure_mcp_logging
 from gg_api_core.mcp_server import get_mcp_server
 from gg_api_core.scopes import set_developer_scopes
 
 from developer_mcp_server.add_health_check import add_health_check
 from developer_mcp_server.register_tools import DEVELOPER_INSTRUCTIONS, register_developer_tools
 
-# Configure more detailed logging
-logging.basicConfig(level=logging.DEBUG, format="%(asctime)s - %(name)s - %(levelname)s - %(message)s")
-
+configure_mcp_logging()
 logger = logging.getLogger(__name__)
 
 # Use our custom GitGuardianFastMCP from the core package
 mcp = get_mcp_server(
     "GitGuardian Developer",
-    log_level="DEBUG",
     instructions=DEVELOPER_INSTRUCTIONS,
 )
 

packages/gg_api_core/src/gg_api_core/__init__.py

@@ -1,3 +1,19 @@
 """Core functionality for GitGuardian MCP servers."""
 
+import logging
+import sys
+
 __version__ = "0.1.0"
+
+
+def configure_mcp_logging(level: int = logging.INFO) -> None:
+    """Configure logging to use stderr (stdout is reserved for MCP JSON-RPC protocol)."""
+    logging.basicConfig(
+        level=level,
+        format="%(asctime)s - %(name)s - %(levelname)s - %(message)s",
+        stream=sys.stderr,
+        force=True,
+    )
+    # Reduce noise from library loggers
+    for name in ("mcp", "fastmcp", "rich"):
+        logging.getLogger(name).setLevel(logging.WARNING)

packages/secops_mcp_server/src/secops_mcp_server/server.py

@@ -17,9 +17,7 @@
 from gg_api_core.tools.write_custom_tags import update_or_create_incident_custom_tags, write_custom_tags
 from pydantic import BaseModel, Field
 
-# Configure more detailed logging
-logging.basicConfig(level=logging.DEBUG, format="%(asctime)s - %(name)s - %(levelname)s - %(message)s")
-
+configure_mcp_logging()
 logger = logging.getLogger(__name__)
 
 
@@ -113,7 +111,6 @@ class ListHoneytokensParams(BaseModel):
 # Use our custom GitGuardianFastMCP from the core package
 mcp = get_mcp_server(
     "GitGuardian SecOps",
-    log_level="DEBUG",
     instructions=SECOPS_INSTRUCTIONS,
 )