prreviewbuddy 0.80.3 → 0.82.1

package/dist/server.js

@@ -1,4 +1,4 @@
-import { $ as resolveWorkspacePort, $i as AgentUnavailableError, $n as attemptIdOf, $r as isIssueOutstanding, $t as moveIds, A as parsePullRequestLink, Ai as resolveTarget, Ar as recentWorkspaces, At as readArchitecture, Bi as resolveAgent, Bn as requestJobStop, Br as isLifecycle, Bt as writeJsonAtomically, C as completeRun, Ci as followUpFindingId, Cn as hasAuthorContent, Cr as lineagePosition, Ct as RUNTIME_WORDS, D as relativeTime$1, Di as describeAuthorship, Dn as CODE_COPY_CLASS, Dt as checkArchitecture, E as stopRun, Ei as objectWithAnyOf, En as CODE_BLOCK_CLASS, Er as mutateWorkspace, Et as ARCHITECTURE_DIR, Fi as AmbiguousRefError, Fn as createWorktree, Fr as workspaceRevision, Ft as ownersOf, G as stopChild, Gn as checkReview, Gr as parseLifecycle, Gt as createArchitectureNotification, Hi as clearAgentPreference, Hr as lifecycleItemLabel, Ht as branchOf, I as settleQueue, Ii as shortNameOf, In as holdWorktree, Ir as workspaceStamps, It as QUOTE_LIMITS, J as writeServerRecord, Ji as agentById, Jn as fetchHead, Jr as queueAddMany, Jt as eventName, Ki as STORE_ROOT, Kn as followedBranch, Kr as sectionOf, Kt as createPrState, L as readIndexToken, Li as shorthandOf, Ln as isWorktreeHeld, Lr as LIFECYCLES, Lt as REPOSITORY_SCOPE, M as runBatch, Mi as DetachedHeadError, Mn as isBusy, Mt as storeArchitecture, N as activateQueued, Ni as isCommitReview, Nn as withReviewLock, Nt as storedScopes, O as batchIdentity, Oi as WITHHELD_REASON, On as escapeHtml$1, Or as presentedKissRunOf, Ot as loadArchitecture, P as compactQueue, Pi as resolveBase, Pn as MANAGED_ROOT, Pr as touchWorkspace, Pt as trackedFiles, Q as parseWorkspacePort, Qi as AgentCancelledError, Qn as allJobs, Qr as doneVerb, Qt as markReadForReview, Ri as git, Rn as readMarker, Rr as activityAttributes, Rt as ancestorsOf, S as beginRun, Si as wasReassessed, Sn as authorPreview, Sr as latestKissRun, St as recordRuntimeBestEffort, T as reportRun, Ti as extractJsonText, Tn as renderMarkdown, Tt as formatDuration, U as serverEntryExists, Ui as readAgentPreference, Un as findExistingReview, Ur as lifecyclePill, Ut as subjectOf, V as readServerRecord, Vi as resolveModel, Vr as lifecycleChip, Vt as announceJob, W as spawnServer, Wi as writeAgentPreference, Wn as checkCodeFreshness, Wr as offersLifecycle, Wt as categoryOf, X as MAX_WORKSPACE_PORT, Xi as detectAgents, Xn as fillFileUrlTemplate, Xr as queueStamp, Xt as markAllRead, Y as DEFAULT_WORKSPACE_PORT, Yi as agentFor$1, Yn as forgeResolver, Yr as queueRemove, Yt as listNotifications, Z as MIN_WORKSPACE_PORT, Zi as isAgentId, Zn as RUN_RECORD_RETENTION_MS, Zr as readQueue, Zt as markRead, _i as gradeScale, _n as isUnchanged, _r as groupByLineage, _t as BUILD_VERSION, a as wasBlocked, ai as processDiscussion, an as toneOf, ar as saveJob, at as settleReview, b as track, bi as reassessIn, bn as chatConversationSection, br as kissDecisionRefusal, bt as waitingHistory, c as RefreshUnavailableError, ci as explanationOf, cn as withUsageRecorded, cr as isTerminalJobPhase, d as mergeUpdateInto, di as ANALYSIS_ONLY, dn as adoptLegacyBaseline, dr as purposeOf, dt as record, ei as isQuestionOutstanding, en as pruneNotifications, er as endedAt, et as statedWorkspacePort, f as updateReview, fi as analysisRefOf, fn as prepareConversationWithin, gi as findingIn, gn as refreshPrContext, gr as followedRefName, h as liveUpdateFor, hi as findingAssessment, hn as unforeseenFailure, hr as decideKissRecommendation, ht as telemetryUploadConsent, i as liveJobsFor, ii as reviewerDispositions, in as retireNotification, ir as pruneJobResidue, it as prepareReview, j as unsupportedReason, ji as originRemote, jn as ReviewBeingDeletedError, jr as reviewedCommit, jt as repositoryKey, k as extractPullRequestLinks, ki as kissOutcome, kn as renderInlineText, kr as previousKissResult, kt as partFiles, l as cancelUpdate, li as isExplanationAction, ln as existingPaths, lr as phasesFor, m as UpdateAlreadyRunningError, mi as currentGradeIn, mn as storedConversation, mt as recordTelemetryUploadConsent, n as deleteReview, ni as questionsOutstanding, nn as readStateOf, nr as isTerminal, nt as failureOf$1, o as discardJob, oi as recommendationsOf, on as unreadAround, ot as reanalyseReview, p as askCheckout, pi as assessmentLookup, pn as saveRefresh, pr as isClaimed, qi as AGENT_IDS, qn as reviewedShaOf, qr as queueAdd, qt as createPrUpdate, r as lineageIds, ri as resolvedByConversation, rn as removeNotification, rr as loadJob, rt as observeReview, s as FETCH_TIMEOUT_MS, si as explanationActionOf, sn as unreadCount$1, sr as isLiveJob, st as runJob, t as deleteLineage, ti as issuesOutstanding, tn as readNotification, tr as fail$1, tt as writePortPreference, u as cancellableHere, ui as withExplanation, un as ASK_REREAD_CEILING_MS, ur as progressSteps, ut as readEvents, v as startKissJob, vi as isGradeFor, vn as agentEnvOf, vr as indexKeptIds, w as endRun, wi as asString, wn as GITHUB_MARKDOWN, wr as loadWorkspace, wt as describeRuntimes, x as RUN_KINDS, xi as sourcePhrase, xn as UNESTABLISHED_MOTIVATION, xr as kissDecisionsOf, xt as mapRuntimes, y as isDriving, yi as promptGrade, yn as EXPLAIN_SIMPLY_PROMPT, yr as indexReviewGroups, yt as feedbackUrl, z as clearServerRecordOwnedBy, zi as refExists, zn as removeWorktree, zr as activityLabel, zt as isArchitectureId } from "./delete_review-CYiIfV5A.js";
+import { $ as MIN_WORKSPACE_PORT, $i as resolveModel, $n as RUN_RECORD_RETENTION_MS, $r as deleteCollection, $t as markRead, A as batchIdentity, Ai as isGradeFor, An as escapeHtml$1, At as loadArchitecture, Bi as WITHHELD_REASON, Bn as readMarker, Br as LIFECYCLES, Bt as ancestorsOf, C as RUN_KINDS, Ci as ANALYSIS_ONLY, Cn as UNESTABLISHED_MOTIVATION, Cr as kissDecisionsOf, Ct as mapRuntimes, D as reportRun, Di as findingAssessment, Dn as renderMarkdown, Dr as loadWorkspace, Dt as formatDuration, E as endRun, Ei as currentGradeIn, En as GITHUB_MARKDOWN, Er as lineagePosition, Et as describeRuntimes, F as activateQueued, Fi as followUpFindingId, Fn as withReviewLock, Ft as storedScopes, G as serverEntryExists, Gi as isCommitReview, Gn as findExistingReview, Gr as lifecycleItemLabel, Gt as subjectOf, Hi as resolveTarget, Hn as requestJobStop, Hr as activityLabel, Ht as writeJsonAtomically, I as compactQueue, Ii as asString, In as MANAGED_ROOT, Ir as summarise, It as trackedFiles, Ji as shortNameOf, Jn as followedBranch, Jr as parseLifecycle, Jt as createPrState, K as spawnServer, Ki as resolveBase, Kn as checkCodeFreshness, Kr as lifecyclePill, Kt as categoryOf, Li as extractJsonText, Ln as createWorktree, Lr as touchWorkspace, Lt as ownersOf, M as parsePullRequestLink, Mi as reassessIn, Mr as previousKissResult, Mt as readArchitecture, N as unsupportedReason, Ni as sourcePhrase, Nn as ReviewBeingDeletedError, Nr as recentWorkspaces, Nt as repositoryKey, O as stopRun, Oi as findingIn, On as CODE_BLOCK_CLASS, Ot as ARCHITECTURE_DIR, P as runBatch, Pi as wasReassessed, Pn as isBusy, Pr as reviewedCommit, Pt as storeArchitecture, Q as MAX_WORKSPACE_PORT, Qi as resolveAgent, Qn as fillFileUrlTemplate, Qr as createWithMembers, Qt as markAllRead, R as settleQueue, Ri as objectWithAnyOf, Rn as holdWorktree, Rr as workspaceRevision, Rt as QUOTE_LIMITS, S as track, Si as withExplanation, Sn as chatConversationSection, Sr as kissDecisionRefusal, St as waitingHistory, T as completeRun, Ti as assessmentLookup, Tn as hasAuthorContent, Tr as lineageKey, Tt as RUNTIME_WORDS, U as readServerRecord, Ui as originRemote, Ur as isLifecycle, Ut as announceJob, V as clearServerRecordOwnedBy, Vi as kissOutcome, Vn as removeWorktree, Vr as activityAttributes, Vt as isArchitectureId, Wi as DetachedHeadError, Wr as lifecycleChip, Wt as branchOf, X as writeServerRecord, Xi as git, Xn as fetchHead, Xr as collectionsStamp, Xt as eventName, Yi as shorthandOf, Yn as reviewedShaOf, Yr as sectionOf, Yt as createPrUpdate, Z as DEFAULT_WORKSPACE_PORT, Zi as refExists, Zn as forgeResolver, Zr as createCollection, Zt as listNotifications, _i as processDiscussion, _n as unforeseenFailure, _r as decideKissRecommendation, _t as telemetryUploadConsent, a as wasBlocked, aa as STORE_ROOT, ai as queueAdd, an as removeNotification, ar as loadJob, at as observeReview, b as startKissJob, bi as explanationOf, bn as agentEnvOf, br as indexKeptIds, c as RefreshUnavailableError, ca as agentFor$1, ci as queueStamp, cn as unreadAround, ct as reanalyseReview, d as mergeUpdateInto, da as isAgentId, di as isIssueOutstanding, dn as existingPaths, dr as phasesFor, ea as agentIdOf, ei as nameProblemText, en as markReadForReview, er as allJobs, et as parseWorkspacePort, fa as parkedReason, fi as isQuestionOutstanding, fn as ASK_REREAD_CEILING_MS, fr as progressSteps, ft as readEvents, g as liveUpdateFor, gi as reviewerDispositions, gn as storedConversation, gt as recordTelemetryUploadConsent, h as UpdateAlreadyRunningError, ha as AgentUnavailableError, hi as resolvedByConversation, hn as saveRefresh, hr as isClaimed, i as liveJobsFor, ii as updateCollections, in as readStateOf, ir as isTerminal, it as failureOf$1, j as extractPullRequestLinks, ji as promptGrade, jn as renderInlineText, jr as presentedKissRunOf, jt as partFiles, k as relativeTime$1, ki as gradeScale, kn as CODE_COPY_CLASS, kr as mutateWorkspace, kt as checkArchitecture, l as cancelUpdate, la as agentIdentityOf, li as readQueue, ln as unreadCount$1, lr as isLiveJob, lt as runJob, m as askCheckout, ma as AgentCancelledError, mi as questionsOutstanding, mn as prepareConversationWithin, n as deleteReview, na as readAgentPreference, ni as renameCollection, nn as pruneNotifications, nr as endedAt, nt as statedWorkspacePort, o as discardJob, oa as AGENT_IDS, oi as queueAddMany, on as retireNotification, or as pruneJobResidue, ot as prepareReview, p as updateReview, pa as runnableAgents, pi as issuesOutstanding, pn as adoptLegacyBaseline, pr as purposeOf, pt as record, q as stopChild, qi as AmbiguousRefError, qn as checkReview, qr as offersLifecycle, qt as createArchitectureNotification, r as lineageIds, ra as writeAgentPreference, ri as setMembership, rn as readNotification, rr as fail$1, rt as writePortPreference, s as FETCH_TIMEOUT_MS, sa as agentById, si as queueRemove, sn as toneOf, sr as saveJob, st as settleReview, t as deleteLineage, ta as clearAgentPreference, ti as readCollections, tn as moveIds, tr as attemptIdOf, tt as resolveWorkspacePort, u as cancellableHere, ua as detectAgents, ui as doneVerb, un as withUsageRecorded, ur as isTerminalJobPhase, v as kissAgentId, vi as recommendationsOf, vn as refreshPrContext, vr as followedRefName, w as beginRun, wi as analysisRefOf, wn as authorPreview, wr as latestKissRun, wt as recordRuntimeBestEffort, x as isDriving, xi as isExplanationAction, xn as EXPLAIN_SIMPLY_PROMPT, xr as indexReviewGroups, xt as feedbackUrl, yi as explanationActionOf, yn as isUnchanged, yr as groupByLineage, yt as BUILD_VERSION, z as readIndexToken, zi as describeAuthorship, zn as isWorktreeHeld, zr as workspaceStamps, zt as REPOSITORY_SCOPE } from "./delete_review-BED4mPxB.js";
 import { basename, isAbsolute, join, relative, resolve } from "node:path";
 import { execFile } from "node:child_process";
 import { existsSync, mkdirSync, readFileSync, readdirSync, realpathSync, rmSync, statSync, writeFileSync } from "node:fs";
