@mrrlin-dev/mcp 0.4.22 → 0.4.23

package/dist/bin.cjs

@@ -20815,6 +20815,66 @@ function browserServeArgs(homeDir = import_node_os.default.homedir()) {
     env: { PLAYWRIGHT_BROWSERS_PATH: browsersPath }
   };
 }
+function defaultListProcesses() {
+  const r = (0, import_node_child_process.spawnSync)("ps", ["-ww", "-eo", "pid,ppid,command"], { encoding: "utf8" });
+  const lines2 = (r.stdout ?? "").split("\n").slice(1);
+  const out = [];
+  for (const line of lines2) {
+    const m = /^\s*(\d+)\s+(\d+)\s+(.*)$/.exec(line);
+    if (!m || m[1] === void 0 || m[2] === void 0 || m[3] === void 0) continue;
+    out.push({ pid: Number(m[1]), ppid: Number(m[2]), command: m[3] });
+  }
+  return out;
+}
+function defaultIsAlive(pid) {
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch (err) {
+    return err.code === "EPERM";
+  }
+}
+function defaultKill(pid, signal) {
+  try {
+    process.kill(pid, signal);
+    return true;
+  } catch (err) {
+    if (err.code === "ESRCH") return true;
+    return false;
+  }
+}
+function safeKill(kill, pid, signal) {
+  try {
+    return kill(pid, signal);
+  } catch {
+    return false;
+  }
+}
+function reapOrphanedBrowserProcesses(opts = {}) {
+  const homeDir = opts.homeDir ?? import_node_os.default.homedir();
+  const { profilePath } = browserPaths(homeDir);
+  const listProcesses = opts.listProcesses ?? defaultListProcesses;
+  const isAlive = opts.isAlive ?? defaultIsAlive;
+  const kill = opts.kill ?? defaultKill;
+  const selfPid = opts.selfPid ?? process.pid;
+  const allProcesses = listProcesses();
+  const candidates = allProcesses.filter(
+    (p) => p.pid !== selfPid && p.command.includes("@playwright/mcp") && p.command.includes(profilePath)
+  );
+  const killedPids = [];
+  const killedChildPids = [];
+  for (const proc of candidates) {
+    if (!isAlive(proc.ppid)) {
+      for (const child of allProcesses) {
+        if (child.ppid === proc.pid) {
+          if (safeKill(kill, child.pid, "SIGKILL")) killedChildPids.push(child.pid);
+        }
+      }
+      if (safeKill(kill, proc.pid, "SIGTERM")) killedPids.push(proc.pid);
+    }
+  }
+  return { checked: candidates.length, killedPids, killedChildPids };
+}
 function externalAgentsServeArgs() {
   return {
     command: process.execPath,
@@ -41664,7 +41724,7 @@ Do NOT derail the current task - if the operator has an in-flight question, answ
 }
 
 // src/_generated/version.ts
-var PKG_VERSION = "0.4.22";
+var PKG_VERSION = "0.4.23";
 
 // src/api-base-url-migrate.ts
 var import_node_fs3 = __toESM(require("node:fs"), 1);
@@ -54052,6 +54112,7 @@ function buildThreadStartParams(msg, config2, desiredModel) {
     "- When creating a task, classify and supply the `domain` field explicitly based on the nature of the work: pass `domain:'code'` for software engineering work, `domain:'seo'` for SEO/optimization, `domain:'marketing'` for marketing/growth, or `domain:'content'` for content creation/writing \u2014 the same way you select category/subcategory. A follow-up task spawned with `parentTaskId` inherits its parent's domain automatically and requires no explicit domain.",
     '- When a task needs a browser (login flows, UI verification/clicks, scraping), pass `prerequisites:[{mcp:"mrrlin-browser",reason:"..."}]` to `create_execution_run`. If that run later fails into a handoff, the operator gets a one-click "Launch browser & resume" that brings up the dedicated browser as the turn starts.',
     "- When you actually drive a browser (calling `browser_navigate`, `browser_click`, `browser_snapshot`, `browser_type`, etc.), invoke those tools from the **`mrrlin-browser`** MCP server ONLY. Do NOT use tools from any other browser MCP (`playwright`, `chrome-devtools`, `puppeteer`, etc.) even if they are also connected on this operator's box \u2014 those use a shared/default Playwright profile that is fragile across runs. Only `mrrlin-browser` uses this project's dedicated Chromium + persistent operator profile (`~/.mrrlin/browser/profile`). If your host exposes multiple browser MCPs with overlapping tool names, prefer the `mrrlin-browser` server unconditionally.",
+    "- Exactly one browser session exists for this operator, shared across every concurrent chat and task on their box \u2014 a deliberate tradeoff, not a bug, so a second concurrent user of the browser is expected, not exceptional. If a `mrrlin-browser` call fails with `Browser is already in use for ...`, that means another one of the operator's own chats or tasks is driving the browser RIGHT NOW. Do not present this as a broken tool or a mysterious error: tell the operator plainly that the browser is busy with another task, ask them to wait for it to finish (or close it), and offer to retry shortly. Never retry-loop this silently.",
     "",
     'Research playbook \u2014 when the user asks about a topic, an area, or "what do we have on X":',
     "- Tasks are the source of truth. Start with `list_tasks` using `q=<keyword>`; the filter is a",
@@ -54146,6 +54207,14 @@ function buildThreadStartParams(msg, config2, desiredModel) {
     "- A failed fetch is a signal to change tool, never a finding. NEVER report that you could not",
     "  check, verify, or reach a publicly reachable target until you have actually tried the",
     "  browser on it in this turn.",
+    "- An internal Mrrlin route (`/deliverables/...`, `/wiki/...`, `/specs/...`, `/items/...`,",
+    "  `/memory/...`) is NEVER a browser target, even when it appears as a literal URL/path in a",
+    "  message or a screenshot. Resolve it through the typed tools instead: `list_wiki_pages` (by",
+    "  `folder`/`q`) or `search_wiki_pages` to find the page, then `get_wiki_page` for its content;",
+    "  `get_task`/`list_tasks` for `/items/<id>`. Opening these in `mrrlin-browser` is slower, can",
+    "  hit an operator login/consent wall the automation cannot clear on its own, and needlessly",
+    "  occupies the one shared browser session. Only reach for the browser on a Mrrlin route after a",
+    "  typed tool call for it has actually failed.",
     "",
     "Initiative contract \u2014 do the obvious next step instead of offering it:",
     "- When a next step is cheap, clearly wanted, and finishable in this turn, just do it and",
@@ -64620,6 +64689,14 @@ async function main() {
       return;
     }
     case "browser-serve": {
+      const reaped = reapOrphanedBrowserProcesses();
+      if (reaped.killedPids.length > 0) {
+        const childNote = reaped.killedChildPids.length > 0 ? ` (plus ${reaped.killedChildPids.length} browser child process(es) they had already opened: ${reaped.killedChildPids.join(", ")})` : "";
+        process.stderr.write(
+          `[mrrlin-mcp browser-serve] reaped ${reaped.killedPids.length} orphaned browser process(es) with a dead parent: ${reaped.killedPids.join(", ")}${childNote}
+`
+        );
+      }
       const { command: command2, args, env } = browserServeArgs();
       const r = (0, import_node_child_process18.spawnSync)(command2, args, { stdio: "inherit", env: { ...process.env, ...env } });
       process.exitCode = r.status ?? 1;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrrlin-dev/mcp",
-  "version": "0.4.22",
+  "version": "0.4.23",
   "type": "module",
   "bin": {
     "mrrlin-mcp": "dist/bin.cjs"