@ouro.bot/cli 0.1.0-alpha.550 → 0.1.0-alpha.551

package/changelog.json

@@ -1,6 +1,12 @@
 {
   "_note": "This changelog is maintained as part of the PR/version-bump workflow. Agent-curated, not auto-generated. Agents read this file directly via read_file to understand what changed between versions.",
   "versions": [
+    {
+      "version": "0.1.0-alpha.551",
+      "changes": [
+        "`ouro up` now waits for SIGTERMed orphan daemon processes to settle before opening the replacement Unix socket, preventing the previous daemon from unlinking the new daemon's command socket during orphan cleanup."
+      ]
+    },
     {
       "version": "0.1.0-alpha.550",
       "changes": [

package/dist/heart/daemon/daemon.js

@@ -37,6 +37,7 @@ exports.OuroDaemon = void 0;
 exports.parseOrphanPidsFromPs = parseOrphanPidsFromPs;
 exports.filterPidfilePidsToActualOrphans = filterPidfilePidsToActualOrphans;
 exports.mergeUniqueOrphanPids = mergeUniqueOrphanPids;
+exports.waitForOrphanProcessesToSettle = waitForOrphanProcessesToSettle;
 exports.killOrphanProcesses = killOrphanProcesses;
 exports.writePidfile = writePidfile;
 exports.handleAgentSenseTurn = handleAgentSenseTurn;
@@ -176,6 +177,40 @@ function mergeUniqueOrphanPids(...sources) {
     }
     return merged;
 }
+const ORPHAN_CLEANUP_SETTLE_TIMEOUT_MS = 5_000;
+const ORPHAN_CLEANUP_SETTLE_POLL_INTERVAL_MS = 50;
+/* v8 ignore start -- process liveness probe; pure wait behavior covered via injected deps @preserve */
+function defaultIsPidAlive(pid) {
+    try {
+        process.kill(pid, 0);
+        return true;
+    }
+    catch (error) {
+        return error.code === "EPERM";
+    }
+}
+/* v8 ignore stop */
+/* v8 ignore start -- real timer wiring; wait behavior covered via injected sleep @preserve */
+async function defaultSettleSleep(ms) {
+    await new Promise((resolve) => setTimeout(resolve, ms));
+}
+/* v8 ignore stop */
+async function waitForOrphanProcessesToSettle(pids, deps = {}) {
+    if (pids.length === 0)
+        return [];
+    const isPidAlive = deps.isPidAlive ?? defaultIsPidAlive;
+    const now = deps.now ?? Date.now;
+    const sleep = deps.sleep ?? defaultSettleSleep;
+    const timeoutMs = deps.timeoutMs ?? ORPHAN_CLEANUP_SETTLE_TIMEOUT_MS;
+    const pollIntervalMs = deps.pollIntervalMs ?? ORPHAN_CLEANUP_SETTLE_POLL_INTERVAL_MS;
+    const deadline = now() + timeoutMs;
+    let survivors = pids.filter(isPidAlive);
+    while (survivors.length > 0 && now() < deadline) {
+        await sleep(pollIntervalMs);
+        survivors = pids.filter(isPidAlive);
+    }
+    return survivors;
+}
 /* v8 ignore start -- shells out to ps; covered by filterPidfilePidsToActualOrphans unit tests via injected runner @preserve */
 function runPsCheck(pids) {
     try {
@@ -217,7 +252,7 @@ function killOrphanProcesses(socketPath = socket_client_1.DEFAULT_DAEMON_SOCKET_
             message: "blocked orphan cleanup for non-production daemon socket",
             meta: { socketPath, pidfilePath: PIDFILE_PATH },
         });
-        return;
+        return [];
     }
     if (isVitestProcess()) {
         (0, runtime_1.emitNervesEvent)({
@@ -227,7 +262,7 @@ function killOrphanProcesses(socketPath = socket_client_1.DEFAULT_DAEMON_SOCKET_
             message: "blocked killOrphanProcesses from touching real pidfile under vitest",
             meta: { pidfilePath: PIDFILE_PATH },
         });
-        return;
+        return [];
     }
     try {
         let pidfileOrphans = [];
@@ -269,6 +304,7 @@ function killOrphanProcesses(socketPath = socket_client_1.DEFAULT_DAEMON_SOCKET_
                 meta: { pids: pidsToKill },
             });
         }
+        return pidsToKill;
     }
     catch (error) {
         (0, runtime_1.emitNervesEvent)({
@@ -278,6 +314,7 @@ function killOrphanProcesses(socketPath = socket_client_1.DEFAULT_DAEMON_SOCKET_
             message: "failed to clean up orphaned ouro processes",
             meta: { error: error instanceof Error ? error.message : String(error) },
         });
+        return [];
     }
 }
 /**
@@ -594,7 +631,8 @@ class OuroDaemon {
         // MCP connections are lazily initialized per-agent during senseTurn
         // (daemon manages multiple agents; agent identity must be set before loading MCP config)
         /* v8 ignore start -- orphan cleanup + pidfile: calls process management functions @preserve */
-        killOrphanProcesses(this.socketPath);
+        const killedOrphanPids = killOrphanProcesses(this.socketPath);
+        await waitForOrphanProcessesToSettle(killedOrphanPids);
         /* v8 ignore stop */
         await this.openCommandSocket();
         this.triggerAutoStartAgents();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ouro.bot/cli",
-  "version": "0.1.0-alpha.550",
+  "version": "0.1.0-alpha.551",
   "main": "dist/heart/daemon/ouro-entry.js",
   "bin": {
     "cli": "dist/heart/daemon/ouro-bot-entry.js",