openclaw-navigator 4.3.4 → 4.3.5

package/cli.mjs

@@ -772,53 +772,73 @@ ${BOLD}How it works:${RESET}
     ok(`MCP server started (PID ${mcpProcess.pid}) — bridge: http://localhost:${port}`);
 
     // ── Auto-register MCP server with mcporter (if available) ─────────
-    // This makes the 15 navigator tools discoverable by the OC agent.
-    // Uses --scope home so it persists in ~/.mcporter/mcporter.json.
+    // Writes directly to ~/.mcporter/mcporter.json for reliability.
+    //
+    // KEY FIX: We resolve stable paths instead of using process.execPath
+    // which may point to a temp npx cache (/Users/x/.npm/_npx/<hash>/node)
+    // that disappears after npx exits. We find the REAL node binary and
+    // the globally-installed mcp.mjs (or realpath of the current one).
     try {
-      const reg = spawn(
-        "mcporter",
-        [
-          "config",
-          "add",
-          "navigator",
-          "--command",
-          process.execPath,
-          "--arg",
-          mcpPath,
-          "--env",
-          `NAVIGATOR_BRIDGE_URL=http://localhost:${port}`,
-          "--description",
-          "Navigator browser control (15 tools: navigate, click, fill, read, etc.)",
-          "--scope",
-          "home",
-        ],
-        { stdio: ["ignore", "pipe", "pipe"] },
-      );
-
-      let regOut = "";
-      reg.stdout?.on("data", (d) => {
-        regOut += d.toString();
-      });
-      reg.stderr?.on("data", (d) => {
-        regOut += d.toString();
-      });
+      const { readFileSync, writeFileSync, mkdirSync, realpathSync } = await import("node:fs");
+      const { homedir } = await import("node:os");
+
+      // Resolve stable node binary (follow symlinks out of npx temp dir)
+      let stableNode;
+      try {
+        stableNode = realpathSync(process.execPath);
+      } catch {
+        stableNode = process.execPath;
+      }
 
-      reg.on("close", (code) => {
-        if (code === 0) {
-          ok("Registered MCP server with mcporter (15 browser tools available to OC agent)");
-        } else if (regOut.includes("already exists")) {
-          info("  mcporter: navigator server already registered");
-        } else {
-          // mcporter not installed or failed — that's fine, not required
-          info("  mcporter not found — OC agent can still use the MCP server directly");
+      // Resolve stable mcp.mjs path
+      let stableMcpPath;
+      try {
+        stableMcpPath = realpathSync(mcpPath);
+      } catch {
+        stableMcpPath = mcpPath;
+      }
+
+      // If paths are still in npx temp cache, try to find the real install
+      if (stableMcpPath.includes("_npx") || stableMcpPath.includes(".npm/_npx")) {
+        // Check if openclaw-navigator is installed globally
+        const nodeDir = dirname(stableNode);
+        const globalMcp = join(dirname(nodeDir), "lib/node_modules/openclaw-navigator/mcp.mjs");
+        if (existsSync(globalMcp)) {
+          stableMcpPath = globalMcp;
         }
-      });
+        // If still temp, that's OK — mcporter will use the path we give it
+        // and the bridge running alongside keeps the MCP server alive anyway
+      }
 
-      reg.on("error", () => {
-        info("  mcporter not found — OC agent can still use the MCP server directly");
-      });
-    } catch {
-      // mcporter not installed — skip silently
+      info(`  mcporter: node=${stableNode}`);
+      info(`  mcporter: mcp=${stableMcpPath}`);
+
+      // Write directly to ~/.mcporter/mcporter.json
+      const mcporterDir = join(homedir(), ".mcporter");
+      const mcporterConfigPath = join(mcporterDir, "mcporter.json");
+
+      let mcporterConfig = { mcpServers: {}, imports: [] };
+      try {
+        mcporterConfig = JSON.parse(readFileSync(mcporterConfigPath, "utf8"));
+        if (!mcporterConfig.mcpServers) mcporterConfig.mcpServers = {};
+      } catch {
+        // File doesn't exist or invalid — start fresh
+      }
+
+      // Set/update the navigator entry with stable paths
+      mcporterConfig.mcpServers.navigator = {
+        command: stableNode,
+        args: [stableMcpPath],
+        env: { NAVIGATOR_BRIDGE_URL: `http://localhost:${port}` },
+      };
+
+      mkdirSync(mcporterDir, { recursive: true });
+      writeFileSync(mcporterConfigPath, JSON.stringify(mcporterConfig, null, 2) + "\n", "utf8");
+      ok("Registered MCP server with mcporter (15 browser tools available)");
+      info(`  Config: ${mcporterConfigPath}`);
+    } catch (err) {
+      warn(`mcporter registration failed: ${err.message}`);
+      info("  You can manually configure mcporter for Navigator MCP");
     }
 
     console.log("");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openclaw-navigator",
-  "version": "4.3.4",
+  "version": "4.3.5",
   "description": "One-command bridge + tunnel for the Navigator browser — works on any machine, any OS",
   "keywords": [
     "browser",