toolroll 0.9.84 → 0.9.86

package/dist/server/console.css

@@ -2730,3 +2730,44 @@ html.so-swap::view-transition-old(*):only-child { animation: so-card-out 160ms e
   html.so-swap::view-transition-group(*), html.so-swap::view-transition-new(*), html.so-swap::view-transition-old(*) { animation: none !important; }
   .worker.flash { animation: none; }
 }
+
+/* Exact-image feedback shares the existing comment form. */
+.result-image { margin: 0; min-width: 0; }
+.result-image > button { margin-top: .5rem; min-height: 44px; white-space: nowrap; }
+.feedback-image { display: flex; align-items: center; flex-wrap: wrap; gap: .75rem; margin: .5rem 0; min-width: 0; }
+.feedback-image[hidden], .feedback-image img[hidden] { display: none; }
+.feedback-image img { width: 5rem; max-height: 5rem; object-fit: contain; border: 1px solid var(--border); border-radius: var(--radius); }
+.feedback-image figcaption { flex: 1 1 12rem; overflow-wrap: anywhere; font-size: .875rem; }
+.feedback-image button { min-height: 44px; white-space: nowrap; }
+
+.result-feedback-actions { display: flex; flex-wrap: wrap; gap: .5rem; }
+.result-feedback-actions button { min-height: 44px; white-space: nowrap; }
+
+/* Put away old tasks (docs/plans/task-tidy.md): grouped checklist on the kit's column. */
+.put-away { max-width: var(--so-settings-width, 760px); }
+.put-away h1 { margin: 0 0 4px; }
+.put-away-days { display: flex; flex-wrap: wrap; align-items: center; gap: 8px; margin: 0 0 16px; }
+.put-away-days label { display: flex; align-items: center; gap: 8px; margin: 0; font-size: 14px; }
+.put-away-days input[type=number] { width: 5rem; margin: 0; }
+.put-away-days button { width: auto; }
+.put-away-group { margin: 0 0 20px; padding: 0; border: 0; }
+.put-away-group legend { display: flex; align-items: center; gap: 8px; padding: 0; margin: 0 0 6px; font-weight: 600; font-size: 15px; }
+.put-away-group legend .count { font-family: var(--so-mono, ui-monospace, monospace); font-size: 11px; font-weight: 500; color: var(--so-muted); }
+.put-away-group ul { list-style: none; margin: 0; padding: 0; border-top: 1px solid var(--so-line); }
+.put-away-group li { border-bottom: 1px solid var(--so-line); }
+.put-away-item { display: grid; grid-template-columns: auto minmax(0, 1fr); column-gap: 10px; align-items: start; margin: 0; padding: 10px 2px; cursor: pointer; }
+.put-away-item input { margin: 3px 0 0; width: auto; min-height: 0; align-self: start; }
+.put-away-title { font-size: 13.5px; font-weight: 500; overflow-wrap: anywhere; }
+.put-away-why { grid-column: 2; font-size: 12.5px; color: var(--so-muted); }
+.put-away-id { grid-column: 2; font-family: var(--so-mono, ui-monospace, monospace); font-size: 11.5px; color: var(--so-muted); overflow-wrap: anywhere; }
+.put-away-cli { margin: 12px 0 0; font-size: 13px; color: var(--so-muted); overflow-wrap: anywhere; }
+.put-away-cli code { font-size: 12px; }
+.task-archived { display: flex; flex-wrap: wrap; align-items: center; justify-content: space-between; gap: 8px 12px; margin: 0 0 12px; padding: 10px 12px; border-radius: 8px; background: var(--so-raised); }
+.task-archived p { margin: 0; font-size: 13.5px; }
+.task-archived form { margin: 0; }
+.task-archived button { width: auto; }
+.work-row-status form.inline { margin: 0; }
+@media (max-width: 900px) {
+  .put-away-item { min-height: 44px; }
+  .put-away-days button, .task-archived button, .work-row-status form.inline button { min-height: 44px; }
+}

package/dist/server/flows.js