@@ -282,15 +282,37 @@ var INDEX_SECTION_NAMES = {
 };
 /**
 * Whether a section starts open. The reviewer's choice wins, and otherwise everything is open but
-* Done, which is the archive. The one place that default is written.
+* the archives: Done, and each collection's completed reviews. A new collection starts open. The
+* one place that default is written.
 */
 function sectionOpen(section, collapsed) {
-	return !(collapsed?.[section] ?? section === "done");
+	return !(collapsed?.[section] ?? (section === "done" || section.startsWith("collection-done:")));
 }
 /** A section the list draws a heading for, and so one whose fold can be stored. */
 function isListSection(value) {
 	return typeof value === "string" && LIST_SECTIONS.includes(value);
 }
+/** A fold the store accepts: a list section, or a collection's, by an id shaped like one. */
+function isFoldKey(value) {
+	if (isListSection(value)) return true;
+	if (typeof value !== "string") return false;
+	const match = /^collection(?:-done)?:(.*)$/.exec(value);
+	return match !== null && isCollectionId(match[1]);
+}
+function collectionFold(id) {
+	return `collection:${id}`;
+}
+function completedFold(id) {
+	return `collection-done:${id}`;
+}
+var COLLECTION_ID = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/;
+/**
+* Whether a value is shaped like a collection's id. Here, in the module both halves share, because
+* the hash and the fold store check it in the browser and on the server alike.
+*/
+function isCollectionId(value) {
+	return typeof value === "string" && COLLECTION_ID.test(value);
+}
 /**
 * Which section a card is drawn in: Pinned when pinned, and otherwise where its lifecycle says.
 *
@@ -302,6 +324,69 @@ function indexSectionOf(pinned, lifecycle, activity, filter) {
 	return pinned && !ordersExplicitly(filter) ? "pinned" : sectionOf(lifecycle, activity);
 }
 /**
+* Where a review's own card is drawn, or absent when it is drawn only inside its collections.
+*
+* Pinned first and exclusive, as it always was: a pinned review is drawn in Pinned and nowhere else,
+* collected or not, and unpinning it puts it back where the rest of these rules say. A review in a
+* collection is drawn there instead of in Active, Waiting or Done, so those three list what nobody
+* has grouped. Under the Queue filter neither applies: the list is the queue in its order, each card
+* in its lifecycle section, the same reason `indexSectionOf` gives for Pinned.
+*/
+function homeSection(pinned, lifecycle, activity, collected, filter) {
+	if (collected && !pinned && !ordersExplicitly(filter)) return void 0;
+	return indexSectionOf(pinned, lifecycle, activity, filter);
+}
+/**
+* Where a review is drawn inside each collection it belongs to: with the collection's reviews still
+* needing attention, under its completed reviews, or not at all.
+*
+* Completed means Done, by the same derivation that places the card everywhere else, so Waiting is
+* still in play and the Reviewed mark has nothing to do with it. One state for every collection the
+* review is in: marking it Done moves it in all of them at once. Not drawn while pinned (Pinned is
+* exclusive) or under the Queue filter (see `homeSection`).
+*/
+function collectionPlacement(pinned, lifecycle, activity, filter) {
+	if (pinned || ordersExplicitly(filter)) return void 0;
+	return sectionOf(lifecycle, activity) === "done" ? "completed" : "active";
+}
+/** A collection heading's count: what still needs attention, and the members drawn in Pinned. */
+function collectionCountLabel(active, pinned) {
+	const label = `${active} active`;
+	return pinned === 0 ? label : `${label} · ${pinned} pinned`;
+}
+/**
+* The quiet line on a card naming its collections, or absent when it would say nothing new.
+*
+* Inside a collection the heading already names that one, so the line says only where else the
+* review is ("Also in Release 0.82"), and nothing for a review in one collection. Outside every
+* collection heading, which is Pinned and the Queue filter, it names them all.
+*/
+function collectionsLine(memberOf, collections, current) {
+	const names = collections.filter((collection) => collection.id !== current && memberOf.includes(collection.id)).map((collection) => collection.name);
+	if (names.length === 0) return void 0;
+	return `${current === void 0 ? "In" : "Also in"} ${names.join(", ")}`;
+}
+/**
+* What a collection says when its main list is empty, or absent when it is not. A collection is
+* never hidden for having nothing active: it is still somewhere its finished reviews live, and
+* somewhere to put the next one.
+*/
+function collectionEmptyText(counts, filtered) {
+	if (counts.active > 0) return void 0;
+	if (filtered) return "Nothing in this collection matches the filters.";
+	if (counts.members === 0) return "Nothing here yet. Add reviews from a card’s menu, or select several and choose Manage collections.";
+	if (counts.pinned > 0 && counts.completed === 0) return "Its reviews are pinned, so they are listed under Pinned.";
+	return "Nothing here needs attention.";
+}
+/** Said under a collection whose oldest members are not drawn, so it never looks smaller than it is. */
+function omittedText(omitted) {
+	return omitted === 1 ? "1 older review in this collection is not listed." : `${omitted} older reviews in this collection are not listed.`;
+}
+/** The disclosure holding a collection's Done reviews. */
+function completedLabel(completed) {
+	return `Show completed (${completed})`;
+}
+/**
 * Whether a filter replaces the activity order with one the reviewer set. Queue is the only one:
 * the list is the queue, in its order.
 * `orderIndex` and `indexSectionOf` both ask this, so the ordering and the placement cannot disagree.
@@ -4228,7 +4313,7 @@ function renderFailureBody(job, options = {}) {
 	if (job.failure.stopped) return `<p>${options.lead === false ? "" : "<strong>Review stopped.</strong> "}Nothing went wrong, and what it had finished is kept.</p>
   <button type="button" class="prb-job-retry" data-job-retry>Carry on</button>
   <p class="prb-job-note">It picks up from the ${escapeHtml$1(noun)}, not from the start.</p>`;
-	const next = job.failure.kind === "agent_authentication_failed" ? `<p class="prb-job-note">Signing that agent in is what changes this. Once it is signed in, update the review.</p>` : `<button type="button" class="prb-job-retry" data-job-retry>Retry</button>
+	const next = job.failure.parked ? "" : job.failure.kind === "agent_authentication_failed" ? `<p class="prb-job-note">Signing that agent in is what changes this. Once it is signed in, update the review.</p>` : `<button type="button" class="prb-job-retry" data-job-retry>Retry</button>
   <p class="prb-job-note">Retry picks up from the ${escapeHtml$1(noun)}, not from the start.</p>`;
 	return `<p>${options.lead === false ? "" : "<strong>Review paused.</strong> "}${escapeHtml$1(job.failure.message)}</p>
   ${next}`;
@@ -7701,6 +7786,8 @@ function startArchitectureRun(workspace, scope, deps = {}) {
 	const now = deps.now ?? Date.now;
 	const startedAt = now();
 	const env = agentEnvOf(workspace);
+	const parked = parkedReason(env.agentId);
+	if (parked) return { refused: parked };
 	const agent = agentFor$1(env);
 	const requestedModel = resolveModel({ review: { session: workspace.session } }) ?? void 0;
 	const record = {
@@ -9531,27 +9618,63 @@ function operationLine(operation) {
 }
 //#endregion
 //#region ../../packages/review-harness/src/workspace/reviews_page.ts
-function renderReviewsPage(view) {
+function indexContext(view) {
 	const now = view.now ?? Date.now();
 	const activityOf = (group) => view.activity?.get(group.latest.id) ?? {
 		at: group.latest.createdAt,
 		kind: "created"
 	};
 	const ordered = [...view.groups].sort((a, b) => activityOf(b).at - activityOf(a).at);
+	const rank = new Map(ordered.map((group, index) => [group, index]));
+	const operations = view.operations ?? [];
+	const inProgressOnly = new Set(ordered.filter((group) => group.history.length === 1 && !group.latest.hasStandardReview && operations.some((operation) => operation.reviewId === group.latest.id && operation.purpose !== "update")).map((group) => group.latest.id));
+	const queue = view.queue ?? [];
+	const positions = new Map(queue.map((id, position) => [id, position]));
+	const collections = view.collections ?? [];
+	const refs = collections.map(({ id, name }) => ({
+		id,
+		name
+	}));
+	const memberOf = membershipsByLineage(collections);
+	const renderCards = (groups, placement, hidden = () => false) => groups.map((group) => renderCard(group, now, activityOf(group), rank.get(group), view.analyses?.get(group.latest.id), positions.get(group.latest.id), group.history.map((review) => view.moved?.get(review.id)).find((kind) => kind !== void 0), prStateOf(group.latest, view.prStates), inProgressOnly.has(group.latest.id), {
+		placement,
+		memberOf: memberOf.get(group.key) ?? [],
+		refs,
+		hidden: hidden(group)
+	})).join("");
+	return {
+		now,
+		ordered,
+		activityOf,
+		inProgressOnly,
+		positions,
+		queue,
+		refs,
+		collections,
+		memberOf,
+		renderCards
+	};
+}
+function renderReviewsPage(view) {
+	const context = indexContext(view);
+	const { ordered, activityOf, inProgressOnly, positions, queue, memberOf, renderCards } = context;
+	const now = context.now;
+	const operations = view.operations ?? [];
 	const sections = {
 		pinned: [],
 		active: [],
 		waiting: [],
 		done: []
 	};
-	for (const group of ordered) sections[indexSectionOf(group.latest.pinned === true, group.latest.lifecycle, activityOf(group), "all")].push(group);
-	const rank = new Map(ordered.map((group, index) => [group, index]));
-	const operations = view.operations ?? [];
-	const inProgressOnly = new Set(ordered.filter((group) => group.history.length === 1 && !group.latest.hasStandardReview && operations.some((operation) => operation.reviewId === group.latest.id && operation.purpose !== "update")).map((group) => group.latest.id));
-	const renderCards = (groups) => groups.map((group) => renderCard(group, now, activityOf(group), rank.get(group), view.analyses?.get(group.latest.id), positions.get(group.latest.id), group.history.map((review) => view.moved?.get(review.id)).find((kind) => kind !== void 0), prStateOf(group.latest, view.prStates), inProgressOnly.has(group.latest.id))).join("");
+	const collectedHome = /* @__PURE__ */ new Set();
+	for (const group of ordered) {
+		const collected = (memberOf.get(group.key) ?? []).length > 0;
+		const home = homeSection(group.latest.pinned === true, group.latest.lifecycle, activityOf(group), collected, "all");
+		if (home === void 0) collectedHome.add(group);
+		sections[home ?? sectionOf(group.latest.lifecycle, activityOf(group))].push(group);
+	}
 	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 renderSectionAt = (section) => renderSection(section, sections[section], (groups) => renderCards(groups, "home", (group) => collectedHome.has(group)), sectionOpen(section, view.sectionsCollapsed), sections[section].filter((group) => !inProgressOnly.has(group.latest.id) && !collectedHome.has(group)).length);
 	return `<!doctype html>
 <html lang="en"${themeAttribute(view.theme)}>
 <head>
@@ -9562,7 +9685,7 @@ function renderReviewsPage(view) {
 <link rel="stylesheet" href="/assets/sidebar.css">
 <link rel="stylesheet" href="/assets/shell.css">
 </head>
-<body class="prb-index-body" data-way-back="${escapeHtml$1(wayBackSentence())}" data-notifications-since="${now}"${view.revision === void 0 ? "" : ` data-index-revision="${escapeHtml$1(view.revision)}"`} data-live="${anythingRunning(view.analyses ?? /* @__PURE__ */ new Map(), operations)}">
+<body class="prb-index-body" data-way-back="${escapeHtml$1(wayBackSentence())}" data-notifications-since="${now}"${view.revision === void 0 ? "" : ` data-index-revision="${escapeHtml$1(view.revision)}"`}${view.snapshot === void 0 ? "" : ` data-cards-snapshot="${escapeHtml$1(view.snapshot)}"`} data-live="${anythingRunning(view.analyses ?? /* @__PURE__ */ new Map(), operations)}">
 ${renderGlobalBar({
 		current: "reviews",
 		unread: view.unreadNotifications ?? 0,
@@ -9572,7 +9695,7 @@ ${renderContextBar("Reviews", ordered.length === 0 ? "" : renderControls$1(queue
 		handle: latest.authorHandle,
 		name: latest.authorName,
 		createdAt: latest.createdAt
-	}))), ordered.some((group) => group.latest.prNumber !== void 0)))}
+	}))), ordered.some((group) => group.latest.prNumber !== void 0), context.refs.length > 0))}
 <div class="prb-shell">
 ${renderWorkspaceRail({
 		current: "reviews",
@@ -9583,11 +9706,14 @@ ${renderWorkspaceRail({
     <p class="prb-index-count" data-count>${escapeHtml$1(countLabel(storedReviews, ordered.length))}</p>
     <button type="button" class="prb-index-select" data-select-mode aria-pressed="false">Select</button>
   </div>`}
