prreviewbuddy 0.46.0 → 0.49.0

package/dist/server.js

@@ -1,4 +1,4 @@
-import { A as createPrState, An as writeAgentPreference, At as allJobs, B as toneOf, Bt as purposeOf, C as readEvents, Ct as forgeFor, D as announceJob, Dn as resolveModel, Dt as checkReview, E as telemetryUploadConsent, Et as checkCodeFreshness, F as markAllRead, Ft as saveJob, G as extractJsonText, H as withUsageRecorded, I as markRead, In as agentFor, J as readMarker, K as ReviewBeingDeletedError, Kt as indexReviewGroups, L as pruneNotifications, Lt as isLiveJob, M as eventName, Mn as STORE_ROOT, Mt as fail, N as headline, Nn as AGENT_IDS, Nt as isTerminal, O as branchOf, On as clearAgentPreference, Ot as followedBranch, P as listNotifications, Pt as loadJob, Q as compactQueue, Qt as mutateWorkspace, R as readNotification, Rn as AgentCancelledError, Rt as phasesFor, T as recordTelemetryUploadConsent, Tn as git, Tt as fillFileUrlTemplate, U as EXPLAIN_SIMPLY_PROMPT, Ut as isClaimed, V as unreadCount$1, W as asString, Wt as followedRefName, Xt as loadWorkspace, Y as removeWorktree, Yt as lineagePosition, Z as relativeTime$1, _ as agentEnvOf, _n as issuesOutstanding, _t as parseWorkspacePort, a as wasBlocked, an as touchWorkspace, bn as processDiscussion, bt as writePortPreference, c as RefreshUnavailableError, cn as dequeueOnComplete, d as refreshPrContext, dn as queueRemove, dt as BUILD_VERSION, en as previousKissResult, et as readIndexToken, f as isUnchanged, fn as queueStamp, gn as isQuestionOutstanding, gt as MIN_WORKSPACE_PORT, h as liveUpdateFor, hn as isIssueOutstanding, ht as MAX_WORKSPACE_PORT, i as liveJobsFor, it as readServerRecord, j as createPrUpdate, jt as endedAt, k as subjectOf, kn as readAgentPreference, kt as reviewedShaOf, l as mergeUpdateInto, ln as queueAdd, m as UpdateAlreadyRunningError, mn as doneVerb, mt as DEFAULT_WORKSPACE_PORT, n as deleteReview, nn as reviewedCommit, nt as clearServerRecord, o as discardJob, on as workspaceRevision, ot as serverEntryExists, p as askCheckout, pn as readQueue, pt as feedbackUrl, q as MANAGED_ROOT, qt as latestKissRun, r as lineageIds, s as FETCH_TIMEOUT_MS, sn as workspaceStamps, st as spawnServer, t as deleteLineage, tn as recentWorkspaces, u as updateReview, un as queueAddMany, ut as writeServerRecord, vn as questionsOutstanding, vt as resolveWorkspacePort, w as record, wn as isCommitReview, wt as remoteHost, x as runJob, xn as describeAuthorship, xt as claimRequestUrl, y as startKissJob, yn as reviewerDispositions, yt as statedWorkspacePort, z as removeNotification, zn as AgentUnavailableError, zt as progressSteps } from "./delete_review-lmVdoybL.js";
+import { $ as compactQueue, A as createPrState, An as processDiscussion, At as reviewedShaOf, B as toneOf, Bn as writeAgentPreference, Bt as progressSteps, C as readEvents, Cn as readQueue, D as announceJob, Dn as issuesOutstanding, Dt as checkCodeFreshness, E as telemetryUploadConsent, En as isQuestionOutstanding, Et as fillFileUrlTemplate, F as markAllRead, Fn as git, Ft as loadJob, G as extractJsonText, Gt as followedRefName, H as withUsageRecorded, Hn as STORE_ROOT, I as markRead, It as saveJob, J as readMarker, Jn as AgentCancelledError, Jt as indexReviewGroups, K as ReviewBeingDeletedError, Kn as agentFor, Kt as groupByLineage, L as pruneNotifications, Ln as resolveModel, M as eventName, Mt as endedAt, N as headline, Nt as fail, O as branchOf, On as questionsOutstanding, Ot as checkReview, P as listNotifications, Pn as isCommitReview, Pt as isTerminal, Q as activateQueued, Qt as loadWorkspace, R as readNotification, Rn as clearAgentPreference, Rt as isLiveJob, Sn as queueStamp, St as claimRequestUrl, T as recordTelemetryUploadConsent, Tn as isIssueOutstanding, Tt as remoteHost, U as EXPLAIN_SIMPLY_PROMPT, Un as AGENT_IDS, V as unreadCount$1, Vt as purposeOf, W as asString, Wt as isClaimed, Y as removeWorktree, Yn as AgentUnavailableError, Yt as latestKissRun, Z as relativeTime$1, Zt as lineagePosition, _ as agentEnvOf, _n as parseLifecycle, _t as MIN_WORKSPACE_PORT, a as wasBlocked, at as readServerRecord, bn as queueAddMany, bt as statedWorkspacePort, c as RefreshUnavailableError, cn as workspaceRevision, ct as spawnServer, d as refreshPrContext, dn as activityLabel, dt as writeServerRecord, en as mutateWorkspace, f as isUnchanged, fn as isLifecycle, ft as BUILD_VERSION, gn as offersLifecycle, gt as MAX_WORKSPACE_PORT, h as liveUpdateFor, hn as lifecyclePill, ht as DEFAULT_WORKSPACE_PORT, i as liveJobsFor, in as reviewedCommit, j as createPrUpdate, jn as describeAuthorship, jt as allJobs, k as subjectOf, kn as reviewerDispositions, kt as followedBranch, l as mergeUpdateInto, ln as workspaceStamps, m as UpdateAlreadyRunningError, mn as lifecycleChip, mt as feedbackUrl, n as deleteReview, nn as previousKissResult, o as discardJob, p as askCheckout, pn as lifecycleAction, q as MANAGED_ROOT, qt as indexKeptIds, r as lineageIds, rn as recentWorkspaces, rt as clearServerRecord, s as FETCH_TIMEOUT_MS, sn as touchWorkspace, st as serverEntryExists, t as deleteLineage, tt as readIndexToken, u as updateReview, un as LIFECYCLES, vn as sectionOf, vt as parseWorkspacePort, w as record, wn as doneVerb, wt as forgeFor, x as runJob, xn as queueRemove, xt as writePortPreference, y as startKissJob, yn as queueAdd, yt as resolveWorkspacePort, z as removeNotification, zn as readAgentPreference, zt as phasesFor } from "./delete_review-BBjXPfMV.js";
 import { basename, dirname, isAbsolute, join, relative, resolve } from "node:path";
 import { existsSync, mkdirSync, readFileSync, readdirSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { createHash, timingSafeEqual } from "node:crypto";
@@ -253,6 +253,14 @@ function saveItemLabel(saved) {
 	return saved ? "Remove from saved" : "Save review";
 }
 /**
+* The queue item, on a card and on the review's own page. "review queue" rather than "queue",
+* because a menu is where somebody meets the idea for the first time. Here because the browser
+* relabels it after a change, on both pages.
+*/
+function queueItemLabel(queued) {
+	return queued ? "Remove from review queue" : "Add to review queue";
+}
+/**
 * What `Continue reviewing` opens, and whether it exists at all.
 *
 * One rule rather than an expression in the template, because "is there anything to continue" and
@@ -4550,6 +4558,46 @@ function renderPrState(state, besideNumber = false) {
 	return `<span class="prb-card-pr-state" data-pr-state="${state}">${state === "merged" ? "Merged" : besideNumber ? "Closed" : "PR closed"}</span>`;
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/theme.ts
+/**
+* Which palette the workspace's furniture is drawn in: the reviewer's device, or a fixed choice.
+*
+* One axis with a registry rather than a flag per value, so the stored type, the settings page's
+* radios and the attribute on `<html>` all derive from the same list and cannot disagree about
+* what the choices are.
+*
+* Pure on purpose: the settings page's script imports this, and the client bundle must not reach
+* anything that pulls in a node builtin.
+*
+* The attribute is the whole mechanism. shell.css sets `color-scheme` from it, and every shell
+* token is a `light-dark()` pair that the browser resolves from `color-scheme`, so System follows
+* the device live, with no listener and no reload.
+*/
+var THEMES = [
+	{
+		id: "system",
+		label: "System",
+		help: "Follow your device setting"
+	},
+	{
+		id: "light",
+		label: "Light",
+		help: "Always use light mode"
+	},
+	{
+		id: "dark",
+		label: "Dark",
+		help: "Always use dark mode"
+	}
+];
+function isTheme(value) {
+	return THEMES.some((theme) => theme.id === value);
+}
+/** The `<html>` attribute, leading space included, so a root reads `<html lang="en"${…}>`. */
+function themeAttribute(theme) {
+	return ` data-theme="${theme ?? "system"}"`;
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/page.ts
 function renderWorkspace(view) {
 	const { session, changeSet } = view;
@@ -4569,7 +4617,7 @@ function renderWorkspace(view) {
 	*/
 	const freshness = result ? view.freshness : void 0;
 	return `<!doctype html>
-<html lang="en">
+<html lang="en"${themeAttribute(view.theme)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -4658,9 +4706,30 @@ function renderReviewBody(view, sections) {
     ${updating(view) ? renderUpdatingDoc(view) : renderReviewDoc(view, sections)}
   </main>
   ${result ? renderAsk(view, sections, updating(view)) : ""}
+  ${result ? renderAskTab() : ""}
 </div>`;
 }
 /**
+* Buddy AI's own way in: a tab on the window's right edge, where its pane opens.
+*
+* Out of the header, which says where the review stands; the assistant is a tool beside the review,
+* and the edge it lives on is the edge this sits on. Hidden by the stylesheet whenever the pane is
+* showing, where the chevron in the pane's own header closes it again. The pane's left border is
+* not free for it: the resize handle runs its full height.
+*
+* Horizontal, and only as wide as its words: the product reads left to right everywhere else, and
+* a vertical label was a pattern it would have used exactly once.
+*
+* Only with a standard review, like the pane. A control that opens nothing is worse than none.
+*/
+function renderAskTab() {
+	return `<button type="button" class="prb-ask-tab" data-ask-toggle aria-controls="prb-ask" aria-expanded="false">${ASK_TAB_ICON}<span class="prb-ask-tab-label">Buddy AI</span></button>`;
+}
+/** Four-pointed, stroked like the rail's icons, so it reads as one of the product's own marks. */
+var ASK_TAB_ICON = "<svg class=\"prb-ask-tab-icon\" viewBox=\"0 0 24 24\" width=\"14\" height=\"14\" focusable=\"false\" aria-hidden=\"true\" fill=\"none\" stroke=\"currentColor\" stroke-width=\"1.75\" stroke-linejoin=\"round\"><path d=\"M12 3l1.8 5.6a2 2 0 0 0 1.3 1.3L21 12l-5.9 2.1a2 2 0 0 0-1.3 1.3L12 21l-1.8-5.6a2 2 0 0 0-1.3-1.3L3 12l5.9-2.1a2 2 0 0 0 1.3-1.3z\"/></svg>";
+/** Two chevrons pointing where the pane goes. */
+var ASK_HIDE_ICON = "<svg viewBox=\"0 0 24 24\" width=\"15\" height=\"15\" focusable=\"false\" aria-hidden=\"true\" fill=\"none\" stroke=\"currentColor\" stroke-width=\"2\" stroke-linecap=\"round\" stroke-linejoin=\"round\"><path d=\"m6 17 5-5-5-5M13 17l5-5-5-5\"/></svg>";
+/**
 * The document while an update runs: its progress and nothing else.
 *
 * Deliberately not a `.prb-view`. Views are what the rail and the hash switch between, and a page
@@ -4967,42 +5036,69 @@ function renderReviewBar(view, freshness) {
     <span><span class="prb-add">+${additions}</span> <span class="prb-del">&minus;${deletions}</span></span>
     ${freshness || view.remote ? (() => {
 		const { label, state } = freshnessLabel(freshness ?? null, view.remote ?? null);
-		return `<span class="prb-fresh-status" data-fresh-status data-state="${state}">${escapeHtml$1(label)}</span>${view.token ? "<button type=\"button\" class=\"prb-fresh-check\" data-fresh-check>Check for updates</button>" : ""}`;
+		return `<span class="prb-fresh-status" data-fresh-status data-state="${state}">${escapeHtml$1(label)}</span>`;
 	})() : ""}
-    ${view.token ? `<button type="button" class="prb-complete" data-complete aria-pressed="${view.complete === true}">${view.complete ? "Reviewed" : "Mark reviewed"}</button>` : ""}
-    ${view.id ? renderIdMenu(view.id, Boolean(view.token), view.saved === true) : ""}
-    ${view.session.result ? "<button type=\"button\" class=\"prb-ask-toggle\" data-ask-toggle aria-expanded=\"false\">Buddy AI</button>" : ""}
+    ${view.token ? renderStatePills(view) : ""}
+    ${view.id ? renderIdMenu(view, freshness) : ""}
   </div>
 </div>`;
 }
 /**
-* This review's ID, where somebody looking at this review would think to find it.
+* Where this review stands, as pills that say and never do.
 *
-* In the header because the header answers "what am I looking at", and an ID is the most literal
-* possible answer to that: it is the name `prreviewbuddy review --review <review> --delete` takes.
-* It was on the index only, so the one page devoted to a single review was the only place that
-* could not tell you which review it was, and the help text's advice amounted to reading it out of
-* the address bar.
+* Two, because they answer two questions. Reviewed or In progress is what the reviewer has done;
+* Waiting or Done is what happens next with the branch. A review can be Reviewed and Waiting, so a
+* single pill would have to drop one of them. Active draws nothing, as an open pull request does
+* not: it is where every review starts, and a pill that never varies teaches nothing.
 *
-* The same `...` and the same menu markup as the index, deliberately. One gesture, in both places
-* it can be wanted, behaving identically. That includes deleting, with the same confirmation in the
-* same panel and the same sentence. The list is where you land afterwards, since the page you were
-* reading no longer exists. It deletes this review only, as a history row on the list does.
+* Never "New" here, because being on this page is what ends it.
+*/
+function renderStatePills(view) {
+	const status = view.complete ? "complete" : "in-progress";
+	const pill = lifecyclePill(view.lifecycle, view.activity);
+	return `<span class="prb-card-status" data-status="${status}" data-page-status>${statusLabel(status)}</span>
+    <span class="prb-card-status prb-page-lifecycle" data-page-lifecycle${pill?.changed ? " data-changed" : ""}${pill ? "" : " hidden"}>${escapeHtml$1(pill?.text ?? "")}</span>`;
+}
+/**
+* Everything that can be done to this review, from the page you are reading it on.
 *
-* No delete where there is no server, which is the offline preview: the item would be a lie there.
-* Save follows the same rule, since saving is a write to the store the preview does not have. It
-* is the index's star, reachable from the page you are reading, so its words are the star's
-* `aria-pressed` pair spelled out: what pressing it will do, never the state it is in.
+* The header only says where the review stands; this is where that is changed. It holds what the
+* Reviews list's card menu holds, in the same words from the same registries, so nobody has to go
+* back to the list to park or queue the review they are in. Check for updates leads because it is
+* the one thing here the list does not offer, and it is here rather than in the header because the
+* page already checks by itself, every minute while visible and on coming back to the tab.
+*
+* Waiting, Done and the queue belong to the branch, so they are offered on its newest review only,
+* as on the list, where the older reviews' rows carry none. The lineage notice already points at
+* the newest one.
+*
+* Delete keeps the index's confirmation, in the same panel with the same sentence. The list is
+* where you land afterwards, since the page you were reading no longer exists.
+*
+* Nothing that writes where there is no server, which is the offline preview: each item would be a
+* lie there. Copy review ID stays, because it is the name `prreviewbuddy review --review <review>
+* --delete` takes, and the page devoted to one review should be able to say which one it is.
 */
-function renderIdMenu(id, canWrite, saved) {
-	const safe = escapeHtml$1(id);
-	return `<div class="prb-menu" data-menu>
+function renderIdMenu(view, freshness) {
+	const safe = escapeHtml$1(view.id ?? "");
+	const canWrite = Boolean(view.token);
+	const latest = !view.lineage || view.lineage.position >= view.lineage.total;
+	const mark = view.complete ? "in-progress" : "complete";
+	const branchItems = canWrite && latest;
+	const items = [
+		canWrite && (freshness || view.remote) ? "<button type=\"button\" class=\"prb-menu-item\" data-fresh-check>Check for updates</button>" : "",
+		canWrite ? `<button type="button" class="prb-menu-item" data-mark-status="${mark}">${markItemLabel(mark)}</button>` : "",
+		...branchItems ? LIFECYCLES.map(({ id }) => `<button type="button" class="prb-menu-item" data-lifecycle-set="${id}"${offersLifecycle(id, view.lifecycle, view.activity) ? "" : " hidden"}>${escapeHtml$1(lifecycleAction(id))}</button>`) : [],
+		branchItems ? `<button type="button" class="prb-menu-item" data-queue-action="${view.queued ? "remove" : "add"}">${queueItemLabel(view.queued === true)}</button>` : "",
+		canWrite ? `<button type="button" class="prb-menu-item" data-save-toggle aria-pressed="${view.saved === true}">${saveItemLabel(view.saved === true)}</button>` : "",
+		`<button type="button" class="prb-menu-item" data-copy-id="${safe}">Copy review ID</button>`,
+		canWrite ? `<button type="button" class="prb-menu-item prb-menu-item-danger" data-delete="${safe}">Delete review</button>` : ""
+	].filter(Boolean);
+	return `<div class="prb-menu" data-menu data-review-menu${[view.lifecycle ? ` data-lifecycle-state="${view.lifecycle.state}" data-lifecycle-at="${view.lifecycle.at}"` : "", view.activity ? ` data-activity-at="${view.activity.at}" data-activity-kind="${view.activity.kind}"` : ""].join("")}>
     <button type="button" class="prb-menu-trigger" data-menu-trigger aria-expanded="false" aria-haspopup="true" aria-label="Actions for this review">&#8943;</button>
     <div class="prb-menu-panel" data-menu-panel hidden>
       <div data-menu-items>
-        ${canWrite ? `<button type="button" class="prb-menu-item" data-save-toggle aria-pressed="${saved}">${saveItemLabel(saved)}</button>` : ""}
-        <button type="button" class="prb-menu-item" data-copy-id="${safe}">Copy review ID</button>
-        ${canWrite ? `<button type="button" class="prb-menu-item prb-menu-item-danger" data-delete="${safe}">Delete review</button>` : ""}
+        ${items.join("\n        ")}
       </div>
       ${canWrite ? `<div class="prb-menu-confirm" data-menu-confirm hidden>
         <p class="prb-menu-question">Delete this review?</p>
@@ -5019,7 +5115,7 @@ function renderIdMenu(id, canWrite, saved) {
 * Everything below the review: where else to go, and what to do to the server.
 *
 * These are in the rail rather than the header because they are not about this review. The header
-* answers "what am I looking at" — branch, size, whether I am done with it, and Buddy AI — and
+* answers "what am I looking at" — branch, size, where the review stands — and
 * anything global sitting up there competed with that. Leaving the review is navigation, and it
 * belongs with the other navigation.
 *
@@ -5529,7 +5625,7 @@ function renderAsk(view, sections, paused = false) {
 	const pending = view.pendingAsk ?? null;
 	const pendingTurn = pending ? `<div class="prb-msg prb-msg-user">${escapeHtml$1(pending.question ?? "")}</div>
       <div class="prb-thinking" data-ask-pending><i></i><i></i><i></i><span>Reading the change…</span></div>` : "";
-	return `<aside class="prb-ask" aria-label="Ask Buddy AI">
+	return `<aside class="prb-ask" id="prb-ask" aria-label="Ask Buddy AI">
   <div
     class="prb-ask-resize"
     data-ask-resize
@@ -5542,6 +5638,7 @@ function renderAsk(view, sections, paused = false) {
     <h2>Ask Buddy AI</h2>
     <span class="prb-ask-scope" data-ask-scope>whole change</span>
     ${renderFocusToggle("buddy", "Buddy AI")}
+    <button type="button" class="prb-ask-hide" data-ask-toggle aria-controls="prb-ask" aria-label="Hide Buddy AI">${ASK_HIDE_ICON}</button>
   </div>
   ${paused ? "" : renderPromptBank(view.prompts ?? [], view.promptsCollapsed === true)}
   <div class="prb-ask-log" data-ask-log>
@@ -5718,7 +5815,8 @@ function normalise$1(entries) {
 		if (typeof raw.observedAt !== "number") continue;
 		states[key] = {
 			state: raw.state,
-			observedAt: raw.observedAt
+			observedAt: raw.observedAt,
+			...typeof raw.changedAt === "number" ? { changedAt: raw.changedAt } : {}
 		};
 	}
 	return states;
@@ -5834,6 +5932,7 @@ function normalise(entries) {
 		if (typeof raw.polledAt === "number") entry.polledAt = raw.polledAt;
 		if (typeof raw.observedSha === "string") entry.observedSha = raw.observedSha;
 		if (typeof raw.observedAt === "number") entry.observedAt = raw.observedAt;
+		if (typeof raw.changedAt === "number") entry.changedAt = raw.changedAt;
 		if (typeof raw.notifiedSha === "string") entry.notifiedSha = raw.notifiedSha;
 		if (typeof raw.notificationId === "string") entry.notificationId = raw.notificationId;
 		if (typeof raw.failures === "number") entry.failures = raw.failures;
@@ -6172,10 +6271,12 @@ async function watchCycle(deps = defaultDeps(), options = {}) {
 		}
 		if (observed.kind === "unresolved") continue;
 		if (observed.prState !== void 0 && !isBranchCandidate(candidate)) recordPrState(candidate, key, observed.sha, observed.prState, prStates, now);
+		const lastSeen = state[key]?.observedSha;
 		state[key] = {
 			...state[key],
 			observedSha: observed.sha,
 			observedAt: seenAt,
+			...lastSeen === void 0 || observed.sha === lastSeen ? {} : { changedAt: seenAt },
 			failures: 0
 		};
 		if (observed.sha === candidate.analysedSha) {
@@ -6220,7 +6321,8 @@ function prunedPrStates(prStates, deps) {
 */
 function recordPrState(candidate, key, headSha, observed, prStates, now) {
 	if (observed === null) return;
-	if (prStates[key]?.state === "open" && observed !== "open") {
+	const previous = prStates[key]?.state;
+	if (previous === "open" && observed !== "open") {
 		const id = `pr-${observed}-${candidate.reviewId}-${headSha}`;
 		if (createPrState({
 			id,
@@ -6232,9 +6334,11 @@ function recordPrState(candidate, key, headSha, observed, prStates, now) {
 			headSha
 		}, now) === null && readNotification(id) === null) return;
 	}
+	const changedAt = previous !== void 0 && previous !== observed ? now : prStates[key]?.changedAt;
 	prStates[key] = {
 		state: observed,
-		observedAt: now
+		observedAt: now,
+		...changedAt === void 0 ? {} : { changedAt }
 	};
 }
 async function observeRequest(candidate, deps, unreachable) {
@@ -6400,6 +6504,12 @@ async function announce(candidate, key, observed, state, deps, now) {
 * and a review with no `completedAt` is one they have not finished. Anything outside those is a
 * review they are done with, and polling it would spend a request on an answer nobody wants.
 *
+* **Waiting and Done are watched whatever else is true of them.** Parking a review takes it out of
+* the queue and is usually done after marking it reviewed, so without this a Waiting review would
+* fall outside all three sets at the moment it starts waiting for a push, and the push that should
+* bring it back to Active would never be seen. Done is watched for "Changed since done": the reviewer
+* asked not to be interrupted, not to be kept uninformed.
+*
 * The one exclusion is structural rather than editorial: a commit review is of one fixed commit
 * that cannot move, and `freshness.ts` calls that state `fixed` for the same reason. A review with
 * no pull request used to be excluded too, having nothing to ask a forge about. It is watched
@@ -6407,7 +6517,7 @@ async function announce(candidate, key, observed, state, deps, now) {
 */
 function isWatchable(workspace, queued) {
 	if (isCommitReview(workspace.changeSet)) return false;
-	return queued.has(workspace.id) || workspace.pinned === true || workspace.completedAt === void 0;
+	return queued.has(workspace.id) || workspace.pinned === true || workspace.completedAt === void 0 || parseLifecycle(workspace.lifecycle) !== void 0;
 }
 /** The request a review is attached to, by URL where there is one and by number otherwise. */
 function requestOf(workspace) {
@@ -6498,6 +6608,50 @@ function prStatesByRequest(states = readPrStates()) {
 	return new Map(Object.entries(states).map(([key, entry]) => [key, entry.state]));
 }
 /**
+* What last happened to each branch's change, by the id of its newest review: the list's order and
+* the time on each card.
+*
+* The latest of three things, each of which is the change moving rather than anybody looking at it:
+* the newest analysis being made, its head moving, and its pull request changing state. Read from
+* the two `changedAt`s, never the `observedAt`s, which every poll rewrites.
+*
+* A watch entry follows the newest analysis of its request, so it is looked for across the whole
+* history, the same way `moved` is: an entry still filed against an older analysis is the same
+* change. A head move or state change from before the newest analysis is already in it, so only a
+* later one counts.
+*/
+function reviewActivity(groups, watch = readWatch(), prStates = readPrStates()) {
+	const headChanges = /* @__PURE__ */ new Map();
+	for (const entry of Object.values(watch)) {
+		if (entry.reviewId === void 0 || entry.changedAt === void 0) continue;
+		headChanges.set(entry.reviewId, Math.max(entry.changedAt, headChanges.get(entry.reviewId) ?? 0));
+	}
+	const activity = /* @__PURE__ */ new Map();
+	for (const group of groups) {
+		const { latest } = group;
+		let current = {
+			at: latest.createdAt,
+			kind: "created"
+		};
+		for (const review of group.history) {
+			const at = headChanges.get(review.id);
+			if (at !== void 0 && at > current.at) current = {
+				at,
+				kind: "updated"
+			};
+		}
+		if (latest.prNumber !== void 0) {
+			const entry = prStates[requestKey(latest.repoPath, latest.prNumber)];
+			if (entry?.changedAt !== void 0 && entry.changedAt > current.at) current = {
+				at: entry.changedAt,
+				kind: entry.state === "open" ? "reopened" : entry.state
+			};
+		}
+		activity.set(latest.id, current);
+	}
+	return activity;
+}
+/**
 * One review's pull request state, for its page, filed under the same key the list looks up. Null
 * for a review with no request or one never observed: unknown, not open. Only ever what the watcher
 * last saw; nothing here reaches the forge.
@@ -6561,6 +6715,23 @@ function watchNow(scope, makeDeps = defaultDeps) {
 	});
 }
 /**
+* A first look at every one of these reviews, however many there are.
+*
+* For parking a selection as Waiting. `watchNow` keeps the request budget of an ordinary cycle, so
+* handed thirty reviews it would look at twenty and leave ten to the timer, and a push to one of
+* those before its turn would be taken as its first observation and never bring it back. So the
+* selection goes through in batches of that budget, one scoped cycle after another: every review is
+* looked at, and no single cycle asks more of the forge than an ordinary one does. The whole
+* selection is the reviewer's own act, which is what bounds how many batches there can be.
+*/
+async function watchAllNow(reviewIds, makeDeps = defaultDeps) {
+	const ids = [...reviewIds];
+	for (let start = 0; start < ids.length; start += 20) {
+		const batch = new Set(ids.slice(start, start + 20));
+		await watchNow((candidate) => batch.has(candidate.reviewId), makeDeps);
+	}
+}
+/**
 * Run the pass on a timer, and return the stop.
 *
 * The same shape as `startSweeping`, and tied into `cli/server.ts` the same way, so a restart
@@ -6600,14 +6771,26 @@ var FETCH_SELECTED_LABEL = "Fetch selected";
 //#region ../../packages/review-harness/src/workspace/reviews_page.ts
 function renderReviewsPage(view) {
 	const now = view.now ?? Date.now();
-	const ordered = [...view.groups].sort((a, b) => Number(b.latest.pinned) - Number(a.latest.pinned) || touchedAt(b.latest) - touchedAt(a.latest));
+	const activityOf = (group) => view.activity?.get(group.latest.id) ?? {
+		at: group.latest.createdAt,
+		kind: "created"
+	};
+	const ordered = [...view.groups].sort((a, b) => Number(b.latest.pinned) - Number(a.latest.pinned) || activityOf(b).at - activityOf(a).at);
+	const sections = {
+		active: [],
+		waiting: [],
+		done: []
+	};
+	for (const group of ordered) sections[sectionOf(group.latest.lifecycle, activityOf(group))].push(group);
+	const rank = new Map(ordered.map((group, index) => [group, index]));
+	const renderCards = (groups) => groups.map((group) => renderCard(group, now, activityOf(group), rank.get(group), view.analyses?.get(group.latest.id), positions.get(group.latest.id), group.history.map((review) => view.moved?.get(review.id)).find((kind) => kind !== void 0), prStateOf(group.latest, view.prStates))).join("");
 	const storedReviews = view.groups.reduce((total, group) => total + group.history.length, 0);
 	const queue = view.queue ?? [];
 	const positions = new Map(queue.map((id, position) => [id, position]));
 	const continues = continueTarget(queue);
 	const front = continues === null ? void 0 : findReview(view.groups, continues);
 	return `<!doctype html>
-<html lang="en">
+<html lang="en"${themeAttribute(view.theme)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -6634,13 +6817,15 @@ ${renderWorkspaceRail({
     <p class="prb-index-count" data-count>${escapeHtml$1(countLabel(storedReviews, ordered.length))}</p>
     <button type="button" class="prb-index-select" data-select-mode aria-pressed="false">Select</button>
   </div>`}
-  <div class="prb-cards" data-cards>
-    ${ordered.map((group) => renderCard(group, now, view.analyses?.get(group.latest.id), positions.get(group.latest.id), group.history.map((review) => view.moved?.get(review.id)).find((kind) => kind !== void 0), prStateOf(group.latest, view.prStates))).join("")}
+  <div class="prb-cards" data-cards="active">
+    ${renderCards(sections.active)}
   </div>
+  ${renderLoadMore("active")}
   ${ordered.length === 0 ? `<p class="prb-index-empty">No reviews stored yet. Ask Claude Code to review a change and one will appear here.</p>` : `<p class="prb-index-none" data-no-matches hidden>Nothing matches that.</p>`}
-  ${ordered.length > 15 ? renderPager() : ""}
+  ${renderSection("waiting", "Waiting", sections.waiting, true, renderCards)}
+  ${renderSection("done", "Done", sections.done, false, renderCards)}
   ${ordered.length === 0 ? "" : renderBulkBar()}
-  <p class="prb-index-note">Reviews are kept for thirty days, on this machine only. Saved reviews and anything in your review queue are kept for ninety days instead, counting from the last time you opened them. Every review has a fixed address, so a link to one is worth keeping and still opens after a restart. Stopping the server deletes nothing; it only stops anything being served until the next review starts it again.</p>
+  <p class="prb-index-note">Reviews are kept for thirty days, on this machine only. Saved reviews, Done reviews and anything in your review queue are kept for ninety days instead, counting from the last time you opened or marked them. Every review has a fixed address, so a link to one is worth keeping and still opens after a restart. Stopping the server deletes nothing; it only stops anything being served until the next review starts it again.</p>
 </main>
 </div>
 <script src="/assets/reviews.js" type="module"><\/script>
@@ -6721,22 +6906,31 @@ function findReview(groups, id) {
 	for (const group of groups) for (const review of group.history) if (review.id === id) return review;
 }
 /**
-* The pager, with its numbers left out.
+* One section's Load more, which the browser shows once that section holds more than it is showing.
 *
-* How many pages there are depends on what the search matched, and matching happens in the browser
-* against the whole list — so the numbers are the one part of this page the server cannot know.
-* Every word of it is still written here, where the house style tests read it; only the digits are
-* filled in later.
+* One per section rather than one pager for the page, so revealing older Done reviews never costs
+* the reviewer their place in Active. Hidden until the client has counted, so a browser with no
+* JavaScript gets every card and no control that does nothing.
+*/
+function renderLoadMore(section) {
+	return `<button type="button" class="prb-index-more" data-load-more="${section}" hidden>Load more</button>`;
+}
+/**
+* Waiting or Done, as a disclosure under the working list.
 *
-* Hidden until the client has counted, so a browser with no JavaScript gets the whole list and no
-* controls that do nothing, rather than an empty pager under it.
+* Active has no heading: it is what the page has always been, and the count line above it already
+* says what it holds. Waiting starts open, because it is work that will come back; Done starts
+* closed, because it is the archive. Rendered empty and hidden rather than left out, so a card the
+* browser moves here after a mark has somewhere to go.
 */
-function renderPager() {
-	return `<nav class="prb-index-pager" data-pager aria-label="Review pages" hidden>
-    <button type="button" class="prb-index-page" data-page-step="-1">&lsaquo; Previous</button>
-    <span class="prb-index-pages" data-pages></span>
-    <button type="button" class="prb-index-page" data-page-step="1">Next &rsaquo;</button>
-  </nav>`;
+function renderSection(section, title, groups, open, renderCards) {
+	return `<details class="prb-index-section" data-section="${section}"${open ? " open" : ""}${groups.length === 0 ? " hidden" : ""}>
+    <summary class="prb-index-section-head">${title} · <span data-section-count>${groups.length}</span></summary>
+    <div class="prb-cards" data-cards="${section}">
+      ${renderCards(groups)}
+    </div>
+    ${renderLoadMore(section)}
+  </details>`;
 }
 /**
 * The bulk bar, with its counts left out.
@@ -6761,6 +6955,7 @@ function renderBulkBar() {
       <button type="button" class="prb-bulk-button" data-bulk-queue disabled>Add to review queue</button>
       <button type="button" class="prb-bulk-button" data-bulk-mark="complete">Mark reviewed</button>
       <button type="button" class="prb-bulk-button" data-bulk-mark="in-progress">Mark in progress</button>
+      ${LIFECYCLES.map((lifecycle) => `<button type="button" class="prb-bulk-button" data-bulk-lifecycle="${lifecycle.id}">${escapeHtml$1(lifecycle.action)}</button>`).join("\n      ")}
       <button type="button" class="prb-bulk-button prb-bulk-button-danger" data-bulk-delete></button>
       <button type="button" class="prb-bulk-button prb-bulk-cancel" data-bulk-cancel>Cancel</button>
     </div>
@@ -6794,20 +6989,22 @@ function renderBulkBar() {
 * card and is shown only in selection mode, so entering that mode is a stylesheet change rather
 * than a rebuild of the list.
 */
-function renderCard(group, now, analysis, queued, moved, prState) {
+function renderCard(group, now, activity, rank, analysis, queued, moved, prState) {
 	const review = group.latest;
 	const status = statusOf(review);
+	const chip = lifecycleChip(review.lifecycle, activity);
 	const href = `/r/${encodeURIComponent(review.id)}?t=${encodeURIComponent(review.token)}`;
 	const followed = review.targetType === "commit" ? `commit ${(review.commitSha ?? review.branch).slice(0, 7)}` : followedRefName(review.branch, review.headResolvedRef) ?? review.branch;
-	return `<article class="prb-card" data-card data-id="${escapeHtml$1(review.id)}" data-status="${status}" data-pinned="${review.pinned}"${queued === void 0 ? "" : ` data-queue="${queued}"`}${prState === void 0 ? "" : ` data-pr-state="${prState}"`}${review.authorHandle ? ` data-author="${escapeHtml$1(review.authorHandle)}"` : ""} data-reviews="${group.history.length}" data-lineage="${escapeHtml$1(group.history.map((entry) => entry.id).join(" "))}" data-search-text="${escapeHtml$1(groupSearchText(group))}">
+	return `<article class="prb-card" data-card data-id="${escapeHtml$1(review.id)}" data-status="${status}" data-pinned="${review.pinned}"${queued === void 0 ? "" : ` data-queue="${queued}"`}${prState === void 0 ? "" : ` data-pr-state="${prState}"`}${review.authorHandle ? ` data-author="${escapeHtml$1(review.authorHandle)}"` : ""} data-lifecycle="${review.lifecycle?.state ?? "active"}"${review.lifecycle ? ` data-lifecycle-at="${review.lifecycle.at}"` : ""} data-activity-at="${activity.at}" data-activity-kind="${activity.kind}" data-rank="${rank}" data-reviews="${group.history.length}" data-lineage="${escapeHtml$1(group.history.map((entry) => entry.id).join(" "))}" data-search-text="${escapeHtml$1(groupSearchText(group))}">
     <input type="checkbox" class="prb-card-select" data-select="${escapeHtml$1(review.id)}" aria-label="Select ${escapeHtml$1(followed)}">
     <button type="button" class="prb-card-pin" data-pin="${escapeHtml$1(review.id)}" aria-pressed="${review.pinned}" aria-label="${review.pinned ? "Unsave" : "Save"} this review">${review.pinned ? "&#9733;" : "&#9734;"}</button>
-    ${renderCardMenu(group, followed, queued !== void 0, status)}
+    ${renderCardMenu(group, followed, queued !== void 0, status, activity)}
     <a class="prb-card-link" href="${escapeHtml$1(href)}">
       <div class="prb-card-head">
         <h2>${escapeHtml$1(review.title)}</h2>
         ${renderStatus(status)}
         ${renderPrState(prState)}
+        ${`<span class="prb-card-lifecycle" data-lifecycle-chip${chip === void 0 ? " hidden" : ""}>${escapeHtml$1(chip ?? "")}</span>`}
       </div>
       ${review.summary ? `<p class="prb-card-summary">${renderInlineText(review.summary)}</p>` : ""}
       <p class="prb-card-where">
@@ -6824,7 +7021,7 @@ function renderCard(group, now, analysis, queued, moved, prState) {
         <span class="prb-card-diffstat"><span class="prb-add">+${review.additions}</span> <span class="prb-del">&minus;${review.deletions}</span></span>
         ${analysis !== void 0 && !review.hasStandardReview ? "" : `<span class="prb-card-left">${escapeHtml$1(whatIsLeft(review))}</span>`}
         ${renderAnalysisState(analysis)}
-        <span class="prb-card-when"><time datetime="${new Date(touchedAt(review)).toISOString()}" title="${escapeHtml$1(new Date(touchedAt(review)).toLocaleString())}">${escapeHtml$1(whenLabel(review, now))}</time></span>
+        <span class="prb-card-when"><time datetime="${new Date(activity.at).toISOString()}" title="${escapeHtml$1(new Date(activity.at).toLocaleString())}">${escapeHtml$1(`${activityLabel(activity.kind)} ${relativeTime(activity.at, now)}`)}</time></span>
       </p>
     </a>
     ${renderHistory(group, now)}
@@ -6847,9 +7044,13 @@ function renderCard(group, now, analysis, queued, moved, prState) {
 *
 * The rows underneath carry the per-review menus, and they only exist once there is more than one.
 */
-function renderCardMenu(group, followed, queued, status) {
+function renderCardMenu(group, followed, queued, status, activity) {
 	const count = group.history.length;
 	const mark = status === "complete" ? "in-progress" : "complete";
+	const lifecycle = {
+		stored: group.latest.lifecycle,
+		activity
+	};
 	if (count < 2) return renderMenu({
 		id: group.latest.id,
 		scope: "review",
@@ -6858,7 +7059,8 @@ function renderCardMenu(group, followed, queued, status) {
 		question: "Delete this review?",
 		copy: "Copy review ID",
 		queue: queued ? "remove" : "add",
-		mark
+		mark,
+		lifecycle
 	});
 	const all = `all ${count} reviews following ${followed}`;
 	return renderMenu({
@@ -6869,6 +7071,7 @@ function renderCardMenu(group, followed, queued, status) {
 		question: `Delete ${all}?`,
 		queue: queued ? "remove" : "add",
 		mark,
+		lifecycle,
 		copy: "Copy latest review ID"
 	});
 }
@@ -6888,12 +7091,14 @@ function renderCardMenu(group, followed, queued, status) {
 function renderMenu(item) {
 	const id = escapeHtml$1(item.id);
 	const markItem = item.mark === void 0 ? "" : `<button type="button" class="prb-menu-item" data-mark-status="${item.mark}" data-mark-id="${id}">${markItemLabel(item.mark)}</button>`;
-	const queueItem = item.queue === void 0 ? "" : `<button type="button" class="prb-menu-item" data-queue-action="${item.queue}" data-queue-id="${id}">${item.queue === "remove" ? "Remove from review queue" : "Add to review queue"}</button>`;
+	const queueItem = item.queue === void 0 ? "" : `<button type="button" class="prb-menu-item" data-queue-action="${item.queue}" data-queue-id="${id}">${queueItemLabel(item.queue === "remove")}</button>`;
+	const lifecycleItems = item.lifecycle === void 0 ? "" : LIFECYCLES.map(({ id: target }) => `<button type="button" class="prb-menu-item" data-lifecycle-set="${target}" data-lifecycle-id="${id}"${offersLifecycle(target, item.lifecycle.stored, item.lifecycle.activity) ? "" : " hidden"}>${escapeHtml$1(lifecycleAction(target))}</button>`).join("\n        ");
 	return `<div class="prb-menu" data-menu>
     <button type="button" class="prb-menu-trigger" data-menu-trigger aria-expanded="false" aria-haspopup="true" aria-label="${escapeHtml$1(item.label)}">&#8943;</button>
     <div class="prb-menu-panel" data-menu-panel hidden>
       <div data-menu-items>
         ${markItem}
+        ${lifecycleItems}
         ${queueItem}
         <button type="button" class="prb-menu-item" data-copy-id="${id}">${escapeHtml$1(item.copy)}</button>
         <button type="button" class="prb-menu-item prb-menu-item-danger" data-delete="${id}" data-scope="${item.scope}">${escapeHtml$1(item.action)}</button>
@@ -7040,13 +7245,6 @@ function renderRisk(risk) {
 	if (risk === "unknown") return "";
 	return `<span class="prb-card-risk" data-risk="${risk}">${risk} risk</span>`;
 }
-/** Last opened where that is known, created otherwise. */
-function touchedAt(review) {
-	return review.lastViewedAt ?? review.createdAt;
-}
-function whenLabel(review, now) {
-	return review.lastViewedAt ? `Opened ${relativeTime(review.lastViewedAt, now)}` : `Created ${relativeTime(review.createdAt, now)}`;
-}
 /**
 * Everything the search field matches, flattened into one string on the card.
 *
@@ -7136,6 +7334,115 @@ function escapeHtml(value) {
 	return value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&#39;");
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/preferences.ts
+/**
+* Workspace preferences that belong to the reviewer rather than to a review.
+*
+* Kept server-side, in their own file. The original reason was that `localStorage` is keyed by
+* origin and this server took whatever port the OS offered on each start, so a restart would
+* silently discard a width the reviewer had deliberately chosen. That reason has gone: the port is
+* fixed, so the origin holds still and `localStorage` would now work. Moving this is a change worth
+* making on its own merits rather than in passing, and there is a second reason it stays for now:
+* `config.json` next door is hand-edited, and writing to a file someone maintains by hand reformats
+* it under them the first time they save, which is why this is not simply a key in there.
+*
+* Everything here is best-effort. A workspace that will not open because a preferences file is
+* unreadable would be a preference costing more than it is worth.
+*/
+var PREFERENCES_PATH = join(STORE_ROOT, "ui.json");
+var MAX_ASK_WIDTH = 1200;
+/** A key is a short identifier, never prose. Anything else in the file is not one of ours. */
+var NOTICE_KEY = /^[a-z-]+:[^\s]{1,200}$/;
+function readPreferences() {
+	try {
+		const parsed = JSON.parse(readFileSync(PREFERENCES_PATH, "utf8"));
+		if (!parsed || typeof parsed !== "object") return {};
+		const { askWidth, dismissedNotices, promptsCollapsed, changeMapRegister, theme } = parsed;
+		return {
+			...typeof askWidth === "number" && Number.isFinite(askWidth) ? { askWidth: clampAskWidth(askWidth) } : {},
+			...Array.isArray(dismissedNotices) ? { dismissedNotices: dismissedNotices.filter((key) => typeof key === "string" && NOTICE_KEY.test(key)) } : {},
+			...typeof promptsCollapsed === "boolean" ? { promptsCollapsed } : {},
+			...changeMapRegister === "simple" || changeMapRegister === "technical" ? { changeMapRegister } : {},
+			...isTheme(theme) ? { theme } : {}
+		};
+	} catch {
+		return {};
+	}
+}
+function writeAskWidth(width) {
+	if (!Number.isFinite(width)) return;
+	try {
+		mkdirSync(STORE_ROOT, { recursive: true });
+		const next = {
+			...readPreferences(),
+			askWidth: clampAskWidth(width)
+		};
+		writeFileSync(PREFERENCES_PATH, `${JSON.stringify(next, null, 2)}\n`);
+	} catch {}
+}
+/** Remember whether "Your prompts" is folded. Read-modify-write, like every other field here. */
+function writePromptsCollapsed(collapsed) {
+	try {
+		mkdirSync(STORE_ROOT, { recursive: true });
+		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
+			...readPreferences(),
+			promptsCollapsed: collapsed
+		}, null, 2)}\n`);
+	} catch {}
+}
+/** Remember which words the change map is read in. Read-modify-write, like every other field here. */
+function writeChangeMapRegister(register) {
+	try {
+		mkdirSync(STORE_ROOT, { recursive: true });
+		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
+			...readPreferences(),
+			changeMapRegister: register
+		}, null, 2)}\n`);
+	} catch {}
+}
+/** Remember which palette the workspace is drawn in. Read-modify-write, like every other field here. */
+function writeTheme(theme) {
+	try {
+		mkdirSync(STORE_ROOT, { recursive: true });
+		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
+			...readPreferences(),
+			theme
+		}, null, 2)}\n`);
+	} catch {}
+}
+/** Remember that the reviewer has read a notice. Refuses anything not shaped like a key. */
+function dismissNotice(key) {
+	if (!NOTICE_KEY.test(key)) return false;
+	try {
+		const current = readPreferences();
+		const dismissedNotices = [.../* @__PURE__ */ new Set([...current.dismissedNotices ?? [], key])];
+		mkdirSync(STORE_ROOT, { recursive: true });
+		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
+			...current,
+			dismissedNotices
+		}, null, 2)}\n`);
+		return true;
+	} catch {
+		return false;
+	}
+}
+/** Forget a stated width, so the stylesheet's proportional default applies again. */
+function clearAskWidth() {
+	try {
+		const { askWidth: _dropped, ...rest } = readPreferences();
+		mkdirSync(STORE_ROOT, { recursive: true });
+		writeFileSync(PREFERENCES_PATH, `${JSON.stringify(rest, null, 2)}\n`);
+	} catch {}
+}
+/**
+* Absolute bounds only. The upper limit the reviewer actually feels is half the viewport, which
+* the page enforces because it is the only party that knows how wide the window is — this is the
+* far cruder job of keeping a hand-edited or stale number from producing an unusable layout.
+*/
+function clampAskWidth(width) {
+	return Math.round(Math.min(MAX_ASK_WIDTH, Math.max(320, width)));
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/settings.ts
 /**
 * The settings the workspace can show and change, as one object rather than as three files.
@@ -7167,7 +7474,8 @@ function readSettings() {
 		},
 		agent: readAgentPreference(),
 		agents: AGENT_IDS,
-		usageCounts: telemetryUploadConsent()
+		usageCounts: telemetryUploadConsent(),
+		theme: readPreferences().theme ?? "system"
 	};
 }
 /** True when what the reader is looking at is served on a port they have since changed. */
@@ -7194,10 +7502,19 @@ function saveSettings(change, minPort, maxPort) {
 		settings: readSettings(),
 		error: `There is no agent called \`${change.agent}\`.`
 	};
+	if (change.theme !== void 0 && !isTheme(change.theme)) {
+		const labels = THEMES.map((theme) => theme.label);
+		const choices = `${labels.slice(0, -1).join(", ")} or ${labels.at(-1)}`;
+		return {
+			settings: readSettings(),
+			error: `There is no theme called \`${change.theme}\`. Choose ${choices}.`
+		};
+	}
 	if (port !== null) writePortPreference(port);
 	if (change.agent === "") clearAgentPreference();
 	else if (change.agent !== void 0) writeAgentPreference(change.agent);
 	if (change.usageCounts !== void 0) recordTelemetryUploadConsent(change.usageCounts);
+	if (change.theme !== void 0 && isTheme(change.theme)) writeTheme(change.theme);
 	return {
 		settings: readSettings(),
 		error: null
@@ -7209,7 +7526,7 @@ function renderSettingsPage(view) {
 	const { settings } = view;
 	const address = `http://localhost:${settings.port.configured}`;
 	return `<!doctype html>
-<html lang="en">
+<html lang="en"${themeAttribute(settings.theme)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -7231,6 +7548,7 @@ ${renderWorkspaceRail({
 		unread: view.unreadNotifications ?? 0
 	})}
 <main class="prb-index prb-settings">
+  ${renderAppearanceSection(settings)}
   ${renderWorkspaceSection(settings, address)}
   ${renderReviewsSection(settings)}
   ${renderPrivacySection(settings)}
@@ -7242,7 +7560,8 @@ ${renderWorkspaceRail({
 
   <p class="prb-index-note">These are kept in <code>~/.prreviewbuddy/config.json</code>, which you can
   edit by hand, and they are the same settings <code>prreviewbuddy config</code> reads and writes.
-  Run <code>prreviewbuddy config path</code> to be told where that file is.</p>
+  Run <code>prreviewbuddy config path</code> to be told where that file is. Appearance is kept with
+  the workspace's other display choices in <code>~/.prreviewbuddy/ui.json</code>.</p>
 
   ${renderServerSection()}
 </main>
@@ -7310,6 +7629,23 @@ function renderWorkspaceSection(settings, address) {
     working tomorrow.${moved ? ` <strong>You are reading this on port ${settings.port.current}.</strong> The port above is used the next time the workspace starts.` : ""}</p>
   </section>`;
 }
+/**
+* The theme, as three radios over the registry in theme.ts.
+*
+* Choosing one repaints this page at once (see `client/settings.ts`), so the reader can see what
+* they are picking; Save is still what keeps it, like every other field here.
+*/
+function renderAppearanceSection(settings) {
+	return `<section class="prb-settings-section">
+    <h2>Appearance</h2>
+    <div class="prb-field prb-settings-field" role="group" aria-labelledby="prb-theme-label">
+      <span id="prb-theme-label">Theme</span>
+      <div class="prb-settings-choice">
+        ${THEMES.map((theme) => `<label><input type="radio" name="prb-theme" value="${theme.id}" data-settings-theme${settings.theme === theme.id ? " checked" : ""}> ${theme.label} <small class="prb-settings-choice-help">${theme.help}</small></label>`).join("\n        ")}
+      </div>
+    </div>
+  </section>`;
+}
 function renderReviewsSection(settings) {
 	return `<section class="prb-settings-section">
     <h2>Reviews</h2>
@@ -7568,7 +7904,7 @@ function defaultFetchDeps() {
 function renderNotificationsPage(view) {
 	const unread = view.notifications.filter((notification) => notification.readAt === void 0).length;
 	return `<!doctype html>
-<html lang="en">
+<html lang="en"${themeAttribute(view.theme)}>
 <head>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
@@ -7765,6 +8101,8 @@ function reviewsRevision(live = analysisStates()) {
 	hash.update(`queue:${queueStamp()}\n`);
 	for (const [id, kind] of [...movedReviews()].sort(([a], [b]) => a.localeCompare(b))) hash.update(`moved:${id}:${kind}\n`);
 	for (const [key, state] of [...prStatesByRequest()].sort(([a], [b]) => a.localeCompare(b))) hash.update(`pr:${key}:${state}\n`);
+	for (const [key, entry] of Object.entries(readWatch()).sort(([a], [b]) => a.localeCompare(b))) if (entry.changedAt !== void 0) hash.update(`head:${key}:${entry.changedAt}\n`);
+	for (const [key, entry] of Object.entries(readPrStates()).sort(([a], [b]) => a.localeCompare(b))) if (entry.changedAt !== void 0) hash.update(`pr-changed:${key}:${entry.changedAt}\n`);
 	return hash.digest("base64url");
 }
 //#endregion
@@ -8643,104 +8981,6 @@ function readConfiguredEditor() {
 		return null;
 	}
 }
-//#endregion
-//#region ../../packages/review-harness/src/workspace/preferences.ts
-/**
-* Workspace preferences that belong to the reviewer rather than to a review.
-*
-* Kept server-side, in their own file. The original reason was that `localStorage` is keyed by
-* origin and this server took whatever port the OS offered on each start, so a restart would
-* silently discard a width the reviewer had deliberately chosen. That reason has gone: the port is
-* fixed, so the origin holds still and `localStorage` would now work. Moving this is a change worth
-* making on its own merits rather than in passing, and there is a second reason it stays for now:
-* `config.json` next door is hand-edited, and writing to a file someone maintains by hand reformats
-* it under them the first time they save, which is why this is not simply a key in there.
-*
-* Everything here is best-effort. A workspace that will not open because a preferences file is
-* unreadable would be a preference costing more than it is worth.
-*/
-var PREFERENCES_PATH = join(STORE_ROOT, "ui.json");
-var MAX_ASK_WIDTH = 1200;
-/** A key is a short identifier, never prose. Anything else in the file is not one of ours. */
-var NOTICE_KEY = /^[a-z-]+:[^\s]{1,200}$/;
-function readPreferences() {
-	try {
-		const parsed = JSON.parse(readFileSync(PREFERENCES_PATH, "utf8"));
-		if (!parsed || typeof parsed !== "object") return {};
-		const { askWidth, dismissedNotices, promptsCollapsed, changeMapRegister } = parsed;
-		return {
-			...typeof askWidth === "number" && Number.isFinite(askWidth) ? { askWidth: clampAskWidth(askWidth) } : {},
-			...Array.isArray(dismissedNotices) ? { dismissedNotices: dismissedNotices.filter((key) => typeof key === "string" && NOTICE_KEY.test(key)) } : {},
-			...typeof promptsCollapsed === "boolean" ? { promptsCollapsed } : {},
-			...changeMapRegister === "simple" || changeMapRegister === "technical" ? { changeMapRegister } : {}
-		};
-	} catch {
-		return {};
-	}
-}
-function writeAskWidth(width) {
-	if (!Number.isFinite(width)) return;
-	try {
-		mkdirSync(STORE_ROOT, { recursive: true });
-		const next = {
-			...readPreferences(),
-			askWidth: clampAskWidth(width)
-		};
-		writeFileSync(PREFERENCES_PATH, `${JSON.stringify(next, null, 2)}\n`);
-	} catch {}
-}
-/** Remember whether "Your prompts" is folded. Read-modify-write, like every other field here. */
-function writePromptsCollapsed(collapsed) {
-	try {
-		mkdirSync(STORE_ROOT, { recursive: true });
-		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
-			...readPreferences(),
-			promptsCollapsed: collapsed
-		}, null, 2)}\n`);
-	} catch {}
-}
-/** Remember which words the change map is read in. Read-modify-write, like every other field here. */
-function writeChangeMapRegister(register) {
-	try {
-		mkdirSync(STORE_ROOT, { recursive: true });
-		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
-			...readPreferences(),
-			changeMapRegister: register
-		}, null, 2)}\n`);
-	} catch {}
-}
-/** Remember that the reviewer has read a notice. Refuses anything not shaped like a key. */
-function dismissNotice(key) {
-	if (!NOTICE_KEY.test(key)) return false;
-	try {
-		const current = readPreferences();
-		const dismissedNotices = [.../* @__PURE__ */ new Set([...current.dismissedNotices ?? [], key])];
-		mkdirSync(STORE_ROOT, { recursive: true });
-		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
-			...current,
-			dismissedNotices
-		}, null, 2)}\n`);
-		return true;
-	} catch {
-		return false;
-	}
-}
-/** Forget a stated width, so the stylesheet's proportional default applies again. */
-function clearAskWidth() {
-	try {
-		const { askWidth: _dropped, ...rest } = readPreferences();
-		mkdirSync(STORE_ROOT, { recursive: true });
-		writeFileSync(PREFERENCES_PATH, `${JSON.stringify(rest, null, 2)}\n`);
-	} catch {}
-}
-/**
-* Absolute bounds only. The upper limit the reviewer actually feels is half the viewport, which
-* the page enforces because it is the only party that knows how wide the window is — this is the
-* far cruder job of keeping a hand-edited or stale number from producing an unusable layout.
-*/
-function clampAskWidth(width) {
-	return Math.round(Math.min(MAX_ASK_WIDTH, Math.max(320, width)));
-}
 /** Long enough for a paragraph of standing instructions, short enough not to crowd the question. */
 var MAX_PROMPT_CHARS = 2e3;
 /**
@@ -9029,6 +9269,7 @@ async function handle(req, res, options) {
 	if (url.pathname === "/queue") return handleQueue(req, res, options.indexToken);
 	if (url.pathname === "/delete") return handleDelete(req, res, options.indexToken);
 	if (url.pathname === "/status") return handleStatus(req, res, options.indexToken);
+	if (url.pathname === "/lifecycle") return handleLifecycle(req, res, options.indexToken, options.establishBaseline ?? establishBaseline);
 	if (url.pathname === "/revision") return handleRevision(req, res, options.indexToken);
 	if (url.pathname === "/fetch-updates") return handleFetchUpdates(req, res, options.indexToken);
 	if (url.pathname === "/shutdown") return handleIndexShutdown(req, res, options);
@@ -9055,6 +9296,8 @@ async function handle(req, res, options) {
 	if (action === "/stop") return handleStop(req, res, workspace);
 	if (action === "/kiss") return handleKiss(req, res, workspace);
 	if (action === "/state") return handleState(req, res, workspace);
+	if (action === "/lifecycle") return handleOwnLifecycle(req, res, workspace, options.establishBaseline ?? establishBaseline);
+	if (action === "/queue") return handleOwnQueue(req, res, workspace);
 	if (action === "/delete") return handleDeleteOwn(req, res, workspace);
 	if (action === "/event") return handleEvent(req, res, workspace);
 	if (action === "/prefs") return handlePrefs(req, res);
@@ -9119,9 +9362,11 @@ function handleIndex(req, res, params, indexToken) {
 	if (onward) return redirect(res, onward, cookie);
 	const states = analysisStates();
 	const queue = readQueue();
-	const kept = new Set(queue);
-	for (const review of recentWorkspaces(Number.MAX_SAFE_INTEGER)) if (review.pinned) kept.add(review.id);
-	const groups = indexReviewGroups(MAX_LISTED_BRANCHES, MAX_KEPT_BRANCHES, kept);
+	const reviews = recentWorkspaces(Number.MAX_SAFE_INTEGER);
+	const kept = indexKeptIds(reviews, queue);
+	const everyGroup = groupByLineage(reviews);
+	const activity = reviewActivity(everyGroup);
+	const groups = indexReviewGroups(MAX_LISTED_BRANCHES, MAX_KEPT_BRANCHES, kept, (group) => activity.get(group.latest.id)?.at ?? group.latest.createdAt, everyGroup);
 	const rendered = new Set(groups.flatMap((group) => group.history.map((review) => review.id)));
 	sendHtml(res, 200, renderReviewsPage({
 		groups,
@@ -9129,9 +9374,11 @@ function handleIndex(req, res, params, indexToken) {
 		analyses: states,
 		moved: movedReviews(),
 		prStates: prStatesByRequest(),
+		activity,
 		revision: reviewsRevision(states),
 		unreadNotifications: unreadCount$1(),
-		driving: isDriving()
+		driving: isDriving(),
+		theme: readPreferences().theme
 	}), cookie);
 }
 /**
@@ -9203,7 +9450,8 @@ async function handleSettings(req, res, params, indexToken) {
 		const { settings, error } = saveSettings({
 			...typeof body?.port === "string" ? { port: body.port } : {},
 			...typeof body?.agent === "string" ? { agent: body.agent } : {},
-			...typeof body?.usageCounts === "boolean" ? { usageCounts: body.usageCounts } : {}
+			...typeof body?.usageCounts === "boolean" ? { usageCounts: body.usageCounts } : {},
+			...typeof body?.theme === "string" ? { theme: body.theme } : {}
 		}, MIN_WORKSPACE_PORT, MAX_WORKSPACE_PORT);
 		return send(res, 200, {
 			port: {
@@ -9212,6 +9460,7 @@ async function handleSettings(req, res, params, indexToken) {
 			},
 			agent: settings.agent,
 			usageCounts: settings.usageCounts,
+			theme: settings.theme,
 			restartRequired: restartRequired(settings),
 			error
 		});
@@ -9303,7 +9552,8 @@ async function handleNotifications(req, res, params, indexToken) {
 		notifications,
 		gone: new Set(notifications.filter((entry) => !loadWorkspace(entry.reviewId)).map((entry) => entry.id)),
 		unreadNotifications: unreadCount$1(),
-		driving: isDriving()
+		driving: isDriving(),
+		theme: readPreferences().theme
 	}), indexCookieHeader(indexToken));
 }
 /**
@@ -9354,7 +9604,6 @@ async function handleStatus(req, res, indexToken) {
 			current.completedAt = complete ? now : void 0;
 			if (!complete) current.lastViewedAt = current.lastViewedAt ?? now;
 		})) continue;
-		dequeueOnComplete(id, complete);
 		changed += 1;
 	}
 	send(res, 200, {
@@ -9363,6 +9612,156 @@ async function handleStatus(req, res, indexToken) {
 	});
 }
 /**
+* Put a selection in Active, Waiting or Done.
+*
+* The same shape and the same credential as `handleStatus` beside it, for its reasons: this is the
+* list acting on itself, and a lifecycle is a note about what happens next, not a change to the
+* analysis, so nothing in flight is a reason to refuse it.
+*
+* **A state that is stated and unknown is refused whole**, naming it and what would have worked.
+* Guessing (reading `archived` as Done, say) would put cards where the reviewer did not ask for
+* them, and skipping it would answer 200 to a request that did nothing.
+*
+* Waiting and Done both leave the queue: it holds work somebody means to do next, and either mark
+* says there is none. Active does not put anything back in, as un-marking a review does not.
+*
+* Marking Waiting again writes a new `at`, which is what settles a Waiting review that changed: the
+* reviewer has seen the change and parked it again.
+*/
+async function handleLifecycle(req, res, indexToken, baseline) {
+	if (!indexToken || !secretMatches(indexToken, indexCookie(req))) return send(res, 403, { error: "Not authorised." });
+	const body = await readJson(req);
+	const state = body?.state;
+	if (!isLifecycle(state)) {
+		const choices = LIFECYCLES.map((lifecycle) => lifecycle.id).join(", ");
+		return send(res, 400, { error: `There is no lifecycle state called \`${String(state)}\`. Choose one of ${choices}.` });
+	}
+	const ids = Array.isArray(body?.ids) ? body.ids.filter((id) => typeof id === "string") : [];
+	const { changed, at } = applyLifecycle(ids, state);
+	send(res, 200, {
+		ok: true,
+		changed,
+		at,
+		queue: readQueue()
+	});
+	baselineParked(ids, state, baseline);
+}
+/**
+* The write behind every lifecycle mark, from the list or from the review's own page. Waiting and
+* Done dequeue; see `handleLifecycle`.
+*/
+function applyLifecycle(ids, state) {
+	const at = Date.now();
+	let changed = 0;
+	for (const id of ids) {
+		if (!mutateWorkspace(id, (current) => {
+			current.lifecycle = state === "active" ? void 0 : {
+				state,
+				at
+			};
+		})) continue;
+		if (state !== "active") queueRemove(id);
+		changed += 1;
+	}
+	return {
+		changed,
+		at
+	};
+}
+/** After the answer is sent, so a first look at the forge never holds up the page that asked. */
+function baselineParked(ids, state, baseline) {
+	if (state !== "waiting") return;
+	const unseen = unobserved(ids);
+	if (unseen.size > 0) baseline(unseen);
+}
+/**
+* Lifecycle and queue, asked by the review's own page on its own token.
+*
+* Both belong to the branch, and the list only ever reads them from the newest review of it, so a
+* mark written on an older one would be a write nobody sees. The page offers neither there; this
+* says why to anything that asks anyway.
+*/
+function refuseUnlessLatest(res, workspace) {
+	const { position, total } = lineagePosition(workspace);
+	if (position >= total) return false;
+	send(res, 409, { error: "A newer review of this branch exists. Waiting, Done and the review queue are set on the newest one." });
+	return true;
+}
+/**
+* What the Reviews list would say about this branch: the newest analysis's creation, a push the
+* watcher saw, or a change of pull request state. Reads every record, so it is asked only where
+* it can change what is drawn.
+*/
+function branchActivity(workspace, all = recentWorkspaces(Number.MAX_SAFE_INTEGER)) {
+	const group = groupByLineage(all).find((candidate) => candidate.history.some((review) => review.id === workspace.id));
+	return group ? reviewActivity([group]).get(group.latest.id) : void 0;
+}
+/**
+* Where the review stands, for the page's own checks to repaint from. Another tab can mark or park
+* it, and the watcher can see the branch move while the page is open.
+*
+* Activity only for a parked review: it decides nothing for an Active one, and it costs a read of
+* every record (measured 27 ms warm, 525 ms cold, on 66 reviews), once a minute per open tab.
+*/
+function pageStanding(workspace) {
+	const lifecycle = parseLifecycle(workspace.lifecycle) ?? null;
+	return {
+		complete: workspace.completedAt !== void 0,
+		lifecycle,
+		queued: readQueue().includes(workspace.id),
+		activity: lifecycle ? branchActivity(workspace) ?? null : null
+	};
+}
+/** Where this review stands after a lifecycle or queue change, which is everything the page repaints. */
+function ownStanding(id) {
+	return {
+		lifecycle: parseLifecycle(loadWorkspace(id)?.lifecycle) ?? null,
+		queued: readQueue().includes(id)
+	};
+}
+async function handleOwnLifecycle(req, res, workspace, baseline) {
+	const state = (await readJson(req))?.state;
+	if (!isLifecycle(state)) {
+		const choices = LIFECYCLES.map((lifecycle) => lifecycle.id).join(", ");
+		return send(res, 400, { error: `There is no lifecycle state called \`${String(state)}\`. Choose one of ${choices}.` });
+	}
+	if (refuseUnlessLatest(res, workspace)) return;
+	applyLifecycle([workspace.id], state);
+	send(res, 200, {
+		ok: true,
+		...ownStanding(workspace.id)
+	});
+	baselineParked([workspace.id], state, baseline);
+}
+async function handleOwnQueue(req, res, workspace) {
+	const action = (await readJson(req))?.action;
+	if (action !== "add" && action !== "remove") return send(res, 400, { error: "Unknown queue action." });
+	if (refuseUnlessLatest(res, workspace)) return;
+	const stored = applyQueue(workspace.id, action);
+	if (stored === null) return send(res, 500, { error: "The queue could not be saved." });
+	if (action === "add" && !stored.includes(workspace.id)) return send(res, 409, { error: `The review queue is full. It holds 50 reviews at most.` });
+	send(res, 200, ownStanding(workspace.id));
+}
+/**
+* Which of these reviews the watcher has never seen the head of.
+*
+* Those are the ones Waiting cannot yet work for: a first observation is not a change (it says
+* nothing about when the head moved), so a push before it would be absorbed as the baseline. A
+* review already observed has its baseline, and polling it again now would only spend a request.
+*/
+function unobserved(ids) {
+	const seen = new Set(Object.values(readWatch()).filter((entry) => entry.observedSha !== void 0).map((entry) => entry.reviewId));
+	return new Set(ids.filter((id) => !seen.has(id)));
+}
+/**
+* The first look itself, by the watcher's own rules. A review that cannot be observed (the forge is
+* unreachable, or it is a commit review with nothing to watch) stays Waiting with no baseline, and
+* no change is invented for it: the next successful poll sets one.
+*/
+function establishBaseline(reviewIds) {
+	watchAllNow(reviewIds).catch(() => {});
+}
+/**
 * Has anything the reviews index draws changed, and is anything still being made.
 *
 * The whole of the index's liveness, in one request that costs the same whether the machine holds
@@ -9454,10 +9853,17 @@ async function handleQueue(req, res, indexToken) {
 		return addMany(res, body.ids.filter((one) => typeof one === "string"));
 	}
 	if (!id || !loadWorkspace(id)) return send(res, 404, { error: "No such review" });
+	const stored = applyQueue(id, action);
+	if (stored === null) return send(res, 500, { error: "The queue could not be saved." });
+	send(res, 200, { queue: stored });
+}
+/** One review in or out of the queue, from a card or from its own page. The queue as stored, or null. */
+function applyQueue(id, action) {
 	if (action === "add") compactQueue(readQueue());
 	const stored = action === "add" ? queueAdd(id) : queueRemove(id);
-	if (stored === null) return send(res, 500, { error: "The queue could not be saved." });
-	send(res, 200, { queue: compactQueue(stored) });
+	if (stored === null) return null;
+	if (action === "add") activateQueued([id], stored);
+	return compactQueue(stored);
 }
 /**
 * Adding a selection, and saying which of it did not go in.
@@ -9473,6 +9879,7 @@ function addMany(res, ids) {
 	compactQueue(readQueue());
 	const outcome = queueAddMany(known);
 	if (outcome === null) return send(res, 500, { error: "The queue could not be saved." });
+	activateQueued(known, outcome.queue);
 	const refused = [...outcome.full.map((one) => ({
 		id: one,
 		error: `The review queue is full. It holds 50 reviews at most.`
@@ -9698,6 +10105,7 @@ async function sendWorkspacePage(res, workspace) {
 	if (notice) mutateWorkspace(workspace.id, (current) => {
 		if (current.lastUpdate?.at === notice.at) current.lastUpdate.acknowledged = true;
 	});
+	const everyReview = recentWorkspaces(Number.MAX_SAFE_INTEGER);
 	const html = renderWorkspace({
 		session: workspace.session,
 		changeSet: workspace.changeSet,
@@ -9709,6 +10117,7 @@ async function sendWorkspacePage(res, workspace) {
 		editor: editorChoice(),
 		repoPath: workspace.repoPath,
 		askWidth: readPreferences().askWidth,
+		theme: readPreferences().theme,
 		dismissedNotices: readPreferences().dismissedNotices,
 		promptsCollapsed: readPreferences().promptsCollapsed === true,
 		changeMapRegister: readPreferences().changeMapRegister ?? "simple",
@@ -9735,7 +10144,10 @@ async function sendWorkspacePage(res, workspace) {
 		changeMap: workspace.changeMap,
 		reviewedSha: reviewedCommit(workspace),
 		revision: workspaceRevision(workspace),
-		lineage: lineagePosition(workspace),
+		lineage: lineagePosition(workspace, everyReview),
+		lifecycle: parseLifecycle(workspace.lifecycle),
+		activity: branchActivity(workspace, everyReview),
+		queued: readQueue().includes(workspace.id),
 		job: workspace.jobId ? loadJob(workspace.jobId) : void 0,
 		updateJob: liveUpdateFor(workspace.id),
 		kissRun,
@@ -9855,7 +10267,8 @@ async function handleFreshness(res, workspace, quiet) {
 		prState,
 		kiss,
 		kissDismissedSha: workspace.dismissedKissFreshnessSha ?? null,
-		revision
+		revision,
+		standing: pageStanding(workspace)
 	});
 	if (!quiet) record({
 		event: "freshness_checked",
@@ -9872,7 +10285,8 @@ async function handleFreshness(res, workspace, quiet) {
 		dismissedSha: workspace.dismissedFreshnessSha ?? null,
 		kiss,
 		kissDismissedSha: workspace.dismissedKissFreshnessSha ?? null,
-		revision
+		revision,
+		standing: pageStanding(workspace)
 	});
 }
 /**
@@ -10330,7 +10744,8 @@ async function handleState(req, res, workspace) {
 	send(res, 200, {
 		ok: true,
 		completedAt: saved.completedAt ?? null,
-		pinned: saved.pinned === true
+		pinned: saved.pinned === true,
+		queued: readQueue().includes(saved.id)
 	});
 }
 /** What `/state` can change. A body naming none of these writes nothing. */
@@ -10348,10 +10763,7 @@ function applyState(workspace, body) {
 	if (Array.isArray(body?.dismissedIssues)) reviewer.dismissedIssues = body.dismissedIssues.filter((entry) => typeof entry?.id === "string" && typeof entry?.sessionId === "string");
 	if (typeof body?.dismissFreshnessSha === "string" || body?.dismissFreshnessSha === null) workspace.dismissedFreshnessSha = body.dismissFreshnessSha ?? void 0;
 	if (typeof body?.dismissKissFreshnessSha === "string" || body?.dismissKissFreshnessSha === null) workspace.dismissedKissFreshnessSha = body.dismissKissFreshnessSha ?? void 0;
-	if (typeof body?.complete === "boolean") {
-		workspace.completedAt = body.complete ? Date.now() : void 0;
-		dequeueOnComplete(workspace.id, body.complete);
-	}
+	if (typeof body?.complete === "boolean") workspace.completedAt = body.complete ? Date.now() : void 0;
 	if (typeof body?.pinned === "boolean") workspace.pinned = body.pinned;
 }
 /** 1 MB is far more than any question or id list, and far less than a memory problem. */