@@ -21,6 +21,7 @@ import { exportFlow, fetchFlowFile, FlowFileError, importFlow, parseFlowFile, pl
 import { addFlowTriggerTo, checkFlowTriggerNow, pressFlowButton, removeFlowTrigger, removeLinearKey, setFlowTriggerOn, renewFlowHook, saveHooksBase, saveLinearKey, saveSigningSecret, setUpFlowHook, changeFlowHookSender, setFlowHookScope, shareFlowButton, stopSharingFlowButton, rawTakesDeliveries, rawAdmitsAnyone } from "../flow-triggers.js";
 import { FLOW_IMPORT_SCRIPT, flowFallbackHtml, flowImportHtml, flowsListHtml, flowView } from "../flows-ui.js";
 import { FLOW_TEMPLATES, FlowContractError, validateFlowDefinition, withZoneNames } from "../flows.js";
+import { revokeFlowBrowser, signFlowBrowser } from "../flow-browser.js";
 import { prepareUnattendedCandidates, parseUnattendedAsk, revokeUnattended, signUnattended, untilWords } from "../flow-unattended.js";
 import { keyStatus } from "../keys.js";
 import { kitPageHtml, kitsGalleryHtml } from "../kits-ui.js";
@@ -656,6 +657,18 @@ export function createFlowsHandlers(runtime) {
         if (action === "unattended-off") {
             return revokeUnattended(store, flow.id, who.name, now) ? settle("It no longer runs on its own. New cards wait for you.") : answer(409, { ok: false, said: "It wasn't running on its own." });
         }
+        if (action === "browser") {
+            // A person's own step-up (a lead's or remote run's token never gets here: route table), then signFlowBrowser
+            // re-reads the flow, its zones and scripts in its one write and refuses unless they are the digest the panel showed.
+            const proved = authenticateApprover(who, body.get("password") ?? "", flow.repo);
+            if (!proved.ok)
+                return answer(403, { ok: false, said: "That password isn't right. Nothing was approved." });
+            const signed = signFlowBrowser(store, { flow: flow.id, signer: who.name, proof: proved.method, sawDigest: body.get("digest") ?? "", console: { generation: who.via === "cookie" ? who.session.generation : -1 } }, now);
+            return signed.ok ? settle(`Browser allowed. ${signed.grant.zones.length === 1 ? "Its script" : "Its scripts"} can start Chromium now.`) : answer(409, { ok: false, said: signed.message });
+        }
+        if (action === "browser-off") {
+            return revokeFlowBrowser(store, flow.id, who.name, now) ? settle("Browser turned off. Its browser zones wait until someone allows it again.") : answer(409, { ok: false, said: "Browser use wasn't allowed for this flow." });
+        }
         if (action === "save") {
             let saved, drawn;
             try {
@@ -975,6 +988,8 @@ export function createFlowsHandlers(runtime) {
         "flow.secrets": flowAct("secrets"),
         "flow.unattended": flowAct("unattended"),
         "flow.unattended-off": flowAct("unattended-off"),
+        "flow.browser": flowAct("browser"),
+        "flow.browser-off": flowAct("browser-off"),
         "flow.triggers": flowAct("triggers"),
         "flow.card.move": cardAct("move"),
         "flow.card.decide": cardAct("decide"),

package/dist/server/pages.js

@@ -1,3 +1,4 @@
+import { projectOutcomes } from '../project-outcomes.js';
 import { steerControlsFor } from "../steer-controls.js";
 import { canReveal, cdCommandOf, localCheckoutOf } from "../checkout-proof.js";
 import { instanceHttpAnswer } from "../console-instance.js";
@@ -967,7 +968,16 @@ export function createPagesHandlers(runtime) {
             peeks[one.path] = peekOf(one.path);
         for (const path of candidates)
             peeks[path] = peekOf(path);
-        return sendScreen(response, status, projectsPage(chromeFor(open, "projects"), recent, [...candidates], open, problem, !restricted() && unscopedMode, !restricted() && (ceiling.roots.length > 0 || unscopedMode), onboardState, peeks, returnTo, who.via === "cookie" ? (path) => publishingOf(store, path).on : undefined, who.via === "cookie" ? (path) => projectCheckLevel(store, path).level : undefined));
+        let activity;
+        if (open !== null && !returnTo.startsWith("/tasks/new") && visible(open) && store.accountCanAccess(who.name, open)) {
+            try {
+                activity = { project: projectName(open), report: projectOutcomes(store, open, now, { principal: 'operator', repos: [open] }) };
+            }
+            catch {
+                activity = { project: projectName(open), report: null };
+            }
+        }
+        return sendScreen(response, status, projectsPage(chromeFor(open, "projects"), recent, [...candidates], open, problem, !restricted() && unscopedMode, !restricted() && (ceiling.roots.length > 0 || unscopedMode), onboardState, peeks, returnTo, who.via === "cookie" ? (path) => publishingOf(store, path).on : undefined, who.via === "cookie" ? (path) => projectCheckLevel(store, path).level : undefined, activity));
     }
     async function instanceEdge(ctx) {
         const { request, response, url } = ctx;

package/dist/server/render-chat.js

@@ -1,3 +1,4 @@
+import { parseFeedbackImage } from "../result-review.js";
 import { assignmentSummaryHtml } from "../assignment-ui.js";
 import {} from "../assignment.js";
 import {} from "../browser-workspace.js";
@@ -610,8 +611,8 @@ export function proposalCardParts(view, inert, decision, returnTo = null) {
         const snapshot = payload["snapshot"];
         const ids = Array.isArray(payload["notes"]) ? payload["notes"] : [];
         const selected = snapshot?.notes.filter(one => ids.includes(one.id)) ?? [];
-        const notes = [...selected, ...(text("note") === "" ? [] : [{ note: text("note"), path: text("path") || null, line: typeof payload["line"] === "number" ? payload["line"] : null }])];
-        what = html `<h3>${text("taskTitle") || task}</h3><ul class="proposal-review-notes">${notes.map(one => html `<li>${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : html `:${one.line}`}</span> · `}${one.note}</li>`)}</ul>${text("operation") === "revise" ? html `<p class="meta">Updates this task. Your approval settings apply.</p>` : html `<p class="meta">No work starts.</p>`}`;
+        const notes = [...selected, ...(text("note") === "" ? [] : [{ note: text("note"), path: text("path") || null, line: typeof payload["line"] === "number" ? payload["line"] : null, image: parseFeedbackImage(payload["image"]) || null }])];
+        what = html `<h3>${text("taskTitle") || task}</h3><ul class="proposal-review-notes">${notes.map(one => html `<li>${one.image == null ? "" : html `<a href="/r/${one.image.run}/evidence/${one.image.artifact}">Screenshot ${one.image.artifact} · result #${one.image.run}</a><br>`}${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : html `:${one.line}`}</span> · `}${one.note}</li>`)}</ul>${text("operation") === "revise" ? html `<p class="meta">Updates this task. Your approval settings apply.</p>` : html `<p class="meta">No work starts.</p>`}`;
     }
     else if (view.kind === "next") {
         what = html `<h3>Move <a href="${taskHref(task)}">${task}</a> to the front</h3>${facts(["current position", html `${String(payload["position"] ?? "?")} of ${String(payload["of"] ?? "?")}`], ["project", mono(repoId)])}`;

package/dist/server/render-pages.d.ts

@@ -1,9 +1,10 @@
 /** Everyday console pages: home, inbox, board, work, projects, fleet and the queue. */
+import { type TidyItem } from "../tidy.js";
 import { type TaskOrigin } from "../task-source.js";
 import { type Html } from "../html.js";
 import { type AssignmentSnapshot } from "../assignment.js";
 import { type BoardCard } from "../board.js";
-import { type BrowserLimits } from "../browser-workspace.js";
+import { type BrowserLimits, type BrowserProjectsView } from "../browser-workspace.js";
 import { type CheckLevel } from "../check-levels.js";
 import { type DispatchDiagnosis } from "../dispatch.js";
 import { type FirstRunStep } from "../first-run.js";
@@ -295,6 +296,8 @@ export declare function workPage(chrome: Chrome, data: {
     now: Date;
     limits?: BrowserLimits | null;
     state?: TaskState | null;
+    /** An approver in a browser session: rows offer Put away (or Restore, archived), and the page offers Put away old tasks. */
+    canPutAway?: boolean;
 }): Screen;
 export declare function browsePage(chrome: Chrome, data: {
     at: string;
@@ -451,7 +454,7 @@ export declare function projectsPage(chrome: Chrome, recent: {
     path: string;
     name: string;
     lastOpenedAt: string;
-}[], candidates: string[], open: string | null, problem: string | null, _unscopedMode: boolean, browsable?: boolean, onboard?: OnboardCardState | null, peeks?: Record<string, ProjectPeek | null>, returnTo?: string, pullRequestsOn?: (path: string) => boolean, checkLevelOf?: (path: string) => CheckLevel): Screen;
+}[], candidates: string[], open: string | null, problem: string | null, _unscopedMode: boolean, browsable?: boolean, onboard?: OnboardCardState | null, peeks?: Record<string, ProjectPeek | null>, returnTo?: string, pullRequestsOn?: (path: string) => boolean, checkLevelOf?: (path: string) => CheckLevel, activity?: BrowserProjectsView["activity"]): Screen;
 /** 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 declare const QUEUE_FRONT = "__TOP__";
@@ -562,3 +565,13 @@ export type PeekAdmission = {
     epoch: string;
     entries: ReturnType<typeof parseBaseTreeSnapshot>;
 };
+/** 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 declare function putAwayPage(chrome: Chrome, data: {
+    items: readonly TidyItem[];
+    days: number;
+    digest: string;
+    now: Date;
+    problem: string | null;
+    releasesSkipped: string | null;
+}): Screen;

package/dist/server/render-pages.js

@@ -1,4 +1,7 @@
+import { activityMetrics, activityScope, activityLimit } from '../project-activity.js';
 /** 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 { originNote } from "../task-source.js";
 import { html, htmlString, joinHtml, postForm, replaceMarkup, reauth } from "../html.js";
 import {} from "../assignment.js";
@@ -631,14 +634,25 @@ export function workPage(chrome, data) {
         return `/work${query.size ? `?${query}` : ''}`;
     };
     const tabs = 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>`;
+    // 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: 'Put away' };
+    const returnTo = href(data.view, undefined);
     const rowHtml = (row) => {
+        if (row.archived !== undefined) {
+            const act = tidyAct(row);
+            const project = data.multiProject || row.repo === null ? html `<span class="project-label">${row.repo === null ? 'Unplaced' : projectName(row.repo)}</span>` : null;
+            return html `<article class="work-row" data-task="${row.rootId}" data-work-status="archived" data-work-views="archived"><div class="work-row-main"><a class="work-title" href="${taskHref(row.rootId)}">${row.title}</a><p class="work-meta">${project}<span>${relativeAge(row.archived.at, data.now)}</span><span class="mono">${row.rootId}</span></p><p class="work-detail" data-archived-by>${archivedWords(row.archived)}</p></div><div class="work-row-status">${act === null ? null : postForm(act.action, html `<button type="submit" class="secondary">${act.label}</button>`, { hidden: { return: returnTo }, attrs: { class: 'inline', 'data-tidy-act': 'restore' } })}</div></article>`;
+        }
         const target = row.primaryAction?.target;
         const actionHref = row.primaryAction?.code === 'open-result' && target?.runId != null
             ? `/review?result=${encodeURIComponent(target.taskId)}&run=${target.runId}`
             : browserWorkActionHref(row);
         const action = actionHref === null || row.primaryAction === null ? null : html `<a class="work-action" data-primary-action href="${actionHref}">${row.primaryAction.label} →</a>`;
         const project = data.multiProject || row.repo === null ? html `<span class="project-label">${row.repo === null ? 'Unplaced' : projectName(row.repo)}</span>` : null;
-        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}${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>`;
+        const act = tidyAct(row);
+        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>`;
     };
     const current = WORK_VIEWS.find(one => one.key === data.view);
     const filtered = (data.state ?? null) !== null;
@@ -653,6 +667,7 @@ export function workPage(chrome, data) {
     const pages = !data.previous && data.work.nextCursor === null ? null : html `<nav class="row work-pagination" aria-label="Task pages">${data.previous ? html `<a class="button-link" href="${href(data.view)}">First page</a>` : null}${data.work.nextCursor === null ? null : html `<a class="button-link" rel="next" href="${href(data.view, data.work.nextCursor)}">Next page</a>`}</nav>`;
     const tools = html `<details class="work-tools"><summary>Work tools${CHEVRON_ICON}</summary><nav class="work-tools-menu">${[['/inbox', 'Inbox'], ['/board', 'Board'], ['/board?view=order', 'Order'], ['/recipes', 'Recipes'], ...(chrome.projectScoped ? [] : [['/workbench', 'Portfolio']]), ['/ledger', 'Action ledger']]
         .map(([path, label]) => html `<a href="${path}">${label}</a>`)}</nav></details>`;
+    const putAwayLink = data.canPutAway === true && data.view !== 'archived' ? { label: 'Put away old tasks', href: '/work/put-away' } : null;
     const toolLinks = [['/inbox', 'Inbox'], ['/board', 'Board'], ['/board?view=order', 'Order'], ['/recipes', 'Recipes'], ...(chrome.projectScoped ? [] : [['/workbench', 'Portfolio']])];
     const view = {
         kind: 'tasks',
@@ -666,6 +681,15 @@ export function workPage(chrome, data) {
             const actionHref = row.primaryAction?.code === 'open-result' && target?.runId != null
                 ? `/review?result=${encodeURIComponent(target.taskId)}&run=${target.runId}`
                 : browserWorkActionHref(row);
+            const act = tidyAct(row);
+            const tidy = act === null ? null : { action: act.action, label: act.label, returnTo };
+            if (row.archived !== undefined)
+                return {
+                    id: row.rootId, title: row.title, href: taskHref(row.rootId),
+                    project: data.multiProject || row.repo === null ? (row.repo === null ? 'Unplaced' : projectName(row.repo)) : null,
+                    age: relativeAge(row.archived.at, data.now), status: { label: 'Archived', tone: 'muted', token: 'archived' },
+                    ask: null, chip: null, group: 'rest', action: null, detail: archivedWords(row.archived), problem: null, notes: [], tidy,
+                };
             const needsYouDetail = (row.status.views.includes('needs-you') || row.status.label === 'Failed') && !['write-scope', 'approve-scope'].includes(row.primaryAction?.code ?? '') && row.status.label !== 'Ready for review' && row.status.detail !== row.status.label && row.status.detail !== row.familyProblem;
             return {
                 id: row.rootId, title: row.title, href: taskHref(row.rootId),
@@ -677,6 +701,7 @@ export function workPage(chrome, data) {
                 detail: needsYouDetail ? row.status.detail : row.progress ?? null,
                 problem: row.familyProblem,
                 notes: (row.status.diagnostics ?? []).map(one => `${one.label} · ${one.detail}`),
+                tidy,
             };
         }),
         empty: data.work.items.length > 0 ? null : filtered && !data.previous ? { text: `No tasks are ${data.state} in this view.`, action: { label: "Show every state", href: href(data.view, undefined, null) } } : {
@@ -686,11 +711,12 @@ export function workPage(chrome, data) {
         pages: { first: data.previous ? href(data.view) : null, next: data.work.nextCursor === null ? null : href(data.view, data.work.nextCursor) },
         tools: toolLinks.map(([path, label]) => ({ label: label, href: path })),
         newTask: { label: 'New task', href: '/tasks/new' },
+        putAway: putAwayLink,
         limits: data.limits ?? null,
         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 }))],
     };
-    return screen('work', html `<div class="work-head"><h1>Tasks</h1>${tools}</div>${limitsHtml(data.limits ?? null)}${tabs}${stateLinks}${list}${pages}`, { chrome, workspace: { view } });
+    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)}${tabs}${stateLinks}${list}${pages}`, { chrome, workspace: { view } });
 }
 export function browsePage(chrome, data) {
     const crumb = data.at === data.root ? projectName(data.root) : `${projectName(data.root)}${data.at.slice(data.root.length)}`;
@@ -921,7 +947,10 @@ export function githubReposPage(chrome, data) {
         html `<p class="row" style="margin-top:.6rem"><a class="badge" href="/projects">← back to projects</a></p>`,
     ], "\n"), { chrome });
 }
-export function projectsPage(chrome, recent, candidates, open, problem, _unscopedMode, browsable = false, onboard = null, peeks = {}, returnTo = "/", pullRequestsOn, checkLevelOf) {
+export function projectsPage(chrome, recent, candidates, open, problem, _unscopedMode, browsable = false, onboard = null, peeks = {}, returnTo = "/", pullRequestsOn, checkLevelOf, activity) {
+    const activityHtml = activity === undefined ? null : html `<details><summary>Activity · ${activity.project}</summary>${activity.report === null
+        ? html `<p role="alert">Activity is unavailable.</p><a href="/projects">Reload</a>`
+        : html `<p>${activityScope(activity.report)}</p>${activity.report.truncated ? html `<p>${activityLimit(activity.report)}</p>` : null}${activityMetrics(activity.report).map(metric => html `<details><summary>${metric.label} · ${metric.value}</summary><p>${metric.detail}</p>${metric.rows.length === 0 ? html `<p>No tasks in this group.</p>` : html `<ul>${metric.rows.map(row => html `<li><a href="${row.href}">${row.title}</a> · Recorded checks: ${row.checks}</li>`)}</ul>`}</details>`)}`}</details>`;
     // The onboarding card (repo onboarding, findings 1-39): preview first,
     // then a password-confirmed clone into a configured root. Disabled
     // states explain themselves in words (finding 28/39).
@@ -1047,11 +1076,12 @@ export function projectsPage(chrome, recent, candidates, open, problem, _unscope
         recentItems.length === 0 && candidateItems.length === 0
             ? html `<div class="card"><p><strong>Nothing to open yet.</strong></p><p class="meta">Add one below — opening it registers it here for next time.</p></div>`
             : null,
+        activityHtml,
         recentItems.length > 0 ? html `<h2>Recent</h2>${cards(recentItems)}` : null,
         candidateItems.length > 0 ? html `<h2>Available</h2>${cards(candidateItems)}` : null,
         addCard,
     ], "\n"), { chrome, workspace: { view: {
-                kind: "projects", choosing, problem, returnTo,
+                kind: "projects", choosing, problem, returnTo, ...(activity === undefined ? {} : { activity }),
                 recent: recent.map(one => rowOf(one.path, one.name, one.lastOpenedAt)),
                 available: candidates.map(path => rowOf(path, projectName(path), null)),
                 add: { browse: browsable ? "/projects/browse" : null, github: onboard === null ? null : "/projects/github", html: htmlString(addForms) },
@@ -1315,3 +1345,21 @@ export function nextPage(chrome, data) {
     }
     return screen("next", joinHtml([header, card], "\n"), { chrome });
 }
+/** 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;
+    const daysArg = data.days === TIDY_DEFAULT_DAYS ? "" : ` --days ${data.days}`;
+    const crumbs = html `<nav class="kit-crumbs" aria-label="Breadcrumb"><ol><li><a href="/work">Tasks</a></li><li><span aria-current="page">Put away old tasks</span></li></ol></nav>`;
+    const daysForm = html `<form method="get" action="/work/put-away" class="put-away-days"><label>Also finished tasks untouched for <input type="number" name="days" min="1" max="365" value="${String(data.days)}" inputmode="numeric"> days</label><button type="submit" class="secondary">Update list</button></form>`;
+    const list = joinHtml(groups.map(group => html `<fieldset class="put-away-group" data-tidy-group="${group.rule}"><legend>${group.label}<span class="count">${String(group.items.length)}</span></legend><ul>${group.items.map(item => html `<li><label class="put-away-item" data-tidy-task="${item.taskId}"><input type="checkbox" name="task" value="${item.taskId}" checked><span class="put-away-title">${item.title}</span><span class="put-away-why">${item.reason}</span><span class="put-away-id">${item.taskId}${item.repo === null ? "" : ` · ${projectName(item.repo)}`}${item.members.length > 1 ? ` · ${item.members.length} versions` : ""}</span></label></li>`)}</ul></fieldset>`));
+    const cli = html `<p class="put-away-cli" data-tidy-cli>Or run: <code>toolroll task tidy --apply --digest ${data.digest}${daysArg}</code></p>`;
+    const body = count === 0
+        ? html `<p class="kit-state">Nothing to put away right now.</p><p><a href="/work?view=archived">See archived tasks →</a></p>`
+        : stepUpForm({ proof: "tasks-put-away", action: "/work/put-away", id: "put-away", hidden: { digest: data.digest, days: data.days },
+            body: html `${list}${cli}`, save: { label: "Put away selected", idle: `${count} ${count === 1 ? "task" : "tasks"} selected` }, confirm: `Put away the selected tasks. You can restore any of them from Archived.` });
+    return screen("Put away old tasks", html `<div class="put-away">${crumbs}<h1>Put away old tasks</h1>\
+<p class="kit-state">${count === 0 ? "Old releases, routine runs and fixed CI notices are put away automatically." : `${count} ${count === 1 ? "task" : "tasks"} can be put away. They leave every list and keep their history.`}</p>\
+${data.problem ? html `<p class="kit-problem" role="alert">${data.problem}</p>` : ""}${data.releasesSkipped === null ? "" : html `<p class="kit-hint">Release tasks were left out: ${data.releasesSkipped}.</p>`}${daysForm}${body}</div>`, { chrome });
+}

package/dist/server/render-results.d.ts

@@ -614,6 +614,8 @@ export type ResultDetail = {
     fileAnchors: ReadonlyMap<string, string>;
     comments: DiffComment[];
     pastComments?: DiffComment[];
+    /** Live saved notes whose screenshot would be refused now (result-actions.ts feedbackImageProblem). */
+    unusableImageNotes?: readonly number[];
     reviewerFindings: DiffComment[];
     /** The build's one automatic review, when its project had review on. */
     automaticReview?: BuildReviewView | null;

package/dist/server/render-results.js

@@ -1068,7 +1068,7 @@ export function proofBundleView(store, run, artifacts, root) {
         catch {
             problem = "the file could not be read";
         }
-        return [{ path, caption: captionFor(path), artifactId: artifact.id, problem }];
+        return [{ path, caption: captionFor(path), artifactId: artifact.id, sha256: artifact.sha256, problem }];
     });
     const stored = new Set(screenshots.map(one => one.path));
     const uncapturedScreenshots = (proof?.screenshots ?? []).map(one => one.path).filter(path => !stored.has(path));
@@ -1598,6 +1598,8 @@ export function resultPanelParts(detail, o) {
     const physicalReasons = receipt.reasons.filter(reason => !assessmentReasons.has(reason));
     const runId = run.id;
     const shots = proof?.screenshots ?? [];
+    const picture = (shot) => html `<figure class="result-image"><a class="receipt-shot" href="/r/${runId}/evidence/${shot.artifactId}"><img src="/r/${runId}/evidence/${shot.artifactId}" alt="${shot.caption}"><span>${shot.caption}</span></a>${detail.canAnnotate && o.csrf !== "" && shot.sha256 !== undefined
+        ? html `<button type="button" class="quiet" data-comment-image="${JSON.stringify({ run: runId, artifact: shot.artifactId, sha256: shot.sha256 })}" data-image-caption="${shot.caption}">Comment</button>` : ""}</figure>`;
     const shown = shots.filter(one => one.problem === null);
     const stat = terminal?.stat ?? null;
     const statOk = stat !== null && !("problem" in stat);
@@ -1694,7 +1696,7 @@ export function resultPanelParts(detail, o) {
     }
     else if (lead === "screenshots") {
         summaryParts.push(html `<div class="receipt-visuals result-visuals" aria-label="validated screenshots">${shown
-            .map(shot => html `<a class="receipt-shot" href="/r/${runId}/evidence/${shot.artifactId}"><img src="/r/${runId}/evidence/${shot.artifactId}" alt="${shot.caption}"><span>${shot.caption}</span></a>`)}</div>`);
+            .map(picture)}</div>`);
     }
     const unavailableShots = shots.filter(one => one.problem !== null);
     if (lead === "changes") {
@@ -1848,7 +1850,7 @@ export function resultPanelParts(detail, o) {
             ? directAssessment && !screenshotRequired ? html `` : html `<p class="meta" data-cockpit-source="screenshots">No screenshots${screenshotRequired ? " were captured, although one was required" : " were needed"}.</p>`
             : lead === "screenshots"
                 ? html `<p class="meta" data-cockpit-source="screenshots">${shown.length} validated screenshot${shown.length === 1 ? "" : "s"} shown in Summary${unavailableShots.length > 0 ? `; ${unavailableShots.length} unavailable` : ""}.</p>`
-                : html `<details class="cockpit-proof-group" data-cockpit-source="screenshots"><summary>Screenshots · ${shown.length}${unavailableShots.length > 0 ? ` (${unavailableShots.length} unavailable)` : ""}</summary><div class="receipt-visuals" aria-label="validated screenshots">${shown.map(shot => html `<a class="receipt-shot" href="/r/${runId}/evidence/${shot.artifactId}"><img src="/r/${runId}/evidence/${shot.artifactId}" alt="${shot.caption}"><span>${shot.caption}</span></a>`)}</div></details>`);
+                : html `<details class="cockpit-proof-group" data-cockpit-source="screenshots"><summary>Screenshots · ${shown.length}${unavailableShots.length > 0 ? ` (${unavailableShots.length} unavailable)` : ""}</summary><div class="receipt-visuals" aria-label="validated screenshots">${shown.map(picture)}</div></details>`);
         checkParts.push(receipt.caveats.length === 0
             ? directAssessment ? html `` : html `<p class="meta" data-cockpit-source="caveats">No caveats were reported.</p>`
             : html `<details class="cockpit-proof-group" data-cockpit-source="caveats"><summary>Agent caveats · ${receipt.caveats.length}</summary><ul>${receipt.caveats.map(one => html `<li>${one}</li>`)}</ul></details>`);
@@ -1863,19 +1865,29 @@ export function resultPanelParts(detail, o) {
         checkParts.push(o.extraChecks);
     // ---- request changes: both feedback styles, one sealed road ------------
     const requestParts = [html `<h3 class="so-sr-only">What should change?</h3>`];
+    const imageNote = (one) => {
+        if (one.image == null)
+            return html ``;
+        const shot = shots.find(shot => shot.artifactId === one.image.artifact && shot.sha256 === one.image.sha256 && one.image.run === runId && shot.problem === null);
+        // A live saved note whose screenshot no longer verifies can be kept without it, so it never blocks a revision.
+        const drop = o.csrf === "" || !(detail.unusableImageNotes ?? []).includes(one.id) ? html ``
+            : postForm(`/r/${runId}/comment`, html `<input type="hidden" name="intent" value="drop-image"><input type="hidden" name="comment" value="${String(one.id)}"><button type="submit" class="quiet">Keep note without screenshot</button>`, { attrs: { class: "feedback-image-drop" }, returnTo: o.returnTo, hidden: { tab: o.tab } });
+        return html `<figure class="feedback-image">${shot === undefined ? html `<figcaption>Screenshot ${one.image.artifact} is unavailable.</figcaption>`
+            : html `<a href="/r/${runId}/evidence/${shot.artifactId}"><img src="/r/${runId}/evidence/${shot.artifactId}" alt="${shot.caption}"></a><figcaption>${shot.caption} · Screenshot ${shot.artifactId}</figcaption>`}${drop}</figure>`;
+    };
     const pathWords = (path, line) => {
         const shownPath = `${path}${line === null ? "" : `:${line}`}`;
         const href = detail.editor === null ? null : editorFileHref(detail.editor.worktree, path, line);
         return href === null ? html `<span class="mono">${shownPath}</span> ` : html `<a class="mono" href="${href}">${shownPath}</a> `;
     };
     if (detail.comments.length > 0) {
-        requestParts.push(html `<p class="meta">Saved for later · ${detail.comments.length}</p><div class="diff-comments" data-result-notes="${detail.comments.length}">${detail.comments.map(one => html `<div class="diff-comment"><span class="diff-comment-pin" aria-hidden="true"></span><p>${one.path === null ? "" : pathWords(one.path, one.line)}${one.note}</p><span class="meta">${one.author} · ${whenTime(one.createdAt)}</span></div>`)}</div>`);
+        requestParts.push(html `<p class="meta">Saved for later · ${detail.comments.length}</p><div class="diff-comments" data-result-notes="${detail.comments.length}">${detail.comments.map(one => html `<div class="diff-comment">${imageNote(one)}<span class="diff-comment-pin" aria-hidden="true"></span><p>${one.path === null ? "" : pathWords(one.path, one.line)}${one.note}</p><span class="meta">${one.author} · ${whenTime(one.createdAt)}</span></div>`)}</div>`);
         if (o.csrf !== "" && !detail.canAnnotate) {
             requestParts.push(postForm(`/r/${runId}/revise`, html `${revisionSealFields(detail.comments, detail.sourceDigest)}<div><strong>${detail.comments.length} note${detail.comments.length === 1 ? "" : "s"} ready</strong></div><button type="submit">Request changes</button>`, { attrs: { class: "card revision-from-comments" }, returnTo: o.returnTo }));
         }
     }
     if ((detail.pastComments?.length ?? 0) > 0) {
-        requestParts.push(html `<details class="result-feedback-history"><summary>Earlier feedback</summary><div class="diff-comments" data-past-feedback>${detail.pastComments.map(one => html `<div class="diff-comment"><p>${one.path === null ? "" : pathWords(one.path, one.line)}${one.note}</p><span class="meta">${one.author} · ${whenTime(one.createdAt)}</span></div>`)}</div></details>`);
+        requestParts.push(html `<details class="result-feedback-history"><summary>Earlier feedback</summary><div class="diff-comments" data-past-feedback>${detail.pastComments.map(one => html `<div class="diff-comment">${imageNote(one)}<p>${one.path === null ? "" : pathWords(one.path, one.line)}${one.note}</p><span class="meta">${one.author} · ${whenTime(one.createdAt)}</span></div>`)}</div></details>`);
     }
     for (const revision of detail.revisions) {
         // The child's standing is the shared status projection's own words
@@ -1892,6 +1904,7 @@ export function resultPanelParts(detail, o) {
         requestParts.push(html `<details class="result-request-open result-request-form"${detail.comments.length > 0 ? " open" : ""}><summary>What should change?</summary>${postForm(`/r/${runId}/comment`, joinHtml([
             revisionSealFields(detail.comments, detail.sourceDigest),
             html `<input type="hidden" data-recorded-requests value="${[...detail.comments, ...(detail.pastComments ?? [])].filter(one => one.sourceKey?.startsWith(`review:${o.user}:`)).map(one => one.sourceKey.slice(`review:${o.user}:`.length)).join(",")}">`,
+            html `<input type="hidden" name="image" value=""><figure class="feedback-image" data-feedback-image hidden><img alt="Selected screenshot" hidden><figcaption></figcaption><button type="button" class="quiet" data-remove-image>Remove image</button></figure>`,
             html `<textarea name="note" rows="2" maxlength="${LIMITS.note}" placeholder="Describe the change…" aria-label="review comment" aria-describedby="comment-note-limit"${o.noted ? " autofocus" : ""}></textarea>`,
             html `<span class="meta diff-comment-limit" id="comment-note-limit">up to ${LIMITS.note} characters</span>`,
             html `<details class="result-pin"><summary>Attach to a file or line</summary>`,
@@ -1980,7 +1993,7 @@ export function personCheckItems(proof, patchText, shots, runId) {
         return {
             id: row.id, statement: row.statement, words: personCheckWords(row.statement), note: notes.length === 0 ? null : notes.join(" "),
             excerpts: files.filter(one => one.lines.length > 0).slice(0, 3),
-            shots: pictures.slice(0, 4).map(shot => ({ src: `/r/${runId}/evidence/${shot.artifactId}`, href: `/r/${runId}/evidence/${shot.artifactId}`, caption: shot.caption })),
+            shots: pictures.slice(0, 4).map(shot => ({ src: `/r/${runId}/evidence/${shot.artifactId}`, href: `/r/${runId}/evidence/${shot.artifactId}`, caption: shot.caption, ...(shot.sha256 === undefined ? {} : { image: { run: runId, artifact: shot.artifactId, sha256: shot.sha256 } }) })),
         };
     });
 }
@@ -2215,7 +2228,7 @@ export function runPage(chrome, run, taskId, artifacts, terminal = null, notes =
             noted,
             requestToken: result.requestToken,
             hrefFor: one => (one === "summary" ? `/r/${run.id}` : `/r/${run.id}?tab=${one}`),
-            returnTo: `/r/${run.id}`,
+            returnTo: `/r/${run.id}?record=1`,
             back: null,
         });
     return screen(`build #${run.id}`, joinHtml([

package/dist/server/render-tasks.d.ts

@@ -398,6 +398,7 @@ export type RevisionView = {
         line: number | null;
         note: string;
         author: string;
+        image?: import("../result-review.js").FeedbackImage | null;
     }[];
     /** The lineage and ACTUAL terms (contract handoff task 2), read from
      * the child's own rows — the same projection chat and the approval
@@ -437,6 +438,14 @@ export declare function taskBodyParts(data: {
     /** Markup the caller rendered (the version history). */
     history?: Html;
     versionLabel?: string | null;
+    /** Put away (v145): by whom and why; and whether the viewer can put it away or restore it (an approver's browser). */
+    archived?: {
+        by: string;
+        byKind: "person" | "tidy";
+        reason: string;
+        at: string;
+    } | null;
+    canPutAway?: boolean;
     status?: DisplayStatus;
     task: Task;
     /** Shared read-side lifecycle answer; the atomic claim still re-proves it. */

package/dist/server/render-tasks.js

@@ -1,4 +1,6 @@
 /** Task pages: approval, agents, plans, controls, decisions and the task body. */
+import { archivedWords } from "../tidy.js";
+import { whenUtc } from "../when-html.js";
 import { networkWords } from "../egress.js";
 import { releaseAsksAgainWords, releaseTermsSentence } from "../scope.js";
 import { RELEASE_CI_RUNNING_WORDS } from "../release-ci-hold.js";
@@ -50,7 +52,7 @@ import { reviewHref, taskChatHref, taskHref } from "./http.js";
 import { coordinatorProposalsSection } from "./render-chat.js";
 import { completionReceiptCard, criterionMatrixHtml, evidenceWords, homePhaseWords, incidentWords, needsCheck, pullRequestCardHtml, reasonWords, receiptStatusOf, repairChainHtml, reportItemsHtml, reviewRetryPanel, runNoun, runOutcomeBadge, semanticCoverageHtml, verificationExplanation, verificationRecovered } from "./render-results.js";
 import {} from "./session.js";
-import { linkWords, originNote } from "../task-source.js";
+import { linkWords, originNote, requestName, requestWords } from "../task-source.js";
 export function contractChangeHtml(change) {
     const tag = html `<span class="contract-change-kind contract-change-${change.kind}">${change.kind}</span>`;
     switch (change.field) {
@@ -203,7 +205,8 @@ const fragmentString = (fragment) => fragment === "" ? "" : htmlString(fragment)
 function originWords(origin) {
     const flow = origin.flow === null ? null : origin.flow === "hidden" ? "from a flow" : origin.flow === "removed" ? "from a flow (removed)" : `from flow “${origin.flow.name}”`;
     const source = origin.source === null ? null : `${origin.source.label} (${origin.source.proved})`;
-    return [flow, source].filter(one => one !== null).join(" · ");
+    const request = origin.request == null ? null : `from ${requestName(origin.request)}`;
+    return [flow, source, request].filter(one => one !== null).join(" · ");
 }
 /** v126: a task's origin as a fact: the flow, linked for those who may open it, and the outside event, linked to where
  * it lives, with that link's host always shown. */
@@ -219,6 +222,12 @@ function originParts(origin) {
         if (origin.source.url !== null)
             parts.push(" · ", { label: linkWords(origin.source.url), href: origin.source.url });
     }
+    if (origin.request != null) {
+        if (parts.length > 0)
+            parts.push(" · ");
+        const where = requestWords(origin.request);
+        parts.push(`${requestName(origin.request)}${where === "" ? "" : `: ${where}`}`);
+    }
     return parts;
 }
 export function approvalSheetHtml(input) {
@@ -294,7 +303,7 @@ export function approvalSheetHtml(input) {
     const revisionDetail = input.revision === null ? "" : detail("Earlier build", joinHtml([
         html `<div class="revision-card" data-revision-feedback>`,
         html `<p><a href="${input.revisionSourceHref}"${input.surface === "chat" ? html ` data-revision-source` : ""}>${input.surface === "chat" ? html `Original result: build #${input.revision.sourceRun} →` : html `build #${input.revision.sourceRun}`}</a></p>`,
-        input.revision.comments.length === 0 ? "" : html `<ul>${input.revision.comments.map(one => html `<li>${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : `:${one.line}`}</span> · `}${one.note} <span class="meta">— ${one.author}</span></li>`)}</ul>`,
+        input.revision.comments.length === 0 ? "" : html `<ul>${input.revision.comments.map(one => html `<li>${revisionImageLink(one.image)}${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : `:${one.line}`}</span> · `}${one.note} <span class="meta">— ${one.author}</span></li>`)}</ul>`,
         revisionLineageHtml(input.revision.lineage),
         html `</div>`,
     ]));
@@ -767,6 +776,8 @@ export function approveRefusalWords(reason) {
         return gateWords({ verdict: "refuse", reason });
     if (reason === "machine-proof")
         return "not approved: only a person can approve this. Approve it on the task page or from your paired chat";
+    if (reason === "archived")
+        return "not approved: this task is put away. Restore it first";
     if (reason === "unrouted")
         return "not approved: this scope predates agent routing and its old approval no longer stands — edit and re-file the scope so it is routed under today’s agents, then approve it";
     if (reason === "profile-unresolved")
@@ -1172,6 +1183,9 @@ export function executionPlanHtml(document, compact = false) {
         html `</div>`,
     ]);
 }
+function revisionImageLink(image) {
+    return image == null ? html `` : html `<a href="/r/${image.run}/evidence/${image.artifact}">Screenshot ${image.artifact} · result #${image.run}</a> · `;
+}
 /**
  * A revision's lineage and ACTUAL terms, in the same words on the task
  * page, the approval card, and chat (contract handoff task 2): which
@@ -1594,7 +1608,7 @@ export function taskBodyParts(data) {
             : joinHtml([
                 html `<div class="revision-card" data-revision-feedback>`,
                 html `<p><strong>${data.revision.kind === "ci-repair" ? "CI repair" : data.revision.kind === "criterion-repair" ? "Criterion repair" : "Revision feedback"}</strong> <span class="meta">from <a href="/r/${data.revision.sourceRun}">build #${data.revision.sourceRun}</a></span></p>`,
-                ...data.revision.comments.map(one => html `<p class="row"><span class="meta">${one.author}</span> ${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : `:${one.line}`}</span> `}${one.note}</p>`),
+                ...data.revision.comments.map(one => html `<p class="row"><span class="meta">${one.author}</span> ${revisionImageLink(one.image)}${one.path === null ? "" : html `<span class="mono">${one.path}${one.line === null ? "" : `:${one.line}`}</span> `}${one.note}</p>`),
                 revisionLineageHtml(data.revision.lineage),
                 html `</div>`,
             ], "\n");
@@ -2058,8 +2072,11 @@ export function taskBodyParts(data) {
         // The title is bare; the receipt or shared task status leads once.
         html `<div class="task-title-row"><h1 class="task-main-title">${data.rootTitle ?? task.title}</h1>${data.csrf === "" ? "" : taskViewSwitch(data.rootId ?? task.id, "overview")}</div>`,
         data.versionLabel == null ? "" : html `<p class="meta">${data.versionLabel} · <a href="${taskHref(data.rootId ?? task.id)}">Current work</a></p>`,
+        data.archived == null ? "" : html `<div class="task-archived" data-archived><p>${archivedWords(data.archived)} · ${whenUtc(data.archived.at)}</p>${data.canPutAway === true && data.csrf !== ""
+            ? postForm(`${taskHref(task.id)}/restore`, html `<button type="submit" class="secondary">Restore</button>`, { hidden: { return: taskHref(task.id) }, attrs: { class: "inline" } }) : ""}</div>`,
         (data.history ?? ""),
-        statusHtml,
+        // A put-away task's banner is its status: no Needs you card, no act, no approval form (each would be refused).
+        data.archived == null ? statusHtml : "",
         controlHtml,
         previousResult === null ? "" : html `<details class="task-previous-result"><summary>${previousResult.title}</summary>${previousResult.html}</details>`,
         progressCard,
@@ -2068,7 +2085,7 @@ export function taskBodyParts(data) {
         mirrorCard,
         problemHtml,
         needsScopeCard,
-        approvalHtml,
+        data.archived == null ? approvalHtml : "",
         html `<details class="task-status-details" id="task-diagnostics"${optionsOpen ? html ` open` : ""}><summary>Task options</summary>${optionsHtml}</details>`,
         // Evidence-first (M5.5): what needs you, then what happened — decisions
         // and incidents above the attempt ledger and spend, the mechanics
@@ -2170,7 +2187,7 @@ export function taskBodyParts(data) {
         const feedback = data.revision;
         thread.push({ key: "revision", at: task.createdAt, kind: "reply", who: "person", author: feedback.comments[0]?.author ?? "You",
             title: feedback.kind === "ci-repair" ? "Asked for a CI repair" : feedback.kind === "criterion-repair" ? "Asked for a repair" : "Asked for changes",
-            text: feedback.comments.map(one => `${one.path === null ? "" : `${one.path}${one.line === null ? "" : `:${one.line}`} — `}${one.note.trim()}`).filter(one => one !== "").join("\n") || null,
+            text: feedback.comments.map(one => `${one.image ? `Screenshot ${one.image.artifact} · result #${one.image.run} — ` : ""}${one.path === null ? "" : `${one.path}${one.line === null ? "" : `:${one.line}`} — `}${one.note.trim()}`).filter(one => one !== "").join("\n") || null,
             link: { label: `From build #${feedback.sourceRun}`, href: `/r/${feedback.sourceRun}` }, html: "", more: null });
     }
     for (const decision of data.decisions) {
@@ -2199,7 +2216,8 @@ export function taskBodyParts(data) {
     // The Details panel: the same facts, grouped and said once.
     const lastRunForAgent = liveRun ?? familyRuns.filter(run => run.role !== "reviewer").sort((a, b) => b.id - a.id)[0];
     const work = [];
-    work.push({ label: "Status", parts: [data.assignment != null && assignmentOptions !== null ? assignmentCardOf(data.assignment, assignmentOptions).status.headline : status.label] });
+    // A put-away task's one status is Archived (its banner says by whom and why).
+    work.push({ label: "Status", parts: [data.archived != null ? "Archived" : data.assignment != null && assignmentOptions !== null ? assignmentCardOf(data.assignment, assignmentOptions).status.headline : status.label] });
     if (data.repo !== null)
         work.push({ label: "Project", parts: [projectName(data.repo)] });
     if (data.base != null)
@@ -2310,6 +2328,7 @@ export function taskBodyParts(data) {
         ...(sectionIds.has("waits-for") ? [{ group: "work", label: "Dependencies", kind: "goto", tab: "plan", target: "waits-for" }] : []),
         ...(sectionIds.has("holds") ? [{ group: "work", label: "Holds", kind: "goto", tab: "plan", target: "holds" }] : []),
         ...(data.csrf !== "" ? [{ group: "work", label: "Task options", kind: "goto", tab: "plan", target: "task-diagnostics" }] : []),
+        ...(data.canPutAway === true && data.csrf !== "" && data.archived == null && !building ? [{ group: "work", label: "Put away", kind: "post", action: `${taskHref(task.id)}/archive`, fields: { return: taskHref(task.id) } }] : []),
         { group: "share", label: "Copy link", kind: "copy" },
         ...(facts.some(one => one.label === "Audit") ? [{ group: "share", label: "Evidence pack", kind: "link", href: `${taskHref(rootOf)}/evidence` }] : []),
         ...(data.repo !== null && data.scope !== null ? [{ group: "share", label: "Reuse as recipe", kind: "link", href: `/recipes/from-task?task=${encodeURIComponent(task.id)}` }] : []),
@@ -2326,10 +2345,12 @@ export function taskBodyParts(data) {
             { label: "Ask", href: taskChatHref(data.rootId ?? task.id), active: false },
         ],
         version: data.versionLabel == null ? null : { label: data.versionLabel, current: { label: "Current work", href: taskHref(data.rootId ?? task.id) } },
-        status: statusCard === null ? null : inPlaceChecks(failedCard ? { ...statusCard, action: null, status: { ...statusCard.status, sentence: data.failure?.line ?? statusCard.status.sentence } }
+        archived: data.archived == null ? null : { words: archivedWords(data.archived), at: data.archived.at,
+            restore: data.canPutAway === true && data.csrf !== "" ? { action: `${taskHref(task.id)}/restore`, returnTo: taskHref(task.id) } : null },
+        status: statusCard === null || data.archived != null ? null : inPlaceChecks(failedCard ? { ...statusCard, action: null, status: { ...statusCard.status, sentence: data.failure?.line ?? statusCard.status.sentence } }
             : progress !== null || record !== null ? { ...statusCard, action: statusCard.action?.href === record?.href ? null : statusCard.action, status: { ...statusCard.status, sentence: progress?.line ?? statusCard.status.sentence } }
                 : statusCard.action !== null && rebuild === null && bareSelfLink(statusCard.action.href) ? { ...statusCard, action: null } : statusCard),
-        statusHtml: htmlString(statusHtml),
+        statusHtml: data.archived != null ? "" : htmlString(statusHtml),
         failure: failedCard && data.failure != null ? { line: data.failure.line, evidence: data.failure.evidence, suggestion: data.failure.suggestion, link: data.failure.link } : null,
         retry,
         runChecks,
@@ -2342,7 +2363,7 @@ export function taskBodyParts(data) {
         ...(data.csrf !== "" && (data.steerRecent?.length ?? 0) > 0 ? { steerRecent: data.steerRecent } : {}),
         record,
         earlier,
-        approval: fragmentString(approvalHtml),
+        approval: data.archived != null ? "" : fragmentString(approvalHtml),
         approvalPlacement: approvalLeads ? "lead" : "plan",
         more,
         confirmStopped: data.csrf !== "" && data.assignment?.primaryAction?.code === "confirm-stopped" && data.assignment.primaryAction.target.runId !== null