prreviewbuddy 0.46.0 → 0.49.0

package/CHANGELOG.md

@@ -1,5 +1,92 @@
 # Changelog
 
+## 0.49.0
+
+**Change a review from the review itself.** The **⋯** menu at the top of a review now has
+everything the card's menu on the reviews list has: **Mark reviewed** or **Mark in progress**,
+**Mark waiting**, **Mark done**, **Move to active**, and **Add to review queue** or **Remove from
+review queue**, as well as Save, Copy review ID and Delete. You no longer have to go back to the
+list to park or queue the review you are reading. Waiting, Done and the queue belong to the branch,
+so they are offered on its newest review; an older review of the same branch points you to it.
+
+**The top bar says where the review stands.** It shows the files, the size of the change, whether
+the review is up to date, and two labels: **Reviewed** or **In progress**, and **Waiting** or
+**Done** when the review is parked. None of them are buttons. **Check for updates** has moved into
+the **⋯** menu, because the page already checks by itself every minute while you are looking at it
+and whenever you come back to the tab. Its answer, such as "No change", appears where the review's
+freshness is shown.
+
+**Check for updates works more than once.** After one check it stayed greyed out until you reloaded
+the page. It now becomes available again as soon as the check finishes, and pressing it while the
+page is checking by itself no longer does nothing: it waits for that check and then runs its own.
+
+**The review page keeps up while it is open.** If a Waiting review's branch gets new commits while
+you are looking at it, the page now says **Changed while waiting** within a minute, without a
+reload, and offers **Mark waiting** again. Changes made from the reviews list in another tab show
+up the same way. Adding a review to a full review queue from its page now says the queue is full
+instead of appearing to work.
+
+**Reviewed reviews stay in your queue.** Marking a review **Reviewed** no longer takes it out of
+the review queue, from the reviews list or from the review itself. Reviewed records that you have
+been through it, not that you are finished with it; **Waiting** and **Done** are how you say that,
+and they still take a review out of the queue. `prreviewbuddy review --queue` now queues a Reviewed
+review instead of refusing it, and, like adding from the workspace, moves a Waiting or Done review
+back to Active when it goes in.
+
+**Buddy AI has its own tab.** It is no longer a button in the top bar. When Buddy AI's panel is
+closed, a **Buddy AI** tab sits on the right edge of the window; press it to open the panel. The
+double arrow in the panel's header closes it.
+
+## 0.48.0
+
+**Active, Waiting and Done.** A review can now be marked **Waiting**, for when you have done your
+pass and expect another round, or **Done**, for when you are finished with it. The reviews list keeps
+your active reviews at the top, with **Waiting** below them and **Done** below that, folded away. Use
+the **⋯** menu on a card, or select several and use the bar at the bottom. **Move to active** puts a
+review back. This is separate from **Reviewed**, which still records that you have been through it.
+
+**Waiting reviews come back when something changes.** If a Waiting review gets new commits, or its
+pull request is merged, closed or reopened, it shows up in the main list again marked **Changed while
+waiting**. Marking it Waiting again puts it back. A Done review never comes back by itself; if it
+changes, it is marked **Changed since done** where it is. Marking a review Waiting starts watching
+it straight away, so a push made soon after still brings it back. Adding a Waiting or Done review to
+your review queue moves it back to Active.
+
+**The list no longer reorders when you open a review.** Reviews are ordered by what last happened to
+the change: when it was analysed, when new commits arrived, or when the pull request was merged,
+closed or reopened. The time on each card now says which of those it was, for example "Updated
+2 hours ago" or "Merged 1 day ago". Saved reviews still come first.
+
+**Load more instead of pages.** Each section shows fifteen reviews and a **Load more** button for the
+rest. Search and the filters still cover every review, in all three sections.
+
+**Done reviews are kept longer.** Like saved reviews, Done reviews are kept for ninety days rather
+than thirty, counting from the last time you opened or marked them.
+
+**Light mode.** The workspace now has a light theme as well as the dark one. The rail, the top bars,
+Buddy AI, the reviews list, Notifications and Settings are drawn in pale greys around a white
+review, so the review stays the brightest thing on the screen. Purple is kept for what you have
+chosen and for the main action, as a pale tint rather than a solid block.
+
+**Follows your device, or stays put.** Settings has a new **Appearance** section with **System**,
+**Light** and **Dark**. System is the default, and it changes with your device at once, without
+reloading the page, when your Mac switches between light and dark. Choosing a theme shows it on the
+Settings page straight away; **Save changes** keeps it. If you have not chosen, the workspace follows
+your device, so on a Mac set to light it now opens in light mode.
+
+**Readable in both.** Every piece of text in the light theme, including timestamps, counts and other
+small details, meets the WCAG 4.5:1 contrast level against whatever it sits on.
+
+**Fixed: buttons in a review that went blank on hover.** In the dark theme, pointing at **Open in**
+your editor, **Show more of this file**, **View files**, **Regenerate** or a diagram's zoom buttons
+turned the label white on the white review, so it disappeared. They now keep their dark label.
+
+**A clearer message when OpenAI refuses Codex while it is signed in.** If Codex says it is signed in
+but OpenAI answers "Incorrect API key", the review no longer suggests signing in again, and no longer
+suggests looking for an API key of your own: the key in that error is Codex's own. It now suggests
+`codex doctor` to see what Codex is using, and `codex features list` to find an experimental Codex
+feature to turn off, which is what fixed it when we saw it.
+
 ## 0.46.0
 
 **The change map: see what a change is about, and how its parts affect each other.** The Diagrams

