mcp-optimizer 0.0.4-alpha.1 → 0.0.5-alpha.1

package/dist/mcpServer.js

@@ -246,10 +246,15 @@ async function startMcpServer() {
     // launched as a stdio child by an MCP host and use the stdio transport.
     const shouldUseStdio = (process.stdin && !process.stdin.isTTY) && (process.stdout && !process.stdout.isTTY);
     if (shouldUseStdio) {
+        // Redirect console output to stderr to avoid corrupting the JSON stdio protocol
+        const _orig = { log: console.log, info: console.info, warn: console.warn };
+        console.log = (...args) => { process.stderr.write(args.map(String).join(' ') + '\n'); };
+        console.info = console.log;
+        console.warn = console.log;
         try {
             stdioTransport = new stdio_js_1.StdioServerTransport(process.stdin, process.stdout);
             await mcp.connect(stdioTransport);
-            console.info('Stdio: connected to parent process over stdio');
+            console.error('Stdio: connected to parent process over stdio');
             // When running over stdio we do not start the HTTP server; stay alive
             // and let the parent coordinate messages. Return a promise that
             // resolves when the transport closes.
@@ -261,6 +266,10 @@ async function startMcpServer() {
         catch (err) {
             console.error('Stdio: failed to start transport, falling back to HTTP:', err);
             stdioTransport = null;
+            // restore console in case we fall back to HTTP server mode
+            console.log = _orig.log;
+            console.info = _orig.info;
+            console.warn = _orig.warn;
         }
     }
     const pendingPosts = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-optimizer",
-  "version": "0.0.4-alpha.1",
+  "version": "0.0.5-alpha.1",
   "description": "An MCP server that runs Lighthouse audits and enables LLMs to automatically optimize your code.",
   "main": "dist/index.js",
   "bin": {

package/src/mcpServer.ts

@@ -231,10 +231,15 @@ export async function startMcpServer(): Promise<void> {
   // launched as a stdio child by an MCP host and use the stdio transport.
   const shouldUseStdio = (process.stdin && !process.stdin.isTTY) && (process.stdout && !process.stdout.isTTY);
   if (shouldUseStdio) {
+    // Redirect console output to stderr to avoid corrupting the JSON stdio protocol
+    const _orig = { log: console.log, info: console.info, warn: console.warn };
+    console.log = (...args: any[]) => { process.stderr.write(args.map(String).join(' ') + '\n'); };
+    console.info = console.log;
+    console.warn = console.log;
     try {
       stdioTransport = new StdioServerTransport(process.stdin, process.stdout);
       await mcp.connect(stdioTransport as unknown as Transport);
-      console.info('Stdio: connected to parent process over stdio');
+      console.error('Stdio: connected to parent process over stdio');
       // When running over stdio we do not start the HTTP server; stay alive
       // and let the parent coordinate messages. Return a promise that
       // resolves when the transport closes.
@@ -245,6 +250,10 @@ export async function startMcpServer(): Promise<void> {
     } catch (err) {
       console.error('Stdio: failed to start transport, falling back to HTTP:', err);
       stdioTransport = null;
+      // restore console in case we fall back to HTTP server mode
+      console.log = _orig.log;
+      console.info = _orig.info;
+      console.warn = _orig.warn;
     }
   }
   const pendingPosts: Array<{ body: string; url: string | undefined; headers: any }> = [];