@feigi/fleet-ctl 3.91.0 → 3.91.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@feigi/fleet-ctl",
-  "version": "3.91.0",
+  "version": "3.91.1",
   "description": "Agent fleet: run-team controller, merge bot, PR reviewer, and the ticket pipeline they share",
   "license": "Apache-2.0",
   "repository": {

package/scripts/board-gather.mjs

@@ -412,10 +412,11 @@ export function mapCi(ciJson, pr) {
 // A receipt with no session to give is `{ error }`, which the panel shows:
 // the board names no session rather than guessing one. So is a recorded
 // session whose NAME is not an omp `<ISO>_<uuid>` session directory:
-// phase0.mjs accepts any existing directory as the session, so a `--session`
-// naming the encoded-cwd project dir instead of its own session child lands in
-// the receipt, and readOmpSpend would book every session under it — the
-// controller's own main-session transcript included — as this run's agents.
+// phase0.mjs refuses a directory with no `<dir>.jsonl` beside it, but a
+// receipt it wrote before that refusal, or one written by hand, can record the
+// encoded-cwd project dir instead of its own session child, and readOmpSpend
+// would book every session under it — the controller's own main-session
+// transcript included — as this run's agents.
 // A recorded directory that does not exist yet is not an error — that is
 // gatherSpend()'s quiet `null`, below.
 export function receiptSpendDir(stateDir) {

package/scripts/phase0.mjs

@@ -8,9 +8,11 @@
 // The session directory is `--session`, else `PI_ARTIFACTS_DIR`, which omp
 // exports to its Python eval kernel and the processes that kernel starts (the
 // bash tool's environment does not carry it); with neither, exit 2. It is
-// accepted when it is an existing directory, or when `<dir>.jsonl` exists
-// beside it — omp creates the directory when the session's first subagent
-// writes there.
+// accepted when `<dir>.jsonl` exists beside it and the path is a directory or
+// does not exist yet — omp creates the directory when the session's first
+// subagent writes there. An existing directory with no `<dir>.jsonl` beside
+// it names no session (the directory holding every session of a checkout is
+// one): from either source it exits 2 naming the path, writing no receipt.
 //
 // Reached through the Resolver and run from the MAIN checkout (a linked
 // worktree refuses: the checks below judge the checkout the controller reads
@@ -528,15 +530,19 @@ function main() {
   }
   const source = given === null ? "PI_ARTIFACTS_DIR" : "--session";
   const session = resolve(given ?? fromEnv);
-  // An existing directory, or `<session>.jsonl` beside a path that does not
-  // exist yet: omp creates the directory when the session's first subagent
-  // writes there.
+  // A session is named by its transcript, `<session>.jsonl`, beside the
+  // directory: an existing directory with none beside it — the one holding
+  // every session of a checkout, say — names no session. The directory itself
+  // may not exist yet: omp creates it when the session's first subagent writes
+  // there.
+  const transcript = existsSync(`${session}.jsonl`);
   try {
     if (!statSync(session).isDirectory()) die(`${source} ${session} is not a directory`);
+    if (!transcript) die(`${source} ${session} has no ${session}.jsonl beside it, so it names no session`);
   } catch (e) {
     if (e.code === undefined) throw e;
     if (e.code !== "ENOENT") die(`${source} ${session} is not readable (${e.code})`);
-    if (!existsSync(`${session}.jsonl`)) die(`${source} ${session} does not exist, and no ${session}.jsonl beside it names a session`);
+    if (!transcript) die(`${source} ${session} does not exist, and no ${session}.jsonl beside it names a session`);
   }
   const positive = (name) => {
     const n = F.numArg(name);

package/skills/run-team/references/phase-0.md

@@ -7,7 +7,12 @@ environment carries `PI_ARTIFACTS_DIR`, this run's own session directory,
 which phase 0 records in its receipt; the bash tool's does not, so there
 phase 0 exits 2. `expanduser` is required: a literal `~/…` fails with
 `FileNotFoundError`. So is the `stderr` redirect: the cell never shows a
-child's stderr. `--session <dir>` overrides the variable.
+child's stderr. `--session <dir>` overrides the variable. Either way the
+directory must have its transcript `<dir>.jsonl` beside it; the directory
+itself may not exist yet. An existing directory with no `<dir>.jsonl` beside
+it — `~/.omp/agent/sessions/<encoded-cwd>`, which holds every session of the
+checkout — names no session: phase 0 exits 2 naming the path and where it
+came from, and writes no receipt.
 
 Exit 3 is a step that failed: its line carries the refusal, so fix the cause
 and run it again.