prreviewbuddy 0.27.2 → 0.30.0

package/dist/server.js

@@ -1,4 +1,4 @@
-import { $ as fillFileUrlTemplate, $t as agentFor, A as removeWorktree, At as workspaceRevision, C as recordTelemetryUploadConsent, Ct as recentReviewGroups, D as ReviewBeingDeletedError, Dt as saveWorkspace, E as EXPLAIN_SIMPLY_PROMPT, F as clearServerRecord, Ft as issuesOutstanding, G as DEFAULT_WORKSPACE_PORT, Gt as clearAgentPreference, H as BUILD_VERSION, Ht as git, It as questionsOutstanding, J as parseWorkspacePort, K as MAX_WORKSPACE_PORT, Kt as readAgentPreference, L as readServerRecord, Lt as reviewerDispositions, M as relativeTime$1, Mt as doneVerb, N as readIndexToken, Nt as isIssueOutstanding, O as MANAGED_ROOT, Pt as isQuestionOutstanding, Rt as processDiscussion, S as record, St as previousKissResult, T as withUsageRecorded, Tt as reviewedCommit, V as writeServerRecord, W as feedbackUrl, Wt as resolveModel, X as statedWorkspacePort, Xt as AGENT_IDS, Y as resolveWorkspacePort, Yt as STORE_ROOT, Z as writePortPreference, _ as startKissJob, a as wasBlocked, at as isTerminal, c as updateReview, d as askCheckout, dt as purposeOf, et as checkCodeFreshness, f as UpdateAlreadyRunningError, gt as latestKissRun, h as agentEnvOf, i as liveJobsFor, it as fail, jt as workspaceStamps, k as readMarker, kt as touchWorkspace, l as refreshPrContext, lt as phasesFor, mt as followedRefName, n as deleteReview, nn as AgentUnavailableError, nt as allJobs, o as discardJob, ot as loadJob, p as liveUpdateFor, pt as isClaimed, q as MIN_WORKSPACE_PORT, qt as writeAgentPreference, r as lineageIds, rt as endedAt, s as RefreshUnavailableError, st as saveJob, t as deleteLineage, tn as AgentCancelledError, tt as checkFreshness, u as isUnchanged, ut as progressSteps, vt as lineagePosition, w as telemetryUploadConsent, x as readEvents, y as runJob, yt as loadWorkspace, zt as describeAuthorship } from "./delete_review-B3xciJ03.js";
+import { $ as fillFileUrlTemplate, $t as writeAgentPreference, A as removeWorktree, At as workspaceRevision, Bt as isQuestionOutstanding, C as recordTelemetryUploadConsent, Ct as previousKissResult, D as ReviewBeingDeletedError, Dt as saveWorkspace, E as EXPLAIN_SIMPLY_PROMPT, F as clearServerRecord, Ft as queueStamp, G as DEFAULT_WORKSPACE_PORT, Gt as describeAuthorship, H as BUILD_VERSION, Ht as questionsOutstanding, It as readQueue, J as parseWorkspacePort, Jt as git, K as MAX_WORKSPACE_PORT, L as readServerRecord, Lt as writeQueue, M as relativeTime$1, Mt as dequeueOnComplete, N as readIndexToken, Nt as queueAdd, O as MANAGED_ROOT, Pt as queueRemove, Qt as readAgentPreference, Rt as doneVerb, S as record, T as withUsageRecorded, Tt as reviewedCommit, Ut as reviewerDispositions, V as writeServerRecord, Vt as issuesOutstanding, W as feedbackUrl, Wt as processDiscussion, X as statedWorkspacePort, Xt as resolveModel, Y as resolveWorkspacePort, Z as writePortPreference, Zt as clearAgentPreference, _ as startKissJob, _t as latestKissRun, a as wasBlocked, an as agentFor, at as isTerminal, bt as loadWorkspace, c as updateReview, cn as AgentUnavailableError, d as askCheckout, dt as purposeOf, et as checkCodeFreshness, f as UpdateAlreadyRunningError, gt as indexReviewGroups, h as agentEnvOf, i as liveJobsFor, it as fail, jt as workspaceStamps, k as readMarker, kt as touchWorkspace, l as refreshPrContext, lt as phasesFor, mt as followedRefName, n as deleteReview, nn as AGENT_IDS, nt as allJobs, o as discardJob, ot as loadJob, p as liveUpdateFor, pt as isClaimed, q as MIN_WORKSPACE_PORT, r as lineageIds, rt as endedAt, s as RefreshUnavailableError, sn as AgentCancelledError, st as saveJob, t as deleteLineage, tn as STORE_ROOT, tt as checkFreshness, u as isUnchanged, ut as progressSteps, w as telemetryUploadConsent, wt as recentWorkspaces, x as readEvents, y as runJob, yt as lineagePosition, zt as isIssueOutstanding } from "./delete_review-B_2J2Wn9.js";
 import { isAbsolute, join, relative, resolve } from "node:path";
 import { existsSync, mkdirSync, readFileSync, readdirSync, statSync, writeFileSync } from "node:fs";
 import { createHash, timingSafeEqual } from "node:crypto";
