do not log to stdio for daemon processes.

Author: Davidyz

src/vectorcode/cli_utils.py

@@ -3,6 +3,7 @@
 import json
 import logging
 import os
+import sys
 from dataclasses import dataclass, field, fields
 from datetime import datetime
 from enum import Enum, StrEnum
@@ -488,7 +489,9 @@ async def expand_globs(
     return list(result)
 
 
-def config_logging(name: str = "vectorcode"):  # pragma: nocover
+def config_logging(
+    name: str = "vectorcode", stdio: bool = True, log_file: bool = False
+):  # pragma: nocover
     """Configure the logging module. This should be called before a `main` function (CLI, LSP or MCP)."""
 
     logging.root.handlers = []
@@ -503,36 +506,37 @@ def config_logging(name: str = "vectorcode"):  # pragma: nocover
             )
 
     handlers = []
-    if level is not None:
+    if level is not None or log_file:
         log_dir = os.path.expanduser("~/.local/share/vectorcode/logs/")
         os.makedirs(log_dir, exist_ok=True)
         # File handler
         log_file_path = os.path.join(
             log_dir, f"{name}-{datetime.now().strftime('%Y-%m-%d_%H-%M-%S')}.log"
         )
         file_handler = logging.FileHandler(log_file_path)
-        file_handler.setLevel(level)
+        file_handler.setLevel(level or logging.WARN)
         handlers.append(file_handler)
 
-    import colorlog
-
-    # Console handler
-    console_handler = colorlog.StreamHandler()
-    console_handler.setFormatter(
-        colorlog.ColoredFormatter(
-            fmt="%(log_color)s%(levelname)s%(reset)s: %(name)s : %(message)s",
-            log_colors={
-                "DEBUG": "cyan",
-                "INFO": "green",
-                "WARNING": "yellow",
-                "ERROR": "red",
-                "CRITICAL": "bold_red",
-            },
-            reset=True,
+    if stdio:
+        import colorlog
+
+        # Console handler
+        console_handler = colorlog.StreamHandler(sys.stderr)
+        console_handler.setFormatter(
+            colorlog.ColoredFormatter(
+                fmt="%(log_color)s%(levelname)s%(reset)s: %(name)s : %(message)s",
+                log_colors={
+                    "DEBUG": "cyan",
+                    "INFO": "green",
+                    "WARNING": "yellow",
+                    "ERROR": "red",
+                    "CRITICAL": "bold_red",
+                },
+                reset=True,
+            )
         )
-    )
-    console_handler.setLevel(level or logging.WARN)
-    handlers.append(console_handler)
+        console_handler.setLevel(level or logging.WARN)
+        handlers.append(console_handler)
 
     logging.basicConfig(
         handlers=handlers,

src/vectorcode/lsp_main.py

@@ -179,7 +179,7 @@ async def lsp_start() -> int:
 
 
 def main():  # pragma: nocover
-    config_logging("vectorcode-lsp-server")
+    config_logging("vectorcode-lsp-server", stdio=False)
     asyncio.run(lsp_start())
 
 

src/vectorcode/mcp_main.py

@@ -20,6 +20,7 @@
 
 from vectorcode.cli_utils import (
     Config,
+    config_logging,
     find_project_config_dir,
     get_project_config,
     load_config_file,
@@ -145,6 +146,7 @@ async def run_server():  # pragma: nocover
 
 
 def main():  # pragma: nocover
+    config_logging("vectorcode-mcp-server", stdio=False)
     return asyncio.run(run_server())