prreviewbuddy 0.62.0 → 0.64.2

package/dist/server.js

@@ -1,4 +1,4 @@
-import { $ as REPOSITORY_SCOPE, $n as presentedKissRunOf, A as reanalyseReview, An as saveJob, Ar as recommendationsOf, At as renderInlineText, B as RUNTIME_WORDS, Bn as followedRefName, Br as clearAgentPreference, Bt as removeWorktree, C as track, Cn as followedBranch, Cr as doneVerb, Ct as authorPreview, D as endRun, Dn as fail, Dr as questionsOutstanding, Dt as CODE_BLOCK_CLASS, E as completeRun, En as endedAt, Er as issuesOutstanding, Et as renderMarkdown, F as recordTelemetryUploadConsent, Fn as purposeOf, Ft as MANAGED_ROOT, G as partFiles, Gn as kissDecisionsOf, Gr as AGENT_IDS, Gt as compactQueue, H as ARCHITECTURE_DIR, Hn as indexKeptIds, Hr as writeAgentPreference, I as telemetryUploadConsent, Ir as isCommitReview, It as createWorktree, J as storeArchitecture, Jn as lineagePosition, Jr as agentFor, Jt as readIndexToken, K as readArchitecture, Kn as latestKissRun, L as waitingHistory, Lr as git, Lt as holdWorktree, Mn as isLiveJob, Mr as WITHHELD_REASON, Mt as extractJsonText, N as readEvents, Nn as phasesFor, Nr as kissOutcome, Nt as objectWithAnyOf, O as reportRun, On as isTerminal, Or as reviewerDispositions, Ot as CODE_COPY_CLASS, P as record, Pn as progressSteps, Pt as ReviewBeingDeletedError, Q as QUOTE_LIMITS, Qt as readServerRecord, R as mapRuntimes, Rn as isClaimed, Rt as isWorktreeHeld, S as isDriving, Sn as checkReview, Sr as readQueue, St as UNESTABLISHED_MOTIVATION, T as beginRun, Tn as allJobs, Tr as isQuestionOutstanding, Tt as GITHUB_MARKDOWN, U as checkArchitecture, Un as indexReviewGroups, Ut as relativeTime$1, V as describeRuntimes, Vn as groupByLineage, Vr as readAgentPreference, Vt as requestJobStop, W as loadArchitecture, Wn as kissDecisionRefusal, Wr as STORE_ROOT, Wt as activateQueued, X as trackedFiles, Xr as AgentCancelledError, Xt as clearServerRecordOwnedBy, Y as storedScopes, Yn as loadWorkspace, Z as ownersOf, Zn as mutateWorkspace, Zr as AgentUnavailableError, _ as liveUpdateFor, _r as sectionOf, _t as removeNotification, a as wasBlocked, an as writeServerRecord, ar as touchWorkspace, at as subjectOf, bn as fillFileUrlTemplate, br as queueRemove, bt as withUsageRecorded, c as RefreshUnavailableError, cn as feedbackUrl, cr as LIFECYCLES, ct as createPrUpdate, d as mergeUpdateInto, dn as MIN_WORKSPACE_PORT, dr as isLifecycle, dt as listNotifications, en as serverEntryExists, er as previousKissResult, et as ancestorsOf, f as updateReview, fn as parseWorkspacePort, fr as lifecycleChip, ft as markAllRead, g as UpdateAlreadyRunningError, gn as claimRequestUrl, gr as parseLifecycle, gt as readNotification, h as askCheckout, hn as writePortPreference, hr as offersLifecycle, ht as pruneNotifications, i as liveJobsFor, it as branchOf, j as runJob, jr as describeAuthorship, jt as asString, k as stopRun, kn as loadJob, kr as processDiscussion, kt as escapeHtml$1, l as cancelUpdate, ln as DEFAULT_WORKSPACE_PORT, lr as activityAttributes, lt as eventName, m as isUnchanged, mn as statedWorkspacePort, mr as lifecyclePill, mt as markReadForReview, n as deleteReview, nn as stopChild, nr as reviewedCommit, nt as writeJsonAtomically, o as discardJob, on as BUILD_VERSION, or as workspaceRevision, ot as createArchitectureNotification, p as refreshPrContext, pn as resolveWorkspacePort, pr as lifecycleItemLabel, pt as markRead, q as repositoryKey, qt as settleQueue, r as lineageIds, rt as announceJob, s as FETCH_TIMEOUT_MS, sr as workspaceStamps, st as createPrState, t as deleteLineage, tn as spawnServer, tr as recentWorkspaces, tt as isArchitectureId, u as cancellableHere, un as MAX_WORKSPACE_PORT, ur as activityLabel, ut as headline, vn as forgeFor, vr as queueAdd, vt as toneOf, w as RUN_KINDS, wn as reviewedShaOf, wr as isIssueOutstanding, wt as hasAuthorContent, x as startKissJob, xn as checkCodeFreshness, xr as queueStamp, xt as EXPLAIN_SIMPLY_PROMPT, y as agentEnvOf, yn as remoteHost, yr as queueAddMany, yt as unreadCount$1, z as recordRuntimeBestEffort, zn as decideKissRecommendation, zr as resolveModel, zt as readMarker } from "./delete_review-LzUgEUzN.js";
+import { $ as storedScopes, $n as loadWorkspace, A as reanalyseReview, An as endedAt, Ar as issuesOutstanding, At as renderMarkdown, Br as isCommitReview, Bt as createWorktree, C as track, Cn as remoteHost, Cr as queueAddMany, Ct as unreadCount$1, D as endRun, Dn as followedBranch, Dr as doneVerb, Dt as authorPreview, E as completeRun, En as checkReview, Er as readQueue, Et as UNESTABLISHED_MOTIVATION, F as record, Fr as describeAuthorship, Ft as asString, G as describeRuntimes, Gn as groupByLineage, Gr as readAgentPreference, Gt as requestJobStop, H as mapRuntimes, Hn as isClaimed, Ht as isWorktreeHeld, In as isLiveJob, Ir as WITHHELD_REASON, It as extractJsonText, J as loadArchitecture, Jn as kissDecisionRefusal, Jr as STORE_ROOT, Jt as activateQueued, K as ARCHITECTURE_DIR, Kn as indexKeptIds, Kr as writeAgentPreference, Ln as phasesFor, Lr as kissOutcome, Lt as objectWithAnyOf, Mn as isTerminal, Mr as reviewerDispositions, Mt as CODE_COPY_CLASS, Nn as loadJob, Nr as processDiscussion, Nt as escapeHtml$1, O as reportRun, On as reviewedShaOf, Or as isIssueOutstanding, Ot as hasAuthorContent, P as readEvents, Pn as saveJob, Pr as recommendationsOf, Pt as renderInlineText, Q as storeArchitecture, Qn as lineagePosition, Qr as agentFor, Qt as readIndexToken, R as recordTelemetryUploadConsent, Rn as progressSteps, Rt as ReviewBeingDeletedError, S as isDriving, Sn as forgeFor, Sr as queueAdd, St as toneOf, T as beginRun, Tn as checkCodeFreshness, Tr as queueStamp, Tt as EXPLAIN_SIMPLY_PROMPT, U as recordRuntimeBestEffort, Un as decideKissRecommendation, Ur as resolveModel, Ut as readMarker, V as waitingHistory, Vr as git, Vt as holdWorktree, W as RUNTIME_WORDS, Wn as followedRefName, Wr as clearAgentPreference, Wt as removeWorktree, X as readArchitecture, Xn as latestKissRun, Y as partFiles, Yn as kissDecisionsOf, Yr as AGENT_IDS, Yt as compactQueue, Z as repositoryKey, Zt as settleQueue, _ as liveUpdateFor, _n as resolveWorkspacePort, _r as lifecycleItemLabel, _t as markRead, a as wasBlocked, an as spawnServer, ar as recentWorkspaces, at as isArchitectureId, bn as claimRequestUrl, br as parseLifecycle, bt as readNotification, c as RefreshUnavailableError, ct as branchOf, d as mergeUpdateInto, dr as workspaceStamps, dt as createPrState, ei as AgentCancelledError, en as clearServerRecordOwnedBy, et as trackedFiles, f as updateReview, fn as feedbackUrl, fr as LIFECYCLES, ft as createPrUpdate, g as UpdateAlreadyRunningError, gn as parseWorkspacePort, gr as lifecycleChip, gt as markAllRead, h as askCheckout, hn as MIN_WORKSPACE_PORT, hr as isLifecycle, ht as listNotifications, i as liveJobsFor, in as serverEntryExists, ir as previousKissResult, it as ancestorsOf, j as runJob, jn as fail, jr as questionsOutstanding, jt as CODE_BLOCK_CLASS, k as stopRun, kn as allJobs, kr as isQuestionOutstanding, kt as GITHUB_MARKDOWN, l as cancelUpdate, ln as writeServerRecord, lr as touchWorkspace, lt as subjectOf, m as isUnchanged, mn as MAX_WORKSPACE_PORT, mr as activityLabel, mt as headline, n as deleteReview, nn as readServerRecord, nt as QUOTE_LIMITS, o as discardJob, on as stopChild, or as reviewedCommit, ot as writeJsonAtomically, p as refreshPrContext, pn as DEFAULT_WORKSPACE_PORT, pr as activityAttributes, pt as eventName, q as checkArchitecture, qn as indexReviewGroups, qt as relativeTime$1, r as lineageIds, rr as presentedKissRunOf, rt as REPOSITORY_SCOPE, s as FETCH_TIMEOUT_MS, st as announceJob, t as deleteLineage, ti as AgentUnavailableError, tr as mutateWorkspace, tt as ownersOf, u as cancellableHere, un as BUILD_VERSION, ur as workspaceRevision, ut as createArchitectureNotification, vn as statedWorkspacePort, vr as lifecyclePill, vt as markReadForReview, w as RUN_KINDS, wn as fillFileUrlTemplate, wr as queueRemove, wt as withUsageRecorded, x as startKissJob, xr as sectionOf, xt as removeNotification, y as agentEnvOf, yn as writePortPreference, yr as offersLifecycle, yt as pruneNotifications, z as telemetryUploadConsent, zn as purposeOf, zt as MANAGED_ROOT } from "./delete_review-l4p_p6oh.js";
 import { basename, isAbsolute, join, relative, resolve } from "node:path";
 import { existsSync, mkdirSync, readFileSync, readdirSync, rmSync, statSync, writeFileSync } from "node:fs";
 import { createHash, randomUUID, timingSafeEqual } from "node:crypto";
