@tablation/crew 0.43.4 → 0.45.0

package/dist/cli.js

@@ -1432,6 +1432,7 @@ var RELEASE_KEYS = /* @__PURE__ */ new Set([
   "ship",
   "ci",
   "verify",
+  "isolateFailures",
   "versioning",
   "versionFiles",
   "changelog",
@@ -1441,6 +1442,14 @@ var RELEASE_KEYS = /* @__PURE__ */ new Set([
 var VERSIONINGS = ["auto", "none"];
 var CI_KEYS = /* @__PURE__ */ new Set(["provider", "ref"]);
 var VERIFY_KEYS = /* @__PURE__ */ new Set(["match", "timeoutSeconds", "intervalSeconds"]);
+var ISOLATE_KEYS = /* @__PURE__ */ new Set(["maxCandidates"]);
+var DEFAULT_ISOLATE_MAX_CANDIDATES = 4;
+function parseIsolateMax(raw, where, fail) {
+  if (raw === void 0 || raw === null) return void 0;
+  const n = Number(raw);
+  if (!Number.isInteger(n) || n < 0) throw fail(`${where}.maxCandidates must be a non-negative integer (got "${raw}")`);
+  return n;
+}
 var MATCHES = ["commit", "version"];
 var MODES = ["local", "integrate", "ci_manual", "ci_auto", "external"];
 var PROVIDERS = ["github", "buildkite", "other", "none"];
@@ -1474,6 +1483,7 @@ function parseRepoConfig(text, file) {
   rejectUnknown(raw.release, RELEASE_KEYS, "release", file);
   rejectUnknown(raw.release?.ci, CI_KEYS, "release.ci", file);
   rejectUnknown(raw.release?.verify, VERIFY_KEYS, "release.verify", file);
+  rejectUnknown(raw.release?.isolateFailures, ISOLATE_KEYS, "release.isolateFailures", file);
   rejectUnknown(raw.branch, BRANCH_KEYS, "branch", file);
   rejectUnknown(raw.worktrees, /* @__PURE__ */ new Set(["copy", "prefix", "name"]), "worktrees", file);
   rejectUnknown(raw.docs, /* @__PURE__ */ new Set(["triagePolicy", "designGuide"]), "docs", file);
@@ -1611,6 +1621,11 @@ function parseRepoConfig(text, file) {
       `${file}: release.verify.intervalSeconds (${verify.intervalSeconds}) exceeds timeoutSeconds (${verify.timeoutSeconds}) \u2014 it would never be asked twice`
     );
   }
+  const isolateMax = parseIsolateMax(
+    raw.release?.isolateFailures?.maxCandidates,
+    "release.isolateFailures",
+    (m) => new RepoConfigError(`${file}: ${m}`)
+  );
   const versioning = String(raw.release?.versioning ?? "auto");
   if (!VERSIONINGS.includes(versioning)) {
     throw new RepoConfigError(`${file}: release.versioning must be auto|none (got "${versioning}")`);
@@ -1664,6 +1679,7 @@ function parseRepoConfig(text, file) {
       ship,
       ci: { provider, ref: raw.release?.ci?.ref ? String(raw.release.ci.ref) : void 0 },
       verify,
+      isolateFailures: isolateMax === void 0 ? void 0 : { maxCandidates: isolateMax },
       versioning,
       versionFiles,
       tag,
@@ -1848,6 +1864,14 @@ function resolveRepoConfig(repo, ship, dir) {
           DEFAULTS.intervalSeconds
         )
       },
+      isolateFailures: {
+        maxCandidates: pick(
+          "release.isolateFailures.maxCandidates",
+          repo?.release.isolateFailures?.maxCandidates,
+          ship?.release?.isolateFailures?.maxCandidates,
+          DEFAULT_ISOLATE_MAX_CANDIDATES
+        )
+      },
       versioning: pick(
         "release.versioning",
         repo?.release.versioning === "auto" ? void 0 : repo?.release.versioning,
@@ -1999,6 +2023,7 @@ var RELEASE_OVERRIDE_KEYS = /* @__PURE__ */ new Set([
   "mode",
   "ci",
   "verify",
+  "isolateFailures",
   "versioning",
   "versionFiles",
   "versionFile",
@@ -2008,6 +2033,7 @@ var RELEASE_OVERRIDE_KEYS = /* @__PURE__ */ new Set([
 ]);
 var CI_OVERRIDE_KEYS = /* @__PURE__ */ new Set(["provider", "ref"]);
 var VERIFY_OVERRIDE_KEYS = /* @__PURE__ */ new Set(["match", "timeoutSeconds", "intervalSeconds"]);
+var ISOLATE_OVERRIDE_KEYS = /* @__PURE__ */ new Set(["maxCandidates"]);
 function rejectUnknownKeys(obj, allowed, where, file) {
   if (!obj || typeof obj !== "object") return;
   const unknown = Object.keys(obj).filter((k) => !allowed.has(k));
@@ -2022,6 +2048,7 @@ function parseReleaseOverride(raw, where, file) {
   rejectUnknownKeys(raw, RELEASE_OVERRIDE_KEYS, `${where}.release`, file);
   if (raw.ci !== void 0) rejectUnknownKeys(raw.ci, CI_OVERRIDE_KEYS, `${where}.release.ci`, file);
   if (raw.verify !== void 0) rejectUnknownKeys(raw.verify, VERIFY_OVERRIDE_KEYS, `${where}.release.verify`, file);
+  if (raw.isolateFailures !== void 0) rejectUnknownKeys(raw.isolateFailures, ISOLATE_OVERRIDE_KEYS, `${where}.release.isolateFailures`, file);
   const mode = raw.mode === void 0 ? void 0 : String(raw.mode);
   if (mode !== void 0 && !MODES.includes(mode)) {
     throw new ConfigError(`${file}: ${where}.release.mode must be one of ${MODES.join("|")} (got "${raw.mode}")`);
@@ -2050,11 +2077,17 @@ function parseReleaseOverride(raw, where, file) {
   };
   const timeoutSeconds = num(raw.verify?.timeoutSeconds, "timeoutSeconds");
   const intervalSeconds = num(raw.verify?.intervalSeconds, "intervalSeconds");
+  const isolateMax = parseIsolateMax(
+    raw.isolateFailures?.maxCandidates,
+    `${where}.release.isolateFailures`,
+    (m) => new ConfigError(`${file}: ${m}`)
+  );
   const versionFiles = Array.isArray(raw.versionFiles) ? raw.versionFiles : raw.versionFile ? [String(raw.versionFile)] : void 0;
   return {
     mode,
     ci: provider !== void 0 || raw.ci?.ref !== void 0 ? { provider, ref: raw.ci?.ref !== void 0 ? String(raw.ci.ref) : void 0 } : void 0,
     verify: match !== void 0 || timeoutSeconds !== void 0 || intervalSeconds !== void 0 ? { match, timeoutSeconds, intervalSeconds } : void 0,
+    ...isolateMax === void 0 ? {} : { isolateFailures: { maxCandidates: isolateMax } },
     versioning,
     versionFiles,
     changelog: raw.changelog !== void 0 ? String(raw.changelog) : void 0,
@@ -2078,10 +2111,12 @@ function mergeRouteRelease(route, override) {
   const match = override?.verify?.match ?? route.verify?.match;
   const timeoutSeconds = override?.verify?.timeoutSeconds ?? route.verify?.timeoutSeconds;
   const intervalSeconds = override?.verify?.intervalSeconds ?? route.verify?.intervalSeconds;
+  const isolateMax = override?.isolateFailures?.maxCandidates ?? route.isolateFailures?.maxCandidates;
   return {
     mode: override?.mode ?? route.mode,
     ci: provider !== void 0 || ref !== void 0 ? { provider, ref } : void 0,
     verify: match !== void 0 || timeoutSeconds !== void 0 || intervalSeconds !== void 0 ? { match, timeoutSeconds, intervalSeconds } : void 0,
+    ...isolateMax === void 0 ? {} : { isolateFailures: { maxCandidates: isolateMax } },
     versioning: override?.versioning ?? route.versioning,
     versionFiles: override?.versionFiles ?? route.versionFiles,
     changelog: override?.changelog ?? route.changelog,
@@ -3199,8 +3234,6 @@ var DEFAULT_CONTRACT = {
     parent: "ticket_id",
     body: "body",
     author: "team_member_id",
-    kind: "kind",
-    eventKind: "event",
     createdAt: "created_at",
     ship: "ship_id"
   },
@@ -3310,6 +3343,7 @@ function validateContract(c) {
   }
   return problems;
 }
+var EVENT_LOG_MISSING = "the Issues table has no event log \u2014 crew writes its audit trail there (CREW-1574). In the app, open the Issues table's Data & lifecycle tab and turn on Event log, then rerun.";
 
 // src/shell.ts
 import { spawn } from "child_process";
@@ -5174,18 +5208,65 @@ var Tracker = class {
     await this.client.records.update(this.models.crew, memberId, patch2);
   }
   /**
-   * An `event` comment — the runner's own audit trail, distinct from a
-   * seat's prose. `kind` is what keeps these out of the "new comment from
-   * someone else" wake signal.
+   * One entry in the runner's audit trail, written to the Issues table's
+   * event log (CREW-1574) rather than to Comments, so it can never be mistaken
+   * for something a seat or person said. `type` is a short snake_case name
+   * (`pushed`, `claimed`, `bounced`, …); `body` is the human message and keeps
+   * whatever wording the caller already used. Attributed to the seat's Crew
+   * row, with the posting ship in `metadata`.
    */
-  async postEvent(ticketId, body, memberId) {
+  async postEvent(ticketId, body, memberId, type = "note") {
     const auth = await this.authorshipProbe();
-    await this.client.records.create(this.models.comments, {
-      ticket_id: ticketId,
-      body,
-      team_member_id: memberId,
-      kind: "event",
-      ...auth.commentShip && auth.shipId ? { [this.contract.comments.ship]: auth.shipId } : {}
+    const res = await this.eventsFetch("POST", `/records/${ticketId}/events`, {
+      type,
+      message: body,
+      actorRecordId: memberId,
+      ...auth.shipId ? { metadata: { ship_id: auth.shipId } } : {}
+    });
+    if (res.status === 404) throw new ConfigError(EVENT_LOG_MISSING);
+    if (!res.ok) throw new Error(`posting a ${type} event failed: HTTP ${res.status}`);
+  }
+  /**
+   * Whether the Issues table has an event log: `GET /data-models/<issues>/events`
+   * answers 200 when it does and 404 when the setting is off. Crew requires it
+   * (CREW-1574); `connect` and doctor report `false` by name.
+   */
+  async hasEventLog() {
+    const res = await this.eventsFetch("GET", "/events?limit=1");
+    if (res.status === 404) return false;
+    if (!res.ok) throw new Error(`reading the event log failed: HTTP ${res.status}`);
+    return true;
+  }
+  /**
+   * Events of `type` across the Issues table, newest first. The runner's own
+   * dedupe/escalation markers (`stalled`, `gate_red`, `conflict_handed_back`,
+   * `stranded_flagged`) live here now, so the readers that used to scan
+   * Comments for them read this instead. A tracker without the log answers 404
+   * and reads as no events: the writes fail by name (`postEvent`), and a
+   * poll must not die on a read.
+   */
+  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();
+    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. */
+  async latestEvent(ticketId, type) {
+    return (await this.events(type)).find((e) => e.record_id === ticketId);
+  }
+  /** The event-log routes hang off the Issues data model; @tablation/client has no resource for them yet. */
+  eventsFetch(method, path, body) {
+    return fetch(`${this.route.baseUrl}/api/data-models/${this.models.issues}${path}`, {
+      method,
+      headers: {
+        Authorization: `Bearer ${resolveApiKey(this.route)}`,
+        "User-Agent": this.userAgent,
+        ...body === void 0 ? {} : { "Content-Type": "application/json" }
+      },
+      ...body === void 0 ? {} : { body: JSON.stringify(body) }
     });
   }
   /**
@@ -5460,7 +5541,7 @@ async function applySweep(writer, steps, memberId, log) {
         { ticket: step.ticket.issue_id, step: "sweep" }
       );
       try {
-        await writer.postEvent(step.ticket.id, sweepComment(step), memberId);
+        await writer.postEvent(step.ticket.id, sweepComment(step), memberId, "blocked_sweep");
       } catch {
         log.warn("status changed but the note failed to post", { ticket: step.ticket.issue_id, step: "sweep" });
       }
@@ -5573,7 +5654,8 @@ async function noteReaccepted(tracker, result, seat, shipLabel) {
     await tracker.postEvent(
       result.ticket.id,
       `Re-accepted while held by ${shipLabel}; resuming there, not re-claiming. Setting a held ticket back to \`accepted\` is not needed \u2014 to move it to another ship, clear \`held_by_ship_id\`.`,
-      seat
+      seat,
+      "reaccepted"
     );
   } catch {
   }
@@ -5606,7 +5688,8 @@ async function noteTakeover(tracker, result, seat, ships, myShipName, continuing
     await tracker.postEvent(
       result.ticket.id,
       `Taken over from ${shipName(ships, result.tookOverFrom)} (not seen recently) by ${myShipName}; ` + (sha ? `continuing from \`${sha}\` on origin.` : "no branch commit was found on origin to continue from."),
-      seat
+      seat,
+      "taken_over"
     );
   } catch {
   }
@@ -5676,7 +5759,7 @@ function buildingRoleHasWork(role, i) {
   );
   const watchedIds = new Set(watched.map((t) => t.id));
   const newComments = i.comments.filter(
-    (c) => c.created_at > i.watermark && c.team_member_id !== me && (c.kind ?? "comment") !== "event" && watchedIds.has(c.ticket_id)
+    (c) => c.created_at > i.watermark && c.team_member_id !== me && watchedIds.has(c.ticket_id)
   ).length;
   const unassigned = mine.filter(
     (t) => t.status === "in_progress" && !t.assignee_id && !i.blocked.has(t.id)
@@ -5933,14 +6016,14 @@ function filedBy(t, i) {
   return t.reporter_name ? `${t.reporter_name} (unverified)` : "\u2014";
 }
 function lastComment(i, ticketId) {
-  const forTicket = i.comments.filter((c) => c.ticket_id === ticketId && (c.kind ?? "comment") !== "event").sort((a, b) => a.created_at < b.created_at ? -1 : a.created_at > b.created_at ? 1 : 0);
+  const forTicket = i.comments.filter((c) => c.ticket_id === ticketId).sort((a, b) => a.created_at < b.created_at ? -1 : a.created_at > b.created_at ? 1 : 0);
   const last = forTicket[forTicket.length - 1];
   if (!last) return "\u2014";
   return `${last.created_at.slice(0, 16)}Z ${author(last, i)}`;
 }
 function newFromOthers(i, ticketId) {
   return i.comments.filter(
-    (c) => c.ticket_id === ticketId && c.team_member_id !== i.me && c.created_at > i.watermark && (c.kind ?? "comment") !== "event"
+    (c) => c.ticket_id === ticketId && c.team_member_id !== i.me && c.created_at > i.watermark
   ).length;
 }
 function who(t, i) {
@@ -6142,7 +6225,7 @@ var ms = (iso) => {
   const n = iso ? Date.parse(iso) : NaN;
   return Number.isFinite(n) ? n : 0;
 };
-function planStalled(tickets, comments, ships, contract, holds2 = /* @__PURE__ */ new Set(), now = Date.now(), wm) {
+function planStalled(tickets, comments, ships, contract, holds2 = /* @__PURE__ */ new Set(), now = Date.now(), wm, events = []) {
   const watched = /* @__PURE__ */ new Set([contract.statuses.building, contract.statuses.handoff, contract.statuses.verifying]);
   const byTicket = /* @__PURE__ */ new Map();
   for (const c of comments) {
@@ -6155,7 +6238,7 @@ function planStalled(tickets, comments, ships, contract, holds2 = /* @__PURE__ *
     if (!watched.has(t.status)) continue;
     if (t.assignee_id && holds2.has(t.assignee_id)) continue;
     const mine = byTicket.get(t.id) ?? [];
-    const lastMarker = Math.max(0, ...mine.filter((c) => (c.body ?? "").includes(STALLED_MARKER)).map((c) => ms(c.created_at)));
+    const lastMarker = Math.max(0, ...events.filter((e) => e.record_id === t.id && e.event_type === "stalled").map((e) => ms(e.occurred_at ?? e.created_at ?? "")));
     const lastActivityAt = Math.max(
       ms(t.updated_at),
       ...mine.filter((c) => !(c.body ?? "").includes(STALLED_MARKER)).map((c) => ms(c.created_at))
@@ -6197,7 +6280,7 @@ async function applyStalled(writer, stalled, ships, memberId, myShipId, log, dry
       continue;
     }
     try {
-      await writer.postEvent(s.ticket.id, stalledComment(s, ships, now), memberId);
+      await writer.postEvent(s.ticket.id, stalledComment(s, ships, now), memberId, "stalled");
       r.announced++;
       log.warn(`stalled at ${s.ticket.status} \u2014 ${s.holderShipId ? "flagged on the holder" : "commented, no holder"}`, x);
     } catch (e) {
@@ -6231,13 +6314,14 @@ async function decideCycle(o) {
   const { route, ship, state, emit } = o;
   const tracker = new Tracker(route, ship);
   emit.enter("poll");
-  const [tickets, comments, crewRows, epicRows, ships, projectModes] = await Promise.all([
+  const [tickets, comments, crewRows, epicRows, ships, projectModes, stalledEvents] = await Promise.all([
     tracker.openTickets(),
     tracker.comments(200),
     tracker.crewRows(),
     tracker.epicRows(),
     tracker.shipRows(),
-    tracker.projectWorkModes()
+    tracker.projectWorkModes(),
+    tracker.events("stalled")
   ]);
   emit.emit(`${tickets.length} open ticket(s), ${comments.length} comment(s)`, {
     data: { tickets: tickets.length, comments: comments.length }
@@ -6281,7 +6365,7 @@ async function decideCycle(o) {
   }
   const sweep = planSweep(tickets, info, blocked, workModes);
   const holdReleases = planHoldReleases(tickets, tracker.contract);
-  const stalled = planStalled(tickets, comments, ships, tracker.contract, holds2, Date.now(), workModes);
+  const stalled = planStalled(tickets, comments, ships, tracker.contract, holds2, Date.now(), workModes, stalledEvents);
   const skipped = skippedByMode(tickets, workModes);
   if (skipped.manual || skipped.hybrid) {
     emit.emit(`sweeps skipped ${skipped.manual} manual/project-less and ${skipped.hybrid} unassigned hybrid ticket(s)`, {
@@ -6567,15 +6651,15 @@ function worktreeForBranch(cwd, branch) {
   }
   return null;
 }
-var priorBounces = (ticket, comments) => comments.filter((c) => c.ticket_id === ticket.id && (c.body ?? "").includes(BOUNCE_MARKER)).length;
-function planConflictBounce(cwd, ticket, branch, base, paths, comments) {
+var priorBounces = (ticket, events) => events.filter((e) => e.record_id === ticket.id && e.event_type === "conflict_handed_back").length;
+function planConflictBounce(cwd, ticket, branch, base, paths, events) {
   return {
     ticket,
     branch,
     base,
     paths,
     worktree: worktreeForBranch(cwd, branch),
-    priorBounces: priorBounces(ticket, comments)
+    priorBounces: priorBounces(ticket, events)
   };
 }
 function refreshBranch(worktree, base) {
@@ -6621,7 +6705,7 @@ async function applyConflictBounce(writer, b, contract, memberId, log, dryRun) {
     }
     try {
       await writer.updateTicket(b.ticket.id, { status: contract.statuses.needsHuman }, b.ticket.updated_at);
-      await writer.postEvent(b.ticket.id, escalateComment(b), memberId).catch(() => {
+      await writer.postEvent(b.ticket.id, escalateComment(b), memberId, "conflict_escalated").catch(() => {
         log.warn("escalated, but the note failed to post", t);
       });
       log.warn(`conflicts again after a hand-back \u2014 escalated to ${contract.statuses.needsHuman}`, t);
@@ -6658,7 +6742,7 @@ async function applyConflictBounce(writer, b, contract, memberId, log, dryRun) {
       { status: contract.statuses.building, assignee_id: null },
       b.ticket.updated_at
     );
-    await writer.postEvent(b.ticket.id, handBackComment(b, merged), memberId).catch(() => {
+    await writer.postEvent(b.ticket.id, handBackComment(b, merged), memberId, "conflict_handed_back").catch(() => {
       log.warn("handed back, but the note failed to post", t);
     });
     log.emit(
@@ -6675,17 +6759,93 @@ async function applyConflictBounce(writer, b, contract, memberId, log, dryRun) {
   }
 }
 
+// src/build-failure.ts
+import { StaleWriteError as StaleWriteError6 } from "@tablation/client";
+var BUILD_FAILURE_MARKER = "<!-- crew:build-failure -->";
+function failingPaths(output, cwd) {
+  const found = /* @__PURE__ */ new Set();
+  const ext = String.raw`[\w.-]+\.[A-Za-z]{1,6}`;
+  const dir = String.raw`(?:[\w@.~-]+/)*`;
+  const patterns = [
+    new RegExp(String.raw`^\s*(${dir}${ext})\(\d+,\d+\)\s*:`, "gm"),
+    new RegExp(String.raw`(?:^|\s)(/?${dir}${ext}):\d+(?::\d+)?`, "gm"),
+    new RegExp(String.raw`(?:FAIL|✖|✕|not ok \d+ -)\s+(/?${dir}${ext})`, "gm")
+  ];
+  const root = cwd.endsWith("/") ? cwd : `${cwd}/`;
+  for (const re of patterns) {
+    for (const m of output.matchAll(re)) {
+      let p = m[1];
+      if (p.startsWith("/")) {
+        if (!p.startsWith(root)) continue;
+        p = p.slice(root.length);
+      }
+      p = p.replace(/^\.\//, "");
+      if (p.includes("node_modules/")) continue;
+      found.add(p);
+    }
+  }
+  return [...found];
+}
+function culpritByFiles(candidates, paths) {
+  if (paths.length === 0) return null;
+  const hits = candidates.filter((c) => paths.some((p) => c.changed.has(p)));
+  return hits.length === 1 ? hits[0] : null;
+}
+function planBuildFailureBounce(cwd, c, base, remote) {
+  return { ...c, base, remote, worktree: worktreeForBranch(cwd, c.branch) };
+}
+function buildFailureComment(b) {
+  const how = b.how === "files" ? `The failing output names files that only \`${b.branch}\` changed among the branches in that release.` : `Built alone on the previous \`${b.base}\`, \`${b.branch}\` fails the same hook.`;
+  return `${BUILD_FAILURE_MARKER}
+**Back with the dev lane \u2014 this branch builds on its own but not on current \`${b.base}\`.**
+
+\`${b.branch}\` was verified, but the release's \`${b.hook}\` hook failed once it was merged with the other branches in the same release. ${how}
+
+First step: \`git merge ${b.remote}/${b.base}\` in ${b.worktree ? `\`${b.worktree}\`` : "the branch's worktree"}, rebuild, and fix what disagrees \u2014 the merge is clean at the text level, so this is a semantic conflict (a duplicate identifier, a changed signature) with work that landed on \`${b.base}\` after this branch was cut. Push as a plain fast-forward and hand back to QA as usual.
+
+\`\`\`
+${b.tail.trimEnd()}
+\`\`\`
+
+Nothing was written to \`${b.base}\` for this ticket; the rest of the release shipped without it.`;
+}
+async function applyBuildFailureBounce(writer, b, contract, memberId, log, dryRun) {
+  const t = { ticket: b.ticket.issue_id, step: "release" };
+  if (dryRun) {
+    log.emit(`would hand ${b.ticket.issue_id} back to ${contract.statuses.building} \u2014 its branch breaks the ${b.hook} hook`, t);
+    return { kind: "handed-back" };
+  }
+  try {
+    await writer.updateTicket(
+      b.ticket.id,
+      { status: contract.statuses.building, assignee_id: null },
+      b.ticket.updated_at
+    );
+    await writer.postEvent(b.ticket.id, buildFailureComment(b), memberId, "build_failed").catch(() => {
+      log.warn("handed back, but the note failed to post", t);
+    });
+    log.emit(`handed back to ${contract.statuses.building} \u2014 breaks the ${b.hook} hook once merged with the rest`, t);
+    return { kind: "handed-back" };
+  } catch (e) {
+    if (e instanceof StaleWriteError6) {
+      log.emit(`${b.ticket.issue_id} changed on the board since this run read it \u2014 not handing it back over it`, t);
+      return { kind: "stale" };
+    }
+    return { kind: "failed", why: e.message };
+  }
+}
+
 // src/stranded-verified.ts
 var STRANDED_MARKER = "<!-- crew:stranded-verified -->";
 var STRANDED_ESCALATE_AFTER_MS = 30 * 60 * 1e3;
-var priorStrandedFlags = (ticket, comments) => comments.filter((c) => c.ticket_id === ticket.id && (c.body ?? "").includes(STRANDED_MARKER)).length;
-function planStrandedVerified(candidate, base, remote, comments) {
-  const stamps = comments.filter((c) => c.ticket_id === candidate.ticket.id && (c.body ?? "").includes(STRANDED_MARKER)).map((c) => Date.parse(c.created_at)).filter((n) => !Number.isNaN(n));
+var priorStrandedFlags = (ticket, events) => events.filter((e) => e.record_id === ticket.id && e.event_type === "stranded_flagged").length;
+function planStrandedVerified(candidate, base, remote, events) {
+  const stamps = events.filter((e) => e.record_id === candidate.ticket.id && e.event_type === "stranded_flagged").map((e) => Date.parse(e.occurred_at ?? e.created_at ?? "")).filter((n) => !Number.isNaN(n));
   return {
     candidate,
     base,
     remote,
-    priorFlags: priorStrandedFlags(candidate.ticket, comments),
+    priorFlags: priorStrandedFlags(candidate.ticket, events),
     lastFlaggedAt: stamps.length ? Math.max(...stamps) : void 0
   };
 }
@@ -6723,7 +6883,7 @@ async function applyStrandedVerified(writer, s, contract, operatorId, memberId,
         status: contract.statuses.needsHuman,
         ...operatorId ? { assignee_id: operatorId } : {}
       });
-      await writer.postEvent(s.candidate.ticket.id, escalateComment2(s), memberId).catch(() => {
+      await writer.postEvent(s.candidate.ticket.id, escalateComment2(s), memberId, "stranded_escalated").catch(() => {
         log.warn("escalated, but the note failed to post", t);
       });
       log.warn(`still nothing to merge after a flag \u2014 escalated to ${contract.statuses.needsHuman}`, t);
@@ -6737,7 +6897,7 @@ async function applyStrandedVerified(writer, s, contract, operatorId, memberId,
     return { kind: "flagged" };
   }
   try {
-    await writer.postEvent(s.candidate.ticket.id, flagComment(s), memberId).catch(() => {
+    await writer.postEvent(s.candidate.ticket.id, flagComment(s), memberId, "stranded_flagged").catch(() => {
       log.warn("flagged, but the note failed to post", t);
     });
     log.emit("flagged as stranded at verified \u2014 will escalate if still unplaceable next cycle", t);
@@ -6772,7 +6932,7 @@ ${g.tail}
 \`\`\`
 `;
 }
-var alreadyReported = (ticket, g, comments) => comments.some((c) => c.ticket_id === ticket.id && (c.body ?? "").includes(`${GATE_RED_MARKER} ${g.sha.slice(0, 8)}`));
+var alreadyReported = (ticket, g, events) => events.some((e) => e.record_id === ticket.id && e.event_type === "gate_red" && (e.message ?? "").includes(`${GATE_RED_MARKER} ${g.sha.slice(0, 8)}`));
 
 // src/agent.ts
 import { spawn as spawn2 } from "child_process";
@@ -7196,12 +7356,12 @@ function expandSkillMarkers(text, lookup, opts = {}) {
 import { readFileSync as readFileSync10, readdirSync as readdirSync3, existsSync as existsSync11 } from "fs";
 import { join as join12 } from "path";
 import { parse as parseYaml } from "yaml";
-import { TablationClient as TablationClient3, StaleWriteError as StaleWriteError7, TablationApiError as TablationApiError2 } from "@tablation/client";
+import { TablationClient as TablationClient3, StaleWriteError as StaleWriteError8, TablationApiError as TablationApiError2 } from "@tablation/client";
 
 // src/agents.ts
 import { readFileSync as readFileSync9, existsSync as existsSync10 } from "fs";
 import { join as join11 } from "path";
-import { TablationClient as TablationClient2, StaleWriteError as StaleWriteError6, TablationApiError } from "@tablation/client";
+import { TablationClient as TablationClient2, StaleWriteError as StaleWriteError7, TablationApiError } from "@tablation/client";
 var PERSONA_NAME = {
   dev: "Developer",
   design: "Design",
@@ -7380,7 +7540,7 @@ async function syncPersonas(route, opts) {
         try {
           anchoredAt = (await client2.records.update(agentsModel.id, row.id, promptSourceFields(agentsModel, defaultPrompt), row.updated_at)).updated_at;
         } catch (e) {
-          if (!(e instanceof StaleWriteError6) && !isPromptGateRejection(e)) throw e;
+          if (!(e instanceof StaleWriteError7) && !isPromptGateRejection(e)) throw e;
         }
       }
       const historyId = await fetchLatestHistoryId(route, agentsModel.id, row.id, opts.userAgent);
@@ -7452,7 +7612,7 @@ async function syncPersonas(route, opts) {
       agentPersonas[role] = { agentId: row.id, lastSyncedUpdatedAt: updated.updated_at, ...historyId ? { historyId } : {} };
       outcomes.push({ role, action: "updated", agentId: row.id });
     } catch (e) {
-      if (e instanceof StaleWriteError6) {
+      if (e instanceof StaleWriteError7) {
         outcomes.push({ role, action: "diverged", agentId: row.id });
         continue;
       }
@@ -7636,7 +7796,7 @@ async function syncSkills(route, opts) {
             row.updated_at
           )).updated_at;
         } catch (e) {
-          if (!(e instanceof StaleWriteError7) && !isPromptGateRejection(e)) throw e;
+          if (!(e instanceof StaleWriteError8) && !isPromptGateRejection(e)) throw e;
         }
       }
       agentSkills[skill.name] = { skillId: row.id, lastSyncedUpdatedAt: anchoredAt };
@@ -7692,7 +7852,7 @@ async function syncSkills(route, opts) {
       agentSkills[skill.name] = { skillId: row.id, lastSyncedUpdatedAt: updated.updated_at };
       outcomes.push({ name: skill.name, action: "updated", skillId: row.id });
     } catch (e) {
-      if (e instanceof StaleWriteError7) {
+      if (e instanceof StaleWriteError8) {
         outcomes.push({ name: skill.name, action: "diverged", skillId: row.id });
         continue;
       }
@@ -8384,7 +8544,7 @@ async function enforceHandoff(i) {
       return { kind: "unverified", why: v.why };
     }
     if (v.kind === "pushed") {
-      await i.writer.postEvent(t.id, pushedEventBody(v.sha, v.remote, v.branch), i.seatId);
+      await i.writer.postEvent(t.id, pushedEventBody(v.sha, v.remote, v.branch), i.seatId, "pushed");
       i.log.emit(`${t.issue_id} pushed at ${v.sha.slice(0, 7)} \u2014 handed on to QA`, log);
       return { kind: "pushed", sha: v.sha };
     }
@@ -8394,7 +8554,7 @@ async function enforceHandoff(i) {
       ...t.assignee_id ? {} : { assignee_id: i.seatId },
       ...i.hold && !heldBy ? { [i.hold.column]: i.hold.shipId } : {}
     });
-    await i.writer.postEvent(t.id, bounceComment(v, i.shipName), i.seatId).catch(() => {
+    await i.writer.postEvent(t.id, bounceComment(v, i.shipName), i.seatId, "bounced").catch(() => {
       i.log.warn("bounced, but the note failed to post", log);
     });
     i.log.warn(`${t.issue_id} bounced back to ${i.contract.statuses.building}: ${v.reason}`, log);
@@ -8452,7 +8612,7 @@ async function retireRunWorktree(tracker, dir, cfg, t, role, seatId, shipName2,
     if (o.kind === "removed") log.emit(`${t.issue_id}: removed the run's worktree ${o.path} \u2014 the branch is on ${cfg.branch.remote}`, at);
     else if (o.kind === "kept-handoff") {
       log.emit(`${t.issue_id}: kept worktree ${o.path} \u2014 this repo's handoff hook serves from it`, at);
-      await tracker.postEvent(t.id, `worktree kept on ship ${shipName2} at \`${o.path}\`: the repo's \`handoff\` hook serves from it. QA on another ship cuts its own from \`${cfg.branch.remote}/${branch}\` instead.`, seatId);
+      await tracker.postEvent(t.id, `worktree kept on ship ${shipName2} at \`${o.path}\`: the repo's \`handoff\` hook serves from it. QA on another ship cuts its own from \`${cfg.branch.remote}/${branch}\` instead.`, seatId, "worktree_kept");
     } else if (o.kind === "kept-dirty") log.warn(`${t.issue_id}: left worktree ${o.path} \u2014 it has uncommitted changes`, at);
     else if (o.kind === "kept-unpushed") log.warn(`${t.issue_id}: left worktree ${o.path} \u2014 its branch has commits that are not on ${cfg.branch.remote}`, at);
   } catch (e) {
@@ -8981,7 +9141,7 @@ ${h.prFailure.tail}
         [contract.columns.status]: reviewing,
         ...h.prRef && columns.prRef ? { [contract.columns.prRef]: h.prRef } : {}
       }, t.updated_at);
-      await tracker.postEvent(t.id, lines.join("\n\n"), seat);
+      await tracker.postEvent(t.id, lines.join("\n\n"), seat, "review");
       emit.emit(`moved to ${reviewing}`, { ticket: t.issue_id });
     } catch (e) {
       emit.warn(`could not move to ${reviewing}: ${e.message}`, { ticket: t.issue_id });
@@ -8996,7 +9156,7 @@ ${h.prFailure.tail}
     }
     try {
       await tracker.updateTicket(t.id, { [contract.columns.status]: contract.statuses.handoff }, t.updated_at);
-      await tracker.postEvent(t.id, body, seat);
+      await tracker.postEvent(t.id, body, seat, "requeued");
       emit.emit(`returned to ${contract.statuses.handoff} \u2014 branch moved past ${q.verified.slice(0, 8)}`, { ticket: t.issue_id });
     } catch (e) {
       emit.warn(`could not return to ${contract.statuses.handoff}: ${e.message}`, { ticket: t.issue_id });
@@ -9474,8 +9634,62 @@ async function runRelease(o) {
     }
   }
 }
-async function mergeAndRelease(o, decision, fresh, releaseBranch) {
+var isolateCap = (o) => o.repo.release.isolateFailures?.maxCandidates ?? DEFAULT_ISOLATE_MAX_CANDIDATES;
+async function isolateAndRetry(o, name, output, merged, preMerge, decision, fresh, releaseBranch) {
+  const applied = merged.filter((c) => c.sha && c.branch);
+  const cap = isolateCap(o);
+  if (o.dryRun || !preMerge || applied.length < 2 || cap === 0) return null;
+  const head = headSha(o.cwd);
+  const withFiles = applied.map((c) => ({
+    c,
+    changed: new Set(
+      (gitOk(o.cwd, ["diff", "--name-only", `${preMerge}...${c.branch}`]) ?? "").split("\n").filter(Boolean)
+    )
+  }));
+  const culprits = [];
+  const byFile = culpritByFiles(withFiles, failingPaths(output, o.cwd));
+  if (byFile) {
+    o.emit.warn(`${name} failed; its output names files only ${byFile.c.branch} changed \u2014 treating it as the culprit`, { ticket: byFile.c.ticket.issue_id });
+    culprits.push({ c: byFile.c, how: "files", tail: lastLines(output, HOOK_FAILURE_TAIL_LINES) });
+  } else if (applied.length > cap) {
+    o.emit.warn(`${name} failed with ${applied.length} branches merged, over release.isolateFailures.maxCandidates (${cap}) \u2014 not isolating`);
+    return null;
+  } else {
+    o.emit.warn(`${name} failed with ${applied.length} branches merged \u2014 building each alone to find the culprit`);
+    for (const c of applied) {
+      gitOk(o.cwd, ["reset", "--hard", preMerge]);
+      const m = mergeOne(o, c);
+      if (typeof m === "object" && "conflict" in m) continue;
+      const r = await hook(o, name);
+      if (r && r.code !== 0) {
+        o.emit.warn(`${c.branch} fails the ${name} hook on its own`, { ticket: c.ticket.issue_id });
+        culprits.push({ c, how: "solo", tail: lastLines(r.output, HOOK_FAILURE_TAIL_LINES) });
+      }
+    }
+  }
+  gitOk(o.cwd, ["reset", "--hard", preMerge]);
+  if (culprits.length === 0) {
+    o.emit.warn("no single branch could be blamed \u2014 failing the release as a whole");
+    gitOk(o.cwd, ["reset", "--hard", head]);
+    return null;
+  }
+  const out = new Set(culprits.map((x) => x.c.ticket.id));
+  const rest = decision.merges.filter((c) => !out.has(c.ticket.id));
+  const retried = await mergeAndRelease(o, { ...decision, merges: rest }, fresh, releaseBranch, true);
+  return {
+    ...retried,
+    buildCulprits: culprits.map((x) => ({
+      ticket: x.c.ticket,
+      branch: x.c.branch,
+      hook: name,
+      tail: x.tail,
+      how: x.how
+    }))
+  };
+}
+async function mergeAndRelease(o, decision, fresh, releaseBranch, isolated = false) {
   o.emit.enter("merge");
+  const preMerge = o.dryRun ? "" : headSha(o.cwd);
   const merged = [];
   const conflicts = [];
   const unbuildable = [];
@@ -9590,6 +9804,10 @@ async function mergeAndRelease(o, decision, fresh, releaseBranch) {
         o.emit.warn("test gate failed with what looks like a worker crash, not a real failure \u2014 retrying once");
         r = await hook(o, "test");
       }
+      if (r && r.code !== 0 && !isolated) {
+        const out = await isolateAndRetry(o, "test", r.output, merged, preMerge, decision, fresh, releaseBranch);
+        if (out) return out;
+      }
       if (r && r.code !== 0) {
         o.emit.error(`test gate FAILED \u2014 not deploying; the target stays on the previous release`);
         const sha = headSha(o.cwd);
@@ -9619,13 +9837,23 @@ async function mergeAndRelease(o, decision, fresh, releaseBranch) {
     }
   }
   if (o.repo.hooks.build) {
-    if (o.dryRun) o.emit.emit(`would run: ${hookLabel(o.repo, "build")}`);
-    else {
+    if (o.dryRun) {
+      o.emit.emit(`would run: ${hookLabel(o.repo, "build")}`);
+      if (applied > 1) {
+        o.emit.emit(
+          `if it fails, would find the one of ${applied} merged branches that broke it (by the files it names, else by building each alone, up to ${isolateCap(o)}), hand that back to the dev lane and ship the rest`
+        );
+      }
+    } else {
       let r = await hook(o, "build");
       if (r && r.code !== 0 && looksLikeTransientResolveFailure(r.output)) {
         o.emit.warn("build failed with what looks like a transient dependency-resolution glitch, not a real failure \u2014 retrying once");
         r = await hook(o, "build");
       }
+      if (r && r.code !== 0 && !isolated) {
+        const out = await isolateAndRetry(o, "build", r.output, merged, preMerge, decision, fresh, releaseBranch);
+        if (out) return out;
+      }
       if (r && r.code !== 0) {
         o.emit.error("build failed \u2014 not deploying");
         if (version && !o.dryRun && headSha(o.cwd) !== preBump) {
@@ -9757,7 +9985,7 @@ async function applyFailureAlert(writer, n, route, openTickets, memberId, log, d
     }
     try {
       await writer.postEvent(existing.id, `${FAILURE_ALERT_MARKER}
-Still failing: ${n.detail ?? n.headline}`, memberId);
+Still failing: ${n.detail ?? n.headline}`, memberId, "failure_alert");
       log.warn(`release failure recurring \u2014 commented on ${label}`, { step: "release" });
       return { kind: "commented", issueId: label };
     } catch (e) {
@@ -9824,7 +10052,7 @@ function formatProjectsReport(report) {
 }
 
 // src/cli.ts
-import { StaleWriteError as StaleWriteError8 } from "@tablation/client";
+import { StaleWriteError as StaleWriteError9 } from "@tablation/client";
 
 // src/release-ship.ts
 var DAY_MS = 24 * 60 * 60 * 1e3;
@@ -10382,6 +10610,13 @@ async function discover(o) {
     } catch {
     }
   }
+  if (out.models.issues) {
+    try {
+      await get(o, `/data-models/${out.models.issues}/events?limit=1`);
+    } catch (e) {
+      if (e instanceof ConnectHttpError && e.status === 404) problems.push(EVENT_LOG_MISSING);
+    }
+  }
   out.areaModelId = find("Projects");
   out.reposModelId = find("Repos");
   out.shipsModelId = find("Ships");
@@ -12926,11 +13161,11 @@ ${changed2.map((c) => `  - ${c}
       if (outcome.gateRed && !dryRun) {
         try {
           const g = outcome.gateRed;
-          const comments = await tracker.comments();
+          const reported = await tracker.events("gate_red");
           const seat = releaseSeat(c);
           for (const t of gateRedTickets(outcome.merged, outcome.decision.merges)) {
-            if (alreadyReported(t, g, comments)) continue;
-            await tracker.postEvent(t.id, gateRedComment(g), seat);
+            if (alreadyReported(t, g, reported)) continue;
+            await tracker.postEvent(t.id, gateRedComment(g), seat, "gate_red");
             remit.warn(`test gate red for ${g.count} cycles \u2014 noted on the ticket`, { ticket: t.issue_id, step: "merge" });
           }
           state.release(scope).noteTestGateReported(g.sha);
@@ -12963,8 +13198,23 @@ ${changed2.map((c) => `  - ${c}
           });
         }
       }
+      if (outcome.buildCulprits?.length) {
+        const seat = releaseSeat(c);
+        for (const cu of outcome.buildCulprits) {
+          try {
+            const bounce = planBuildFailureBounce(target.dir, cu, repo.branch.base, repo.branch.remote);
+            const r = await applyBuildFailureBounce(tracker, bounce, tracker.contract, seat, remit, dryRun);
+            if (r.kind === "failed") {
+              remit.warn(`could not hand back ${cu.ticket.issue_id}: ${r.why}`, { step: "release" });
+            }
+          } catch (e) {
+            remit.warn(`could not hand back ${cu.ticket.issue_id}: ${e.message}`, { step: "release" });
+          }
+        }
+      }
       if (outcome.conflicts?.length || outcome.unbuildable?.length) {
-        const comments = await tracker.comments();
+        const handedBack = await tracker.events("conflict_handed_back");
+        const flagged = await tracker.events("stranded_flagged");
         const seat = releaseSeat(c);
         for (const f of outcome.conflicts ?? []) {
           try {
@@ -12974,7 +13224,7 @@ ${changed2.map((c) => `  - ${c}
               f.candidate.branch,
               repo.branch.base,
               f.paths,
-              comments
+              handedBack
             );
             const r = await applyConflictBounce(tracker, bounce, tracker.contract, seat, remit, dryRun);
             if (r.kind === "failed") {
@@ -12989,7 +13239,7 @@ ${changed2.map((c) => `  - ${c}
         }
         for (const cand of outcome.unbuildable ?? []) {
           try {
-            const stranded = planStrandedVerified(cand, repo.branch.base, repo.branch.remote, comments);
+            const stranded = planStrandedVerified(cand, repo.branch.base, repo.branch.remote, flagged);
             const r = await applyStrandedVerified(
               tracker,
               stranded,
@@ -14658,7 +14908,7 @@ Signed in as ${result.identity.email} \u2014 minted an API key in workspace "${r
         await tracker3.updateTicket(ticket.id, { [c.columns.assignee]: null }, ticket.updated_at);
         emit.emit(`cleared assignee on ${label} \u2014 back up for grabs next cycle`);
       } catch (e) {
-        if (e instanceof StaleWriteError8) {
+        if (e instanceof StaleWriteError9) {
           process.stderr.write(`unassign: ${label} changed on the board since it was read \u2014 re-run to retry
 `);
           process.exit(2);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.43.4",
+  "version": "0.45.0",
   "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/personas/lane-qa.md

@@ -56,7 +56,7 @@ pass.
 4. **Get the branch and read the diff.** The branch on the remote is what
    you test, and you never need the builder's worktree: cut your own with
    `crew worktree <number>` (it prints the path), unless the builder's
-   hand-off event says its ship kept a worktree and a server for you, in
+   hand-off event (in the ticket's event log) says its ship kept a worktree and a server for you, in
    which case you may open that URL instead. Remove yours when you hand the
    ticket on. The digest's `branch` column reads `<name> (on <remote> only)`
    when another ship built it — still testable; only **MISSING** (gone
@@ -65,7 +65,9 @@ pass.
 
    **Name the sha you test.** Your starting comment states the
    `<remote>/<branch>` tip you fetched. The runner records the pushed tip on
-   the ticket as `pushed <sha> to <remote>/<branch>`; if your tip differs
+   the ticket's activity (its event log, not a comment) as a `pushed` event
+   reading `pushed <sha> to <remote>/<branch>`; read it with the events list
+   on the Issues table, filtered to the `pushed` type. If your tip differs
    from the latest such event, say so and verify the remote tip anyway — the
    remote is the only git truth.
 
@@ -161,6 +163,11 @@ pass.
   base, the bounce says to merge `<remote>/<base>` into the branch, resolve and
   push fast-forward; a rebase would need a force push the builder cannot make.
   Or pass the ticket and let the release's conflict path do the merge.
+- **A branch that came back from a conflict or build-failure bounce is
+  re-verified against the current base, not only its own tip.** Before
+  passing it, merge current `<remote>/<base>` into the worktree (or run the
+  build on a throwaway merge) and run the build: a clean text merge can still
+  fail to compile, and it is far cheaper to find that here than at release.
 - **You only ever enter the worktree of the ticket you are verifying.**
 - Throwaway verification scripts go in a scratch directory outside the
   repository — not the worktree root either.

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

@@ -363,7 +363,7 @@ before going further, and never change a ticket's `report_type` yourself.
    `fixed` and `<remote>/<branch>` is missing or not at your worktree's
    HEAD, it sets the ticket back to `in_progress` with an event saying what
    it found, and QA never sees it. When it matches, the runner records
-   `pushed <sha> to <remote>/<branch>` on the ticket; you still state the sha
+   `pushed <sha> to <remote>/<branch>` in the ticket's activity (its event log); you still state the sha
    in your last progress comment.
 
    **The branch on the remote is what QA tests.** Leave it unmerged, and do

package/prompts/dev-qa/personas/lane-qa.md

@@ -55,7 +55,7 @@ pass.
 4. **Get the branch and read the diff.** The branch on the remote is what
    you test, and you never need the builder's worktree: cut your own with
    `crew worktree <number>` (it prints the path), unless the builder's
-   hand-off event says its ship kept a worktree and a server for you, in
+   hand-off event (in the ticket's event log) says its ship kept a worktree and a server for you, in
    which case you may open that URL instead. Remove yours when you hand the
    ticket on. The digest's `branch` column reads `<name> (on <remote> only)`
    when another ship built it — still testable; only **MISSING** (gone
@@ -64,7 +64,9 @@ pass.
 
    **Name the sha you test.** Your starting comment states the
    `<remote>/<branch>` tip you fetched. The runner records the pushed tip on
-   the ticket as `pushed <sha> to <remote>/<branch>`; if your tip differs
+   the ticket's activity (its event log, not a comment) as a `pushed` event
+   reading `pushed <sha> to <remote>/<branch>`; read it with the events list
+   on the Issues table, filtered to the `pushed` type. If your tip differs
    from the latest such event, say so and verify the remote tip anyway — the
    remote is the only git truth.
 
@@ -160,6 +162,11 @@ pass.
   base, the bounce says to merge `<remote>/<base>` into the branch, resolve and
   push fast-forward; a rebase would need a force push the builder cannot make.
   Or pass the ticket and let the release's conflict path do the merge.
+- **A branch that came back from a conflict or build-failure bounce is
+  re-verified against the current base, not only its own tip.** Before
+  passing it, merge current `<remote>/<base>` into the worktree (or run the
+  build on a throwaway merge) and run the build: a clean text merge can still
+  fail to compile, and it is far cheaper to find that here than at release.
 - **You only ever enter the worktree of the ticket you are verifying.**
 - Throwaway verification scripts go in a scratch directory outside the
   repository — not the worktree root either.

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

@@ -363,7 +363,7 @@ before going further, and never change a ticket's `report_type` yourself.
    `fixed` and `<remote>/<branch>` is missing or not at your worktree's
    HEAD, it sets the ticket back to `in_progress` with an event saying what
    it found, and QA never sees it. When it matches, the runner records
-   `pushed <sha> to <remote>/<branch>` on the ticket; you still state the sha
+   `pushed <sha> to <remote>/<branch>` in the ticket's activity (its event log); you still state the sha
    in your last progress comment.
 
    **The branch on the remote is what QA tests.** Leave it unmerged, and do