From 4908b422b46534cd62ed6c254b11e9a6f9805b35 Mon Sep 17 00:00:00 2001
From: root <root@luna.joylo.com>
Date: Thu, 9 Oct 2025 06:55:19 +0200
Subject: [PATCH] Fix: Disable logging in stdio mode to prevent MCP protocol
 errors
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Detect stdio mode and set logger to 'silent' level
- stdout must be reserved exclusively for JSON-RPC MCP messages
- Prevents 'Unexpected token' JSON parsing errors in Claude Code
- Only log when using HTTP transport mode

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 src/utils/logger.ts | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/utils/logger.ts b/src/utils/logger.ts
index dde431f..6ffaf4c 100644
--- a/src/utils/logger.ts
+++ b/src/utils/logger.ts
@@ -1,9 +1,12 @@
 import pino from 'pino';
 import { config } from '../config';
 
+// Detect if running in stdio mode (stdout must be reserved for MCP protocol)
+const isStdioMode = !process.env.MCP_TRANSPORT || process.env.MCP_TRANSPORT === 'stdio';
+
 export const logger = pino({
-  level: config.mcp.logLevel,
-  transport: {
+  level: isStdioMode ? 'silent' : config.mcp.logLevel,
+  transport: isStdioMode ? undefined : {
     target: 'pino-pretty',
     options: {
       colorize: true,
@@ -11,6 +14,10 @@ export const logger = pino({
       ignore: 'pid,hostname',
     },
   },
+  // In stdio mode, send any logs to stderr instead of stdout
+  ...(isStdioMode && {
+    browser: { write: () => {} },
+  }),
 });
 
 export function createLogger(name: string): pino.Logger {