tumwater 0.1.2 → 0.2.0

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

@@ -1,6 +1,7 @@
+// vocabulary: plain
 /** The dashboard's detail drawer, browser-side: a sheet on the right that opens over the page
  * without losing it. For a loop it shows the state, the loop's controls (prompt, wake now or later,
- * pause/resume, abort), what it is working on, its last result, the metrics the fleet table
+ * pause/resume, abort), what it is working on, its last result, the metrics the team table
  * leaves out (tokens, context, cost, timings), its recent ticks (/api/history filtered to the
  * loop), and its live transcript (/api/transcript, re-fetched on the 1 s poll and kept pinned
  * to the bottom while the reader is there). For a backlog entry it shows the entry in full
@@ -12,10 +13,19 @@
 export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind: "loop", role } | { kind: "entry", file, index } while open
   let drawerTicks = null; // the open loop's recent history rows (null until fetched)
   let drawerTicksAt = 0;
-  let drawerChange = null; // the open loop's unlanded change: { role, state: "loading"|"ok"|"error", view, message }
+  let drawerChange = null; // the open loop's unmerged change: { role, state: "loading"|"ok"|"error", view, message }
   let drawerRole = null; // the open loop's /api/role payload: { role, state: "loading"|"ok"|"error", view, message }
   const openEntryKey = () => (drawer && drawer.kind === "entry" ? drawer.file + ":" + drawer.index : null);
-  const openLoopRole = () => (drawer && drawer.kind === "loop" ? drawer.role : null);
+  const openLoopRole = () => (drawer && drawer.kind === "loop" ? drawer.role : null); // vocabulary-ok: the drawer kind stays "loop"
+  // One key/value row builder shared by the loop-metrics table (renderLoopDrawer) and the
+  // Configuration section (drawerConfigHtml): identical markup, formerly copy-pasted into two
+  // local consts (kv and row) so the row shape could drift between the two sections.
+  const kvRow = (k, v, title) => "<div><dt>" + esc(k) + "</dt><dd title='" + esc(title || v) + "'>" + esc(v) + "</dd></div>";
+  // A diff block in the Change section (the committed change and the dirty-tree uncommitted
+  // diff, drawerChangeHtml's two arms): trim the trailing newline the diff carriage-returned,
+  // escape the rest, and wrap it in the shared .diff pre. Formerly copy-pasted into both arms
+  // so the trim/escape could drift between them.
+  const diffPre = (text) => "<pre class='mono diff'>" + esc(String(text).replace(/\n+$/, "")) + "</pre>";
 
   function showDrawer(bodyHtml) {
     for (const id of ["drawerhead", "draweractions", "drawermeta", "drawerrole", "transcript", "entrybody"]) delete lastPaint[id];
@@ -29,8 +39,14 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     drawer = null;
     entryEditing = null;
     noteEditing = null;
-    confirmNoteClear = null;
-    if (wasLoop && location.hash.startsWith("#loop/")) history.replaceState(null, "", "#" + activeView);
+    capEditing = null;
+    instructionsEditing = null;
+    quietEditing = null;
+    intervalEditing = null;
+    instancesEditing = null;
+    modelEditing = null;
+    disarmNoteClear();
+    if (wasLoop && (location.hash.startsWith("#role/") || location.hash.startsWith("#loop/"))) history.replaceState(null, "", "#" + activeView); // vocabulary-ok: legacy dashboard hash, still routed
     drawerTicks = null;
     drawerChange = null;
     drawerRole = null;
@@ -39,12 +55,12 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     $("drawerhead").innerHTML = "";
     $("drawerbody").innerHTML = "";
     for (const id of ["drawerhead", "draweractions", "drawermeta", "drawerrole", "transcript", "entrybody"]) delete lastPaint[id];
-    if (lastStatus) renderFleet(lastStatus); // drop the row highlight
+    if (lastStatus) renderTeam(lastStatus); // drop the row highlight
   }
 
   function openLoop(role) {
-    drawer = { kind: "loop", role };
-    const hash = "#loop/" + encodeURIComponent(role);
+    drawer = { kind: "loop", role }; // vocabulary-ok: the drawer kind stays "loop"
+    const hash = "#role/" + encodeURIComponent(role);
     if (location.hash !== hash) history.replaceState(null, "", hash);
     drawerTicks = null;
     drawerTicksAt = 0;
@@ -55,7 +71,7 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
       "<pre id='transcript' class='transcript'><span class='muted'>Loading…</span></pre></section>");
     if (lastStatus) {
       renderLoopDrawer(lastStatus);
-      renderFleet(lastStatus);
+      renderTeam(lastStatus);
     }
     loadLoopTicks(true);
     loadDrawerChange(role);
@@ -70,16 +86,16 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     drawer = { kind: "entry", file, index: Number(index) };
     entryEditing = null;
     showDrawer("<div id='entrybody'><div class='empty'>Loading…</div></div>");
-    if (lastStatus) renderFleet(lastStatus);
+    if (lastStatus) renderTeam(lastStatus);
     refreshEntry();
   }
   function toggleEntry(file, index) {
     if (openEntryKey() === file + ":" + index) closeDrawer();
     else openEntry(file, index);
   }
-  // The change the landing slot holds for a role — what a landing (or approved) row is about.
-  function landingSummary(d, role) {
-    const f = d.landQueue && d.landQueue.inFlight;
+  // The change the merge slot holds for a role — what a merge (or approved) row is about.
+  function mergeSummary(d, role) {
+    const f = d.mergeQueue && d.mergeQueue.inFlight;
     if (!f) return "";
     const change = (Array.isArray(f.changes) ? f.changes : [f]).find((c) => c.role === role);
     return change && change.summary ? change.summary : "";
@@ -88,35 +104,36 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
   function renderLoopDrawer(d) {
     const role = openLoopRole();
     if (!role) return;
-    const l = (d.loops || []).find((x) => x.role === role);
+    const l = (d.roles || []).find((x) => x.role === role);
     if (!l) {
-      paintPanel("drawerhead", "<div class='kicker'>Loop</div><h2 id='drawertitle' class='mono'>" + esc(role) +
-        "</h2><span class='muted'>This loop is no longer part of the fleet.</span>");
+      paintPanel("drawerhead", "<div class='kicker'>Role</div><h2 id='drawertitle' class='mono'>" + esc(roleLabel(role)) +
+        "</h2><span class='muted'>This role is no longer part of the team.</span>");
       paintPanel("draweractions", "");
       paintPanel("drawermeta", drawerChangeSection());
       paintPanel("drawerrole", drawerRoleSection());
       return;
     }
     const info = phaseInfo(l.phase);
-    const paused = (d.pausedRoles || []).includes(l.role);
+    const paused = (d.pausedLoops || []).includes(l.role);
     const detail = phaseDetail(l, info, d);
-    paintPanel("drawerhead", "<div class='kicker'>Loop" + (l.custom ? " · user-defined" : "") + "</div>" +
-      "<h2 id='drawertitle' class='mono'>" + esc(l.role) + "</h2><div class='chips'>" + pill(info) +
+    paintPanel("drawerhead", "<div class='kicker'>Role" + (l.custom ? " · user-defined" : "") + "</div>" +
+      "<h2 id='drawertitle' class='mono'>" + esc(roleLabel(l.role)) + "</h2><div class='chips'>" + pill(info) +
       (detail ? "<span class='muted'>" + esc(detail) + "</span>" : "") + "</div>");
 
     const btn = (action, ic, label, cls) => "<button type='button' class='btn btn-sm rowaction" + (cls ? " " + cls : "") + "' data-action='" + action +
       "' data-role='" + esc(l.role) + "'>" + icon(ic) + esc(label) + "</button>";
-    let actions = btn("prompt", "chat", "Prompt this loop");
+    let actions = btn("prompt", "chat", "Message this role");
     // The running set (info.live), not the permit-holder inFlight: the exempt running director
     // has an abortable tick without holding a maxConcurrent permit (BUGS.md 2026-10-06).
-    if (info.live) actions += abortConfirming(l.role) ? btn("abort", "stop", "Click again to abort", "btn-danger confirming") : btn("abort", "stop", "Abort this tick", "btn-danger");
+    if (info.live) actions += abortConfirming(l.role) ? btn("abort", "stop", "Click again to stop", "btn-danger confirming") : btn("abort", "stop", "Stop this run", "btn-danger");
     else {
       actions += btn("wake", "bolt", "Wake now");
       if (roleWakeMenuOpen(l.role)) {
         actions += ROLE_WAKE_CHOICES.map((c) => "<button type='button' class='btn btn-sm rowaction' data-action='wake-in' data-for='" + c[0] +
           "' data-role='" + esc(l.role) + "'>in " + esc(c[1]) + "</button>").join("");
       } else actions += btn("wake-in", "clock", "Wake later…");
-      // Exact id only: the cancel matches ids exactly, so an instance named via its base role would cancel nothing.
+      // Exact id only: the cancel matches ids exactly, so an instance named via its base role
+      // would cancel nothing.
       if (scheduledWakesFor(l, d).some((w) => w.roles.includes(l.role))) actions += btn("wake-cancel", "x", "Cancel scheduled wake");
     }
     if (paused) actions += btn("resume", "play", "Resume");
@@ -125,44 +142,47 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
         "' data-role='" + esc(l.role) + "'>" + esc(c[1]) + "</button>").join("");
     } else actions += btn("pause", "pause", "Pause") + btn("pause-for", "clock", "Pause for…");
     actions += btn("reset-counters", "refresh", "Reset counters");
+    // The two-click Disable, the drawer half of the enable/disable pair (the Settings card's
+    // Enable re-enables). The director cannot be disabled, so it shows no Disable button.
+    if (l.role !== "director") actions += disableConfirming(l.role) ? btn("disable", "x", "Click again to disable", "btn-danger confirming") : btn("disable", "x", "Disable", "btn-danger");
 
-    const working = info.live || info.key === "vetted";
+    const working = info.live || info.key === "vetted"; // vocabulary-ok: the persisted phase key
     const now = working
-      ? "<section><div class='sec-head'><h3>" + (info.key === "working" ? "Working on" : info.key === "reviewing" ? "Under review" : "Landing") +
-        "</h3></div><div class='note'>" + esc(l.currentWork || landingSummary(d, l.role) || (info.key === "working" ? "Starting up…" : "Waiting for its first words…")) + "</div></section>"
+      ? "<section><div class='sec-head'><h3>" + (info.key === "working" ? "Working on" : info.key === "reviewing" ? "Under review" : "Merging") +
+        "</h3></div><div class='note'>" + esc(l.currentWork || mergeSummary(d, l.role) || (info.key === "working" ? "Starting up…" : "Waiting for its first words…")) + "</div></section>"
       : "";
     const r = l.lastResult ? resultInfo(l.lastResult) : null;
     const why = resultWhy(l);
-    const last = "<section><div class='sec-head'><h3>Last result</h3>" + (l.lastTickEndedAt ? "<span class='muted' style='font-size:13px'>" + esc(fmtAgo(l.lastTickEndedAt)) + "</span>" : "") +
-      "</div><div class='note'>" + (r ? "<span class='res t-" + r.tone + "'>" + esc(r.label) + "</span>" + (why ? " — " + esc(why) : "") : "<span class='muted'>No completed ticks yet.</span>") + "</div></section>";
-    const kv = (k, v, title) => "<div><dt>" + esc(k) + "</dt><dd title='" + esc(title || v) + "'>" + esc(v) + "</dd></div>";
+    const last = "<section><div class='sec-head'><h3>Last result</h3>" + (l.lastRunEndedAt ? "<span class='muted' style='font-size:13px'>" + esc(fmtAgo(l.lastRunEndedAt)) + "</span>" : "") +
+      "</div><div class='note'>" + (r ? "<span class='res t-" + r.tone + "'>" + esc(r.label) + "</span>" + (why ? " — " + esc(why) : "") : "<span class='muted'>No completed runs yet.</span>") + "</div></section>";
     const next = fmtNextRun(l, d.running);
     // fmtNextRun's shapes: "-", "now" (possibly with a " ×N" yield suffix), and a bare or
     // "backoff "-prefixed countdown. A backoff countdown keeps the label but as a suffix;
     // a suffixed "now" passes through as-is — it is due now, not a future time.
     const nextNote = next === "-" ? "—"
       : next === "now" || next.startsWith("now ") ? next
-      : next.startsWith("backoff ") ? "in " + next.slice(8) + " (backoff)"
+      : next.startsWith("sleeping ") ? "in " + next.slice(9) + " (sleeping)"
       : "in " + next;
-    const tickWord = info.live ? "this tick" : "last tick";
-    const stats = "<dl class='kv'>" + kv("Commits", String(l.commits)) + kv("Ticks", String(l.ticks)) +
-      kv("Tokens, " + tickWord, fmtTokens(l.generated)) + kv("Peak context, " + tickWord, fmtTokens(l.peakCtx)) +
-      kv("Spent today", fmtUsd(l.todayUsd)) + kv("Spent in total", fmtUsd(l.costUsd)) +
-      kv("Last tick ended", l.lastTickEndedAt ? fmtAgo(l.lastTickEndedAt) : "never", fmtLastTick(l.lastTickEndedAt)) +
-      kv("Next run", nextNote) + "</dl>";
+    const tickWord = info.live ? "this run" : "last run";
+    const stats = "<dl class='kv'>" + kvRow("Commits", String(l.commits)) + kvRow("Runs", String(l.runs)) +
+      kvRow("Tokens, " + tickWord, fmtTokens(l.generated)) + kvRow("Peak context, " + tickWord, fmtTokens(l.peakCtx)) +
+      kvRow("Spent today", fmtUsd(l.todayUsd)) + kvRow("Spent in total", fmtUsd(l.costUsd)) +
+      kvRow("Last run ended", l.lastRunEndedAt ? fmtAgo(l.lastRunEndedAt) : "never", fmtLastTick(l.lastRunEndedAt)) +
+      kvRow("Last merged", l.lastMergedAt ? fmtAgo(l.lastMergedAt) : "never") +
+      kvRow("Next run", nextNote) + "</dl>";
     const ticks = drawerTicks === null
       ? "<p class='muted'>Loading…</p>"
       : drawerTicks.length
-        ? "<ul class='ticks'>" + drawerTicks.map((t) => {
+        ? "<ul class='ticks'>" + drawerTicks.map((t) => { // vocabulary-ok: the ticks class stays
           const res = resultInfo(t.result);
-          return "<li><span class='when' title='" + esc(t.time) + "'>#" + esc(t.tick) + " · " + esc(fmtAgo(t.ts)) + "</span>" +
+          return "<li><span class='when' title='" + esc(t.time) + "'>#" + esc(t.run) + " · " + esc(fmtAgo(t.ts)) + "</span>" +
             "<span class='clamp1' title='" + esc(t.detail) + "'><span class='res t-" + res.tone + "'>" + esc(res.label) + "</span> " + esc(t.detail) + "</span>" +
             "<span class='when'>" + (t.durationMs === null ? "" : esc(fmtSpan(Number(t.durationMs)))) + "</span></li>";
         }).join("") + "</ul>"
-        : "<p class='muted'>No completed ticks in the log yet.</p>";
+        : "<p class='muted'>No completed runs in the log yet.</p>";
     paintPanel("draweractions", actions);
     paintPanel("drawermeta", now + last + stats +
-      "<section><div class='sec-head'><h3>Recent ticks</h3></div>" + ticks + "</section>" + drawerChangeSection());
+      "<section><div class='sec-head'><h3>Recent runs</h3></div>" + ticks + "</section>" + drawerChangeSection());
     // The notebook/prompt live in their own panel, outside drawermeta: the poll rewrites
     // drawermeta whenever its countdown or tick-age text changes, and an innerHTML rewrite
     // would collapse an opened Next tick prompt. This panel's markup is stable while its
@@ -170,7 +190,7 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     paintPanel("drawerrole", drawerRoleSection());
   }
 
-  // The open loop's unlanded change, as /api/diff?role serves it: the full patch the CLI's
+  // The open loop's unmerged change, as /api/diff?role serves it: the full patch the CLI's
   // tumwater diff --role prints. Degrades like the collector — a missing worktree, a missing
   // base branch, or no work each get a plain statement, and an unreadable payload says so
   // rather than rendering a half-view. Pure of side effects; the view paints what it returns.
@@ -178,17 +198,17 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     if (!view || typeof view !== "object") return "<p class='muted'>Pending change unavailable.</p>";
     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 worktree yet — this loop has not run.</p>";
+    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 !== "ready" || (view.ahead === 0 && dirty.length === 0)) return "<p class='muted'>No unlanded work.</p>";
     let html = "<p class='note'>" + esc(view.branch) + " · " + esc(plural(view.ahead, "commit")) +
       " ahead of " + esc(view.mainBranch) +
       (dirty.length ? " · " + esc(plural(dirty.length, "uncommitted file")) : "") + "</p>";
     html += commitRowsHtml(commits);
-    if (view.diff) html += "<pre class='mono diff'>" + esc(String(view.diff).replace(/\n+$/, "")) + "</pre>";
+    if (view.diff) html += diffPre(view.diff);
     if (dirty.length) {
       html += "<p class='note'>Uncommitted: " + esc(dirty.join(", ")) + "</p>";
-      if (view.uncommittedDiff) html += "<pre class='mono diff'>" + esc(String(view.uncommittedDiff).replace(/\n+$/, "")) + "</pre>";
+      if (view.uncommittedDiff) html += diffPre(view.uncommittedDiff);
     }
     return html;
   }
@@ -218,7 +238,7 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     }
     if (lastStatus) renderLoopDrawer(lastStatus);
   }
-  // The open loop's unlanded change, as /api/diff?role serves it.
+  // The open loop's unmerged change, as /api/diff?role serves it.
   function loadDrawerChange(role) {
     return loadDrawerSection(role, "/api/diff?role=", "Pending change unavailable", (v) => { drawerChange = v; });
   }
@@ -235,11 +255,12 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     const r = drawerRole;
     let body;
     if (!r || r.state === "loading") body = "<p class='muted'>Loading…</p>";
-    else if (r.state === "error") body = "<p class='muted'>" + esc(r.message || "Notebook and prompt unavailable") + "</p>";
+    else if (r.state === "error") body = "<p class='muted'>" + esc(r.message || "Notes and prompt unavailable") + "</p>";
     else {
       const view = r.view && typeof r.view === "object" ? r.view : {};
-      const note = typeof view.note === "string" && view.note.trim() !== "" ? view.note : null;
+      const note = typeof view.notes === "string" && view.notes.trim() !== "" ? view.notes : null;
       const nextPrompt = typeof view.nextPrompt === "string" && view.nextPrompt.trim() !== "" ? view.nextPrompt : null;
+      const outcomes = Array.isArray(view.recentOutcomes) ? view.recentOutcomes : [];
       const canEdit = r.role !== "director";
       const editing = canEdit && noteEditing && noteEditing.role === r.role;
       const buttons = !canEdit ? "" : "<span style='margin-left:auto'><button type='button' class='btn btn-sm' data-act='note-edit' data-arg='" + esc(r.role) + "'>Edit</button>" +
@@ -248,31 +269,267 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
         ? "<textarea id='noteedittext' rows='12' class='mono' style='width:100%'>" + esc(noteEditing.text) + "</textarea>" +
           "<div><button type='button' class='btn btn-sm btn-primary' data-act='note-save' data-arg='" + esc(r.role) + "'>Save</button> " +
           "<button type='button' class='btn btn-sm' data-act='note-discard' data-arg='" + esc(r.role) + "'>Discard</button></div>"
-        : note ? "<pre class='mono'>" + esc(note) + "</pre>" : "<p class='muted'>No notebook yet.</p>";
-      body = "<div class='sec-head'><h3>Notebook</h3>" + (editing ? "" : buttons) + "</div>" + noteBody +
-        "<details><summary>Next tick prompt</summary>" + (nextPrompt ? "<pre class='mono'>" + esc(nextPrompt) + "</pre>" : "<p class='muted'>Nothing to run (empty inbox).</p>") + "</details>";
+        : note ? "<pre class='mono'>" + esc(note) + "</pre>" : "<p class='muted'>No notes yet.</p>";
+      body = "<div class='sec-head'><h3>Notes</h3>" + (editing ? "" : buttons) + "</div>" + noteBody +
+        "<details><summary>Next run's prompt</summary>" + (nextPrompt ? "<pre class='mono'>" + esc(nextPrompt) + "</pre>" : "<p class='muted'>Nothing to run (empty inbox).</p>") + "</details>" +
+        // The loop's recent tick outcomes (rolePayload's recentOutcomes, read from the event
+        // log), rendered like the Recent ticks list above: each with its relative age, result
+        // label, summary, and duration. Mirrors the Recent tick outcomes section of
+        // tumwater role <id> — no backticks allowed here (this whole script is one big
+        // String.raw template), so the page and the CLI read one loop's outcomes identically.
+        (outcomes.length ? "<section><div class='sec-head'><h3>Recent outcomes" +
+          "<span class='muted' style='font-size:13px'>· " + outcomes.length + "</span></h3></div>" +
+          "<ul class='ticks'>" + outcomes.map((o) => { // vocabulary-ok: the ticks class stays
+            const info = resultInfo(o.result);
+            const detail = typeof o.summary === "string" && o.summary.trim() !== "" ? o.summary : "";
+            return "<li><span class='when' title='" + esc(new Date(Number(o.ts)).toLocaleString()) + "'>" + esc(fmtAgo(Number(o.ts))) + "</span>" +
+              "<span class='clamp1' title='" + esc(detail) + "'><span class='res t-" + info.tone + "'>" + esc(info.label) + "</span>" + (detail ? " " + esc(detail) : "") + "</span>" +
+              "<span class='when'>" + (o.durationMs == null ? "" : esc(fmtSpan(Number(o.durationMs)))) + "</span></li>";
+          }).join("") + "</ul></section>" : "") +
+        drawerConfigHtml(view, r.role);
     }
     return "<section>" + body + "</section>";
   }
+  // 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 loop RUNS ON (model/tier, provider cap, image reading, the
+  // fallback pairs, billing, min tick 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
+  // 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("/");
+    const singleLine = (s) => String(s).replace(/\s+/g, " ").trim();
+    // A verbatim value's single-line clamped rendering (80 chars + ellipsis) with the full
+    // text riding the title tooltip — the same clamp+tooltip idiom the tick summaries use.
+    const clamp = (t) => { const one = singleLine(t); return one.length > 80 ? one.slice(0, 80) + "…" : one; };
+    const rows = [];
+    // A role that runs on pi's own saved default carries no provider/model but a piDefault
+    // selector (rolePayload's resolvedModelFields leaves both undefined), and the CLI's
+    // Model line names that default (<agent> default (<selector>), agentDefault). Mirror
+    // that branch so the drawer names the model a default pi team actually runs on.
+    const defaultLine = typeof view.piDefault === "string"
+      ? (view.backend && view.backend !== "pi" ? view.backend : "pi") + " default (" + view.piDefault + ")"
+      : "";
+    const modelLine = defaultLine !== "" ? defaultLine : pair(view.provider, view.model);
+    if (modelLine !== "" || view.modelTier !== undefined) {
+      rows.push(kvRow("Model", (modelLine !== "" ? modelLine + " " : "") + (view.modelTier !== undefined ? "(" + view.modelTier + " tier)" : "").trim()));
+    }
+    // The per-role model triple (roles.<base>.provider/.model/.thinking, the file's own raw
+    // overrides rolePayload's providerOverride/modelOverride/thinkingOverride carry) — the
+    // one per-role write the Model row above still renders read-only, the dashboard half of
+    // the CLI's --provider/--model/--thinking. An always-shown row (every role, the director
+    // included, since the CLI writes its triple too) whose body names the payload's raw
+    // overrides, or no overrides, and whose control is a model-edit button (Edit/Set model);
+    // while the editor is open for this role, three text inputs prefilled with the raw
+    // overrides (empty when unset) plus Save/Discard replace the body, mirroring the interval
+    // editor's markup.
+    const rawTriple = [];
+    if (typeof view.providerOverride === "string" && view.providerOverride !== "") rawTriple.push("provider: " + view.providerOverride);
+    if (typeof view.modelOverride === "string" && view.modelOverride !== "") rawTriple.push("model: " + view.modelOverride);
+    if (typeof view.thinkingOverride === "string" && view.thinkingOverride !== "") rawTriple.push("thinking: " + view.thinkingOverride);
+    const modelBody = rawTriple.length ? rawTriple.join(" · ") : "no overrides";
+    const editModel = modelEditing && modelEditing.role === role;
+    const modelControl = editModel
+      ? "<input id='modeleditprovider' type='text' placeholder='provider' value='" + esc(modelEditing.provider) + "'>" +
+        "<input id='modeleditmodel' type='text' placeholder='model' value='" + esc(modelEditing.model) + "'>" +
+        "<input id='modeleditthinking' type='text' placeholder='thinking' value='" + esc(modelEditing.thinking) + "'>" +
+        "<div><button type='button' class='btn btn-sm btn-primary' data-act='model-save' data-arg='" + esc(role) + "'>Save</button> " +
+        "<button type='button' class='btn btn-sm' data-act='model-discard' data-arg='" + esc(role) + "'>Discard</button></div>"
+      : "<button type='button' class='btn btn-sm' data-act='model-edit' data-arg='" + esc(role) + "'>" + (rawTriple.length ? "Edit" : "Set model") + "</button>";
+    rows.push("<div><dt>Model overrides</dt><dd title='" + esc(modelBody) + "'>" + esc(modelBody) + modelControl + "</dd></div>");
+    // Context window: the resolved primary seam's declared window - the same seam '- Next run:'
+    // previews. Guard optional to keep an older payload that lacks the field rendering as before.
+    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")
+        : "unknown";
+      rows.push(kvRow("Images", images));
+    }
+    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", pair(fb.provider, fb.model) + " — on fallback" + when + " (primary failing: " + (fb.reason ?? "") + ")"));
+    }
+    if (view.fallback && typeof view.fallback === "object" && view.fallback.model !== undefined) {
+      rows.push(kvRow("Budget fallback", pair(view.fallback.provider, view.fallback.model) + " (" + (view.fallbackFree ? "free" : "priced") + ")"));
+    }
+    if (view.billing !== undefined) {
+      const window = view.billing === "subscription" && view.billingWindow ? " (" + view.billingWindow + " window)" : "";
+      rows.push(kvRow("Billing", (view.billing === "api" ? "API" : view.billing) + window));
+    }
+    // The min run interval (rolePayload's minRunIntervalSeconds), worded like role-render.ts's
+    // Min run interval line, plus a control to set/clear the per-role override — the last
+    // per-role hold the drawer edits. Shown whenever the payload names an interval (always —
+    // the effective value is the role's override or the team default); the body tags whether
+    // the shown value is the role's own override or the team default. While the editor is open
+    // for this role, an input prefilled with the override (empty when none) and Save/Discard
+    // replace the sentence, mirroring the daily-cap editor's markup; a role with no override
+    // still seeds an empty input, so an empty buffer on Save clears.
+    if (typeof view.minRunIntervalSeconds === "number") {
+      const editing = intervalEditing && intervalEditing.role === role;
+      const intervalBody = String(view.minRunIntervalSeconds) + "s" +
+        (typeof view.minRunIntervalOverride === "number" ? " (override)" : " (team default)");
+      const seed = intervalEditing ? intervalEditing.value : null;
+      const intervalControl = editing
+        ? "<input id='intervaleditinput' type='number' min='0' step='1' value='" + (seed === "" || seed == null ? "" : String(seed)) + "' placeholder='team default'>" +
+          "<div><button type='button' class='btn btn-sm btn-primary' data-act='interval-save' data-arg='" + esc(role) + "'>Save</button> " +
+          "<button type='button' class='btn btn-sm' data-act='interval-discard' data-arg='" + esc(role) + "'>Discard</button></div>"
+        : "<button type='button' class='btn btn-sm' data-act='interval-edit' data-arg='" + esc(role) + "'>" + (typeof view.minRunIntervalOverride === "number" ? "Edit" : "Set interval") + "</button>";
+      rows.push("<div><dt>Min run interval</dt><dd title='" + esc(intervalBody) + "'>" + esc(intervalBody) + intervalControl + "</dd></div>");
+    }
+    // The parallel-runner count (roles.<base>.instances), worded like role-render.ts's
+    // Instances line — the last per-role hold the drawer edits. Rendered only for a role that
+    // may split (its base, an instance strip, is feature or bugfix): the count is meaningless
+    // for any other role, which role-view never carries. The body reads the payload's numeric
+    // count (roles.<base>.instances) tagged (set) or falls back to 1 tagged (default) when the
+    // file leaves it unset — role-view carries instances only when the file sets it. While the
+    // editor is open for this role, an input prefilled with the count (empty when none) and
+    // Save/Discard
+    // replace the sentence, mirroring the interval editor's markup; a role with no count still
+    // seeds an empty input, so an empty buffer on Save clears.
+    const splitBase = role.replace(/-\d+$/, "");
+    if (splitBase === "feature" || splitBase === "bugfix") {
+      const hasInstances = typeof view.instances === "number";
+      const instCount = hasInstances ? view.instances : 1;
+      const instBody = String(instCount) + (hasInstances ? " (set)" : " (default)");
+      const editingInst = instancesEditing && instancesEditing.role === role;
+      const seed = instancesEditing ? instancesEditing.value : null;
+      const instControl = editingInst
+        ? "<input id='instanceseditinput' type='number' min='1' max='8' step='1' value='" + (seed === "" || seed == null ? "" : String(seed)) + "' placeholder='default 1'>" +
+          "<div><button type='button' class='btn btn-sm btn-primary' data-act='instances-save' data-arg='" + esc(role) + "'>Save</button> " +
+          "<button type='button' class='btn btn-sm' data-act='instances-discard' data-arg='" + esc(role) + "'>Discard</button></div>"
+        : "<button type='button' class='btn btn-sm' data-act='instances-edit' data-arg='" + esc(role) + "'>" + (hasInstances ? "Edit" : "Set instances") + "</button>";
+      rows.push("<div><dt>Instances</dt><dd title='" + esc(instBody) + "'>" + esc(instBody) + instControl + "</dd></div>");
+    }
+    // The windows governing the loop, worded like role-render.ts's Quiet hours line; the end time
+    // mirrors quietWindowEnd (browser code cannot import it). A non-director role's own window
+    // is an always-shown editable row (the Daily cap row's rationale: the control sits where
+    // the hold is shown) — including when the payload carries no role-scope window ("no own
+    // window"); any other window stays a read-only row worded exactly as before ("team window").
+    const own = Array.isArray(view.quietHours) ? view.quietHours.find(function (w) { return w && w.scope === "role"; }) : undefined;
+    const ownWindow = own && typeof own.window === "string" ? own.window : "";
+    if (role !== "director") {
+      const ownEnd = String(ownWindow).split("-")[1];
+      const ownState = own && own.inWindow
+        ? "holding now — quiet until " + (ownEnd ? ownEnd.trim() : "")
+        : "not holding now";
+      const editQuiet = quietEditing && quietEditing.role === role;
+      const quietBody = ownWindow !== "" ? ownWindow + " (own window; " + ownState + ")" : "no own window";
+      const quietControl = editQuiet
+        ? "<input id='quieteditinput' type='text' placeholder='23:00-07:00' value='" + esc(quietEditing.window) + "'>" +
+          "<div><button type='button' class='btn btn-sm btn-primary' data-act='quiet-save' data-arg='" + esc(role) + "'>Save</button> " +
+          "<button type='button' class='btn btn-sm' data-act='quiet-discard' data-arg='" + esc(role) + "'>Discard</button></div>"
+        : "<button type='button' class='btn btn-sm' data-act='quiet-edit' data-arg='" + esc(role) + "'>" + (ownWindow !== "" ? "Edit" : "Set quiet hours") + "</button>";
+      rows.push("<div><dt>Quiet hours</dt><dd title='" + esc(quietBody) + "'>" + esc(quietBody) + quietControl + "</dd></div>");
+    }
+    if (Array.isArray(view.quietHours)) view.quietHours.forEach(function (w) {
+      if (!w || w.scope === "role" || typeof w.window !== "string") return;
+      const end = String(w.window).split("-")[1];
+      const state = w.inWindow ? "holding now — quiet until " + (end ? end.trim() : "") : "not holding now";
+      rows.push(kvRow("Quiet hours", w.window + " (team window; " + state + ")"));
+    });
+    if (typeof view.inboxCount === "number") rows.push(kvRow("Queued prompts", String(view.inboxCount)));
+    // The loop's own spend over its retained event window (rolePayload's usage, summed from the
+    // same run_end events recentOutcomes reads): how much the loop burned on tokens and cost,
+    // with a notional suffix only when a claude-subscription seam burned non-zero notional spend
+    // — mirroring role-render.ts's Spend line, so the drawer and tumwater role <id> read one
+    // loop's spend identically. Each numeric field is coerced (foreign shapes read as 0) and an
+    // all-zero usage is absent on the payload, so this row never renders for a never-costly loop.
+    if (view.usage && typeof view.usage === "object") {
+      const u = view.usage;
+      const real = Number(u.costUsd) || 0;
+      const tokens = Number(u.tokens) || 0;
+      const notional = Number(u.notionalCostUsd) || 0;
+      let spend = fmtUsd(real) + " · " + fmtTokens(tokens) + " tokens";
+      if (notional !== 0) spend += " · " + fmtUsd(notional) + " notional";
+      rows.push(kvRow("Spend", spend));
+    }
+    // The per-role daily cap (rolePayload's dailyCap), worded like role-render.ts's Daily cap line.
+    // The per-role daily cap (rolePayload's dailyCap), worded like role-render.ts's Daily cap
+    // line. Shown for every non-director role — always, cap or not — because the "Set a cap"/
+    // "Edit cap" control is the end of the amber role-cap alert's path: the alert's remedy
+    // leads here. A director shows no row (role-view.ts's guard exempts it from a cap). While
+    // the cap editor is open for this role, an input prefilled with the current cap (empty for
+    // none) and Save/Discard replace the sentence, mirroring the notebook editor's markup.
+    if (role !== "director") {
+      const dc = view.dailyCap && typeof view.dailyCap === "object" ? view.dailyCap : null;
+      const editing = capEditing && capEditing.role === role;
+      const capBody = dc
+        ? fmtUsd(Number(dc.spentUsd) || 0) + " of " + fmtUsd(Number(dc.capUsd) || 0) + " spent today (" + (dc.over ? "holding now — no new runs until local midnight" : "not holding") + ")" +
+          (Array.isArray(dc.roleIds) && dc.roleIds.length > 1 ? " · shared by " + dc.roleIds.join(", ") : "")
+        : "no cap";
+      const capControl = editing
+        ? "<input id='capeditinput' type='number' min='0' step='0.01' value='" + (capEditing.value ? String(capEditing.value) : "") + "' placeholder='no cap'>" +
+          "<div><button type='button' class='btn btn-sm btn-primary' data-act='cap-save' data-arg='" + esc(role) + "'>Save</button> " +
+          "<button type='button' class='btn btn-sm' data-act='cap-discard' data-arg='" + esc(role) + "'>Discard</button></div>"
+        : "<button type='button' class='btn btn-sm' data-act='cap-edit' data-arg='" + esc(role) + "'>" + (dc ? "Edit cap" : "Set a cap") + "</button>";
+      rows.push("<div><dt>Daily cap</dt><dd title='" + esc(capBody) + "'>" + esc(capBody) + capControl + "</dd></div>");
+    }
+    // The standing instructions (roles.<base>.instructions, resolved through the base role),
+    // worded like role-render.ts's Instructions override. Shown for every role — always,
+    // override or not — because the Edit/Save control is how the director's and instance
+    // runs' instructions get written from the dashboard: the same always-shown rationale as
+    // the Daily cap row above. While the editor is open for this role, a textarea prefilled
+    // with the current text and Save/Discard replace the sentence, mirroring the notebook
+    // editor's markup; a role with no override still seeds an empty textarea, so an empty
+    // buffer on Save clears.
+    const instEditing = instructionsEditing && instructionsEditing.role === role;
+    const instText = typeof view.instructions === "string" ? view.instructions : "";
+    const instBody = instText.trim() !== "" ? clamp(instText) : "No standing instructions.";
+    const instControl = instEditing
+      ? "<textarea id='instructionsedittext' rows='10' class='mono' style='width:100%'>" + esc(instructionsEditing.text) + "</textarea>" +
+        "<div><button type='button' class='btn btn-sm btn-primary' data-act='instructions-save' data-arg='" + esc(role) + "'>Save</button> " +
+        "<button type='button' class='btn btn-sm' data-act='instructions-discard' data-arg='" + esc(role) + "'>Discard</button></div>"
+      : "<button type='button' class='btn btn-sm' data-act='instructions-edit' data-arg='" + esc(role) + "'>Edit</button>";
+    rows.push("<div><dt>Instructions override</dt><dd title='" + esc(instText) + "'>" + esc(instBody) + instControl + "</dd></div>");
+    if (typeof view.find === "string" && view.find.trim() !== "") rows.push(kvRow("Find text", clamp(view.find), view.find));
+    return rows.length ? "<section><div class='sec-head'><h3>Configuration</h3></div><dl class='kv'>" + rows.join("") + "</dl></section>" : "";
+  }
   // An open notebook edit holds the draft's starting text, not the live payload, so a poll
   // refetch neither wipes the textarea nor changes the painted panel string. Clear waits for a
   // confirming click like entry cancel: the first click arms it for 5 s, the second posts.
   let noteEditing = null; // { role, text }
-  let confirmNoteClear = null; // { role, until }
+  let capEditing = null; // { role, value }
+  let instructionsEditing = null; // { role, text }
+  let quietEditing = null; // { role, window }
+  let intervalEditing = null; // { role, value } — value is the seeded override (number) or "" when none
+  let instancesEditing = null; // { role, value } — value is the seeded count (number) or "" when none
+  let modelEditing = null; // { role, provider, model, thinking } — the seeded raw overrides ("" when unset)
   function repaintDrawerRole() { if (lastStatus) renderLoopDrawer(lastStatus); }
-  function startNoteEdit(role, text) { noteEditing = { role, text }; confirmNoteClear = null; repaintDrawerRole(); }
+  // The notebook clear and the backlog entry cancel each build a two-click confirm slot from
+  // the shared confirmSlot factory (defined in the loops splice, an earlier one), so the arm,
+  // guard, and disarm logic lives once.
+  const noteClear = confirmSlot(repaintDrawerRole);
+  const noteClearArmed = (role) => noteClear.armed(role);
+  function armNoteClear(role) { noteClear.arm(role); }
+  function disarmNoteClear() { noteClear.disarm(); }
+  function startNoteEdit(role, text) { noteEditing = { role, text }; disarmNoteClear(); repaintDrawerRole(); }
   function stopNoteEdit() { noteEditing = null; repaintDrawerRole(); }
-  const noteClearArmed = (role) => Boolean(confirmNoteClear && confirmNoteClear.role === role && confirmNoteClear.until > Date.now());
-  function disarmNoteClear() { confirmNoteClear = null; }
-  function armNoteClear(role) {
-    confirmNoteClear = { role, until: Date.now() + 5000 };
-    repaintDrawerRole();
-    setTimeout(() => { if (confirmNoteClear && confirmNoteClear.role === role && confirmNoteClear.until <= Date.now()) { confirmNoteClear = null; repaintDrawerRole(); } }, 5100);
-  }
+  function startCapEdit(role, capUsd) { capEditing = { role, value: capUsd ? Number(capUsd) || 0 : 0 }; repaintDrawerRole(); }
+  function stopCapEdit() { capEditing = null; repaintDrawerRole(); }
+  function startInstructionsEdit(role, text) { instructionsEditing = { role, text: String(text ?? "") }; repaintDrawerRole(); }
+  function stopInstructionsEdit() { instructionsEditing = null; repaintDrawerRole(); }
+  function startIntervalEdit(role, value) { intervalEditing = { role, value: value === "" || value == null ? "" : Number(value) }; repaintDrawerRole(); }
+  function stopIntervalEdit() { intervalEditing = null; repaintDrawerRole(); }
+  function startInstancesEdit(role, value) { instancesEditing = { role, value: value === "" || value == null ? "" : Number(value) }; repaintDrawerRole(); }
+  function stopInstancesEdit() { instancesEditing = null; repaintDrawerRole(); }
+  function startModelEdit(role, seed) { modelEditing = { role, provider: String((seed && seed.provider) ?? ""), model: String((seed && seed.model) ?? ""), thinking: String((seed && seed.thinking) ?? "") }; repaintDrawerRole(); }
+  function stopModelEdit() { modelEditing = null; repaintDrawerRole(); }
+  function startQuietEdit(role, window) { quietEditing = { role, window: String(window ?? "") }; repaintDrawerRole(); }
+  function stopQuietEdit() { quietEditing = null; repaintDrawerRole(); }
   // The notebook fetch shares loadDrawerSection; closing the drawer clears drawerRole, so
   // reopening refetches.
   function loadDrawerRole(role) {
-    return loadDrawerSection(role, "/api/role?role=", "Notebook and prompt unavailable", (v) => { drawerRole = v; });
+    return loadDrawerSection(role, "/api/role?role=", "Notes and prompt unavailable", (v) => { drawerRole = v; });
   }
 
   async function loadLoopTicks(force) {
@@ -309,7 +566,7 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     const pre = $("transcript");
     if (openLoopRole() !== role || !pre) return;
     const lines = Array.isArray(d.lines) ? d.lines : [];
-    const html = lines.length ? transcriptHtml(lines) : "<span class='muted'>No transcript yet — it fills in once this loop's next tick starts.</span>";
+    const html = lines.length ? transcriptHtml(lines) : "<span class='muted'>No transcript yet — it fills in once this role's next run starts.</span>";
     // Stay pinned to the newest line while the reader is at the bottom; leave them be if they
     // scrolled up to read.
     const atBottom = pre.scrollTop + pre.clientHeight >= pre.scrollHeight - 12;
@@ -317,17 +574,13 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
   }
 
   // A plan/bug cancel waits for its confirming click, the confirmAbort shape: the first click
-  // arms it for 5 s, a second inside the window fires, and a timeout disarms.
-  let confirmEntryCancel = null; // { key, until }
-  // The arm records the title too: a list that shifted between the clicks must re-arm, not
-  // cancel whatever entry now sits at that position.
-  const entryCancelArmed = (key, title) => Boolean(confirmEntryCancel && confirmEntryCancel.key === key && (title === undefined || confirmEntryCancel.title === title) && confirmEntryCancel.until > Date.now());
-  function disarmEntryCancel() { confirmEntryCancel = null; }
-  function armEntryCancel(key, title) {
-    confirmEntryCancel = { key, title, until: Date.now() + 5000 };
-    refreshEntry();
-    setTimeout(() => { if (confirmEntryCancel && confirmEntryCancel.key === key && confirmEntryCancel.until <= Date.now()) { confirmEntryCancel = null; refreshEntry(); } }, 5100);
-  }
+  // arms it for 5 s, a second inside the window fires, and a timeout disarms. The slot records
+  // the title too: a list that shifted between the clicks must re-arm, not cancel whatever
+  // entry now sits at that position.
+  const entryCancel = confirmSlot(refreshEntry);
+  const entryCancelArmed = (key, title) => entryCancel.armed(key, title);
+  function disarmEntryCancel() { entryCancel.disarm(); }
+  function armEntryCancel(key, title) { entryCancel.arm(key, title); }
   // An open edit form: painted once, so a poll never overwrites the operator's draft; the
   // server's title check catches an entry that changed underneath it at save time.
   let entryEditing = null; // { key, title, painted }
@@ -336,7 +589,7 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
     refreshEntry();
   }
   function stopEntryEdit() { entryEditing = null; }
-  const ENTRY_KINDS = { plans: "Planned feature", bugs: "Open bug", questions: "Open question" };
+  const ENTRY_KINDS = { plans: "Planned feature", bugs: "Open bug", questions: "Open question", done: "Done plan", fixed: "Fixed bug", answered: "Answered question" };
   async function refreshEntry() {
     if (!drawer || drawer.kind !== "entry") return;
     const file = drawer.file;
@@ -363,21 +616,23 @@ export const GUI_CLIENT_DRAWER_JS = String.raw `  let drawer = null; // { kind:
         "<textarea id='entryeditbody' rows='14'>" + esc(d.body || "") + "</textarea>");
       return;
     }
-    const action = file === "questions"
+    const action = file === "done" || file === "fixed" || file === "answered" ? "" : file === "questions"
       ? "<textarea id='answertext' rows='3' placeholder='Your answer — recorded in QUESTIONS.md'></textarea>" +
         "<button type='button' class='btn btn-sm btn-primary' data-act='answer-record' data-arg='" + index + "'>" + icon("chat") + "Record answer</button>" +
         "<button type='button' class='btn btn-sm' data-act='answer' data-arg='" + index + "'>Ask the director instead</button>"
       : "<button type='button' class='btn btn-sm' data-act='mention' data-arg='" + esc(file + ":" + index) + "'>" + icon("chat") + "Ask the director about this</button>" +
+        (d.refused || d.waiting ? "<button type='button' class='btn btn-sm btn-primary' data-act='entry-release' data-arg='" + esc(file + ":" + index) + "'>Release this entry</button>" : "") +
+        (index > 0 ? "<button type='button' class='btn btn-sm' data-act='entry-top' data-arg='" + esc(file + ":" + index) + "'>Move to top</button>" : "") +
         "<button type='button' class='btn btn-sm' data-act='entry-edit' data-arg='" + esc(file + ":" + index) + "'>Edit this entry</button>" +
         "<button type='button' class='btn btn-sm btn-danger' data-act='entry-cancel' data-arg='" + esc(file + ":" + index) + "'>" + (entryCancelArmed(file + ":" + index) ? "Click again to cancel" : "Cancel this entry") + "</button>";
     // The body is model-written Markdown: renderMarkdown escapes every piece of it.
-    paintPanel("entrybody", "<div class='drawer-actions'>" + action + "</div><div class='md'>" +
+    paintPanel("entrybody", (action ? "<div class='drawer-actions'>" + action + "</div>" : "") + "<div class='md'>" +
       (d.body ? renderMarkdown(d.body) : "<p class='muted'>No details for this entry.</p>") + "</div>");
   }
 
   async function refreshDrawer() {
     if (!drawer) return;
-    if (drawer.kind === "loop") {
+    if (drawer.kind === "loop") { // vocabulary-ok: the drawer kind stays "loop"
       if (lastStatus) renderLoopDrawer(lastStatus);
       await refreshTranscript(false);
     } else {