tumwater 0.4.2 → 0.4.3

package/dist/src/ui/badges.js

@@ -1,4 +1,3 @@
-import { sortEntriesByKey } from "../collections.js";
 import { utilizationPct, windowLabel } from "../agent/subscription-account.js";
 import { MODEL_TIERS } from "../config/config-schema.js";
 import { budgetGate, budgetReached, fallbackTierEntries } from "../budget/budget.js";
@@ -383,21 +382,12 @@ export function diskBadge(disk) {
             ? ` · disk ${free}`
             : "";
 }
-/** The permit pool's header fragment (plans/provider-concurrency.md §8): `slots 5/8 · omlx
- * 3/3 · claude 2/6` — the team total's in-use/cap followed by each capped provider key's own
- * in-use/cap. The WHOLE fragment appears only while at least one key carries a cap (the
- * `maxConcurrentPerProvider` map is set), so a team without it renders the header exactly
- * as before; uncapped keys that happen to be in the snapshot are dropped rather than shown
- * with a bare in-use that would read like a cap. */
+/** The permit pool's header fragment (plans/provider-concurrency.md §8): `slots 5/8` — the
+ * team total's in-use/cap. Empty when no snapshot was published. */
 export function permitsBadge(permits) {
     if (!permits)
         return "";
-    const providers = sortEntriesByKey(Object.entries(permits.byKey).filter(([, e]) => e.limit !== undefined))
-        .map(([k, e]) => `${k} ${e.inUse}/${e.limit}`)
-        .join(" · ");
-    if (providers.length === 0)
-        return "";
-    return ` · running ${permits.inUse} of ${permits.limit} · ${providers}`;
+    return ` · running ${permits.inUse} of ${permits.limit}`;
 }
 /** The check-permit pool's reading, ` · checks 1/2` plus `, 3 waiting` while checks queue. Empty
  * when no check runs or waits, so an idle check pool leaves the header byte-identical. */

package/dist/src/ui/gui/gui-client-boot.js

