prreviewbuddy 0.63.0 → 0.69.1

package/dist/server.js

@@ -1,7 +1,9 @@
-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-CeJo9t7h.js";
+import { $ as MAX_WORKSPACE_PORT, $n as isClaimed, $r as resolveAgent, $t as markReadForReview, A as relativeTime$1, An as isWorktreeHeld, Ar as parseLifecycle, At as partFiles, B as readIndexToken, Bn as reviewedShaOf, Br as issuesOutstanding, Bt as isArchitectureId, C as track, Cn as originRemote, Cr as activityAttributes, Ct as mapRuntimes, D as endRun, Dn as MANAGED_ROOT, Dr as lifecycleItemLabel, Dt as ARCHITECTURE_DIR, E as completeRun, En as ReviewBeingDeletedError, Er as lifecycleChip, Et as describeRuntimes, F as runBatch, Fr as queueStamp, Ft as trackedFiles, Gn as loadJob, Gr as describeAuthorship, Gt as createArchitectureNotification, H as clearServerRecordOwnedBy, Hn as endedAt, Hr as reviewerDispositions, Ht as announceJob, I as activateQueued, In as findExistingReview, Ir as readQueue, It as ownersOf, J as stopChild, Jn as isLiveJob, Jr as resolveTarget, Jt as eventName, K as serverEntryExists, Kn as saveJob, Kr as WITHHELD_REASON, Kt as createPrState, L as compactQueue, Ln as checkCodeFreshness, Lr as doneVerb, Lt as QUOTE_LIMITS, M as extractPullRequestLinks, Mn as removeWorktree, Mr as queueAdd, Mt as repositoryKey, N as parsePullRequestLink, Nn as requestJobStop, Nr as queueAddMany, Nt as storeArchitecture, O as reportRun, On as createWorktree, Or as lifecyclePill, Ot as checkArchitecture, P as unsupportedReason, Pr as queueRemove, Pt as storedScopes, Q as DEFAULT_WORKSPACE_PORT, Qr as git, Qt as markRead, Rn as checkReview, Rr as isIssueOutstanding, Rt as REPOSITORY_SCOPE, S as isDriving, Sn as remoteHost, Sr as LIFECYCLES, St as waitingHistory, T as beginRun, Tr as isLifecycle, Tt as RUNTIME_WORDS, Un as fail, Ur as processDiscussion, Ut as branchOf, Vn as allJobs, Vr as questionsOutstanding, Vt as writeJsonAtomically, W as readServerRecord, Wn as isTerminal, Wr as recommendationsOf, Wt as subjectOf, Xn as progressSteps, Xr as isCommitReview, Xt as listNotifications, Yn as phasesFor, Yr as DetachedHeadError, Yt as headline, Z as writeServerRecord, Zn as purposeOf, Zr as resolveBase, Zt as markAllRead, _ as liveUpdateFor, _n as extractJsonText, _t as telemetryUploadConsent, a as wasBlocked, ai as STORE_ROOT, an as withUsageRecorded, ar as kissDecisionRefusal, at as failureOf$1, bn as forgeResolver, br as workspaceRevision, c as RefreshUnavailableError, ci as agentFor$1, cn as authorPreview, cr as lineagePosition, ct as reanalyseReview, d as mergeUpdateInto, di as AgentUnavailableError, dn as renderMarkdown, dr as mutateWorkspace, ei as resolveModel, en as pruneNotifications, er as decideKissRecommendation, et as MIN_WORKSPACE_PORT, f as updateReview, fn as CODE_BLOCK_CLASS, ft as readEvents, g as UpdateAlreadyRunningError, gn as asString, gr as reviewedCommit, gt as recordTelemetryUploadConsent, h as askCheckout, hn as renderInlineText, hr as recentWorkspaces, i as liveJobsFor, in as unreadCount$1, ir as indexReviewGroups, it as writePortPreference, j as batchIdentity, jn as readMarker, jr as sectionOf, jt as readArchitecture, k as stopRun, kn as holdWorktree, kr as offersLifecycle, kt as loadArchitecture, l as cancelUpdate, li as detectAgents, ln as hasAuthorContent, lr as loadWorkspace, lt as runJob, m as isUnchanged, mn as escapeHtml$1, mr as previousKissResult, n as deleteReview, ni as readAgentPreference, nn as removeNotification, nr as groupByLineage, nt as resolveWorkspacePort, o as discardJob, oi as AGENT_IDS, on as EXPLAIN_SIMPLY_PROMPT, or as kissDecisionsOf, ot as prepareReview, p as refreshPrContext, pn as CODE_COPY_CLASS, pr as presentedKissRunOf, pt as record, q as spawnServer, qr as kissOutcome, qt as createPrUpdate, r as lineageIds, ri as writeAgentPreference, rn as toneOf, rr as indexKeptIds, rt as statedWorkspacePort, s as FETCH_TIMEOUT_MS, si as agentById, sn as UNESTABLISHED_MOTIVATION, sr as latestKissRun, st as settleReview, t as deleteLineage, ti as clearAgentPreference, tn as readNotification, tr as followedRefName, tt as parseWorkspacePort, u as cancellableHere, ui as AgentCancelledError, un as GITHUB_MARKDOWN, vn as objectWithAnyOf, w as RUN_KINDS, wn as fillFileUrlTemplate, wr as activityLabel, wt as recordRuntimeBestEffort, x as startKissJob, xn as forgeFor, xr as workspaceStamps, xt as feedbackUrl, y as agentEnvOf, yn as claimRequestUrl, yr as touchWorkspace, yt as BUILD_VERSION, z as settleQueue, zn as followedBranch, zr as isQuestionOutstanding, zt as ancestorsOf } from "./delete_review-CRPRIRKg.js";
 import { basename, isAbsolute, join, relative, resolve } from "node:path";
