@tablation/crew 0.21.0 → 0.23.0

package/dist/cli.js

@@ -3273,6 +3273,26 @@ function isAutomatic(t, wm) {
   if (!wm || wm.modes.size === 0) return true;
   return modeOf(t, wm) === "automatic";
 }
+function skippedByMode(tickets, wm) {
+  const out = { manual: 0, hybrid: 0 };
+  if (!wm || wm.modes.size === 0) return out;
+  const kept = new Set(withinWorkMode(tickets, wm));
+  for (const t of tickets) {
+    if (kept.has(t)) continue;
+    out[modeOf(t, wm) === "manual" ? "manual" : "hybrid"]++;
+  }
+  return out;
+}
+function withoutManualVerified(tickets, verifiedStatus, wm) {
+  if (!wm || wm.modes.size === 0) return tickets;
+  return tickets.filter((t) => t.status !== verifiedStatus || modeOf(t, wm) !== "manual");
+}
+function projectModeLabels(rows) {
+  return rows.map((r) => {
+    const name = r.name ?? r.id;
+    return { id: r.id, name, mode: r.mode, label: `${name} (${r.mode})` };
+  });
+}
 
 // src/board-lock.ts
 import { StaleWriteError } from "@tablation/client";
@@ -3517,6 +3537,12 @@ var Tracker = class {
     if (!model) return [];
     return this.client.records.list(model, { limit: 200 });
   }
