@tablation/crew 0.44.0 → 0.45.1

package/dist/cli.js

@@ -3234,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"
   },
@@ -3345,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";
@@ -5209,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) }
     });
   }
   /**
@@ -5495,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" });
       }
@@ -5608,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 {
   }
@@ -5641,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 {
   }
@@ -5711,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)
@@ -5968,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) {
@@ -6177,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) {
@@ -6190,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))
@@ -6232,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) {
@@ -6266,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 }
@@ -6316,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)`, {
@@ -6602,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) {
@@ -6656,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);
@@ -6693,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(
@@ -6772,7 +6821,7 @@ async function applyBuildFailureBounce(writer, b, contract, memberId, log, dryRu
       { status: contract.statuses.building, assignee_id: null },
       b.ticket.updated_at
     );
-    await writer.postEvent(b.ticket.id, buildFailureComment(b), memberId).catch(() => {
+    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);
@@ -6789,14 +6838,14 @@ async function applyBuildFailureBounce(writer, b, contract, memberId, log, dryRu
 // 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
   };
 }
@@ -6834,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);
@@ -6848,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);
@@ -6883,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";
@@ -8495,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 };
     }
@@ -8505,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);
@@ -8563,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) {
@@ -9092,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 });
@@ -9107,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 });
@@ -9936,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) {
@@ -10561,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");
@@ -13105,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);
@@ -13157,7 +13213,8 @@ ${changed2.map((c) => `  - ${c}
         }
       }
       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 {
@@ -13167,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") {
@@ -13182,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,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.44.0",
+  "version": "0.45.1",
   "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.
 

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.
 

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