@ouro.bot/cli 0.1.0-alpha.508 → 0.1.0-alpha.509

package/changelog.json

@@ -1,6 +1,14 @@
 {
   "_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.509",
+      "changes": [
+        "New `Trips` category in `ouro doctor`. Operators previously had no quick way to verify the trip ledger was healthy — they had to call `trip_status` from inside the agent or open `state/trips/ledger.json` by hand. With the trip ledger now load-bearing for trip planning workflows, doctor coverage matters.",
+        "The check walks each agent bundle and reports per-agent trip health: pass when the ledger is absent (optional feature, not yet ensured), warn when `state/trips/` exists but `ledger.json` is missing or lacks the `ledgerId` field, fail when `ledger.json` is unreadable, unparseable, or missing the `privateKeyPem` (encrypted records would be unreadable). Healthy ledgers report `<ledgerId> (<N> records)` so the operator sees record count without opening anything.",
+        "7 new tests cover: no-agents (warn), no-ledger-dir (pass — optional), missing ledger.json (warn), unparseable JSON (fail), missing ledgerId (warn), missing privateKeyPem (fail), and the healthy passing path with record-counting that ignores non-`.json` files in the records dir. Wired between `Security` and `Disk` in the `CATEGORY_CHECKERS` array — same orchestration shape as the existing categories."
+      ]
+    },
     {
       "version": "0.1.0-alpha.508",
       "changes": [

package/dist/heart/daemon/doctor.js

@@ -13,6 +13,7 @@ exports.checkAgents = checkAgents;
 exports.checkSenses = checkSenses;
 exports.checkHabits = checkHabits;
 exports.checkSecurity = checkSecurity;
+exports.checkTrips = checkTrips;
 exports.checkDisk = checkDisk;
 exports.checkLifecycle = checkLifecycle;
 exports.runDoctorChecks = runDoctorChecks;
@@ -379,6 +380,73 @@ function checkSecurity(deps) {
     }
     return { name: "Security", checks };
 }
+function checkTrips(deps) {
+    const checks = [];
+    const agents = discoverAgents(deps);
+    if (agents.length === 0) {
+        checks.push({ label: "trip ledger", status: "warn", detail: "no agent bundles found" });
+        return { name: "Trips", checks };
+    }
+    for (const agentDir of agents) {
+        const tripsRootPath = `${deps.bundlesRoot}/${agentDir}/state/trips`;
+        if (!deps.existsSync(tripsRootPath)) {
+            // Trip ledger is optional; absence is fine. Pass with a hint.
+            checks.push({ label: `${agentDir} trip ledger`, status: "pass", detail: "no ledger directory (no trips ensured yet)" });
+            continue;
+        }
+        const ledgerPath = `${tripsRootPath}/ledger.json`;
+        if (!deps.existsSync(ledgerPath)) {
+            checks.push({ label: `${agentDir} trip ledger`, status: "warn", detail: "state/trips/ exists but ledger.json missing — run trip_ensure_ledger" });
+            continue;
+        }
+        let raw;
+        /* v8 ignore start -- defensive: readFileSync failure after existsSync passes is a race-condition fallback @preserve */
+        try {
+            raw = deps.readFileSync(ledgerPath);
+        }
+        catch {
+            checks.push({ label: `${agentDir} trip ledger`, status: "fail", detail: "ledger.json could not be read" });
+            continue;
+        }
+        /* v8 ignore stop */
+        let parsed;
+        try {
+            parsed = JSON.parse(raw);
+        }
+        catch {
+            checks.push({ label: `${agentDir} trip ledger`, status: "fail", detail: "ledger.json is not valid JSON" });
+            continue;
+        }
+        const ledgerId = typeof parsed.ledgerId === "string" ? parsed.ledgerId : null;
+        const hasPrivateKey = typeof parsed.privateKeyPem === "string" && parsed.privateKeyPem.includes("BEGIN");
+        if (!ledgerId) {
+            checks.push({ label: `${agentDir} trip ledger`, status: "warn", detail: "ledger.json missing ledgerId field" });
+            continue;
+        }
+        if (!hasPrivateKey) {
+            checks.push({ label: `${agentDir} trip ledger`, status: "fail", detail: `${ledgerId}: privateKeyPem missing — encrypted trip records cannot be read` });
+            continue;
+        }
+        let recordCount = 0;
+        const recordsDir = `${tripsRootPath}/records`;
+        /* v8 ignore start -- defensive: records dir presence and readdir error are filesystem-state branches not all exercised by tests; pluralization branch likewise depends on record count fixtures @preserve */
+        if (deps.existsSync(recordsDir)) {
+            try {
+                recordCount = deps.readdirSync(recordsDir).filter((name) => name.endsWith(".json")).length;
+            }
+            catch {
+                // ignore — the warn detail will still report 0 records
+            }
+        }
+        checks.push({
+            label: `${agentDir} trip ledger`,
+            status: "pass",
+            detail: `${ledgerId} (${recordCount} record${recordCount === 1 ? "" : "s"})`,
+        });
+        /* v8 ignore stop */
+    }
+    return { name: "Trips", checks };
+}
 function checkDisk(deps) {
     const checks = [];
     const addLogSizeCheck = (labelPrefix, logsDir) => {
@@ -584,6 +652,7 @@ const CATEGORY_CHECKERS = [
     { name: "Senses", fn: checkSenses },
     { name: "Habits", fn: checkHabits },
     { name: "Security", fn: checkSecurity },
+    { name: "Trips", fn: checkTrips },
     { name: "Disk", fn: checkDisk },
 ];
 async function runDoctorChecks(deps) {

package/dist/heart/identity.js

@@ -50,6 +50,7 @@ exports.getAgentDaemonStateRoot = getAgentDaemonStateRoot;
 exports.getAgentDaemonLogsDir = getAgentDaemonLogsDir;
 exports.getAgentDaemonLoggingConfigPath = getAgentDaemonLoggingConfigPath;
 exports.getAgentMessagesRoot = getAgentMessagesRoot;
+exports.getAgentMailroomRoot = getAgentMailroomRoot;
 exports.getAgentToolsRoot = getAgentToolsRoot;
 exports.loadAgentConfig = loadAgentConfig;
 exports.setAgentName = setAgentName;
@@ -291,6 +292,9 @@ function getAgentDaemonLoggingConfigPath(agentName) {
 function getAgentMessagesRoot(agentName) {
     return path.join(getAgentStateRoot(resolveOptionalAgentName(agentName)), "messages");
 }
+function getAgentMailroomRoot(agentName) {
+    return path.join(getAgentStateRoot(resolveOptionalAgentName(agentName)), "mailroom");
+}
 function getAgentToolsRoot(agentName) {
     return path.join(getAgentStateRoot(resolveOptionalAgentName(agentName)), "tools");
 }

package/dist/mailroom/reader.js

@@ -122,7 +122,7 @@ function createMailroomStore(config, agentName) {
             storeLabel: `${config.azureAccountUrl}/${containerName}`,
         };
     }
-    const storePath = config.storePath ?? path.join((0, identity_2.getAgentRoot)(agentName), "state", "mailroom");
+    const storePath = config.storePath ?? (0, identity_2.getAgentMailroomRoot)(agentName);
     return {
         store: new file_store_1.FileMailroomStore({ rootDir: storePath }),
         storeKind: "file",

package/package.json

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