-  ${renderInProgress$1(operations, ordered, positions, view.prStates, now, sectionOpen("in-progress", view.sectionsCollapsed))}
-  ${INDEX_SECTIONS.map((section) => renderSection(section, sections[section], renderCards, sectionOpen(section, view.sectionsCollapsed), sections[section].filter((group) => !inProgressOnly.has(group.latest.id)).length)).join("\n  ")}
+  ${renderInProgress$1(operations, ordered, positions, view.prStates, now, sectionOpen("in-progress", view.sectionsCollapsed), memberOf)}
+  ${renderSectionAt("pinned")}
+  ${ordered.length === 0 ? "" : collectionsArea(context, view.sectionsCollapsed)}
+  ${INDEX_SECTIONS.filter((section) => section !== "pinned").map(renderSectionAt).join("\n  ")}
   ${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 === 0 ? "" : renderBulkBar()}
-  <p class="prb-index-note">Reviews are kept for thirty days, on this machine only. Pinned reviews, Done reviews and anything in your review queue are kept for ninety days instead, counting from the last time you opened or marked them. Every review has a fixed address, so a link to one is worth keeping and still opens after a restart. Stopping the server deletes nothing; it only stops anything being served until the next review starts it again.</p>
+  ${ordered.length === 0 ? "" : renderCollectionsDialog()}
+  <p class="prb-index-note">Reviews are kept for thirty days, on this machine only. Pinned, Done and queued reviews, and reviews in a collection, are kept for ninety days instead, counting from the last time you opened or marked them. Every review has a fixed address, so a link to one is worth keeping and still opens after a restart. Stopping the server deletes nothing; it only stops anything being served until the next review starts it again.</p>
 </main>
 </div>
 <script src="/assets/reviews.js" type="module"><\/script>