@@ -322,11 +322,35 @@ function countLabel(reviews, branches) {
 	return `${label} · ${branches} branch${branches === 1 ? "" : "es"}`;
 }
 /**
-* Every login the author dropdown offers, once each, in alphabetical order regardless of case.
-* Read from the cards the page is showing, so it never lists somebody with nothing to show.
+* Every login the author dropdown offers, once each, in alphabetical order of what the reader sees:
+* the name where there is one, the login otherwise, regardless of case. Read from the cards the page
+* is showing, so it never lists somebody with nothing to show.
+*
+* A login's name is the one on its newest review that has a name, chosen here by `createdAt` rather
+* than by the order the caller happens to pass, so an account renamed upstream reads as it does now.
 */
-function authorOptions(handles) {
-	return [...new Set(handles.filter((handle) => Boolean(handle)))].sort((a, b) => a.localeCompare(b, void 0, { sensitivity: "base" }));
+function authorOptions(cards) {
+	const newest = /* @__PURE__ */ new Map();
+	for (const card of cards) {
+		if (!card.handle) continue;
+		const seen = newest.get(card.handle);
+		if (!seen) newest.set(card.handle, {
+			name: card.name || void 0,
+			createdAt: card.createdAt
+		});
+		else if (card.name && (!seen.name || card.createdAt > seen.createdAt)) newest.set(card.handle, {
+			name: card.name,
+			createdAt: card.createdAt
+		});
+	}
+	return [...newest].map(([handle, { name }]) => name && name !== handle ? {
+		handle,
+		name
+	} : { handle }).sort((a, b) => (a.name ?? a.handle).localeCompare(b.name ?? b.handle, void 0, { sensitivity: "base" }));
+}
+/** An author option as the dropdown spells it: the name and the login, or the login alone. */
+function authorOptionLabel(option) {
+	return option.name ? `${option.name} · @${option.handle}` : `@${option.handle}`;
 }
 /**
 * A card menu's status item, which sets the state it names. The bulk bar's words for the same two
@@ -334,7 +358,7 @@ function authorOptions(handles) {
 * the browser relabels the item after a change, and that renderer reaches the store.
 */
 function markItemLabel(mark) {
-	return mark === "complete" ? "Mark reviewed" : "Mark in progress";
+	return mark === "complete" ? "Mark reviewed" : "Mark not reviewed";
 }
 /**
 * The review page's Pin item, which says what pressing it will do. The words of the index's star,
@@ -371,7 +395,7 @@ function queueOptionLabel(queued) {
 */
 function statusLabel(status) {
 	if (status === "complete") return "Reviewed";
-	return status === "in-progress" ? "In progress" : "New";
+	return status === "in-progress" ? "Not reviewed" : "New";
 }
 /**
 * The bulk bar's way to tick everything the filters show, across pages: "Select all 7".
@@ -422,6 +446,206 @@ function anythingRunning(states) {
 var FETCH_REVIEW_LABEL = "Fetch remote updates";
 var FETCH_SELECTED_LABEL = "Fetch selected";
 //#endregion
+//#region ../../packages/review-harness/src/domain/ui/components/message_copy.ts
+var MESSAGE_COPY_CLASS = "pr-helper-msg-copy";
+var MESSAGE_ACTIONS_CLASS = "pr-helper-chat-actions";
+/** The Markdown itself, for surfaces that hold no message state to resolve an id against. */
+var MESSAGE_SOURCE_ATTR = "data-copy-source";
+var STATES$1 = {
+	idle: {
+		icon: "<path d=\"M0 6.75C0 5.784.784 5 1.75 5h1.5a.75.75 0 0 1 0 1.5h-1.5a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-1.5a.75.75 0 0 1 1.5 0v1.5A1.75 1.75 0 0 1 9.25 16h-7.5A1.75 1.75 0 0 1 0 14.25Z\"></path><path d=\"M5 1.75C5 .784 5.784 0 6.75 0h7.5C15.216 0 16 .784 16 1.75v7.5A1.75 1.75 0 0 1 14.25 11h-7.5A1.75 1.75 0 0 1 5 9.25Zm1.75-.25a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-7.5a.25.25 0 0 0-.25-.25Z\"></path>",
+		label: "",
+		aria: "Copy message"
+	},
+	copied: {
+		icon: "<path d=\"M13.78 4.22a.75.75 0 0 1 0 1.06l-7.25 7.25a.75.75 0 0 1-1.06 0L2.22 9.28a.751.751 0 0 1 .018-1.042.751.751 0 0 1 1.042-.018L6 10.94l6.72-6.72a.75.75 0 0 1 1.06 0Z\"></path>",
+		label: "Copied",
+		aria: "Copied"
+	},
+	failed: {
+		icon: "<path d=\"M6.457 1.047c.659-1.234 2.427-1.234 3.086 0l6.082 11.378A1.75 1.75 0 0 1 14.082 15H1.918a1.75 1.75 0 0 1-1.543-2.575Zm1.763.707a.25.25 0 0 0-.44 0L1.698 13.132a.25.25 0 0 0 .22.368h12.164a.25.25 0 0 0 .22-.368Zm.53 3.996v2.5a.75.75 0 0 1-1.5 0v-2.5a.75.75 0 0 1 1.5 0ZM9 11a1 1 0 1 1-2 0 1 1 0 0 1 2 0Z\"></path>",
+		label: "Copy failed",
+		aria: "Copy failed"
+	}
+};
+/**
+* The same row, carrying the Markdown itself.
+*
+* For a surface with nothing to resolve an id against — the local workspace renders its stored
+* transcript to HTML server-side and appends live answers straight to the DOM, so there is no
+* conversation object on the client at all. Duplicating the source into the attribute is the
+* honest trade there, and it is the same shape the prompt bank already uses to ship its data.
+*/
+function renderMessageCopySource(source) {
+	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"`);
+}
+/** The class that places a finding's copy control in its card's heading, and gives it its tooltip. */
+var FINDING_COPY_CLASS = "pr-helper-finding-copy";
+/**
+* The same control on an issue or question card, carrying that finding as Markdown.
+*
+* It sits in the card's heading, not its footer. It copies the whole finding, so the heading is
+* the thing it acts on. And it is a different kind of act from Investigate, Dismiss and Mark
+* answered, which change the review: those keep the footer to themselves, so the footer holds
+* only actions that change something.
+*
+* Icon only at rest, like a chat reply's. It used to show a word because it sat in a row of worded
+* buttons, where an icon alone read as decoration. In the heading it is on its own, and a word
+* would give a utility the weight of the title beside it. What it does is said by a hover
+* tooltip (`data-tip`, the same CSS tooltip the pills use, not a native `title`) and by its
+* accessible name. After a click it shows a green check and says "Copied" (or "Copy failed") in
+* the bubble below it, the words used everywhere else, but not inline: beside it is Ask Buddy. The
+* resting name and tip are kept on the button so the reset returns to "Copy finding" rather than to
+* a chat reply's "Copy message".
+*/
+function renderFindingCopySource(source, kind) {
+	const name = kind === "issue" ? "Copy finding" : kind === "question" ? "Copy question" : "Copy recommendation";
+	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"`, {
+		label: "",
+		aria: name,
+		tip: name
+	});
+}
+/**
+* The same control with its word showing, for a footer of worded actions.
+*
+* A KISS card's Copy draft comment sits among Plan to simplify and Leave it, where an icon alone
+* would read as decoration. `extra` is for attributes the caller's own client needs, such as the
+* handle Copy recommendations is found by; the source may be updated after render, since the
+* listener reads it at the moment of the click.
+*/
+function renderLabelledCopySource(source, label, extra = "") {
+	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"${extra ? ` ${extra}` : ""}`, {
+		label,
+		aria: label,
+		tip: label
+	});
+}
+/** Where a finding's button keeps its resting word and name. Absent on a chat reply's. */
+var IDLE_LABEL_ATTR = "data-copy-idle-label";
+var IDLE_ARIA_ATTR = "data-copy-idle-aria";
+var IDLE_TIP_ATTR = "data-copy-idle-tip";
+function button(handle, idle) {
+	const { icon } = STATES$1.idle;
+	const label = idle?.label ?? STATES$1.idle.label;
+	const aria = idle?.aria ?? STATES$1.idle.aria;
+	const remembered = idle ? ` ${IDLE_LABEL_ATTR}="${escapeHtml$1(label)}" ${IDLE_ARIA_ATTR}="${escapeHtml$1(aria)}" ${IDLE_TIP_ATTR}="${escapeHtml$1(idle.tip)}"` : "";
+	const control = `<button type="button" class="${idle ? `${MESSAGE_COPY_CLASS} ${FINDING_COPY_CLASS}` : MESSAGE_COPY_CLASS}" ${handle}${remembered}${idle ? ` data-tip="${escapeHtml$1(idle.tip)}"` : ""} aria-label="${escapeHtml$1(aria)}"><span class="${MESSAGE_COPY_CLASS}-icon" aria-hidden="true">${svg(icon)}</span><span class="${MESSAGE_COPY_CLASS}-label"${label ? "" : " hidden"}>${escapeHtml$1(label)}</span></button>`;
+	return idle ? control : `<div class="${MESSAGE_ACTIONS_CLASS}">${control}</div>`;
+}
+function svg(paths) {
+	return `<svg viewBox="0 0 16 16" width="14" height="14" fill="currentColor">${paths}</svg>`;
+}
+//#endregion
+//#region ../../packages/review-harness/src/workspace/feedback_view.ts
+/**
+* Feedback selection: the reviewer picks the Issues and Questions they mean to raise with the
+* author, across both panes, and takes them away as one Markdown draft.
+*
+* The words and the composition, in one module the server renders from and the client imports.
+* The same split the inbox's bulk bar settled (`reviews_index.ts`): every sentence here carries a
+* number only the browser knows, so the whole sentence lives where both sides can reach it, and
+* where the copy style and vocabulary scans already look. Nothing here may import a node builtin.
+*
+* Each item is the text the card's own Copy button carries (`finding_copy.ts`), read from the card.
+* There is one rendering of a finding as text, so what this exports, what Copy gives and what Ask
+* Buddy is asked about can never disagree. That is also why no severity, theme or status appears:
+* those sort the findings for the reviewer and are not part of what the author is told.
+*/
+function count(n, noun) {
+	return `${n} ${noun}${n === 1 ? "" : "s"}`;
+}
+/**
+* The tray's count line, in its three parts: "5 chosen", "2 issues", "3 questions".
+*
+* Chosen rather than selected: the reviewer is choosing what to tell the author, not operating a
+* list.
+* "Nothing chosen" rather than "0 chosen", the inbox bar's reason: the mode opens in this
+* state, and a zero reads as a count of something that happened. A side with nothing picked is
+* empty, and left out, rather than said as a zero beside the side that has some. The tray draws
+* the two sides as buttons back to their panes; everywhere else reads the joined sentence.
+*/
+function feedbackSelectionParts(issues, questions) {
+	const total = issues + questions;
+	return {
+		total: total === 0 ? "Nothing chosen" : `${total} chosen`,
+		issues: issues ? count(issues, "issue") : "",
+		questions: questions ? count(questions, "question") : ""
+	};
+}
+/** "5 chosen · 2 issues · 3 questions", "1 chosen · 1 question", "Nothing chosen". */
+function feedbackSelectionLabel(issues, questions) {
+	const parts = feedbackSelectionParts(issues, questions);
+	return [
+		parts.total,
+		parts.issues,
+		parts.questions
+	].filter(Boolean).join(" · ");
+}
+/**
+* The pane's way into the mode, and out of it. Named for what it is for rather than for the
+* mechanism ("Select"). It shares its name with the tray's last step on purpose, and the two are
+* never on screen together: pressed, this one reads Done with a tick, the same act as the tray's
+* Done, and that is when the tray appears. A checklist, not a sparkle, because nothing here is AI.
+* Both faces are in the markup and the stylesheet shows the one `aria-pressed` calls for, so the
+* hidden one is not part of the name a screen reader hears.
+*/
+var PREPARE_LABEL = "Prepare feedback";
+var DONE_LABEL = "Done";
+var ICON = (paths) => `<svg viewBox="0 0 24 24" width="14" height="14" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true">${paths}</svg>`;
+var LIST_CHECK = ICON("<path d=\"m3 17 2 2 4-4M3 7l2 2 4-4M13 6h8M13 12h8M13 18h8\"/>");
+var CHECK = ICON("<path d=\"M20 6 9 17l-5-5\"/>");
+function renderFeedbackSelectButton() {
+	return `<button type="button" class="prb-selection-toggle" data-feedback-select aria-pressed="false"><span class="prb-selection-toggle-idle">${LIST_CHECK}${PREPARE_LABEL}</span><span class="prb-selection-toggle-active">${CHECK}${DONE_LABEL}</span></button>`;
+}
+/**
+* The tray, at the foot of the document pane on every view while the mode is on. The mode belongs
+* to the review rather than to a pane, so leaving Issues does not take the tray with it, and its
+* counts are the way back to each pane.
+*/
+function renderFeedbackTray() {
+	return `<div class="prb-selection-tray" data-feedback-tray role="region" aria-label="Chosen for feedback" hidden>
+  <p class="prb-selection-count" aria-live="polite">
+    <span data-feedback-total>${escapeHtml$1(feedbackSelectionLabel(0, 0))}</span>${renderGoto("issues")}${renderGoto("questions")}
+  </p>
+  <span class="prb-selection-spacer"></span>
+  <button type="button" class="prb-selection-button" data-feedback-clear>Clear</button>
+  <button type="button" class="prb-selection-button" data-feedback-done>${DONE_LABEL}</button>
+  <button type="button" class="prb-selection-button prb-selection-primary" data-feedback-prepare disabled>${PREPARE_LABEL}</button>
+</div>`;
+}
+/** A side's count, which is also the way back to its pane. The dot is text, so the line reads whole. */
+function renderGoto(pane) {
+	return `<button type="button" class="prb-selection-goto" data-feedback-goto="${pane}" hidden><span class="prb-selection-sep" aria-hidden="true"> · </span><span data-feedback-goto-label></span></button>`;
+}
+/**
+* Where the draft is read, edited and copied.
+*
+* Two views of one draft, as tabs, because that is what they are: Preview, the default, for reading
+* what the author will see, and Markdown for changing it. Both are drawn from the same text, the
+* text area's, so an edit is in the preview the moment the tab changes, and Copy takes exactly what
+* is in the box: the client rewrites the button's source on every keystroke.
+*/
+function renderFeedbackDialog() {
+	const tab = (name, label, selected) => `<button type="button" role="tab" class="prb-draft-tab" id="prb-draft-tab-${name}" data-draft-tab="${name}" aria-controls="prb-draft-${name}" aria-selected="${selected}" tabindex="${selected ? 0 : -1}">${label}</button>`;
+	return `<dialog class="prb-dialog prb-draft-dialog" data-feedback-dialog aria-labelledby="prb-feedback-title">
+  <div class="prb-draft-head">
+    <h2 id="prb-feedback-title">Prepare feedback</h2>
+    <p class="prb-draft-count" data-feedback-dialog-count></p>
+  </div>
+  <div class="prb-draft-tabs" role="tablist" aria-label="Draft">${tab("preview", "Preview", true)}${tab("markdown", "Markdown", false)}</div>
+  <div class="prb-draft-panel prb-draft-preview prb-md" id="prb-draft-preview" role="tabpanel" aria-labelledby="prb-draft-tab-preview" tabindex="0" data-feedback-preview></div>
+  <div class="prb-draft-panel" id="prb-draft-markdown" role="tabpanel" aria-labelledby="prb-draft-tab-markdown" hidden>
+    <textarea class="prb-draft-text" data-feedback-text aria-label="Markdown for the author" spellcheck="true" autocomplete="off"></textarea>
+  </div>
+  <div class="prb-dialog-actions">
+    <span class="prb-dialog-spacer"></span>
+    <button type="button" data-feedback-close>Close</button>
+    ${renderLabelledCopySource("", "Copy Markdown", "data-feedback-copy")}
+  </div>
+</dialog>`;
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/way_back.ts
 var CLI = {
 	command: "prreviewbuddy open",
@@ -842,97 +1066,6 @@ function deriveReviewMetrics(result) {
 	};
 }
 //#endregion
-//#region ../../packages/review-harness/src/domain/ui/components/message_copy.ts
-var MESSAGE_COPY_CLASS = "pr-helper-msg-copy";
-var MESSAGE_ACTIONS_CLASS = "pr-helper-chat-actions";
-/** The Markdown itself, for surfaces that hold no message state to resolve an id against. */
-var MESSAGE_SOURCE_ATTR = "data-copy-source";
-var STATES$1 = {
-	idle: {
-		icon: "<path d=\"M0 6.75C0 5.784.784 5 1.75 5h1.5a.75.75 0 0 1 0 1.5h-1.5a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-1.5a.75.75 0 0 1 1.5 0v1.5A1.75 1.75 0 0 1 9.25 16h-7.5A1.75 1.75 0 0 1 0 14.25Z\"></path><path d=\"M5 1.75C5 .784 5.784 0 6.75 0h7.5C15.216 0 16 .784 16 1.75v7.5A1.75 1.75 0 0 1 14.25 11h-7.5A1.75 1.75 0 0 1 5 9.25Zm1.75-.25a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-7.5a.25.25 0 0 0-.25-.25Z\"></path>",
-		label: "",
-		aria: "Copy message"
-	},
-	copied: {
-		icon: "<path d=\"M13.78 4.22a.75.75 0 0 1 0 1.06l-7.25 7.25a.75.75 0 0 1-1.06 0L2.22 9.28a.751.751 0 0 1 .018-1.042.751.751 0 0 1 1.042-.018L6 10.94l6.72-6.72a.75.75 0 0 1 1.06 0Z\"></path>",
-		label: "Copied",
-		aria: "Copied"
-	},
-	failed: {
-		icon: "<path d=\"M6.457 1.047c.659-1.234 2.427-1.234 3.086 0l6.082 11.378A1.75 1.75 0 0 1 14.082 15H1.918a1.75 1.75 0 0 1-1.543-2.575Zm1.763.707a.25.25 0 0 0-.44 0L1.698 13.132a.25.25 0 0 0 .22.368h12.164a.25.25 0 0 0 .22-.368Zm.53 3.996v2.5a.75.75 0 0 1-1.5 0v-2.5a.75.75 0 0 1 1.5 0ZM9 11a1 1 0 1 1-2 0 1 1 0 0 1 2 0Z\"></path>",
-		label: "Copy failed",
-		aria: "Copy failed"
-	}
-};
-/**
-* The same row, carrying the Markdown itself.
-*
-* For a surface with nothing to resolve an id against — the local workspace renders its stored
-* transcript to HTML server-side and appends live answers straight to the DOM, so there is no
-* conversation object on the client at all. Duplicating the source into the attribute is the
-* honest trade there, and it is the same shape the prompt bank already uses to ship its data.
-*/
-function renderMessageCopySource(source) {
-	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"`);
-}
-/** The class that places a finding's copy control in its card's heading, and gives it its tooltip. */
-var FINDING_COPY_CLASS = "pr-helper-finding-copy";
-/**
-* The same control on an issue or question card, carrying that finding as Markdown.
-*
-* It sits in the card's heading, not its footer. It copies the whole finding, so the heading is
-* the thing it acts on. And it is a different kind of act from Investigate, Dismiss and Mark
-* answered, which change the review: those keep the footer to themselves, so the footer holds
-* only actions that change something.
-*
-* Icon only at rest, like a chat reply's. It used to show a word because it sat in a row of worded
-* buttons, where an icon alone read as decoration. In the heading it is on its own, and a word
-* would give a utility the weight of the title beside it. What it does is said by a hover
-* tooltip (`data-tip`, the same CSS tooltip the pills use, not a native `title`) and by its
-* accessible name. After a click it shows a green check and says "Copied" (or "Copy failed") in
-* the bubble below it, the words used everywhere else, but not inline: beside it is Ask Buddy. The
-* resting name and tip are kept on the button so the reset returns to "Copy finding" rather than to
-* a chat reply's "Copy message".
-*/
-function renderFindingCopySource(source, kind) {
-	const name = kind === "issue" ? "Copy finding" : kind === "question" ? "Copy question" : "Copy recommendation";
-	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"`, {
-		label: "",
-		aria: name,
-		tip: name
-	});
-}
-/**
-* The same control with its word showing, for a footer of worded actions.
-*
-* A KISS card's Copy draft comment sits among Plan to simplify and Leave it, where an icon alone
-* would read as decoration. `extra` is for attributes the caller's own client needs, such as the
-* handle Copy recommendations is found by; the source may be updated after render, since the
-* listener reads it at the moment of the click.
-*/
-function renderLabelledCopySource(source, label, extra = "") {
-	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"${extra ? ` ${extra}` : ""}`, {
-		label,
-		aria: label,
-		tip: label
-	});
-}
-/** Where a finding's button keeps its resting word and name. Absent on a chat reply's. */
-var IDLE_LABEL_ATTR = "data-copy-idle-label";
-var IDLE_ARIA_ATTR = "data-copy-idle-aria";
-var IDLE_TIP_ATTR = "data-copy-idle-tip";
-function button(handle, idle) {
-	const { icon } = STATES$1.idle;
-	const label = idle?.label ?? STATES$1.idle.label;
-	const aria = idle?.aria ?? STATES$1.idle.aria;
-	const remembered = idle ? ` ${IDLE_LABEL_ATTR}="${escapeHtml$1(label)}" ${IDLE_ARIA_ATTR}="${escapeHtml$1(aria)}" ${IDLE_TIP_ATTR}="${escapeHtml$1(idle.tip)}"` : "";
-	const control = `<button type="button" class="${idle ? `${MESSAGE_COPY_CLASS} ${FINDING_COPY_CLASS}` : MESSAGE_COPY_CLASS}" ${handle}${remembered}${idle ? ` data-tip="${escapeHtml$1(idle.tip)}"` : ""} aria-label="${escapeHtml$1(aria)}"><span class="${MESSAGE_COPY_CLASS}-icon" aria-hidden="true">${svg(icon)}</span><span class="${MESSAGE_COPY_CLASS}-label"${label ? "" : " hidden"}>${escapeHtml$1(label)}</span></button>`;
-	return idle ? control : `<div class="${MESSAGE_ACTIONS_CLASS}">${control}</div>`;
-}
-function svg(paths) {
-	return `<svg viewBox="0 0 16 16" width="14" height="14" fill="currentColor">${paths}</svg>`;
-}
-//#endregion
 //#region ../../packages/review-harness/src/domain/ui/components/ask_buddy_menu.ts
 /**
 * Ask Buddy on a finding card: a sparkle beside Copy that opens the assistant's actions for the
@@ -2397,6 +2530,7 @@ function renderIssuesTab(session, investigatingId, options) {
     <div class="pr-helper-issue-card pr-helper-severity-${issue.severity}${issue.reconciliationStatus ? ` pr-helper-issue-${issue.reconciliationStatus}` : ""}${issue.investigationImpact ? ` pr-helper-issue-investigation-${issue.investigationImpact}` : ""}${isDismissed && !stale ? " pr-helper-issue-card-dismissed" : ""}" data-finding-card="${escapeHtml$1(issue.id)}">
       <div class="pr-helper-issue-header">
         <div class="pr-helper-issue-title-row">
+          ${options?.feedbackSelect ? renderFeedbackPick("issue", issue.title) : ""}
           <p class="pr-helper-issue-title">${renderInlineText(issue.title)}</p>
           ${renderFindingCopySource(issueMarkdown(issue), "issue")}
           ${options?.askBuddy ? renderAskBuddyMenu("issue") : ""}
@@ -2499,6 +2633,7 @@ function renderQuestionCard(q, sections, answered, records, now = Date.now(), in
     <div class="pr-helper-question-card${answered ? " pr-helper-question-card-answered" : ""}${q.investigationImpact ? ` pr-helper-issue-investigation-${q.investigationImpact}` : ""}" data-finding-card="${escapeHtml$1(q.id)}"${questionHome(q) ? ` data-section-id="${escapeHtml$1(questionHome(q))}"` : ""}>
       <div class="pr-helper-issue-header">
         <div class="pr-helper-issue-title-row">
+          ${options?.feedbackSelect ? renderFeedbackPick("question", q.question) : ""}
           <p class="pr-helper-issue-title">${renderInlineText(q.question)}</p>
           ${renderFindingCopySource(questionMarkdown(q), "question")}
           ${options?.askBuddy ? renderAskBuddyMenu("question") : ""}
@@ -2550,6 +2685,17 @@ function renderQuestionsProgress(questions, answered) {
 	if (counts.total < 2) return "";
 	return `<div class="pr-helper-questions-progress" role="status">${questionsProgressBody(counts.total, counts.open)}</div>`;
 }
+/**
+* The card's feedback checkbox. Named by the finding, so a list of them is not one word announced
+* fifteen times. The id is the card's own `data-finding-card`.
+*
+* `autocomplete="off"` because the browser otherwise restores a box's tick when the page is loaded
+* again from history, and the selection it belonged to went with the page that made it: Back to a
+* review showed a ticked box that nothing had selected.
+*/
+function renderFeedbackPick(kind, title) {
+	return `<input type="checkbox" class="pr-helper-feedback-pick" data-feedback-pick="${kind}" autocomplete="off" aria-label="${escapeHtml$1(`Include in feedback: ${title.trim()}`)}">`;
+}
 function renderQuestionJump(file, options) {
 	if (!file || options?.suppressFileJump) return "";
 	return `<button type="button" class="pr-helper-question-jump" data-file="${escapeHtml$1(file)}">Jump to file →</button>`;
@@ -5441,6 +5587,7 @@ ${updating(view) ? "" : renderUpdateOutcome(view.lastUpdate)}
 ${updating(view) ? "" : renderLineageNotice(view)}
 ${renderReviewBody(view, sections)}
 ${view.token ? renderPromptDialog() : ""}
+${offersFeedback(view) ? renderFeedbackDialog() : ""}
 <script type="module" src="/assets/workspace.js"><\/script>
 </body>
 </html>`;
@@ -5498,12 +5645,22 @@ function renderReviewBody(view, sections) {
   ${renderRail(view, sections, updating(view))}
   <main class="prb-doc">
     ${updating(view) ? renderUpdatingDoc(view) : renderReviewDoc(view, sections)}
+    ${offersFeedback(view) ? renderFeedbackTray() : ""}
   </main>
   ${result ? renderAsk(view, sections, updating(view)) : ""}
   ${result ? renderAskTab() : ""}
 </div>`;
 }
 /**
+* Whether the reviewer can choose findings to raise with the author: a standard result with at least
+* one issue or question, and no update running over it. Select, the tray and the dialog are drawn
+* together or not at all.
+*/
+function offersFeedback(view) {
+	const result = view.session.result;
+	return Boolean(result && (result.issues.length > 0 || result.questions.length > 0)) && !updating(view);
+}
+/**
 * 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,
@@ -5658,15 +5815,19 @@ function renderStandardViews(view, sections) {
         ${renderDiagrams(view.diagrams ?? [], sections, view.reviewedSha ?? null)}
       </section>
       <section class="prb-view" data-view="issues" hidden>
-        ${renderDocHead("Issues", renderFocusToggle("issues", "Issues"))}
+        ${renderDocHead("Issues", `${(result?.issues.length ?? 0) > 0 ? renderFeedbackSelectButton() : ""}${renderFocusToggle("issues", "Issues")}`)}
         ${renderAttributionAge(view)}
-        ${renderIssuesTab(session, null, { askBuddy: true })}
+        ${renderIssuesTab(session, null, {
+		askBuddy: true,
+		feedbackSelect: true
+	})}
       </section>
       <section class="prb-view" data-view="questions" hidden>
-        ${renderDocHead("Questions", renderFocusToggle("questions", "Questions"))}
+        ${renderDocHead("Questions", `${(result?.questions.length ?? 0) > 0 ? renderFeedbackSelectButton() : ""}${renderFocusToggle("questions", "Questions")}`)}
         ${renderQuestionsTab(session, null, {
 		suppressFileJump: true,
-		askBuddy: true
+		askBuddy: true,
+		feedbackSelect: true
 	})}
       </section>
       <section class="prb-view" data-view="prcontext" hidden>
@@ -5763,7 +5924,7 @@ function renderLineageNotice(view) {
 * Two facts per row and no more. The commit is the one that does the real work, because two
 * analyses an hour apart are told apart by what they were of rather than by when they happened;
 * the risk says whether the newer one found something worse, which is the reason to go and read
-* it. There is deliberately no `New` / `In progress` / `Reviewed` here: those describe where
+* it. There is deliberately no `New` / `Not reviewed` / `Reviewed` here: those describe where
 * somebody is in a piece of work, and on a superseded review that is a claim about work nobody
 * should be continuing. The index's history rows withhold it for the same reason.
 */
