negotium 0.1.25 → 0.1.26

package/dist/runtime/src/platform/playwright/headed-launch.ts

@@ -0,0 +1,61 @@
+import { accessSync, constants } from "node:fs";
+import { delimiter, isAbsolute, resolve } from "node:path";
+
+export interface HeadedPlaywrightSpawnSpec {
+  command: string;
+  args: string[];
+  virtualDisplay: boolean;
+}
+
+interface HeadedPlaywrightSpawnOptions {
+  platform?: NodeJS.Platform;
+  environment?: NodeJS.ProcessEnv;
+  findExecutable?: (command: string, environment: NodeJS.ProcessEnv) => string | null;
+}
+
+function findExecutableOnPath(
+  command: string,
+  environment: NodeJS.ProcessEnv = process.env,
+): string | null {
+  const candidates = isAbsolute(command)
+    ? [command]
+    : (environment.PATH ?? "")
+        .split(delimiter)
+        .filter(Boolean)
+        .map((directory) => resolve(directory, command));
+  for (const candidate of candidates) {
+    try {
+      accessSync(candidate, constants.X_OK);
+      return candidate;
+    } catch {
+      // Keep searching PATH.
+    }
+  }
+  return null;
+}
+
+/** Select the safe argv-only launcher for a visible browser MCP. */
+export function resolveHeadedPlaywrightSpawn(
+  command: string,
+  args: readonly string[],
+  options: HeadedPlaywrightSpawnOptions = {},
+): HeadedPlaywrightSpawnSpec {
+  const platform = options.platform ?? process.platform;
+  const environment = options.environment ?? process.env;
+  const hasDisplay = Boolean(environment.DISPLAY?.trim() || environment.WAYLAND_DISPLAY?.trim());
+  if (platform !== "linux" || hasDisplay) {
+    return { command, args: [...args], virtualDisplay: false };
+  }
+
+  const xvfbRun = (options.findExecutable ?? findExecutableOnPath)("xvfb-run", environment);
+  if (!xvfbRun) {
+    throw new Error(
+      "Headed Playwright on Linux requires DISPLAY/WAYLAND_DISPLAY or xvfb-run; install Xvfb and ensure xvfb-run is on PATH",
+    );
+  }
+  return {
+    command: xvfbRun,
+    args: ["-a", "-s", "-screen 0 1440x1000x24", command, ...args],
+    virtualDisplay: true,
+  };
+}

package/dist/runtime/src/platform/playwright/manager.ts

@@ -22,8 +22,11 @@ import {
   resolveBrowserProxy,
 } from "#platform/config";
 import { delay } from "#platform/delay";
-
 import { logger } from "#platform/logger";
+import {
+  type HeadedPlaywrightSpawnSpec,
+  resolveHeadedPlaywrightSpawn,
+} from "#platform/playwright/headed-launch";
 import { sanitizeTopicName } from "#security/sanitize";
 import {
   assignTopicBrowserProfile,
@@ -783,11 +786,25 @@ async function spawnPlaywright(
 
   const command = browserBin.endsWith(".mjs") ? process.execPath : browserBin;
   const args = browserBin.endsWith(".mjs") ? [browserBin, ...mcpArgs] : mcpArgs;
-  const proc = spawn(command, args, {
-    stdio: "ignore",
-    detached: false,
-    env: childEnv,
-  });
+  let spawnSpec: HeadedPlaywrightSpawnSpec;
+  try {
+    spawnSpec = resolveHeadedPlaywrightSpawn(command, args, { environment: childEnv });
+  } catch (err) {
+    releasePort(port);
+    throw err;
+  }
+  let proc: ChildProcess;
+  try {
+    proc = spawn(spawnSpec.command, spawnSpec.args, {
+      stdio: "ignore",
+      detached: false,
+      env: childEnv,
+    });
+  } catch (err) {
+    releasePort(port);
+    throw err;
+  }
+  const spawnError = waitForChildProcessSpawnError(proc);
 
   // When the MCP dies on its own (crash / OOM), its Chrome subtree is reparented
   // to init and escapes the tracked-instance map. Reap it by user-data-dir here
@@ -835,8 +852,11 @@ async function spawnPlaywright(
     capability,
   });
 
-  const ready =
-    (await waitForServer(port, 10_000)) && (await supportsOwnerCleanup(port, capability));
+  const ready = await Promise.race([
+    (async () =>
+      (await waitForServer(port, 10_000)) && (await supportsOwnerCleanup(port, capability)))(),
+    spawnError,
+  ]);
   if (!ready) {
     const exitCode = proc.exitCode;
     killInstance(instanceKey);
@@ -853,7 +873,10 @@ async function spawnPlaywright(
     );
   }
   writePortFile(instanceKey, port);
-  logger.info({ instanceKey, port, pid: proc.pid, ready }, "Playwright MCP started");
+  logger.info(
+    { instanceKey, port, pid: proc.pid, ready, virtualDisplay: spawnSpec.virtualDisplay },
+    "Playwright MCP started",
+  );
   return port;
 }
 
@@ -1234,6 +1257,13 @@ export function waitForChildProcessExit(proc: ChildProcess, timeoutMs: number):
   });
 }
 
+/** Reject immediately when the OS cannot launch a child process. */
+export function waitForChildProcessSpawnError(proc: Pick<ChildProcess, "once">): Promise<never> {
+  return new Promise<never>((_resolve, reject) => {
+    proc.once("error", reject);
+  });
+}
+
 // Proactively evict all idle instances every 30 minutes
 // Every 30 minutes: evict idle instances, then reap any orphaned browser the
 // tracked-instance map has lost sight of. The orphan reap is only a backstop —

package/dist/runtime/src/version.ts

@@ -1 +1 @@
-export const NEGOTIUM_VERSION = "0.1.25";
+export const NEGOTIUM_VERSION = "0.1.26";

package/dist/types/packages/core/src/version.d.ts

@@ -1 +1 @@
-export declare const NEGOTIUM_VERSION = "0.1.25";
+export declare const NEGOTIUM_VERSION = "0.1.26";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "negotium",
-  "version": "0.1.25",
+  "version": "0.1.26",
   "type": "module",
   "description": "Install the Negotium multi-agent runtime and CLI with one package",
   "license": "Apache-2.0",