prreviewbuddy 0.93.0 → 0.95.2

package/dist/server.js

@@ -1,9 +1,9 @@
-import { $a as versionFrom, $i as isExplanationAction, $n as hasAuthorContent, $r as latestKissRun, $t as waitingHistory, A as settleQueue, Aa as isCommitReview, Ai as deleteCollection, An as markRead, Ar as endedAt, At as askCheckout, B as humanizeModelId, Ba as clearAgentPreference, Bi as readQueue, Bn as unreadCount$1, Br as progressSteps, Bt as runJob, C as parsePullRequestLink, Ca as describeAuthorship, Ci as lifecyclePillLabel, Cn as categoryOf, Cr as reviewedShaOf, Ct as RefreshUnavailableError, D as activateQueued, Da as originRemote, Di as collectionsStamp, Dn as eventName, Dr as RUN_RECORD_RETENTION_MS, E as relativeTime$1, Ea as resolveTarget, Ei as sectionOf$1, En as createPrUpdate, Er as fillFileUrlTemplate, Et as mergeUpdateInto, Fa as git, Fi as updateCollections, Fn as readStateOf, Fr as saveJob, Ft as failureOf$1, G as reviewTag, Ga as AGENT_IDS, Gi as questionsOutstanding, Gn as saveRefresh, H as findingRef, Ha as writeAgentPreference, Hi as isIssueOutstanding, Hn as existingPaths, Ia as refExists, Ii as queueAdd, In as removeNotification, It as observeReview, Ja as agentIdentityOf, Ji as verdictClosure, Jn as agentEnvOf, Jr as groupByLineage, Jt as telemetryUploadConsent, K as reviewTagsIn, Ka as agentById, Ki as resolvedByConversation, Kn as storedConversation, Kr as decideKissRecommendation, L as modelProvenanceOf, La as resolveAgent, Li as queueAddMany, Ln as retireNotification, Lr as isLiveJob, Lt as prepareReview, Ma as AmbiguousRefError, Mi as readCollections, Mn as moveIds, Mr as isTerminal, Mt as UpdateAlreadyRunningError, N as cliEntry, Na as shortNameOf, Ni as renameCollection, Nn as pruneNotifications, Nr as loadJob, Nt as liveUpdateFor, O as compactQueue, Oa as DetachedHeadError, Oi as createCollection, On as listNotifications, Or as allJobs, Ot as recheckConversation, Pa as shorthandOf, Pi as setMembership, Pn as readNotification, Pr as pruneJobResidue, Pt as runningJobs, Q as markAskRunning, Qa as runnableAgents, Qi as explanationOf, Qn as authorPreview, Qr as kissDecisionsOf, Qt as feedbackUrl, R as provenanceDetailLines, Ra as resolveModel, Ri as queueRemove, Rn as toneOf, Rr as isTerminalJobPhase, Rt as settleReview, S as extractPullRequestLinks, Sa as objectWithAnyOf, Si as lifecyclePillAction, Sn as subjectOf, Sr as followedBranch, St as FETCH_TIMEOUT_MS, T as runBatch, Ta as kissOutcome, Ti as parseLifecycle, Tn as createPrState, Tr as forgeResolver, Tt as cancellableHere, U as findingRefs, Ui as isQuestionOutstanding, Un as ASK_REREAD_CEILING_MS, Ut as readEvents, V as findingAtRef, Va as readAgentPreference, Vi as doneVerb, Vn as withUsageRecorded, Vr as purposeOf, W as refOf, Wa as STORE_ROOT, Wi as issuesOutstanding, Wn as prepareConversationWithin, Wr as isClaimed, Wt as record, X as isRequestId, Xa as isAgentId, Xi as recommendationsOf, Xn as chatConversationSection, Xr as indexReviewGroups, Xt as BUILD_VERSION, Ya as detectAgents, Yi as processDiscussion, Yn as EXPLAIN_SIMPLY_PROMPT, Yr as indexKeptIds, Za as parkedReason, Zi as explanationActionOf, Zn as UNESTABLISHED_MOTIVATION, Zr as kissDecisionRefusal, _ as endRun, _a as wasReassessed, _i as activityLabel, _n as ancestorsOf, _t as parseWorkspacePort, a as lineageIds, aa as closesFinding, an as ARCHITECTURE_DIR, ar as renderInlineText, b as stopRun, ba as asString, bi as lifecycleItemLabel, bn as announceJob, br as checkCodeFreshness, bt as writePortPreference, c as discardJob, ca as findingIn, ci as recentWorkspaces, cn as partFiles, cr as isBusy, ct as serverEntryExists, d as startKissJob, da as isGradeFor, di as summarise, dn as storeArchitecture, dr as createWorktree, ea as withExplanation, ei as lineageKey, en as mapRuntimes, er as GITHUB_MARKDOWN, f as isDriving, fa as isOffered, fi as touchWorkspace, fn as storedScopes, fr as holdWorktree, g as completeRun, ga as statusLookup, gi as activityAttributes, gn as REPOSITORY_SCOPE, gr as requestJobStop, gt as MIN_WORKSPACE_PORT, h as beginRun, ha as sourcePhrase, hi as LIFECYCLES, hn as QUOTE_LIMITS, hr as removeWorktree, ht as MAX_WORKSPACE_PORT, i as deleteReview, ia as assessmentLookup, ii as mutateWorkspace, in as formatDuration, ir as escapeHtml$1, ja as resolveBase, ji as nameProblemText, jn as markReadForReview, jr as fail$1, jt as findingsChanged, ki as createWithMembers, kn as markAllRead, kr as attemptIdOf, kt as updateReview, l as kissAgentId, la as findingStatus, li as reviewedCommit, ln as readArchitecture, lr as withReviewLock, lt as spawnServer, m as RUN_KINDS, ma as reassessIn, mi as workspaceStamps, mn as ownersOf, mr as readMarker, mt as DEFAULT_WORKSPACE_PORT, n as MCP_TOOL_NAMES, na as NO_STATUS, ni as loadWorkspace, nn as RUNTIME_WORDS, no as AgentUnavailableError, nr as CODE_BLOCK_CLASS, o as liveJobsFor, oa as currentGradeIn, oi as presentedKissRunOf, on as checkArchitecture, ot as readServerRecord, p as track, pa as promptGrade, pi as workspaceRevision, pn as trackedFiles, pr as isWorktreeHeld, pt as writeServerRecord, q as endAskRequest, qa as agentFor$1, qi as reviewerDispositions, qn as isUnchanged, qr as followedRefName, qt as recordTelemetryUploadConsent, r as deleteLineage, ra as analysisRefOf, rn as describeRuntimes, rr as CODE_COPY_CLASS, rt as clearServerRecordOwnedBy, s as wasBlocked, sa as findingAssessment, si as previousKissResult, sn as loadArchitecture, sr as ReviewBeingDeletedError, ta as ANALYSIS_ONLY, ti as lineagePosition, tn as recordRuntimeBestEffort, to as AgentCancelledError, tr as renderMarkdown, tt as readIndexToken, ua as gradeScale, un as repositoryKey, ur as MANAGED_ROOT, ut as stopChild, v as heldRuns, va as followUpFindingId, vi as isLifecycle, vn as isArchitectureId, vt as resolveWorkspacePort, w as unsupportedReason, wa as WITHHELD_REASON, wi as offersLifecycle, wn as createArchitectureNotification, wr as fetchHead, wt as cancelUpdate, x as batchIdentity, xa as extractJsonText, xi as lifecycleName$1, xn as branchOf, xr as checkReview, y as reportRun, ya as SEVERITIES, yi as lifecycleChip, yn as writeJsonAtomically, yr as findExistingReview, yt as statedWorkspacePort, z as agentDisplayName, za as agentIdOf, zi as queueStamp, zn as unreadAround, zr as phasesFor, zt as reanalyseReview } from "./mcp_tools-DbFR9Z90.js";
-import { basename, delimiter, isAbsolute, join, relative, resolve } from "node:path";
-import { execFile, spawn } from "node:child_process";
+import { $ as statusPillLabel, $i as lifecycleItemLabel, $n as writeJsonAtomically, $r as findExistingReview, $t as statedWorkspacePort, A as anythingRunning, Aa as currentGradeIn, Ai as indexReviewGroups, An as BUILD_VERSION, Ao as deleteCollection, Ar as chatConversationSection, At as isRequestId, B as completedLabel, Ba as statusLookup, Bi as previousKissResult, Bn as checkArchitecture, Bt as readServerRecord, C as parsePullRequestLink, Ca as isExplanationAction, Cn as readEvents, Cr as ASK_REREAD_CEILING_MS, Ct as findingRefs, D as INDEX_SECTIONS, Da as analysisRefOf, Di as followedRefName, Dn as recordTelemetryUploadConsent, Do as collectionsStamp, Dr as isUnchanged, Dt as endAskRequest, E as relativeTime$1, Ea as NO_STATUS, Ei as decideKissRecommendation, Er as storedConversation, Et as reviewTagsIn, F as collectionFold, Fa as isGradeFor, Fi as loadWorkspace, Fn as recordRuntimeBestEffort, Fo as STORE_ROOT, Fr as renderMarkdown, Ft as readIndexToken, G as markItemLabel, Ga as extractJsonText, Gi as touchWorkspace, Gn as storeArchitecture, Gr as createWorktree, H as homeSection, Ha as followUpFindingId, Hi as reviewedCommit, Hn as partFiles, Hr as isBusy, Ht as serverEntryExists, Ia as isOffered, In as RUNTIME_WORDS, Ir as CODE_BLOCK_CLASS, J as queueItemLabel, Ja as WITHHELD_REASON, Ji as LIFECYCLES, Jn as ownersOf, Jr as readMarker, Jt as DEFAULT_WORKSPACE_PORT, K as omittedText, Ka as objectWithAnyOf, Ki as workspaceRevision, Kn as storedScopes, Kr as holdWorktree, L as collectionPlacement, La as promptGrade, Li as mutateWorkspace, Ln as describeRuntimes, Lr as CODE_COPY_CLASS, Lt as clearServerRecordOwnedBy, M as authorOptions, Ma as findingIn, Mi as kissDecisionsOf, Mn as feedbackUrl, Mo as readCollections, Mr as authorPreview, Mt as markAskRunning, N as collectionCountLabel, Na as findingStatus, Ni as latestKissRun, Nn as waitingHistory, No as renameCollection, Nr as hasAuthorContent, O as INDEX_SECTION_NAMES, Oa as assessmentLookup, Oi as groupByLineage, On as telemetryUploadConsent, Oo as createCollection, Or as agentEnvOf, P as collectionEmptyText, Pa as gradeScale, Pi as lineagePosition, Pn as mapRuntimes, Po as setMembership, Pr as GITHUB_MARKDOWN, Q as statusLabel, Qa as DetachedHeadError, Qi as lifecycleChip, Qn as isArchitectureId, Qt as resolveWorkspacePort, R as collectionsLine, Ra as reassessIn, Rn as formatDuration, Rr as escapeHtml$1, S as extractPullRequestLinks, Sa as explanationOf, So as versionFrom, Sr as existingPaths, St as findingRef, T as runBatch, Ta as ANALYSIS_ONLY, To as AgentUnavailableError, Tr as saveRefresh, Tt as reviewTag, U as isCollectionId, Ua as SEVERITIES, Un as readArchitecture, Ur as withReviewLock, Ut as spawnServer, V as countLabel, Va as wasReassessed, Vi as recentWorkspaces, Vn as loadArchitecture, Vr as ReviewBeingDeletedError, W as isFoldKey, Wa as asString, Wn as repositoryKey, Wr as MANAGED_ROOT, Wt as stopChild, X as sectionOpen, Xa as resolveTarget, Xi as activityLabel, Xn as REPOSITORY_SCOPE, Xr as requestJobStop, Xt as MIN_WORKSPACE_PORT, Y as queueOptionLabel, Ya as kissOutcome, Yi as activityAttributes, Yn as QUOTE_LIMITS, Yr as removeWorktree, Yt as MAX_WORKSPACE_PORT, Z as selectAllLabel, Za as originRemote, Zi as isLifecycle, Zn as ancestorsOf, Zt as parseWorkspacePort, _ as endRun, _a as reviewerDispositions, _i as isLiveJob, _n as prepareReview, _o as agentIdentityOf, _r as retireNotification, _t as modelProvenanceOf, a as lineageIds, aa as sectionOf$1, ai as forgeResolver, an as cancellableHere, ao as git, ar as createPrState, at as lineageKeysFor, b as stopRun, ba as recommendationsOf, bi as progressSteps, bn as runJob, bo as parkedReason, br as unreadCount$1, bt as humanizeModelId, c as discardJob, ca as queueRemove, ci as allJobs, cn as recheckConversation, co as resolveModel, cr as listNotifications, ct as compactQueue, d as startKissJob, da as doneVerb, di as fail$1, dn as findingsChanged, do as readAgentPreference, dr as markReadForReview, ea as lifecycleName, ei as checkCodeFreshness, en as writePortPreference, eo as isCommitReview, er as announceJob, et as deliver, f as isDriving, fa as isIssueOutstanding, fi as isTerminal, fn as UpdateAlreadyRunningError, fo as writeAgentPreference, fr as moveIds, g as completeRun, ga as resolvedByConversation, gn as observeReview, go as agentFor$1, gr as removeNotification, h as beginRun, ha as questionsOutstanding, hi as saveJob, hn as failureOf$1, ho as agentById, hr as readStateOf, i as deleteReview, ia as parseLifecycle, ii as fetchHead, in as cancelUpdate, io as shorthandOf, ir as createArchitectureNotification, it as retryableMiss, j as authorOptionLabel, ja as findingAssessment, ji as kissDecisionRefusal, jo as nameProblemText, jr as UNESTABLISHED_MOTIVATION, k as PR_FILTER_OPTIONS, ka as closesFinding, ki as indexKeptIds, ko as createWithMembers, kr as EXPLAIN_SIMPLY_PROMPT, l as kissAgentId, la as queueStamp, li as attemptIdOf, ln as updateReview, lo as agentIdOf, lr as markAllRead, m as RUN_KINDS, ma as issuesOutstanding, mi as pruneJobResidue, mn as runningJobs, mo as AGENT_IDS, mr as readNotification, n as MCP_TOOL_NAMES, na as lifecyclePillLabel, ni as followedBranch, nn as FETCH_TIMEOUT_MS, no as AmbiguousRefError, nr as subjectOf, o as liveJobsFor, oa as queueAdd, oi as fillFileUrlTemplate, on as mergeUpdateInto, oo as refExists, or as createPrUpdate, ot as settleCollections, p as track, pa as isQuestionOutstanding, pi as loadJob, pn as liveUpdateFor, pr as pruneNotifications, pt as cliEntry, q as pinItemLabel, qa as describeAuthorship, qi as workspaceStamps, qn as trackedFiles, qr as isWorktreeHeld, qt as writeServerRecord, r as deleteLineage, ra as offersLifecycle, ri as reviewedShaOf, rn as RefreshUnavailableError, ro as shortNameOf, rr as categoryOf, rt as openDestination, s as wasBlocked, sa as queueAddMany, si as RUN_RECORD_RETENTION_MS, so as resolveAgent, sr as eventName, st as activateQueued, ta as lifecyclePillAction, ti as checkReview, to as resolveBase, tr as branchOf, ua as readQueue, ui as endedAt, un as askCheckout, uo as clearAgentPreference, ur as markRead, ut as settleQueue, v as heldRuns, va as verdictClosure, vi as isTerminalJobPhase, vn as settleReview, vo as detectAgents, vr as toneOf, vt as provenanceDetailLines, w as unsupportedReason, wa as withExplanation, wi as isClaimed, wn as record, wo as AgentCancelledError, wr as prepareConversationWithin, wt as refOf, x as batchIdentity, xa as explanationActionOf, xi as purposeOf, xo as runnableAgents, xr as withUsageRecorded, xt as findingAtRef, y as reportRun, ya as processDiscussion, yi as phasesFor, yn as reanalyseReview, yo as isAgentId, yr as unreadAround, yt as agentDisplayName, z as completedFold, za as sourcePhrase, zi as presentedKissRunOf, zn as ARCHITECTURE_DIR, zr as renderInlineText } from "./mcp_tools-DDbcXys2.js";
+import { createHash, randomUUID, timingSafeEqual } from "node:crypto";
 import { accessSync, constants, existsSync, mkdirSync, readFileSync, readdirSync, realpathSync, rmSync, statSync, writeFileSync } from "node:fs";
