open-agents-ai 0.187.539 → 0.187.541

package/dist/index.js

@@ -585913,17 +585913,17 @@ async function runGraphicalSudo(opts) {
         }
       }, 2e3);
     }, timeoutMs);
-    child.stdout.on("data", (d2) => {
+    child.stdout?.on("data", (d2) => {
       stdout += d2.toString("utf-8");
     });
-    child.stderr.on("data", (d2) => {
+    child.stderr?.on("data", (d2) => {
       stderr += d2.toString("utf-8");
     });
-    child.on("error", (e2) => {
+    child.once("error", (e2) => {
       clearTimeout(timer);
       reject(e2);
     });
-    child.on("close", (code8) => {
+    child.once("close", (code8) => {
       clearTimeout(timer);
       if (timedOut) return reject(new Error(`graphical sudo timed out after ${timeoutMs}ms`));
       resolve43({ code: code8 ?? -1, stdout, stderr, helper });

package/dist/postinstall-daemon.cjs

@@ -85,18 +85,28 @@ function tryHealth(port, cb) {
   req.end();
 }
 
-// Resolve the `oa` launcher script. On npm global install:
-//   Linux/macOS:  $(npm prefix -g)/bin/oa
-//   Windows:      %APPDATA%/npm/oa.cmd
+// Resolve the `oa` launcher script.
+//
+// PRIORITY ORDER (most stable first):
+//   1. Sibling launcher.cjs — guaranteed to live next to this postinstall,
+//      survives across upgrades because it's part of the package itself.
+//      Avoids the npx cache trap and the fnm shim trap.
+//   2. npm-global bin (if npm_config_prefix is set) — stable across reboots.
+//   3. PATH-resolved oa — works in dev installs but can resolve to ephemeral
+//      shims (npx cache, fnm multishell symlinks).
 function resolveOaBinary() {
-  // 1. If npm set npm_config_prefix, use it.
-  var prefix = process.env.npm_config_prefix || "";
-  if (!prefix) {
-    // 2. Ask npm.
-    prefix = runCapture("npm prefix -g");
-  }
-
   var candidates = [];
+
+  // 1. Self-relative — IDEAL: the launcher ships in the same directory as
+  //    this postinstall script, so as long as the package is installed, it
+  //    exists. This path is what should land in the systemd ExecStart.
+  try {
+    candidates.push(path.resolve(__dirname, "launcher.cjs"));
+  } catch (e) {}
+
+  // 2. npm prefix bin
+  var prefix = process.env.npm_config_prefix || "";
+  if (!prefix) prefix = runCapture("npm prefix -g");
   if (prefix) {
     if (IS_WIN) {
       candidates.push(path.join(prefix, "oa.cmd"));
@@ -106,14 +116,9 @@ function resolveOaBinary() {
       candidates.push(path.join(prefix, "bin", "open-agents"));
     }
   }
-  // 3. PATH fallback via which/where.
+  // 3. PATH fallback (last resort — can resolve to npx cache or fnm shim).
   var found = runCapture(IS_WIN ? "where oa" : "which oa").split(/\r?\n/)[0];
-  if (found) candidates.push(found);
-
-  // 4. Relative to this script: <pkg>/dist/launcher.cjs
-  try {
-    candidates.push(path.resolve(__dirname, "launcher.cjs"));
-  } catch (e) {}
+  if (found && !/_npx|fnm_multishells/.test(found)) candidates.push(found);
 
   for (var i = 0; i < candidates.length; i++) {
     try {
@@ -315,8 +320,14 @@ function installSystemd(nodeBin, oaScript, user) {
     "Environment=OA_PORT=" + PORT,
     "Environment=NODE_ENV=production",
     "ExecStart=" + nodeBin + " " + oaScript + " serve --daemon --quiet",
-    "Restart=on-failure",
+    // Restart=always (was on-failure) — also relaunch on clean exit.
+    // Some upgrade flows trigger process.exit(0) (e.g. /update reload,
+    // SIGTERM during npm install), which Restart=on-failure ignores.
+    // Pair with StartLimitIntervalSec to prevent thrash loops.
+    "Restart=always",
     "RestartSec=3",
+    "StartLimitIntervalSec=30",
+    "StartLimitBurst=10",
     "StandardOutput=append:" + path.join(logDir, "daemon.log"),
     "StandardError=append:" + path.join(logDir, "daemon.err.log"),
     "",

package/dist/preinstall.cjs

@@ -1,4 +1,92 @@
 #!/usr/bin/env node
-// Preinstall hook shim — no-op by default
-process.exit(0);
+/* eslint-disable */
+/**
+ * preinstall — runs BEFORE npm replaces files on disk. Gracefully stops
+ * the running OA daemon so the install window doesn't strand it with
+ * a half-replaced binary that crashes on next relaunch.
+ *
+ * The postinstall hook restarts the daemon after install completes with
+ * the new code in place.
+ *
+ * Opt-out: OA_SKIP_DAEMON_INSTALL=1 (matches postinstall semantics).
+ */
+
+"use strict";
+
+if (process.env.OA_SKIP_DAEMON_INSTALL === "1") {
+  process.exit(0);
+}
+
+var os = require("os");
+var path = require("path");
+var fs = require("fs");
+var cp = require("child_process");
+
+var IS_WIN = os.platform() === "win32";
+var IS_LINUX = os.platform() === "linux";
+var IS_MAC = os.platform() === "darwin";
+var HOME = os.homedir();
+var SERVICE_LABEL = "open-agents-daemon";
+var LAUNCHD_LABEL = "ai.open-agents.daemon";
+var WIN_TASK_NAME = "OpenAgentsDaemon";
+
+function runQuiet(cmd) {
+  try { cp.execSync(cmd, { stdio: "pipe", timeout: 8000 }); return true; } catch (e) { return false; }
+}
+
+function log(msg) { process.stdout.write("  [preinstall] " + msg + "\n"); }
+
+function stopServiceManager() {
+  // Stop via the registered service manager if one exists. This is
+  // graceful (sends SIGTERM, waits for exit) AND prevents the manager
+  // from auto-restarting the daemon mid-install.
+  try {
+    if (IS_LINUX) {
+      // Pause + stop. Don't disable — we want it to come back after
+      // postinstall re-enables/restarts it.
+      runQuiet("systemctl --user stop " + SERVICE_LABEL + ".service");
+    } else if (IS_MAC) {
+      var plist = path.join(HOME, "Library", "LaunchAgents", LAUNCHD_LABEL + ".plist");
+      if (fs.existsSync(plist)) runQuiet("launchctl unload " + plist);
+    } else if (IS_WIN) {
+      runQuiet('schtasks /End /TN "' + WIN_TASK_NAME + '"');
+    }
+  } catch (e) { /* best-effort */ }
+}
+
+function killPidFile(pidFile) {
+  try {
+    if (!fs.existsSync(pidFile)) return false;
+    var n = parseInt(fs.readFileSync(pidFile, "utf8").trim(), 10);
+    if (!n || n <= 0) return false;
+    try { process.kill(n, "SIGTERM"); log("SIGTERM " + pidFile + " (pid " + n + ")"); return true; }
+    catch (e) { /* dead */ }
+  } catch (e) {}
+  return false;
+}
+
+stopServiceManager();
+killPidFile(path.join(HOME, ".open-agents", "daemon.pid"));
+
+// Final: lsof-based sweep for any process still holding 11435.
+try {
+  var port = parseInt(process.env.OA_PORT || "11435", 10);
+  var out = "";
+  try {
+    out = cp.execSync("lsof -ti :" + port + " 2>/dev/null || true", {
+      encoding: "utf8", timeout: 3000,
+    }).trim();
+  } catch (e) {}
+  if (out) {
+    out.split(/\s+/).forEach(function (s) {
+      var n = parseInt(s, 10);
+      if (Number.isFinite(n) && n > 0 && n !== process.pid) {
+        try { process.kill(n, "SIGTERM"); log("SIGTERM port-holder pid " + n); } catch (e) {}
+      }
+    });
+  }
+} catch (e) {}
+
+// 1.5s grace so SIGTERM handlers can flush state.
+setTimeout(function () { process.exit(0); }, 1500);
 

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "open-agents-ai",
-  "version": "0.187.539",
+  "version": "0.187.541",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "open-agents-ai",
-      "version": "0.187.539",
+      "version": "0.187.541",
       "hasInstallScript": true,
       "license": "CC-BY-NC-4.0",
       "dependencies": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "open-agents-ai",
-  "version": "0.187.539",
+  "version": "0.187.541",
   "description": "AI coding agent powered by open-source models (Ollama/vLLM) — interactive TUI with agentic tool-calling loop",
   "type": "module",
   "main": "./dist/index.js",