+  /** Every Projects row with its raw `work_mode`, for `crew status`. */
+  async projectRowsWithModes() {
+    const rows = await this.projectRows();
+    const modes = await this.projectWorkModes();
+    return rows.map((r) => ({ ...r, mode: modes.get(r.id) ?? "automatic" }));
+  }
   /** Create a `Repos` row. Throws when the workspace has no Repos table. */
   async createRepoRow(fields) {
     const model = this.route.resolved?.reposModelId;
@@ -3929,9 +3955,9 @@ function sweepDiagnostics(tickets, info) {
   return { dangling, selfBlocked: [...new Set(selfBlocked)] };
 }
 var describeBlockers = (t, info) => (t.blocked_by ?? []).map((id) => `${info[id]?.issue_id ?? "?"} (${info[id]?.status ?? "unknown"})`).join(", ");
-function planSweep(tickets, info, blocked) {
+function planSweep(tickets, info, blocked, wm) {
   const steps = [];
-  for (const t of tickets) {
+  for (const t of withinWorkMode(tickets, wm)) {
     if (t.status === "accepted" && blocked.has(t.id)) {
       steps.push({ action: "park", ticket: t, blockers: describeBlockers(t, info), to: "blocked" });
     } else if (t.status === "blocked" && !blocked.has(t.id)) {
@@ -4500,6 +4526,9 @@ function blockers(t, i) {
   });
   return list.length === 0 ? "\u2014" : list.join(", ");
 }
+function modeCell(t, i) {
+  return i.workModes && i.workModes.modes.size > 0 ? modeOf(t, i.workModes) : "\u2014";
+}
 var sorted = (ts) => [...ts].sort(compareRank);
 var stream = (segments) => `${segments.join("\n")}
 `;
@@ -4511,14 +4540,14 @@ ${sorted(ts).map(row).join("\n")}
 `;
 }
 function buildingDigest(i) {
-  const header = "| ticket | repo | branch | worktree | 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 | 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} | ${t.status} | ${who(t, i)} | ${filedBy(t, i)} | ${t.severity ?? "\u2014"} | ${t.priority ?? "\u2014"} | p${effectivePriority(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"} | p${effectivePriority(t)} | ${t.updated_at.slice(0, 16)}Z | ${lastComment(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" : `
@@ -4564,7 +4593,7 @@ ${sorted(held).map((t) => `| ${displayKey2(t)} | ${stopOf(t)} |`).join("\n")}
   ]);
 }
 function qaDigest(i) {
-  const header = "| ticket | repo | status | built by | assignee | filed by | sev | pri | eff | branch | worktree | updated | last comment | new since last poll |\n|---|---|---|---|---|---|---|---|---|---|---|---|---|---|";
+  const header = "| ticket | repo | mode | status | built by | assignee | filed by | sev | pri | eff | branch | worktree | updated | last comment | new since last poll |\n|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|";
   const branchCell = (t) => {
     if (i.dirFor && i.dirFor(t) === null) return "\u2014";
     if (!i.existingBranchFor) return "";
@@ -4579,7 +4608,7 @@ function qaDigest(i) {
     const n = newFromOthers(i, t.id);
     const dir = i.dirFor ? i.dirFor(t) : null;
     const repo = i.dirFor ? dir ?? "**NO CHECKOUT**" : "";
-    return `| ${displayKey2(t)} | ${repo} | ${t.status} | ${builtBy(t)} | ${who(t, i)} | ${filedBy(t, i)} | ${t.severity ?? "\u2014"} | ${t.priority ?? "\u2014"} | p${effectivePriority(t)} | ${branchCell(t)} | ${worktreeCell(t)} | ${t.updated_at.slice(0, 16)}Z | ${lastComment(i, t.id)} | ${n > 0 ? `**${n} new**` : "\u2014"} |`;
+    return `| ${displayKey2(t)} | ${repo} | ${modeCell(t, i)} | ${t.status} | ${builtBy(t)} | ${who(t, i)} | ${filedBy(t, i)} | ${t.severity ?? "\u2014"} | ${t.priority ?? "\u2014"} | p${effectivePriority(t)} | ${branchCell(t)} | ${worktreeCell(t)} | ${t.updated_at.slice(0, 16)}Z | ${lastComment(i, t.id)} | ${n > 0 ? `**${n} new**` : "\u2014"} |`;
   };
   return stream([
     "## Current queue \u2014 built for you by the poll\n",
@@ -4649,7 +4678,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()) {
+function planStalled(tickets, comments, ships, contract, holds2 = /* @__PURE__ */ new Set(), now = Date.now(), wm) {
   const watched = /* @__PURE__ */ new Set([contract.statuses.building, contract.statuses.handoff, contract.statuses.verifying]);
   const byTicket = /* @__PURE__ */ new Map();
   for (const c of comments) {
@@ -4658,7 +4687,7 @@ function planStalled(tickets, comments, ships, contract, holds2 = /* @__PURE__ *
     else byTicket.set(c.ticket_id, [c]);
   }
   const out = [];
-  for (const t of tickets) {
+  for (const t of withinWorkMode(tickets, wm)) {
     if (!watched.has(t.status)) continue;
     if (t.assignee_id && holds2.has(t.assignee_id)) continue;
     const mine = byTicket.get(t.id) ?? [];
@@ -4782,9 +4811,15 @@ async function decideCycle(o) {
   if (diag.selfBlocked.length) {
     emit.warn(`ticket(s) blocking themselves, parked permanently: ${diag.selfBlocked.join(", ")}`);
   }
-  const sweep = planSweep(tickets, info, blocked);
+  const sweep = planSweep(tickets, info, blocked, workModes);
   const holdReleases = planHoldReleases(tickets, tracker.contract);
-  const stalled = planStalled(tickets, comments, ships, tracker.contract, holds2);
+  const stalled = planStalled(tickets, comments, ships, tracker.contract, holds2, Date.now(), workModes);
+  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)`, {
+      data: { sweepSkipped: skipped }
+    });
+  }
   const blockedIssues = tickets.filter((t) => blocked.has(t.id)).map((t) => t.issue_id);
   emit.emit(`${blockedIssues.length} ticket(s) parked as blocked`, { data: { blocked: blockedIssues } });
   const qaHeld = tickets.filter((t) => t.status === tracker.contract.statuses.verifying).map((t) => t.issue_id);
@@ -5446,6 +5481,18 @@ function renderEnvironment(i) {
       "another (the digest prints other ships by name) \u2014 write its name, never a Ships row id."
     );
   }
+  const seatIds = Object.entries(route.resolved?.seats ?? {}).filter(([, id]) => !!id);
+  if (seatIds.length > 0) {
+    lines.push(
+      "",
+      "## Seats on this ship",
+      "",
+      "The Crew row id of each role's seat on this ship, to write verbatim where a hand-off",
+      "assigns the next lane (a ticket whose digest `mode` is `hybrid`):",
+      "",
+      ...seatIds.map(([role, id]) => `- ${role}: \`${id}\``)
+    );
+  }
   lines.push(
     "",
     "## What the columns are called here",
@@ -11019,7 +11066,12 @@ ${changed2.map((c) => `  - ${c}
         let seen = precheckTickets.get(c.route);
         if (!seen) precheckTickets.set(c.route, seen = tracker.openTickets());
         const { byRepo: preByRepo, unplaceable: preUnplaceable } = ticketsByRepo(c, await seen);
-        const pre = preByRepo.get(target.name) ?? [];
+        const preModes = { modes: await tracker.projectWorkModes(), agentAssignees: /* @__PURE__ */ new Set() };
+        const pre = withoutManualVerified(
+          preByRepo.get(target.name) ?? [],
+          tracker.contract.statuses.verified,
+          preModes
+        );
         reportUnplaceable(c.route, preUnplaceable, tracker.contract.statuses.verified, remit);
         const d = decideRelease(target.dir, pre, tracker.contract, {
           tagPattern: repo.release.tagPattern ?? "v*",
@@ -11067,7 +11119,12 @@ ${changed2.map((c) => `  - ${c}
       }
       const all = await tracker.openTickets();
       const { byRepo, unplaceable } = ticketsByRepo(c, all);
-      const tickets = byRepo.get(target.name) ?? [];
+      const releaseModes = { modes: await tracker.projectWorkModes(), agentAssignees: /* @__PURE__ */ new Set() };
+      const tickets = withoutManualVerified(
+        byRepo.get(target.name) ?? [],
+        tracker.contract.statuses.verified,
+        releaseModes
+      );
       reportUnplaceable(c.route, unplaceable, tracker.contract.statuses.verified, remit);
       const outcome = await runRelease({
         cwd: target.dir,
@@ -12952,6 +13009,7 @@ Signed in as ${result.identity.email} \u2014 minted an API key in workspace "${r
       const engagedTickets = engagedTicketIds.length ? await statusTracker.ticketsByIds(engagedTicketIds) : [];
       const issueLabelById = new Map(engagedTickets.map((t) => [t.id, t.issue_id]));
       const todo = await statusTracker.openTickets().then((open) => operatorTodo(open, statusTracker.contract, route.resolved?.operator ?? "")).catch(() => []);
+      const projects = projectModeLabels(await statusTracker.projectRowsWithModes().catch(() => []));
       if (flag("json")) {
         const repos = await resolvedRepos(route);
         const seatIds = route.resolved?.seats ?? {};
@@ -13001,6 +13059,8 @@ Signed in as ${result.identity.email} \u2014 minted an API key in workspace "${r
           watermark: state.watermark(),
           route: {
             name: route.route,
+            // Additive (CREW-1496): each project and its work mode. `version` stays 1.
+            projects: projects.map((p) => ({ id: p.id, name: p.name, workMode: p.mode })),
             dir: route.dir,
             enabled: route.enabled,
             area: route.area ?? null,
@@ -13058,6 +13118,8 @@ watermark:  ${state.watermark()}
         if (state.isRolePaused(r)) process.stdout.write(`role ${r}: paused
 `);
       }
+      if (projects.length) process.stdout.write(`projects:   ${projects.map((p) => p.label).join(", ")}
+`);
       for (const i of openShipAttention(state)) {
         process.stdout.write(`attention:  ${i.kind} \u2014 ${i.message}
 `);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.21.0",
+  "version": "0.23.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/common.md

@@ -305,10 +305,18 @@ Never start work on a ticket without first setting `assignee_id` to your
 own id (Step 3.3 already does this) — that claim is itself part of what
 keeps a concurrent interactive session from re-entering the same ticket.
 
-A project in **manual work mode** (its Projects row says so) is never consumed
-automatically: the digest lists its tickets only when a person has assigned them to
-your seat, and status alone never makes one yours. The digest says so when it
-applies; do not go looking for the missing tickets. QA is unaffected.
+Every ticket's project has a **work mode**, shown in the digest's `mode`
+column: `automatic` (lanes work tickets by status), `hybrid` (a lane considers a
+ticket only when it is assigned to an agent — a Crew row backed by an Agent, not a
+hold), or `manual` (no automation touches it; a ticket with no project counts as
+manual). The digest says so when a project is not automatic; do not go looking for
+a ticket it leaves out. **One hand-off rule:** in a `hybrid` project, hand a ticket
+to the next lane by *assigning that lane's seat on this ship* (ids under "Seats on
+this ship" in the Environment section, else the roster); in an `automatic` project,
+clear `assignee_id` as before. If there is no seat for the target role, assign any
+Crew row with that role's Agent; if there is none, clear it and say so in the
+hand-off comment. Parks (`needs_info`, `needs_planning` → the operator) are the
+same in every mode.
 
 Never touch a ticket still at `new` — only `accepted` tickets are yours to
 pick up; triage (a separate process) is what promotes `new` → `accepted`/
@@ -558,7 +566,9 @@ if `report_type` is either of those.
    this is your own isolated worktree, no special permission needed for this
    part).
 9. Set the ticket's `status` to `fixed` once you have verified it yourself,
-   and **clear `assignee_id`** — that pair is the hand-off to the QA lane.
+   and hand the ticket to the QA lane: in an `automatic` project **clear `assignee_id`**; in a
+   `hybrid` project **set `assignee_id` to this ship's QA seat** instead (the digest's `mode`
+   column says which). That pair is the hand-off to the QA lane.
    If the Issues table has a `held_by_ship_id` column, clear it in the same
    write (the hold is for the building roles; QA on any ship may take a `fixed`
    ticket). A QA bounce or `needs_info` park leaves it set; QA clears it again

package/prompts/default/personas/lane-design.md

@@ -98,6 +98,11 @@ stopping for this run, do the following instead:
     it) — revise the comp against what they said and post again. This cycle
     repeats as many times as the operator wants.
 
+  In a `hybrid` project (the digest's `mode` column), where the operator
+  leaves the hand-off to you, assign this ship's Developer seat when
+  `needs_design` is cleared rather than leaving the assignee blank, so the dev lane
+  still sees the ticket; in an `automatic` project nothing changes.
+
   Never implement past the comp to "just finish it" because the fix looks
   small, and never set `needs_design`, `accepted`, or clear
   `needs_planning`/`needs_review` yourself — all of that is the operator's

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

@@ -141,6 +141,11 @@ human review, so "probably fine" is not a pass.
      bump the version, and deploy this cycle or the next. Say in
      the comment what you exercised, so the record shows what "verified"
      covered.
+   - **Person-built hybrid ticket with no branch** — a `hybrid` ticket at
+     `fixed` whose branch under the repo's naming convention does not exist
+     (the digest's `branch` column says **MISSING**) is not a bounce to a
+     builder: set `needs_info` and assign the operator, with a comment naming the
+     branch you expected. A person built it; the operator has to push it.
    - **It doesn't** → `status` = `in_progress`, `assignee_id` = the building
      lane's own Crew row id from the roster above — the dev seat if
      `needs_design` is false or null, the design seat if true. **Exception:** when the Issues table has a `held_by_ship_id` column, **clear** `assignee_id` *and* `held_by_ship_id` instead of naming a seat — the

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

@@ -286,6 +286,19 @@ Never start work on a ticket without first setting `assignee_id` to your
 own id (Step 3.3 already does this) — that claim is itself part of what
 keeps a concurrent interactive session from re-entering the same ticket.
 
+Every ticket's project has a **work mode**, shown in the digest's `mode`
+column: `automatic` (lanes work tickets by status), `hybrid` (a lane considers a
+ticket only when it is assigned to an agent — a Crew row backed by an Agent, not a
+hold), or `manual` (no automation touches it; a ticket with no project counts as
+manual). The digest says so when a project is not automatic; do not go looking for
+a ticket it leaves out. **One hand-off rule:** in a `hybrid` project, hand a ticket
+to the next lane by *assigning that lane's seat on this ship* (ids under "Seats on
+this ship" in the Environment section, else the roster); in an `automatic` project,
+clear `assignee_id` as before. If there is no seat for the target role, assign any
+Crew row with that role's Agent; if there is none, clear it and say so in the
+hand-off comment. Parks (`needs_info`, `needs_planning` → the operator) are the
+same in every mode.
+
 Never touch a ticket still at `new` — only `accepted` tickets are yours to
 pick up; triage (a separate process) is what promotes `new` → `accepted`/
 `needs_info`.
@@ -535,7 +548,9 @@ if `report_type` is either of those.
    this is your own isolated worktree, no special permission needed for this
    part).
 9. Set the ticket's `status` to `fixed` once you have verified it yourself,
-   and **clear `assignee_id`** — that pair is the hand-off to the QA lane.
+   and hand the ticket to the QA lane: in an `automatic` project **clear `assignee_id`**; in a
+   `hybrid` project **set `assignee_id` to this ship's QA seat** instead (the digest's `mode`
+   column says which). That pair is the hand-off to the QA lane.
    If the Issues table has a `held_by_ship_id` column, clear it in the same
    write (the hold is for the building roles; QA on any ship may take a `fixed`
    ticket). A QA bounce or `needs_info` park leaves it set; QA clears it again

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

@@ -128,6 +128,11 @@ human review, so "probably fine" is not a pass.
      bump the version, and deploy this cycle or the next. Say in
      the comment what you exercised, so the record shows what "verified"
      covered.
+   - **Person-built hybrid ticket with no branch** — a `hybrid` ticket at
+     `fixed` whose branch under the repo's naming convention does not exist
+     (the digest's `branch` column says **MISSING**) is not a bounce to a
+     builder: set `needs_info` and assign the operator, with a comment naming the
+     branch you expected. A person built it; the operator has to push it.
    - **It doesn't** → `status` = `in_progress`, `assignee_id` = the dev
      seat's own Crew row id from the roster above. **Exception:** when the Issues table has a `held_by_ship_id` column, **clear** `assignee_id` *and* `held_by_ship_id` instead of naming a seat — the
      hold on a `qa` ticket is yours, as verifier, and seats are per ship, so