fix: respect PLUGINS_LOG_LEVEL environment variable in all runtime.py files (#48)

* fix: respect PLUGINS_LOG_LEVEL environment variable in all runtime.py files

- Updated grpc/server/runtime.py to check PLUGINS_LOG_LEVEL env var before command-line arg
- Updated mcp/server/runtime.py to add logging configuration with PLUGINS_LOG_LEVEL support
- Updated unix/server/runtime.py to respect PLUGINS_LOG_LEVEL instead of hardcoded INFO level
- All implementations now log to stderr and follow consistent pattern

* refactor: use get_settings().log_level instead of hardcoded env var

Address PR review feedback to use the settings infrastructure instead of
directly accessing environment variables. This approach:
- Keeps the env var name defined in one place (PluginsSettings model)
- Gets .env file support for free
- Stays consistent with how other settings are read in the codebase

Changes:
- grpc/server/runtime.py: Use get_settings().log_level with fallback to args
- mcp/server/runtime.py: Use get_settings().log_level
- unix/server/runtime.py: Use get_settings().log_level

---------

Co-authored-by: Bogdan-Marius-Catanus <bogdan-marius.catanus@ibm.com>

Author: bogdanmariusc10

cpex/framework/external/grpc/server/runtime.py

@@ -289,10 +289,14 @@ def main() -> None:
 
     args = parser.parse_args()
 
-    # Configure logging
+    # Configure logging - respect PLUGINS_LOG_LEVEL environment variable
+    settings = get_settings()
+    log_level_str = settings.log_level or args.log_level
+    log_level = getattr(logging, log_level_str.upper(), logging.INFO)
     logging.basicConfig(
-        level=getattr(logging, args.log_level),
+        level=log_level,
         format="%(asctime)s - %(name)s - %(levelname)s - %(message)s",
+        stream=sys.stderr,
     )
 
     # Run the server

cpex/framework/external/mcp/server/runtime.py

@@ -79,7 +79,18 @@
     MCP_SERVER_INSTRUCTIONS,
     MCP_SERVER_NAME,
 )
-from cpex.framework.settings import get_transport_settings
+
+# Configure logging - respect PLUGINS_LOG_LEVEL environment variable
+from cpex.framework.settings import get_settings, get_transport_settings
+
+settings = get_settings()
+log_level_str = settings.log_level
+log_level = getattr(logging, log_level_str.upper(), logging.INFO)
+logging.basicConfig(
+    level=log_level,
+    format="%(asctime)s - %(name)s - %(levelname)s - %(message)s",
+    stream=sys.stderr,
+)
 
 logger = logging.getLogger(__name__)
 

cpex/framework/external/unix/server/runtime.py

@@ -33,9 +33,12 @@
 from cpex.framework.external.unix.server.server import run_server
 from cpex.framework.settings import get_settings
 
-# Configure logging
+# Configure logging - respect PLUGINS_LOG_LEVEL environment variable
+settings = get_settings()
+log_level_str = settings.log_level
+log_level = getattr(logging, log_level_str.upper(), logging.INFO)
 logging.basicConfig(
-    level=logging.INFO,
+    level=log_level,
     format="%(asctime)s - %(name)s - %(levelname)s - %(message)s",
     stream=sys.stderr,  # Log to stderr to keep stdout clean for coordination
 )