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

This reverts commit 03bcf59.

Author: KNedelec

packages/developer_mcp_server/src/developer_mcp_server/server.py

@@ -2,19 +2,21 @@
 
 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_mcp_logging()
+# Configure more detailed logging
+logging.basicConfig(level=logging.DEBUG, format="%(asctime)s - %(name)s - %(levelname)s - %(message)s")
+
 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,19 +1,3 @@
 """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

@@ -6,7 +6,6 @@
 from developer_mcp_server.add_health_check import add_health_check
 from developer_mcp_server.register_tools import register_developer_tools
 from fastmcp.exceptions import ToolError
-from gg_api_core import configure_mcp_logging
 from gg_api_core.mcp_server import get_mcp_server, register_common_tools
 from gg_api_core.scopes import set_secops_scopes
 from gg_api_core.tools.assign_incident import assign_incident
@@ -18,7 +17,9 @@
 from gg_api_core.tools.write_custom_tags import update_or_create_incident_custom_tags, write_custom_tags
 from pydantic import BaseModel, Field
 
-configure_mcp_logging()
+# Configure more detailed logging
+logging.basicConfig(level=logging.DEBUG, format="%(asctime)s - %(name)s - %(levelname)s - %(message)s")
+
 logger = logging.getLogger(__name__)
 
 
@@ -112,6 +113,7 @@ class ListHoneytokensParams(BaseModel):
 # Use our custom GitGuardianFastMCP from the core package
 mcp = get_mcp_server(
     "GitGuardian SecOps",
+    log_level="DEBUG",
     instructions=SECOPS_INSTRUCTIONS,
 )