@@ -73,13 +73,13 @@ export const GUI_CLIENT_BOOT_JS = String.raw `  // One tail for the "…written
   // submit listener saves the row's key (saveSetting, gui-client-settings). No per-row wiring,
   // so rows can repaint freely.
   document.addEventListener("submit", (ev) => {
-    const form = ev.target instanceof Element ? ev.target.closest("form.settings-row") : null;
+    const form = clickClosest(ev, "form.settings-row");
     if (form) { ev.preventDefault(); saveSetting(form); }
   });
   // The Brief card's Save: its own form class, so the settings listener above never
   // routes it to saveSetting.
   document.addEventListener("submit", (ev) => {
-    const form = ev.target instanceof Element ? ev.target.closest("form.brief-form") : null;
+    const form = clickClosest(ev, "form.brief-form");
     if (form) { ev.preventDefault(); saveBrief(form); }
   });
   // The Settings view's Clear buttons: one delegated click listener resolves the row's Clear

package/dist/src/ui/gui/gui-client-composer.js

@@ -221,7 +221,7 @@ export const GUI_CLIENT_COMPOSER_JS = String.raw `  // ---- composer: one box fo
       "<button type='button' data-idx='" + i + "' title='Remove' aria-label='Remove " + esc(img.name) + "'>×</button></span>").join("");
   }
   $("promptimages").addEventListener("click", (ev) => {
-    const b = ev.target instanceof Element ? ev.target.closest("button[data-idx]") : null;
+    const b = clickClosest(ev, "button[data-idx]");
     if (!b) return;
     promptImages.splice(Number(b.dataset.idx), 1);
     renderPromptImages();

package/dist/src/ui/gui/gui-client-drawer-config.js

@@ -7,22 +7,20 @@
 export const GUI_CLIENT_DRAWER_CONFIG_JS = String.raw `
   // A Configuration section built from the same /api/role payload the CLI's tumwater role
   // <id> renders (rolePayload), so the drawer's and the terminal's rows read one document and
-  // cannot drift: what the role RUNS ON (model/tier, provider cap, image reading, the
+  // cannot drift: what the role RUNS ON (model/tier, image reading, the
   // fallback pairs, billing, min run interval) and CARRIES (queued prompts, the instructions
   // override, find text). Each row is guarded by a present/own check so an absent field
-  // renders no row — a cap-less team shows no Provider row, no fallback shows no fallback
-  // rows, and a notes-only payload adds an empty (omitted) section. Labels and wording mirror
+  // renders no row — no fallback shows no fallback rows, and a notes-only payload adds an
+  // empty (omitted) section. Labels and wording mirror
   // role-render.ts's family; no backticks allowed here (this whole script is one big
   // String.raw template).
   function drawerConfigHtml(view, role) {
     const pair = (p, m) => [p, m].filter(Boolean).join("/");
     // A fallback pair's label, mirroring role-render.ts's fallbackPairLabel so the drawer's
-    // Model fallback and Budget fallback rows name a non-pi backend's namespace the same way
-    // the CLI's fallback lines do: a non-pi backend with a model reads <backend>/<model>, else
-    // <provider>/<model>, else the model, else the provider, else the pair-less default. The
-    // pi provider/model cases and a budget pair (whose provider already holds the backend) stay
-    // byte-identical; only a model-fallback non-pi pair (modelFallbackView drops pi's provider
-    // for a non-pi backend) gains the namespace the CLI already shows.
+    // Budget fallback row names a non-pi backend's namespace the same way the CLI's fallback
+    // lines do: a non-pi backend with a model reads <backend>/<model>, else <provider>/<model>,
+    // else the model, else the provider, else the pair-less default. A budget pair's provider
+    // already holds the backend, so its row stays byte-identical.
     // Local wall-clock HH:MM for the role-cap forecast's ~cap at stamp — the drawer is browser
     // code and cannot import ../text/datetime.js (the quietWindowEnd precedent's own note), so
     // it routes through the bundle's shared clockHm in FORMAT_JS, the same helper the operator
@@ -92,9 +90,6 @@ export const GUI_CLIENT_DRAWER_CONFIG_JS = String.raw `
     if (view.contextWindow !== undefined) {
       rows.push(kvRow("Context window", view.contextWindow === null ? "unknown" : fmtTokens(view.contextWindow) + " tokens"));
     }
-    if (view.providerKey !== undefined && view.providerCap !== undefined) {
-      rows.push(kvRow("Provider", view.providerKey + " (cap " + view.providerCap + ")"));
-    }
     if (view.imageInput !== undefined) {
       const images = view.imageInput === true ? "yes"
         : view.imageInput === false ? (view.imageReroute ? "no — reruns on " + view.imageReroute : "no")
@@ -115,11 +110,6 @@ export const GUI_CLIENT_DRAWER_CONFIG_JS = String.raw `
       const revControl = "<button type='button' class='btn btn-sm' data-act='review-toggle' data-arg='" + esc(role) + "'>" + (revAfter ? "Review before merge" : "Review after merge") + "</button>";
       rows.push("<div><dt>Review</dt><dd title='" + esc(revBody) + "'>" + esc(revBody) + revControl + "</dd></div>");
     }
-    if (view.modelFallback && typeof view.modelFallback === "object" && view.modelFallback.model !== undefined) {
-      const fb = view.modelFallback;
-      const when = typeof fb.since === "number" ? " since " + fmtAgo(fb.since) : "";
-      rows.push(kvRow("Model fallback", fallbackLabel(fb, "") + " — on fallback" + when + " (primary failing: " + (fb.reason ?? "") + ")"));
-    }
     if (view.fallback && typeof view.fallback === "object" && view.fallback.model !== undefined) {
       rows.push(kvRow("Budget fallback", fallbackLabel(view.fallback, "") + " (" + (view.fallbackFree ? "free" : "priced") + ")"));
     }

package/dist/src/ui/gui/gui-client-drawer.js

@@ -208,7 +208,7 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     const commits = Array.isArray(view.commits) ? view.commits : [];
     const dirty = Array.isArray(view.dirtyFiles) ? view.dirtyFiles : [];
     if (view.state === "absent") return "<p class='muted'>No workspace yet — this role has not run.</p>";
-    if (view.state === "no-base") return "<p class='muted'>Main branch " + esc(view.mainBranch) + " does not exist.</p>";
+    if (view.state === "no-base") return "<p class='muted'>" + mainBranchMissingPhrase(view.mainBranch) + ".</p>";
     const scoped = typeof view.path === "string";
     if (scoped && view.state === "ready" && commits.length === 0 && dirty.length === 0) {
       return "<p class='muted'>No pending change to " + esc(view.path) + ".</p>";

package/dist/src/ui/gui/gui-client-history.js

@@ -71,13 +71,13 @@ export const GUI_CLIENT_HISTORY_JS = String.raw `  // The counts the "Runs to sh
       fetchHistory();
     });
     $("histfilter").addEventListener("click", (ev) => {
-      const b = ev.target.closest("button[data-hist]");
+      const b = clickClosest(ev, "button[data-hist]");
       if (!b) return;
       histFilter = b.dataset.hist;
       renderHistory();
     });
     $("histsum").addEventListener("click", (ev) => {
-      const b = ev.target.closest("button[data-histsum]");
+      const b = clickClosest(ev, "button[data-histsum]");
       if (!b) return;
       histMode = b.dataset.histsum;
       store("hist-mode", histMode);
@@ -98,7 +98,7 @@ export const GUI_CLIENT_HISTORY_JS = String.raw `  // The counts the "Runs to sh
     // data-rundetail button).
     // A By model row's label drills into that model's runs: the table, filtered to the selector.
     $("histbody").addEventListener("click", (ev) => {
-      const b = ev.target instanceof Element ? ev.target.closest("button[data-histmodel]") : null;
+      const b = clickClosest(ev, "button[data-histmodel]");
       if (!b) return;
       histModel = b.dataset.histmodel;
       histMode = "table";
@@ -106,43 +106,43 @@ export const GUI_CLIENT_HISTORY_JS = String.raw `  // The counts the "Runs to sh
       fetchHistory();
     });
     $("histbody").addEventListener("click", (ev) => {
-      const b = ev.target instanceof Element ? ev.target.closest("button[data-rundetail]") : null;
+      const b = clickClosest(ev, "button[data-rundetail]");
       if (!b) return;
       toggleHistDetail(b.dataset.role, b.dataset.run);
     });
     // The drill-down's revert buttons: the first click previews, the preview's own button commits.
     $("histbody").addEventListener("click", (ev) => {
-      const b = ev.target instanceof Element ? ev.target.closest("button[data-hist-revert], button[data-hist-revert-go]") : null;
+      const b = clickClosest(ev, "button[data-hist-revert], button[data-hist-revert-go]");
       if (!b) return;
       if (b.dataset.histRevertGo !== undefined) runHistRevert(b.dataset.histRevertGo);
       else previewHistRevert(b.dataset.histRevert);
     });
     // The drill-down's "Show patch" / "Hide patch" button.
     $("histbody").addEventListener("click", (ev) => {
-      const b = ev.target instanceof Element ? ev.target.closest("button[data-hist-patch]") : null;
+      const b = clickClosest(ev, "button[data-hist-patch]");
       if (!b) return;
       toggleHistPatch(b.dataset.histPatch);
     });
     // The drill-down's "Show transcript" / "Hide transcript" button.
     $("histbody").addEventListener("click", (ev) => {
-      const b = ev.target instanceof Element ? ev.target.closest("button[data-hist-transcript]") : null;
+      const b = clickClosest(ev, "button[data-hist-transcript]");
       if (!b) return;
       toggleHistTranscript(b.dataset.histTranscript);
     });
     // The open transcript's "Show prompts" / "Hide prompts" button.
     $("histbody").addEventListener("click", (ev) => {
-      const b = ev.target instanceof Element ? ev.target.closest("button[data-hist-transcript-prompts]") : null;
+      const b = clickClosest(ev, "button[data-hist-transcript-prompts]");
       if (!b) return;
       toggleHistTranscriptPrompts(b.dataset.histTranscriptPrompts);
     });
     // The open transcript's search box (applied on Enter or blur: the card re-renders each change).
     $("histbody").addEventListener("change", (ev) => {
-      const el = ev.target instanceof Element ? ev.target.closest("input[data-hist-transcript-grep]") : null;
+      const el = clickClosest(ev, "input[data-hist-transcript-grep]");
       if (el) searchHistTranscript(el.dataset.histTranscriptGrep, el.value);
     });
     // The pinned newest-failure card's × closes it.
     $("histbody").addEventListener("click", (ev) => {
-      const b = ev.target instanceof Element ? ev.target.closest("button[data-hist-newest-close]") : null;
+      const b = clickClosest(ev, "button[data-hist-newest-close]");
       if (!b) return;
       closeHistNewest();
     });
@@ -187,11 +187,16 @@ export const GUI_CLIENT_HISTORY_JS = String.raw `  // The counts the "Runs to sh
     renderHistory();
     try {
       const d = await getJson("/api/run?last=1&result=problems");
-      // Name the endpoint and the run it answered for: a 200 without a text or role field is a
-      // server too old (or too new) for this dashboard's client, not a lost run.
+      // Name the endpoint, the run it answered for, and the actually missing field: a 200
+      // without either text or role is a server too old (or too new) for this dashboard's
+      // client, not a lost run. text is the usual miss (the card would render nothing) and
+      // role decides the card's cache key, so name role when text is present but the role
+      // is not rather than blaming text for a field the reply did carry.
       if (!d || typeof d.text !== "string" || typeof d.role !== "string")
         throw new Error(
-          "/api/run answered the newest failing run without a text field — the running tumwater may not match this dashboard",
+          "/api/run answered the newest failing run without a " +
+            (!d || typeof d.text !== "string" ? "text" : "role") +
+            " field — the running tumwater may not match this dashboard",
         );
       const key = histDetailKey(d.role, d.run);
       histDetails.set(key, histDetailOk(d));
@@ -299,7 +304,16 @@ export const GUI_CLIENT_HISTORY_JS = String.raw `  // The counts the "Runs to sh
   // drift apart: every one fetches runApiUrl, then insists the reply carries its field.
   async function fetchRunField(key, query, field) {
     const d = await getJson(runApiUrl(key, query));
-    if (!d || typeof d[field] !== "string") throw new Error("/api/run answered without a " + field + " field");
+    if (!d || typeof d[field] !== "string") {
+      // Name the endpoint, the run it answered for, and the likely cause — the same
+      // ownership fetchHistDetail and openHistNewest give their /api/run errors: a 200
+      // without the promised field is a server too old (or too new) for this dashboard's
+      // client, not a silently dropped patch or transcript.
+      const at = key.lastIndexOf("#");
+      throw new Error(
+        "/api/run answered " + key.slice(0, at) + " run " + key.slice(at + 1) + " without a " + field + " field — the running tumwater may not match this dashboard",
+      );
+    }
     return d;
   }
   // Fetch or toggle one card add-on field (the patch or the transcript) at key ("role#run").

package/dist/src/ui/gui/gui-client-model.js

@@ -29,18 +29,6 @@ export const GUI_CLIENT_MODEL_JS = String.raw `  // view-model:start
     if (p.startsWith("merging")) return { key: "landing", label: "Merging", tone: "orange", live: true, detail: after(7) }; // vocabulary-ok: internal key the row classes and filters match
     if (p.startsWith("approved")) return { key: "vetted", label: "Approved", tone: "indigo", live: false, detail: "waiting to merge" }; // vocabulary-ok: internal key the row classes and filters match
     if (p.startsWith("waiting to start")) return { key: "awaiting", label: "Waiting to start", tone: "gray", live: false, detail: after(16) ? "for " + after(16) : "" };
-    // A waiter parked on a full provider key (provider-concurrency part 5/5): the phase reads
-    // waiting for <key> 2m, and this chip names the provider alongside the wait, so the
-    // operator sees which cap is full — the same chip and tone as a team-total waiter.
-    if (p.startsWith("waiting for ")) {
-      const m = after(12).match(/^(\S+)(?:\s+(.*))?$/);
-      const prov = m ? m[1] : "";
-      const dur = m && m[2] ? m[2].trim() : "";
-      return {
-        key: "awaiting", label: "Waiting to start", tone: "gray", live: false,
-        detail: (prov ? "on " + prov : "") + (dur ? " for " + dur : ""),
-      };
-    }
     if (p === "failing") return { key: "failing", label: "Failing", tone: "red", live: false, detail: "failed runs in a row" };
     if (p === "main's check is failing") return { key: "mainred", label: "Main's check failing", tone: "red", live: false, detail: "change pinned until main goes green" };
     if (p.startsWith("paused")) return { key: "paused", label: "Paused", tone: "amber", live: false, detail: "starts no new runs" };

package/dist/src/ui/gui/gui-client-pending.js

@@ -58,7 +58,7 @@ export const GUI_CLIENT_PENDING_JS = String.raw `  let pendingTeam = null;
     if (!team || !Array.isArray(team.roles) || team.roles.length === 0)
       return "<div class='empty'><strong>No roles yet</strong>Roles appear here once the team is configured.</div>";
     if (team.roles.every((r) => r.state === "no-base"))
-      return "<div class='empty'><strong>Main branch " + esc(team.mainBranch) + " does not exist</strong>There is no base to compare unmerged work against.</div>";
+      return "<div class='empty'><strong>" + mainBranchMissingPhrase(team.mainBranch) + "</strong>There is no base to compare unmerged work against.</div>";
     const rows = team.roles.map((r) =>
       "<tr class='clickable' data-open='" + esc(r.role) + "'>" +
       "<td class='mono'>" + esc(roleLabel(r.role)) + "</td>" +
@@ -202,7 +202,7 @@ export const GUI_CLIENT_PENDING_JS = String.raw `  let pendingTeam = null;
     let body;
     let head = "<div class='card-head'><h2>Overlapping files</h2></div>";
     if (ok && view.state === "no-base") {
-      body = "<div class='empty'>Main branch " + esc(view.mainBranch) + " does not exist</div>";
+      body = "<div class='empty'>" + mainBranchMissingPhrase(view.mainBranch) + "</div>";
     } else if (!ok || !Array.isArray(view.overlaps) || view.overlaps.length === 0) {
       body = "<div class='empty'>No file is touched by two pending changes</div>";
     } else {
@@ -267,6 +267,7 @@ export const GUI_CLIENT_PENDING_JS = String.raw `  let pendingTeam = null;
       case "fixed":
         return "[fixed — the repair " + shortSha(e.fixed) + " already merged on main]";
       case "rejected":
+        if (!e.rejected || e.rejected.length === 0) return "[rejected]";
         return "[rejected — " + e.rejected.map((r) => squashText(r, 60)).join("; ") + "]";
       case "failing":
         if (e && typeof e.lastFailure === "string" && e.lastFailure.length > 0) {
@@ -388,7 +389,7 @@ export const GUI_CLIENT_PENDING_JS = String.raw `  let pendingTeam = null;
       // A Merge queue row's Move to top: a single click posts /api/queue-top by sha, moving
       // that entry to the head of the queue and refetching (the move is reversible and
       // non-destructive, so it needs no two-step arm).
-      const t = e.target.closest("[data-top]");
+      const t = clickClosest(e, "[data-top]");
       if (t) {
         const sha = t.getAttribute("data-top");
         if (sha) {
@@ -397,7 +398,7 @@ export const GUI_CLIENT_PENDING_JS = String.raw `  let pendingTeam = null;
         }
         return;
       }
-      const b = e.target.closest("[data-cancel]");
+      const b = clickClosest(e, "[data-cancel]");
       if (!b) return;
       const sha = b.getAttribute("data-cancel");
       if (!sha) return;
@@ -415,7 +416,7 @@ export const GUI_CLIENT_PENDING_JS = String.raw `  let pendingTeam = null;
     // (repainting that button as Confirm accept), a second on the armed commit within 5 s posts
     // /api/post-review-accept then clears the arm and refetches so the removed row disappears.
     $("pendingreviews").addEventListener("click", async (e) => {
-      const b = e.target.closest("[data-accept]");
+      const b = clickClosest(e, "[data-accept]");
       if (!b) return;
       const commit = b.getAttribute("data-accept");
       if (!commit) return;
@@ -445,7 +446,7 @@ export const GUI_CLIENT_PENDING_JS = String.raw `  let pendingTeam = null;
       }
     });
     $("pendingoverlaps").addEventListener("click", (e) => {
-      const b = e.target.closest("[data-overlap]");
+      const b = clickClosest(e, "[data-overlap]");
       if (b) togglePendingOverlap(Number(b.dataset.overlap));
     });
     $("pendingoverlaps").addEventListener("change", (e) => {

package/dist/src/ui/gui/gui-client-report-chart.js

@@ -48,7 +48,7 @@ export const GUI_CLIENT_REPORT_CHART_JS = String.raw `
   // Object.prototype key whose inherited value would otherwise seed the counter and satisfy
   // the membership test (the browser twin of addTo's fix; BUGS.md 2026-10-10).
   function reportRoleOrder(data) {
-    return tallyAcross(data.series, ["runsByRole"], ["costByRole", "notionalByRole", "rejectionsByRole", "revisionsByRole", "refusalsByRole", "fallbacksByRole", "failedRunsByRole", "mergeFailedByRole", "commitsByRole", "mergedByRole"]);
+    return tallyAcross(data.series, ["runsByRole"], ["costByRole", "notionalByRole", "rejectionsByRole", "revisionsByRole", "refusalsByRole", "imageReroutesByRole", "failedRunsByRole", "mergeFailedByRole", "commitsByRole", "mergedByRole"]);
   }
 
   // X-axis labels: MM-DD, thinned to at most seven, counted back from the newest day so today
@@ -371,10 +371,10 @@ export const GUI_CLIENT_REPORT_CHART_JS = String.raw `
   function chartRefusalsByRole(data) {
     return roleStackChart(data, "refusalsByRole", (n) => plural(n, "refusal"), countAxis, true);
   }
-  // The team's model-fallback episodes, per role, on the same count axis: one segment per
-  // (day, role) with a fallback.
-  function chartFallbacksByRole(data) {
-    return roleStackChart(data, "fallbacksByRole", (n) => plural(n, "fallback"), countAxis, true);
+  // Image reroutes per role, drawn like the other role stacks: one segment per (day, role)
+  // with a reroute.
+  function chartImageReroutesByRole(data) {
+    return roleStackChart(data, "imageReroutesByRole", (n) => plural(n, "image reroute"), countAxis, true);
   }
   // Runs that ended in an error, per role, drawn the same way.
   function chartFailedRunsByRole(data) {

package/dist/src/ui/gui/gui-client-report.js

@@ -335,7 +335,7 @@ export const GUI_CLIENT_REPORT_JS = String.raw `// report-tip:start
         (seriesHasAny(d, "postMergeRejectionsByRole") ? block("Post-merge rejections by role", chartPostMergeRejectionsByRole(d)) : "") +
         (seriesHasAny(d, "revisionsByRole") ? block("Revisions by role", chartRevisionsByRole(d)) : "") +
         (seriesHasAny(d, "refusalsByRole") ? block("Refusals by role", chartRefusalsByRole(d)) : "") +
-        (seriesHasAny(d, "fallbacksByRole") ? block("Fallbacks by role", chartFallbacksByRole(d)) : "") +
+        (seriesHasAny(d, "imageReroutesByRole") ? block("Image reroutes by role", chartImageReroutesByRole(d)) : "") +
         (seriesHasAny(d, "failedRunsByRole") ? block("Failed runs by role", chartFailedRunsByRole(d)) : "") +
         (seriesHasAny(d, "mergedByRole") ? block("Merged by role", chartMergedByRole(d)) : "") +
         (!d.role && seriesHasAny(d, "runsByModel") ? block("Runs by model", chartRunsByModel(d)) : "") +

package/dist/src/ui/gui/gui-client-roles.js

@@ -163,7 +163,6 @@ export const GUI_CLIENT_ROLES_JS = String.raw `
     const name = "<div class='role-name'><button type='button' class='linkish role' data-open='" + esc(l.role) + "' title='Open " + esc(roleLabel(l.role)) + "'>" + esc(roleLabel(l.role)) + "</button>" +
       (l.custom ? "<span class='tag' title='A custom role from tumwater.json'>custom</span>" : "") +
       (l.modelTier ? "<span class='tag' title='The model tier this role resolves at'>" + esc(l.modelTier) + "</span>" : "") +
-      (l.fallback ? "<span class='tag t-amber' title='Running on the model fallback while its primary keeps failing provider-class errors'>fallback</span>" : "") +
       (queued ? "<span class='tag t-indigo' title='" + esc(plural(queued, "message") + " in this role's inbox") + "'>" + icon("chat") + queued + "</span>" : "") +
       "</div><div class='sub'>" + esc(plural(l.commits, "commit") + " · " + plural(l.runs, "run") + " · merged " + (l.lastMergedAt ? fmtAgo(l.lastMergedAt) : "never")) + (l.model ? " · " + esc(l.model) : "") +
       (l.modelFromPi ? " <span class='tag' title='pi&#39;s own default model, from its settings.json: tumwater.json names no model'>(pi default)</span>" : "") + "</div>";

package/dist/src/ui/gui/gui-client-settings.js

@@ -34,10 +34,8 @@ export const GUI_CLIENT_SETTINGS_JS = String.raw `  // settings-view:start
     ["fallback", "Fallback (free at the spend cap)"],
     ["maxDailyCostUsd", "Daily spend cap (USD)"],
     ["maxConcurrent", "Max concurrent runs"],
-    ["maxConcurrentPerProvider", "Provider concurrency caps (JSON)"],
     ["maxConcurrentChecks", "Max concurrent checks"],
     ["worktreeSlots", "Worktree slots"],
-    ["claudeWindowReserve", "Claude window reserve (backends.claude.windowReserve)"],
     ["quietHours", "Quiet hours"],
     ["notify", "Notify hook"],
     ["backends", "Agent backends (JSON)"],
@@ -162,13 +160,7 @@ export const GUI_CLIENT_SETTINGS_JS = String.raw `  // settings-view:start
   // The row's Enable button: re-enable the role through POST /api/role-enable (writing
   // roles.<base>.enabled), flash the server's message, and refetch so the card drops the row.
   async function enableRole(role) {
-    try {
-      const d = await postJson("/api/role-enable", { role: role });
-      showFlash(d.message);
-    } catch (err) {
-      showFlash("error: " + clientErrorMessage(err));
-    }
-    await fetchRetirable();
+    await postSetting("/api/role-enable", { role: role }, fetchRetirable, (d) => d.message);
   }
   // First click arms the role's button; a second within the confirm window posts the retire.
   async function confirmRetire(role) {
@@ -177,13 +169,42 @@ export const GUI_CLIENT_SETTINGS_JS = String.raw `  // settings-view:start
       return;
     }
     retireSlot.disarm();
+    await postSetting("/api/retire", { role: role }, fetchRetirable, (d) => d.message);
+  }
+  // POST one Settings action and flash its outcome, then refetch the card it changed — the
+  // try/catch tail every Enable/Retire/Health-button handler shares, so the thrown-error
+  // coercion and the success/error wording live in one place. news derives the success line
+  // from the endpoint's answer: the default is the team-wide "ok: / error: " prefix, Enable
+  // and Retire pass the server's message through verbatim, and Retire-leftovers overrides the
+  // none-left fallback. Deliberately not named postAction (gui-client.ts's core helper, which
+  // closes menus, refreshes the standing poll, and does not refetch on failure — this one
+  // refetches the card its action changed even when the post failed).
+  async function postSetting(path, body, refetch, news) {
     try {
-      const d = await postJson("/api/retire", { role: role });
-      showFlash(d.message);
+      const d = await postJson(path, body);
+      showFlash(news ? news(d) : (d.ok ? "" : "error: ") + d.message);
     } catch (err) {
       showFlash("error: " + clientErrorMessage(err));
     }
-    await fetchRetirable();
+    await refetch();
+  }
+  // POST one two-click Health-card action after its slot was armed by a prior click: the first
+  // click only arms the slot (the handler's button repaints to "Confirm …") and returns; a
+  // second click within the confirm window disarms, repaints, flashes the action line, posts,
+  // and re-runs the card's checks. The Health card's four repair buttons (Kill orphans, Drop
+  // broken pins, Retire leftovers, Archive backlog) share this one confirm shape: only the slot
+  // key, the paint fn, the flash line, the endpoint, and an optional news override (passed
+  // through to postSetting, e.g. Retire-leftovers' outcome-line override) differ, so they fold
+  // into this one helper instead of four copies of the arm/disarm/paint/flash/postSetting tail.
+  async function postConfirmedHealthAction(slot, key, paint, flash, path, news) {
+    if (!slot.armed(key)) {
+      slot.arm(key);
+      return;
+    }
+    slot.disarm();
+    paint();
+    showFlash(flash);
+    await postSetting(path, {}, fetchDoctor, news);
   }
   // The Health card's Kill orphans button: a two-click confirm, then POST /api/kill-orphans runs
   // the same kill as "tumwater doctor --kill-orphans" and the card re-runs its checks.
@@ -193,20 +214,7 @@ export const GUI_CLIENT_SETTINGS_JS = String.raw `  // settings-view:start
   }
   const killSlot = confirmSlot(paintKillButton);
   async function killOrphansClick() {
-    if (!killSlot.armed("orphans")) {
-      killSlot.arm("orphans");
-      return;
-    }
-    killSlot.disarm();
-    paintKillButton();
-    showFlash("Killing orphaned processes…");
-    try {
-      const d = await postJson("/api/kill-orphans", {});
-      showFlash((d.ok ? "" : "error: ") + d.message);
-    } catch (err) {
-      showFlash("error: " + clientErrorMessage(err));
-    }
-    await fetchDoctor();
+    await postConfirmedHealthAction(killSlot, "orphans", paintKillButton, "Killing orphaned processes…", "/api/kill-orphans");
   }
   // The Health card's Drop broken pins button: a two-click confirm, then POST
   // /api/doctor-drop-broken-pins runs the same drop as "tumwater doctor --drop-broken-pins" and
@@ -217,20 +225,7 @@ export const GUI_CLIENT_SETTINGS_JS = String.raw `  // settings-view:start
   }
   const dropSlot = confirmSlot(paintDropButton);
   async function dropBrokenPinsClick() {
-    if (!dropSlot.armed("broken-pins")) {
-      dropSlot.arm("broken-pins");
-      return;
-    }
-    dropSlot.disarm();
-    paintDropButton();
-    showFlash("Dropping broken pins…");
-    try {
-      const d = await postJson("/api/doctor-drop-broken-pins", {});
-      showFlash((d.ok ? "" : "error: ") + d.message);
-    } catch (err) {
-      showFlash("error: " + clientErrorMessage(err));
-    }
-    await fetchDoctor();
+    await postConfirmedHealthAction(dropSlot, "broken-pins", paintDropButton, "Dropping broken pins…", "/api/doctor-drop-broken-pins");
   }
   // The Health card's Retire leftovers button: a two-click confirm, then POST
   // /api/doctor-retire-leftovers runs the same reap as "tumwater doctor --retire-leftovers" and
@@ -242,22 +237,11 @@ export const GUI_CLIENT_SETTINGS_JS = String.raw `  // settings-view:start
   }
   const retireLeftoversSlot = confirmSlot(paintRetireLeftoversButton);
   async function retireLeftoversClick() {
-    if (!retireLeftoversSlot.armed("retire-leftovers")) {
-      retireLeftoversSlot.arm("retire-leftovers");
-      return;
-    }
-    retireLeftoversSlot.disarm();
-    paintRetireLeftoversButton();
-    showFlash("Retiring every disabled role's leftovers…");
-    try {
-      const d = await postJson("/api/doctor-retire-leftovers", {});
-      // The shared leftoverRetireLines join is empty when nothing is retirable, so the
-      // button falls back to the same none-left sentence the TUI Health key flashes.
-      showFlash((d.ok ? "" : "error: ") + (d.message || "no leftover roles to retire"));
-    } catch (err) {
-      showFlash("error: " + clientErrorMessage(err));
-    }
-    await fetchDoctor();
+    // The news override's none-left fallback matches the none-left sentence the TUI Health key
+    // flashes when nothing is retirable: the shared leftoverRetireLines join is empty when
+    // nothing is retirable, so the button falls back to that same sentence.
+    await postConfirmedHealthAction(retireLeftoversSlot, "retire-leftovers", paintRetireLeftoversButton, "Retiring every disabled role's leftovers…", "/api/doctor-retire-leftovers",
+      (d) => (d.ok ? "" : "error: ") + (d.message || "no leftover roles to retire"));
   }
   // The Health card's Archive backlog button: a two-click confirm, then POST
   // /api/doctor-archive-backlog runs the same archive as "tumwater archive" (holding the merge
@@ -270,20 +254,7 @@ export const GUI_CLIENT_SETTINGS_JS = String.raw `  // settings-view:start
   }
   const archiveBacklogSlot = confirmSlot(paintArchiveBacklogButton);
   async function archiveBacklogClick() {
-    if (!archiveBacklogSlot.armed("archive-backlog")) {
-      archiveBacklogSlot.arm("archive-backlog");
-      return;
-    }
-    archiveBacklogSlot.disarm();
-    paintArchiveBacklogButton();
-    showFlash("Archiving the finished backlog entries…");
-    try {
-      const d = await postJson("/api/doctor-archive-backlog", {});
-      showFlash((d.ok ? "" : "error: ") + d.message);
-    } catch (err) {
-      showFlash("error: " + clientErrorMessage(err));
-    }
-    await fetchDoctor();
+    await postConfirmedHealthAction(archiveBacklogSlot, "archive-backlog", paintArchiveBacklogButton, "Archiving the finished backlog entries…", "/api/doctor-archive-backlog");
   }
   // The Health card's Commit backlog button: a single click, because the commit is the doctor's
   // own recommended repair and is reversible (unlike the two-click Kill orphans arm), POSTing
@@ -292,13 +263,7 @@ export const GUI_CLIENT_SETTINGS_JS = String.raw `  // settings-view:start
   // clean), then re-running the checks so the uncommitted-backlog row repaints.
   async function commitBacklogClick() {
     showFlash("Committing the uncommitted backlog/brief files…");
-    try {
-      const d = await postJson("/api/doctor-commit-backlog", {});
-      showFlash((d.ok ? "" : "error: ") + d.message);
-    } catch (err) {
-      showFlash("error: " + clientErrorMessage(err));
-    }
-    await fetchDoctor();
+    await postSetting("/api/doctor-commit-backlog", {}, fetchDoctor);
   }
   // The Health card's Re-sync install button: a single click, because the re-sync is the doctor's
   // own recommended repair and is a no-op when the install already matches the lockfile, POSTing
@@ -307,13 +272,7 @@ export const GUI_CLIENT_SETTINGS_JS = String.raw `  // settings-view:start
   // the checks so the root-install row and the install amber alert clear once the drift is gone.
   async function resyncInstallClick() {
     showFlash("Re-syncing the root install…");
-    try {
-      const d = await postJson("/api/doctor-resync-install", {});
-      showFlash((d.ok ? "" : "error: ") + d.message);
-    } catch (err) {
-      showFlash("error: " + clientErrorMessage(err));
-    }
-    await fetchDoctor();
+    await postSetting("/api/doctor-resync-install", {}, fetchDoctor);
   }
   async function fetchSettings() {
     buildSettingsView();

package/dist/src/ui/gui/gui-client-team.js

@@ -95,13 +95,13 @@ export const GUI_CLIENT_TEAM_JS = String.raw `  // ---- sidebar: project, team s
               : "");
       }
       // The permit pool (plans/provider-concurrency.md §8): the payload's preformatted
-      // permitsBadge — running 5 of 8 · omlx 3/3 · claude 2/6 — upcased for the row's label
-      // style, the same reading the TUI/status header carries, so the two surfaces cannot
-      // drift. Empty on a map-less team, so the sidebar stays unchanged.
+      // permitsBadge — running 5 of 8 — upcased for the row's label style, the same reading
+      // the TUI/status header carries, so the two surfaces cannot drift. Present for any
+      // team with a published permit snapshot.
       if (d.permitsBadge) {
         const tok = String(d.permitsBadge).replace(/^ · /, "");
         rows += row("blue", icon("gauge"), esc(tok),
-          "Roles running at once out of the team limit, then each capped provider's own running/cap; a role waits to start when its provider is at its cap.");
+          "Roles running at once out of the team limit (maxConcurrent); a role waits to start when the team is at its limit.");
       }
       // The check-permit pool: the payload's checksBadge, present only while a check runs or waits.
       if (d.checksBadge) {

package/dist/src/ui/gui/gui-client.js

@@ -151,6 +151,12 @@ const FORMAT_JS = String.raw `  // format:start
   // section had the byte-identical map; a missing or non-array list paints nothing.
   const commitRowsHtml = (commits) =>
     (Array.isArray(commits) ? commits : []).map((c) => "<div class='mono clamp1'>" + esc(c.sha) + " " + esc(c.subject || "—") + "</div>").join("");
+  // The empty state for a missing main branch, shared by the Pending roster, its
+  // overlapping-files card, and the role drawer's Pending change section so the esced branch
+  // name and the wording cannot drift between them — change-render.ts's mainMissingPhrase,
+  // whose terminal spelling is lowercase/unescaped. The browser client cannot import that TS
+  // module, so this is its own copy, capitalized and esced for the HTML sentence it leads.
+  const mainBranchMissingPhrase = (mainBranch) => "Main branch " + esc(mainBranch) + " does not exist";
   // Title-case a phrase's first character and keep the rest — backend-traits.ts's
   // backendDisplayName rule, the one home of the upcasing gui-client-model.ts's result labels
   // and gui-client-history.ts's window options each inlined, so a label cannot drift between
@@ -226,7 +232,7 @@ const FORMAT_JS = String.raw `  // format:start
     // phase rather than throwing: the browser copy is untyped, and its sibling phaseInfo/
     // roleRank already coerce the same way, so one bad row must not blank the whole table.
     const phase = String(l.phase || "");
-    const active = isActivePhase(phase) || phase.startsWith("waiting to start") || phase.startsWith("waiting for ") || phase.startsWith("director working");
+    const active = isActivePhase(phase) || phase.startsWith("waiting to start") || phase.startsWith("director working");
     if (!teamRunning || active) return "-";
     const sec = Math.round((l.nextRunAt - Date.now()) / 1000);
     const suffix = l.yieldMultiplier > 1 ? " ×" + l.yieldMultiplier : "";