+import { basename, delimiter, isAbsolute, join, relative, resolve } from "node:path";
 import { homedir } from "node:os";
-import { createHash, randomUUID, timingSafeEqual } from "node:crypto";
+import { execFile, spawn } from "node:child_process";
 import { fileURLToPath } from "node:url";
 import { createServer } from "node:http";
 //#region ../../packages/review-harness/src/domain/ai_review/review_composition.ts
@@ -477,317 +477,6 @@ function renderIntegrationsNotice(names) {
 </div>`;
 }
 //#endregion
-//#region ../../packages/review-harness/src/workspace/reviews_index.ts
-/**
-* The sections of the list, top to bottom.
-*
-* Pinned is not a lifecycle state. It is where a card is drawn, on top of a lifecycle it keeps: a
-* pinned review can be Active, Waiting or Done, and unpinning it puts it back in the section that
-* says. That is why it lives here, beside the other placement rules, and not in `LIFECYCLES`.
-*/
-var INDEX_SECTIONS = [
-	"pinned",
-	"active",
-	"waiting",
-	"done"
-];
-/**
-* Every section the list draws, top to bottom: In progress, then the card sections.
-*
-* In progress is not in `INDEX_SECTIONS` because no card is ever placed there. It lists work, one
-* entry per running job or waiting row (`deriveOperations`), and a review being updated stays a
-* card in its own section while its update is listed here. Windows, Load more, the selection and
-* the counts are all about cards, so they keep reading `INDEX_SECTIONS`; folding, its memory and its
-* keyboard commands are about headings, so they read this.
-*/
-var LIST_SECTIONS = ["in-progress", ...INDEX_SECTIONS];
-/**
-* Each section's name, as its heading says it, and the key that folds it from the keyboard
-* (⌘K L and this key). The commands are made from this table, so a section added here is a section
-* the keyboard can fold, with no second list to remember.
-*/
-var lifecycleName = (id) => LIFECYCLES.find((lifecycle) => lifecycle.id === id).name;
-var INDEX_SECTION_NAMES = {
-	"in-progress": {
-		title: "In progress",
-		key: "i"
-	},
-	pinned: {
-		title: "Pinned",
-		key: "p"
-	},
-	active: {
-		title: lifecycleName("active"),
-		key: "a"
-	},
-	waiting: {
-		title: lifecycleName("waiting"),
-		key: "w"
-	},
-	done: {
-		title: lifecycleName("done"),
-		key: "d"
-	}
-};
-/**
-* Whether a section starts open. The reviewer's choice wins, and otherwise everything is open but
-* 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" || 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.
-*
-* Pinning controls prominence in the general list and never overrides an ordering the reviewer
-* asked for. Under such a filter a pinned card stays in its lifecycle section, in the order the
-* filter gives, and only its star says it is pinned.
-*/
-function indexSectionOf(pinned, lifecycle, activity, filter) {
-	return pinned && !ordersExplicitly(filter) ? "pinned" : sectionOf$1(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$1(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 "No active reviews";
-}
-/** 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.
-*/
-function ordersExplicitly(filter) {
-	return filter === "queue";
-}
-/**
-* "45 reviews · 13 branches", and "13 reviews" when those are the same thirteen things.
-*
-* The two numbers are different questions and only both worth asking once they differ. A page of
-* thirteen branches reviewed once each would read "13 reviews · 13 branches", which invites the
-* reader to work out what the distinction is on the one page where there isn't one.
-*
-* `of` is every review on the page, given while the search or a filter is narrowing the list:
-* "12 of 71 reviews" says the list is filtered and by how much, where "12 reviews" reads as all
-* there are.
-*/
-function countLabel(reviews, branches, of) {
-	const label = of === void 0 ? `${reviews} review${reviews === 1 ? "" : "s"}` : `${reviews} of ${of} review${of === 1 ? "" : "s"}`;
-	if (reviews === branches) return label;
-	return `${label} · ${branches} branch${branches === 1 ? "" : "es"}`;
-}
-/**
-* Every login the author dropdown offers, once each, in alphabetical order of what the reader sees:
-* the name where there is one, the login otherwise, regardless of case. Read from the cards the page
-* is showing, so it never lists somebody with nothing to show.
-*
-* A login's name is the one on its newest review that has a name, chosen here by `createdAt` rather
-* than by the order the caller happens to pass, so an account renamed upstream reads as it does now.
-*/
-function authorOptions(cards) {
-	const newest = /* @__PURE__ */ new Map();
-	for (const card of cards) {
-		if (!card.handle) continue;
-		const seen = newest.get(card.handle);
-		if (!seen) newest.set(card.handle, {
-			name: card.name || void 0,
-			createdAt: card.createdAt
-		});
-		else if (card.name && (!seen.name || card.createdAt > seen.createdAt)) newest.set(card.handle, {
-			name: card.name,
-			createdAt: card.createdAt
-		});
-	}
-	return [...newest].map(([handle, { name }]) => name && name !== handle ? {
-		handle,
-		name
-	} : { handle }).sort((a, b) => (a.name ?? a.handle).localeCompare(b.name ?? b.handle, void 0, { sensitivity: "base" }));
-}
-/** An author option as the dropdown spells it: the name and the login, or the login alone. */
-function authorOptionLabel(option) {
-	return option.name ? `${option.name} · @${option.handle}` : `@${option.handle}`;
-}
-/**
-* A card menu's status item, which sets the state it names. The bulk bar's words for the same two
-* 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 not reviewed";
-}
-/**
-* The review page's Pin item, which says what pressing it will do. The words of the index's star,
-* so pinning from either place reads as the same act. Here because the browser relabels the item
-* after the change, for the reason `markItemLabel` is here.
-*/
-function pinItemLabel(pinned) {
-	return pinned ? "Unpin review" : "Pin review";
-}
-/**
-* The queue item, on a card and on the review's own page. "review queue" rather than "queue",
-* because a menu is where somebody meets the idea for the first time. Here because the browser
-* relabels it after a change, on both pages.
-*/
-function queueItemLabel(queued) {
-	return queued ? "Remove from review queue" : "Add to review queue";
-}
-/**
-* 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 queueOptionLabel(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
-* selection reviewed repaints it without asking the server for the page again, and two copies of
-* these three words is how a page ends up saying "Reviewed" in one place and "Complete" in the
-* other about the same review.
-*/
-function statusLabel(status) {
-	if (status === "complete") return "Reviewed";
-	return status === "in-progress" ? "Not reviewed" : "New";
-}
-/**
-* The status pill's accessible name: where the reviewer is, then what pressing it does. The visible
-* words are the state, so a screen reader hears both rather than only a state it cannot act on.
-*/
-function statusPillLabel(status) {
-	return `${statusLabel(status)}. Mark as ${status === "complete" ? "not reviewed" : "reviewed"}`;
-}
-/**
-* The bulk bar's way to tick everything the filters show, across pages.
-*
-* Counted ("Select all 7") only while the headline is not already saying "of 7", so the number is
-* on the bar once. With PR set to Merged this is how a reviewer clears the merged reviews in one go,
-* by way of the existing Mark reviewed.
-*/
-function selectAllLabel(inView, scopeShown = false) {
-	return inView === 0 || scopeShown ? "Select all" : `Select all ${inView}`;
-}
-/** The PR filter's options, in order, as the select shows them. */
-var PR_FILTER_OPTIONS = [
-	{
-		value: "any",
-		label: "Any"
-	},
-	{
-		value: "open",
-		label: "Open"
-	},
-	{
-		value: "merged",
-		label: "Merged"
-	},
-	{
-		value: "closed",
-		label: "Closed"
-	}
-];
-/**
-* Whether any of that is work still going on, which is what sets the polling cadence.
-*
-* Here rather than beside `analysisStates` because this module is the DOM-free, filesystem-free
-* arithmetic both halves of the index share, and the renderer needs this one answer without
-* needing an import that reads the jobs directory to get it.
-*
-* A stopped analysis is a state, not an activity. Counting it as live would leave the index asking
-* once a second about a review that finished failing an hour ago, for as long as the tab stayed
-* open.
-*/
-function anythingRunning(states, operations = []) {
-	if (operations.length > 0) return true;
-	for (const state of states.values()) if (state.phase !== "failed") return true;
-	return false;
-}
-//#endregion
 //#region ../../packages/review-harness/src/workspace/fetch_updates_view.ts
 /** A card's and a review page's item. Named for what it reaches: a remote branch, not the list. */
 var FETCH_REVIEW_LABEL = "Fetch remote updates";
@@ -1033,7 +722,7 @@ function statusPillHtml(status, attributes = "") {
 */
 function lifecycleControlHtml(lifecycle, activity, options) {
 	const state = lifecycle?.state ?? "active";
-	const name = escapeHtml$1(lifecycleName$1(lifecycle));
+	const name = escapeHtml$1(lifecycleName(lifecycle));
 	const attributes = options.attributes ?? "";
 	if (!options.interactive) return `<span class="prb-state-pill prb-lifecycle-pill" data-lifecycle-pill="${state}"${attributes}>${name}</span>`;
 	const action = lifecyclePillAction(lifecycle);
@@ -10395,7 +10084,8 @@ function renderReviewsPage(view) {
 	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");
+		const pinned = group.latest.pinned === true;
+		const home = homeSection(pinned, group.latest.lifecycle, activityOf(group), collected, "all");
 		if (home === void 0) collectedHome.add(group);
 		sections[home ?? sectionOf$1(group.latest.lifecycle, activityOf(group))].push(group);
 	}
@@ -12903,7 +12593,14 @@ feature:                   E</pre>
     a time unless <code>--concurrency</code> says otherwise.</p>
     <p>One that fails does not stop the others. Ctrl-C stops them all, and running the same command
     again carries each one on from where it got to. Add <code>--queue</code> to put each review on
-    your review queue as soon as it exists, rather than when it is ready.</p>`
+    your review queue as soon as it exists, rather than when it is ready.</p>
+    <p>To keep a batch together, add it to a collection: <code>--collection "Release review"</code>
+    for one you have, or <code>--new-collection "Friday batch"</code> to make one, with
+    <code>--also-queue</code> to put the reviews on your review queue as well. In New review, choose
+    the same from <b>Add to</b>: <b>Reviews list only</b>, <b>Review queue</b>, a collection, or
+    <b>Create collection…</b>, and tick <b>Also add to review queue</b> for both. When the batch
+    ends, <b>Open collection</b> shows it. A retry adds only what is missing, to the same
+    collection.</p>`
 	},
 	{
 		id: "review-lenses",
@@ -13196,50 +12893,6 @@ 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.
@@ -13448,7 +13101,13 @@ function realAddReviewDeps() {
 			} : null;
 		},
 		agentName: agentNameOf,
-		home: homedir()
+		home: homedir(),
+		collections: () => readCollections().map((collection) => ({
+			id: collection.id,
+			name: collection.name
+		})),
+		openDestination,
+		deliver
 	};
 }
 function modelHelpOf(agentId) {
@@ -13589,7 +13248,8 @@ async function addReviewOptions(deps) {
 				kind: "refused",
 				error: unresolvedAgent(resolution)
 			},
-			canChooseFolder: deps.chooseFolder !== null
+			canChooseFolder: deps.chooseFolder !== null,
+			collections: deps.collections()
 		}
 	};
 }
@@ -13703,12 +13363,13 @@ function pendingBatchRows() {
 function forgetFinished(now) {
 	for (const [id, batch] of batches) if (batch.finishedAt !== null && now - Math.max(batch.finishedAt, batch.seenAt) > FINISHED_BATCH_TTL_MS) batches.delete(id);
 }
-function newBatch(rows) {
+function newBatch(rows, destination = { queue: false }) {
 	forgetFinished(Date.now());
 	const now = Date.now();
 	const batch = {
 		id: randomUUID(),
 		rows,
+		destination,
 		finishedAt: now,
 		seenAt: now,
 		running: 0,
@@ -13735,7 +13396,14 @@ function fail(row, error) {
 	row.reason = refusal.error;
 	if (refusal.detail) row.detail = refusal.detail;
 }
-var rowsOf = (batch) => batch.rows.map((row) => ({ ...row }));
+var rowsOf = (batch) => batch.rows.map((row) => ({
+	...row,
+	...missToRetry(row, batch) ? { addAgain: true } : {}
+}));
+/** What of the batch's destination this row's review is still worth adding to, or null. */
+function missToRetry(row, batch) {
+	return row.reviewId && row.delivered ? retryableMiss(row.delivered, batch.destination) : null;
+}
 /**
 * Create the review for a row's target, or find the one it has, under the per-target lock. The
 * caller holds an analysis slot: nothing is created without one, so nothing created waits.
@@ -13795,6 +13463,7 @@ async function stillThere(row, target, what, deps) {
 */
 async function retryReview(row, batch, deps) {
 	const reviewId = row.reviewId;
+	deliverRow(row, batch, deps);
 	const kept = deps.resumeOf(reviewId);
 	begin(row);
 	if (kept) {
@@ -13808,6 +13477,14 @@ async function retryReview(row, batch, deps) {
 * target, the old review, and still `stale`. So the row keeps offering the fresh start, and a plain
 * Retry finds the old commit gone again, rather than resuming the old review against it.
 */
+/**
+* Put a row's review where its batch goes, as soon as it has one. Idempotent, so it runs again on
+* every retry and fills in only what is missing. A batch for the reviews list alone does nothing.
+*/
+function deliverRow(row, batch, deps) {
+	if (!row.reviewId || !batch.destination.queue && !batch.destination.collection) return;
+	row.delivered = deps.deliver(row.reviewId, batch.destination);
+}
 function keepFreshPending(row, replaced) {
 	if (row.status === "failed" && !replaced) row.stale = true;
 }
@@ -13846,13 +13523,18 @@ async function startPullRequestBatch(body, deps) {
 	}
 	const model = modelOf(body?.model);
 	if (isAnswer(model)) return model;
+	const requested = destinationRequestOf(body?.destination);
+	if (isAnswer(requested)) return requested;
 	const agent = await agentFor(body?.agent, deps);
 	if (isAnswer(agent)) return agent;
+	const opened = deps.openDestination(requested);
+	if (!opened.ok) return destinationRefusal(opened, deps);
+	const destination = opened.destination;
 	const batch = newBatch(items.map((item) => ({
 		key: item.key,
 		label: `${item.owner}/${item.repo} #${item.number}`,
 		status: "waiting"
-	})));
+	})), destination);
 	batch.rows.forEach((row, index) => noteRow(row, {
 		repoPath: items[index].folder,
 		prNumber: items[index].number
@@ -13891,6 +13573,7 @@ async function startPullRequestBatch(body, deps) {
 			}
 			claimed.set(identity, index);
 			const prepared = await prepareRow(row, target, agent.agentId, model, deps, fresh);
+			deliverRow(row, batch, deps);
 			if (fresh) {
 				resolved[index] = target;
 				replaced = true;
@@ -13912,23 +13595,114 @@ async function startPullRequestBatch(body, deps) {
 		status: 200,
 		body: {
 			batchId: batch.id,
-			rows: rowsOf(batch)
+			rows: rowsOf(batch),
+			destination
 		}
 	};
 }
 /**
+* What the panel's Add to asked for. A collection it picked comes as an id and is matched as one,
+* never as a name; one it is to make comes as a name. Nothing, or the reviews list, is no
+* destination at all.
+*/
+function destinationRequestOf(raw) {
+	const choose = () => refuse(400, "Choose where the reviews go from Add to.", { field: "destination" });
+	if (raw === void 0 || raw === null) return { queue: false };
+	if (typeof raw !== "object") return choose();
+	const value = raw;
+	const queue = value.queue === true;
+	switch (value.kind) {
+		case "list": return { queue: false };
+		case "queue": return { queue: true };
+		case "collection": return typeof value.id === "string" && value.id !== "" ? {
+			collectionId: value.id,
+			queue
+		} : choose();
+		case "new-collection": return typeof value.name === "string" ? {
+			newCollection: value.name,
+			queue
+		} : choose();
+		default: return choose();
+	}
+}
+/** A destination that could not be opened, said beside Add to, with the collections as they are now. */
+function destinationRefusal(refusal, deps) {
+	const extra = {
+		field: "destination",
+		collections: deps.collections()
+	};
+	switch (refusal.reason) {
+		case "missing": return refuse(409, "That collection no longer exists. Choose again from Add to. Nothing was started.", extra);
+		case "empty":
+		case "too-long":
+		case "duplicate": return refuse(400, `${nameProblemText(refusal.reason)} Nothing was started.`, extra);
+		case "full": return refuse(409, `You have 100 collections, which is as many as there can be. Delete one, or add these to one you have. Nothing was started.`, extra);
+		default: return refuse(500, "The collection could not be saved. Nothing was started. Try again.", extra);
+	}
+}
+/**
+* The batch's destination as the panel shows it, under the collection's current name: a rename
+* keeps the id, so the batch still delivers to it, and the result should call it what it is now.
+*/
+function destinationNow(batch, deps) {
+	const collection = batch.destination.collection;
+	if (!collection || !deps) return batch.destination;
+	const current = deps.collections().find((one) => one.id === collection.id);
+	if (!current) return {
+		...batch.destination,
+		collection: {
+			...collection,
+			deleted: true
+		}
+	};
+	return current.name !== collection.name ? {
+		...batch.destination,
+		collection: {
+			...collection,
+			name: current.name
+		}
+	} : batch.destination;
+}
+/**
 * A batch's rows as they stand. Asking counts as watching: a batch an open panel is still asking
 * about is never forgotten, so its Retry stays good for as long as the panel is open.
 */
-function batchRows(id) {
+function batchRows(id, deps) {
+	const batch = batches.get(id);
+	if (!batch) return lostBatch();
+	batch.seenAt = Date.now();
+	return {
+		status: 200,
+		body: {
+			rows: rowsOf(batch),
+			finished: batch.finishedAt !== null,
+			destination: destinationNow(batch, deps)
+		}
+	};
+}
+/**
+* Put one row's review where the batch goes again, and nothing else: for a review that exists but
+* missed its collection or the queue (a full queue, a write that failed). The review is not run
+* again. A row with no review yet is Retry's business, which delivers once the review exists.
+*/
+function deliverBatchRow(id, key, deps) {
 	const batch = batches.get(id);
 	if (!batch) return lostBatch();
+	const row = batch.rows.find((one) => one.key === key);
+	if (!row) return refuse(404, "That review is not part of this batch.");
+	const again = missToRetry(row, batch);
+	if (!again || !row.reviewId) return refuse(409, "Only a review that missed somewhere it can still be added can be added again.");
+	row.delivered = {
+		...row.delivered,
+		...deps.deliver(row.reviewId, again)
+	};
 	batch.seenAt = Date.now();
 	return {
 		status: 200,
 		body: {
 			rows: rowsOf(batch),
-			finished: batch.finishedAt !== null
+			finished: batch.finishedAt !== null,
+			destination: destinationNow(batch, deps)
 		}
 	};
 }
@@ -17945,7 +17719,7 @@ async function handleAddReview(req, res, pathname, options) {
 			if (route === "options") return addReviewOptions(deps);
 			if (route === "folders") return listFolders(deps);
 			const batch = /^batches\/([0-9a-f-]{36})$/.exec(route);
-			if (batch) return batchRows(batch[1]);
+			if (batch) return batchRows(batch[1], deps);
 			return null;
 		}
 		if (req.method !== "POST") return {
@@ -17960,6 +17734,8 @@ async function handleAddReview(req, res, pathname, options) {
 		if (route === "branch") return startBranchReview(body, deps);
 		const retry = /^batches\/([0-9a-f-]{36})\/retry$/.exec(route);
 		if (retry) return retryBatchRow(retry[1], body?.key, body?.fresh);
+		const again = /^batches\/([0-9a-f-]{36})\/deliver$/.exec(route);
+		if (again) return deliverBatchRow(again[1], body?.key, deps);
 		return null;
 	};
 	const result = await answer();