mcp-optimizer 0.0.1-alpha.1 → 0.0.1

package/README.md

@@ -26,6 +26,21 @@ curl -X POST http://localhost:3000/audit -H "Content-Type: application/json" -d
 ```
 
 Notes
-- `src/runner/lighthouseRunner.ts` — runs Lighthouse via `chrome-launcher` and returns the LHR.
-- `src/fix/fixer.ts` — placeholder to convert LHR into actionable fixes; integrate LLM (e.g., via `@modelcontextprotocol/sdk`) here.
+This section provides an example of launching via an MCP host (stdio).
+
+If you want to configure an MCP host to spawn the optimizer via `npx`, add a server entry like the following to your host config:
+
+```json
+{
+	"mcpServers": {
+		"mcp-optimizer": {
+			"command": "npx",
+			"args": ["-y", "mcp-optimizer@0.0.6-alpha.1", "--","--port", "5000"]
+		}
+	}
+}
+```
+
+This will instruct the host to spawn `npx -y mcp-optimizer@0.0.6-alpha.1 -- --port 5000` and communicate with the child over stdio.
+
 

package/bin/mcp-optimizer.js

@@ -1,9 +1,37 @@
 #!/usr/bin/env node
 
 // Lightweight CLI wrapper for the MCP Optimizer package.
-// This file is intentionally small so `npx mcp-optimizer` can run it.
+// Accept `--port <n>` or `--port=<n>` (or `--audit-port`) to configure the server
+// port when launching via `npx` (e.g. `npx -y mcp-optimizer -- --port 6000`).
 
 try {
+  const args = process.argv.slice(2);
+  for (let i = 0; i < args.length; i++) {
+    const a = args[i];
+    // Accept environment-style args like PORT=6000
+    if (a.includes('=') && !a.startsWith('--')) {
+      const [k, v] = a.split('=', 2);
+      if (k && v !== undefined) {
+        process.env[k] = v;
+        continue;
+      }
+    }
+    if (a.startsWith('--port=')) {
+      process.env.PORT = a.split('=')[1];
+    }
+    else if (a === '--port' && args[i + 1]) {
+      process.env.PORT = args[i + 1];
+      i++;
+    }
+    else if (a.startsWith('--audit-port=')) {
+      process.env.AUDIT_PORT = a.split('=')[1];
+    }
+    else if (a === '--audit-port' && args[i + 1]) {
+      process.env.AUDIT_PORT = args[i + 1];
+      i++;
+    }
+  }
+
   // Require built output. When installed via npm the package root
   // will contain `dist/index.js` after `npm pack` / `npm publish`.
   require('../dist/index.js');

package/dist/mcpServer.js

@@ -39,6 +39,7 @@ const mcp_js_1 = require("@modelcontextprotocol/sdk/server/mcp.js");
 const zod_1 = require("zod");
 const lighthouseRunner_1 = require("./runner/lighthouseRunner");
 const sse_js_1 = require("@modelcontextprotocol/sdk/server/sse.js");
+const stdio_js_1 = require("@modelcontextprotocol/sdk/server/stdio.js");
 const http = __importStar(require("http"));
 const fixer_1 = require("./fix/fixer");
 class LighthouseMcpServer {
@@ -205,9 +206,72 @@ exports.LighthouseMcpServer = LighthouseMcpServer;
 exports.default = LighthouseMcpServer;
 async function startMcpServer() {
     // Run HTTP/SSE server
-    const port = Number(process.env.PORT || process.env.AUDIT_PORT || 5000);
+    // Prefer explicit environment variables, but also allow parsing CLI args
+    // (e.g. when launched directly without the lightweight `bin` wrapper).
+    let portEnv = process.env.PORT || process.env.AUDIT_PORT;
+    if (!portEnv) {
+        const args = process.argv.slice(2);
+        for (let i = 0; i < args.length; i++) {
+            const a = args[i];
+            if (a.includes('=') && !a.startsWith('--')) {
+                const [k, v] = a.split('=', 2);
+                if ((k === 'PORT' || k === 'AUDIT_PORT') && v !== undefined) {
+                    portEnv = v;
+                    break;
+                }
+            }
+            if (a.startsWith('--port=')) {
+                portEnv = a.split('=')[1];
+                break;
+            }
+            else if (a === '--port' && args[i + 1]) {
+                portEnv = args[i + 1];
+                break;
+            }
+            else if (a.startsWith('--audit-port=')) {
+                portEnv = a.split('=')[1];
+                break;
+            }
+            else if (a === '--audit-port' && args[i + 1]) {
+                portEnv = args[i + 1];
+                break;
+            }
+        }
+    }
+    const port = Number(portEnv || 5000);
     const mcp = new LighthouseMcpServer();
     let sseTransport = null;
+    let stdioTransport = null;
+    // If process stdin/stdout appear to be non-TTY pipes, assume we're being
+    // 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.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.
+            return new Promise((resolve, reject) => {
+                stdioTransport.onclose = () => resolve();
+                stdioTransport.onerror = (err) => reject(err);
+            });
+        }
+        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 = [];
     const { Readable, Writable } = await (async () => {
         const mod = await Promise.resolve().then(() => __importStar(require('stream')));

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.1-alpha.1",
+  "version": "0.0.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

@@ -3,6 +3,7 @@ import { z } from "zod";
 import { runLighthouseAudit } from "./runner/lighthouseRunner";
 import { Transport } from "@modelcontextprotocol/sdk/shared/transport.js";
 import { SSEServerTransport } from "@modelcontextprotocol/sdk/server/sse.js";
+import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import * as http from 'http';
 import { IncomingMessage, ServerResponse } from 'http';
 import { autoFixFromReport } from './fix/fixer';
@@ -192,9 +193,69 @@ export default LighthouseMcpServer;
 
 export async function startMcpServer(): Promise<void> {
   // Run HTTP/SSE server
-  const port = Number(process.env.PORT || process.env.AUDIT_PORT || 5000);
+  // Prefer explicit environment variables, but also allow parsing CLI args
+  // (e.g. when launched directly without the lightweight `bin` wrapper).
+  let portEnv = process.env.PORT || process.env.AUDIT_PORT;
+  if (!portEnv) {
+    const args = process.argv.slice(2);
+    for (let i = 0; i < args.length; i++) {
+      const a = args[i];
+      if (a.includes('=') && !a.startsWith('--')) {
+        const [k, v] = a.split('=', 2);
+        if ((k === 'PORT' || k === 'AUDIT_PORT') && v !== undefined) {
+          portEnv = v;
+          break;
+        }
+      }
+      if (a.startsWith('--port=')) {
+        portEnv = a.split('=')[1];
+        break;
+      } else if (a === '--port' && args[i + 1]) {
+        portEnv = args[i + 1];
+        break;
+      } else if (a.startsWith('--audit-port=')) {
+        portEnv = a.split('=')[1];
+        break;
+      } else if (a === '--audit-port' && args[i + 1]) {
+        portEnv = args[i + 1];
+        break;
+      }
+    }
+  }
+
+  const port = Number(portEnv || 5000);
   const mcp = new LighthouseMcpServer();
   let sseTransport: SSEServerTransport | null = null;
+  let stdioTransport: StdioServerTransport | null = null;
+  // If process stdin/stdout appear to be non-TTY pipes, assume we're being
+  // 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.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.
+      return new Promise((resolve, reject) => {
+        stdioTransport!.onclose = () => resolve();
+        stdioTransport!.onerror = (err: any) => reject(err);
+      });
+    } 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 }> = [];
 
   const { Readable, Writable } = await (async () => {

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));

package/dist/server.js

@@ -1,79 +0,0 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.startHttpServer = startHttpServer;
-const http = __importStar(require("http"));
-const lighthouseRunner_1 = require("./runner/lighthouseRunner");
-const fixer_1 = require("./fix/fixer");
-async function startHttpServer(port) {
-    const p = Number(port ?? process.env.PORT ?? process.env.AUDIT_PORT ?? 5000);
-    const requestHandler = async (req, res) => {
-        if (req.method === 'POST' && req.url === '/audit') {
-            let body = '';
-            for await (const chunk of req) {
-                body += chunk;
-            }
-            const parsed = JSON.parse(body || '{}');
-            const url = parsed.url;
-            if (!url) {
-                res.writeHead(400);
-                res.end('missing url');
-                return;
-            }
-            try {
-                const report = await (0, lighthouseRunner_1.runLighthouseAudit)(url);
-                const fix = await (0, fixer_1.autoFixFromReport)(report);
-                res.writeHead(200, { 'Content-Type': 'application/json' });
-                res.end(JSON.stringify({ summary: report.lhr?.categories || null, fix }));
-            }
-            catch (err) {
-                res.writeHead(500);
-                res.end(String(err));
-            }
-        }
-        else {
-            res.writeHead(200, { 'Content-Type': 'text/plain' });
-            res.end('MCP Optimizer running — POST /audit { "url": "https://..." }');
-        }
-    };
-    const server = http.createServer(requestHandler);
-    return new Promise((resolve, reject) => {
-        server.listen(p, () => {
-            console.log(`Server listening on ${p}`);
-            resolve(server);
-        });
-        server.on('error', reject);
-    });
-}