package/dist/{delete_review-lmVdoybL.js → delete_review-BBjXPfMV.js}

@@ -1129,6 +1129,19 @@ var CodexUnavailableError = class extends AgentUnavailableError {};
 */
 var AUTHENTICATION = /not (?:logged|signed) in|unauthori[sz]ed|401|authenticat|credential|sign[- ]?in|log ?in/i;
 /**
+* OpenAI refusing a request Codex believes is signed in, which is not nobody being signed in.
+*
+* Seen with Codex reporting "Logged in using ChatGPT" and no API key anywhere in the reviewer's
+* environment: the key named in the error is Codex's own, not one of theirs. Signing in again did
+* not clear it; turning off an experimental Codex feature did. So the advice is Codex's own
+* diagnosis and its feature switches, never a login loop and never a hunt for an API key. Still an
+* authentication failure, so no Retry is offered.
+*/
+var REJECTED_KEY = /incorrect api key|invalid[_ ]api[_ ]key/i;
+function isAuthentication(detail) {
+	return AUTHENTICATION.test(detail) || REJECTED_KEY.test(detail);
+}
+/**
 * Which kind of failure the agent's own sentence describes.
 *
 * Only the one distinction that changes what a person is offered: a run that failed because nobody
@@ -1137,7 +1150,7 @@ var AUTHENTICATION = /not (?:logged|signed) in|unauthori[sz]ed|401|authenticat|c
 * label for them.
 */
 function kindOf(detail) {
-	return AUTHENTICATION.test(detail) ? "agent_authentication_failed" : "agent_execution_failed";
+	return isAuthentication(detail) ? "agent_authentication_failed" : "agent_execution_failed";
 }
 var codexAgent = {
 	id: CODEX_AGENT_ID,
@@ -1147,8 +1160,10 @@ var codexAgent = {
 	captureEnv: captureEnv$1,
 	/** Which tool, which profile, and the one command that changes it. See `agent.ts` for the budget. */
 	explainFailure(detail, env = {}) {
-		if (!AUTHENTICATION.test(detail)) return null;
-		return `Codex could not sign in. This review was started using ${env.CODEX_HOME ? `the Codex profile at \`${env.CODEX_HOME}\`` : "your default Codex profile"}, so run \`codex login\` against that one.`;
+		if (!isAuthentication(detail)) return null;
+		const profile = env.CODEX_HOME ? `the Codex profile at \`${env.CODEX_HOME}\`` : "your default Codex profile";
+		if (REJECTED_KEY.test(detail)) return `OpenAI refused a request from ${profile} even though Codex reports being signed in, which usually points to Codex's own setup rather than your sign-in. Run \`codex doctor\` to see what it is using, and check \`codex features list\` for an experimental feature to turn off with \`codex features disable <name>\`.`;
+		return `Codex could not sign in. This review was started using ${profile}, so run \`codex login\` against that one.`;
 	},
 	/**
 	* Whether the command is here and runs. Deliberately not whether it is signed in.
@@ -4074,19 +4089,6 @@ function queueRemove(id) {
 	return updateQueue((current) => current.includes(id) ? current.filter((queued) => queued !== id) : null);
 }
 /**
-* The invariant, as one named thing: the queue holds unfinished work only.
-*
-* Removing on completion is half of it. The other half is that this **does nothing** when
-* `complete` is false, and that half only exists as a written rule -- there is no line of code for
-* it to be visible in. Re-adding there would make membership something inferred from status rather
-* than chosen, so a reviewer who took a review out of the queue and later moved it back to In
-* progress would find it queued again with no idea why.
-*/
-function dequeueOnComplete(id, complete) {
-	if (!complete) return;
-	queueRemove(id);
-}
-/**
 * A token for the index's revision, equality only.
 *
 * The queue changes only when a person deliberately changes it, so unlike a running job's progress
@@ -4112,6 +4114,115 @@ function normalise(ids) {
 	return seen;
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/lifecycle.ts
+/**
+* What happens next with a change, as the reviewer says it: still in play, parked until something
+* moves, or finished with.
+*
+* A separate axis from New / In progress / Reviewed, which says what the reviewer has *done*. A
+* change can be reviewed and waiting for another push, or reviewed and done, and one field cannot
+* hold both answers.
+*
+* One registry that the stored type, the card, the menus, the bulk bar and the route all derive
+* from. Pure on purpose: the reviews page's script imports it to move a card between sections after
+* a mark, without asking the server for the page again.
+*/
+var LIFECYCLES = [
+	{
+		id: "active",
+		name: "Active",
+		action: "Move to active"
+	},
+	{
+		id: "waiting",
+		name: "Waiting",
+		action: "Mark waiting"
+	},
+	{
+		id: "done",
+		name: "Done",
+		action: "Mark done"
+	}
+];
+function isLifecycle(value) {
+	return LIFECYCLES.some((lifecycle) => lifecycle.id === value);
+}
+/** A stored value read defensively: anything this code did not write reads as Active. */
+function parseLifecycle(raw) {
+	if (typeof raw !== "object" || raw === null) return void 0;
+	const { state, at } = raw;
+	if (state !== "waiting" && state !== "done" || typeof at !== "number" || !Number.isFinite(at)) return void 0;
+	return {
+		state,
+		at
+	};
+}
+function lifecycleAction(lifecycle) {
+	return LIFECYCLES.find((entry) => entry.id === lifecycle).action;
+}
+/**
+* The review page's lifecycle pill, or absent for Active, which says nothing (as an open pull
+* request says nothing). A parked review that has changed since reads as the card's chip does, so
+* the page and the list give one answer about the same review.
+*/
+function lifecyclePill(lifecycle, activity) {
+	if (!lifecycle) return void 0;
+	const chip = lifecycleChip(lifecycle, activity);
+	if (chip !== void 0) return {
+		text: chip,
+		changed: true
+	};
+	return {
+		text: LIFECYCLES.find((entry) => entry.id === lifecycle.state).name,
+		changed: false
+	};
+}
+var ACTIVITY_LABELS = {
+	created: "Created",
+	updated: "Updated",
+	merged: "Merged",
+	closed: "Closed",
+	reopened: "Reopened"
+};
+function activityLabel(kind) {
+	return ACTIVITY_LABELS[kind];
+}
+/**
+* Which section a card is drawn in.
+*
+* Waiting means "bring this back when something changes", so a Waiting review whose change moved
+* after it was parked is drawn in Active. Derived, and nothing is written: the stored state stays
+* Waiting until the reviewer says otherwise, and marking it Waiting again moves the baseline.
+*
+* Done means "leave me alone", so it never comes back by itself. `created` can never resurface
+* anything either, because a new analysis is a new record, and a new record is Active.
+*/
+function sectionOf(lifecycle, activity) {
+	if (!lifecycle) return "active";
+	if (lifecycle.state === "waiting" && changedSince(lifecycle, activity)) return "active";
+	return lifecycle.state;
+}
+/** The chip that says why a card is where it is, or absent when its section already says it. */
+function lifecycleChip(lifecycle, activity) {
+	if (!lifecycle || !changedSince(lifecycle, activity)) return void 0;
+	return lifecycle.state === "waiting" ? "Changed while waiting" : "Changed since done";
+}
+function changedSince(lifecycle, activity) {
+	return activity !== void 0 && activity.kind !== "created" && activity.at > lifecycle.at;
+}
+/**
+* Which lifecycle items a card's menu offers.
+*
+* Every state but the one the card is already in, which would be an item that does nothing. A card
+* carrying a chip is the exception and offers all three: re-marking it is how the reviewer says
+* they have seen the change, so "Mark waiting" on a Waiting review that changed is the settle
+* action, not a no-op.
+*/
+function offersLifecycle(target, lifecycle, activity) {
+	if (lifecycleChip(lifecycle, activity) !== void 0) return true;
+	return target !== (lifecycle?.state ?? "active");
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/store.ts
 var WORKSPACES_DIR = join(STORE_ROOT, "workspaces");
 /**
@@ -4481,8 +4592,20 @@ function followedRefName(branch, followedRef) {
 * can return is the two added together. Neither constant says that on its own, which is why it is
 * said here.
 */
-function indexReviewGroups(limit, keptLimit, keptIds) {
-	return selectIndexGroups(groupByLineage(recentWorkspaces(Number.MAX_SAFE_INTEGER)), limit, keptLimit, keptIds);
+function indexReviewGroups(limit, keptLimit, keptIds, rank, groups = groupByLineage(recentWorkspaces(Number.MAX_SAFE_INTEGER))) {
+	return selectIndexGroups(groups, limit, keptLimit, keptIds, rank);
+}
+/**
+* What the reviewer explicitly kept, which is what stops the recency window hiding it: the queue,
+* Saved, and anything marked Waiting or Done.
+*
+* Done is the reviewer's archive, and an archive the recency window can empty is not one. Waiting
+* is work they expect back, and a push to it must have a card to come back on.
+*/
+function indexKeptIds(reviews, queue) {
+	const kept = new Set(queue);
+	for (const review of reviews) if (review.pinned || review.lifecycle !== void 0) kept.add(review.id);
+	return kept;
 }
 /**
 * The choosing, with the disk left out, in the same spirit as `groupByLineage` next door.
@@ -4494,12 +4617,18 @@ function indexReviewGroups(limit, keptLimit, keptIds) {
 *
 * The kept side is capped for the same reason the recent side is: `queue.json` is a file on a disk
 * somebody can edit, and an uncapped read of it is an index with five thousand cards on it.
+*
+* **Windowed by `rank`, which is what the list is ordered by.** The index passes what last happened
+* to each change, so a branch analysed two months ago and pushed to this morning is inside the
+* window rather than cut before its activity is ever looked at. Absent, it is when the newest
+* review was made, which is what every other caller has always meant by recent.
 */
-function selectIndexGroups(groups, limit, keptLimit, keptIds) {
+function selectIndexGroups(groups, limit, keptLimit, keptIds, rank = (group) => group.latest.createdAt) {
 	const kept = [];
 	const rest = [];
 	for (const group of groups) (group.history.some((review) => keptIds.has(review.id)) ? kept : rest).push(group);
-	return [...rest.slice(0, limit), ...kept.slice(0, keptLimit)].sort((a, b) => b.latest.createdAt - a.latest.createdAt);
+	const byRank = (a, b) => rank(b) - rank(a);
+	return [...rest.sort(byRank).slice(0, limit), ...kept.sort(byRank).slice(0, keptLimit)].sort(byRank);
 }
 /**
 * The grouping itself, with the disk left out.
@@ -4536,9 +4665,8 @@ function groupByLineage(reviews) {
 * on an old review is "then what is the current one", and answering it by sending them to the
 * index costs them the page they were reading. See `RelatedReview` for what may travel and why.
 */
-function lineagePosition(workspace) {
+function lineagePosition(workspace, all = recentWorkspaces(Number.MAX_SAFE_INTEGER)) {
 	const review = summarise(workspace);
-	const all = recentWorkspaces(Number.MAX_SAFE_INTEGER);
 	return {
 		...positionInLineage(review, all),
 		reviews: relatedReviews(review, all)
@@ -4626,6 +4754,7 @@ function summarise(workspace) {
 		lastViewedAt: workspace.lastViewedAt,
 		completedAt: workspace.completedAt,
 		pinned: workspace.pinned === true,
+		lifecycle: parseLifecycle(workspace.lifecycle),
 		paths: files.slice(0, MAX_INDEXED_PATHS).map((file) => file.name),
 		prNumber: workspace.changeSet.data.pullRequest?.number || workspace.prContext?.pullRequest.number,
 		commitSha: workspace.session.metadata?.commitSha || (workspace.changeSet.targetType === "commit" ? workspace.changeSet.data.pullRequest.head.sha : "") || void 0,
@@ -4803,7 +4932,8 @@ function isKept(entry, queued) {
 	const id = workspaceIdFrom(entry);
 	if (id === null) return false;
 	if (queued.has(id)) return true;
-	return loadWorkspace(id)?.pinned === true;
+	const workspace = loadWorkspace(id);
+	return workspace?.pinned === true || parseLifecycle(workspace?.lifecycle)?.state === "done";
 }
 /** The review an entry belongs to, whether it is the record or the usage sidecar beside it. */
 function workspaceIdFrom(entry) {
@@ -7526,9 +7656,10 @@ function compactQueue(stored) {
 /**
 * Put one review on the queue, the way the index's Add does, and say how that went.
 *
-* **A completed review is left out**, not queued and not reopened. The queue holds unfinished work
-* only -- `dequeueOnComplete` is the other half of that rule -- and quietly moving somebody's review
-* back to In progress because a command mentioned it is a status change nobody asked for.
+* **A Reviewed review is queued like any other.** Reviewed says what the reviewer has done with it,
+* not that the work is over; that is Done. A Waiting or Done review that goes in is put back in
+* Active, as it is from the page (`activateQueued`), because asking for it in the queue is asking
+* to work on it again.
 *
 * Already queued is `already`, not a move: the same rule `queueAdd` keeps, for the same reason.
 *
@@ -7536,11 +7667,9 @@ function compactQueue(stored) {
 * was queued would otherwise hold a place, and a live review could be refused as "full" by it.
 */
 function queueForReview(id) {
-	const workspace = loadWorkspace(id);
-	if (!workspace) return "gone";
-	if (workspace.completedAt !== void 0) return "complete";
+	if (!loadWorkspace(id)) return "gone";
 	let result = "added";
-	return updateQueue((current) => {
+	const stored = updateQueue((current) => {
 		const live = current.filter(isLive);
 		const unchanged = live.length === current.length ? null : live;
 		if (live.includes(id)) {
@@ -7552,7 +7681,29 @@ function queueForReview(id) {
 			return unchanged;
 		}
 		return [...live, id];
-	}) === null ? "unsaved" : result;
+	});
+	if (stored === null) return "unsaved";
+	if (result === "added") activateQueued([id], stored);
+	return result;
+}
+/**
+* Queueing a review puts it back in Active.
+*
+* The queue is work the reviewer wants surfaced next, and Waiting and Done both say the opposite; a
+* queued review folded away in Done would be a contradiction the page then has to draw. Parking
+* takes a review out of the queue, and this is the same rule read the other way. Only what actually
+* went in: a review refused because the queue was full keeps the lifecycle it had.
+*
+* Every way in calls this: a card, the bulk bar, the review's own page and `--queue`.
+*/
+function activateQueued(ids, queue) {
+	const queued = new Set(queue);
+	for (const one of ids) {
+		if (!queued.has(one) || loadWorkspace(one)?.lifecycle === void 0) continue;
+		mutateWorkspace(one, (current) => {
+			current.lifecycle = void 0;
+		});
+	}
 }
 function isLive(id) {
 	return loadWorkspace(id) !== null;
@@ -12651,4 +12802,4 @@ function lineageIds(id) {
 	return [.../* @__PURE__ */ new Set([id, ...lineage.map((review) => review.id)])];
 }
 //#endregion
-export { queueForReview as $, positionInLineage as $t, createPrState as A, writeAgentPreference as An, allJobs as At, toneOf as B, purposeOf as Bt, readEvents as C, displayRef as Cn, forgeFor as Ct, announceJob as D, resolveModel as Dn, checkReview as Dt, telemetryUploadConsent as E, resolveAgent as En, checkCodeFreshness as Et, markAllRead as F, agentById as Fn, saveJob as Ft, extractJsonText as G, groupByLineage as Gt, withUsageRecorded as H, driverGone as Ht, markRead as I, agentFor as In, PHASES as It, readMarker as J, lineageKeyFor as Jt, ReviewBeingDeletedError as K, indexReviewGroups as Kt, pruneNotifications as L, detectAgents as Ln, isLiveJob as Lt, eventName as M, STORE_ROOT as Mn, fail as Mt, headline as N, AGENT_IDS as Nn, isTerminal as Nt, branchOf as O, clearAgentPreference as On, followedBranch as Ot, listNotifications as P, DEFAULT_AGENT_ID as Pn, loadJob as Pt, compactQueue as Q, mutateWorkspace as Qt, readNotification as R, AgentCancelledError as Rn, phasesFor as Rt, startJob as S, resolveTarget as Sn, forgeResolver as St, recordTelemetryUploadConsent as T, git as Tn, fillFileUrlTemplate as Tt, EXPLAIN_SIMPLY_PROMPT as U, isClaimed as Ut, unreadCount as V, clearClaim as Vt, asString as W, followedRefName as Wt, modelHelpLines as X, loadWorkspace as Xt, removeWorktree as Y, lineagePosition as Yt, relativeTime as Z, matchingWorkspaceIds as Zt, agentEnvOf as _, issuesOutstanding as _n, parseWorkspacePort as _t, wasBlocked as a, touchWorkspace as an, reviewsUrl as at, reanalyseReview as b, processDiscussion as bn, writePortPreference as bt, RefreshUnavailableError as c, dequeueOnComplete as cn, stopServer as ct, refreshPrContext as d, queueRemove as dn, BUILD_VERSION as dt, previousKissResult as en, readIndexToken as et, isUnchanged as f, queueStamp as fn, PACKAGE_NAME as ft, runningJobs as g, isQuestionOutstanding as gn, MIN_WORKSPACE_PORT as gt, liveUpdateFor as h, isIssueOutstanding as hn, MAX_WORKSPACE_PORT as ht, liveJobsFor as i, summarise as in, readServerRecord as it, createPrUpdate as j, CONFIG_PATH as jn, endedAt as jt, subjectOf as k, readAgentPreference as kn, reviewedShaOf as kt, mergeUpdateInto as l, queueAdd as ln, workspaceUrl as lt, UpdateAlreadyRunningError as m, doneVerb as mn, DEFAULT_WORKSPACE_PORT as mt, deleteReview as n, reviewedCommit as nn, clearServerRecord as nt, discardJob as o, workspaceRevision as on, serverEntryExists as ot, askCheckout as p, readQueue as pn, feedbackUrl as pt, MANAGED_ROOT as q, latestKissRun as qt, lineageIds as r, reviewedRepositories as rn, ensureServer as rt, FETCH_TIMEOUT_MS as s, workspaceStamps as sn, spawnServer as st, deleteLineage as t, recentWorkspaces as tn, bootstrapUrl as tt, updateReview as u, queueAddMany as un, writeServerRecord as ut, recordKissRun as v, questionsOutstanding as vn, resolveWorkspacePort as vt, record as w, isCommitReview as wn, remoteHost as wt, runJob as x, describeAuthorship as xn, claimRequestUrl as xt, startKissJob as y, reviewerDispositions as yn, statedWorkspacePort as yt, removeNotification as z, AgentUnavailableError as zn, progressSteps as zt };
+export { compactQueue as $, matchingWorkspaceIds as $t, createPrState as A, processDiscussion as An, reviewedShaOf as At, toneOf as B, writeAgentPreference as Bn, progressSteps as Bt, readEvents as C, readQueue as Cn, forgeResolver as Ct, announceJob as D, issuesOutstanding as Dn, checkCodeFreshness as Dt, telemetryUploadConsent as E, isQuestionOutstanding as En, fillFileUrlTemplate as Et, markAllRead as F, git as Fn, loadJob as Ft, extractJsonText as G, agentById as Gn, followedRefName as Gt, withUsageRecorded as H, STORE_ROOT as Hn, clearClaim as Ht, markRead as I, resolveAgent as In, saveJob as It, readMarker as J, AgentCancelledError as Jn, indexReviewGroups as Jt, ReviewBeingDeletedError as K, agentFor as Kn, groupByLineage as Kt, pruneNotifications as L, resolveModel as Ln, PHASES as Lt, eventName as M, resolveTarget as Mn, endedAt as Mt, headline as N, displayRef as Nn, fail as Nt, branchOf as O, questionsOutstanding as On, checkReview as Ot, listNotifications as P, isCommitReview as Pn, isTerminal as Pt, activateQueued as Q, loadWorkspace as Qt, readNotification as R, clearAgentPreference as Rn, isLiveJob as Rt, startJob as S, queueStamp as Sn, claimRequestUrl as St, recordTelemetryUploadConsent as T, isIssueOutstanding as Tn, remoteHost as Tt, EXPLAIN_SIMPLY_PROMPT as U, AGENT_IDS as Un, driverGone as Ut, unreadCount as V, CONFIG_PATH as Vn, purposeOf as Vt, asString as W, DEFAULT_AGENT_ID as Wn, isClaimed as Wt, modelHelpLines as X, lineageKeyFor as Xt, removeWorktree as Y, AgentUnavailableError as Yn, latestKissRun as Yt, relativeTime as Z, lineagePosition as Zt, agentEnvOf as _, parseLifecycle as _n, MIN_WORKSPACE_PORT as _t, wasBlocked as a, reviewedRepositories as an, readServerRecord as at, reanalyseReview as b, queueAddMany as bn, statedWorkspacePort as bt, RefreshUnavailableError as c, workspaceRevision as cn, spawnServer as ct, refreshPrContext as d, activityLabel as dn, writeServerRecord as dt, mutateWorkspace as en, queueForReview as et, isUnchanged as f, isLifecycle as fn, BUILD_VERSION as ft, runningJobs as g, offersLifecycle as gn, MAX_WORKSPACE_PORT as gt, liveUpdateFor as h, lifecyclePill as hn, DEFAULT_WORKSPACE_PORT as ht, liveJobsFor as i, reviewedCommit as in, ensureServer as it, createPrUpdate as j, describeAuthorship as jn, allJobs as jt, subjectOf as k, reviewerDispositions as kn, followedBranch as kt, mergeUpdateInto as l, workspaceStamps as ln, stopServer as lt, UpdateAlreadyRunningError as m, lifecycleChip as mn, feedbackUrl as mt, deleteReview as n, previousKissResult as nn, bootstrapUrl as nt, discardJob as o, summarise as on, reviewsUrl as ot, askCheckout as p, lifecycleAction as pn, PACKAGE_NAME as pt, MANAGED_ROOT as q, detectAgents as qn, indexKeptIds as qt, lineageIds as r, recentWorkspaces as rn, clearServerRecord as rt, FETCH_TIMEOUT_MS as s, touchWorkspace as sn, serverEntryExists as st, deleteLineage as t, positionInLineage as tn, readIndexToken as tt, updateReview as u, LIFECYCLES as un, workspaceUrl as ut, recordKissRun as v, sectionOf as vn, parseWorkspacePort as vt, record as w, doneVerb as wn, forgeFor as wt, runJob as x, queueRemove as xn, writePortPreference as xt, startKissJob as y, queueAdd as yn, resolveWorkspacePort as yt, removeNotification as z, readAgentPreference as zn, phasesFor as zt };

package/dist/main.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { $ as queueForReview, $t as positionInLineage, An as writeAgentPreference, At as allJobs, Bt as purposeOf, Cn as displayRef, Dn as resolveModel, Dt as checkReview, E as telemetryUploadConsent, En as resolveAgent, Fn as agentById, Gt as groupByLineage, Ht as driverGone, It as PHASES, J as readMarker, Jt as lineageKeyFor, Ln as detectAgents, Mn as STORE_ROOT, Nn as AGENT_IDS, Nt as isTerminal, Pn as DEFAULT_AGENT_ID, Pt as loadJob, Qt as mutateWorkspace, S as startJob, Sn as resolveTarget, St as forgeResolver, T as recordTelemetryUploadConsent, Tn as git, Ut as isClaimed, Vt as clearClaim, Wt as followedRefName, X as modelHelpLines, Xt as loadWorkspace, Y as removeWorktree, Z as relativeTime, Zt as matchingWorkspaceIds, _t as parseWorkspacePort, a as wasBlocked, at as reviewsUrl, b as reanalyseReview, bt as writePortPreference, ct as stopServer, dt as BUILD_VERSION, et as readIndexToken, ft as PACKAGE_NAME, g as runningJobs, gt as MIN_WORKSPACE_PORT, h as liveUpdateFor, ht as MAX_WORKSPACE_PORT, i as liveJobsFor, in as summarise, jn as CONFIG_PATH, kn as readAgentPreference, l as mergeUpdateInto, lt as workspaceUrl, m as UpdateAlreadyRunningError, mt as DEFAULT_WORKSPACE_PORT, n as deleteReview$1, q as MANAGED_ROOT, rn as reviewedRepositories, rt as ensureServer, tn as recentWorkspaces, tt as bootstrapUrl, u as updateReview, v as recordKissRun, w as record$1, x as runJob, yt as statedWorkspacePort, zt as progressSteps } from "./delete_review-lmVdoybL.js";
+import { $t as matchingWorkspaceIds, Bn as writeAgentPreference, Bt as progressSteps, Ct as forgeResolver, E as telemetryUploadConsent, Fn as git, Ft as loadJob, Gn as agentById, Gt as followedRefName, Hn as STORE_ROOT, Ht as clearClaim, In as resolveAgent, J as readMarker, Kt as groupByLineage, Ln as resolveModel, Lt as PHASES, Mn as resolveTarget, Nn as displayRef, Ot as checkReview, Pt as isTerminal, Qt as loadWorkspace, S as startJob, T as recordTelemetryUploadConsent, Un as AGENT_IDS, Ut as driverGone, Vn as CONFIG_PATH, Vt as purposeOf, Wn as DEFAULT_AGENT_ID, Wt as isClaimed, X as modelHelpLines, Xt as lineageKeyFor, Y as removeWorktree, Z as relativeTime, _t as MIN_WORKSPACE_PORT, a as wasBlocked, an as reviewedRepositories, b as reanalyseReview, bt as statedWorkspacePort, en as mutateWorkspace, et as queueForReview, ft as BUILD_VERSION, g as runningJobs, gt as MAX_WORKSPACE_PORT, h as liveUpdateFor, ht as DEFAULT_WORKSPACE_PORT, i as liveJobsFor, it as ensureServer, jt as allJobs, l as mergeUpdateInto, lt as stopServer, m as UpdateAlreadyRunningError, n as deleteReview$1, nt as bootstrapUrl, on as summarise, ot as reviewsUrl, pt as PACKAGE_NAME, q as MANAGED_ROOT, qn as detectAgents, rn as recentWorkspaces, tn as positionInLineage, tt as readIndexToken, u as updateReview, ut as workspaceUrl, v as recordKissRun, vt as parseWorkspacePort, w as record$1, x as runJob, xt as writePortPreference, zn as readAgentPreference } from "./delete_review-BBjXPfMV.js";
 import { basename, dirname, join, resolve } from "node:path";
 import { spawn } from "node:child_process";
 import { existsSync, mkdirSync, readdirSync, realpathSync, rmSync, statSync, writeFileSync } from "node:fs";
@@ -1455,7 +1455,6 @@ function batchQueueLines(rows) {
 }
 function notQueuedBecause(subject, result) {
 	switch (result) {
-		case "complete": return `${subject} is marked complete. Move it back to In progress in the workspace to queue it.`;
 		case "full": return `${subject}, because the review queue is full. It holds 50 reviews at most.`;
 		default: return `${subject}, because the review queue could not be saved.`;
 	}
@@ -1468,7 +1467,6 @@ function reviewQueued(result) {
 	switch (result) {
 		case "added": return "Added to review queue.";
 		case "already": return "Already in your review queue.";
-		case "complete": return "Not added to your review queue: this review is marked complete. Move it back to In progress in the workspace to queue it.";
 		case "full": return `Not added to your review queue: it is full, and holds 50 reviews at most.`;
 		default: return "Not added to your review queue: the queue could not be saved.";
 	}