@tablation/crew 0.49.4 → 0.49.6

package/dist/cli.js

@@ -4931,6 +4931,8 @@ var Tracker = class {
    * writes pass its seat's agent key so the event log and rule context see an
    * agent, not the ship's human principal.
    */
+  /** Told once per failed best-effort read (`events`); poll.ts points it at the pass's emitter. */
+  onReadFailure;
   constructor(route, ship, opts = {}) {
     this.route = route;
     if (!route.resolved) {
@@ -5372,10 +5374,16 @@ var Tracker = class {
    */
   async events(type, limit = 500) {
     const filters = encodeURIComponent(JSON.stringify([{ columnName: "event_type", operator: "EQ", value: type }]));
-    const res = await this.eventsFetch("GET", `/events?limit=${limit}&filters=${filters}`);
-    if (res.status === 404) return [];
-    if (!res.ok) throw new Error(`reading ${type} events failed: HTTP ${res.status}`);
-    const rows = await res.json();
+    let rows;
+    try {
+      const res = await this.eventsFetch("GET", `/events?limit=${limit}&filters=${filters}`);
+      if (res.status === 404) return [];
+      if (!res.ok) throw new Error(`HTTP ${res.status}`);
+      rows = await res.json();
+    } catch (err) {
+      this.onReadFailure?.(`reading ${type} events failed: ${err instanceof Error ? err.message : String(err)}`);
+      return [];
+    }
     return rows.filter((e) => e.event_type === type).sort((a, b) => Date.parse(b.occurred_at ?? b.created_at ?? "") - Date.parse(a.occurred_at ?? a.created_at ?? ""));
   }
   /** The newest event of `type` on a ticket, or undefined — e.g. the `pushed` tip QA verifies against. */
@@ -6112,6 +6120,7 @@ function operatorTodo(tickets, contract, operatorId) {
 }
 
 // src/digest.ts
+var DIRECTION_EVENT_TYPES = ["conflict_handed_back", "requeued", "bounced", "conflict_escalated"];
 function displayKey2(t) {
   return t.issue_tag ?? t.issue_id ?? "?";
 }
@@ -6146,10 +6155,24 @@ function lastComment(i, ticketId) {
   if (!last) return "\u2014";
   return `${last.created_at.slice(0, 16)}Z ${author(last, i)}`;
 }
+var eventAt = (e) => e.occurred_at ?? e.created_at ?? "";
+function pendingDirection(i, ticketId) {
+  const mine = (i.events ?? []).filter((e) => e.record_id === ticketId);
+  const pushed = mine.filter((e) => e.event_type === "pushed").map(eventAt).sort().pop() ?? "";
+  return mine.filter((e) => DIRECTION_EVENT_TYPES.includes(e.event_type) && eventAt(e) > pushed).sort((a, b) => eventAt(a) < eventAt(b) ? 1 : -1)[0];
+}
+function directionCell(i, ticketId) {
+  const d = pendingDirection(i, ticketId);
+  if (!d) return "\u2014";
+  const msg = (d.message ?? "").replace(/\s+/g, " ").trim().slice(0, 160);
+  return `${d.event_type} ${eventAt(d).slice(0, 16)}Z${msg ? ` \u2014 ${msg}` : ""}`.replace(/\|/g, "/");
+}
 function newFromOthers(i, ticketId) {
-  return i.comments.filter(
+  const comments = i.comments.filter(
     (c) => c.ticket_id === ticketId && c.team_member_id !== i.me && c.created_at > i.watermark
   ).length;
+  const d = pendingDirection(i, ticketId);
+  return comments + (d && eventAt(d) > i.watermark ? 1 : 0);
 }
 function who(t, i) {
   return whoAssigned(t, i) + heldBySuffix(t, i);
@@ -6188,14 +6211,14 @@ ${sorted(ts).map(row).join("\n")}
 `;
 }
 function buildingDigest(i) {
-  const header = "| ticket | repo | branch | worktree | mode | status | assignee | filed by | sev | pri | eff | updated | last comment | new since last poll |\n|---|---|---|---|---|---|---|---|---|---|---|---|---|---|";
+  const header = "| ticket | repo | branch | worktree | mode | status | assignee | filed by | sev | pri | eff | updated | last comment | runner direction | new since last poll |\n|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|";
   const row = (t) => {
     const n = newFromOthers(i, t.id);
     const branch = i.branchFor ? i.branchFor(t) : "";
     const worktree = i.worktreeFor ? i.worktreeFor(t) : "";
     const dir = i.dirFor ? i.dirFor(t) : null;
     const repo = i.dirFor ? dir ?? "**NO CHECKOUT**" : "";
-    return `| ${displayKey2(t)} | ${repo} | ${branch} | ${worktree} | ${modeCell(t, i)} | ${t.status} | ${who(t, i)} | ${filedBy(t, i)} | ${t.severity ?? "\u2014"} | ${t.priority ?? "\u2014"} | ${effCell(t)} | ${t.updated_at.slice(0, 16)}Z | ${lastComment(i, t.id)} | ${n > 0 ? `**${n} new**` : "\u2014"} |`;
+    return `| ${displayKey2(t)} | ${repo} | ${branch} | ${worktree} | ${modeCell(t, i)} | ${t.status} | ${who(t, i)} | ${filedBy(t, i)} | ${t.severity ?? "\u2014"} | ${t.priority ?? "\u2014"} | ${effCell(t)} | ${t.updated_at.slice(0, 16)}Z | ${lastComment(i, t.id)} | ${directionCell(i, t.id)} | ${n > 0 ? `**${n} new**` : "\u2014"} |`;
   };
   const blockedRow = (t) => `| ${displayKey2(t)} | ${t.status} | ${who(t, i)} | p${effectivePriority(t)} | ${blockers(t, i)} |`;
   const blockedTable = (ts) => ts.length === 0 ? "\n_None._\n" : `
@@ -6226,6 +6249,7 @@ ${sorted(held).map((t) => `| ${displayKey2(t)} | ${stopOf(t)} |`).join("\n")}
     "## Current queue \u2014 built for you by the poll\n",
     "\nAlready filtered to your lane, and already ordered by the Step 2 rule.\nTicket bodies are deliberately omitted: fetch the full record of only the\nticket you actually pick up. **Do not re-fetch the whole tracker.** This\ndigest comes from the same API call the poll just made, moments ago.\n",
     '\n"new since last poll" counts comments from someone other than you since\nthe poll watermark \u2014 the same signal that woke this run.\n',
+    '\n"runner direction" is the newest instruction the crew itself posted to the\nevent log (not Comments) that no push has answered yet: `conflict_handed_back`\n(merge the base into the branch and resolve), `requeued`, `bounced`,\n`conflict_escalated`. It is direction, same as a comment from a hold: act on it.\n',
     "\n**`repo` is the checkout a ticket's work happens in** \u2014 an area spans\nseveral repositories, so cut the worktree beside THAT directory, not beside\nwhichever one you started in. A ticket marked **NO CHECKOUT** is not yours:\nthis ship has no clone of its repository, and another ship may serve it.\n\n**Use the `branch` and `worktree` columns verbatim.** They are that\nrepository's own naming convention, rendered for THIS ticket \u2014 cut the\nworktree at `<repo>/../<worktree>`, on branch `<branch>`. Do not derive\neither yourself, and prefer these over the Environment section's own worked\nexample if the two ever disagree \u2014 that example is illustrative only,\nrendered before any ticket is picked.\n",
     ...i.workModes && [...i.workModes.modes.values()].some((m) => m !== "automatic") ? ["\n**Manual / hybrid projects.** Some projects on this board are not fully automatic. In a `manual` project no lane touches any ticket, and a ticket with no project is treated the same way, so none appear below. In a `hybrid` project a ticket appears below only when it is assigned to an agent. An `accepted` ticket that is absent for either reason is not yours to claim \u2014 do not go looking for it.\n"] : [],
     "\n### Step 1 \u2014 open tickets that may be yours to act on\n",
@@ -6439,16 +6463,26 @@ async function runStalledSweep(o, stalled) {
 async function decideCycle(o) {
   const { route, ship, state, emit } = o;
   const tracker = new Tracker(route, ship);
+  tracker.onReadFailure = (m) => emit.warn(m);
   emit.enter("poll");
+  const optional = async (what, read, fallback) => {
+    try {
+      return await read;
+    } catch (err) {
+      emit.warn(`reading ${what} failed: ${err instanceof Error ? err.message : String(err)}`);
+      return fallback;
+    }
+  };
   const [tickets, comments, crewRows, epicRows, ships, projectModes, stalledEvents] = await Promise.all([
     tracker.openTickets(),
     tracker.comments(200),
     tracker.crewRows(),
-    tracker.epicRows(),
-    tracker.shipRows(),
-    tracker.projectWorkModes(),
+    optional("epics", tracker.epicRows(), []),
+    optional("ships", tracker.shipRows(), []),
+    optional("project work modes", tracker.projectWorkModes(), /* @__PURE__ */ new Map()),
     tracker.events("stalled")
   ]);
+  const directionEvents = (await Promise.all([...DIRECTION_EVENT_TYPES, "pushed"].map((t) => tracker.events(t)))).flat();
   emit.emit(`${tickets.length} open ticket(s), ${comments.length} comment(s)`, {
     data: { tickets: tickets.length, comments: comments.length }
   });
@@ -6594,6 +6628,7 @@ async function decideCycle(o) {
     selection,
     actionable,
     watermark,
+    directionEvents,
     selectionInput,
     attention: attentionCurrent
   };
@@ -6673,6 +6708,7 @@ function writeDigest(o, d, role, stateDir) {
       ships: d.ships,
       myShipId: d.ships.find((s) => (s.name ?? "").trim() === o.ship.name.trim())?.id ?? null,
       watermark: d.watermark,
+      events: d.directionEvents,
       blocked: d.blocked,
       blockerInfo: d.info
     };
@@ -6777,7 +6813,14 @@ function worktreeForBranch(cwd, branch) {
   }
   return null;
 }
-var priorBounces = (ticket, events) => events.filter((e) => e.record_id === ticket.id && e.event_type === "conflict_handed_back").length;
+var stampOf = (e) => e.occurred_at ?? e.created_at ?? "";
+var priorBounces = (ticket, events) => {
+  const mine = events.filter((e) => e.record_id === ticket.id);
+  const pushes = mine.filter((e) => e.event_type === "pushed").map(stampOf);
+  return mine.filter(
+    (e) => e.event_type === "conflict_handed_back" && pushes.some((p) => p > stampOf(e))
+  ).length;
+};
 function planConflictBounce(cwd, ticket, branch, base, paths, events) {
   return {
     ticket,
@@ -12521,7 +12564,18 @@ async function runDaemonLoop(o) {
   let passes = 0;
   for (; ; ) {
     if (o.signal?.aborted) return "aborted";
-    const result = await o.runPass();
+    let result;
+    try {
+      result = await o.runPass();
+    } catch (err) {
+      passes++;
+      o.onPassError?.(err);
+      if (o.maxPasses !== void 0 && passes >= o.maxPasses) return "maxPasses";
+      if (o.signal?.aborted) return "aborted";
+      await sleep(backoff);
+      backoff = Math.min(backoff * 2, ceiling);
+      continue;
+    }
     passes++;
     if (result.started.length > 0) {
       backoff = floor;
@@ -13822,7 +13876,7 @@ ${changed2.map((c) => `  - ${c}
         }
       }
       if (outcome.conflicts?.length || outcome.unbuildable?.length) {
-        const handedBack = await tracker.events("conflict_handed_back");
+        const handedBack = [...await tracker.events("conflict_handed_back"), ...await tracker.events("pushed")];
         const flagged = await tracker.events("stranded_flagged");
         const seat = releaseSeat(c);
         for (const f of outcome.conflicts ?? []) {
@@ -14447,6 +14501,9 @@ ${changed2.map((c) => `  - ${c}
 `)
           })
         }),
+        onPassError: (err) => emit.error(
+          `daemon pass failed, continuing: ${err instanceof Error ? err.stack ?? err.message : String(err)}`
+        ),
         checkStale: makeStaleChecker(watchPaths, { reloadSentinel: reloadSentinelFor(cfg.ship.stateDir) }),
         onPass: (result) => {
           if (result.started.length) emit.emit(`daemon pass started: ${result.started.join(", ")}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.49.4",
+  "version": "0.49.6",
   "description": "A standing team of headless agents that picks work off a Tablation board, does it on a machine you control, and reports back.",
   "type": "module",
   "license": "MIT",

package/prompts/default/skills/crew-lane-agent.md

@@ -141,6 +141,12 @@ Then:
   the ticket again because the status was left alone.
 - A hold gave new direction on an `in_progress` ticket → adjust the branch
   and comment on what changed.
+- A ticket whose `runner direction` cell is not `—` has been handed back by
+  the crew itself (a conflict with the base, a requeue, a bounce). That is
+  direction exactly as a comment from a hold is, and "no new comment" does not
+  cancel it: for `conflict_handed_back`, merge `<remote>/<base>` into the
+  branch, resolve, run the suite, push as a fast-forward and set `fixed`
+  (Step 4.4). Never answer it with "nothing to build".
 - An `in_progress` ticket with no assignee and no new comment → still "up
   for grabs, more work to do": a hold clearing the assignee *is* the signal.
 - **An `in_progress` ticket assigned to you with no new comment is your own
@@ -397,8 +403,10 @@ worktree, runs the test gate and deploys. Three consequences:
    lines from Step 3.5 are the only input the release phase has; nobody
    reads the diff later to fill them in.
 3. **A branch that no longer squash-merges cleanly comes back to you** at
-   `in_progress`, reassigned to your lane, with a comment naming the
-   conflict. Fix it in the same worktree and take it to `fixed` again. "Never
+   `in_progress`, reassigned to your lane, with the hand-back
+   posted to the event log, which the digest's `runner direction` column
+   shows (a ticket there reads as new direction even with no new comment).
+   Fix it in the same worktree and take it to `fixed` again. "Never
    merge" above means never merge INTO the base; merging the base INTO your
    own ticket branch is the expected fix (rule 4), and it is not "by hand to
    help".

package/prompts/dev-qa/skills/crew-lane-agent.md

@@ -142,6 +142,12 @@ Then:
   the ticket again because the status was left alone.
 - A hold gave new direction on an `in_progress` ticket → adjust the branch
   and comment on what changed.
+- A ticket whose `runner direction` cell is not `—` has been handed back by
+  the crew itself (a conflict with the base, a requeue, a bounce). That is
+  direction exactly as a comment from a hold is, and "no new comment" does not
+  cancel it: for `conflict_handed_back`, merge `<remote>/<base>` into the
+  branch, resolve, run the suite, push as a fast-forward and set `fixed`
+  (Step 4.4). Never answer it with "nothing to build".
 - An `in_progress` ticket with no assignee and no new comment → still "up
   for grabs, more work to do": a hold clearing the assignee *is* the signal.
 - **An `in_progress` ticket assigned to you with no new comment is your own
@@ -397,8 +403,10 @@ worktree, runs the test gate and deploys. Three consequences:
    lines from Step 3.5 are the only input the release phase has; nobody
    reads the diff later to fill them in.
 3. **A branch that no longer squash-merges cleanly comes back to you** at
-   `in_progress`, reassigned to your lane, with a comment naming the
-   conflict. Fix it in the same worktree and take it to `fixed` again. "Never
+   `in_progress`, reassigned to your lane, with the hand-back
+   posted to the event log, which the digest's `runner direction` column
+   shows (a ticket there reads as new direction even with no new comment).
+   Fix it in the same worktree and take it to `fixed` again. "Never
    merge" above means never merge INTO the base; merging the base INTO your
    own ticket branch is the expected fix (rule 4), and it is not "by hand to
    help".