@ouro.bot/cli 0.1.0-alpha.242 → 0.1.0-alpha.243

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.243",
+      "changes": [
+        "fix(daemon): per-agent Git Sync in `ouro status` \u2014 was always showing `disabled` regardless of any agent's `agent.json`, because the daemon process has no argv-derived agent identity so `getSyncConfig()` fell into its catch. Status payload now carries a per-agent `sync` array (one row per enabled bundle) rendered as its own section like Senses and Workers, instead of a single global field on the overview block."
+      ]
+    },
     {
       "version": "0.1.0-alpha.241",
       "changes": [

package/dist/heart/daemon/agent-discovery.js

@@ -34,6 +34,7 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.listEnabledBundleAgents = listEnabledBundleAgents;
+exports.listBundleSyncRows = listBundleSyncRows;
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const identity_1 = require("../identity");
@@ -79,3 +80,36 @@ function listEnabledBundleAgents(options = {}) {
     }
     return discovered.sort((left, right) => left.localeCompare(right));
 }
+/**
+ * Read the per-agent sync block from each enabled bundle's agent.json.
+ * Used by the daemon (and stopped-state status renderer) to build per-agent
+ * sync rows without depending on argv-derived global identity. Bundles that
+ * cannot be read are skipped silently.
+ */
+function listBundleSyncRows(options = {}) {
+    const bundlesRoot = options.bundlesRoot ?? (0, identity_1.getAgentBundlesRoot)();
+    const readFileSync = options.readFileSync ?? fs.readFileSync;
+    const agents = listEnabledBundleAgents(options);
+    const rows = [];
+    for (const agent of agents) {
+        const configPath = path.join(bundlesRoot, `${agent}.ouro`, "agent.json");
+        let enabled = false;
+        let remote = "origin";
+        try {
+            const raw = readFileSync(configPath, "utf-8");
+            const parsed = JSON.parse(raw);
+            if (parsed.sync && typeof parsed.sync === "object") {
+                if (typeof parsed.sync.enabled === "boolean")
+                    enabled = parsed.sync.enabled;
+                if (typeof parsed.sync.remote === "string" && parsed.sync.remote.length > 0) {
+                    remote = parsed.sync.remote;
+                }
+            }
+        }
+        catch {
+            // Best-effort: bundle without readable config still gets a row with defaults
+        }
+        rows.push({ agent, enabled, remote });
+    }
+    return rows;
+}

package/dist/heart/daemon/cli-render.js

@@ -54,6 +54,7 @@ const runtime_mode_1 = require("./runtime-mode");
 const identity_1 = require("../identity");
 const daemon_tombstone_1 = require("./daemon-tombstone");
 const daemon_health_1 = require("./daemon-health");
+const agent_discovery_1 = require("./agent-discovery");
 // ── Field extractors ──
 function stringField(value) {
     return typeof value === "string" ? value : null;
@@ -72,10 +73,14 @@ function parseStatusPayload(data) {
     const overview = raw.overview;
     const senses = raw.senses;
     const workers = raw.workers;
+    const sync = raw.sync;
     if (!overview || typeof overview !== "object" || Array.isArray(overview))
         return null;
     if (!Array.isArray(senses) || !Array.isArray(workers))
         return null;
+    // sync is optional for backward compatibility — older daemons may omit it
+    if (sync !== undefined && !Array.isArray(sync))
+        return null;
     const parsedOverview = {
         daemon: stringField(overview.daemon) ?? "unknown",
         health: stringField(overview.health) ?? "unknown",
@@ -89,8 +94,6 @@ function parseStatusPayload(data) {
         senseCount: numberField(overview.senseCount) ?? 0,
         entryPath: stringField(overview.entryPath) ?? "unknown",
         mode: stringField(overview.mode) ?? "unknown",
-        syncEnabled: booleanField(overview.syncEnabled) ?? false,
-        syncRemote: stringField(overview.syncRemote) ?? "origin",
     };
     const parsedSenses = senses.map((entry) => {
         if (!entry || typeof entry !== "object" || Array.isArray(entry))
@@ -135,12 +138,26 @@ function parseStatusPayload(data) {
             lastSignal: stringField(row.lastSignal) ?? null,
         };
     });
-    if (parsedSenses.some((row) => row === null) || parsedWorkers.some((row) => row === null))
+    const parsedSync = (sync ?? []).map((entry) => {
+        if (!entry || typeof entry !== "object" || Array.isArray(entry))
+            return null;
+        const row = entry;
+        const agent = stringField(row.agent);
+        const enabled = booleanField(row.enabled);
+        const remote = stringField(row.remote);
+        if (!agent || enabled === null || !remote)
+            return null;
+        return { agent, enabled, remote };
+    });
+    if (parsedSenses.some((row) => row === null) ||
+        parsedWorkers.some((row) => row === null) ||
+        parsedSync.some((row) => row === null))
         return null;
     return {
         overview: parsedOverview,
         senses: parsedSenses,
         workers: parsedWorkers,
+        sync: parsedSync,
     };
 }
 // ── ANSI color helpers (private) ──
@@ -232,9 +249,6 @@ function formatDaemonStatusOutput(response, fallback) {
     lines.push(kvLine("Socket", ov.socketPath));
     lines.push(kvLine("Outlook", ov.outlookUrl));
     lines.push(kvLine("Health", `${statusDot(ov.health)} ${ov.health}`));
-    /* v8 ignore next -- cosmetic: sync-enabled branch requires agent with sync configured @preserve */
-    const syncLabel = ov.syncEnabled ? `${green("●")} enabled (remote: ${ov.syncRemote})` : `${dim("○")} disabled`;
-    lines.push(kvLine("Git Sync", syncLabel));
     lines.push(kvLine("Updated", ov.lastUpdated));
     lines.push("");
     // ── Senses ──
@@ -295,6 +309,19 @@ function formatDaemonStatusOutput(response, fallback) {
         }
         lines.push("");
     }
+    // ── Git Sync (per agent) ──
+    if (payload.sync.length > 0) {
+        lines.push(`  ${teal("──")} ${bold("Git Sync")} ${teal("─".repeat(35))}`);
+        const agentNameWidth = Math.max(12, ...payload.sync.map((r) => r.agent.length));
+        for (const row of payload.sync) {
+            const name = row.agent.padEnd(agentNameWidth);
+            const dot = row.enabled ? green("●") : dim("○");
+            const stateText = (row.enabled ? "enabled " : "disabled").padEnd(10);
+            const detail = row.enabled ? `remote: ${row.remote}` : "";
+            lines.push(`    ${name} ${dot} ${stateText}  ${dim(detail)}`);
+        }
+        lines.push("");
+    }
     return lines.join("\n");
 }
 function formatVersionOutput() {
@@ -304,7 +331,7 @@ function formatVersionOutput() {
     return mode === "dev" ? `${version} (dev)` : version;
     /* v8 ignore stop */
 }
-function buildStoppedStatusPayload(socketPath) {
+function buildStoppedStatusPayload(socketPath, syncRows = []) {
     const metadata = (0, runtime_metadata_1.getRuntimeMetadata)();
     const repoRoot = (0, identity_1.getRepoRoot)();
     return {
@@ -321,14 +348,23 @@ function buildStoppedStatusPayload(socketPath) {
             senseCount: 0,
             entryPath: path.join(repoRoot, "dist", "heart", "daemon", "daemon-entry.js"),
             mode: (0, runtime_mode_1.detectRuntimeMode)(repoRoot),
-            syncEnabled: false,
-            syncRemote: "origin",
         },
         senses: [],
         workers: [],
+        sync: syncRows,
     };
 }
 function daemonUnavailableStatusOutput(socketPath, healthFilePath) {
+    // Read per-agent sync config from disk so the user still sees it when the daemon is down.
+    // Best-effort: any fs error returns [] and the section is omitted.
+    let syncRows = [];
+    try {
+        syncRows = (0, agent_discovery_1.listBundleSyncRows)();
+    }
+    catch {
+        // listBundleSyncRows already swallows fs errors internally; this catch is a defensive
+        // safety net for environments where the fs module itself is partially mocked.
+    }
     /* v8 ignore start — tombstone read tested in daemon-status-tombstone.test; branch misreported @preserve */
     const tombstone = (0, daemon_tombstone_1.readDaemonTombstone)();
     const deathLine = tombstone
@@ -339,7 +375,7 @@ function daemonUnavailableStatusOutput(socketPath, healthFilePath) {
         formatDaemonStatusOutput({
             ok: true,
             summary: "daemon not running",
-            data: buildStoppedStatusPayload(socketPath),
+            data: buildStoppedStatusPayload(socketPath, syncRows),
         }, "daemon not running"),
         "",
     ];

package/dist/heart/daemon/daemon.js

@@ -42,7 +42,7 @@ const net = __importStar(require("net"));
 const os = __importStar(require("os"));
 const path = __importStar(require("path"));
 const identity_1 = require("../identity");
-const config_1 = require("../config");
+const agent_discovery_1 = require("./agent-discovery");
 const runtime_1 = require("../../nerves/runtime");
 const runtime_metadata_1 = require("./runtime-metadata");
 const runtime_mode_1 = require("./runtime-mode");
@@ -234,7 +234,7 @@ class OuroDaemon {
         const workers = buildWorkerRows(snapshots);
         const senses = this.senseManager?.listSenseRows() ?? [];
         const repoRoot = (0, identity_1.getRepoRoot)();
-        const syncConfig = (0, config_1.getSyncConfig)();
+        const sync = (0, agent_discovery_1.listBundleSyncRows)({ bundlesRoot: this.bundlesRoot });
         return {
             overview: {
                 daemon: "running",
@@ -246,11 +246,10 @@ class OuroDaemon {
                 senseCount: senses.length,
                 entryPath: path.join(repoRoot, "dist", "heart", "daemon", "daemon-entry.js"),
                 mode: (0, runtime_mode_1.detectRuntimeMode)(repoRoot),
-                syncEnabled: syncConfig.enabled,
-                syncRemote: syncConfig.remote,
             },
             workers,
             senses,
+            sync,
         };
     }
     async start() {

package/package.json

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