-import { existsSync, mkdirSync, readFileSync, readdirSync, rmSync, statSync, writeFileSync } from "node:fs";
+import { execFile } from "node:child_process";
+import { existsSync, mkdirSync, readFileSync, readdirSync, realpathSync, rmSync, statSync, writeFileSync } from "node:fs";
 import { createHash, randomUUID, timingSafeEqual } from "node:crypto";
+import { homedir } from "node:os";
 import { fileURLToPath } from "node:url";
 import { createServer } from "node:http";
 //#region ../../packages/review-harness/src/domain/ai_review/review_composition.ts
@@ -322,11 +324,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 +360,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 +397,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 +448,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 +1068,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 +2532,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 +2635,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 +2687,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 +5589,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 +5647,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 +5817,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 +5926,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 +6057,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 +6210,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 +6418,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>`;
 }
@@ -7075,7 +7243,7 @@ function startArchitectureRun(workspace, scope, deps = {}) {
 	const now = deps.now ?? Date.now;
 	const startedAt = now();
 	const env = agentEnvOf(workspace);
-	const agent = agentFor(env);
+	const agent = agentFor$1(env);
 	const requestedModel = resolveModel({ review: { session: workspace.session } }) ?? void 0;
 	const record = {
 		version: 1,
@@ -7427,7 +7595,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 +8943,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 +8973,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 +8993,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 +9009,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 +9080,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 +9277,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.
 */
@@ -10193,6 +10374,656 @@ function reviewsRevision(live = analysisStates()) {
 	return hash.digest("base64url");
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/working_folders.ts
+/**
+* The local Git working folders PR Review Buddy can find right now.
+*
+* One question and nothing more: which folders on this machine could a review run in. The answer is
+* derived each time from what is already known, the folders past reviews ran in and the other
+* worktrees of those repositories, and nothing is stored. A worktree an agent made yesterday shows
+* up because `git worktree list` names it, not because anything remembered it.
+*
+* A folder is a folder. A linked worktree is its own entry with its own path and is never folded
+* into the clone it belongs to: a review is filed under the folder it ran in, and the worktree is
+* where the branch is checked out.
+*
+* Not a registry. If one is wanted later it replaces `discoverWorkingFolders` and keeps the shape.
+*/
+function canonical(path) {
+	try {
+		return realpathSync(path);
+	} catch {
+		return path;
+	}
+}
+/** `git worktree list --porcelain`, main worktree first, bare and prunable entries left out. */
+async function worktreesOf(repoPath) {
+	let output;
+	try {
+		output = await git(repoPath, [
+			"worktree",
+			"list",
+			"--porcelain"
+		]);
+	} catch {
+		return [];
+	}
+	const entries = [];
+	for (const block of output.split(/\n\s*\n/)) {
+		const lines = block.split("\n");
+		const path = lines.find((line) => line.startsWith("worktree "))?.slice(9);
+		if (!path || lines.includes("bare") || lines.some((line) => line.startsWith("prunable"))) continue;
+		const ref = lines.find((line) => line.startsWith("branch "))?.slice(7) ?? null;
+		entries.push({
+			path,
+			branch: ref ? ref.replace(/^refs\/heads\//, "") : null
+		});
+	}
+	return entries;
+}
+async function discoverWorkingFolders(summaries = recentWorkspaces(Number.MAX_SAFE_INTEGER)) {
+	const lastReviewed = /* @__PURE__ */ new Map();
+	for (const summary of summaries) {
+		if (!summary.repoPath) continue;
+		const key = canonical(summary.repoPath);
+		lastReviewed.set(key, Math.max(lastReviewed.get(key) ?? 0, summary.createdAt));
+	}
+	const found = /* @__PURE__ */ new Map();
+	for (const seed of lastReviewed.keys()) {
+		if (found.has(seed) || !existsSync(seed)) continue;
+		(await worktreesOf(seed)).forEach((entry, index) => {
+			const key = canonical(entry.path);
+			if (!found.has(key) && existsSync(key)) found.set(key, {
+				entry: {
+					...entry,
+					path: key
+				},
+				main: index === 0
+			});
+		});
+	}
+	return (await Promise.all([...found.values()].map(async ({ entry, main }) => ({
+		path: entry.path,
+		remote: await originRemote(entry.path),
+		branch: entry.branch,
+		main,
+		lastReviewedAt: lastReviewed.get(entry.path) ?? null
+	})))).sort((a, b) => (b.lastReviewedAt ?? 0) - (a.lastReviewedAt ?? 0) || Number(b.main) - Number(a.main) || a.path.localeCompare(b.path));
+}
+/** Whether a remote is this github.com repository. GitHub names are case-insensitive. */
+function isGitHubRepository(remote, owner, repo) {
+	return Boolean(remote && remote.host === "github.com" && remote.owner.toLowerCase() === owner.toLowerCase() && remote.repo.toLowerCase() === repo.toLowerCase());
+}
+/**
+* The folders a pull request could be reviewed in, the suggested one first.
+*
+* Every match is returned, because which one is right is the reviewer's call: an agent's worktree
+* and the main clone hold the same objects and differ in what is checked out. The order only
+* decides the default. A folder that already has a review of this pull request comes first, so
+* reviewing it again finds that review rather than starting a second one filed under another
+* folder; then the main worktree; then whichever was reviewed in most recently.
+*/
+function candidatesFor(folders, link, summaries) {
+	const reviewedHere = new Set(summaries.filter((summary) => summary.prNumber === link.number).map((summary) => canonical(summary.repoPath)));
+	const matching = folders.filter((folder) => isGitHubRepository(folder.remote, link.owner, link.repo));
+	const rank = (folder) => reviewedHere.has(folder.path) ? 0 : folder.main ? 1 : 2;
+	return [...matching].sort((a, b) => rank(a) - rank(b) || (b.lastReviewedAt ?? 0) - (a.lastReviewedAt ?? 0));
+}
+/** The review of this pull request already filed under one of these folders, if there is one. */
+function reviewedIn(folders, link, summaries) {
+	const paths = new Set(folders.map((folder) => folder.path));
+	return summaries.find((summary) => summary.prNumber === link.number && paths.has(canonical(summary.repoPath))) ?? null;
+}
+//#endregion
+//#region ../../packages/review-harness/src/workspace/choose_folder.ts
+/**
+* Ask the person at this machine to pick a folder, with the operating system's own picker.
+*
+* The workspace is a page served by a local daemon, and a page cannot learn a folder's path from
+* the browser's directory picker, so the daemon asks instead. The operation is "choose a working
+* folder"; how a platform does it stays in this file, so neither the routes nor the page know.
+*
+* Only macOS has an implementation, through `osascript`. Elsewhere there is no chooser, and the
+* Add review panel falls back to the path field and the Recent list, which is all v1 needs.
+*/
+/** Long enough for somebody to browse; short enough that a forgotten dialog does not hold a request for ever. */
+var CHOOSER_TIMEOUT_MS = 3e5;
+function folderChooser(platform = process.platform) {
+	if (platform !== "darwin") return null;
+	return () => new Promise((resolve, reject) => {
+		execFile("osascript", ["-e", "POSIX path of (choose folder with prompt \"Choose the working folder to review\")"], { timeout: CHOOSER_TIMEOUT_MS }, (error, stdout, stderr) => {
+			if (error) return /-128/.test(String(stderr)) ? resolve(null) : reject(/* @__PURE__ */ new Error("The folder picker could not be opened."));
+			resolve(stdout.trim().replace(/\/$/, "") || null);
+		});
+	});
+}
+function createSlots(capacity) {
+	let free = capacity;
+	const waiting = [];
+	const releaser = () => {
+		let given = false;
+		return () => {
+			if (given) return;
+			given = true;
+			const next = waiting.shift();
+			if (next) next(releaser());
+			else free += 1;
+		};
+	};
+	return {
+		acquire: () => {
+			if (free > 0) {
+				free -= 1;
+				return Promise.resolve(releaser());
+			}
+			return new Promise((resolve) => waiting.push(resolve));
+		},
+		tryAcquire: () => {
+			if (free === 0 || waiting.length > 0) return null;
+			free -= 1;
+			return releaser();
+		}
+	};
+}
+/** How many pull requests one submission may name. See `MAX_PULL_REQUESTS_PER_BATCH`. */
+var MAX_BATCH = 20;
+/** How long a finished batch stays answerable, for a panel that is still open on it. */
+var FINISHED_BATCH_TTL_MS = 6e5;
+function realAddReviewDeps() {
+	return {
+		resolveAgent: (flag) => resolveAgent(flag ? { flag } : {}),
+		agents: async () => (await detectAgents()).map((agent) => ({
+			id: agent.id,
+			displayName: agent.displayName,
+			installed: agent.state === "installed"
+		})),
+		folders: () => discoverWorkingFolders(),
+		summaries: () => recentWorkspaces(Number.MAX_SAFE_INTEGER),
+		remoteOf: originRemote,
+		resolvePullRequest: (folder, url) => resolveTarget({
+			repoPath: folder,
+			prUrl: url,
+			resolveRequest: forgeResolver
+		}),
+		resolveBranch: (folder, base) => resolveTarget({
+			repoPath: folder,
+			...base ? { base } : {}
+		}),
+		defaultBase: async (root) => {
+			try {
+				const base = await resolveBase(root, void 0);
+				return (base.followRef ?? base.display).replace(/^refs\/(?:heads|remotes)\//, "");
+			} catch {
+				return null;
+			}
+		},
+		findExisting: findExistingReview,
+		prepare: prepareReview,
+		settle: (workspaceId) => settleReview(workspaceId, "standard"),
+		track,
+		chooseFolder: folderChooser(),
+		concurrency: 2,
+		slots: createSlots(2),
+		resumeOf: (reviewId) => {
+			const failure = failureOf$1(reviewId);
+			return failure ? {
+				agent: agentNameOf(failure.agentId),
+				...failure.requestedModel ? { model: failure.requestedModel } : {}
+			} : null;
+		},
+		agentName: agentNameOf,
+		home: homedir()
+	};
+}
+function agentNameOf(agentId) {
+	try {
+		return agentById(agentId).displayName;
+	} catch {
+		return agentId;
+	}
+}
+var refuse = (status, error) => ({
+	status,
+	body: { error }
+});
+/** A path as a person reads it, with their home folder as `~`. */
+function displayPath(path, home) {
+	return home && (path === home || path.startsWith(`${home}/`)) ? `~${path.slice(home.length)}` : path;
+}
+/** A path a person typed, made absolute: `~` is their home folder, and anything relative is refused. */
+function expandPath(raw, home) {
+	const trimmed = raw.trim().replace(/(.)\/+$/, "$1");
+	if (!trimmed) return null;
+	const expanded = trimmed === "~" ? home : trimmed.startsWith("~/") ? `${home}${trimmed.slice(1)}` : trimmed;
+	return isAbsolute(expanded) ? expanded : null;
+}
+function message(error) {
+	return error instanceof Error ? error.message : String(error);
+}
+/** Why no agent could be used, in a sentence for the panel. The CLI's own wording names flags. */
+function unresolvedAgent(resolution) {
+	switch (resolution.kind) {
+		case "none": return "No review agent is installed on this machine. Install one of the agents PR Review Buddy supports, then try again.";
+		case "unknown-id": return `"${resolution.id}" is not an agent PR Review Buddy knows. Choose another under Review options, or change the default in Settings.`;
+		case "unavailable": return `${resolution.agent.displayName} can't be used from the workspace: ${resolution.reason}`;
+	}
+}
+/**
+* The agent a submission runs under: the one named under Review options, or else the same answer
+* the CLI reaches with no flag (the configured default, or the only one installed). Never the
+* harness's own fallback: an id is always passed down, so `startJob` never has to guess.
+*/
+async function agentFor(flag, deps) {
+	if (flag !== void 0 && typeof flag !== "string") return refuse(400, "Say which agent by its id.");
+	const resolution = await deps.resolveAgent(flag || void 0);
+	if (resolution.kind === "resolved") return { agentId: resolution.agent.id };
+	if (resolution.kind === "ambiguous") return refuse(400, "More than one review agent is installed. Choose one under Review options.");
+	return refuse(resolution.kind === "unknown-id" ? 400 : 409, unresolvedAgent(resolution));
+}
+function modelOf(value) {
+	if (value === void 0 || value === null || value === "") return void 0;
+	if (typeof value !== "string" || value.length > 200) return refuse(400, "The model must be a short name.");
+	return value.trim() || void 0;
+}
+/**
+* The tail of each target's queue of creations, by batch identity. In-process, like the fetch
+* queue in `target.ts`.
+*
+* `prepareReview` looks for an existing review and then creates one, and nothing between the two
+* stops a second start of the same target, from another tab or another paste, from looking in the
+* same gap and creating a second review of it. One at a time per target closes that gap for every
+* start this server makes; the second one finds the first one's review.
+*/
+var creations = /* @__PURE__ */ new Map();
+function oneCreationAtATime(target, work) {
+	const key = batchIdentity(target);
+	const mine = (creations.get(key) ?? Promise.resolve()).then(work, work);
+	const tail = mine.catch(() => {});
+	creations.set(key, tail);
+	tail.then(() => {
+		if (creations.get(key) === tail) creations.delete(key);
+	});
+	return mine;
+}
+var isAnswer = (value) => typeof value === "object" && value !== null && "status" in value && "body" in value;
+async function addReviewOptions(deps) {
+	const [agents, resolution] = await Promise.all([deps.agents(), deps.resolveAgent()]);
+	return {
+		status: 200,
+		body: {
+			agents,
+			agent: resolution.kind === "resolved" ? {
+				kind: "resolved",
+				id: resolution.agent.id,
+				displayName: resolution.agent.displayName
+			} : resolution.kind === "ambiguous" ? { kind: "ambiguous" } : {
+				kind: "refused",
+				error: unresolvedAgent(resolution)
+			},
+			canChooseFolder: deps.chooseFolder !== null
+		}
+	};
+}
+function reviewedRow(reviewId, deps) {
+	const kept = deps.resumeOf(reviewId);
+	return {
+		reviewId,
+		...kept ? { kept } : {}
+	};
+}
+function folderOption(folder, home) {
+	return {
+		path: folder.path,
+		display: displayPath(folder.path, home),
+		branch: folder.branch
+	};
+}
+/**
+* What a paste would start, without starting anything: rows, the folders each could run in, and
+* which already have a review. No forge is asked and nothing is fetched, so it is cheap enough to
+* run on every pause in typing.
+*
+* Every link gets a row, however many there are. Past the batch limit the panel refuses to start
+* rather than this quietly dropping the tail: a reviewer who pasted twenty-one pull requests and
+* got twenty reviews would not find out which one was missing.
+*/
+async function previewPullRequests(text, deps) {
+	if (typeof text !== "string") return refuse(400, "Paste the pull request links to review.");
+	const links = extractPullRequestLinks(text);
+	const [folders, summaries] = links.some((link) => link.kind === "github") ? [await deps.folders(), deps.summaries()] : [[], []];
+	return {
+		status: 200,
+		body: {
+			rows: links.map((link) => {
+				if (link.kind === "unsupported") return {
+					key: link.key,
+					label: `${link.provider} ${link.provider === "GitLab" ? "merge request" : "pull request"}`,
+					url: link.url,
+					ok: false,
+					reason: unsupportedReason(link.provider),
+					folders: []
+				};
+				const label = `${link.owner}/${link.repo} #${link.number}`;
+				const candidates = candidatesFor(folders, link, summaries);
+				if (candidates.length === 0) return {
+					key: link.key,
+					label,
+					url: link.url,
+					ok: false,
+					reason: "Local repository not found",
+					needsFolder: true,
+					folders: []
+				};
+				const reviewed = reviewedIn(candidates, link, summaries);
+				return {
+					key: link.key,
+					label,
+					url: link.url,
+					ok: true,
+					folders: candidates.map((folder) => folderOption(folder, deps.home)),
+					folder: candidates[0].path,
+					...reviewed ? { reviewed: reviewedRow(reviewed.id, deps) } : {}
+				};
+			}),
+			limit: MAX_BATCH
+		}
+	};
+}
+var batches = /* @__PURE__ */ new Map();
+function forgetFinished(now) {
+	for (const [id, batch] of batches) if (batch.finishedAt !== null && now - batch.finishedAt > FINISHED_BATCH_TTL_MS) batches.delete(id);
+}
+function newBatch(rows) {
+	forgetFinished(Date.now());
+	const batch = {
+		id: randomUUID(),
+		rows,
+		finishedAt: null
+	};
+	batches.set(batch.id, batch);
+	return batch;
+}
+var rowsOf = (batch) => batch.rows.map((row) => ({ ...row }));
+/**
+* Create the review for a row's target, or find the one it has, under the per-target lock. The
+* caller holds an analysis slot: nothing is created without one, so nothing created waits.
+*/
+async function prepareRow(row, target, agentId, model, deps) {
+	const prepared = await oneCreationAtATime(target, () => deps.prepare(target, agentId, false, model));
+	row.reviewId = prepared.kind === "new" ? prepared.workspaceId : prepared.existing.id;
+	if (prepared.kind === "existing") row.status = "already-reviewed";
+	else row.status = "reviewing";
+	if (prepared.kind === "resume") row.kept = keptOf(prepared.failure.agentId, prepared.failure.requestedModel, deps);
+	return prepared;
+}
+async function settleRow(row, deps) {
+	const settled = await deps.settle(row.reviewId);
+	if (settled.ok) row.status = "ready";
+	else {
+		row.status = "failed";
+		row.reason = settled.stopped ? "Stopped" : settled.reason;
+	}
+}
+function keptOf(agentId, model, deps) {
+	return {
+		agent: deps.agentName(agentId),
+		...model ? { model } : {}
+	};
+}
+/**
+* Start a batch and answer at once. The analyses take minutes, and the panel is not a progress
+* dialog: it may watch the batch while it stays open, and the reviews appear in the list either way.
+*
+* Each pull request waits for one of the server's analysis slots (`deps.slots`), shared with every
+* other batch and every branch review, and is resolved and created only once it has one.
+*/
+async function startPullRequestBatch(body, deps) {
+	const raw = Array.isArray(body?.items) ? body.items : null;
+	if (!raw || raw.length === 0) return refuse(400, "Say which pull requests to review.");
+	if (raw.length > MAX_BATCH) return refuse(400, `At most ${MAX_BATCH} pull requests can be started at once.`);
+	const items = [];
+	for (const entry of raw) {
+		const url = typeof entry?.url === "string" ? entry.url : "";
+		const link = parsePullRequestLink(url);
+		if (!link || link.kind !== "github") return refuse(400, `${url || "That"} is not a github.com pull request link.`);
+		const folder = typeof entry?.folder === "string" ? expandPath(entry.folder, deps.home) : null;
+		if (!folder) return refuse(400, `Choose a local folder for ${link.owner}/${link.repo} #${link.number}.`);
+		items.push({
+			url: link.url,
+			owner: link.owner,
+			repo: link.repo,
+			number: link.number,
+			key: link.key,
+			folder
+		});
+	}
+	const model = modelOf(body?.model);
+	if (isAnswer(model)) return model;
+	const agent = await agentFor(body?.agent, deps);
+	if (isAnswer(agent)) return agent;
+	const batch = newBatch(items.map((item) => ({
+		key: item.key,
+		label: `${item.owner}/${item.repo} #${item.number}`,
+		status: "waiting"
+	})));
+	/** Which row claimed each pull request, by what it resolved to rather than how it was spelled. */
+	const claimed = /* @__PURE__ */ new Map();
+	const lane = async (item, index) => {
+		const row = batch.rows[index];
+		const release = await deps.slots.acquire();
+		row.status = "reviewing";
+		try {
+			const remote = await deps.remoteOf(item.folder);
+			if (!isGitHubRepository(remote, item.owner, item.repo)) {
+				row.status = "failed";
+				row.reason = remote ? `This folder is a clone of ${remote.owner}/${remote.repo}, not ${item.owner}/${item.repo}` : `This folder is not a clone of ${item.owner}/${item.repo}`;
+				return;
+			}
+			const target = await deps.resolvePullRequest(item.folder, item.url);
+			const identity = batchIdentity(target);
+			const first = claimed.get(identity);
+			if (first !== void 0) {
+				row.status = "already-reviewed";
+				row.reason = `Same pull request as ${batch.rows[first].label}`;
+				return;
+			}
+			claimed.set(identity, index);
+			if ((await prepareRow(row, target, agent.agentId, model, deps)).kind !== "existing") await settleRow(row, deps);
+		} catch (error) {
+			row.status = "failed";
+			row.reason = message(error);
+		} finally {
+			release();
+		}
+	};
+	deps.track(runBatch(items, {
+		concurrency: deps.concurrency,
+		run: lane
+	})).catch(() => {}).finally(() => {
+		batch.finishedAt = Date.now();
+	});
+	return {
+		status: 200,
+		body: {
+			batchId: batch.id,
+			rows: rowsOf(batch)
+		}
+	};
+}
+/**
+* A batch's rows as they stand. A batch this server does not know was either finished long ago,
+* which an open panel never asks about because it stops asking once a batch finishes, or lost to a
+* restart, which is the case worth saying plainly: whatever had not started never will.
+*/
+function batchRows(id) {
+	const batch = batches.get(id);
+	if (!batch) return refuse(404, "The workspace server no longer knows this batch, most likely because it restarted. Reviews that had started are in your list. Any that were still waiting did not start.");
+	return {
+		status: 200,
+		body: {
+			rows: rowsOf(batch),
+			finished: batch.finishedAt !== null
+		}
+	};
+}
+/** A target refusal in the panel's words where the CLI's would name a command. */
+function targetRefusal(error) {
+	if (error instanceof DetachedHeadError) return "This folder is on a detached HEAD, so there is no branch to review. Check out a branch in it, or review its pull request instead.";
+	return message(error);
+}
+/** The recent and discoverable folders, for the Branch panel's list. */
+async function listFolders(deps) {
+	return {
+		status: 200,
+		body: { folders: (await deps.folders()).map((folder) => ({
+			...folderOption(folder, deps.home),
+			repo: folder.remote ? folder.remote.repo : basename(folder.path)
+		})) }
+	};
+}
+/**
+* What reviewing this folder would review, read without changing anything: the same target a bare
+* `prreviewbuddy` here would resolve, and the review it would find.
+*
+* With the base the reviewer named, when they named one, so what this checks is the comparison
+* that would be submitted. Without it, a repository whose default base is ambiguous could never be
+* reviewed from here, though naming the base is exactly what settles it.
+*/
+async function inspectFolder(rawPath, deps, rawBase) {
+	const path = typeof rawPath === "string" ? expandPath(rawPath, deps.home) : null;
+	if (!path) return {
+		status: 200,
+		body: {
+			ok: false,
+			error: "Enter the full path of a folder, or choose one."
+		}
+	};
+	const named = typeof rawBase === "string" && rawBase.trim() ? rawBase.trim() : void 0;
+	let target;
+	try {
+		target = await deps.resolveBranch(path, named);
+	} catch (error) {
+		return {
+			status: 200,
+			body: {
+				ok: false,
+				error: targetRefusal(error)
+			}
+		};
+	}
+	const [remote, base, existing] = await Promise.all([
+		deps.remoteOf(target.originRepoPath),
+		named ? Promise.resolve(named) : deps.defaultBase(target.originRepoPath),
+		deps.findExisting(target)
+	]);
+	const kept = existing ? deps.resumeOf(existing.id) : null;
+	return {
+		status: 200,
+		body: {
+			ok: true,
+			path: target.originRepoPath,
+			display: displayPath(target.originRepoPath, deps.home),
+			repo: remote ? remote.repo : basename(target.originRepoPath),
+			branch: target.branch,
+			base,
+			...existing ? { reviewed: {
+				reviewId: existing.id,
+				inProgress: existing.inProgress,
+				...kept ? { kept } : {}
+			} } : {}
+		}
+	};
+}
+/**
+* Review the branch checked out in a folder: `prreviewbuddy` run there.
+*
+* With a free analysis slot it answers once the review exists, and the analysis runs on in this
+* process, counted by `track` like a Retry. With none free it answers `waiting` and a one-row batch
+* to watch, and creates nothing until a slot frees: a review created to wait is a job nobody is
+* running, which the sweep would fail. A review the branch already has needs no slot to hand back.
+*/
+async function startBranchReview(body, deps) {
+	const path = typeof body?.path === "string" ? expandPath(body.path, deps.home) : null;
+	if (!path) return refuse(400, "Choose the working folder to review.");
+	const base = typeof body?.base === "string" && body.base.trim() ? body.base.trim() : void 0;
+	const model = modelOf(body?.model);
+	if (isAnswer(model)) return model;
+	let target;
+	try {
+		target = await deps.resolveBranch(path, base);
+	} catch (error) {
+		return refuse(400, targetRefusal(error));
+	}
+	const agent = await agentFor(body?.agent, deps);
+	if (isAnswer(agent)) return agent;
+	const row = {
+		key: batchIdentity(target),
+		label: target.branch,
+		status: "waiting"
+	};
+	const release = deps.slots.tryAcquire();
+	if (!release) {
+		const existing = await deps.findExisting(target);
+		if (existing && !deps.resumeOf(existing.id)) return {
+			status: 200,
+			body: {
+				outcome: "existing",
+				reviewId: existing.id
+			}
+		};
+		const batch = newBatch([row]);
+		deps.track((async () => {
+			const later = await deps.slots.acquire();
+			try {
+				if ((await prepareRow(row, target, agent.agentId, model, deps)).kind !== "existing") await settleRow(row, deps);
+			} catch (error) {
+				row.status = "failed";
+				row.reason = message(error);
+			} finally {
+				later();
+			}
+		})()).finally(() => {
+			batch.finishedAt = Date.now();
+		});
+		return {
+			status: 200,
+			body: {
+				outcome: "waiting",
+				batchId: batch.id
+			}
+		};
+	}
+	let prepared;
+	try {
+		prepared = await prepareRow(row, target, agent.agentId, model, deps);
+	} catch (error) {
+		release();
+		return refuse(409, message(error));
+	}
+	if (prepared.kind === "existing") release();
+	else deps.track(deps.settle(row.reviewId)).catch(() => {}).finally(release);
+	return {
+		status: 200,
+		body: {
+			outcome: prepared.kind,
+			reviewId: row.reviewId,
+			...row.kept ? { kept: row.kept } : {}
+		}
+	};
+}
+async function chooseFolder(deps) {
+	if (!deps.chooseFolder) return refuse(404, "Choosing a folder is not available on this system. Type its path instead.");
+	try {
+		const path = await deps.chooseFolder();
+		return {
+			status: 200,
+			body: path ? { path } : { cancelled: true }
+		};
+	} catch (error) {
+		return refuse(500, message(error));
+	}
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/ask.ts
 /** Long enough for a real look at the repository; short enough that a hang is not forever. */
 var ASK_TIMEOUT_MS = 18e4;
@@ -10233,7 +11064,7 @@ async function askAboutChange(workspace, question, sectionId, signal) {
 	const env = agentEnvOf(workspace);
 	const requestedModel = resolveModel({ review: { session: workspace.session } }) ?? void 0;
 	try {
-		return extractAnswer(await withUsageRecorded(workspace.id, "ask", env, (onUsage) => agentFor(env).run({
+		return extractAnswer(await withUsageRecorded(workspace.id, "ask", env, (onUsage) => agentFor$1(env).run({
 			capability: "read-code",
 			prompt: buildPrompt$2(workspace, question, sectionId, cwd),
 			cwd,
@@ -10269,7 +11100,7 @@ async function askAboutChange(workspace, question, sectionId, signal) {
 function explained(error, env) {
 	if (!(error instanceof Error) || error instanceof AgentCancelledError) return error;
 	try {
-		const plain = agentFor(env).explainFailure?.(error.message, env);
+		const plain = agentFor$1(env).explainFailure?.(error.message, env);
 		return plain ? new AskFailedError(plain) : error;
 	} catch {
 		return error;
@@ -10381,7 +11212,7 @@ async function drawDiagram(workspace, sectionId, signal) {
 	const requestedModel = resolveModel({ review: { session: workspace.session } }) ?? void 0;
 	let raw;
 	try {
-		raw = await withUsageRecorded(workspace.id, "diagram", env, (onUsage) => agentFor(env).run({
+		raw = await withUsageRecorded(workspace.id, "diagram", env, (onUsage) => agentFor$1(env).run({
 			capability: "read-code",
 			prompt: buildPrompt$1(workspace, sectionId, cwd),
 			cwd,
@@ -10568,7 +11399,7 @@ async function drawChangeMap(workspace, signal) {
 	const requestedModel = resolveModel({ review: { session: workspace.session } }) ?? void 0;
 	let raw;
 	try {
-		raw = await withUsageRecorded(workspace.id, "change-map", env, (onUsage) => agentFor(env).run({
+		raw = await withUsageRecorded(workspace.id, "change-map", env, (onUsage) => agentFor$1(env).run({
 			capability: "read-code",
 			prompt: buildPrompt(workspace, cwd),
 			cwd,
@@ -11266,6 +12097,7 @@ async function handle(req, res, options) {
 	if (url.pathname === "/fetch-updates") return handleFetchUpdates(req, res, options.indexToken);
 	if (url.pathname === "/shutdown") return handleIndexShutdown(req, res, options);
 	if (url.pathname === "/restart") return handleIndexRestart(req, res, options);
+	if (url.pathname.startsWith("/add-review/")) return handleAddReview(req, res, url.pathname, options);
 	const match = /^\/(?:r|api)\/([0-9a-f-]{36})(\/[a-z]+)?$/.exec(url.pathname);
 	if (!match) return send(res, 404, { error: "Not found" });
 	const [, id, action] = match;
@@ -11583,17 +12415,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,
@@ -11823,6 +12655,43 @@ async function handleFetchUpdates(req, res, indexToken) {
 	send(res, 200, await runFetchUpdates(new Set(ids)));
 }
 /**
+* Add review: start pull request and branch reviews from any workspace page.
+*
+* The index credential, as the cookie, because starting a review is the machine's business rather
+* than any one review's, the same authority as `/queue` and `/fetch-updates`. The cookie is set on
+* `Path=/`, so a review page carries it too. See `add_review.ts` for what each route does.
+*/
+async function handleAddReview(req, res, pathname, options) {
+	if (!options.indexToken || !secretMatches(options.indexToken, indexCookie(req))) return send(res, 403, { error: "Not authorised." });
+	const deps = options.addReview ?? (addReviewDeps ??= realAddReviewDeps());
+	const route = pathname.slice(12);
+	const answer = async () => {
+		if (req.method === "GET") {
+			if (route === "options") return addReviewOptions(deps);
+			if (route === "folders") return listFolders(deps);
+			const batch = /^batches\/([0-9a-f-]{36})$/.exec(route);
+			if (batch) return batchRows(batch[1]);
+			return null;
+		}
+		if (req.method !== "POST") return {
+			status: 405,
+			body: { error: "Method not allowed." }
+		};
+		if (route === "folder/choose") return chooseFolder(deps);
+		const body = await readJson(req);
+		if (route === "pull-requests/preview") return previewPullRequests(body?.text, deps);
+		if (route === "pull-requests") return startPullRequestBatch(body, deps);
+		if (route === "folder/inspect") return inspectFolder(body?.path, deps, body?.base);
+		if (route === "branch") return startBranchReview(body, deps);
+		return null;
+	};
+	const result = await answer();
+	if (!result) return send(res, 404, { error: "Not found." });
+	send(res, result.status, result.body);
+}
+/** Built on first use, so a server nobody adds a review on never runs the detectors. */
+var addReviewDeps;
+/**
 * Fetch remote updates, from one review's own page, on its own token.
 *
 * The list's fetch scoped to this review: `GET` says whether it applies (the page draws the item
@@ -12936,7 +13805,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. */