@proagentstore/cli 0.4.41 → 0.4.42

package/dist/browser-runner/coding/repo.js

@@ -1,15 +1,49 @@
 import { execFileSync } from "node:child_process";
-import { existsSync, mkdirSync, readdirSync, rmSync } from "node:fs";
+import { existsSync, mkdirSync, readdirSync, rmSync, statSync } from "node:fs";
 import { join } from "node:path";
 /**
- * Repo/workdir helpers for the coding engine.
- *
- * These lived in `coding/tmux.ts` and have nothing to do with tmux — `ensureRepo` runs `git
- * clone`. The coding engine stopped using tmux when it moved to the structured stream-json
- * interface, so anyone cleaning up that module found its two most load-bearing functions
- * inside it (#247). Split out so the tmux module is only tmux, and only the terminal-operator
- * agents depend on it.
+ * Inspect a workdir: does it exist, does it hold anything, is it a checkout. Never throws —
+ * every failure is a `false`, because this function exists to REPORT a broken path and one that
+ * threw would be reported as a broken runner instead.
  */
+export function checkWorkdir(dir) {
+    const absent = { checked: true, path: dir, exists: false, isDirectory: false, entryCount: 0, insideWorkTree: false, gitChecked: true };
+    let isDirectory = false;
+    try {
+        isDirectory = statSync(dir).isDirectory();
+    }
+    catch {
+        return { ...absent };
+    }
+    if (!isDirectory)
+        return { ...absent, exists: true };
+    let entryCount = 0;
+    try {
+        entryCount = readdirSync(dir).length;
+    }
+    catch {
+        // Unreadable (permissions) — reported as empty rather than as a crash; the cloud's
+        // message for "empty" tells the owner to look at the path either way.
+    }
+    let insideWorkTree = false;
+    let gitChecked = true;
+    try {
+        const out = execFileSync("git", ["rev-parse", "--is-inside-work-tree"], {
+            cwd: dir,
+            encoding: "utf-8",
+            timeout: 10_000,
+            stdio: ["ignore", "pipe", "ignore"],
+        });
+        insideWorkTree = out.trim() === "true";
+    }
+    catch (e) {
+        // Exit 128 ("not a git repository") is an ANSWER. ENOENT is git missing from PATH, which
+        // is not — and reporting it as "not a checkout" would condemn every repo on that machine.
+        if (e?.code === "ENOENT")
+            gitChecked = false;
+    }
+    return { checked: true, path: dir, exists: true, isDirectory, entryCount, insideWorkTree, gitChecked };
+}
 /**
  * A safe, collision-resistant label derived from an arbitrary string.
  *

package/dist/browser-runner/coding/runtime.js

@@ -2,7 +2,7 @@ import { homedir } from "node:os";
 import { join, resolve } from "node:path";
 import { defaultStatePath, HeadlessSession } from "./headless.js";
 import { InspectError, readGitRemoteOrigin, readRepoFile, repoTree, runRepoGit } from "./inspect.js";
-import { ensureRepo, sanitizeSessionName } from "./repo.js";
+import { checkWorkdir, ensureRepo, sanitizeSessionName } from "./repo.js";
 /** Hard cap on a pane returned to the brain/console (matches the worker MAX_PANE_CHARS). */
 const MAX_PANE = 64 * 1024;
 export class CodingRuntime {
@@ -58,6 +58,16 @@ export class CodingRuntime {
     gitRemote(input) {
         return { remote: readGitRemoteOrigin(this.resolveWorkDir(input)) };
     }
+    /**
+     * Is the configured workdir usable AT ALL — present, non-empty, a checkout (#405)?
+     *
+     * The only endpoint here that answers a question about the PATH rather than about its
+     * contents, and the one the cloud needs before it may call a repo `ready`. Reports the
+     * resolved path so a message can name the thing the owner actually typed, `~` and all.
+     */
+    checkRepo(input) {
+        return checkWorkdir(this.resolveWorkDir(input));
+    }
     static taskTypes() {
         return ["coding.session"];
     }

package/dist/browser-runner/server.js

@@ -232,6 +232,18 @@ async function route(runner, req, res) {
             return json(res, 400, { error: e instanceof Error ? e.message : String(e) });
         }
     }
+    // Does the configured workdir exist / hold files / belong to a checkout (#405)? An older
+    // runner 404s this; the cloud reads the missing `checked:true` as "unverified" and leaves
+    // the repo's status alone rather than condemning it.
+    if (req.method === "POST" && path === "/coding/repo-check") {
+        const b = await readJson(req);
+        try {
+            return json(res, 200, runner.coding.checkRepo(b));
+        }
+        catch (e) {
+            return json(res, 400, { error: e instanceof Error ? e.message : String(e) });
+        }
+    }
     if (req.method === "POST" && path === "/coding/tree") {
         const b = await readJson(req);
         try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@proagentstore/cli",
-  "version": "0.4.41",
+  "version": "0.4.42",
   "description": "CLI for creating, publishing, and running ProAgentStore agents",
   "license": "MIT",
   "type": "module",