@@ -9603,7 +9729,7 @@ ${renderWorkspaceRail({
 * 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$1(queued, authors, anyRequest) {
+function renderControls$1(queued, authors, anyRequest, anyCollection) {
 	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)}
@@ -9616,10 +9742,29 @@ function renderControls$1(queued, authors, anyRequest) {
       </select>
     </label>
     ${anyRequest ? renderPrFilter() : ""}
+    ${renderCollectionsFilter(anyCollection)}
     ${renderClearFilters()}
   </div>`;
 }
 /**
+* Which collections to show, any of several, and Uncollected for the reviews in none.
+*
+* A menu holding checkboxes rather than a select, because a select chooses one thing and this
+* chooses several, and because the list can be long enough to want its own search. The options are
+* filled in by the browser from the page's list of collections, so one made or renamed on the page
+* appears here without a reload. Hidden while there are no collections: a filter whose only choice
+* is Uncollected is not a choice.
+*/
+function renderCollectionsFilter(anyCollection) {
+	return `<div class="prb-index-status prb-collections-filter" data-menu data-menu-form data-collections-filter${anyCollection ? "" : " hidden"}>
+      <button type="button" class="prb-collections-filter-trigger" data-menu-trigger aria-expanded="false" aria-haspopup="true" aria-label="Filter by collection">Collections: <span data-collections-filter-label>All</span></button>
+      <div class="prb-menu-panel prb-collections-filter-panel" data-menu-panel hidden>
+        <input type="search" class="prb-collections-filter-search" data-collections-filter-search placeholder="Find a collection" aria-label="Find a collection" spellcheck="false">
+        <div class="prb-collections-filter-options" data-collections-filter-options role="group" aria-label="Collections"></div>
+      </div>
+    </div>`;
+}
+/**
 * Puts the search and every filter back at once. Hidden until the browser finds one narrowing the
 * list: the hash is the browser's to read, and without script there is nothing to clear.
 */
@@ -9669,6 +9814,141 @@ function renderAuthorFilter(authors) {
     </label>`;
 }
 /**
+* The Collections area, between Pinned and Active: one disclosure per collection, each listing its
+* reviews that still need attention, with its Done reviews folded beneath.
+*
+* Exported for the routes that change collections, which answer with this markup so the browser can
+* swap the area in place: every card instance is drawn by `renderCard`, here and on a full load, and
+* never copied in the browser.
+*/
+/** Collection ids by lineage key, in collection order: what a card's `data-collections` says. */
+function membershipsByLineage(collections) {
+	const memberOf = /* @__PURE__ */ new Map();
+	for (const collection of collections) for (const key of collection.members) memberOf.set(key, [...memberOf.get(key) ?? [], collection.id]);
+	return memberOf;
+}
+function renderCollectionsArea(view) {
+	return collectionsArea(indexContext(view), view.sectionsCollapsed);
+}
+function collectionsArea(context, collapsed) {
+	const { ordered, refs, collections, memberOf, activityOf, inProgressOnly, renderCards } = context;
+	const registry = JSON.stringify(refs).replace(/</g, "\\u003c");
+	const blocks = collections.map((collection) => {
+		const members = ordered.filter((group) => (memberOf.get(group.key) ?? []).includes(collection.id));
+		const place = (group) => collectionPlacement(group.latest.pinned === true, group.latest.lifecycle, activityOf(group), "all");
+		const drawn = members.filter((group) => !inProgressOnly.has(group.latest.id));
+		const active = members.filter((group) => place(group) !== "completed");
+		const completed = members.filter((group) => place(group) === "completed");
+		return renderCollection(collection, {
+			active: drawn.filter((group) => place(group) === "active").length,
+			completed: drawn.filter((group) => place(group) === "completed").length,
+			pinned: drawn.filter((group) => group.latest.pinned === true).length,
+			members: members.length
+		}, collection.members.length - members.length, collapsed, {
+			active: renderCards(active, { collection: collection.id }, (group) => place(group) === void 0),
+			completed: renderCards(completed, { collection: collection.id })
+		});
+	});
+	return `<section class="prb-collections" data-collections-area aria-label="Collections">
+    <script type="application/json" data-collections-registry>${registry}<\/script>
+    <div class="prb-collections-head">
+      <h2 class="prb-collections-title">Collections</h2>
+      ${collections.length === 0 ? "<p class=\"prb-collections-hint\">Group reviews that belong together, such as one refactor or one release.</p>" : ""}
+    </div>
+    ${blocks.join("\n    ")}
+    ${renderNewCollection()}
+  </section>`;
+}
+function renderCollection(collection, counts, omitted, collapsed, cards) {
+	const id = escapeHtml$1(collection.id);
+	const empty = collectionEmptyText(counts, false);
+	return `<div class="prb-collection" data-collection="${id}">
+      <details class="prb-index-section" data-section="${escapeHtml$1(collectionFold(collection.id))}"${sectionOpen(collectionFold(collection.id), collapsed) ? " open" : ""}>
+        <summary class="prb-index-section-head"><span data-collection-name>${escapeHtml$1(collection.name)}</span> · <span data-section-count>${escapeHtml$1(collectionCountLabel(counts.active, counts.pinned))}</span></summary>
+        <p class="prb-collection-empty" data-collection-empty${empty === void 0 ? " hidden" : ""}>${escapeHtml$1(empty ?? "")}</p>
+        <div class="prb-cards" data-cards="${escapeHtml$1(collectionFold(collection.id))}">
+          ${cards.active}
+        </div>
+        ${renderLoadMore$1(collectionFold(collection.id))}
+        ${omitted > 0 ? `<p class="prb-collection-empty" data-collection-omitted>${escapeHtml$1(omittedText(omitted))}</p>` : ""}
+        <details class="prb-collection-completed" data-section="${escapeHtml$1(completedFold(collection.id))}"${sectionOpen(completedFold(collection.id), collapsed) ? " open" : ""}${counts.completed === 0 ? " hidden" : ""}>
+          <summary class="prb-collection-completed-head" data-completed-label>${escapeHtml$1(completedLabel(counts.completed))}</summary>
+          <div class="prb-cards" data-cards="${escapeHtml$1(completedFold(collection.id))}">
+            ${cards.completed}
+          </div>
+          ${renderLoadMore$1(completedFold(collection.id))}
+        </details>
+      </details>
+      ${renderCollectionMenu(collection)}
+    </div>`;
+}
+/**
+* A collection's own menu: rename it in place, or delete it.
+*
+* Deleting says what it does not do, which is the thing a reader worries about: the reviews stay.
+* It says what does change, too, because collected reviews are kept for longer and these go back to
+* the ordinary thirty days unless something else keeps them.
+*/
+function renderCollectionMenu(collection) {
+	const id = escapeHtml$1(collection.id);
+	return `<div class="prb-menu prb-collection-menu" data-menu data-menu-form>
+        <button type="button" class="prb-menu-trigger" data-menu-trigger aria-expanded="false" aria-haspopup="true" aria-label="${escapeHtml$1(`Actions for collection ${collection.name}`)}">&#8943;</button>
+        <div class="prb-menu-panel" data-menu-panel hidden>
+          <div data-menu-items>
+            <button type="button" class="prb-menu-item" data-collection-rename="${id}">Rename collection</button>
+            <button type="button" class="prb-menu-item prb-menu-item-danger" data-collection-delete="${id}">Delete collection</button>
+          </div>
+          <form class="prb-collection-rename" data-collection-rename-form="${id}" hidden>
+            <label class="prb-menu-question" for="prb-rename-${id}">New name</label>
+            <input type="text" id="prb-rename-${id}" class="prb-collection-name" data-collection-name-input value="${escapeHtml$1(collection.name)}" maxlength="60" autocomplete="off" spellcheck="false">
+            <p class="prb-collection-error" data-collection-error hidden role="alert"></p>
+            <div class="prb-menu-buttons">
+              <button type="submit" class="prb-menu-item">Save</button>
+              <button type="button" class="prb-menu-item" data-collection-cancel>Cancel</button>
+            </div>
+          </form>
+          <div class="prb-menu-confirm" data-menu-confirm hidden>
+            <p class="prb-menu-question">${escapeHtml$1(`Delete the collection ${collection.name}?`)}</p>
+            <p class="prb-menu-note">Its reviews stay where they are. They go back to being kept for thirty days unless they are pinned, Done or in your review queue.</p>
+            <div class="prb-menu-buttons">
+              <button type="button" class="prb-menu-item prb-menu-item-danger" data-collection-confirm-delete="${id}">Delete</button>
+              <button type="button" class="prb-menu-item" data-cancel-delete>Cancel</button>
+            </div>
+          </div>
+        </div>
+      </div>`;
+}
+/** New collection, which opens a name field in place. */
+function renderNewCollection() {
+	return `<div class="prb-collection-new">
+      <button type="button" class="prb-index-more prb-collection-new-button" data-collection-new><span aria-hidden="true">+</span> New collection</button>
+      <form class="prb-collection-new-form" data-collection-new-form hidden>
+        <input type="text" class="prb-collection-name" data-collection-name-input placeholder="Collection name" aria-label="New collection name" maxlength="60" autocomplete="off" spellcheck="false">
+        <button type="submit" class="prb-bulk-button">Create</button>
+        <button type="button" class="prb-bulk-button" data-collection-cancel>Cancel</button>
+        <p class="prb-collection-error" data-collection-error hidden role="alert"></p>
+      </form>
+    </div>`;
+}
+/**
+* Manage collections, for one card or for a selection. Built once; the browser fills the list from
+* the page's collections and the reviews it was opened for, because the boxes' states depend on
+* which reviews those are.
+*/
+function renderCollectionsDialog() {
+	return `<dialog class="prb-dialog prb-collections-dialog" data-collections-dialog aria-labelledby="prb-collections-dialog-title">
+  <h2 id="prb-collections-dialog-title">Collections</h2>
+  <p data-collections-dialog-for></p>
+  <input type="search" class="prb-collections-dialog-search" data-collections-dialog-search placeholder="Find or create a collection" aria-label="Find or create a collection" spellcheck="false" autocomplete="off">
+  <div class="prb-collections-dialog-list" data-collections-dialog-list role="group" aria-label="Collections"></div>
+  <button type="button" class="prb-collections-dialog-create" data-collections-dialog-create hidden></button>
+  <p class="prb-collection-error" data-collections-dialog-error hidden role="alert"></p>
+  <div class="prb-dialog-actions">
+    <button type="button" data-collections-dialog-close>Done</button>
+  </div>
+</dialog>`;
+}
+/**
 * One section's Load more, which the browser shows once that section holds more than it is showing.
 *
 * One per section rather than one pager for the page, so revealing older Done reviews never costs
@@ -9706,10 +9986,11 @@ function renderSection(section, groups, renderCards, open, shown = groups.length
 * index token and arrives as a reload. Its count is of work, not reviews, so a review being updated
 * counts once here and once in its own section, which is the point.
 */
-function renderInProgress$1(operations, groups, positions, prStates, now, open) {
+function renderInProgress$1(operations, groups, positions, prStates, now, open, memberOf = /* @__PURE__ */ new Map()) {
 	if (operations.length === 0) return "";
 	const reviews = new Map(groups.flatMap((group) => group.history.map((review) => [review.id, review])));
-	const entries = operations.map((operation) => renderOperation(operation, reviewOf(operation, reviews), positions, prStates, now)).join("");
+	const collections = operationCollections(operations, groups, memberOf);
+	const entries = operations.map((operation) => renderOperation(operation, reviewOf(operation, reviews), positions, prStates, now, collections[operation.key] ?? [])).join("");
 	return `<details class="prb-index-section" data-section="in-progress"${open ? " open" : ""}>
     <summary class="prb-index-section-head">${INDEX_SECTION_NAMES["in-progress"].title} · <span data-section-count>${operations.length}</span></summary>
     <div class="prb-cards" data-operations>
@@ -9723,6 +10004,23 @@ function renderInProgress$1(operations, groups, positions, prStates, now, open)
 * one, which is what that review's title, author and state still are: the filters then treat a
 * re-review of a known pull request as that pull request, rather than as somebody unknown.
 */
+/**
+* The collections each piece of work's review is in, by the work's key: the lineage of whichever
+* review `reviewOf` finds, so a waiting row with no review yet still answers the Collections filter
+* the way its card will. Also what the collections routes send back, so a change of membership
+* reaches the In progress section without a reload.
+*/
+function operationCollections(operations, groups, memberOf) {
+	const reviews = new Map(groups.flatMap((group) => group.history.map((review) => [review.id, review])));
+	const lineage = new Map(groups.flatMap((group) => group.history.map((review) => [review.id, group.key])));
+	const answer = {};
+	for (const operation of operations) {
+		const review = reviewOf(operation, reviews);
+		const key = review ? lineage.get(review.id) : void 0;
+		answer[operation.key] = key === void 0 ? [] : [...memberOf.get(key) ?? []];
+	}
+	return answer;
+}
 function reviewOf(operation, reviews) {
 	if (operation.reviewId !== void 0) return reviews.get(operation.reviewId);
 	const row = operation.row;
@@ -9744,7 +10042,7 @@ function reviewOf(operation, reviews) {
 * A link to the review, whose own page shows the steps, the cancel and the Retry. A row still waiting
 * for a slot has no review to open, so it is not a link.
 */
-function renderOperation(operation, review, positions, prStates, now) {
+function renderOperation(operation, review, positions, prStates, now, collections = []) {
 	const row = operation.row;
 	const own = operation.reviewId !== void 0 && review?.id === operation.reviewId ? review : void 0;
 	const repoPath = review?.repoPath ?? row?.repoPath;
@@ -9775,7 +10073,7 @@ function renderOperation(operation, review, positions, prStates, now) {
         ${operation.startedAt ? `<span class="prb-card-when"><time datetime="${new Date(operation.startedAt).toISOString()}">${escapeHtml$1(`Started ${relativeTime(operation.startedAt, now)}`)}</time></span>` : ""}
       </p>`;
 	const href = own ? `/r/${encodeURIComponent(own.id)}?t=${encodeURIComponent(own.token)}` : void 0;
-	return `<article class="prb-card" data-operation="${escapeHtml$1(operation.kind)}" data-key="${escapeHtml$1(operation.key)}" data-id="${escapeHtml$1(operation.reviewId ?? "")}" data-status="${status}"${queued === void 0 ? "" : ` data-queue="${queued}"`}${prState === void 0 ? "" : ` data-pr-state="${prState}"`}${review?.authorHandle ? ` data-author="${escapeHtml$1(review.authorHandle)}"` : ""} data-search-text="${escapeHtml$1(search)}">
+	return `<article class="prb-card" data-operation="${escapeHtml$1(operation.kind)}" data-key="${escapeHtml$1(operation.key)}" data-id="${escapeHtml$1(operation.reviewId ?? "")}" data-status="${status}"${queued === void 0 ? "" : ` data-queue="${queued}"`}${prState === void 0 ? "" : ` data-pr-state="${prState}"`}${review?.authorHandle ? ` data-author="${escapeHtml$1(review.authorHandle)}"` : ""} data-search-text="${escapeHtml$1(search)}" data-collections="${escapeHtml$1(collections.join(" "))}">
     ${href ? `<a class="prb-card-link" href="${escapeHtml$1(href)}">${body}</a>` : `<div class="prb-card-link">${body}</div>`}
   </article>`;
 }
@@ -9800,6 +10098,7 @@ 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-collections disabled>Manage collections</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      ")}
@@ -9836,13 +10135,21 @@ 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, activity, rank, analysis, queued, moved, prState, inProgressOnly = false) {
+function renderCard(group, now, activity, rank, analysis, queued, moved, prState, inProgressOnly = false, instance = {
+	placement: "home",
+	memberOf: [],
+	refs: [],
+	hidden: false
+}) {
 	const review = group.latest;
+	const placement = instance.placement === "home" ? "home" : `collection:${instance.placement.collection}`;
+	const line = collectionsLine(instance.memberOf, instance.refs, instance.placement === "home" ? void 0 : instance.placement.collection);
+	const hidden = inProgressOnly || instance.hidden;
 	const status = statusOf(review);
 	const chip = cardLifecycleChip(review.pinned === true, review.lifecycle, activity);
 	const href = `/r/${encodeURIComponent(review.id)}?t=${encodeURIComponent(review.token)}`;
 	const followed = review.targetType === "commit" ? `commit ${(review.commitSha ?? review.branch).slice(0, 7)}` : followedRefName(review.branch, review.headResolvedRef) ?? review.branch;
-	return `<article class="prb-card" data-card data-id="${escapeHtml$1(review.id)}" data-status="${status}" data-pinned="${review.pinned}"${queued === void 0 ? "" : ` data-queue="${queued}"`}${prState === void 0 ? "" : ` data-pr-state="${prState}"`}${review.authorHandle ? ` data-author="${escapeHtml$1(review.authorHandle)}"` : ""} data-lifecycle="${review.lifecycle?.state ?? "active"}"${review.lifecycle ? ` data-lifecycle-at="${review.lifecycle.at}"` : ""}${activityAttributes(activity)} data-rank="${rank}" data-reviews="${group.history.length}" data-lineage="${escapeHtml$1(group.history.map((entry) => entry.id).join(" "))}" data-search-text="${escapeHtml$1(groupSearchText(group))}"${inProgressOnly ? " data-in-progress-only hidden" : ""}>
+	return `<article class="prb-card" data-card data-id="${escapeHtml$1(review.id)}" data-status="${status}" data-pinned="${review.pinned}"${queued === void 0 ? "" : ` data-queue="${queued}"`}${prState === void 0 ? "" : ` data-pr-state="${prState}"`}${review.authorHandle ? ` data-author="${escapeHtml$1(review.authorHandle)}"` : ""} data-lifecycle="${review.lifecycle?.state ?? "active"}"${review.lifecycle ? ` data-lifecycle-at="${review.lifecycle.at}"` : ""}${activityAttributes(activity)} data-rank="${rank}" data-reviews="${group.history.length}" data-lineage="${escapeHtml$1(group.history.map((entry) => entry.id).join(" "))}" data-search-text="${escapeHtml$1(groupSearchText(group))}" data-placement="${escapeHtml$1(placement)}" data-collections="${escapeHtml$1(instance.memberOf.join(" "))}"${inProgressOnly ? " data-in-progress-only" : ""}${hidden ? " hidden" : ""}>
     <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">${review.pinned ? "&#9733;" : "&#9734;"}</button>
     ${renderCardMenu(group, followed, queued !== void 0, status, activity)}
@@ -9868,6 +10175,7 @@ function renderCard(group, now, activity, rank, analysis, queued, moved, prState
         <span class="prb-card-diffstat"><span class="prb-add">+${review.additions}</span> <span class="prb-del">&minus;${review.deletions}</span></span>
         ${analysis !== void 0 && !review.hasStandardReview ? "" : `<span class="prb-card-left">${escapeHtml$1(whatIsLeft(review))}</span>`}
         ${renderAnalysisState(analysis)}
+        <span class="prb-card-collections" data-collections-line${line === void 0 ? " hidden" : ""}>${escapeHtml$1(line ?? "")}</span>
         <span class="prb-card-when"><time datetime="${new Date(activity.at).toISOString()}" title="${escapeHtml$1(new Date(activity.at).toLocaleString())}">${escapeHtml$1(`${activityLabel(activity.kind)} ${relativeTime(activity.at, now)}`)}</time></span>
       </p>
     </a>
@@ -9908,7 +10216,8 @@ function renderCardMenu(group, followed, queued, status, activity) {
 		queue: queued ? "remove" : "add",
 		mark,
 		lifecycle,
-		fetch: true
+		fetch: true,
+		collections: true
 	});
 	const all = `all ${count} reviews following ${followed}`;
 	return renderMenu({
@@ -9921,6 +10230,7 @@ function renderCardMenu(group, followed, queued, status, activity) {
 		mark,
 		lifecycle,
 		fetch: true,
+		collections: true,
 		copy: "Copy latest review ID"
 	});
 }
@@ -9951,6 +10261,7 @@ function renderMenu(item) {
         ${markItem}
         ${lifecycleItems}
         ${queueItem}
+        ${item.collections ? `<button type="button" class="prb-menu-item" data-collections-for="${id}">Manage collections…</button>` : ""}
         <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>
@@ -10361,14 +10672,15 @@ function writeChangeMapRegister(register) {
 		}, null, 2)}\n`);
 	} catch {}
 }
-/** Known sections with a boolean only. A hand-edited key we do not draw is dropped on the next write. */
+/**
+* Known sections and collection folds with a boolean only. A hand-edited key we do not draw is
+* dropped on the next write. A fold for a collection since deleted elsewhere is kept until
+* `forgetSections` is told, which costs one line of JSON and nothing on the page.
+*/
 function readReviewSections(value) {
 	if (!value || typeof value !== "object") return {};
 	const sections = {};
-	for (const section of LIST_SECTIONS) {
-		const collapsed = value[section];
-		if (typeof collapsed === "boolean") sections[section] = collapsed;
-	}
+	for (const [section, collapsed] of Object.entries(value)) if (isFoldKey(section) && typeof collapsed === "boolean") sections[section] = collapsed;
 	return Object.keys(sections).length === 0 ? {} : { reviewSections: sections };
 }
 /**
@@ -10390,6 +10702,19 @@ function writeSectionCollapsed(section, collapsed) {
 		}, null, 2)}\n`);
 	} catch {}
 }
+/** Drop folds nothing will draw again: a deleted collection's. Writes nothing when none was stored. */
+function forgetSections(sections) {
+	try {
+		const current = readPreferences();
+		const stored = current.reviewSections;
+		if (!stored || !sections.some((section) => section in stored)) return;
+		const reviewSections = Object.fromEntries(Object.entries(stored).filter(([section]) => !sections.includes(section)));
+		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
+			...current,
+			reviewSections
+		}, null, 2)}\n`);
+	} catch {}
+}
 /** As `readReviewSections`, for the Runs page. */
 function readRunsSections(value) {
 	if (!value || typeof value !== "object") return {};
@@ -10489,6 +10814,7 @@ function readSettings() {
 		},
 		agent: readAgentPreference(),
 		agents: AGENT_IDS,
+		agentParked: parkedReason(readAgentPreference() ?? void 0) ?? null,
 		usageCounts: telemetryUploadConsent(),
 		theme: readPreferences().theme ?? "system"
 	};
@@ -10513,6 +10839,11 @@ function saveSettings(change, minPort, maxPort) {
 			error: `\`${change.port}\` is not a port PR Review Buddy can use. Choose one between ${minPort} and ${maxPort}.`
 		};
 	}
