@tablation/crew 0.19.4 → 0.20.0

package/dist/cli.js

@@ -1792,6 +1792,7 @@ var DEFAULT_CONTRACT = {
     blockedBy: "blocked_by",
     updatedAt: "updated_at",
     needsDesign: "needs_design",
+    workMode: "work_mode",
     reportType: "report_type",
     epic: "epic_id",
     author: "filed_by_id",
@@ -1840,6 +1841,7 @@ var DEFAULT_CONTRACT = {
   severityOrder: ["s1", "s2", "s3", "s4"],
   unknownPriorityRank: 2,
   coordinatingValue: "coordinating",
+  manualWorkMode: "manual",
   repoExemptReportTypes: ["coordinating", "question", "investigation"]
 };
 function closedStatuses(c) {
@@ -1860,6 +1862,7 @@ function resolveContract(override) {
     severityOrder: override.severityOrder ?? DEFAULT_CONTRACT.severityOrder,
     unknownPriorityRank: override.unknownPriorityRank ?? DEFAULT_CONTRACT.unknownPriorityRank,
     coordinatingValue: override.coordinatingValue ?? DEFAULT_CONTRACT.coordinatingValue,
+    manualWorkMode: override.manualWorkMode ?? DEFAULT_CONTRACT.manualWorkMode,
     // A workspace overriding `coordinatingValue` alone (renaming the rollup
     // marker) still gets that renamed value exempted here by default,
     // without also having to repeat it — only an explicit
@@ -3239,6 +3242,17 @@ var SHELL_WORDS = /* @__PURE__ */ new Set([
   "ok"
 ]);
 
+// src/work-mode.ts
+function manualProjectIds(rows, contract) {
+  const out = /* @__PURE__ */ new Set();
+  for (const r of rows) if (r[contract.columns.workMode] === contract.manualWorkMode) out.add(r.id);
+  return out;
+}
+function withinWorkMode(tickets, me, manual) {
+  if (!manual || manual.size === 0) return tickets;
+  return tickets.filter((t) => !t.project_id || !manual.has(t.project_id) || !!me && t.assignee_id === me);
+}
+
 // src/board-lock.ts
 import { StaleWriteError } from "@tablation/client";
 function releaseLockScope(route, repo) {
@@ -3460,6 +3474,22 @@ var Tracker = class {
       return void 0;
     }
   }
+  /**
+   * Ids of the Projects (area) rows in `manual` work mode (CREW-1445). One
+   * extra read per cycle. A workspace with no Projects table, no `work_mode`
+   * column, a null value or an unreadable table yields the empty set: every
+   * project stays automatic, exactly as before the column existed.
+   */
+  async manualProjectIds() {
+    const model = this.route.resolved?.areaModelId;
+    if (!model) return /* @__PURE__ */ new Set();
+    try {
+      const rows = await this.client.records.list(model, { limit: 200 });
+      return manualProjectIds(rows, this.contract);
+    } catch {
+      return /* @__PURE__ */ new Set();
+    }
+  }
   /** Every `Projects` (area) row — what `crew repos add` offers when a new Repos row needs one. */
   async projectRows() {
     const model = this.route.resolved?.areaModelId;
@@ -4154,7 +4184,7 @@ function notHeldByOtherShip(slice, i) {
 function buildingRoleHasWork(role, i) {
   const me = i.seats[role];
   if (!me) return { hasWork: false, reason: "this ship does not crew that role" };
-  const mine = notInStoppedRepo(notHeldByOtherShip(sliceFor(i.tickets, role), i), i);
+  const mine = withinWorkMode(notInStoppedRepo(notHeldByOtherShip(sliceFor(i.tickets, role), i), i), me, i.manualProjects);
   const startable = mine.filter(
     (t) => (t.status === "accepted" || t.status === "blocked") && !i.blocked.has(t.id) && !isHeld(t, i.holds)
   );
@@ -4204,7 +4234,7 @@ function roleCandidates(role, i) {
   if (role === "pair") return [];
   const me = i.seats[role];
   if (!me) return [];
-  return notInStoppedRepo(notHeldByOtherShip(sliceFor(i.tickets, role), i), i).filter(
+  return withinWorkMode(notInStoppedRepo(notHeldByOtherShip(sliceFor(i.tickets, role), i), i), me, i.manualProjects).filter(
     (t) => ((t.status === "accepted" || t.status === "blocked") && !i.blocked.has(t.id) || t.status === "in_progress" && (t.assignee_id === me || !t.assignee_id)) && !isHeld(t, i.holds)
   );
 }
@@ -4499,6 +4529,7 @@ ${sorted(held).map((t) => `| ${displayKey2(t)} | ${stopOf(t)} |`).join("\n")}
     "\nAlready filtered to your lane, and already ordered by the Step 2 rule.\nTicket bodies are deliberately omitted: fetch the full record of only the\nticket you actually pick up. **Do not re-fetch the whole tracker.** This\ndigest comes from the same API call the poll just made, moments ago.\n",
     '\n"new since last poll" counts comments from someone other than you since\nthe poll watermark \u2014 the same signal that woke this run.\n',
     "\n**`repo` is the checkout a ticket's work happens in** \u2014 an area spans\nseveral repositories, so cut the worktree beside THAT directory, not beside\nwhichever one you started in. A ticket marked **NO CHECKOUT** is not yours:\nthis ship has no clone of its repository, and another ship may serve it.\n\n**Use the `branch` and `worktree` columns verbatim.** They are that\nrepository's own naming convention, rendered for THIS ticket \u2014 cut the\nworktree at `<repo>/../<worktree>`, on branch `<branch>`. Do not derive\neither yourself, and prefer these over the Environment section's own worked\nexample if the two ever disagree \u2014 that example is illustrative only,\nrendered before any ticket is picked.\n",
+    ...i.manualProjects && i.manualProjects.size > 0 ? ["\n**Manual projects.** Some projects on this board are in manual work mode: their tickets appear below only when a person has assigned them to YOU. An unassigned `accepted` ticket in such a project is not yours to claim and is deliberately absent \u2014 do not go looking for it.\n"] : [],
     "\n### Step 1 \u2014 open tickets that may be yours to act on\n",
     "\n`fixed` tickets are deliberately absent: they belong to the QA lane.\n",
     table(step1, i, header, row),
@@ -4686,12 +4717,13 @@ 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] = await Promise.all([
+  const [tickets, comments, crewRows, epicRows, ships, manualProjects] = await Promise.all([
     tracker.openTickets(),
     tracker.comments(200),
     tracker.crewRows(),
     tracker.epicRows(),
-    tracker.shipRows()
+    tracker.shipRows(),
+    tracker.manualProjectIds()
   ]);
   emit.emit(`${tickets.length} open ticket(s), ${comments.length} comment(s)`, {
     data: { tickets: tickets.length, comments: comments.length }
@@ -4799,6 +4831,7 @@ async function decideCycle(o) {
   await checkRepoDisks({ state, emit, route, ship });
   const selectionInput = {
     repoStop: repoStopFor(openShipAttention(state), route),
+    manualProjects,
     tickets,
     comments,
     watermark,
@@ -4893,7 +4926,8 @@ function writeDigest(o, d, role, stateDir) {
         }
         return existingBranchForTicket(dir, repoFor(dir), t, role);
       },
-      tickets: role === "triage" ? d.tickets.filter((t) => t.assignee_id === o.route.resolved?.seats.triage) : sliceFor(d.tickets, role),
+      tickets: role === "triage" ? d.tickets.filter((t) => t.assignee_id === o.route.resolved?.seats.triage) : role === "dev" || role === "design" ? withinWorkMode(sliceFor(d.tickets, role), me, d.selectionInput.manualProjects) : sliceFor(d.tickets, role),
+      manualProjects: d.selectionInput.manualProjects,
       comments: d.comments,
       me,
       roster: d.roster,
@@ -6434,6 +6468,15 @@ function describeUnplaceable(unplaceable, verifiedStatus) {
   }
   return notes;
 }
+var unplaceableReported = /* @__PURE__ */ new Set();
+function reportUnplaceable(routeName, unplaceable, verified, emit) {
+  if (unplaceableReported.has(routeName)) return;
+  unplaceableReported.add(routeName);
+  for (const note of describeUnplaceable(unplaceable, verified)) {
+    if (note.level === "warn") emit.warn(note.message, { step: "release" });
+    else emit.emit(note.message, { step: "release" });
+  }
+}
 function checkGuards(cwd, base = "main") {
   const branch = currentBranch(cwd);
   if (branch !== base) {
@@ -6585,6 +6628,10 @@ function decideRelease(cwd, tickets, contract, opts = {}) {
     unseeded: lastTag === null
   };
 }
+function releaseIdle(i) {
+  if (i.force || i.external || i.unseeded || !i.upToDate || i.unstamped > 0) return false;
+  return !i.tickets.some((t) => t.status === i.verifiedStatus);
+}
 var renderTag = (template, version) => template.replace(/\{version\}/g, version);
 
 // src/stamp.ts
@@ -10526,15 +10573,8 @@ if (command === "passenger-sync-daemon") {
     const out = refreshBaseBranch(r.dir, remote, base, true);
     const unsafe = baseBranchUnsafe(out);
     return { action: out.action, unsafe, detail: unsafe ? describeUnsafeBase(out, remote, base) : out.detail };
-  }, reportUnplaceable = function(c, unplaceable, verified, emit2) {
-    if (unplaceableReported.has(c.route)) return;
-    unplaceableReported.add(c.route);
-    for (const note of describeUnplaceable(unplaceable, verified)) {
-      if (note.level === "warn") emit2.warn(note.message, { step: "release" });
-      else emit2.emit(note.message, { step: "release" });
-    }
   };
-  requireArmed2 = requireArmed, parseTicketNumber2 = parseTicketNumber, passengerApiKeys2 = passengerApiKeys, endpointApiFor2 = endpointApiFor, endpointReconcileDeps2 = endpointReconcileDeps, releaseRefused2 = releaseRefused, baseHealth2 = baseHealth, reportUnplaceable2 = reportUnplaceable;
+  requireArmed2 = requireArmed, parseTicketNumber2 = parseTicketNumber, passengerApiKeys2 = passengerApiKeys, endpointApiFor2 = endpointApiFor, endpointReconcileDeps2 = endpointReconcileDeps, releaseRefused2 = releaseRefused, baseHealth2 = baseHealth;
   let cfg;
   let firstRun;
   try {
@@ -10668,6 +10708,7 @@ if (command === "passenger-sync-daemon") {
   }
   async function releaseFleet(opts = {}) {
     if (releaseRefused()) return;
+    precheckTickets.clear();
     const summaries = [];
     if (!dryRun) state.releaseHeartbeat().touch();
     for (const c of cfg.routes) {
@@ -10679,6 +10720,7 @@ if (command === "passenger-sync-daemon") {
   }
   async function releaseTargets(opts = {}) {
     if (releaseRefused()) return;
+    precheckTickets.clear();
     const only = value("repo");
     const targets = reposOf(route);
     const chosen = only ? targets.filter((t) => t.name === only) : targets;
@@ -10915,7 +10957,7 @@ ${changed2.map((c) => `  - ${c}
     }
     return `${who2}; ${lock2}`;
   }
-  const unplaceableReported = /* @__PURE__ */ new Set();
+  const precheckTickets = /* @__PURE__ */ new Map();
   async function releasePhase(c, target, opts = {}) {
     const scope = releaseLockScope(c.route, target.name);
     const remit = emit.forRoute(c.route);
@@ -10950,6 +10992,38 @@ ${changed2.map((c) => `  - ${c}
       }
       if (!dryRun && !c.enabled) return { scope, tests: "skipped", outcome: "skipped", detail: "route not enabled" };
       const tracker = new Tracker(c, cfg.ship);
+      if (!dryRun) {
+        const force2 = command === "deploy" || flag("force");
+        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) ?? [];
+        reportUnplaceable(c.route, preUnplaceable, tracker.contract.statuses.verified, remit);
+        const d = decideRelease(target.dir, pre, tracker.contract, {
+          tagPattern: repo.release.tagPattern ?? "v*",
+          base: repo.branch.base
+        });
+        const unstamped = planReleasedBranches(
+          target.dir,
+          pre,
+          tracker.contract,
+          (t) => locateBranchForTicket(target.dir, repo, t)?.branch ?? null,
+          repo.branch.base,
+          repo.release.tagPattern ?? "v*"
+        ).length;
+        if (releaseIdle({
+          tickets: pre,
+          verifiedStatus: tracker.contract.statuses.verified,
+          force: force2,
+          external: repo.release.mode === "external",
+          upToDate: d.upToDate,
+          unseeded: d.unseeded,
+          unstamped
+        })) {
+          remit.emit(`${target.name}: nothing verified \u2014 not locking`, { step: "release" });
+          return { scope, tests: "skipped", outcome: "skipped", detail: "nothing verified \u2014 not locking" };
+        }
+      }
       if (!dryRun) {
         const holderLabel = `${cfg.ship.name}:${process.pid}`;
         const got = await tracker.acquireBoardLock(scope, holderLabel, RELEASE_LOCK_TTL_MS);
@@ -10972,7 +11046,7 @@ ${changed2.map((c) => `  - ${c}
       const all = await tracker.openTickets();
       const { byRepo, unplaceable } = ticketsByRepo(c, all);
       const tickets = byRepo.get(target.name) ?? [];
-      reportUnplaceable(c, unplaceable, tracker.contract.statuses.verified, remit);
+      reportUnplaceable(c.route, unplaceable, tracker.contract.statuses.verified, remit);
       const outcome = await runRelease({
         cwd: target.dir,
         repo,
@@ -13258,4 +13332,3 @@ var endpointApiFor2;
 var endpointReconcileDeps2;
 var releaseRefused2;
 var baseHealth2;
-var reportUnplaceable2;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.19.4",
+  "version": "0.20.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,6 +305,11 @@ 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.
+
 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`.