toolroll 0.9.103 → 0.9.105

package/dist/server/render-chat.js

@@ -37,9 +37,7 @@ import { chatResultHref, projectChatHref, taskChatHref, taskHref } from "./http.
 import {} from "./render-pages.js";
 import { completionReceiptCard } from "./render-results.js";
 import { routeEvidenceHtml, agentsAvailabilityHtml, agentsBadgesHtml, agentsStripHtml, agentsSummaryWords, agentsWhyHtml, approvalSheetHtml, checkProgressHtml, consentClosedHtml, consentDoorOf, decisionAnswerCard, dispatchHeadline, milestoneProgressHtml, planRevisionLedgerHtml, profileWords, sizeConsequence, taskControlDetailsHtml, taskStatusCard, taskViewSwitch } from "./render-tasks.js";
-/** The installation fact that the lead was turned on by default, once (its value: the provider). */
 export const LEAD_BY_DEFAULT_FACT = "lead-on-by-default";
-/** The installation fact that the phone card was put away. */
 export const PHONE_CARD_FACT = "phone-card-dismissed";
 export const TASK_COMPOSER_MODES = {
     build: "The operator chose Build: if they ask for a change to the result, read it with get_result and propose a revision of this task with propose_review (operation revise) for them to confirm. Nothing builds until they confirm the card.",
@@ -47,18 +45,13 @@ export const TASK_COMPOSER_MODES = {
     answer: "The operator chose Just answer: answer from what you can read. Do not propose any action this turn.",
 };
 export function taskChatContext(focus) {
-    return html `<aside class="task-chat-context" aria-label="current task"><div class="task-chat-context-head"><span class="eyebrow">current task</span></div><h2>${focus.title}</h2><p class="meta mono">${focus.id}${focus.project === null ? "" : html ` · ${focus.project}`}</p>${
-    // The agents in chat (v47): the same summary the task page and CLI
-    // print — who does what — with the reasons one tap away, so a
-    // conversation never hides which agent runs.
-    focus.route === null
+    return html `<aside class="task-chat-context" aria-label="current task"><div class="task-chat-context-head"><span class="eyebrow">current task</span></div><h2>${focus.title}</h2><p class="meta mono">${focus.id}${focus.project === null ? "" : html ` · ${focus.project}`}</p>${focus.route === null
         ? ""
         : html `<div class="task-chat-agents-aside"><span class="eyebrow">agents</span><p class="agents-summary">${agentsSummaryWords(focus.route)}</p>${routeEvidenceHtml(focus.route)}<div class="agents-badges">${agentsBadgesHtml(focus.route)}</div>${focus.route.projection === null ? "" : html `${agentsAvailabilityHtml(focus.route.projection)}${agentsWhyHtml(focus.route.projection)}`}<p class="meta"><a href="${taskHref(focus.id)}#agents">Change agents on the task →</a></p></div>`}<div class="task-chat-overview-actions"><a class="task-chat-overview-link" href="${taskHref(focus.id)}">Open full overview →</a></div></aside>`;
 }
 export function taskChatApproval(focus, csrf) {
     const approval = focus.approval;
     if (focus.plan === "requested") {
-        // The task status already names planning. No second status card.
         return html ``;
     }
     if (approval === null)
@@ -74,8 +67,6 @@ export function taskChatApproval(focus, csrf) {
     if (approval.nonce === "") {
         return html `<section class="card chat-action-card" id="task-chat-action"><span class="eyebrow">approval needs attention</span><h2>The agent setup isn’t ready yet</h2>${profileWords(scope)}<a class="button-link" href="${taskHref(focus.id)}#scope">Fix the agent setup →</a></section>`;
     }
-    // The same sheet as the task page (approval critique, Oct 2): the plan
-    // open in plain rows, one Approve & start, the rest in Details.
     return html `<section class="card chat-action-card chat-plan" id="task-chat-action" data-approval="${approval.digest}">${approvalSheetHtml({
         surface: "chat",
         action: `${taskHref(focus.executionId)}/approve`,
@@ -95,7 +86,6 @@ export function taskChatApproval(focus, csrf) {
         deliverable: approval.deliverable,
         releaseCiRunning: approval.releaseCiRunning === true,
         rebuild: focus.assignment?.need != null && "key" in focus.assignment.need && focus.assignment.need.key === "replan" ? focus.assignment.need.build : null,
-        // Edit plan opens the task page's sheet with its fields open for editing.
         editHref: `${taskHref(focus.id)}?edit=plan#plan-editor`,
         notNowHref: "/chat",
         sticky: false,
@@ -103,16 +93,12 @@ export function taskChatApproval(focus, csrf) {
         edit: null,
     })}</section>`;
 }
-/** The chat's status sentence, worded as the task page words it: what a failed attempt missed, or which step a live
- * build is on (and the step it is stuck on). Undefined keeps the shared status sentence. */
 export function chatStatusSentence(focus) {
     if (focus.state === "failed" && focus.failure != null)
         return [focus.failure.line, focus.failure.evidence].filter(Boolean).join(" ");
     const steps = focus.liveRun === null ? null : buildProgressOf(focus.milestoneProgress);
     return steps === null ? undefined : steps.line;
 }
-/** One live, server-derived journey from request to proof. The fragment is
- * safe to refresh independently, so an in-progress message is never lost. */
 export function taskChatLiveRegion(focus, csrf, fragment = false, inert = false, approvalHref = taskChatHref(focus.id) + "#task-chat-action") {
     const receiptLeads = focus.state === "done" && focus.result !== null;
     const approvalContent = (inert && focus.approval !== null && focus.plan !== "requested"
@@ -120,7 +106,6 @@ export function taskChatLiveRegion(focus, csrf, fragment = false, inert = false,
         : fragment && focus.approval !== null && focus.plan !== "requested"
             ? html `<section class="card chat-action-card chat-refresh-action"><a class="button-link" href="${approvalHref}" data-primary-action>Approve plan</a></section>`
             : taskChatApproval(focus, csrf));
-    // The approval leads when dispatch waits on it or the plan was rewritten after a build (the assignment's one action).
     const approvalLeads = focus.dispatch?.action === "approve-scope" ||
         (focus.assignment?.primaryAction?.code === "approve-scope" && focus.assignment.primaryAction.target.taskId === focus.executionId);
     const approvalCard = focus.approval !== null && !approvalLeads
@@ -131,20 +116,12 @@ export function taskChatLiveRegion(focus, csrf, fragment = false, inert = false,
         : receiptLeads ? completionReceiptCard(focus.result, focus.id, "chat", focus.status) : taskStatusCard(focus.status, focus.id, focus.dispatch, focus.liveRun?.id ?? null, focus.approval !== null && approvalLeads);
     const publication = focus.publication;
     const prUrl = publication === null ? null : safePrUrl(publication.prUrl);
-    return html `<section id="task-chat-live" aria-live="polite" data-task="${focus.id}" data-execution="${focus.executionId}" data-source="/chat/task-status?task=${encodeURIComponent(focus.id)}" data-poll="${polling ? "1" : "0"}" data-approval="${focus.approval?.digest ?? ""}" data-plan="${focus.plan ?? ""}">${focus.guide === true && focus.state !== "cancelled" ? firstTaskJourneyHtml(focus) : ""}<div class="task-live-summary">${status}${checkProgressHtml(focus.checkProgress)}${
-    // The exact-run control (v52): the SAME component the task page
-    // renders, refreshed with the live region — typed input in the
-    // composer is untouched because only this region is replaced.
-    taskControlDetailsHtml(focus.control, focus.executionId, csrf, "chat", inert)}${
-    // The compact agents strip (v47): always visible, phones included,
-    // where the desktop context panel is hidden.
-    agentsStripHtml(focus.route, focus.executionId)}</div>${milestoneProgressHtml(focus.milestoneProgress)}${planRevisionLedgerHtml(focus.planRevisions, focus.executionId, csrf)}${approvalCard}${focus.history}${focus.decisions.length === 0
+    return html `<section id="task-chat-live" aria-live="polite" data-task="${focus.id}" data-execution="${focus.executionId}" data-source="/chat/task-status?task=${encodeURIComponent(focus.id)}" data-poll="${polling ? "1" : "0"}" data-approval="${focus.approval?.digest ?? ""}" data-plan="${focus.plan ?? ""}">${focus.guide === true && focus.state !== "cancelled" ? firstTaskJourneyHtml(focus) : ""}<div class="task-live-summary">${status}${checkProgressHtml(focus.checkProgress)}${taskControlDetailsHtml(focus.control, focus.executionId, csrf, "chat", inert)}${agentsStripHtml(focus.route, focus.executionId)}</div>${milestoneProgressHtml(focus.milestoneProgress)}${planRevisionLedgerHtml(focus.planRevisions, focus.executionId, csrf)}${approvalCard}${focus.history}${focus.decisions.length === 0
         ? ""
         : html `<section class="chat-decisions"><div class="chat-section-head"><h2>Needs your answer</h2></div>${focus.decisions.map(one => focus.approval !== null || inert
             ? html `<div class="decide-card"><p class="q">${one.question}</p><p class="meta">${oneLineOf(one.recap, 160)}</p><a href="/d/${one.id}?return=${encodeURIComponent(taskChatHref(focus.id))}">Review and answer →</a></div>`
             : decisionAnswerCard(one, csrf, focus.now, false, taskChatHref(focus.id)))}</section>`}${focus.result === null || receiptLeads ? "" : html `<details class="task-previous-result"><summary>Previous result</summary>${completionReceiptCard(focus.result, focus.id, "chat")}</details>`}${publication === null ? "" : html `<p class="chat-publication meta">Published as ${prUrl === null ? html `<span class="mono">PR #${publication.prNumber ?? "?"}</span>` : html `<a href="${prUrl}">PR #${publication.prNumber ?? "?"}</a>`} · ${publication.state}${publication.lastCheckState === null ? "" : html ` · CI ${publication.lastCheckState}`}</p>`}</section>`;
 }
-/** The first task's way to Ready, filled in as it moves (onboarding): the same stage every surface reads. */
 export function firstTaskJourneyHtml(focus) {
     const planning = focus.plan === "requested" && focus.approval === null;
     const read = focus.assignment !== null ? assignmentStageOf(focus.assignment, { token: focus.status.token }, planning) : stageOfCode(focus.status.token, { planning });
@@ -158,17 +135,11 @@ export function taskChatHeading(focus) {
 export function chatWorkspace(content, projects, inert, focus, resultPanel = null) {
     if (focus === null)
         return html `<div class="chat-workspace">${chatProjectRail(projects, inert)}<section class="chat-main">${content}</section></div>`;
-    // The result detail (package 3) is the ONE auxiliary panel when open:
-    // it takes the context panel's place beside the conversation on a wide
-    // screen and becomes the dedicated view, with Back to chat, when the
-    // screen has no room for both (CSS decides; the markup is the same).
     if (resultPanel !== null) {
         return html `<div class="chat-workspace task-chat-workspace result-open" data-chat-result-open><section class="chat-main">${content}</section><aside class="chat-result t-panel-slide" data-open="true" aria-label="result">${resultPanel}</aside></div>`;
     }
     return html `<div class="chat-workspace task-chat-workspace">${taskChatContext(focus)}<section class="chat-main">${content}</section></div>`;
 }
-/** The folded overview's one-line summary (UI polish 2026-09-13): the
- * two counts a phone reader scans before deciding to open it. */
 export function chatOverviewSummaryHtml(projects, attention) {
     const total = (key) => projects.reduce((sum, one) => sum + (one.peek?.[key] ?? 0), 0);
     const needsYou = attention?.count ?? total("waiting");
@@ -198,8 +169,6 @@ export function chatFleetOverview(snapshot, projects, interactive) {
     for (const task of snapshot.tasks.filter(one => one.state === "failed").slice(0, Math.max(0, 3 - rows.length))) {
         rows.push(html `<a class="chat-overview-item failed" href="${taskHref(task.rootId ?? task.id)}"><span class="chat-overview-icon">${strokeIcon(html `<path d="M12 9v4"/><path d="M12 17h.01"/><path d="m10.3 2.9-8.6 15A2 2 0 0 0 3.4 21h17.2a2 2 0 0 0 1.7-3.1l-8.6-15a2 2 0 0 0-3.4 0z"/>`)}</span><span class="chat-overview-copy"><strong>${task.title}</strong><span>${projectOf(task.repoIndex)} · ${task.id} · failed</span></span><span class="chat-overview-arrow" aria-hidden="true">→</span></a>`);
     }
-    // Current assignments own attention. The saved assessment is history,
-    // never a second queue after the lead marked the result complete.
     for (const task of snapshot.tasks
         .filter(one => one.state === "done" && ["ready-to-check", "needs-decision"].includes(snapshot.assignmentStates?.[one.id]?.state ?? ""))
         .slice(0, Math.max(0, 3 - rows.length))) {
@@ -214,12 +183,6 @@ export function chatFleetOverview(snapshot, projects, interactive) {
         ? postForm("/chat", html `<button type="submit" name="message" value="${briefing}" class="quiet">Brief me</button>`, { attrs: { class: "inline" } })
         : html `<a href="/tasks?view=board&scope=all" class="chat-overview-link">open board</a>`}</div><div class="chat-overview-stats"><a href="/tasks?view=needs-you" class="chat-overview-stat attention"><b>${needsYou}${snapshot.attentionCount?.saturated ? "+" : ""}</b><span>need you</span></a><a href="/board?scope=all" class="chat-overview-stat live"><b>${running}</b><span>building</span></a><a href="/board?scope=all&amp;view=order" class="chat-overview-stat"><b>${queued}</b><span>queued</span></a><a href="/tasks" class="chat-overview-stat"><b>${done}</b><span>finished today</span></a></div>${rows.length === 0 ? html `<p class="chat-overview-clear">${needsYou === 0 && running === 0 ? html `<span class="dot dot-ok"></span>No tasks are waiting and no builds are running.` : html `<a href="/tasks">Open Tasks to inspect current work.</a>`}</p>` : html `<div class="chat-overview-items">${rows}</div>`}${completedWorkHtml(snapshot, projects.map(project => project.label), snapshot.assignmentStates)}${saturated ? html `<p class="meta chat-overview-note">Some items aren’t shown. Open Work to see more.</p>` : ""}</section>`;
 }
-/**
- * The lead's project rail: one bounded pulse per admitted project, plus
- * two roads that preserve the plane's contracts. "ask" sends the stable
- * rN alias the model already sees; "board" uses the existing POST switch
- * instead of smuggling a project change through a GET parameter.
- */
 export function chatProjectRail(projects, inert) {
     const statusOf = (peek) => peek === null
         ? "unavailable"
@@ -241,7 +204,6 @@ export function chatProjectRail(projects, inert) {
     });
     return html `<aside class="chat-projects" id="chat-project-panel" aria-label="projects in this conversation"><div class="chat-projects-head"><h2>Projects</h2><span class="badge">${projects.length}</span><button type="button" class="chat-project-close quiet" aria-label="close projects">×</button></div><div class="chat-project-list">${rows}</div></aside>`;
 }
-/** Spend-authorized one-click questions: ordinary /chat posts, not a new door. */
 export function leadPromptStarters(focus = null) {
     const prompts = focus === null
         ? [
@@ -261,10 +223,6 @@ export function chatHeading(copy, projectCount, showProjectToggle = true) {
         ? html `<button type="button" class="chat-project-toggle quiet" aria-controls="chat-project-panel" aria-expanded="false" title="show or hide projects">${strokeIcon(html `<path d="M4 20h16a2 2 0 0 0 2-2V8a2 2 0 0 0-2-2h-7.9a2 2 0 0 1-1.69-.9L9.6 3.9A2 2 0 0 0 7.93 3H4a2 2 0 0 0-2 2v13c0 1.1.9 2 2 2Z"/>`)}<span>projects</span><span class="badge">${projectCount}</span></button>`
         : ""}</div></div>`;
 }
-/** A deliberately small rich-text grammar for model copy. The reply is shaped first (the lead's voice, enforced:
- * no headers, at most three bold anchors, labelled links, no internal ids unless `asked` wanted them); input is
- * escaped before tags are introduced: bullets, numbered steps, bold, inline code, fenced code blocks and labelled
- * http(s) links are presentation only—never executable HTML. */
 export function renderChatText(raw, asked) {
     const inline = replyHtmlInline;
     const text = shapeReply(raw, { appOrigin: requestContext.getStore()?.appOrigins ?? null, ...(asked === undefined ? {} : { asked }) });
@@ -284,7 +242,6 @@ export function renderChatText(raw, asked) {
         out.push(list.tag === "ul" ? html `<ul>${list.items}</ul>` : html `<ol>${list.items}</ol>`);
         list = null;
     };
-    // A fenced block (shaping leaves it exactly as written) is shown as written: escaped, never parsed.
     let fence = null;
     const closeFence = () => {
         const open = fence;
@@ -318,8 +275,6 @@ export function renderChatText(raw, asked) {
                 closeList();
                 list = { tag: wanted, items: [] };
             }
-            // Model replies often separate steps with blank lines. Each remains
-            // its stated number even when that blank line starts another <ol>.
             list.items.push(html `<li${numbered === null ? "" : html ` value="${Number(numbered[1])}"`}>${inline(bullet?.[1] ?? numbered?.[2] ?? "")}</li>`);
         }
         else if (line.trim() === "") {
@@ -331,23 +286,15 @@ export function renderChatText(raw, asked) {
             paragraph.push(line);
         }
     }
-    // An unclosed fence still shows its lines as code: the reply may have been cut.
     flushParagraph();
     closeList();
     closeFence();
     return html `<div class="chat-copy">${out}</div>`;
 }
-/** A line comment from the chat Workbench (src/browser/line-steer.tsx steerMessage): On "<file, as a JSON string>":<line>
- * [ (removed line)] (run #<n>): <comment>, then a blank line and "> <the line>" (just ">" for a blank line, once the
- * chat POST has trimmed the message), always last.
- * The words the lead reads stay exactly as sent; the thread shows them in three parts: a small reference line, the quoted
- * code as written, and the comment. Null for any other message, which keeps its own plain rendering. Everything is
- * escaped by the html template. */
 export function steerCommentHtml(text) {
     const steer = /^On ("(?:[^"\\\n]|\\.){1,1200}"):(\d{1,9})( \(removed line\))? \(run #(\d{1,12})\): ([\s\S]+)\n\n>(?: ([^\n]*))?$/.exec(text);
     if (steer === null)
         return null;
-    // A blank line's quote ("> ") reaches the store as ">": the chat POST trims the message (src/server/chat.ts).
     const [, quoted, line, removed, run, comment, code = ""] = steer;
     let path;
     try {
@@ -388,9 +335,6 @@ export const CHAT_UI_SCRIPT = `(function(){var workspace=document.querySelector(
     `var box=document.querySelector(".composer textarea");if(box){` +
     `function size(){box.style.height="auto";box.style.height=Math.min(box.scrollHeight,208)+"px";}size();box.addEventListener("input",size);` +
     `box.addEventListener("keydown",function(ev){if(ev.isComposing||ev.key!=="Enter"||ev.shiftKey||!window.matchMedia("(min-width: 761px)").matches)return;ev.preventDefault();if(box.value.trim()!=="")box.form.requestSubmit();});}})();`;
-/** Provider, model, and limits under ONE disclosure (UI polish
- * 2026-09-13): the facts stay one tap away on every chat surface, and a
- * membership never shows a dollar figure as if it were a charge. */
 export function chatLimitsHtml(facts) {
     const rows = [
         html `<span class="mono">${facts.provider} · ${facts.model}</span>`,
@@ -413,10 +357,6 @@ export function chatPage(chrome, data) {
         parts.push(html `<div class="problem">${data.problem}</div>`);
     if (!data.enabled.ok) {
         const code = data.enabled.code;
-        // The sandbox shows no conversation at all: chat evidence is a real
-        // subscription-backed plane, never a seeded transcript (v48 authority repair).
-        // With no lead yet, the first run says what happens next (it turns on with the signed-in agent, or shows the one
-        // command to run); otherwise one line, and its settings live in Settings → Lead (onboarding).
         const settingsLink = data.canManage ? html ` <a href="/settings/lead">Settings → Lead</a>` : "";
         if (!(code === "unconfigured" && data.firstRunShown === true))
             parts.push(code === "demo"
@@ -430,8 +370,6 @@ export function chatPage(chrome, data) {
     }
     const config = data.enabled.config;
     const subscription = isSubscriptionChatProvider(config.provider);
-    // The saved catch-up owns orientation. Keep the older overview only as
-    // a fallback; provider and spend terms remain available before consent.
     if (!data.canManage) {
         parts.push(html `<div class="card chat-readonly"><strong>Read-only view</strong><p class="meta">An approver can start the unified conversation and confirm its proposed actions. You can still open every live card and project board here.</p></div>`);
     }
@@ -470,7 +408,7 @@ export function chatPage(chrome, data) {
         ], "\n"), { attrs: { class: "card" } }));
     }
     if (data.canManage) {
-        parts.push(html `<p class="meta" id="chat-settings"><a href="/settings/lead">Lead settings</a> · provider, model and limits</p>`);
+        parts.push(html `<p class="meta" id="chat-settings"><a href="/settings/lead">Chat settings</a> · provider, model and limits</p>`);
     }
     if (data.recent.length > 0) {
         parts.push(html `<details class="chat-turn-history"><summary>Conversation activity <span class="meta">${data.recent.length} recent turns</span></summary>\n${joinHtml(data.recent.map(turn => html `<p class="row"><span class="mono">#${turn.id}</span> ${turn.state}${turn.failureReason === null ? "" : html ` · ${turn.failureReason}`} <span class="right meta">${turn.tokensIn ?? "–"} in / ${turn.tokensOut ?? "–"} out · ${subscription ? "membership" : `${chatMoney(turn.settledMicrousd ?? turn.reservedMicrousd)}${turn.settledMicrousd === null ? " reserved" : ""}`}</span></p>`), "\n")}\n</details>`);
@@ -478,8 +416,6 @@ export function chatPage(chrome, data) {
     return screen("chat", chatWorkspace(joinHtml(parts, "\n"), data.projects, true, data.focusTask, data.resultPanel ?? null), { chrome, functional: { script: CHAT_UI_SCRIPT + (data.focusTask === null ? "" : RESULT_REVIEW_SCRIPT), fetches: data.focusTask !== null } });
 }
 export function chatAckPage(chrome, turn, nonce, reply = null) {
-    // A lead reply's step reserves nothing itself: the reply set the money aside, and a reply whose cost is unknown has
-    // already counted that whole amount against its conversation. Say that amount, never the step's zero.
     const replyCharge = reply === null ? null : reply.settledMicrousd ?? reply.reservedMicrousd;
     return screen("chat", joinHtml([
         html `<h1>Unknown spend</h1>`,
@@ -496,7 +432,6 @@ export function chatAckPage(chrome, turn, nonce, reply = null) {
         html `</div>`,
     ], "\n"), { chrome });
 }
-/** The card that starts a conversation: the one password ceremony (mate arc §1). */
 export function leadMintCard(enabled, returnTo = "/chat") {
     const subscription = enabled.billing === "subscription";
     return html `<div class="card mate-mint" id="latest">\n<p><strong>Start a conversation</strong></p>\n${postForm("/chat/mate/mint", joinHtml([
@@ -509,7 +444,6 @@ export function leadMintCard(enabled, returnTo = "/chat") {
         html `<button type="submit">Start chat</button>`,
     ], "\n"), { returnTo })}\n</div>`;
 }
-/** The decisions the cards on a page name, for the answer card's consequences and the builder's recommendation. */
 export function decisionsFor(store, proposals) {
     const out = new Map();
     for (const one of proposals) {
@@ -521,33 +455,21 @@ export function decisionsFor(store, proposals) {
     }
     return out;
 }
-/**
- * A proposal card: what, then confirm/dismiss, or the door's answer. An
- * `answer` card shows the question, every option WITH its consequence,
- * the builder's recommendation beside the proposer's pick, and — for an
- * irreversible option — the explicit confirmation field the decision
- * page itself uses (ruling 12).
- */
 export function proposalCard(view, inert, decision, returnTo = null) {
     return proposalCardParts(view, inert, decision, returnTo).html;
 }
-/** The card's HTML and, for the chat's confirm-in-place cards, the same
- * card as data: the body is the HTML's own, the act is the same door. */
 export function proposalCardParts(view, inert, decision, returnTo = null) {
     const payload = view.payload;
     const text = (key) => (typeof payload[key] === "string" ? payload[key] : "");
     const task = text("task");
     const repoId = text("repoId");
     if (view.kind === "answer" && decision?.supersededReason != null) {
-        // The proposal remains in history, but its question no longer needs an answer.
-        // Both the server page and browser card use the same closed view, without stale consent controls.
         const body = html `<p><a href="/d/${decision.id}">${decision.question}</a></p><p class="meta">${closedQuestionReason(decision.supersededReason)}</p>`;
         return {
             html: html `<article class="card proposal proposal-answer" data-card-kind="answer"><header class="proposal-head"><strong>Question closed</strong></header><div class="proposal-body">${body}</div></article>`,
             card: { id: view.id, kind: view.kind, label: "Question closed", state: view.state, body: htmlString(body), said: null, links: [], primary: null, dismissable: false, note: null },
         };
     }
-    // A Toolroll card: data to read, with its fixed links. It confirms nothing, so it waits on nobody.
     if (isCardProposal(view)) {
         const shown = cardOfProposal(view);
         const body = shown === null ? html `<p>${cardPlainOf(view)}</p>` : webCardHtml(shown);
@@ -600,7 +522,6 @@ export function proposalCardParts(view, inert, decision, returnTo = null) {
     }
     else if (view.kind === "action") {
         const action = sharedActionPayload(payload);
-        // A Start card (one-step-chat.md C2) names what starts and where; its whole terms and the confirm are on the review page.
         what = action === null ? html `<p>This action is unavailable.</p>` : action.operation === "task_start" ? html `<h3>${action.title}</h3>${facts(["project", mono(action.repo.split(/[\\/]/).filter(Boolean).at(-1) ?? action.repo)])}` : (sharedActionNeedsReview(action) ? html `` : html `${action.terms.map(term => html `<p style="white-space:pre-wrap;overflow-wrap:anywhere">${term}</p>`)}`);
     }
     else if (view.kind === "task_action") {
@@ -610,8 +531,6 @@ export function proposalCardParts(view, inert, decision, returnTo = null) {
     else if (view.kind === "control") {
         const control = payload["control"];
         const label = isChatControl(control) ? CHAT_CONTROLS[control].label : "Control unavailable";
-        // Navigation does not wait for approval. Give it a destination and one
-        // link, not a proposal header that falsely reads as unfinished work.
         if (isChatControl(control) && view.state === "pending" && !inert) {
             const title = text("taskTitle");
             const href = chatControlHref(control, task, payload["run"], payload["project"]);
@@ -658,9 +577,6 @@ export function proposalCardParts(view, inert, decision, returnTo = null) {
         what = html `<h3>Guidance for <a href="${taskHref(task)}">${taskTitle}</a></h3><p class="proposal-summary">${text("note")}</p>${facts(["project", mono(repoId)])}<p class="meta proposal-disclosure">A running agent reads it when it next checks for notes; otherwise the next attempt does. The task’s scope doesn’t change.</p>`;
     }
     else if (view.kind === "agents") {
-        // The agents card (v48): exactly what changes, in the same words the
-        // task page uses — the role, the exact agent, the size — and the
-        // consequence for the standing approval.
         const taskTitle = text("taskTitle") || task;
         const role = text("role");
         const clear = payload["clear"] === true;
@@ -678,9 +594,7 @@ export function proposalCardParts(view, inert, decision, returnTo = null) {
         what = html `<h3>${heading}</h3>${text("why") === "" ? "" : html `<p class="proposal-summary">${text("why")}</p>`}${facts(["agents now", text("before")], ["size", size === "" ? "" : sizeConsequence(size, payload["risky"] === true)], ["role", role === "" ? "" : html `${role} → ${clear ? "the recommendation" : mono(agentWords)}`], ["project", mono(repoId)])}<p class="meta proposal-disclosure">Recorded under your name when you confirm. ${text("approval") === "approved" ? "The current approval no longer covers the task afterwards — approve it again on the task." : "The next approval seals these agents."}</p>`;
     }
     else if (view.kind === "scope") {
-        what = html `<h3>Rewrite <a href="${taskHref(task)}">${task}</a></h3><p class="proposal-summary">${text("goal")}</p>${
-        // One path per line (UI polish 2026-09-13): a comma run broke mid-token on a phone.
-        facts(["out of scope", text("not")], ["may touch", monoLines(payload["touches"])], ["project", mono(repoId)])}`;
+        what = html `<h3>Rewrite <a href="${taskHref(task)}">${task}</a></h3><p class="proposal-summary">${text("goal")}</p>${facts(["out of scope", text("not")], ["may touch", monoLines(payload["touches"])], ["project", mono(repoId)])}`;
     }
     else if (view.kind === "repair") {
         const blocker = text("blocker");
@@ -739,10 +653,7 @@ export function proposalCardParts(view, inert, decision, returnTo = null) {
         const filed = outcome !== null && typeof outcome.taskId === "string" ? outcome.taskId : null;
         acts = html `<p class="done">${said ?? "confirmed"}${(view.kind === "scope" || view.kind === "agents" || (view.kind === "review" && text("operation") === "revise")) && filed !== null
             ? html ` — <a href="${taskChatHref(filed)}#task-chat-action">review & start in chat</a>`
-            : ""}${flowHref === null ? "" : html ` — <a href="${flowHref}">open the flow</a>`}</p>${
-        // The created task, by its recorded id (package 2): one clear road
-        // into its lens — the same conversation, focused — and the overview.
-        filed !== null && view.kind === "task"
+            : ""}${flowHref === null ? "" : html ` — <a href="${flowHref}">open the flow</a>`}</p>${filed !== null && view.kind === "task"
             ? html `<p class="proposal-filed"><a class="button-link" href="${taskChatHref(filed)}" data-filed-task="${filed}">Open task <span class="mono">${filed}</span> →</a> <a href="${taskHref(filed)}">overview</a></p>`
             : ""}`;
     }
@@ -754,7 +665,6 @@ export function proposalCardParts(view, inert, decision, returnTo = null) {
     }
     const stateClass = view.state === "confirmed" ? "badge-done" : view.state === "refused" ? "badge-failed" : "";
     const cardHtml = html `<article class="card proposal proposal-${view.kind} ${view.state}" data-card-kind="${view.kind}"><header class="proposal-head"><span class="proposal-icon">${strokeIcon(presentation.icon)}</span><span><strong>${presentation.label}</strong><small>proposed by ${provenance}</small></span><span class="badge ${stateClass}">${sentenceCase(view.state)}</span></header><div class="proposal-body">${what}</div><footer class="proposal-actions">${acts}</footer></article>`;
-    // The same card as data (chat cards): one act, in the door's own words.
     const filedTask = outcome !== null && typeof outcome.taskId === "string" ? outcome.taskId : null;
     const reviewAction = view.kind === "action" ? sharedActionPayload(payload) : null;
     const sentence = (words) => words.charAt(0).toUpperCase() + words.slice(1);
@@ -779,7 +689,6 @@ export function proposalCardParts(view, inert, decision, returnTo = null) {
     };
     return { html: cardHtml, card };
 }
-/** The review link's words: a Start card's is "Review and start". */
 function reviewWords(action) {
     return action?.operation === "task_start" ? "Review and start" : "Review action";
 }
@@ -789,8 +698,6 @@ export function leadProposalCard(proposal, inert, decision, returnTo = null) {
 export function leadProposalCardParts(proposal, inert, decision, returnTo = null) {
     return proposalCardParts({ id: proposal.id, kind: proposal.kind, payload: proposal.payload, state: proposal.state, outcome: proposal.outcome, by: { mate: true }, actionBase: "/chat/proposal" }, inert, decision, returnTo);
 }
-/** Review and inline team cards share the same server-decided controls and reasons. The team API renders these
- * outside a console request, so the session's own `csrf` is given to the forms here. */
 export function teamProposalCardParts(store, actor, proposal, snapshot, csrf, decision, now, provider) {
     const gate = proposalActGate(store, actor, proposal.thread, now, provider);
     const reason = !gate.ok ? gate.said : null;
@@ -805,21 +712,18 @@ export function teamProposalCardParts(store, actor, proposal, snapshot, csrf, de
 export function coordinatorProposalCard(proposal, now, decision, returnTo = null) {
     return proposalCard({ id: proposal.id, kind: proposal.kind, payload: proposal.payload, state: proposal.state, outcome: proposal.outcome, by: { mate: false, name: proposal.name, ago: relativeAge(proposal.createdAt, now) }, actionBase: "/proposals" }, false, decision, returnTo);
 }
-/** The section shared by /chat (both modes) and the task page: pending coordinator proposals as cards. */
 export function coordinatorProposalsSection(proposals, decisions, now, heading = true, returnTo = null) {
     if (proposals.length === 0)
         return html ``;
     return html `${heading ? html `<h2>Proposed by coordinators <span class="meta">${proposals.length}</span></h2>` : ""}<div class="coordinator-proposals">${proposals.map(one => coordinatorProposalCard(one, now, decisions.get(typeof one.payload["decision"] === "number" ? one.payload["decision"] : -1) ?? null, returnTo))}</div>`;
 }
 export const REPLACED_THREAD_DIVIDER = "New conversation — the projects I can reach changed";
-/** The replaced thread as the React conversation renders it: words and card outcomes, no buttons. */
 export function replacedBrowserMessages(previous) {
     return leadBrowserMessages({ ...previous, pending: null, ask: null, asks: new Map() }, null).map(message => ({
         ...message, cardsHtml: "",
         cards: (message.cards ?? []).map(card => ({ ...card, primary: null, dismissable: false })),
     }));
 }
-/** The replaced thread for the server-rendered page: read only, then the divider. */
 export function replacedThreadHtml(previous) {
     if (previous == null)
         return html ``;
@@ -827,8 +731,6 @@ export function replacedThreadHtml(previous) {
         ? html `<div class="msg op" data-previous-message="${one.id}"><p style="white-space:pre-wrap">${one.text}</p></div>`
         : html `<div class="msg mate" data-previous-message="${one.id}">${renderChatText(one.text)}</div>`)}<p class="chat-previous-divider" role="separator" data-thread-divider>${REPLACED_THREAD_DIVIDER}</p></section>`;
 }
-/** A queued message's state under its words: waiting (with Remove), or why it was not sent. Nothing for one that ran.
- * A div, not a p: the Remove form can't sit inside a paragraph (the parser would close it first). */
 function queuedStateHtml(row, back) {
     if (row.status === "queued") {
         const waiting = row.error?.startsWith("Waiting: ") === true ? row.error.slice("Waiting: ".length) : "Waiting for your earlier messages…";
@@ -842,8 +744,6 @@ function queuedStateHtml(row, back) {
         return html `<div class="so-queued" data-queued="${row.message}" data-queue-state="uncertain"><span class="so-queued-state">Not confirmed</span> · ${row.error ?? "Delivery could not be confirmed. Nothing was sent again."}</div>`;
     return html ``;
 }
-/** The lead's question as buttons: each option sends itself as the next message (the same POST as a typed one);
- * "Something else" moves to the composer. Nothing shows while a reply is running. */
 export function leadAskHtml(ask, target, composer, open = true, stale = false) {
     if (stale)
         return html `<div class="so-owner-ask" data-ask="${ask.turn}" data-ask-stale><p class="so-owner-ask-question"><strong>${ask.question}</strong></p><p class="so-owner-ask-stale">Out of date: the task changed after this was asked. Its status above says what it needs now.</p></div>`;
@@ -852,16 +752,12 @@ export function leadAskHtml(ask, target, composer, open = true, stale = false) {
     const hidden = target.task !== null ? { task: target.task } : target.project !== null ? { project: target.project } : {};
     return html `<div class="so-owner-ask" data-ask="${ask.turn}"><p class="so-owner-ask-question"><strong>${ask.question}</strong></p><div class="so-owner-ask-options" role="group" aria-label="Answer options">${ask.options.map(option => postForm("/chat", html `<button type="submit" name="message" value="${option}" class="so-suggestion quiet">${option}</button>`, { attrs: { class: "inline" }, hidden }))}<label for="${composer}" class="so-suggestion so-owner-ask-other">${LEAD_ASK_OTHER}</label></div></div>`;
 }
-/** The thread's messages as the React conversation renders them, each
- * with the proposal cards its turn produced; `back` is where a card's
- * confirm or dismiss returns. */
 export function leadBrowserMessages(rows, back, target = { task: null, project: null }) {
     const open = rows.pending === null ? rows.ask?.turn ?? null : null;
     const asked = (turn) => {
         const ask = turn === null ? undefined : rows.asks?.get(turn);
         return ask === undefined ? "" : leadAskHtml(ask, target, target.task === null ? "lead-message" : "task-message", ask.turn === open, ask.turn === rows.staleAsk);
     };
-    // The owner's message each reply answers: when it asked for ids, the reply keeps them.
     const askedBy = new Map();
     rows.messages.reduce((last, message) => { if (message.role === 'operator')
         return message.text; if (last !== undefined)
@@ -873,19 +769,12 @@ export function leadBrowserMessages(rows, back, target = { task: null, project:
             : html `${renderChatText(message.text, askedBy.get(message.id))}${asked(message.turn)}`),
         activity: message.activity, createdAt: message.createdAt,
         ...(() => {
-            // Under the lead's message only, as the server-rendered thread does: the person's message shares the turn.
             const parts = message.turn === null || message.role === 'operator' ? [] : cardsFirst(rows.proposals.filter(one => one.turn === message.turn))
                 .map(one => leadProposalCardParts(one, rows.pending !== null, rows.decisions.get(typeof one.payload['decision'] === 'number' ? one.payload['decision'] : -1) ?? null, back));
             return { cardsHtml: htmlString(joinHtml(parts.map(one => one.html))), cards: parts.map(one => one.card) };
         })(),
     }));
 }
-/** The version of the DISPLAYED conversation (package 2): every fact a
- * thread or task fragment renders — message identities, card states and
- * outcomes, the live turn and its step count, the last turn's state, the
- * decisions the cards name, and the task lens's own facts — and nothing
- * that merely ticks: no relative age, no freshly minted nonce, no csrf.
- * Equal versions mean equal fragments; the poll fetches nothing else. */
 export function leadChatVersion(rows) {
     const focus = rows.focusTask;
     const facts = [
@@ -916,12 +805,6 @@ export function leadChatVersion(rows) {
     ];
     return createHash("sha256").update(JSON.stringify(facts)).digest("hex").slice(0, 16);
 }
-/** The thread region — the one safe region the live refresh reconciles
- * (package 2): coordinator cards, the operator/assistant messages with
- * their cards, the reply-in-progress card, and the starters that follow a
- * populated thread. Every child carries a stable `data-key` so the page
- * keeps an unchanged node (its open disclosures, its focus) and replaces
- * only what the server rendered differently. The composer is never here. */
 export function leadThreadHtml(data) {
     const returnTo = data.focusTask === null ? "/chat" : taskChatHref(data.focusTask.id);
     const parts = [];
@@ -946,7 +829,6 @@ export function leadThreadHtml(data) {
     for (const message of data.messages) {
         if (message.role === "operator") {
             asked = message.text;
-            // A message that waits its turn (the lead queue) says so, with Remove while it still waits — as in the React thread.
             const queued = data.queue?.get(message.id);
             thread.push(html `<div class="msg op" data-message-role="operator" data-key="m${message.id}">${message.text === "" ? "" : steerCommentHtml(message.text) ?? html `<p style="white-space:pre-wrap">${message.text}</p>`}${queued === undefined ? "" : queuedStateHtml(queued, returnTo)}</div>`);
             continue;
@@ -965,17 +847,11 @@ export function leadThreadHtml(data) {
             stopForm: postForm("/chat/mate/stop", html `<button type="submit" class="quiet" aria-label="Stop chat response">Stop</button>`, { attrs: { class: "inline" }, returnTo, hidden: { turn: data.pending.id } }),
         }));
     }
-    // Suggestions help start a conversation; repeating them after every
-    // response competes with the actual result and its available actions.
     return html `<div id="chat-thread" data-chat-region="thread">${joinHtml(parts, "\n")}</div>`;
 }
-/** What follows the composer (package 2): the starters of an EMPTY thread
- * sit under the box; once the first message lands they move above it. */
 export function leadAfterComposerHtml(data) {
     return html `<div id="chat-after-composer" data-chat-region="after">${data.messages.length === 0 && data.pending === null ? leadPromptStarters(data.focusTask) : ""}</div>`;
 }
-/** A conversation shown before it exists (quiet page visits): its page binds sends to this session id, and the first
- * send starts the conversation (chatSend). A visit never starts one. */
 export const NOT_STARTED_SESSION = 0;
 export function leadPage(chrome, data) {
     const subscription = isSubscriptionChatProvider(data.config.provider);
@@ -997,7 +873,7 @@ export function leadPage(chrome, data) {
                 ? ""
                 : html `<p class="meta">Recent turns: ${joinHtml(data.recent
                     .map(turn => html `<span class="mono">#${turn.id}</span> ${turn.state}${turn.failureReason === null ? "" : html ` · ${turn.failureReason}`} · ${subscription ? "membership" : chatMoney(turn.settledMicrousd ?? turn.reservedMicrousd)}`), " · ")}</p>`,
-            html `<p class="meta"><a href="/settings/lead">Lead settings</a> · provider, model and limits</p>`,
+            html `<p class="meta"><a href="/settings/lead">Chat settings</a> · provider, model and limits</p>`,
         ], "\n")}\n</details>`,
     ], "\n");
     const conversation = [
@@ -1005,7 +881,6 @@ export function leadPage(chrome, data) {
             ? chatHeading("", data.projects.length)
             : taskChatHeading(data.focusTask),
         data.focusTask === null ? (data.catchUp ?? "") : taskChatLiveRegion(data.focusTask, data.csrf, false, data.pending !== null),
-        // The DB catch-up replaces the older, duplicate portfolio summary.
         data.focusTask === null && !data.catchUp ? html `<details class="chat-fleet-context">${chatOverviewSummaryHtml(data.projects, data.fleetSnapshot?.attentionCount)}${chatFleetOverview(data.fleetSnapshot, data.projects, data.pending === null)}</details>` : "",
     ];
     if (data.problem !== null)
@@ -1017,24 +892,13 @@ export function leadPage(chrome, data) {
     const latestReply = data.pending === null && lastMessage?.role === "assistant" ? lastMessage.id : null;
     conversation.push(replacedThreadHtml(data.previous));
     conversation.push(leadThreadHtml({ ...data, problem: null, chatProject }));
-    conversation.push(
-    // The New update action (package 2): hidden until a live update lands
-    // while the reader is above the latest message; a real button, so the
-    // keyboard reaches it. Only this act moves the reader.
-    html `<div class="chat-new-update-holder"><button type="button" class="chat-new-update" id="chat-new-update" hidden>New update ↓</button></div>`, postForm("/chat", joinHtml([
+    conversation.push(html `<div class="chat-new-update-holder"><button type="button" class="chat-new-update" id="chat-new-update" hidden>New update ↓</button></div>`, postForm("/chat", joinHtml([
         html `<label>Message<textarea id="chat-message" name="message" rows="1" maxlength="${LEAD_MESSAGE_MAX_CHARS}" placeholder="${data.focusTask !== null ? "Ask about this task…" : chatProject !== null ? `Ask about ${projectName(chatProject)}…` : "Describe what you want done…"}"></textarea></label>`,
         html `<button type="submit" aria-label="${data.pending === null ? "send message" : "send message — it waits for the current reply"}">Send</button>`,
     ], "\n"), {
         attrs: { class: "card composer", id: latestReply === null && data.pending === null ? "latest" : "chat-composer", "aria-label": "message the lead", "data-chat-session": sessionId, "data-chat-task": data.focusTask?.id ?? "", "data-chat-user": approver, "data-chat-busy": data.pending === null ? "0" : "1", "data-chat-version": leadChatVersion(data), "data-chat-approval": data.focusTask?.approval?.digest ?? "" },
         hidden: { request: randomBytes(16).toString("hex"), "request-session": sessionId, task: data.focusTask?.id, project: chatProject, result: data.resultRunId },
-    }), 
-    // Concise pass (2026-09-13): the status line speaks only when there is
-    // a state to report — a reply in progress here, the connection from
-    // the continuity script — and the composer carries no second intro.
-    html `<p class="meta composer-hint" id="chat-connection" role="status" aria-live="polite">${data.pending === null ? "" : "Reply in progress. A message you send now waits its turn."}</p>`, 
-    // The explicit reconnection (package 2): shown only once the session
-    // or sign-in changed under this page; it reloads on the reader's act.
-    html `<p class="meta composer-hint" id="chat-reconnect" hidden><button type="button" class="quiet">Reconnect</button></p>`, leadAfterComposerHtml(data), controlsHtml);
+    }), html `<p class="meta composer-hint" id="chat-connection" role="status" aria-live="polite">${data.pending === null ? "" : "Reply in progress. A message you send now waits its turn."}</p>`, html `<p class="meta composer-hint" id="chat-reconnect" hidden><button type="button" class="quiet">Reconnect</button></p>`, leadAfterComposerHtml(data), controlsHtml);
     return screen("chat", chatWorkspace(joinHtml(conversation, "\n"), data.projects, false, data.focusTask, data.resultPanel ?? null), { chrome, functional: { script: CHAT_CONTINUITY_SCRIPT + CHAT_UI_SCRIPT + (data.focusTask === null ? "" : RESULT_REVIEW_SCRIPT), fetches: true },
         workspace: {
             conversation: {

package/dist/server/render-code-map.js

@@ -1,7 +1,3 @@
-/**
- * The Code map page (docs/plans/code-map.md §6): read-only, from the shared reader. Every name on it came from the
- * repository: `html` escapes it, links carry it through encodeURIComponent, and the handler looks it up exactly.
- */
 import { githubRepoOf } from "../pull-request-flow.js";
 import { areaRows, areaView, fileView, searchMap } from "../code-map-read.js";
 import { NOT_ENOUGH, reportRows } from "../code-map-report.js";
@@ -96,7 +92,6 @@ function provenance(data, map) {
         ...(report.enforced === undefined ? [] : [note(LIMITS_WORDS[report.enforced])]),
     ]));
 }
-/** "Builds with and without the map": read by people; nothing switches on it. */
 export function codeMapReportBlock(report) {
     if (report.arms.map.families + report.arms.holdout.families === 0)
         return null;

package/dist/server/render-flaky.js

@@ -1,21 +1,14 @@
-/**
- * The flaky tests list (docs/plans/learning-loops.md, "The list"): a project's tests with mixed outcomes under recorded
- * conditions, on the kit. Reads only; the one action is "Make a task to fix it", the ordinary /tasks/add form, and a
- * person approves that task as usual. Test names are candidate-written text, escaped at the sink like everything else.
- */
 import { flakyCoverageWords, flakyStatusWords } from "../flaky-cli.js";
 import { fixTaskOf } from "../flaky.js";
 import { html, joinHtml, postForm } from "../html.js";
 import { relativeAge, screen } from "./chrome.js";
 import { btn, chip, disclose, doc, docHeader, empty, facts, note, phoneHeader, row } from "./kit.js";
 const KIND_WORDS = { regate: "Checks run again on the same commit", "same-tree": "Two checks of the same code", "in-check-retry": "Passed on a retry inside one check", github: "GitHub re-run" };
-/** "Make a task to fix it": the ordinary task form, filled in. */
 function fixForm(repo, data, returnTo) {
     const task = fixTaskOf(data);
     return postForm("/tasks/add", btn("Make a task to fix it", { sm: true, type: "submit" }), { returnTo, hidden: { title: task.title, goal: task.goal, repo } });
 }
 const statusChip = (data) => chip(flakyStatusWords(data), "", { dot: false });
-/** The list. */
 export function flakyPage(chrome, data) {
     const { list } = data;
     const rowHref = (key) => `${data.selfHref}${data.selfHref.includes("?") ? "&" : "?"}row=${encodeURIComponent(key)}`;
@@ -38,7 +31,6 @@ export function flakyPage(chrome, data) {
         ]))),
     ], "\n"), { chrome });
 }
-/** One row's history. */
 export function flakyRowPage(chrome, data) {
     const one = data.row;
     const history = data.history.length === 0 ? note("No history recorded.") : joinHtml(data.history.map(entry => row({

package/dist/server/render-follow-through.js

@@ -1,10 +1,7 @@
-/** The task page's follow-through sections (docs/plans/fleet-follow-through.md): findings, failed checks, before and after,
- * and the handover note. All of a finding's text is model output or repository text: it renders as text, in `html`'s
- * escaping, and no action here is reachable from it. Every form posts to the one action in tasks.ts. */
 import { checkRepairsOfTask, checkRepairWords } from "../check-repair.js";
 import { evidencePairsOf, evidenceWords, failingFirstOf, failingFirstWords } from "../evidence-pair.js";
 import { findingLine, findingFromWords, findingsOfTask, findingStandingWords } from "../findings.js";
-import { handoverLines, handoverNoteOf } from "../handover-note.js";
+import { handoverHasContent, handoverLines, handoverNoteOf } from "../handover-note.js";
 import { html, joinHtml, postForm } from "../html.js";
 import { providerCaptures } from "../provider-capture.js";
 import { btn, disclose, field, input, note, textarea } from "./kit.js";
@@ -63,6 +60,6 @@ export function followThroughView(store, root, taskId, taskHref, canAct, now) {
             canAct ? act(taskHref, step === null ? "failing-first-on" : "failing-first-off", step === null ? "Ask for a failing test first" : "Turn off failing test first") : "",
             captures.length === 0 ? "" : disclose("What the provider said", html `<ul class="result-changes">${captures.map(one => captureRow(one, taskHref, canAct))}</ul>`),
         ]),
-        handover: note1 === null ? "" : html `<pre class="mono handover-note">${handoverLines(note1).join("\n")}</pre>`,
+        handover: note1 === null || !handoverHasContent(note1) ? "" : disclose("Show the note", html `<pre class="mono handover-note">${handoverLines(note1).join("\n")}</pre>`),
     };
 }