@ouro.bot/cli 0.1.0-alpha.119 → 0.1.0-alpha.120

package/changelog.json

@@ -1,6 +1,13 @@
 {
   "_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.120",
+      "changes": [
+        "Daemon startup now kills ALL orphaned ouro processes (daemons AND agents) from previous instances \u2014 fixes stale-version processes handling requests after every update.",
+        "Failover error messages no longer contain raw JSON API response bodies. Error messages are sanitized at the source and the failover summary uses clean classification labels only."
+      ]
+    },
     {
       "version": "0.1.0-alpha.119",
       "changes": [

package/dist/heart/daemon/daemon.js

@@ -34,7 +34,7 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.OuroDaemon = void 0;
-exports.killOrphanAgentProcesses = killOrphanAgentProcesses;
+exports.killOrphanProcesses = killOrphanProcesses;
 const fs = __importStar(require("fs"));
 const net = __importStar(require("net"));
 const path = __importStar(require("path"));
@@ -52,18 +52,21 @@ const pending_1 = require("../../mind/pending");
 const channel_1 = require("../../mind/friends/channel");
 const mcp_manager_1 = require("../../repertoire/mcp-manager");
 /**
- * Kill orphaned agent-entry.js processes from previous daemon instances.
- * The process manager only tracks agents it spawns in-memory — agents from
- * a previous daemon are invisible and keep handling requests on old code.
+ * Kill ALL orphaned ouro processes from previous daemon instances.
+ * On each `ouro up`, old daemon AND agent processes persist because:
+ * - Daemons are spawned detached and never killed on restart
+ * - Each old daemon keeps respawning agents from its own (stale) code
+ * This scans for both daemon-entry.js and agent-entry.js processes
+ * and kills everything except the current process.
  */
-/* v8 ignore start -- orphan cleanup: uses ps/kill which can't be unit-tested @preserve */
-function killOrphanAgentProcesses() {
+/* v8 ignore start -- orphan cleanup: uses ps/kill, tested via deployment @preserve */
+function killOrphanProcesses() {
     try {
         const myPid = process.pid;
         const result = (0, child_process_1.execSync)("ps -eo pid,command", { encoding: "utf-8", timeout: 5000 });
         const pidsToKill = [];
         for (const line of result.split("\n")) {
-            if (!line.includes("agent-entry.js"))
+            if (!line.includes("agent-entry.js") && !line.includes("daemon-entry.js"))
                 continue;
             const trimmed = line.trim();
             const pid = parseInt(trimmed, 10);
@@ -81,7 +84,7 @@ function killOrphanAgentProcesses() {
             (0, runtime_1.emitNervesEvent)({
                 component: "daemon",
                 event: "daemon.orphan_cleanup",
-                message: `killed ${pidsToKill.length} orphaned agent processes`,
+                message: `killed ${pidsToKill.length} orphaned ouro processes`,
                 meta: { pids: pidsToKill },
             });
         }
@@ -91,7 +94,7 @@ function killOrphanAgentProcesses() {
             level: "warn",
             component: "daemon",
             event: "daemon.orphan_cleanup_error",
-            message: "failed to clean up orphaned agent processes",
+            message: "failed to clean up orphaned ouro processes",
             meta: { error: error instanceof Error ? error.message : String(error) },
         });
     }
@@ -204,7 +207,7 @@ class OuroDaemon {
         // Pre-initialize MCP connections so they're ready for the first command (non-blocking)
         /* v8 ignore next -- catch callback: getSharedMcpManager logs errors internally @preserve */
         (0, mcp_manager_1.getSharedMcpManager)().catch(() => { });
-        killOrphanAgentProcesses();
+        killOrphanProcesses();
         await this.processManager.startAutoStartAgents();
         await this.senseManager?.startAutoStartSenses();
         this.scheduler.start?.();

package/dist/heart/provider-failover.js

@@ -19,12 +19,10 @@ const CLASSIFICATION_LABELS = {
     "network-error": "is unreachable (network error)",
     "unknown": "encountered an error",
 };
-function buildFailoverContext(errorMessage, classification, currentProvider, currentModel, agentName, inventory, providerModels) {
+function buildFailoverContext(_errorMessage, classification, currentProvider, currentModel, agentName, inventory, providerModels) {
     const label = CLASSIFICATION_LABELS[classification];
     const providerWithModel = currentModel ? `${currentProvider} (${currentModel})` : currentProvider;
-    const errorSummary = errorMessage
-        ? `${providerWithModel} ${label} (${errorMessage})`
-        : `${providerWithModel} ${label}`;
+    const errorSummary = `${providerWithModel} ${label}`;
     const workingProviders = [];
     const unconfiguredProviders = [];
     const failingProviders = [];

package/dist/heart/provider-ping.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.sanitizeErrorMessage = sanitizeErrorMessage;
 exports.pingProvider = pingProvider;
 exports.runHealthInventory = runHealthInventory;
 const anthropic_1 = require("./providers/anthropic");
@@ -10,6 +11,28 @@ const github_copilot_1 = require("./providers/github-copilot");
 const auth_flow_1 = require("./daemon/auth-flow");
 const runtime_1 = require("../nerves/runtime");
 const PING_TIMEOUT_MS = 10_000;
+/**
+ * Strip raw JSON API response bodies from error messages.
+ * SDK errors often include the full response body: "400 {"type":"error","error":...}".
+ * Extract just the HTTP status prefix or a short summary.
+ */
+function sanitizeErrorMessage(message) {
+    // Match "NNN {json...}" pattern — keep the status code, drop the JSON
+    const match = message.match(/^(\d{3})\s*\{/);
+    if (match) {
+        // Try to extract the inner message from the JSON
+        try {
+            const json = JSON.parse(message.slice(match[1].length).trim());
+            const inner = json?.error?.message;
+            if (typeof inner === "string" && inner && inner !== "Error") {
+                return `${match[1]} ${inner}`;
+            }
+        }
+        catch { /* not valid JSON, fall through */ }
+        return `HTTP ${match[1]}`;
+    }
+    return message;
+}
 function hasEmptyCredentials(provider, config) {
     switch (provider) {
         case "anthropic":
@@ -106,7 +129,7 @@ async function pingProvider(provider, config) {
             message: `provider ping failed: ${provider}`,
             meta: { provider, classification, error: err.message },
         });
-        return { ok: false, classification, message: err.message };
+        return { ok: false, classification, message: sanitizeErrorMessage(err.message) };
     }
 }
 const PINGABLE_PROVIDERS = ["anthropic", "openai-codex", "azure", "minimax", "github-copilot"];

package/package.json

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