@vortex-os/computer-use 0.2.0 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vortex-os/computer-use",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Add-on — read-only screen perception (structured UIA tree + pixel fallback + change watch) exposed as an MCP server, layered on @vortex-os/base. Windows-first. Control (mouse/keyboard) is intentionally out of scope.",
   "license": "MIT",
   "author": "vortex-os-project",

package/scripts/mcp-stdio.mjs

@@ -5,11 +5,8 @@
 //   - default: run the stdio server (what an MCP host launches).
 //   - `install`: self-register into the project `.mcp.json` under the non-reserved key
 //     `vortex-computer-use` (merge-safe). e.g. `npx vortex-mcp-computer-use install`.
-// Optional dep: @modelcontextprotocol/sdk (declared optional; imported by this entry — `install`
-// just registers and exits without connecting a transport).
-import { Server } from '@modelcontextprotocol/sdk/server/index.js';
-import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
-import { ListToolsRequestSchema, CallToolRequestSchema } from '@modelcontextprotocol/sdk/types.js';
+// Optional dep: @modelcontextprotocol/sdk — loaded DYNAMICALLY only on the serve path (see the
+// bottom dispatch), so `install` registers and exits without needing the SDK present.
 import { spawnSync, spawn } from 'node:child_process';
 import { fileURLToPath } from 'node:url';
 import { dirname, join } from 'node:path';
@@ -412,11 +409,8 @@ const TOOLS = [
   },
 ];
 
-const server = new Server({ name: 'computer-use', version: '0.0.1-poc' }, { capabilities: { tools: {} } });
-
-server.setRequestHandler(ListToolsRequestSchema, async () => ({ tools: TOOLS }));
-
-server.setRequestHandler(CallToolRequestSchema, async (req) => {
+// The CallTool handler — standalone (no SDK types), wired to the server only on the serve path.
+async function handleCallTool(req) {
   const { name, arguments: a = {} } = req.params;
   const useWorker = plat === 'win32';   // the resident worker is Windows PowerShell backend only
   let result;
@@ -538,7 +532,7 @@ server.setRequestHandler(CallToolRequestSchema, async (req) => {
   } finally {
     if (reqDir) { try { rmSync(reqDir, { recursive: true, force: true }); } catch {} }
   }
-});
+}
 
 // Clean up the worker on server shutdown (it would also auto-terminate via stdin EOF when the parent dies, but do it explicitly).
 process.on('exit', () => workerMgr.dispose());
@@ -611,7 +605,13 @@ function runInstall(argv) {
 if (process.argv.slice(2).includes('install')) {
   runInstall(process.argv.slice(2));
 } else {
-  const transport = new StdioServerTransport();
-  await server.connect(transport);
+  // Serve path: load the MCP SDK dynamically (so `install` never requires it), wire the handlers, connect.
+  const { Server } = await import('@modelcontextprotocol/sdk/server/index.js');
+  const { StdioServerTransport } = await import('@modelcontextprotocol/sdk/server/stdio.js');
+  const { ListToolsRequestSchema, CallToolRequestSchema } = await import('@modelcontextprotocol/sdk/types.js');
+  const server = new Server({ name: 'computer-use', version: '0.2.1' }, { capabilities: { tools: {} } });
+  server.setRequestHandler(ListToolsRequestSchema, async () => ({ tools: TOOLS }));
+  server.setRequestHandler(CallToolRequestSchema, handleCallTool);
+  await server.connect(new StdioServerTransport());
   process.stderr.write(`[computer-use MCP] ready on stdio (worker=${plat === 'win32' ? 'on' : 'off'}; tools: probe, read_ui, capture_screen, watch_capture, poll_change, beep)\n`);
 }