mcp-optimizer 0.0.4-alpha.1 → 0.0.6-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/dist/runner/lighthouseRunner.js

@@ -30,7 +30,9 @@ async function runLighthouseAudit(url, opts) {
         // Run via npx to ensure local package is used
         const cmd = process.platform === 'win32' ? 'npx.cmd' : 'npx';
         const reportJson = await new Promise((resolve, reject) => {
-            execFile(cmd, args, { maxBuffer: 10 * 1024 * 1024 }, (err, stdout, stderr) => {
+            // Use a shell on Windows to ensure .cmd/.bat wrappers are executed
+            // correctly (avoids spawn EINVAL in some environments).
+            execFile(cmd, args, { maxBuffer: 10 * 1024 * 1024, shell: true, env: process.env }, (err, stdout, stderr) => {
                 if (err) {
                     const message = stderr || (err && err.message) || String(err);
                     return reject(new Error(message));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-optimizer",
-  "version": "0.0.4-alpha.1",
+  "version": "0.0.6-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 }> = [];

package/src/runner/lighthouseRunner.ts

@@ -31,7 +31,9 @@ export async function runLighthouseAudit(
     const cmd = process.platform === 'win32' ? 'npx.cmd' : 'npx';
 
     const reportJson: string = await new Promise((resolve, reject) => {
-      execFile(cmd, args, { maxBuffer: 10 * 1024 * 1024 }, (err: any, stdout: string, stderr: string) => {
+      // Use a shell on Windows to ensure .cmd/.bat wrappers are executed
+      // correctly (avoids spawn EINVAL in some environments).
+      execFile(cmd, args, { maxBuffer: 10 * 1024 * 1024, shell: true, env: process.env }, (err: any, stdout: string, stderr: string) => {
         if (err) {
           const message = stderr || (err && err.message) || String(err);
           return reject(new Error(message));