+	const parked = change.agent ? parkedReason(change.agent) : void 0;
+	if (parked) return {
+		settings: readSettings(),
+		error: parked
+	};
 	if (change.agent !== void 0 && change.agent !== "" && !AGENT_IDS.includes(change.agent)) return {
 		settings: readSettings(),
 		error: `There is no agent called \`${change.agent}\`.`
@@ -10668,11 +10999,15 @@ function renderReviewsSection(settings) {
     <h2>Reviews</h2>
     <label class="prb-field prb-settings-field">
       <span>Default coding agent</span>
-      <select data-settings-agent aria-describedby="prb-agent-help">${[`<option value=""${settings.agent ? "" : " selected"}>Ask each time</option>`, ...settings.agents.map((id) => `<option value="${escapeHtml$1(id)}"${settings.agent === id ? " selected" : ""}>${escapeHtml$1(id)}</option>`)].join("")}</select>
+      <select data-settings-agent aria-describedby="prb-agent-help">${[
+		`<option value=""${settings.agent ? "" : " selected"}>Ask each time</option>`,
+		...settings.agents.map((id) => `<option value="${escapeHtml$1(id)}"${settings.agent === id ? " selected" : ""}>${escapeHtml$1(id)}</option>`),
+		...settings.agentParked && settings.agent ? [`<option value="${escapeHtml$1(settings.agent)}" selected disabled>${escapeHtml$1(settings.agent)} (turned off)</option>`] : []
+	].join("")}</select>
     </label>
     <p class="prb-settings-help" id="prb-agent-help">Used when a review does not name one. PR Review
     Buddy uses an agent already installed on this machine and its existing sign-in, so this changes
-    which one runs and never asks you for a credential.</p>
+    which one runs and never asks you for a credential.</p>${settings.agentParked ? `\n    <p class="prb-settings-help" role="alert">${escapeHtml$1(settings.agentParked)} Choose another default, or Ask each time.</p>` : ""}
   </section>`;
 }
 /**
@@ -11152,6 +11487,25 @@ feature:                   E</pre>
     <p>Deleting a review removes its record and the isolated checkouts it was made in. It never
     touches your branch or its commits. Delete one from its menu, or tick reviews in the Reviews
     list, one at a time or a whole branch at once, and delete them together.</p>`
+	},
+	{
+		id: "collections",
+		title: "Collections",
+		body: `<p>A collection groups reviews that belong together, such as the pull requests of one
+    refactor or one release. It says what belongs together; whether each review still needs your
+    attention is still Active, Waiting or Done, so adding or removing a review never changes its
+    state.</p>
+    <p>Make one with New collection on the Reviews page, or from Manage collections in a card's menu.
+    To add several reviews at once, choose Select, tick them, and choose Manage collections. A review
+    can be in more than one collection. Ticking a collection adds every selected review to it and
+    leaves their other collections alone.</p>
+    <p>Each collection lists its reviews that are not Done. Marking one Done moves it under Show
+    completed, in every collection it is in, and Move to active brings it back. A review in a
+    collection is listed there rather than under Active, Waiting or Done. A pinned review stays under
+    Pinned. The Collections filter shows chosen collections, or Uncollected for reviews in none.</p>
+    <p>Reviews in a collection are kept for ninety days, like pinned and Done reviews, counted from
+    when you last opened or marked them. Adding one to a collection doesn't restart that count.
+    Deleting a collection keeps its reviews.</p>`
 	},
 	{
 		id: "several-pull-requests",
@@ -11205,8 +11559,8 @@ feature:                   E</pre>
     automatically:</p>
     <ul>
       <li><b>Reviews</b> are kept for thirty days after you last opened or changed them.</li>
-      <li><b>Reviews you kept</b> (saved, marked Done, or on your review queue) are kept for ninety
-      days instead.</li>
+      <li><b>Reviews you kept</b> (saved, marked Done, in a collection, or on your review queue) are
+      kept for ninety days instead.</li>
       <li><b>Isolated checkouts</b> are removed six hours after their work finishes. See
       <a href="#isolated-checkouts">Isolated checkouts</a>.</li>
       <li><b>Runs</b>, the record on the Runs page of what ran and how it ended, are kept for seven
@@ -11383,6 +11737,50 @@ function renderNotificationRow(notification, gone) {
   </li>`;
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/collections_live.ts
+/**
+* Collections as they relate to reviews that actually exist.
+*
+* `collections.ts` stores lineage keys and must not learn to read the review store, for the cycle
+* `queue_live.ts` describes: the prune in `store.ts` imports it. Anything that asks both questions
+* lives here, which imports both and is imported by neither.
+*/
+/**
+* Drop members whose lineage has no review any more, deleted elsewhere or pruned by age, and store
+* the result if that changed anything. Read against every group, not the windowed list, so a branch
+* the recency window happens to hide keeps its place.
+*
+* Compacted again under the lock rather than written from what was read, so a member somebody added
+* since is kept. On a failed write the compacted copy still stands in, so the page draws what exists.
+*/
+function settleCollections(groups) {
+	const live = new Set(groups.map((group) => group.key));
+	const compact = (collections) => collections.some((collection) => collection.members.some((key) => !live.has(key))) ? collections.map((collection) => ({
+		...collection,
+		members: collection.members.filter((key) => live.has(key))
+	})) : null;
+	const read = readCollections();
+	const compacted = compact(read);
+	if (compacted === null) return read;
+	return updateCollections((current) => compact(current)) ?? compacted;
+}
+/**
+* The lineage each review id belongs to, for the routes: the page sends the ids on its cards, and
+* the lineage is the server's to derive. An id with no review, or one that cannot be summarised, is
+* left out, and the caller says so.
+*/
+function lineageKeysFor(ids) {
+	const keys = /* @__PURE__ */ new Map();
+	for (const id of new Set(ids)) {
+		const workspace = loadWorkspace(id);
+		if (!workspace) continue;
+		try {
+			keys.set(id, lineageKey(summarise(workspace)));
+		} catch {}
+	}
+	return keys;
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/working_folders.ts
 /**
 * The local Git working folders PR Review Buddy can find right now.
@@ -11602,11 +12000,7 @@ function modelHelpOf(agentId) {
 	}
 }
 function agentNameOf(agentId) {
-	try {
-		return agentById(agentId).displayName;
-	} catch {
-		return agentId;
-	}
+	return isAgentId(agentId) ? agentIdentityOf(agentId).displayName : agentId;
 }
 var refuse = (status, error, extra = {}) => ({
 	status,
@@ -12602,6 +12996,7 @@ function reviewsRevision(live = analysisStates(), operations = liveOperations())
 	}
 	for (const operation of [...operations].sort((a, b) => a.key < b.key ? -1 : a.key > b.key ? 1 : 0)) hash.update(`op:${operation.key}:${operation.kind}:${operation.phase ?? ""}:${operation.reviewId ?? ""}\n`);
 	hash.update(`queue:${queueStamp()}\n`);
+	hash.update(`collections:${collectionsStamp()}\n`);
 	for (const [id, kind] of [...movedReviews()].sort(([a], [b]) => a.localeCompare(b))) hash.update(`moved:${id}:${kind}\n`);
 	for (const [key, state] of [...prStatesByRequest()].sort(([a], [b]) => a.localeCompare(b))) hash.update(`pr:${key}:${state}\n`);
 	for (const [key, entry] of Object.entries(readWatch()).sort(([a], [b]) => a.localeCompare(b))) if (entry.changedAt !== void 0) hash.update(`head:${key}:${entry.changedAt}\n`);
@@ -12793,7 +13188,7 @@ function reviewHref(review) {
 }
 /** The agent's name, only where the record names one this build knows. */
 function agentName(agentId) {
-	return agentId && isAgentId(agentId) ? agentById(agentId).displayName : void 0;
+	return agentId && isAgentId(agentId) ? agentIdentityOf(agentId).displayName : void 0;
 }
 /** Who asked and who ran it, each only when known. Two facts, never one guessed from the other. */
 function provenance(source, agentId) {
@@ -14598,6 +14993,8 @@ async function handle(req, res, options) {
 	if (url.pathname === "/pin") return handlePin(req, res, options.indexToken);
 	if (url.pathname === "/sections") return handleSections(req, res, options.indexToken);
 	if (url.pathname === "/queue") return handleQueue(req, res, options.indexToken);
+	if (url.pathname === "/collections") return handleCollections(req, res, options.indexToken);
+	if (url.pathname === "/collections/members") return handleCollectionMembers(req, res, options.indexToken);
 	if (url.pathname === "/delete") return handleDelete(req, res, options.indexToken);
 	if (url.pathname === "/status") return handleStatus(req, res, options.indexToken);
 	if (url.pathname === "/lifecycle") return handleLifecycle(req, res, options.indexToken, options.establishBaseline ?? establishBaseline);
@@ -14697,18 +15094,44 @@ function handleIndex(req, res, params, indexToken) {
 	}
 	const cookie = indexCookieHeader(indexToken);
 	if (onward) return redirect(res, onward, cookie);
+	sendHtml(res, 200, renderReviewsPage(indexView()), cookie);
+}
+/**
+* Which cards a page is drawn with: each branch, by its newest review. Equality only. Changes when a
+* branch is analysed again, or a card is added or deleted, which are the changes that make a
+* fragment drawn now wrong for a page drawn then. Nothing else moves it.
+*/
+function cardsSnapshot(groups) {
+	const hash = createHash("sha1");
+	for (const line of groups.map((group) => `${group.key}\t${group.latest.id}`).sort()) hash.update(`${line}\n`);
+	return hash.digest("hex").slice(0, 16);
+}
+/**
+* Everything the reviews index is drawn from, read in one pass.
+*
+* Shared by the page and by the collections routes, which answer with the Collections area drawn
+* from the same view, so a card in a collection after a change is the card a reload would draw.
+*/
+function indexView() {
 	const states = analysisStates();
 	const operations = liveOperations();
 	const reviews = recentWorkspaces(Number.MAX_SAFE_INTEGER);
 	const everyGroup = groupByLineage(reviews);
 	const queue = settleQueue(everyGroup);
+	const collections = settleCollections(everyGroup);
 	const kept = indexKeptIds(reviews, queue);
 	const activity = reviewActivity(everyGroup);
-	const groups = indexReviewGroups(MAX_LISTED_BRANCHES, MAX_KEPT_BRANCHES, kept, (group) => activity.get(group.latest.id)?.at ?? group.latest.createdAt, everyGroup);
+	const groups = indexReviewGroups(MAX_LISTED_BRANCHES, MAX_KEPT_BRANCHES, kept, (group) => activity.get(group.latest.id)?.at ?? group.latest.createdAt, everyGroup, {
+		keys: new Set(collections.flatMap((collection) => collection.members)),
+		limit: MAX_COLLECTED_BRANCHES
+	});
 	const rendered = new Set(groups.flatMap((group) => group.history.map((review) => review.id)));
-	sendHtml(res, 200, renderReviewsPage({
+	const live = queue.filter((id) => rendered.has(id));
+	return {
 		groups,
-		queue: queue.filter((id) => rendered.has(id)),
+		snapshot: cardsSnapshot(groups),
+		queue: live,
+		collections,
 		analyses: states,
 		operations,
 		moved: movedReviews(),
@@ -14719,7 +15142,7 @@ function handleIndex(req, res, params, indexToken) {
 		driving: isDriving(),
 		theme: readPreferences().theme,
 		sectionsCollapsed: readPreferences().reviewSections
-	}), cookie);
+	};
 }
 /**
 * Where `?go=` is allowed to send someone, which is one shape and no other.
@@ -15373,7 +15796,7 @@ async function handleSections(req, res, indexToken) {
 	if (req.method !== "POST") return send(res, 405, { error: "Use POST." });
 	if (!indexToken || !secretMatches(indexToken, indexCookie(req))) return send(res, 403, { error: "Not authorised." });
 	const body = await readJson(req);
-	if (!isListSection(body?.section) || typeof body?.collapsed !== "boolean") return send(res, 400, { error: "Not a section this page can fold." });
+	if (!isFoldKey(body?.section) || typeof body?.collapsed !== "boolean") return send(res, 400, { error: "Not a section this page can fold." });
 	writeSectionCollapsed(body.section, body.collapsed);
 	send(res, 200, { ok: true });
 }
@@ -15442,6 +15865,102 @@ function addMany(res, ids) {
 	});
 }
 /**
+* Making, renaming and deleting collections, from the Reviews page.
+*
+* The index cookie, like the queue: this is the list acting on itself. Every answer that changed
+* something carries the collections as stored and the Collections area redrawn from them, so the
+* page swaps in exactly what a reload would show rather than building cards of its own.
+*/
+async function handleCollections(req, res, indexToken) {
+	if (req.method !== "POST") return send(res, 405, { error: "Use POST." });
+	if (!indexToken || !secretMatches(indexToken, indexCookie(req))) return send(res, 403, { error: "Not authorised." });
+	const body = await readJson(req);
+	const action = body?.action;
+	const name = typeof body?.name === "string" ? body.name : "";
+	const id = isCollectionId(body?.id) ? body.id : null;
+	const snapshot = typeof body?.snapshot === "string" ? body.snapshot : "";
+	if (action === "create") return answerCollections(res, createCollection(name), [], snapshot);
+	if (action === "rename" && id) return answerCollections(res, renameCollection(id, name), [], snapshot);
+	if (action === "delete" && id) {
+		const outcome = deleteCollection(id);
+		if (outcome.ok) forgetSections([collectionFold(id), completedFold(id)]);
+		return answerCollections(res, outcome, [], snapshot);
+	}
+	send(res, 400, { error: "Unknown collections action." });
+}
+/**
+* Putting reviews into one collection, or taking them out of it: one card's menu or the bulk bar.
+*
+* The page sends review ids, the ids on its cards, and the lineage each belongs to is derived here,
+* because a lineage is what a collection holds and the server is the only side that can say which.
+* Only the named collection changes, whatever else the reviews belong to. `create` makes the
+* collection first, by name, so "Create Refactor" from the panel is one request and cannot leave an
+* empty collection behind a failed assignment without saying so.
+*/
+async function handleCollectionMembers(req, res, indexToken) {
+	if (req.method !== "POST") return send(res, 405, { error: "Use POST." });
+	if (!indexToken || !secretMatches(indexToken, indexCookie(req))) return send(res, 403, { error: "Not authorised." });
+	const body = await readJson(req);
+	const action = body?.action;
+	const ids = Array.isArray(body?.ids) ? body.ids.filter((one) => typeof one === "string") : [];
+	if (action !== "add" && action !== "remove" || ids.length === 0) return send(res, 400, { error: "Say which reviews, and whether to add or remove them." });
+	const keys = lineageKeysFor(ids);
+	const gone = ids.filter((one) => !keys.has(one));
+	const snapshot = typeof body?.snapshot === "string" ? body.snapshot : "";
+	if (action === "add" && typeof body?.create === "string" && !isCollectionId(body?.collection)) {
+		if (gone.length > 0) return send(res, 409, { error: "A review you selected has been deleted, so nothing was created. Reload the page and try again." });
+		return answerCollections(res, createWithMembers(body.create, [...keys.values()]), [], snapshot);
+	}
+	const collection = isCollectionId(body?.collection) ? body.collection : null;
+	if (collection === null) return send(res, 400, { error: "Say which collection." });
+	answerCollections(res, setMembership([...keys.values()], collection, action), gone.map((one) => ({
+		id: one,
+		error: "A review you selected has been deleted."
+	})), snapshot);
+}
+/**
+* The answer every collections route gives: what is stored, and the area drawn from it, or why not.
+*
+* **Only for the page it would fit.** The area's cards and the memberships are keyed by each
+* branch's newest review, and a page drawn before a branch was analysed again, or before one was
+* added or deleted, holds cards under other ids. Swapping a fragment drawn from today's reviews into
+* that page would mix the two. So the page says which cards it was drawn with (`snapshot`, from
+* `cardsSnapshot`), and when that is not what the server would draw now the answer is `stale`, with
+* no fragment: the change itself is stored either way, by lineage, and the page reloads to show it.
+*/
+function answerCollections(res, outcome, refused, snapshot) {
+	if (!outcome.ok) {
+		if (outcome.reason === "missing") return send(res, 404, { error: "That collection has been deleted." });
+		if (outcome.reason === "full") return send(res, 409, { error: "That is more than a collection can hold." });
+		if (outcome.reason === "unsaved") return send(res, 500, { error: "The collections could not be saved." });
+		return send(res, 409, {
+			error: nameProblemText(outcome.reason),
+			reason: outcome.reason
+		});
+	}
+	const view = indexView();
+	const collections = (view.collections ?? []).map(({ id, name }) => ({
+		id,
+		name
+	}));
+	if (snapshot === "" || snapshot !== view.snapshot) return send(res, 200, {
+		stale: true,
+		collections,
+		...outcome.id === void 0 ? {} : { id: outcome.id },
+		refused
+	});
+	const memberOf = {};
+	for (const group of view.groups) memberOf[group.latest.id] = (view.collections ?? []).filter((one) => one.members.includes(group.key)).map((one) => one.id);
+	send(res, 200, {
+		collections,
+		...outcome.id === void 0 ? {} : { id: outcome.id },
+		memberOf,
+		operations: operationCollections(view.operations ?? [], view.groups, membershipsByLineage(view.collections ?? [])),
+		area: renderCollectionsArea(view),
+		refused
+	});
+}
+/**
 * 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,
@@ -15598,6 +16117,13 @@ var MAX_LISTED_BRANCHES = 50;
 */
 var MAX_KEPT_BRANCHES = 50;
 /**
+* How many collected branches the list draws, in a window of their own beside the two above, so a
+* collection never competes with Pinned or the queue for a place. One collection's worth: a single
+* collection is always drawn whole, and only collections holding more than this between them leave
+* their oldest members out, which each collection then says.
+*/
+var MAX_COLLECTED_BRANCHES = 500;
+/**
 * A finished review's workspace was served.
 *
 * **This counts page serves, not people and not sessions.** A reload, a restored tab, the CLI
@@ -15996,6 +16522,8 @@ async function handleReanalyse(req, res, workspace) {
 	if (req.method !== "POST") return send(res, 405, { error: "Analysing a review again needs a POST." });
 	const current = workspace.jobId ? loadJob(workspace.jobId) : null;
 	if (current && (!isTerminal(current) || isClaimed(current.id))) return send(res, 409, { error: "This review is still being made. Wait for it to finish before analysing it again." });
+	const parked = parkedReason(agentIdOf(workspace.session));
+	if (parked) return send(res, 409, { error: `${parked} To analyse this review again with another agent, run \`prreviewbuddy review --review ${workspace.id} --reanalyse\` with ${runnableAgents().flags}.` });
 	await new Promise((resolve) => {
 		const run = reanalyseReview(workspace.id, void 0, () => {
 			send(res, 200, { ok: true });
@@ -16036,6 +16564,8 @@ async function handleKiss(req, res, workspace) {
 	if (!audience) return send(res, 400, { error: "A KISS review is written either for the person who wrote the code or for somebody reviewing it. Choose which before running it." });
 	const current = latestKissRun(workspace);
 	if (current && current.status === "generating" && isClaimed(current.jobId)) return send(res, 409, { error: "A KISS review of this change is already running." });
+	const parked = parkedReason(kissAgentId(workspace));
+	if (parked) return send(res, 409, { error: `${parked} A KISS review started here uses the agent this review was made with. To run one with another agent, start it from the terminal with \`--lens kiss\` and ${runnableAgents().flags}.` });
 	startKissJob({
 		workspaceId: workspace.id,
 		audience,