@ouro.bot/cli 0.1.0-alpha.249 → 0.1.0-alpha.250

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.250",
+      "changes": [
+        "fix(sync): surface 'bundle is not a git repo' as an actionable error instead of silently failing. Previously, enabling `sync.enabled` on a bundle that had never been `git init`'d produced a generic `git status` failure buried in nerves logs; the agent saw nothing in its start-of-turn packet and the user saw nothing in `ouro status`. Now: (1) `preTurnPull` and `postTurnPush` detect the missing `.git` directory before touching git and return an actionable error with the bundle path and the `git init` hint; (2) this error propagates via `ctx.syncFailure` into the agent's Sync warning, so the agent can offer to run `git init` or just do it; (3) `ouro status` shows a red `error` state with `not a git repo — run \\`git init\\` to enable sync` next to the offending bundle. New nerves event: `heart.sync_not_a_repo`."
+      ]
+    },
     {
       "version": "0.1.0-alpha.249",
       "changes": [

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

@@ -87,15 +87,16 @@ function listEnabledBundleAgents(options = {}) {
  * sync rows without depending on argv-derived global identity. Bundles that
  * cannot be read are skipped silently.
  *
- * For rows with sync enabled, also resolves the remote URL by shelling out to
- * `git remote get-url <remote>` in the bundle directory. On any error (no
- * remote, not a git repo, git missing) the URL is left undefined and the
- * status renderer treats the bundle as "local only".
+ * For rows with sync enabled, also checks whether the bundle is a git repo
+ * (via .git directory presence) and resolves the remote URL via
+ * `git remote get-url <remote>`. On any error the URL is left undefined and
+ * the status renderer falls back to "local only" or "not a git repo".
  */
 function listBundleSyncRows(options = {}) {
     const bundlesRoot = options.bundlesRoot ?? (0, identity_1.getAgentBundlesRoot)();
     const readFileSync = options.readFileSync ?? fs.readFileSync;
     const execFileSync = options.execFileSync ?? child_process_1.execFileSync;
+    const existsSync = options.existsSync ?? fs.existsSync;
     const agents = listEnabledBundleAgents(options);
     const rows = [];
     for (const agent of agents) {
@@ -117,22 +118,28 @@ function listBundleSyncRows(options = {}) {
         catch {
             // Best-effort: bundle without readable config still gets a row with defaults
         }
-        let remoteUrl;
+        const row = { agent, enabled, remote };
         if (enabled) {
-            try {
-                const out = execFileSync("git", ["remote", "get-url", remote], {
-                    cwd: bundleRoot,
-                    stdio: "pipe",
-                    timeout: 5000,
-                }).toString().trim();
-                if (out.length > 0)
-                    remoteUrl = out;
-            }
-            catch {
-                // No remote configured, not a git repo, or git missing — leave undefined.
+            // Only meaningful when sync is enabled — we don't care about disabled bundles'
+            // git state. Check .git presence before attempting any git invocation.
+            const gitInitialized = existsSync(path.join(bundleRoot, ".git"));
+            row.gitInitialized = gitInitialized;
+            if (gitInitialized) {
+                try {
+                    const out = execFileSync("git", ["remote", "get-url", remote], {
+                        cwd: bundleRoot,
+                        stdio: "pipe",
+                        timeout: 5000,
+                    }).toString().trim();
+                    if (out.length > 0)
+                        row.remoteUrl = out;
+                }
+                catch {
+                    // No remote configured or git missing — leave remoteUrl undefined.
+                }
             }
         }
-        rows.push(remoteUrl !== undefined ? { agent, enabled, remote, remoteUrl } : { agent, enabled, remote });
+        rows.push(row);
     }
     return rows;
 }

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

@@ -147,10 +147,14 @@ function parseStatusPayload(data) {
         const remote = stringField(row.remote);
         if (!agent || enabled === null || !remote)
             return null;
-        const remoteUrl = stringField(row.remoteUrl) ?? undefined;
-        return remoteUrl !== undefined
-            ? { agent, enabled, remote, remoteUrl }
-            : { agent, enabled, remote };
+        const parsed = { agent, enabled, remote };
+        const remoteUrl = stringField(row.remoteUrl);
+        if (remoteUrl !== null)
+            parsed.remoteUrl = remoteUrl;
+        const gitInitialized = booleanField(row.gitInitialized);
+        if (gitInitialized !== null)
+            parsed.gitInitialized = gitInitialized;
+        return parsed;
     });
     if (parsedSenses.some((row) => row === null) ||
         parsedWorkers.some((row) => row === null) ||
@@ -318,11 +322,30 @@ function formatDaemonStatusOutput(response, fallback) {
         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);
-            let detail = "";
-            if (row.enabled) {
-                detail = row.remoteUrl !== undefined ? `${row.remote} → ${row.remoteUrl}` : "local only";
+            // Three states for enabled rows: error (not a repo), ok (remote URL), ok (local only).
+            // Disabled rows show the usual dim dot + "disabled".
+            let dot;
+            let stateText;
+            let detail;
+            if (!row.enabled) {
+                dot = dim("○");
+                stateText = "disabled".padEnd(10);
+                detail = "";
+            }
+            else if (row.gitInitialized === false) {
+                dot = red("●");
+                stateText = "error   ".padEnd(10);
+                detail = "not a git repo — run `git init` to enable sync";
+            }
+            else if (row.remoteUrl !== undefined) {
+                dot = green("●");
+                stateText = "enabled ".padEnd(10);
+                detail = `${row.remote} → ${row.remoteUrl}`;
+            }
+            else {
+                dot = green("●");
+                stateText = "enabled ".padEnd(10);
+                detail = "local only";
             }
             lines.push(`    ${name} ${dot} ${stateText}  ${dim(detail)}`);
         }

package/dist/heart/sync.js

@@ -39,6 +39,21 @@ const child_process_1 = require("child_process");
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const runtime_1 = require("../nerves/runtime");
+/**
+ * Check whether the bundle is initialized as a git repo.
+ * Used by both pre-turn pull and post-turn push to surface a clear,
+ * actionable error when sync is enabled but the user never ran `git init`
+ * inside their bundle. This error is propagated all the way into the
+ * agent's start-of-turn packet as a Sync warning, so the agent can
+ * either ask the user or run `git init` itself.
+ */
+function ensureGitRepo(agentRoot) {
+    if (fs.existsSync(path.join(agentRoot, ".git"))) {
+        return { ok: true };
+    }
+    const error = `bundle is not a git repo; run \`git init\` inside ${agentRoot} to enable sync (or disable sync in agent.json)`;
+    return { ok: false, error };
+}
 /**
  * Pre-turn pull: sync the agent bundle from remote before assembling the start-of-turn packet.
  *
@@ -54,6 +69,20 @@ function preTurnPull(agentRoot, config) {
         message: "pre-turn pull starting",
         meta: { agentRoot, remote: config.remote },
     });
+    // Check that the bundle is actually a git repo before touching git at all.
+    // Surfaces a clear, actionable error via syncFailure → start-of-turn packet
+    // so the agent can propose running `git init` (or just do it).
+    const repoCheck = ensureGitRepo(agentRoot);
+    if (!repoCheck.ok) {
+        (0, runtime_1.emitNervesEvent)({
+            level: "warn",
+            component: "heart",
+            event: "heart.sync_not_a_repo",
+            message: "pre-turn pull failed: bundle is not a git repo",
+            meta: { agentRoot },
+        });
+        return repoCheck;
+    }
     // Check if any remote is configured. If not, skip the pull (local-only mode).
     try {
         const remoteOutput = (0, child_process_1.execFileSync)("git", ["remote"], {
@@ -118,6 +147,22 @@ function postTurnPush(agentRoot, config) {
         message: "post-turn push starting",
         meta: { agentRoot, remote: config.remote },
     });
+    // Same git-repo check as preTurnPull. This is the more common failure path
+    // since postTurnPush runs after every turn while preTurnPull only runs on
+    // user-initiated turns. Prior to this guard, an un-init'd bundle would fail
+    // the git-status invocation below with a generic "not a git repository"
+    // error; now we catch it explicitly with an actionable message.
+    const repoCheck = ensureGitRepo(agentRoot);
+    if (!repoCheck.ok) {
+        (0, runtime_1.emitNervesEvent)({
+            level: "warn",
+            component: "heart",
+            event: "heart.sync_not_a_repo",
+            message: "post-turn push failed: bundle is not a git repo",
+            meta: { agentRoot },
+        });
+        return repoCheck;
+    }
     let statusOutput;
     try {
         statusOutput = (0, child_process_1.execFileSync)("git", ["status", "--porcelain"], {

package/package.json

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