@@ -5894,7 +6055,7 @@ function renderReviewBar(view, freshness) {
 /**
 * Where this review stands, as pills that say and never do.
 *
-* Two, because they answer two questions. Reviewed or In progress is what the reviewer has done;
+* Two, because they answer two questions. Reviewed or Not reviewed 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.
@@ -6047,8 +6208,8 @@ function renderRail(view, sections, stale = false) {
       ${themes || "<p class=\"prb-rail-heading\">No themes</p>"}
     </div>
     ${railLink("map", "diagrams", "Diagrams", diagramsBadge(diagramCount(view)), void 0, "diagram architecture")}
-    ${railLink("issues", "issues", "Issues", findingsBadge(issuesOutstanding(issues, dispositions)))}
-    ${railLink("questions", "questions", "Questions", remainingBadge(questionsOutstanding(questions, answered), "question", "answered"))}
+    ${railLink("issues", "issues", "Issues", findingsBadge(issuesOutstanding(issues, dispositions)), void 0, void 0, true)}
+    ${railLink("questions", "questions", "Questions", remainingBadge(questionsOutstanding(questions, answered), "question", "answered"), void 0, void 0, true)}
     ${railLink("files", "files", "Files", totalBadge(view.changeSet.data.files.length, "files"))}
     ${view.prContext ? railLink("prcontext", "prcontext", "PR context", totalBadge(view.prContext.processed.stats.totalThreads, "PR comment threads")) : railLink("prcontext", "prcontext", "PR context", null, "No pull request was connected to this review")}`
 	}) : "";
@@ -6255,10 +6416,15 @@ function renderBadge(badge) {
 	return `<span class="prb-rail-count${badge.done ? " prb-rail-count-done" : ""}" aria-label="${escapeHtml$1(badge.label)}">${escapeHtml$1(badge.text)}</span>`;
 }
 /** A null badge renders nothing, for the case where zero would be a claim rather than a total. */
-function railLink(view, icon, label, badge, note, alsoCurrentOn) {
+/**
+* `selectable` leaves room for the feedback selection's quiet "2 selected" under the title, empty and
+* hidden until the client has something to say there.
+*/
+function railLink(view, icon, label, badge, note, alsoCurrentOn, selectable = false) {
+	const selected = selectable ? `<span class="prb-rail-selected" data-rail-selected="${view}" hidden></span>` : "";
 	return `<a class="prb-rail-item prb-rail-link" href="${escapeHtml$1(hashForView(view))}" data-view-nav="${view}"${alsoCurrentOn ? ` data-view-also="${alsoCurrentOn}"` : ""}${note ? ` title="${escapeHtml$1(note)}"` : ""}>
     ${railIcon(icon)}
-    <span class="prb-rail-label"><span class="prb-rail-title">${escapeHtml$1(label)}</span></span>
+    <span class="prb-rail-label"><span class="prb-rail-title">${escapeHtml$1(label)}</span>${selected}</span>
     ${renderBadge(badge)}
   </a>`;
 }
@@ -7427,7 +7593,7 @@ async function fetchHead(cwd, request, env = {}) {
 * What each pull request's forge last said about whether it is still open.
 *
 * **A fact about the request, never about the review.** A review can be `Reviewed` while its PR is
-* open, and `In progress` while its PR has merged; the reviewer's decision lives on the workspace
+* open, and `Not reviewed` while its PR has merged; the reviewer's decision lives on the workspace
 * record as `completedAt`, and nothing here reads or writes it.
 *
 * **Its own file, not a field of `watch.json`**, because the two have different lives. `watch.json`
@@ -8775,7 +8941,11 @@ ${renderGlobalBar({
 		unread: view.unreadNotifications ?? 0,
 		driving: view.driving
 	})}
-${renderContextBar("Reviews", ordered.length === 0 ? "" : renderControls(queue.length, authorOptions(ordered.map((group) => group.latest.authorHandle)), ordered.some((group) => group.latest.prNumber !== void 0)))}
+${renderContextBar("Reviews", ordered.length === 0 ? "" : renderControls(queue.length, authorOptions(ordered.map(({ latest }) => ({
+		handle: latest.authorHandle,
+		name: latest.authorName,
+		createdAt: latest.createdAt
+	}))), ordered.some((group) => group.latest.prNumber !== void 0)))}
 <div class="prb-shell">
 ${renderWorkspaceRail({
 		current: "reviews",
@@ -8801,19 +8971,19 @@ ${renderWorkspaceRail({
 *
 * Search covers everything a person might remember about a review — the repository, the branch,
 * the title, the model's summary, and the paths it touched — because which of those they recall
-* is not predictable. The filters are the three states and nothing else; a filter per risk level
-* would be four more controls for a question ("show me only the risky ones") that sorting by
-* recency already answers badly and that scanning answers well.
+* is not predictable. The review state filter is the reviewer's own say and nothing else; a filter
+* per risk level would be four more controls for a question ("show me only the risky ones") that
+* sorting by recency already answers badly and that scanning answers well.
 */
 function renderControls(queued, authors, anyRequest) {
 	return `<div class="prb-index-controls">
     <input type="search" class="prb-index-search" data-search placeholder="Search repo, branch, title, summary or file…" aria-label="Search reviews" spellcheck="false">
     ${renderAuthorFilter(authors)}
-    <label class="prb-index-status">Status:
-      <select data-status-filter aria-label="Filter by status">
+    <label class="prb-index-status">Review state:
+      <select data-status-filter aria-label="Filter by review state">
         <option value="all">All</option>
         <option value="queue" data-queue-option>${escapeHtml$1(queueOptionLabel(queued))}</option>
-        <option value="in-progress">In progress</option>
+        <option value="in-progress">Not reviewed</option>
         <option value="complete">Reviewed</option>
       </select>
     </label>
@@ -8821,13 +8991,13 @@ function renderControls(queued, authors, anyRequest) {
   </div>`;
 }
 /**
-* The pull request's state upstream, beside Status and not inside it, so the two combine.
+* The pull request's state upstream, beside Review state and not inside it, so the two combine.
 *
 * Absent when no card is of a pull request, for the author filter's reason: a select whose every
 * choice but Any shows nothing is not a choice.
 */
 function renderPrFilter() {
-	return `<label class="prb-index-status">PR:
+	return `<label class="prb-index-status prb-index-forge">${GITHUB_MARK}PR:
       <select data-pr-filter aria-label="Filter by pull request state">${PR_FILTER_OPTIONS.map((option) => `<option value="${option.value}">${escapeHtml$1(option.label)}</option>`).join("")}</select>
     </label>`;
 }
@@ -8837,16 +9007,25 @@ function prStateOf(review, states) {
 	return states?.get(requestKey(review.repoPath, review.prNumber));
 }
 /**
-* Whose pull requests to show.
+* Marks the filters whose answers come from GitHub, Author and PR, so the mark means one thing.
+* Review state is the reviewer's own and goes without. Hidden from assistive technology: each
+* select's label already names it.
+*/
+var GITHUB_MARK = "<svg class=\"prb-index-forge-mark\" viewBox=\"0 0 16 16\" width=\"14\" height=\"14\" focusable=\"false\" aria-hidden=\"true\" fill=\"currentColor\"><path d=\"M8 0C3.58 0 0 3.58 0 8c0 3.54 2.29 6.53 5.47 7.59.4.07.55-.17.55-.38 0-.19-.01-.82-.01-1.49-2.01.37-2.53-.49-2.69-.94-.09-.23-.48-.94-.82-1.13-.28-.15-.68-.52-.01-.53.63-.01 1.08.58 1.23.82.72 1.21 1.87.87 2.33.66.07-.52.28-.87.51-1.07-1.78-.2-3.64-.89-3.64-3.95 0-.87.31-1.59.82-2.15-.08-.2-.36-1.02.08-2.12 0 0 .67-.21 2.2.82.64-.18 1.32-.27 2-.27.68 0 1.36.09 2 .27 1.53-1.04 2.2-.82 2.2-.82.44 1.1.16 1.92.08 2.12.51.56.82 1.27.82 2.15 0 3.07-1.87 3.75-3.65 3.95.29.25.54.73.54 1.48 0 1.07-.01 1.93-.01 2.2 0 .21.15.46.55.38A8.013 8.013 0 0016 8c0-4.42-3.58-8-8-8z\"/></svg>";
+/**
+* Whose pull requests to show, by name where GitHub gave one. The login is still what it filters
+* by, and is kept beside the name because two people can share a name and nobody shares a login.
 *
 * Absent when no card carries a login, which is every page of local-branch reviews: a dropdown
 * whose only choice is Everyone is not a choice.
 */
 function renderAuthorFilter(authors) {
 	if (authors.length === 0) return "";
-	return `<select class="prb-index-author" name="author" data-author-filter aria-label="Filter by pull request author">
-      <option value="">Everyone</option>${authors.map((handle) => `<option value="${escapeHtml$1(handle)}">@${escapeHtml$1(handle)}</option>`).join("")}
-    </select>`;
+	return `<label class="prb-index-status prb-index-forge">${GITHUB_MARK}Author:
+      <select name="author" data-author-filter aria-label="Filter by pull request author">
+        <option value="">Everyone</option>${authors.map((author) => `<option value="${escapeHtml$1(author.handle)}">${escapeHtml$1(authorOptionLabel(author))}</option>`).join("")}
+      </select>
+    </label>`;
 }
 /**
 * One section's Load more, which the browser shows once that section holds more than it is showing.
@@ -8899,8 +9078,8 @@ function renderBulkBar() {
       <button type="button" class="prb-bulk-button" data-bulk-select-all>${escapeHtml$1(selectAllLabel(0))}</button>
       <button type="button" class="prb-bulk-button" data-bulk-fetch hidden>${escapeHtml$1(FETCH_SELECTED_LABEL)}</button>
       <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>
+      <button type="button" class="prb-bulk-button" data-bulk-mark="complete">${markItemLabel("complete")}</button>
+      <button type="button" class="prb-bulk-button" data-bulk-mark="in-progress">${markItemLabel("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>
@@ -9096,9 +9275,9 @@ function renderHistory(group, now) {
 * branch started. The same numbering the workspace itself uses, so "Review 1 of 3" there and
 * "Review 1" here are the same review.
 *
-* **Only the latest row carries a workflow status.** `New`, `In progress` and `Reviewed` describe
+* **Only the latest row carries a workflow status.** `New`, `Not reviewed` and `Reviewed` describe
 * where someone is in a piece of work, and on a superseded review that is a claim about work
-* nobody should be continuing: a row reading `IN PROGRESS` under a newer review of the same branch
+* nobody should be continuing: a row reading `NOT REVIEWED` under a newer review of the same branch
 * invites exactly the wrong click. The second slot is the lineage instead, `Latest` or `Previous`,
 * which is the only thing about an older review that a reader needs before opening it.
 */
@@ -11583,17 +11762,17 @@ function handleNotificationOpen(req, res, params, indexToken) {
 	return redirect(res, `/r/${workspace.id}?t=${encodeURIComponent(workspace.token)}${hash}`);
 }
 /**
-* Mark a selection reviewed, or put it back to in progress.
+* Mark a selection reviewed, or put it back to not reviewed.
 *
 * Authorised by the index credential, like pinning and deleting, because this is the list acting on
 * itself. Deliberately not `/api/<id>/state`, which needs a review's own token: routing a bulk mark
 * through it would mean putting fifteen credentials into one request, and the index is the one page
 * that holds them all.
 *
-* **Marking in progress fills in `lastViewedAt` where there is none.** Clearing `completedAt` alone
+* **Marking not reviewed fills in `lastViewedAt` where there is none.** Clearing `completedAt` alone
 * would leave a review that was never opened showing `New`, and a button that names a state has to
-* produce it. Saying a change is in progress is a claim that the work has started, which is what
-* that field records. It is only ever filled in, never moved, so a review that has been opened
+* produce it. Taking the mark back is a claim that the work has started, which is what that field
+* records. It is only ever filled in, never moved, so a review that has been opened
 * keeps the time it was opened.
 *
 * No lock and no running-job check, unlike deletion. A mark is a note about where the reviewer is,
@@ -12936,7 +13115,11 @@ 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;
+	if (typeof body?.complete === "boolean") {
+		const now = Date.now();
+		workspace.completedAt = body.complete ? now : void 0;
+		if (!body.complete) workspace.lastViewedAt = workspace.lastViewedAt ?? now;
+	}
 	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. */