@@ -229,6 +229,36 @@ function wayBackSentence(way = wayBack()) {
 	return way.commands.map(({ command, where }) => `${command} in ${where}`).join(", or ");
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/rail_icons.ts
+/**
+* The rail's icons, one per row that is not a theme.
+*
+* A pure module, because the client draws the Diagrams row itself when the first drawing lands and
+* cannot import `page.ts`. Themes keep their numbers: the order is information, and an icon there
+* would replace it with decoration.
+*
+* Stroked in `currentColor` on a 24-unit grid, so every icon takes the row's colour, including
+* Stop server's red on hover.
+*/
+var PATHS = {
+	review: "<path d=\"M9 5H7a2 2 0 0 0-2 2v12a2 2 0 0 0 2 2h10a2 2 0 0 0 2-2V7a2 2 0 0 0-2-2h-2\"/><rect x=\"9\" y=\"3\" width=\"6\" height=\"4\" rx=\"1\"/><path d=\"m9 14 2 2 4-4\"/>",
+	kiss: "<path d=\"M12 3v4M12 17v4M3 12h4M17 12h4\"/><circle cx=\"12\" cy=\"12\" r=\"2\"/>",
+	diagrams: "<rect x=\"3\" y=\"3\" width=\"7\" height=\"6\" rx=\"1\"/><rect x=\"14\" y=\"15\" width=\"7\" height=\"6\" rx=\"1\"/><path d=\"M6.5 9v3a2 2 0 0 0 2 2h7a2 2 0 0 1 2 2v-1\"/>",
+	issues: "<circle cx=\"12\" cy=\"12\" r=\"9\"/><path d=\"M12 8v4M12 16h.01\"/>",
+	questions: "<circle cx=\"12\" cy=\"12\" r=\"9\"/><path d=\"M9.1 9a3 3 0 0 1 5.8 1c0 2-3 3-3 3M12 17h.01\"/>",
+	files: "<path d=\"M14 3H7a2 2 0 0 0-2 2v14a2 2 0 0 0 2 2h10a2 2 0 0 0 2-2V8z\"/><path d=\"M14 3v5h5M9 13h6M9 17h6\"/>",
+	prcontext: "<path d=\"M21 15a2 2 0 0 1-2 2H7l-4 4V5a2 2 0 0 1 2-2h14a2 2 0 0 1 2 2z\"/>",
+	recommendations: "<path d=\"M9 18h6M10 22h4\"/><path d=\"M12 2a7 7 0 0 0-4 12.7c.6.5 1 1.2 1 2V17h6v-.3c0-.8.4-1.5 1-2A7 7 0 0 0 12 2z\"/>",
+	reviews: "<path d=\"M8 6h13M8 12h13M8 18h13M3 6h.01M3 12h.01M3 18h.01\"/>",
+	settings: "<circle cx=\"12\" cy=\"12\" r=\"3\"/><path d=\"M19.4 15a1.7 1.7 0 0 0 .3 1.8l.1.1a2 2 0 1 1-2.8 2.8l-.1-.1a1.7 1.7 0 0 0-1.8-.3 1.7 1.7 0 0 0-1 1.5V21a2 2 0 1 1-4 0v-.1a1.7 1.7 0 0 0-1.1-1.5 1.7 1.7 0 0 0-1.8.3l-.1.1a2 2 0 1 1-2.8-2.8l.1-.1a1.7 1.7 0 0 0 .3-1.8 1.7 1.7 0 0 0-1.5-1H3a2 2 0 1 1 0-4h.1a1.7 1.7 0 0 0 1.5-1.1 1.7 1.7 0 0 0-.3-1.8l-.1-.1a2 2 0 1 1 2.8-2.8l.1.1a1.7 1.7 0 0 0 1.8.3H9a1.7 1.7 0 0 0 1-1.5V3a2 2 0 1 1 4 0v.1a1.7 1.7 0 0 0 1 1.5 1.7 1.7 0 0 0 1.8-.3l.1-.1a2 2 0 1 1 2.8 2.8l-.1.1a1.7 1.7 0 0 0-.3 1.8V9a1.7 1.7 0 0 0 1.5 1H21a2 2 0 1 1 0 4h-.1a1.7 1.7 0 0 0-1.5 1z\"/>",
+	feedback: "<path d=\"m22 2-7 20-4-9-9-4z\"/><path d=\"M22 2 11 13\"/>",
+	stop: "<path d=\"M12 2v10\"/><path d=\"M18.4 6.6a9 9 0 1 1-12.8 0\"/>"
+};
+/** The icon slot for one rail row. Hidden from assistive technology: the label beside it says it. */
+function railIcon(icon) {
+	return `<span class="prb-rail-icon" aria-hidden="true"><svg viewBox="0 0 24 24" focusable="false" fill="none" stroke="currentColor" stroke-width="1.75" stroke-linecap="round" stroke-linejoin="round">${PATHS[icon]}</svg></span>`;
+}
+//#endregion
 //#region ../../packages/review-harness/src/domain/utils/file_path.ts
 /**
 * Splits a repo-relative path for display, so the filename can be shown prominently and the
@@ -1363,7 +1393,7 @@ function renderGuideSectionCard(section, counts, activeGuidedSectionId, status,
 	const statusBadge = isCompleted ? `<span class="pr-helper-guide-badge pr-helper-completed-badge">Completed</span>` : status === "active" ? `<span class="pr-helper-guide-badge pr-helper-start-here-badge">${escapeHtml$1(activeBadgeLabel)}</span>` : "";
 	const undo = isCompleted ? `<button type="button" class="pr-helper-guide-uncomplete" data-guide-uncomplete="${escapeHtml$1(section.id)}">Mark as not reviewed</button>` : "";
 	return `
-    <section class="pr-helper-guide-section${isCompleted ? " pr-helper-guide-section-completed" : ""}" id="pr-helper-guide-section-${escapeHtml$1(section.id)}" data-guide-section-card="${escapeHtml$1(section.id)}">
+    <section class="pr-helper-guide-section${isCompleted ? " pr-helper-guide-section-completed" : ""}${options?.collapsedSections ? " pr-helper-guide-section-collapsed" : ""}" id="pr-helper-guide-section-${escapeHtml$1(section.id)}" data-guide-section-card="${escapeHtml$1(section.id)}">
       <button type="button" class="pr-helper-guide-section-header" data-guide-toggle="${escapeHtml$1(section.id)}">
         <div class="pr-helper-guide-section-badges">
           ${check}
@@ -3483,7 +3513,10 @@ function renderStandardViews(view, sections) {
 		diagrams: true,
 		detailsInHeader: true
 	})}</section>
-      <section class="prb-view" data-view="guide" hidden>${renderGuideTab(session, null, null, { diagrams: true })}</section>
+      <section class="prb-view" data-view="guide" hidden>${renderGuideTab(session, null, null, {
+		diagrams: true,
+		collapsedSections: true
+	})}</section>
       <section class="prb-view" data-view="diagram" hidden data-diagram-pane${view.pendingDiagram ? ` data-diagram-running="${escapeHtml$1(view.pendingDiagram.sectionId ?? "")}"` : ""}>
         <h3 class="prb-doc-title">Diagrams</h3>
         ${renderDiagrams(view.diagrams ?? [], sections, view.reviewedSha ?? null)}
@@ -3652,7 +3685,7 @@ function renderHeader(view, freshness) {
 		return `<button type="button" class="prb-fresh-check" data-fresh-check data-state="${state}" aria-expanded="false" title="How this review stands against the branch as it is now">${escapeHtml$1(label)}</button>`;
 	})() : ""}
     ${view.token ? `<button type="button" class="prb-complete" data-complete aria-pressed="${view.complete === true}">${view.complete ? "Reviewed" : "Mark reviewed"}</button>` : ""}
-    ${view.id ? renderIdMenu(view.id) : ""}
+    ${view.id ? renderIdMenu(view.id, Boolean(view.token)) : ""}
     ${view.session.result ? "<button type=\"button\" class=\"prb-ask-toggle\" data-ask-toggle aria-expanded=\"false\">Buddy AI</button>" : ""}
   </div>
 </header>`;
@@ -3667,17 +3700,29 @@ function renderHeader(view, freshness) {
 * the address bar.
 *
 * The same `...` and the same menu markup as the index, deliberately. One gesture, in both places
-* it can be wanted, behaving identically. Copy alone and no delete: removing the review you are
-* currently reading is a different act needing somewhere to send you afterwards, and the list is
-* one click away in the rail.
+* it can be wanted, behaving identically. That includes deleting, with the same confirmation in the
+* same panel and the same sentence. The list is where you land afterwards, since the page you were
+* reading no longer exists. It deletes this review only, as a history row on the list does.
+*
+* No delete where there is no server, which is the offline preview: the item would be a lie there.
 */
-function renderIdMenu(id) {
+function renderIdMenu(id, canDelete) {
+	const safe = escapeHtml$1(id);
 	return `<div class="prb-menu" data-menu>
     <button type="button" class="prb-menu-trigger" data-menu-trigger aria-expanded="false" aria-haspopup="true" aria-label="Actions for this review">&#8943;</button>
     <div class="prb-menu-panel" data-menu-panel hidden>
       <div data-menu-items>
-        <button type="button" class="prb-menu-item" data-copy-id="${escapeHtml$1(id)}">Copy review ID</button>
+        <button type="button" class="prb-menu-item" data-copy-id="${safe}">Copy review ID</button>
+        ${canDelete ? `<button type="button" class="prb-menu-item prb-menu-item-danger" data-delete="${safe}">Delete review</button>` : ""}
       </div>
+      ${canDelete ? `<div class="prb-menu-confirm" data-menu-confirm hidden>
+        <p class="prb-menu-question">Delete this review?</p>
+        <p class="prb-menu-note">This cannot be undone. Your branch and its commits are untouched.</p>
+        <div class="prb-menu-buttons">
+          <button type="button" class="prb-menu-item prb-menu-item-danger" data-confirm-delete>Delete</button>
+          <button type="button" class="prb-menu-item" data-cancel-delete>Cancel</button>
+        </div>
+      </div>` : ""}
     </div>
   </div>`;
 }
@@ -3699,22 +3744,22 @@ function renderIdMenu(id) {
 */
 function renderRailFoot(view) {
 	return `<div class="prb-rail-group prb-rail-foot">
-    ${view.token ? `<a class="prb-rail-item" href="/">
-             <span class="prb-rail-order">&lsaquo;</span>
+    ${view.token ? `<a class="prb-rail-item prb-rail-link" href="/">
+             ${railIcon("reviews")}
              <span class="prb-rail-label"><span class="prb-rail-title">All reviews</span></span>
            </a>
-     <a class="prb-rail-item" href="/settings">
-       <span class="prb-rail-order">&#9881;</span>
+     <a class="prb-rail-item prb-rail-link" href="/settings">
+       ${railIcon("settings")}
        <span class="prb-rail-label"><span class="prb-rail-title">Settings</span></span>
      </a>` : ""}
-    <a class="prb-rail-item" href="${escapeHtml$1(feedbackUrl())}" target="_blank" rel="noopener noreferrer">
-      <span class="prb-rail-order"></span>
+    <a class="prb-rail-item prb-rail-link" href="${escapeHtml$1(feedbackUrl())}" target="_blank" rel="noopener noreferrer">
+      ${railIcon("feedback")}
       <span class="prb-rail-label"><span class="prb-rail-title">Send feedback</span></span>
     </a>
   </div>
   <div class="prb-rail-group prb-rail-ops">
-    ${view.token ? `<button type="button" class="prb-rail-item prb-rail-danger" data-shutdown>
-             <span class="prb-rail-order"></span>
+    ${view.token ? `<button type="button" class="prb-rail-item prb-rail-link prb-rail-danger" data-shutdown>
+             ${railIcon("stop")}
              <span class="prb-rail-label"><span class="prb-rail-title">Stop server</span></span>
            </button>` : ""}
     <p class="prb-build">v${escapeHtml$1(BUILD_VERSION)}</p>
@@ -3766,6 +3811,7 @@ function renderRail(view, sections) {
   ${(view.session.result ? railGroup({
 		id: "prb-rail-review",
 		title: "Review",
+		icon: "review",
 		view: "overview",
 		current: true,
 		badge: null,
@@ -3777,11 +3823,11 @@ function renderRail(view, sections) {
     <div class="prb-rail-path" id="prb-rail-path">
       ${themes || "<p class=\"prb-rail-heading\">No themes</p>"}
     </div>
-    ${(view.diagrams?.length ?? 0) > 0 ? railLink("diagram", "Diagrams", diagramsBadge(view.diagrams?.length ?? 0)) : ""}
-    ${railLink("issues", "Issues", findingsBadge(issuesOutstanding(issues, dispositions)))}
-    ${railLink("questions", "Questions", remainingBadge(questionsOutstanding(questions, answered), "question", "answered"))}
-    ${railLink("files", "Files", totalBadge(view.changeSet.data.files.length, "files"))}
-    ${view.prContext ? railLink("prcontext", "PR context", totalBadge(view.prContext.processed.stats.totalThreads, "PR comment threads")) : railLink("prcontext", "PR context", null, "No pull request was connected to this review")}`
+    ${(view.diagrams?.length ?? 0) > 0 ? railLink("diagram", "diagrams", "Diagrams", diagramsBadge(view.diagrams?.length ?? 0)) : ""}
+    ${railLink("issues", "issues", "Issues", findingsBadge(issuesOutstanding(issues, dispositions)))}
+    ${railLink("questions", "questions", "Questions", remainingBadge(questionsOutstanding(questions, answered), "question", "answered"))}
+    ${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")}`
 	}) : "") || renderRunningRail(view)}
   ${renderKissRail(view)}
   ${renderRailFoot(view)}
@@ -3807,11 +3853,12 @@ function renderRunningRail(view) {
 	return railGroup({
 		id: "prb-rail-review",
 		title: "Review",
+		icon: "review",
 		view: "progress",
 		current: true,
 		badge: null,
 		collapses: "the review",
-		body: view.job && changeIsRead(view.job) ? railLink("files", "Files", totalBadge(files, "files")) : ""
+		body: view.job && changeIsRead(view.job) ? railLink("files", "files", "Files", totalBadge(files, "files")) : ""
 	});
 }
 /**
@@ -3834,11 +3881,12 @@ function renderKissRail(view) {
 	return railGroup({
 		id: "prb-rail-kiss",
 		title: "KISS",
+		icon: "kiss",
 		view: "kiss",
 		current: view.session.result === null && !showsProgress(view),
 		badge: result ? recommendationsBadge(result.recommendations.length) : null,
 		collapses: "the KISS review",
-		body: result ? railLink("kissrecs", "Recommendations", recommendationsBadge(result.recommendations.length)) : ""
+		body: result ? railLink("kissrecs", "recommendations", "Recommendations", recommendationsBadge(result.recommendations.length)) : ""
 	});
 }
 /** The one chevron both twisties draw. Rotated by CSS off `aria-expanded`, never by markup. */
@@ -3858,7 +3906,7 @@ function railGroup(group) {
 	return `<div class="prb-rail-group">
     <div class="prb-rail-row">
       <button type="button" class="prb-rail-item prb-rail-group-title" data-view-nav="${escapeHtml$1(group.view)}"${group.current ? " aria-current=\"true\"" : ""}>
-        <span class="prb-rail-order"></span>
+        ${railIcon(group.icon)}
         <span class="prb-rail-label"><span class="prb-rail-title">${escapeHtml$1(group.title)}</span></span>
         ${renderBadge(group.badge)}
       </button>
@@ -3944,9 +3992,9 @@ 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, label, badge, note) {
-	return `<button type="button" class="prb-rail-item" data-view-nav="${view}"${note ? ` title="${escapeHtml$1(note)}"` : ""}>
-    <span class="prb-rail-order"></span>
+function railLink(view, icon, label, badge, note) {
+	return `<button type="button" class="prb-rail-item prb-rail-link" data-view-nav="${view}"${note ? ` title="${escapeHtml$1(note)}"` : ""}>
+    ${railIcon(icon)}
     <span class="prb-rail-label"><span class="prb-rail-title">${escapeHtml$1(label)}</span></span>
     ${renderBadge(badge)}
   </button>`;
@@ -4276,6 +4324,40 @@ 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.
+*/
+function authorOptions(handles) {
+	return [...new Set(handles.filter((handle) => Boolean(handle)))].sort((a, b) => a.localeCompare(b, void 0, { sensitivity: "base" }));
+}
+/**
+* A card menu's status item, which sets the state it names. The bulk bar's words for the same two
+* actions, so a card and a selection never disagree. Here rather than in the page renderer because
+* 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";
+}
+/**
+* What `Continue reviewing` opens, and whether it exists at all.
+*
+* One rule rather than an expression in the template, because "is there anything to continue" and
+* "what is first" are the same question and must not be able to answer differently -- a button that
+* is present but navigates nowhere is worse than no button.
+*/
+function continueTarget(queue) {
+	return queue[0] ?? null;
+}
+/**
+* The Queue pill, which carries a number only once there is one.
+*
+* "Queue 0" is a count of something that is not there, and reads as a fault. Bare "Queue" reads as
+* the place it is -- empty, and still the thing you would click to put something in it.
+*/
+function queuePillLabel(queued) {
+	return queued === 0 ? "Queue" : `Queue ${queued}`;
+}
+/**
 * The words on the status chip.
 *
 * Here rather than in the renderer because the browser now writes this chip too: marking a
@@ -4308,6 +4390,10 @@ function renderReviewsPage(view) {
 	const now = view.now ?? Date.now();
 	const ordered = [...view.groups].sort((a, b) => Number(b.latest.pinned) - Number(a.latest.pinned) || touchedAt(b.latest) - touchedAt(a.latest));
 	const storedReviews = view.groups.reduce((total, group) => total + group.history.length, 0);
+	const queue = view.queue ?? [];
+	const positions = new Map(queue.map((id, position) => [id, position]));
+	const continues = continueTarget(queue);
+	const front = continues === null ? void 0 : findReview(view.groups, continues);
 	return `<!doctype html>
 <html lang="en">
 <head>
@@ -4329,18 +4415,19 @@ function renderReviewsPage(view) {
   </div>
 </header>
 <main class="prb-index">
-  ${ordered.length === 0 ? "" : renderControls()}
+  ${ordered.length === 0 ? "" : renderControls(queue.length, authorOptions(ordered.map((group) => group.latest.authorHandle)))}
+  ${front === void 0 ? "" : renderContinue(front)}
   ${ordered.length === 0 ? "" : `<div class="prb-index-summary">
     <p class="prb-index-count" data-count>${escapeHtml$1(countLabel(storedReviews, ordered.length))}</p>
     <button type="button" class="prb-index-select" data-select-mode aria-pressed="false">Select</button>
   </div>`}
   <div class="prb-cards" data-cards>
-    ${ordered.map((group) => renderCard(group, now, view.analyses?.get(group.latest.id))).join("")}
+    ${ordered.map((group) => renderCard(group, now, view.analyses?.get(group.latest.id), positions.get(group.latest.id))).join("")}
   </div>
   ${ordered.length === 0 ? `<p class="prb-index-empty">No reviews stored yet. Ask Claude Code to review a change and one will appear here.</p>` : `<p class="prb-index-none" data-no-matches hidden>Nothing matches that.</p>`}
   ${ordered.length > 15 ? renderPager() : ""}
   ${ordered.length === 0 ? "" : renderBulkBar()}
-  <p class="prb-index-note">Reviews are kept for thirty days, on this machine only. Every review has a fixed address, so a link to one is worth keeping and still opens after a restart. Stopping the server deletes nothing; it only stops anything being served until the next review starts it again.</p>
+  <p class="prb-index-note">Reviews are kept for thirty days, on this machine only. Saved reviews and anything in your review queue are kept for ninety days instead, counting from the last time you opened them. Every review has a fixed address, so a link to one is worth keeping and still opens after a restart. Stopping the server deletes nothing; it only stops anything being served until the next review starts it again.</p>
 </main>
 <script src="/assets/reviews.js" type="module"><\/script>
 </body>
@@ -4355,17 +4442,51 @@ function renderReviewsPage(view) {
 * 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() {
+function renderControls(queued, authors) {
 	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)}
     <div class="prb-index-filters" role="group" aria-label="Filter reviews">
       <button type="button" data-filter="all" aria-pressed="true">All</button>
+      <button type="button" data-filter="queue" aria-pressed="false" data-queue-pill>${escapeHtml$1(queuePillLabel(queued))}</button>
+      <button type="button" data-filter="saved" aria-pressed="false">Saved</button>
       <button type="button" data-filter="in-progress" aria-pressed="false">In progress</button>
       <button type="button" data-filter="complete" aria-pressed="false">Reviewed</button>
     </div>
   </div>`;
 }
 /**
+* Whose pull requests to show.
+*
+* 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>`;
+}
+/**
+* The way back into the batch, above the list.
+*
+* A link rather than a button, and carrying the review's own token, because that is what every
+* other way into a review on this page is -- the list is the only thing holding every credential.
+*
+* It names the front of the queue rather than saying only "Continue reviewing", so somebody
+* returning after an hour can tell whether it is about to open what they expect before they click.
+*/
+function renderContinue(review) {
+	return `<a class="prb-index-continue" href="${escapeHtml$1(`/r/${encodeURIComponent(review.id)}?t=${encodeURIComponent(review.token)}`)}">
+    <span class="prb-index-continue-label">Continue reviewing</span>
+    <span class="prb-index-continue-title">${escapeHtml$1(review.title)}</span>
+  </a>`;
+}
+/** The review a queued id refers to. The caller has already guaranteed it is one this page shows. */
+function findReview(groups, id) {
+	for (const group of groups) for (const review of group.history) if (review.id === id) return review;
+}
+/**
 * The pager, with its numbers left out.
 *
 * How many pages there are depends on what the search matched, and matching happens in the browser
@@ -4436,15 +4557,15 @@ function renderBulkBar() {
 * card and is shown only in selection mode, so entering that mode is a stylesheet change rather
 * than a rebuild of the list.
 */
-function renderCard(group, now, analysis) {
+function renderCard(group, now, analysis, queued) {
 	const review = group.latest;
 	const status = statusOf(review);
 	const href = `/r/${encodeURIComponent(review.id)}?t=${encodeURIComponent(review.token)}`;
 	const followed = followedRefName(review.branch, review.headResolvedRef) ?? review.branch;
-	return `<article class="prb-card" data-card data-id="${escapeHtml$1(review.id)}" data-status="${status}" data-pinned="${review.pinned}" data-reviews="${group.history.length}" data-search-text="${escapeHtml$1(groupSearchText(group))}">
+	return `<article class="prb-card" data-card data-id="${escapeHtml$1(review.id)}" data-status="${status}" data-pinned="${review.pinned}"${queued === void 0 ? "" : ` data-queue="${queued}"`}${review.authorHandle ? ` data-author="${escapeHtml$1(review.authorHandle)}"` : ""} data-reviews="${group.history.length}" data-search-text="${escapeHtml$1(groupSearchText(group))}">
     <input type="checkbox" class="prb-card-select" data-select="${escapeHtml$1(review.id)}" aria-label="Select ${escapeHtml$1(followed)}">
-    <button type="button" class="prb-card-pin" data-pin="${escapeHtml$1(review.id)}" aria-pressed="${review.pinned}" aria-label="${review.pinned ? "Unpin" : "Pin"} this review" title="${review.pinned ? "Unpin" : "Pin to the top"}">${review.pinned ? "&#9733;" : "&#9734;"}</button>
-    ${renderCardMenu(group, followed)}
+    <button type="button" class="prb-card-pin" data-pin="${escapeHtml$1(review.id)}" aria-pressed="${review.pinned}" aria-label="${review.pinned ? "Unsave" : "Save"} this review">${review.pinned ? "&#9733;" : "&#9734;"}</button>
+    ${renderCardMenu(group, followed, queued !== void 0, status)}
     <a class="prb-card-link" href="${escapeHtml$1(href)}">
       <div class="prb-card-head">
         <h2>${escapeHtml$1(review.title)}</h2>
@@ -4487,15 +4608,18 @@ function renderCard(group, now, analysis) {
 *
 * The rows underneath carry the per-review menus, and they only exist once there is more than one.
 */
-function renderCardMenu(group, followed) {
+function renderCardMenu(group, followed, queued, status) {
 	const count = group.history.length;
+	const mark = status === "complete" ? "in-progress" : "complete";
 	if (count < 2) return renderMenu({
 		id: group.latest.id,
 		scope: "review",
 		label: `Actions for ${followed}`,
 		action: "Delete review",
 		question: "Delete this review?",
-		copy: "Copy review ID"
+		copy: "Copy review ID",
+		queue: queued ? "remove" : "add",
+		mark
 	});
 	const all = `all ${count} reviews following ${followed}`;
 	return renderMenu({
@@ -4504,6 +4628,8 @@ function renderCardMenu(group, followed) {
 		label: `Actions for ${followed}`,
 		action: `Delete ${all}`,
 		question: `Delete ${all}?`,
+		queue: queued ? "remove" : "add",
+		mark,
 		copy: "Copy latest review ID"
 	});
 }
@@ -4522,10 +4648,14 @@ function renderCardMenu(group, followed) {
 */
 function renderMenu(item) {
 	const id = escapeHtml$1(item.id);
+	const markItem = item.mark === void 0 ? "" : `<button type="button" class="prb-menu-item" data-mark-status="${item.mark}" data-mark-id="${id}">${markItemLabel(item.mark)}</button>`;
+	const queueItem = item.queue === void 0 ? "" : `<button type="button" class="prb-menu-item" data-queue-action="${item.queue}" data-queue-id="${id}">${item.queue === "remove" ? "Remove from review queue" : "Add to review queue"}</button>`;
 	return `<div class="prb-menu" data-menu>
     <button type="button" class="prb-menu-trigger" data-menu-trigger aria-expanded="false" aria-haspopup="true" aria-label="${escapeHtml$1(item.label)}">&#8943;</button>
     <div class="prb-menu-panel" data-menu-panel hidden>
       <div data-menu-items>
+        ${markItem}
+        ${queueItem}
         <button type="button" class="prb-menu-item" data-copy-id="${id}">${escapeHtml$1(item.copy)}</button>
         <button type="button" class="prb-menu-item prb-menu-item-danger" data-delete="${id}" data-scope="${item.scope}">${escapeHtml$1(item.action)}</button>
       </div>
@@ -4928,27 +5058,31 @@ function renderReviewsSection(settings) {
   </section>`;
 }
 /**
-* The telemetry answer, visible.
+* The telemetry setting, visible.
 *
 * It has always been changeable, and only from the command line, which is a poor place for the one
-* setting a person is most likely to want to check rather than to set. Three states rather than
-* two: never having answered is not the same as having said no, and a page that showed it as "off"
-* would be answering a question on somebody's behalf.
+* setting a person is most likely to want to check rather than to set.
+*
+* Two radios over three stored states. Until 0.30.0 the third state meant nothing was being sent,
+* so neither radio could be checked without answering on somebody's behalf. Now it means the
+* default is applying and the default is on, so Send it is checked and the help line says which of
+* the two reasons it is on for. The storage keeps all three, because a stored no has to outrank any
+* future change of default.
 */
 function renderPrivacySection(settings) {
-	const on = settings.usageCounts === true;
 	const off = settings.usageCounts === false;
 	return `<section class="prb-settings-section">
     <h2>Privacy</h2>
     <div class="prb-field prb-settings-field" role="group" aria-labelledby="prb-usage-label" aria-describedby="prb-usage-help">
-      <span id="prb-usage-label">Anonymous usage counts</span>
+      <span id="prb-usage-label">Non-identifying usage telemetry</span>
       <div class="prb-settings-choice">
-        <label><input type="radio" name="prb-usage" value="on" data-settings-usage${on ? " checked" : ""}> Send them</label>
-        <label><input type="radio" name="prb-usage" value="off" data-settings-usage${off ? " checked" : ""}> Do not send them</label>
+        <label><input type="radio" name="prb-usage" value="on" data-settings-usage${off ? "" : " checked"}> Send it</label>
+        <label><input type="radio" name="prb-usage" value="off" data-settings-usage${off ? " checked" : ""}> Do not send it</label>
       </div>
     </div>
-    <p class="prb-settings-help" id="prb-usage-help">Counts only, never your code, your diffs, your
-    questions or the names of anything you review.${settings.usageCounts === null ? " You have not answered this yet, so nothing is being sent." : ""}</p>
+    <p class="prb-settings-help" id="prb-usage-help">When a review starts and finishes, with the
+    version, platform, agent, whether it succeeded and how long it took. Never your code, your diffs,
+    your questions or the names of anything you review.${settings.usageCounts === null ? " It is on by default; you have not changed it." : ""}</p>
   </section>`;
 }
 //#endregion
@@ -5075,6 +5209,7 @@ function reviewsRevision(live = analysisStates()) {
 		const state = live.get(id);
 		hash.update(`${id}@${state.purpose}:${state.phase}${state.stopped ? ":stopped" : ""}\n`);
 	}
+	hash.update(`queue:${queueStamp()}\n`);
 	return hash.digest("base64url");
 }
 //#endregion
@@ -5903,6 +6038,7 @@ async function handle(req, res, options) {
 	if (url.pathname === "/open") return handleOpen(req, res, url.searchParams, options.indexToken);
 	if (url.pathname === "/settings") return void handleSettings(req, res, url.searchParams, options.indexToken);
 	if (url.pathname === "/pin") return handlePin(req, res, options.indexToken);
+	if (url.pathname === "/queue") return handleQueue(req, res, options.indexToken);
 	if (url.pathname === "/delete") return handleDelete(req, res, options.indexToken);
 	if (url.pathname === "/status") return handleStatus(req, res, options.indexToken);
 	if (url.pathname === "/revision") return handleRevision(req, res, options.indexToken);
@@ -5928,6 +6064,7 @@ async function handle(req, res, options) {
 	if (action === "/stop") return handleStop(req, res, workspace);
 	if (action === "/kiss") return handleKiss(req, res, workspace);
 	if (action === "/state") return handleState(req, res, workspace);
+	if (action === "/delete") return handleDeleteOwn(req, res, workspace);
 	if (action === "/event") return handleEvent(req, res, workspace);
 	if (action === "/prefs") return handlePrefs(req, res);
 	if (action === "/prompts") return handlePrompts(req, res);
@@ -5990,8 +6127,14 @@ function handleIndex(req, res, params, indexToken) {
 	const cookie = indexCookieHeader(indexToken);
 	if (onward) return redirect(res, onward, cookie);
 	const states = analysisStates();
+	const queue = readQueue();
+	const kept = new Set(queue);
+	for (const review of recentWorkspaces(Number.MAX_SAFE_INTEGER)) if (review.pinned) kept.add(review.id);
+	const groups = indexReviewGroups(MAX_LISTED_BRANCHES, MAX_KEPT_BRANCHES, kept);
+	const rendered = new Set(groups.flatMap((group) => group.history.map((review) => review.id)));
 	sendHtml(res, 200, renderReviewsPage({
-		groups: recentReviewGroups(MAX_LISTED_BRANCHES),
+		groups,
+		queue: queue.filter((id) => rendered.has(id)),
 		analyses: states,
 		revision: reviewsRevision(states)
 	}), cookie);
@@ -6144,6 +6287,7 @@ async function handleStatus(req, res, indexToken) {
 		workspace.completedAt = complete ? now : void 0;
 		if (!complete) workspace.lastViewedAt = workspace.lastViewedAt ?? now;
 		saveWorkspace(workspace);
+		dequeueOnComplete(id, complete);
 		changed += 1;
 	}
 	send(res, 200, {
@@ -6197,6 +6341,32 @@ async function handlePin(req, res, indexToken) {
 	send(res, 200, { ok: true });
 }
 /**
+* Adding to and taking out of the working set.
+*
+* The index cookie for the same reason as `handlePin` above: this is the list acting on itself, and
+* a review's own token must not be able to rearrange the other reviews around it.
+*
+* The answer carries the queue **as stored**, which is not always the queue the page asked for --
+* the cap applies, duplicates collapse, and a review somebody deleted in another tab drops out on
+* the next write. The browser paints from this rather than from its own optimism, which is the one
+* thing that keeps a pill claiming "Queue 4" honest.
+*
+* An unrecognised action is a 400 rather than a no-op 200. A route that silently accepts a word it
+* does not implement is how a later `top` appears to work while doing nothing.
+*/
+async function handleQueue(req, res, indexToken) {
+	if (!indexToken || !secretMatches(indexToken, indexCookie(req))) return send(res, 403, { error: "Not authorised." });
+	const body = await readJson(req);
+	const id = typeof body?.id === "string" ? body.id : null;
+	const action = body?.action;
+	if (action !== "add" && action !== "remove") return send(res, 400, { error: "Unknown queue action." });
+	if (!id || !loadWorkspace(id)) return send(res, 404, { error: "No such review" });
+	const stored = action === "add" ? queueAdd(id) : queueRemove(id);
+	if (stored === null) return send(res, 500, { error: "The queue could not be saved." });
+	const live = stored.filter((queued) => loadWorkspace(queued) !== null);
+	send(res, 200, { queue: live.length === stored.length ? stored : writeQueue(live) ?? stored });
+}
+/**
 * Delete one review, or everything on one card.
 *
 * Authorised the same way pinning is, and for the same reason: this is the list acting on itself,
@@ -6224,7 +6394,35 @@ async function handleDelete(req, res, indexToken) {
 	const live = (wholeCard ? lineageIds(id) : [id]).flatMap((each) => liveJobsFor(each));
 	if (live.length > 0) return send(res, 409, { error: stillRunning(live.map((one) => one.job.target.branch)) });
 	const report = wholeCard ? await deleteLineage(id) : await deleteReview(id);
-	if (wasBlocked(report)) return send(res, 409, { error: report.live.length > 0 ? stillRunning(report.live.map((one) => one.job.target.branch)) : "This review is being worked on right now, so nothing was deleted. Try again in a moment." });
+	if (wasBlocked(report)) return send(res, 409, { error: blockedSentence(report) });
+	send(res, 200, {
+		ok: true,
+		deleted: report.deleted.length,
+		refusals: report.refusals
+	});
+}
+/** Why a deletion that was allowed to start still removed nothing. */
+function blockedSentence(report) {
+	return report.live.length > 0 ? stillRunning(report.live.map((one) => one.job.target.branch)) : "This review is being worked on right now, so nothing was deleted. Try again in a moment.";
+}
+/**
+* Delete the review a workspace page is showing, from that page.
+*
+* The review's own token rather than the index credential. A page opened from a terminal link has
+* never been shown the list and so holds no index cookie, and this token already writes the review's
+* state. Deleting it is the same authority over the same one record, and it reaches no other review.
+*
+* Only this review, never its card. A page showing one analysis does not get to decide the fate of
+* the others in its lineage; the list is where a whole branch is deleted.
+*
+* POST only, because it is the one route under `/api/` that destroys something.
+*/
+async function handleDeleteOwn(req, res, workspace) {
+	if (req.method !== "POST") return send(res, 405, { error: "Deleting a review needs a POST." });
+	const live = liveJobsFor(workspace.id);
+	if (live.length > 0) return send(res, 409, { error: stillRunning(live.map((one) => one.job.target.branch)) });
+	const report = await deleteReview(workspace.id);
+	if (wasBlocked(report)) return send(res, 409, { error: blockedSentence(report) });
 	send(res, 200, {
 		ok: true,
 		deleted: report.deleted.length,
@@ -6263,7 +6461,7 @@ async function deleteCards(ids) {
 		if (wasBlocked(report)) {
 			blocked.push({
 				id,
-				error: report.live.length > 0 ? stillRunning(report.live.map((one) => one.job.target.branch)) : "This review is being worked on right now, so nothing was deleted. Try again in a moment."
+				error: blockedSentence(report)
 			});
 			continue;
 		}
@@ -6316,6 +6514,15 @@ function indexCookie(req) {
 /** Branches, not reviews: a branch reviewed five times is one card and must not cost five slots. */
 var MAX_LISTED_BRANCHES = 50;
 /**
+* How many branches the reviewer's own working set may add on top of the recency window.
+*
+* Its own constant rather than a second use of the one above, because the two bound different
+* things and sharing a name would imply a ceiling the page does not have: what it lists is at most
+* these two added together. Saved and Queue exist to defeat recency, so they cannot be made to
+* compete for its slots -- saving eleven reviews must not push eleven recent ones off the page.
+*/
+var MAX_KEPT_BRANCHES = 50;
+/**
 * A finished review's workspace was served.
 *
 * **This counts page serves, not people and not sessions.** A reload, a restored tab, the CLI
@@ -6904,6 +7111,7 @@ async function handleState(req, res, workspace) {
 	}
 	if (typeof body?.complete === "boolean") {
 		workspace.completedAt = body.complete ? Date.now() : void 0;
+		dequeueOnComplete(workspace.id, body.complete);
 		changed = true;
 	}
 	if (changed) saveWorkspace(workspace);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prreviewbuddy",
-  "version": "0.27.2",
+  "version": "0.30.0",
   "description": "Review a branch or pull request in an isolated checkout, from your terminal, with the coding agent you already have installed.",
   "license": "UNLICENSED",
   "homepage": "https://prreviewbuddy.com",