toolroll 0.9.103 → 0.9.105

package/dist/server/render-history.js

@@ -7,24 +7,16 @@ import { taskHref } from "./http.js";
 import { btn, chip, disclose, doc, docHeader, dock, facts, note, notice, phoneHeader, radios, slots, status } from "./kit.js";
 import { editorFileHref, evidenceBundleCard, evidenceWords, humanBytes, prefillScript, resultPanelHtml, revisionSealFields, runFactsRows, runNoun, terminalDiffCard, } from "./render-results.js";
 import { publicationFactsOf } from "./render-tasks.js";
-/** A section of a reading page: its heading and body (kit `k-sec`). */
 export const sec = (title, body, attrs = html ``) => html `<section class="k-sec"${attrs}><h3>${title}</h3>${body}</section>`;
-/** The root every kit server page carries, so the workspace frames it as a kit page (no padding, panel or desk header). */
 const kitPage = (attrs, ...parts) => html `<div class="h-page" data-kit-page${attrs}>${parts}</div>`;
-// ---- /r/<id>: one attempt ----------------------------------------------------------------------------------------
 export function runPage(chrome, run, taskId, artifacts, terminal = null, notes = [], csrf = "", comments = [], ciRepair = null, liveScript, peekable = false, running = false, editor = null, editorToggle = null, noted = false, structuredHandoff = null, proofBundle = null, route = null, publicationRow = null, review = null, result = null, sourceDigest = null) {
     const title = `${sentenceCase(runNoun(run))} #${run.id}`;
-    // The facts region: a live run's is what the poller swaps; a finished run's stands still.
     const summary = sec("Summary", html `<div id="run-facts">${runFactsRows(run, taskId, running, route)}</div>${running ? html `<p class="k-note" id="run-facts-stamp"></p>` : ""}`);
-    // The live transcript (arc 1): the agent's own words, streamed to a file beside the run and polled as raw text.
-    // Honesty stated on the surface: display only, and the machine running the agent could alter it.
     const transcript = !running ? "" : sec("What the agent is saying", !peekable
         ? note(html `The live transcript is off. Start serve with <span class="k-mono">--runner &lt;name&gt;</span> naming this machine's worker, and it appears here.`)
         : run.provider !== "claude"
             ? note(`The live transcript needs the claude harness for now; this build runs on ${run.provider}.`)
             : html `<pre id="live-transcript" class="k-mono h-pre h-live"></pre><p class="k-note" id="live-transcript-state"></p>`);
-    // The live peek region (A2): the poller fills it only on a serve that asserted its runner; otherwise the section
-    // says why it is empty, so the click never lands on silence.
     const peek = !running ? "" : sec("What is changing right now", peekable
         ? html `<div id="run-peek">${note("Watching… the first look lands within 15 seconds.")}</div><p class="k-note" id="run-peek-stamp"></p>`
         : note(html `The live file view is off. Start serve with <span class="k-mono">--runner &lt;name&gt;</span> naming this machine's worker, and it appears here.`));
@@ -33,8 +25,6 @@ export function runPage(chrome, run, taskId, artifacts, terminal = null, notes =
     const handoff = resultSummary === null ? "" : sec("Result", html `<p class="h-read">${resultSummary}</p>${structuredHandoff === null ? "" : [
         list("Completed", structuredHandoff.changes), list("Checks the agent reported", structuredHandoff.verification), list("Follow-up", structuredHandoff.followUps)
     ]}`);
-    // Review comments on the immutable terminal diff (M6.8): listed, added, and sealed into ONE unapproved revision task.
-    // The seal only creates the unapproved task; its approval screen restates the batch.
     const hasTerminalDiff = terminal !== null && terminal.patch !== null && !("problem" in terminal.patch);
     const commentPath = (path, line) => {
         const shown = `${path}${line === null ? "" : `:${line}`}`;
@@ -47,8 +37,6 @@ export function runPage(chrome, run, taskId, artifacts, terminal = null, notes =
 <div class="k-field"><label class="k-label" for="comment-line">Line</label><input class="k-input" id="comment-line" type="text" name="line" placeholder="—" inputmode="numeric"></div></div>\
 <div class="k-field"><label class="k-label" for="comment-note">Change requested</label><textarea class="k-textarea" id="comment-note" name="note" rows="2" maxlength="${LIMITS.note}" placeholder="What should change, and why" aria-describedby="comment-note-limit"${noted ? html ` autofocus` : ""}></textarea>\
 <p class="k-hint" id="comment-note-limit">Up to ${LIMITS.note} characters</p></div><div class="k-acts">${btn("Add annotation", { sm: true, type: "submit", attrs: { "data-k-busy": "Adding…" } })}</div>`, { attrs: { class: "h-form", id: "comment-form" } });
-    // The device-side half of the editor-link activation (arc 6, finding 1): only when the capability exists and this
-    // run belongs to this machine's runner; the person at the browser flips it.
     const editorToggleForm = editorToggle === null || csrf === "" ? "" : postForm("/session/editor-links", html `${btn(editorToggle.on ? "Stop opening files in VS Code here" : "Open files in VS Code from this device", { variant: "ghost", sm: true, type: "submit" })}<p class="k-note">Only useful when this browser runs on the machine that holds the worktrees.</p>`, { attrs: { class: "h-inline" }, returnTo: `/r/${run.id}`, hidden: { on: editorToggle.on ? "0" : "1" } });
     const reviseForm = csrf === "" || comments.length === 0 ? "" : postForm(`/r/${run.id}/revise`, html `${revisionSealFields(comments, sourceDigest)}${slots({
         status: html `<h2>${comments.length} ${comments.length === 1 ? "annotation" : "annotations"} ready</h2>`,
@@ -56,8 +44,6 @@ export function runPage(chrome, run, taskId, artifacts, terminal = null, notes =
         acts: btn("Create revision from annotations", { variant: "primary", type: "submit", attrs: { "data-k-busy": "Creating…" } }),
         terms: "Nothing builds until its scope is approved.",
     })}`, { attrs: { class: "revision-from-comments" } });
-    // CI repair, suggestion-first (M8.18): the observed red episode earns a button; the button drafts ONE unapproved
-    // task; a person approves it.
     const repair = ciRepair === null || csrf === "" ? "" : sec(`Checks are failing on pull request #${ciRepair.pr}`, html `${note("Seen by the pull request watcher. Drafting a repair makes one unapproved task; you approve its scope before anything builds.")}${postForm(`/r/${run.id}/draft-repair`, btn("Draft a repair task", { sm: true, type: "submit", attrs: { "data-k-busy": "Drafting…" } }), { attrs: { class: "h-inline" } })}`);
     const reviewSec = commentRows.length === 0 && commentForm === null ? "" : sec("Review and revise", html `${note("Annotate the diff above. When the batch is ready, create one scoped revision task from it.")}${commentRows.length === 0 ? "" : html `<ul class="h-notes diff-comments">${commentRows}</ul>`}${commentForm ?? ""}${reviseForm}${editorToggleForm}`);
     const saved = artifacts.length === 0 ? "" : sec("Saved files", html `<ul class="h-files">${artifacts.map(artifact => html `<li><a class="k-link" href="/r/${run.id}/evidence/${artifact.id}">${evidenceWords(artifact.kind)}</a>\
@@ -65,8 +51,6 @@ export function runPage(chrome, run, taskId, artifacts, terminal = null, notes =
     const noteRows = notes.map(one => html `<li><span class="k-num k-muted">${whenTime(one.createdAt)}</span><span><b>${one.author}</b> ${one.note}</span></li>`);
     const noteForm = csrf === "" ? "" : postForm(`/r/${run.id}/note`, html `<label class="k-sr" for="run-note">Add a note</label><input class="k-input" id="run-note" type="text" name="note" placeholder="A note for whoever reads this attempt next" autocomplete="off">${btn("Add note", { sm: true, type: "submit", attrs: { "data-k-busy": "Adding…" } })}`, { attrs: { class: "h-addnote" } });
     const notesSec = notes.length === 0 && noteForm === "" ? "" : sec("Notes", html `${noteRows.length === 0 ? note("No notes yet. A note stays with this attempt for whoever reads it next.") : html `<ul class="h-notes">${noteRows}</ul>`}${noteForm}`);
-    // A finished result (package 3) is the ONE shared panel (Summary / Changes / Checks, Request changes beside it), the
-    // same markup the review cockpit and the chat's result view render. Every other run keeps its record by record page.
     const resultPanel = result === null ? null : resultPanelHtml(result.detail, {
         place: "run", tab: result.tab, csrf, user: result.user, noted, requestToken: result.requestToken,
         hrefFor: one => (one === "summary" ? `/r/${run.id}` : `/r/${run.id}?tab=${one}`), returnTo: `/r/${run.id}?record=1`, back: null,
@@ -76,21 +60,12 @@ export function runPage(chrome, run, taskId, artifacts, terminal = null, notes =
         : [html `<div class="h-panel">${resultPanel}</div>`, editorToggleForm]), saved, notesSec));
     return screen(title, body, {
         chrome,
-        // One composed functional script (arc 4 contract): the pollers when the run is live, the result panel's script or
-        // the comment prefill when a form exists. Neither fetches beyond the pollers.
         ...(liveScript === undefined && commentForm === null && resultPanel === null
             ? {}
             : { functional: { script: (liveScript ?? "") + (resultPanel !== null ? RESULT_REVIEW_SCRIPT : commentForm === null ? "" : prefillScript()), fetches: liveScript !== undefined } }),
     });
 }
-/**
- * The question an agent asked, on top: what it asked and why, the options as radio cards with its recommendation
- * marked, an optional note, and one Answer (kind none: one click). An irreversible option asks for a tick first; the
- * answer route checks it again. A refusal shows in the box with nothing changed; an answer replaces the box.
- */
-export function decisionPage(chrome, decision, taskId, artifacts, now, place, returnTo = null, refusal = null, 
-/** What a refused answer carried: the choice and the note stay in the box. */
-draft = null) {
+export function decisionPage(chrome, decision, taskId, artifacts, now, place, returnTo = null, refusal = null, draft = null) {
     const back = returnTo === null ? ["/", "Home"] : [returnTo, "Back to the chat"];
     const runWords = place.run === null ? null : `${runNoun(place.run)} #${place.run.id}`;
     const project = place.repo === null ? null : projectName(place.repo);
@@ -135,14 +110,9 @@ draft = null) {
     const looked = artifacts.length === 0 ? "" : sec("What the agent looked at", html `<ul class="h-files">${artifacts.map(artifact => html `<li><a class="k-link" href="/d/${decision.id}/evidence/${artifact.id}">${evidenceWords(artifact.kind)}</a><span class="k-num k-muted">${humanBytes(artifact.bytesStored)}${artifact.truncated ? " · partial" : ""}</span></li>`)}</ul>`);
     const stopped = !open || place.run === null ? "" : sec("Where it stopped", html `<p class="h-read k-muted">${sentenceCase(runWords ?? "")} asked this and holds its work until you answer. <a class="k-link" href="/r/${place.run.id}?record=1">See the attempt</a></p>`);
     const answering = open && decision.supersededReason === null;
-    const body = kitPage(html ``, phoneHeader(project ?? "Question", { back: back[0] }), 
-    // The shell's phone header already says this page's title (the screen title), so the h1 is drawn once.
-    doc(docHeader({ title: place.title ?? taskId, back, meta, same: true }), box, looked, stopped), 
-    // On a phone the one primary lives in the dock; it submits the same form.
-    answering ? dock(btn("Answer", { variant: "primary", type: "submit", attrs: { form: "answer-form", "data-k-busy": "Answering…" } })) : "");
+    const body = kitPage(html ``, phoneHeader(project ?? "Question", { back: back[0] }), doc(docHeader({ title: place.title ?? taskId, back, meta, same: true }), box, looked, stopped), answering ? dock(btn("Answer", { variant: "primary", type: "submit", attrs: { form: "answer-form", "data-k-busy": "Answering…" } })) : "");
     return screen(place.title ?? `Question · ${taskId}`, body, { chrome });
 }
-// ---- /t/<id>/evidence: the full record ----------------------------------------------------------------------------
 const at = (iso) => iso === null ? "—" : whenTime(iso);
 const shortHash = (hash) => hash === null ? "—" : html `<span class="k-mono" title="${hash}">${hash.slice(0, 12)}</span>`;
 const usd = (value) => value === null ? "—" : `$${value.toFixed(value < 1 && value > 0 ? 4 : 2)}`;
@@ -156,8 +126,6 @@ function stateChip(state) {
     return state === "done" ? status("finished") : state === "failed" ? status("failed") : state === "running" ? status("running")
         : state === "queued" ? status("queued") : chip(sentenceCase(state));
 }
-/** A task's full record, read-only: what was asked and approved, every agent that ran, the result, and its sealed audit
- * log entries, with the JSON download and Print. The same facts as the pack's JSON (evidence-pack.ts). */
 export function evidenceRecordPage(chrome, pack) {
     const task = pack.task;
     const href = `/t/${encodeURIComponent(task.id)}`;

package/dist/server/render-new-task.js

@@ -1,9 +1,3 @@
-/**
- * New task (Wave E, src/design/KIT.md): one describe box, the project it lands in and "Read code before planning" in
- * the box's bottom row, the round send; every other choice behind More options; ideas under it. The form posts the
- * same fields /tasks/add has always read (title, repo, plan-first with planning-policy=choice, quality-mode, after,
- * goal, acceptance, not, touches, scout, id, permission-mode, projectRevision), so every server check is unchanged.
- */
 import { html, joinHtml, postForm } from "../html.js";
 import { projectName } from "../project.js";
 import { screen } from "./chrome.js";
@@ -13,8 +7,6 @@ import { TASK_TEXT_LIMITS } from "../task-text.js";
 const FORM_ID = "nt-form";
 const PLAN_ON = "The agent reads the code and asks you anything unclear, then writes the plan.";
 const PLAN_OFF = "The agent writes the plan straight from your words. Faster for small, obvious changes.";
-/** The projects as the select offers them: names, with the parent folder when two share a name; the open one first
- * picked. Values are the paths the server proves. */
 function projectItems(projects, open) {
     const all = open !== null && !projects.some(one => one.path === open) ? [{ path: open, name: projectName(open) }, ...projects] : [...projects];
     const counts = new Map();
@@ -27,22 +19,11 @@ function projectItems(projects, open) {
         return [one.path, `${one.name} (${parent})`];
     });
 }
-export function newTaskPage(chrome, project, 
-/** undefined: keep the revision the refused form posted (its draft), so a project switch since still refuses. */
-projectRevision, problem, candidates = [], permissionDefault = "auto", qualityDefault = "default", projects = [], 
-/** A shipped template's fields, or a refused filing's own words, back in the form. */
-prefill = null, 
-/** Words an idea's link put in the box (GET ?text=). */
-draft = null, 
-/** Ideas for the chosen project: one tap drafts the words into the box. */
-suggestions = [], 
-/** A link the lead or a person put in the address (GET ?from=). */
-fromLink = null) {
+export function newTaskPage(chrome, project, projectRevision, problem, candidates = [], permissionDefault = "auto", qualityDefault = "default", projects = [], prefill = null, draft = null, suggestions = [], fromLink = null) {
     const head = html `${phoneHeader("New task", { back: "/tasks" })}`;
     const header = docHeader({ title: "New task", back: ["/tasks", "Tasks"], meta: "Nothing is built until you approve its plan.", same: true });
     const items = projectItems(projects, project);
     const values = prefill?.values;
-    // No project to choose from: the task names its folder (the server proves it), and adding a project is one link away.
     const typed = items.length === 0;
     const chosen = values?.get("repo") ?? project ?? items[0]?.[0] ?? "";
     const planOn = values === undefined || values.get("plan-first") === "1";
@@ -50,7 +31,6 @@ fromLink = null) {
     const after = values?.get("after") ?? "";
     const permission = values?.get("permission-mode") === "bypassPermissions" ? "bypassPermissions" : values?.get("permission-mode") === "auto" ? "auto" : permissionDefault;
     const text = prefill?.title ?? draft ?? "";
-    // More options opens on a template, or when a refused filing had used any of them.
     const optionsUsed = fromLink !== null || prefill !== null && (values === undefined || ["from", "goal", "acceptance", "not", "touches", "id", "after"].some(name => (values.get(name) ?? "") !== "") || values.get("scout") === "1");
     const folder = !typed ? "" : field({ id: "nt-repo", label: "Project folder", hint: "No projects yet: name the folder of a git repository.",
         control: input({ id: "nt-repo", name: "repo", value: chosen, hint: true, attrs: { required: true, placeholder: "/path/to/repository", autocapitalize: "none", spellcheck: "false" } }) });
@@ -63,9 +43,6 @@ fromLink = null) {
         value: text, rows: 4, maxRows: 10, before: bar, error: problem,
         send: sendButton({ label: "File task" }),
         note: html `<span class="k-on-note">${PLAN_ON}</span><span class="k-off-note">${PLAN_OFF}</span>`,
-        // The box holds what a title holds: a task's words are its title, and the planner reads nothing else of them when no
-        // goal is given, so longer words would be lost, not kept. Filing's whole-word cut (task-text.ts titleWithin) is a
-        // belt for other callers only.
         textAttrs: { maxlength: TASK_TEXT_LIMITS.title, autofocus: true },
     });
     const options = html `<div class="k-form">\
@@ -73,7 +50,7 @@ ${field({ id: "nt-from", label: "From an issue or PR (optional)", hint: "A githu
 ${field({ id: "nt-quality", label: "Checks", control: select({ id: "nt-quality", name: "quality-mode", items: [["default", "Default checks"], ["strict", "Strict (release)"]], current: quality, wide: true }) })}\
 ${candidates.length === 0 ? "" : field({ id: "nt-after", label: "Starts", control: select({ id: "nt-after", name: "after", wide: true, current: after,
             items: [["", "Right away"], ...candidates.map(one => [one.id, `After ${one.id}: ${one.title}`])] }) })}\
-${field({ id: "nt-goal", label: "Goal (optional)", hint: "With acceptance below, automatic approval can sign an unchanged plan.", control: textarea({ id: "nt-goal", name: "goal", rows: 2, value: prefill?.goal ?? "", attrs: { placeholder: "What success looks like" } }) })}\
+${field({ id: "nt-goal", label: "Goal (optional)", hint: "With requirements below, an unchanged plan can be approved automatically.", control: textarea({ id: "nt-goal", name: "goal", rows: 2, value: prefill?.goal ?? "", attrs: { placeholder: "What success looks like" } }) })}\
 ${field({ id: "nt-acceptance", label: "Acceptance", hint: "Needed with a goal. One per line: statement | evidence,kinds | how", control: textarea({ id: "nt-acceptance", name: "acceptance", rows: 2, value: prefill?.acceptance ?? "", attrs: { placeholder: "Requests over the limit return 429 | check" } }) })}\
 ${field({ id: "nt-not", label: "Not this (optional)", control: input({ id: "nt-not", name: "not", value: prefill?.not ?? "" }) })}\
 ${field({ id: "nt-touches", label: "Likely touches (optional)", hint: "Paths, separated by commas.", control: input({ id: "nt-touches", name: "touches", value: prefill?.touches ?? "", hint: true }) })}\
@@ -102,12 +79,9 @@ ${field({ id: "nt-id", label: "Task id (optional)", control: input({ id: "nt-id"
     const recipes = note(html `Work you do often? <a class="k-link" href="/recipes">Use a saved recipe</a>.`);
     return screen("New task", html `${head}${doc(header, form, ideaList, recipes)}${dock(btn("File task", { variant: "primary", type: "submit", attrs: { form: FORM_ID } }))}`, { chrome });
 }
-/** The terms as label and words: the lines every approval surface reads (fast-terms.ts), with nothing hidden. */
 export function fastTermsRows(store, scope, repo) {
     return fastTermsLines(store, scope, repo).map(line => [line.label, line.words]);
 }
-/** Build (docs/plans/plain-toolroll.md P2.4): the exact terms this small task would be approved on, and the one button
- * that approves them. The fields of "Plan it first" are the person's own words, on the ordinary road. */
 export function fastStartPage(chrome, input) {
     const head = html `${phoneHeader("Build this?", { back: "/tasks/new" })}`;
     const header = docHeader({ title: "Build this?", back: ["/tasks/new", "New task"], meta: "Build approves exactly these terms.", same: true });
@@ -115,7 +89,6 @@ export function fastStartPage(chrome, input) {
     const plan = postForm("/tasks/add", btn("Plan it first", { type: "submit" }), { attrs: { class: "k-form" }, hidden: { plain: "1", title: input.title, repo: input.repo, "planning-policy": "choice", "plan-first": "1" } });
     return screen("Build this?", html `${head}${doc(header, facts(input.terms.map(([label, words]) => [label, joinHtml(words.split("\n").map(line => html `<div>${line}</div>`))])), html `<div class="k-acts">${build}${plan}</div>`)}`, { chrome });
 }
-/** An issue or pull request, read and quoted before anything is filed. Add task files it for the ordinary approval. */
 export function importPage(chrome, input) {
     const name = input.kind === "pull" ? `Pull request #${input.number}` : `Issue #${input.number}`;
     const head = html `${phoneHeader(name, { back: "/tasks/new" })}`;

package/dist/server/render-pages.js

@@ -1,4 +1,3 @@
-/** Everyday console pages: home, inbox, board, work, projects, fleet and the queue. */
 import { archivedWords, tidyGroups, TIDY_DEFAULT_DAYS } from "../tidy.js";
 import { stepUpForm } from "../settings-kit.js";
 import { html, joinHtml, postForm, replaceMarkup, reauth } from "../html.js";
@@ -29,16 +28,6 @@ import { reviewHref, taskHref } from "./http.js";
 import { criterionMatrixSummary, incidentWords, PHASE_WORDS, phaseWords } from "./render-results.js";
 import { decisionAnswerCard, decisionAnswerScript, dispatchHeadline, statusLineHtml, workDiagnosticsHtml } from "./render-tasks.js";
 export const TASK_STATES = ["queued", "running", "done", "failed", "cancelled"];
-/**
- * The inbox (v3): only things stalling progress without a person, one card
- * per underlying stall, each with its verb inline or one step away. No
- * auto-refresh — approval links lead to the step-up screen, and a page
- * that might hold typed input never re-renders itself.
- */
-/** Start a builder: the one command, as text to select (/start-builder, where every "Connect a builder" leads, and
- * the Agents page). `project`: the open project's name, when the viewer may see it; `connected`: whether a builder
- * that may build there is checking in (null: say nothing); `managed`: whether this person can start one themselves
- * (registering a builder takes an instance operator), else who to ask. */
 export function startBuilderHtml(data) {
     const where = data.project === null ? null : html ` to <strong>${data.project}</strong>`;
     const state = data.connected === null ? null
@@ -49,14 +38,6 @@ export function startBuilderHtml(data) {
         ? html `<p class="meta">Keep it running. Approved tasks start on their own. The console alone (<code>toolroll serve</code>) doesn't build.</p>`
         : html `<p class="meta">Starting a builder takes someone who manages this Toolroll. Ask them to run it there; your tasks start on their own once it connects.</p>`}</div></section>`;
 }
-/**
- * A build's stage as one of three steps — set up, writing, checking and
- * committing — read from the machine's own phase, never a percent. The
- * steps only move forward: evidence and proof checks run both before and
- * after the commit (builder.ts), so everything after the agent's turn is
- * one step, and the stage words say which part. Null for runs that are not
- * builds (planning, review, investigation): those have no honest steps.
- */
 export const BUILD_STEPS = ["setting up", "writing", "checking and committing"];
 export function buildStepOf(claim) {
     if (claim.role === "planner" || claim.role === "reviewer" || claim.role === "scout")
@@ -67,28 +48,15 @@ export function buildStepOf(claim) {
         return 1;
     return 2;
 }
-/** A clock that ticks in the page (boardTickScript) from the server's own start time; the text is the no-script answer. */
 function tickingClock(startedIso, now) {
     const minutes = Math.max(0, Math.round((now.getTime() - new Date(startedIso).getTime()) / 60_000));
     return html `<time class="clock" datetime="${startedIso}" data-tick>${minutes}m</time>`;
 }
-/**
- * The workers a board's strip may show: not retired, bound to at least one
- * project the server's ceiling admits (never a runner serving only projects
- * this server must not speak about), and — on a one-project board — bound to
- * that project.
- */
 export function boardWorkers(runners, scope) {
     return runners.filter(runner => runner.retiredAt === null &&
         runner.repos.some(repo => scope.visible(repo)) &&
         (scope.all || scope.project === null || runner.repos.includes(scope.project)));
 }
-/**
- * The worker strip across the top of the board: every worker this board's
- * work can go to, what it is building here, and its stage and clock. Only
- * builds the board already shows are named — a worker busy in a project
- * this reader cannot see reads "no build on this board", never "free".
- */
 function workerStrip(runners, cards, now) {
     if (runners.length === 0)
         return html `<p class="meta crew-empty">No builders yet — work waits in the queue until one joins.</p>`;
@@ -110,14 +78,6 @@ function workerStrip(runners, cards, now) {
     });
     return html `<section class="crew" aria-label="Workers">${tiles}</section>`;
 }
-/**
- * The board: the pipeline as lanes, position as meaning — attention, then
- * queued, then waiting, then building, then recently done — under a strip
- * of the workers doing it. Read-only by construction (every card is a link,
- * no forms, no nonces), which is what makes it safe to re-render itself
- * while somebody watches; a card that changes lane slides there
- * (regionScript's animate swap, keyed by data-vt).
- */
 export function boardBody(data, ciRed) {
     const chip = (repo) => !data.all || repo === null ? null : html `<span class="badge">${projectName(repo)}</span>`;
     const CAP = 30;
@@ -129,22 +89,14 @@ export function boardBody(data, ciRed) {
             return `${Math.round(minutes / 60)}h`;
         return `${Math.round(minutes / (24 * 60))}d`;
     };
-    // A lane's head is a status chip in its own tone (after ReUI's badge with a dot): solid ink only for the lane that
-    // waits on a person, blue for building, green for done, neutral otherwise. The word carries the state.
     const LANE_CHIP = { attention: "chip-needs", building: "chip-live", done: "chip-done", queued: "", waiting: "" };
     const laneChip = (key, title) => html `<span class="chip ${LANE_CHIP[key]}"><span class="chip-dot" aria-hidden="true"></span>${title}</span>`;
     const lane = (key, title, hint, renderOne) => {
         const cards = data.cards.filter(card => card.lane === key);
-        // The longest-stalled card leads the board: attention sorts by how
-        // long it has waited, everything else stays newest-first — except the
-        // queued lane, which reads in DISPATCH order: moved-up work first,
-        // exactly as the next free worker will pick it.
         if (key === "attention") {
             cards.sort((a, b) => (a.stalledSince ?? "9999").localeCompare(b.stalledSince ?? "9999"));
         }
         if (key === "queued") {
-            // Group: the shared queue first, then reserved columns by worker;
-            // dispatch order (rank) within each — never a cross-column rank race.
             cards.sort((a, b) => {
                 const ka = a.assignedRunner ?? "";
                 const kb = b.assignedRunner ?? "";
@@ -155,15 +107,8 @@ export function boardBody(data, ciRed) {
         }
         const shown = cards.slice(0, CAP);
         const more = cards.length - shown.length;
-        // A lane is a section that can fold (board pass): open when it holds
-        // cards, folded when empty, so a phone reads the counts first and a
-        // desktop column never spends its height on "nothing here".
         return html `<details class="lane lane-${key}"${shown.length === 0 ? null : html ` open`}><summary><h2>${laneChip(key === "attention" && cards.length === 0 ? "queued" : key, title)} <span class="lane-count">${cards.length}${data.saturated ? "+" : ""}</span></h2></summary><p class="hint">${hint}</p>${shown.length === 0 ? html `<p class="meta lane-empty">nothing here</p>` : shown.map((card, index) => renderOne(card, index))}${more > 0 ? html `<a class="lane-more" href="/tasks">+${more} more in Tasks</a>` : null}</details>`;
     };
-    // The card's facts: mono key–value pairs under the title (board pass) —
-    // task, worker, runtime — the same grammar on every lane, so the eye
-    // learns one card. Chips carry the words (project, reservation).
-    // A value is mono only when a machine said it (a worker, model or branch name, a clock); words stay Geist.
     const facts = (rows) => rows.length === 0
         ? null
         : html `<span class="facts">${rows.map(([k, v, mono]) => html `<span class="fact"><span class="k">${k}</span><span class="${mono === undefined ? "v" : "v mono"}">${v}</span></span>`)}</span>`;
@@ -176,10 +121,6 @@ export function boardBody(data, ciRed) {
     ])}${chips([
         chip(card.repo),
     ])}</a>`;
-    // The queued lane: ranks compare only within a COLUMN (queue-columns
-    // review, finding 14), so the badge is per column head — the shared
-    // queue's front card says "next up"; a reserved column's front card
-    // says whose turn it is. No cross-column comparison is claimed.
     const queuedHeads = (() => {
         const heads = new Map();
         for (const card of data.cards.filter(one => one.lane === "queued")) {
@@ -203,18 +144,12 @@ export function boardBody(data, ciRed) {
         if (claim === null)
             return plain(card);
         const workspace = claim.worktree === null ? null : (claim.worktree.split("/").pop() ?? claim.worktree);
-        // Chip copy: unknown phases show nothing rather than raw tokens.
         const phase = claim.phase === null ? undefined : PHASE_WORDS[claim.phase];
-        // The live strip is the run's own phase and clock — never a percent:
-        // a build has no honest progress figure, only a stage and an elapsed.
         const live = claim.model === null ? "preparing workspace" : (phase ?? "the agent is working");
-        // The steps: where the build is among set up → writing → checking →
-        // committing, from its phase — a stage, never a percent.
         const step = buildStepOf(claim);
         const steps = step === null
             ? null
             : html `<span class="steps" role="img" aria-label="${`step ${step + 1} of ${BUILD_STEPS.length}: ${BUILD_STEPS[step]}`}">${BUILD_STEPS.map((_, index) => html `<i class="${index < step ? "on" : index === step ? "now" : ""}"></i>`)}</span>`;
-        // Who, which model and which branch, as one muted line: words in Geist, the model and branch in mono.
         const model = claim.model === null ? null : claim.provider === null ? modelName(null, claim.model) : agentModelName(claim.provider, claim.model);
         const branch = claim.branch === null ? null : `${claim.branch}${workspace === null ? "" : ` · ${workspace}`}`;
         const who = joinHtml([
@@ -276,8 +211,6 @@ export function boardBody(data, ciRed) {
 }
 export function homePage(chrome, data) {
     const { summary } = data;
-    // The night's harvest is the page's reason to exist — strong figures in a
-    // sentence, colored by what they mean, never a metric-card grid.
     const ledger = html `<p class="ledger"><span class="good"><b>${summary.built.length}</b> built</span> · <span${summary.failed.length > 0 ? html ` class="bad"` : null}><b>${summary.failed.length}</b> failed</span> · <b>${summary.refused.length}</b> refused${summary.cutDown.length > 0 ? html ` · <span class="bad"><b>${summary.cutDown.length}</b> cut down mid-flight</span>` : null}</p>`;
     const decide = data.decisions.length === 0
         ? html `<p class="meta">Nothing waits on you. No questions came up.</p>`
@@ -291,15 +224,10 @@ export function homePage(chrome, data) {
     const gaps = data.gaps === null || data.gaps.length === 0
         ? null
         : html `<h2>Missing requirements</h2><p class="hint">tools or credentials builds need — checked before any money is spent</p>${joinHtml(data.gaps.map(gap => html `<p class="row"><a href="/caps">${gap.key}</a> — ${gap.state}<span class="right meta">how to fix →</span></p>`), "\n")}`;
-    // Live at the fidelity the moment deserves: fast while something builds,
-    // gentle when the page is just a briefing. GET-only, so refresh is safe.
     const refresh = data.building.length > 0 ? 10 : 60;
-    // BUILDING RIGHT NOW: each live claim as a pulsing card — the one moment
-    // an operator actually watches this page, so it re-renders itself.
     const building = data.building.length === 0
         ? null
         : html `<h2>Building now</h2><p class="hint">live builds — this page refreshes itself every 10 seconds while anything runs</p><div class="cards">${data.building.map(claim => html `<a class="stat-card" href="${taskHref(claim.taskId)}" style="text-decoration:none"><span class="k"><span class="dot dot-ok pulse"></span>${claim.taskId}</span><span class="v">${claim.runner} · ${Math.max(1, Math.round((data.now.getTime() - new Date(claim.claimedAt).getTime()) / 60_000))}m elapsed${claim.model === null ? null : html ` · ${modelName(null, claim.model)}`}</span></a>`)}</div>`;
-    // THE FLEET: runners by heartbeat age, worktrees by lease state, the watch.
     const nowMs = data.now.getTime();
     const runnerCards = data.runners
         .filter(one => one.retiredAt === null)
@@ -357,13 +285,11 @@ export function homePage(chrome, data) {
     ], "\n"), { chrome, refreshSeconds: refresh });
 }
 export const TASK_GROUP_LABEL = { ...ASK_LABEL, building: 'Building', rest: 'Recent' };
-/** A list row's chip (src/design/KIT.md): a waiting row names its ask ("Plan", "Result") in the ask tone, or says
- * Needs you; every other row wears its headline in the tone of the status it belongs to. */
 export function workRowChip(row) {
     if (row.archived !== undefined)
         return { word: 'Archived', tone: '' };
     if (row.ask !== null)
-        return row.chip === null ? { word: 'Needs you', tone: 'needs' } : { word: row.chip, tone: 'ask' };
+        return { word: row.chip ?? 'Needs you', tone: 'ask' };
     const label = row.status.label;
     if (HEADLINES.includes(label)) {
         const chip = chipOfHeadline(label);
@@ -372,7 +298,6 @@ export function workRowChip(row) {
     const tone = row.status.tone;
     return { word: label, tone: tone === 'live' ? 'live' : tone === 'done' ? 'done' : tone === 'problem' ? 'failed' : '' };
 }
-/** Where a row's one action goes: a result opens on its task's Result tab (the review page is the task page now). */
 function workActionHref(row) {
     const target = row.primaryAction?.target;
     if (row.primaryAction?.code === 'open-result' && target?.runId != null)
@@ -381,26 +306,21 @@ function workActionHref(row) {
 }
 export function workPage(chrome, data) {
     const layout = data.board != null ? 'board' : 'list';
-    // The project and the stored-state filter ride every link; a change of filter starts on page one.
     const query = (over = {}) => {
         const project = over.project === undefined ? (data.projectFilter ?? (data.all === true ? 'all' : null)) : over.project;
         const board = (over.layout ?? layout) === 'board';
         const state = over.state === undefined ? data.state ?? null : over.state;
-        // Tasks' own address (docs/plans/plain-toolroll.md P1.4): Board is a view; every project is scope=all.
         return tasksHref({ scope: project === 'all' ? 'all' : null, project: project === 'all' ? null : project, view: board ? 'board' : over.view ?? data.view,
             state: board ? null : state, cursor: over.cursor ?? null });
     };
     const href = (view, cursor, state = data.state ?? null) => query({ view, ...(cursor === undefined ? {} : { cursor }), state });
-    // Put away (or Restore, in Archived): one quiet act per row, never on a task with work going on.
     const tidyAct = (row) => data.canPutAway !== true ? null : row.archived !== undefined ? { action: `${taskHref(row.rootId)}/restore`, label: 'Restore' }
         : row.liveRunId !== null || row.unfinishedRunId !== null || row.status.views.includes('running') ? null : { action: `${taskHref(row.rootId)}/archive`, label: 'Archive' };
     const returnTo = href(data.view, undefined);
     const current = WORK_VIEWS.find(one => one.key === data.view);
     const filtered = (data.state ?? null) !== null;
     const projectOf = (row) => data.multiProject || row.repo === null ? (row.repo === null ? 'Unplaced' : projectName(row.repo)) : null;
-    // The status filters, as addresses, each with its count; Archived sits at the foot of Finished, not in the track.
     const tabs = WORK_VIEWS.filter(one => one.key !== 'archived').map(one => ({ key: one.key, label: one.label, href: href(one.key), count: data.work.totals[one.key], active: one.key === data.view }));
-    // The project filter: All and each project in view, when there is more than one.
     const known = chrome.projects ?? [];
     const lens = data.lens === undefined ? chrome.project : data.lens;
     const projects = known.length < 2 ? null : [
@@ -423,7 +343,6 @@ export function workPage(chrome, data) {
             id: row.rootId, title: row.title, href: taskHref(row.rootId), project: projectOf(row), age: relativeAge(row.updatedAt, data.now),
             status: { label: row.status.label, tone: row.status.tone, token: row.status.token }, chip: row.chip, word: chip,
             ask: row.ask, group: row.ask ?? (row.status.rank === 1 ? 'building' : 'rest'), rank: row.status.rank,
-            // One quiet action, on a row that waits on you; every other row is its own link.
             action: actionHref === null || row.primaryAction === null || row.ask === null ? null : { label: row.primaryAction.label, href: actionHref },
             detail: needsYouDetail ? row.status.detail : row.progress ?? (row.ask === null && row.status.detail !== row.status.label ? row.status.detail : null),
             problem: row.familyProblem,
@@ -468,8 +387,6 @@ export function workPage(chrome, data) {
         states: [{ label: 'Any state', href: href(data.view, undefined, null), active: (data.state ?? null) === null },
             ...TASK_STATES.map(one => ({ label: one.charAt(0).toUpperCase() + one.slice(1), href: href(data.view, undefined, one), active: data.state === one }))],
     };
-    // The page without the workspace (and its first paint) keeps the older rows the server tests read; the workspace
-    // replaces it with the kit view above.
     const legacyTabs = html `<nav class="work-views" aria-label="Task views">${WORK_VIEWS.map(one => html `<a href="${href(one.key)}"${one.key === data.view ? html ` class="active" aria-current="page"` : null}>${one.label}<span class="count">${data.work.totals[one.key]}</span></a>`)}</nav>`;
     const rowHtml = (row) => {
         if (row.archived !== undefined) {
@@ -487,7 +404,6 @@ export function workPage(chrome, data) {
         const putAway = act === null ? null : postForm(act.action, html `<button type="submit" class="quiet">${act.label}</button>`, { hidden: { return: returnTo }, attrs: { class: 'inline', 'data-tidy-act': 'put-away' } });
         return html `<article class="work-row" data-task="${row.rootId}" data-work-status="${row.status.token}" data-work-views="${row.status.views.join(' ')}"><div class="work-row-main"><a class="work-title" href="${taskHref(row.rootId)}">${row.title}</a><p class="work-meta">${project}<span>${relativeAge(row.updatedAt, data.now)}</span><span class="mono">${row.rootId}</span></p></div><div class="work-row-status">${statusLineHtml(row.status)}${action}${putAway}${row.familyProblem === null ? null : html `<p class="problem">${row.familyProblem}</p>`}${row.status.views.includes('needs-you') && !['write-scope', 'approve-scope'].includes(row.primaryAction?.code ?? '') && row.status.detail !== row.status.label && row.status.detail !== row.familyProblem ? html `<p class="work-detail">${row.status.detail}</p>` : null}${workDiagnosticsHtml(row.status.diagnostics)}</div></article>`;
     };
-    // The state filter, as real links (the old task list's), so a page without the React view can still change it.
     const stateLinks = html `<nav class="work-states" aria-label="Task state"><span class="meta">State:</span> ${joinHtml([html `<a href="${href(data.view, undefined, null)}"${filtered ? null : html ` aria-current="page"`}>Any</a>`,
         ...TASK_STATES.map(one => html `<a href="${href(data.view, undefined, one)}"${data.state === one ? html ` aria-current="page"` : null}>${one}</a>`)], html ` · `)}</nav>`;
     const list = data.work.items.length === 0 && filtered && !data.previous
@@ -500,12 +416,6 @@ export function workPage(chrome, data) {
         .map(([path, label]) => html `<a href="${path}">${label}</a>`)}</nav></details>`;
     return screen('work', html `<div class="work-head"><h1>Tasks</h1>${putAwayLink === null ? null : html `<a class="button-link" href="${putAwayLink.href}" data-put-away-old>${putAwayLink.label}</a>`}${tools}</div>${limitsHtml(data.limits ?? null)}${legacyTabs}${stateLinks}${list}${pages}`, { chrome, workspace: { view } });
 }
-/**
- * owner/name from a git remote URL — FULLY ANCHORED https/ssh github.com
- * forms only (ghlist review, finding 2): a foreign host carrying
- * "github.com" in its path, or a non-github host, must never read as a
- * GitHub identity. Everything else is null.
- */
 export function githubIdentityOf(remoteUrl) {
     const trimmed = remoteUrl.trim();
     const https = /^https:\/\/github\.com\/([A-Za-z0-9-]+)\/([A-Za-z0-9._-]+?)(?:\.git)?\/?$/.exec(trimmed);
@@ -516,13 +426,8 @@ export function githubIdentityOf(remoteUrl) {
         return `${ssh[1]}/${ssh[2]}`;
     return null;
 }
-/** The no-script "move to the front" sentinel: the form cannot name the
- * front of a partition, so the handler resolves it (slice 1b, fix 1). */
 export const QUEUE_FRONT = "__TOP__";
 export function queueCard(one, revision, queueRevision, workers, column) {
-    // Presentation over queueScoped()'s shape only: state, scope, blockers,
-    // and the reservation owner. Money is not in this query and is not
-    // invented here — it stays on the task page, labeled.
     const state = one.taken
         ? html `<span class="badge">Starting — a builder has it</span>`
         : column === "anyone"
@@ -535,7 +440,6 @@ export function queueCard(one, revision, queueRevision, workers, column) {
         : html `${postForm("/queue/move", html `<button type="submit" class="icon-button" aria-label="move to the front">${TO_FRONT_ICON}</button>`, { ...options, hidden: { queueRevision, task: one.id, column, before: QUEUE_FRONT } })}${postForm("/queue/move", html `<select name="column" aria-label="reserve for"><option value="anyone"${column === "anyone" ? html ` selected` : null}>anyone</option>${workers.filter(worker => !worker.retired).map(worker => html `<option value="${worker.name}"${column === worker.name ? html ` selected` : null}>${worker.name}</option>`)}</select><button type="submit">Move</button>`, { ...options, hidden: { queueRevision, task: one.id } })}`;
     return html `<div class="card queue-card" data-task="${one.id}" data-taken="${one.taken ? "1" : "0"}"${one.dispatch === null || one.dispatch === undefined ? null : html ` data-dispatch-status="${one.dispatch.code}"`}><p class="row">${one.taken ? null : GRIP_HANDLE}<a href="${taskHref(one.id)}">${one.title}</a>${chips}</p><p class="row meta"><span class="mono">${one.id}</span> ${controls}</p></div>`;
 }
-/** The queue columns fragment — shared queue first, then each worker. */
 export function queueBody(tasks, workers, revision, queueRevision) {
     const columnOf = (runner) => tasks.filter(one => one.assignedRunner === runner);
     const projectChips = (rows) => {
@@ -554,12 +458,6 @@ export function queueBody(tasks, workers, revision, queueRevision) {
         : html `<p class="meta mono">${worker.building} building in this project · unattended capacity ${worker.capacity}</p>${postForm("/queue/note", html `<input type="text" name="note" value="${worker.note ?? ""}" data-initial="${worker.note ?? ""}" placeholder="what this worker is working through" aria-label="column note" maxlength="200"><button type="submit">Save</button>`, { attrs: { class: "row" }, projectRevision: revision, hidden: { runner: worker.name } })}<p class="meta">Takes from the shared queue when this column is empty</p>`;
     return html `<div class="lanes" data-queue-revision="${queueRevision}">${column("shared queue", "anyone", html `<p class="meta">Workers take from here when their column is empty — top card first</p>`, shared, "nothing waiting — every task is reserved or running")}${joinHtml(workers.map(worker => column(worker.name + (worker.retired ? " (retired)" : ""), worker.name, noteForm(worker), columnOf(worker.name), "nothing queued — this builder takes from the shared queue")), "\n")}</div>`;
 }
-/** The /queue page's one nonce'd script: delegated pointer-event drag (it
- * survives every fragment swap — finding 17) plus a poller that re-checks
- * focus, dirty inputs, and an in-flight drag AT SWAP TIME, never only
- * before the fetch. Select dirtiness compares against data-initial
- * (finding 18); missing that, a select counts clean.
- */
 export function queueScript() {
     return (`(function(){var region=document.getElementById("queue-region");if(!region)return;` +
         `var stamp=document.getElementById("queue-region-stamp");var dragging=null;var ghost=null;` +
@@ -570,7 +468,6 @@ export function queueScript() {
         `var base=selects[j].getAttribute("data-initial");if(base!==null&&selects[j].value!==base)return true;}` +
         `return false;}` +
         `function paused(){return dragging!==null||dirty();}` +
-        // the poller: pause is re-checked at SWAP time
         `var wait=15000;var last=Date.now();var busy=false;` +
         `function tell(){if(!stamp)return;if(paused()){stamp.textContent="paused while you edit";return;}` +
         `stamp.textContent="updated "+Math.round((Date.now()-last)/1000)+"s ago";}setInterval(tell,1000);` +
@@ -581,7 +478,6 @@ export function queueScript() {
         `.then(function(t){if(t&&!paused()){region.innerHTML=t;last=Date.now();}})` +
         `.catch(function(){})` +
         `.then(function(){busy=false;tell();setTimeout(cycle,wait);});}setTimeout(cycle,wait);` +
-        // the drag: delegated from the stable region element
         `region.addEventListener("pointerdown",function(e){var handle=e.target.closest(".queue-handle");if(!handle)return;` +
         `var card=handle.closest(".queue-card");if(!card||card.getAttribute("data-taken")==="1")return;` +
         `e.preventDefault();dragging={task:card.getAttribute("data-task"),card:card};card.style.opacity="0.5";});` +
@@ -601,10 +497,6 @@ export function queueScript() {
         `var fields={respond:"fragment",csrf:(region.querySelector("input[name=csrf]")||{value:""}).value,projectRevision:(region.querySelector("input[name=projectRevision]")||{value:""}).value,` +
         `queueRevision:wrap?wrap.getAttribute("data-queue-revision"):"",task:drag.task,column:column,before:before};` +
         `var post=new URLSearchParams();Object.keys(fields).forEach(function(k){if(fields[k]!==""||k==="respond")post.append(k,fields[k]);});` +
-        // A refused move (slice 1b, fix 2): the handler's typed text/plain 409
-        // lands on the card as a problem row — textContent, never markup. Only
-        // an authenticated plain-text 409 is inlined; an HTML refusal (the
-        // stale-project page), a login bounce, or anything unexpected navigates.
         `function problem(task,text){var card=region.querySelector('.queue-card[data-task="'+task.replace(/["\\\\]/g,"")+'"]');if(!card)return false;` +
         `var old=card.querySelector(".queue-problem");if(old)old.remove();` +
         `var row=document.createElement("p");row.className="problem queue-problem";row.setAttribute("role","alert");row.textContent=text;card.appendChild(row);return true;}` +
@@ -620,8 +512,6 @@ export function queueScript() {
         `.then(function(){tell();});});` +
         `})();`);
 }
-/** Put away old tasks (docs/plans/task-tidy.md): exactly what would be put away, grouped, each with why, every one
- * ticked; one step-up puts away the ticked ones, bound to this preview's digest. The CLI twin is named beside it. */
 export function putAwayPage(chrome, data) {
     const groups = tidyGroups(data.items, data.days);
     const count = data.items.length;