prreviewbuddy 0.42.0 → 0.44.0

package/dist/main.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { $ as ensureServer, $t as reviewedRepositories, An as AGENT_IDS, At as loadJob, Bt as followedRefName, Cn as resolveAgent, Ct as checkReview, Dn as writeAgentPreference, E as telemetryUploadConsent, En as readAgentPreference, Et as allJobs, Ft as progressSteps, G as MANAGED_ROOT, It as purposeOf, J as modelHelpLines, Jt as mutateWorkspace, K as readMarker, Kt as loadWorkspace, Lt as clearClaim, Mn as agentById, Mt as PHASES, On as CONFIG_PATH, Pn as detectAgents, Rt as driverGone, S as startJob, Sn as git, T as recordTelemetryUploadConsent, Vt as groupByLineage, Wt as lineageKeyFor, X as readIndexToken, Y as relativeTime, Yt as positionInLineage, Z as bootstrapUrl, Zt as recentWorkspaces, a as wasBlocked, at as workspaceUrl, b as reanalyseReview, bn as displayRef, ct as PACKAGE_NAME, dt as MAX_WORKSPACE_PORT, en as summarise, ft as MIN_WORKSPACE_PORT, g as runningJobs, gt as writePortPreference, h as liveUpdateFor, ht as statedWorkspacePort, i as liveJobsFor, it as stopServer, jn as DEFAULT_AGENT_ID, kn as STORE_ROOT, kt as isTerminal, l as mergeUpdateInto, m as UpdateAlreadyRunningError, n as deleteReview$1, pt as parseWorkspacePort, q as removeWorktree, qt as matchingWorkspaceIds, st as BUILD_VERSION, tt as reviewsUrl, u as updateReview, ut as DEFAULT_WORKSPACE_PORT, v as recordKissRun, vt as forgeResolver, w as record$1, wn as resolveModel, x as runJob, yn as resolveTarget, zt as isClaimed } from "./delete_review-DPRL5IPt.js";
+import { $ as bootstrapUrl, $t as recentWorkspaces, An as STORE_ROOT, Bt as driverGone, Cn as git, Dn as readAgentPreference, E as telemetryUploadConsent, Fn as detectAgents, G as MANAGED_ROOT, Ht as followedRefName, J as modelHelpLines, Jt as loadWorkspace, K as readMarker, Kt as lineageKeyFor, Lt as progressSteps, Mn as DEFAULT_AGENT_ID, Mt as loadJob, Nn as agentById, On as writeAgentPreference, Ot as allJobs, Pt as PHASES, Q as readIndexToken, Rt as purposeOf, S as startJob, T as recordTelemetryUploadConsent, Tn as resolveModel, Tt as checkReview, Ut as groupByLineage, Vt as isClaimed, Xt as mutateWorkspace, Y as relativeTime, Yt as matchingWorkspaceIds, Z as queueForReview, Zt as positionInLineage, _t as statedWorkspacePort, a as wasBlocked, b as reanalyseReview, bn as resolveTarget, bt as forgeResolver, ft as DEFAULT_WORKSPACE_PORT, g as runningJobs, h as liveUpdateFor, ht as parseWorkspacePort, i as liveJobsFor, jn as AGENT_IDS, jt as isTerminal, kn as CONFIG_PATH, l as mergeUpdateInto, lt as BUILD_VERSION, m as UpdateAlreadyRunningError, mt as MIN_WORKSPACE_PORT, n as deleteReview$1, nn as summarise, ot as stopServer, pt as MAX_WORKSPACE_PORT, q as removeWorktree, rt as reviewsUrl, st as workspaceUrl, tn as reviewedRepositories, tt as ensureServer, u as updateReview, ut as PACKAGE_NAME, v as recordKissRun, vt as writePortPreference, w as record$1, wn as resolveAgent, x as runJob, xn as displayRef, zt as clearClaim } from "./delete_review-BWMmO81Q.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";
@@ -33,7 +33,8 @@ var BARE = {
 		"--update",
 		"--reanalyse",
 		"--delete",
-		"--yes"
+		"--yes",
+		"--queue"
 	],
 	open: [],
 	agents: [],
@@ -117,16 +118,21 @@ function parse(argv) {
 	const values = {};
 	const flags = /* @__PURE__ */ new Set();
 	const positional = [];
+	/** The flag each positional came after, as typed, so a stray URL can be told where it went wrong. */
+	const after = /* @__PURE__ */ new Map();
+	let lastFlag;
 	/** Every URL `--pr` took. `review` has no positionals, so what follows `--pr` up to the next flag is its. */
 	const prs = [];
 	for (let index = 0; index < rest.length; index += 1) {
 		const argument = rest[index];
 		if (!argument.startsWith("--")) {
 			positional.push(argument);
+			if (lastFlag && !after.has(argument)) after.set(argument, lastFlag);
 			continue;
 		}
 		const equals = argument.indexOf("=");
 		const name = equals === -1 ? argument : argument.slice(0, equals);
+		lastFlag = name;
 		if (VALUED[head]?.includes(name)) {
 			const value = equals === -1 ? rest[index + 1] : argument.slice(equals + 1);
 			if (!value) return {
@@ -153,10 +159,18 @@ function parse(argv) {
 			message: `\`${head}\` does not take ${name}. Run \`prreviewbuddy --help\` to see what it takes.`
 		};
 	}
-	if (head === "review" && positional[0]) return {
-		name: "error",
-		message: strayArgument(positional[0])
-	};
+	if (head === "review" && positional[0]) {
+		const stray = positional[0];
+		const flag = after.get(stray);
+		if (prs.length > 0 && flag && LOOKS_LIKE_A_REQUEST.test(stray)) return {
+			name: "error",
+			message: `\`${stray}\` comes after \`${flag}\`, so \`--pr\` did not take it: \`--pr\` takes the URLs straight after it, up to the next flag. Put all the URLs together after \`--pr\`, and the other flags before or after them.`
+		};
+		return {
+			name: "error",
+			message: strayArgument(stray)
+		};
+	}
 	if (head === "uninstall" && positional[0]) return {
 		name: "error",
 		message: `\`uninstall\` removes everything on this machine, so it takes no target and \`${positional[0]}\` means nothing to it. It takes \`--keep-reviews\`, \`--yes\` and \`--force\`.`
@@ -184,7 +198,8 @@ function parse(argv) {
 			...values["--concurrency"] ? { concurrency: Number(values["--concurrency"]) } : {},
 			...values["--agent"] ? { agentId: values["--agent"] } : {},
 			...values["--model"] ? { requestedModel: values["--model"] } : {},
-			...flags.has("--fresh") ? { operation: "fresh" } : {}
+			...flags.has("--fresh") ? { operation: "fresh" } : {},
+			...flags.has("--queue") ? { queue: true } : {}
 		};
 		const target = Object.keys(TARGETS).find((flag) => values[flag]);
 		const operation = Object.keys(OPERATIONS).find((flag) => flags.has(flag));
@@ -197,7 +212,8 @@ function parse(argv) {
 			...operation ? { operation: OPERATIONS[operation][0] } : {},
 			...values["--lens"] ? { lens: values["--lens"] } : {},
 			...values["--audience"] ? { audience: values["--audience"] } : {},
-			...flags.has("--yes") ? { yes: true } : {}
+			...flags.has("--yes") ? { yes: true } : {},
+			...flags.has("--queue") ? { queue: true } : {}
 		};
 	}
 	if (head === "uninstall") return {
@@ -313,6 +329,7 @@ function refuseCombination(values, flags) {
 		}
 		if (values["--agent"]) return "`--agent` chooses who analyses a change, and `--delete` analyses nothing. Drop it, or drop `--delete` if you meant to review something.";
 		if (values["--model"]) return "`--model` chooses which model analyses a change, and `--delete` analyses nothing. Drop it, or drop `--delete` if you meant to review something.";
+		if (flags.has("--queue")) return "`--queue` puts a review on your review queue to work through, and `--delete` removes the review, so there would be nothing left to queue. Ask for one of them.";
 	}
 	if (flags.has("--yes") && !flags.has("--delete")) return "`--yes` answers the confirmation that `--delete` asks, and nothing here is asking one. Add `--delete` if you meant to remove a review.";
 	if (values["--review"]) {
@@ -980,6 +997,8 @@ your own checkout is never read or written while it works.
   prreviewbuddy review --pr <url> <url> ...
                                           review several pull requests, each on its own, two at a
                                           time; --concurrency <n> changes how many
+  prreviewbuddy review ... --queue        also put each review on your review queue, as soon as
+                                          it exists rather than when it is ready
   prreviewbuddy review --base <ref>       compare against a base of your choosing
   prreviewbuddy review --agent <agent>    review with ${AGENT_NAMES}
   prreviewbuddy review --model <model>    ask that agent for a particular model. Reviews keep
@@ -1408,11 +1427,52 @@ function batchFinished(rows, index) {
 		lines.push(`${lead}${row.detail ?? row.url ?? ""}`.trimEnd());
 		if (row.detail && row.url) lines.push(`${" ".repeat(lead.length)}${row.url}`);
 	}
-	if (rows.some((row) => row.outcome === "stopped" || row.outcome === "not-started" || row.outcome === "failed" && row.url)) lines.push("", "Run the same command again to carry on with the reviews that did not finish, from where each got to.");
+	const unfinished = rows.some((row) => row.outcome === "stopped" || row.outcome === "not-started" || row.outcome === "failed" && row.url);
+	lines.push(...batchQueueLines(rows));
+	if (unfinished) lines.push("", "Run the same command again to carry on with the reviews that did not finish, from where each got to.");
 	const all = indexLinks(index);
 	if (all) lines.push("", all);
 	return lines.join("\n");
 }
+/**
+* What `--queue` did across a batch, or nothing at all when it was not asked for.
+*
+* One line for what is in the queue, rather than a word on every row: the rows say how each review
+* ended, and the queue is where the reviewer goes next, so it is said once, together. What was left
+* out is named with its reason, because each reason has a different way round it.
+*/
+function batchQueueLines(rows) {
+	const asked = rows.filter((row) => row.queue !== void 0);
+	if (asked.length === 0) return [];
+	const lines = [""];
+	const queued = asked.filter((row) => row.queue === "added" || row.queue === "already");
+	if (queued.length > 0) lines.push(`In your review queue: ${queued.map((row) => row.label).join(", ")}`);
+	for (const row of asked) {
+		if (row.queue === "added" || row.queue === "already") continue;
+		lines.push(`Not added to your review queue: ${notQueuedBecause(row.label, row.queue)}`);
+	}
+	return lines;
+}
+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.`;
+	}
+}
+/**
+* What `--queue` did with the one review a single command acted on. To stderr, like every line that
+* is not the review's own link.
+*/
+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.";
+	}
+}
 function noReviews() {
 	return "No reviews stored yet. Run `prreviewbuddy review` in a repository to make one.";
 }
@@ -1908,16 +1968,16 @@ async function reviewBatch(command, repoPath, deps = realDeps()) {
 		}
 		claimed.set(identity, index);
 		const prepared = await deps.prepare(target, agent.id, command.operation === "fresh", command.requestedModel);
+		const workspaceId = prepared.kind === "new" ? prepared.workspaceId : prepared.existing.id;
+		row.workspaceId = workspaceId;
+		if (command.queue) row.reviewQueue = deps.enqueue(workspaceId);
 		if (prepared.kind === "existing") {
-			row.workspaceId = prepared.existing.id;
 			row.state = {
 				kind: "existing",
 				inProgress: prepared.existing.inProgress
 			};
 			return;
 		}
-		const workspaceId = prepared.kind === "resume" ? prepared.existing.id : prepared.workspaceId;
-		row.workspaceId = workspaceId;
 		row.state = {
 			kind: "running",
 			jobId: deps.jobIdOf(workspaceId) ?? ""
@@ -1949,7 +2009,11 @@ async function reviewBatch(command, repoPath, deps = realDeps()) {
 }
 function summaryRow(row, link) {
 	const url = row.workspaceId ? link(row.workspaceId) : void 0;
-	const withUrl = url ? { url } : {};
+	const queued = row.reviewQueue ? { queue: row.reviewQueue } : {};
+	const withUrl = {
+		...url ? { url } : {},
+		...queued
+	};
 	switch (row.state.kind) {
 		case "ready": return {
 			label: row.label,
@@ -1979,7 +2043,8 @@ function summaryRow(row, link) {
 		};
 		default: return {
 			label: row.label,
-			outcome: "not-started"
+			outcome: "not-started",
+			...queued
 		};
 	}
 }
@@ -1994,6 +2059,7 @@ function realDeps() {
 		prepare: prepareReview,
 		settle: (workspaceId, signal) => settleReview(workspaceId, "standard", signal),
 		jobIdOf: (workspaceId) => loadWorkspace(workspaceId)?.jobId,
+		enqueue: queueForReview,
 		links: async () => {
 			try {
 				const port = await ensureServer();
@@ -2257,6 +2323,7 @@ async function review(command, repoPath) {
 	const agent = await chooseFor(command);
 	if (!agent) return 1;
 	const prepared = await prepareReview(target, agent.id, command.operation === "fresh", command.requestedModel);
+	enqueue(command, prepared.kind === "new" ? prepared.workspaceId : prepared.existing.id);
 	if (prepared.kind === "existing") {
 		const url = linkTo(await ensureServer(), prepared.existing.id, prepared.existing.token);
 		out(existingReview(prepared.existing, url));
@@ -2311,6 +2378,7 @@ async function kiss(command, target) {
 		...workspace ? { review: { session: workspace.session } } : {}
 	}) ?? void 0;
 	if (workspace) {
+		enqueue(command, workspace.id);
 		const job = await recordKissRun({
 			workspaceId: workspace.id,
 			audience,
@@ -2330,6 +2398,7 @@ async function kiss(command, target) {
 		agentId: agent.id,
 		...requestedModel ? { requestedModel } : {}
 	});
+	enqueue(command, workspaceId);
 	const fresh = loadWorkspace(workspaceId);
 	const run = (fresh.kissRuns ?? [])[0];
 	if (!run) {
@@ -2390,6 +2459,7 @@ async function operate(command, workspace) {
 		err(commitReviewCannotUpdate(workspace.changeSet.data.pullRequest.head.sha));
 		return 1;
 	}
+	enqueue(command, workspace.id);
 	if (command.operation === "reanalyse") {
 		const agent = await chooseFor(command, workspace);
 		if (!agent) return 1;
@@ -2443,6 +2513,17 @@ async function operate(command, workspace) {
 	return 0;
 }
 /**
+* `--queue`: put the review on the reviewer's review queue, now, and say how that went.
+*
+* Called the moment a review exists and before its analysis is awaited, so the card is in the
+* workspace's Queue while the review is still being made. Never a reason to fail the command: the
+* review is what was asked for, and the queue is bookkeeping around it. To stderr, so a piped link
+* is still only a link.
+*/
+function enqueue(command, workspaceId) {
+	if (command.queue) err(reviewQueued(queueForReview(workspaceId)));
+}
+/**
 * Draw a job's progress block, from what the job itself records about its target.
 *
 * From the record rather than the caller, for the reason `drive` gives: the job knows what it is

package/dist/server.js

@@ -1,4 +1,4 @@
-import { A as createPrState, An as AGENT_IDS, At as loadJob, B as toneOf, Bt as followedRefName, C as readEvents, Ct as checkReview, D as announceJob, Dn as writeAgentPreference, Dt as endedAt, E as telemetryUploadConsent, En as readAgentPreference, Et as allJobs, F as markAllRead, Fn as AgentCancelledError, Ft as progressSteps, G as MANAGED_ROOT, Gt as lineagePosition, H as withUsageRecorded, Ht as indexReviewGroups, I as markRead, In as AgentUnavailableError, It as purposeOf, Jt as mutateWorkspace, K as readMarker, Kt as loadWorkspace, L as pruneNotifications, M as eventName, N as headline, Nn as agentFor, Nt as isLiveJob, O as branchOf, Ot as fail, P as listNotifications, Pt as phasesFor, Q as clearServerRecord, Qt as reviewedCommit, R as readNotification, Sn as git, St as checkCodeFreshness, T as recordTelemetryUploadConsent, Tn as clearAgentPreference, Tt as reviewedShaOf, U as EXPLAIN_SIMPLY_PROMPT, Ut as latestKissRun, V as unreadCount$1, W as ReviewBeingDeletedError, X as readIndexToken, Xt as previousKissResult, Y as relativeTime$1, Zt as recentWorkspaces, _ as agentEnvOf, _n as processDiscussion, _t as claimRequestUrl, a as wasBlocked, an as queueAdd, bt as remoteHost, c as RefreshUnavailableError, cn as queueStamp, d as refreshPrContext, dn as doneVerb, dt as MAX_WORKSPACE_PORT, et as readServerRecord, f as isUnchanged, fn as isIssueOutstanding, ft as MIN_WORKSPACE_PORT, gn as reviewerDispositions, gt as writePortPreference, h as liveUpdateFor, hn as questionsOutstanding, ht as statedWorkspacePort, i as liveJobsFor, in as dequeueOnComplete, j as createPrUpdate, jt as saveJob, k as subjectOf, kn as STORE_ROOT, kt as isTerminal, l as mergeUpdateInto, ln as readQueue, lt as feedbackUrl, m as UpdateAlreadyRunningError, mn as issuesOutstanding, mt as resolveWorkspacePort, n as deleteReview, nn as workspaceRevision, nt as serverEntryExists, o as discardJob, on as queueAddMany, ot as writeServerRecord, p as askCheckout, pn as isQuestionOutstanding, pt as parseWorkspacePort, q as removeWorktree, r as lineageIds, rn as workspaceStamps, rt as spawnServer, s as FETCH_TIMEOUT_MS, sn as queueRemove, st as BUILD_VERSION, t as deleteLineage, tn as touchWorkspace, u as updateReview, un as writeQueue, ut as DEFAULT_WORKSPACE_PORT, vn as describeAuthorship, w as record, wn as resolveModel, wt as followedBranch, x as runJob, xn as isCommitReview, xt as fillFileUrlTemplate, y as startKissJob, yt as forgeFor, z as removeNotification, zt as isClaimed } from "./delete_review-DPRL5IPt.js";
+import { $t as recentWorkspaces, A as createPrState, An as STORE_ROOT, At as fail, B as toneOf, C as readEvents, Cn as git, Ct as fillFileUrlTemplate, D as announceJob, Dn as readAgentPreference, Dt as reviewedShaOf, E as telemetryUploadConsent, En as clearAgentPreference, Et as followedBranch, F as markAllRead, Ft as isLiveJob, G as MANAGED_ROOT, Gt as latestKissRun, H as withUsageRecorded, Ht as followedRefName, I as markRead, In as AgentCancelledError, It as phasesFor, Jt as loadWorkspace, K as readMarker, L as pruneNotifications, Ln as AgentUnavailableError, Lt as progressSteps, M as eventName, Mt as loadJob, N as headline, Nt as saveJob, O as branchOf, On as writeAgentPreference, Ot as allJobs, P as listNotifications, Pn as agentFor, Q as readIndexToken, Qt as previousKissResult, R as readNotification, Rt as purposeOf, Sn as isCommitReview, St as remoteHost, T as recordTelemetryUploadConsent, Tn as resolveModel, Tt as checkReview, U as EXPLAIN_SIMPLY_PROMPT, V as unreadCount$1, Vt as isClaimed, W as ReviewBeingDeletedError, Wt as indexReviewGroups, X as compactQueue, Xt as mutateWorkspace, Y as relativeTime$1, _ as agentEnvOf, _n as reviewerDispositions, _t as statedWorkspacePort, a as wasBlocked, an as workspaceStamps, at as spawnServer, c as RefreshUnavailableError, cn as queueAddMany, ct as writeServerRecord, d as refreshPrContext, dn as readQueue, dt as feedbackUrl, en as reviewedCommit, et as clearServerRecord, f as isUnchanged, fn as doneVerb, ft as DEFAULT_WORKSPACE_PORT, gn as questionsOutstanding, gt as resolveWorkspacePort, h as liveUpdateFor, hn as issuesOutstanding, ht as parseWorkspacePort, i as liveJobsFor, in as workspaceRevision, it as serverEntryExists, j as createPrUpdate, jn as AGENT_IDS, jt as isTerminal, k as subjectOf, kt as endedAt, l as mergeUpdateInto, ln as queueRemove, lt as BUILD_VERSION, m as UpdateAlreadyRunningError, mn as isQuestionOutstanding, mt as MIN_WORKSPACE_PORT, n as deleteReview, nt as readServerRecord, o as discardJob, on as dequeueOnComplete, p as askCheckout, pn as isIssueOutstanding, pt as MAX_WORKSPACE_PORT, q as removeWorktree, qt as lineagePosition, r as lineageIds, rn as touchWorkspace, s as FETCH_TIMEOUT_MS, sn as queueAdd, t as deleteLineage, u as updateReview, un as queueStamp, vn as processDiscussion, vt as writePortPreference, w as record, wt as checkCodeFreshness, x as runJob, xt as forgeFor, y as startKissJob, yn as describeAuthorship, yt as claimRequestUrl, z as removeNotification } from "./delete_review-BWMmO81Q.js";
 import { basename, dirname, isAbsolute, join, relative, resolve } from "node:path";
 import { existsSync, mkdirSync, readFileSync, readdirSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { createHash, timingSafeEqual } from "node:crypto";
@@ -744,19 +744,30 @@ var STATES$1 = {
 function renderMessageCopySource(source) {
 	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"`);
 }
+/** The class that places a finding's copy control in its card's heading, and gives it its tooltip. */
+var FINDING_COPY_CLASS = "pr-helper-finding-copy";
 /**
 * The same control on an issue or question card, carrying that finding as Markdown.
 *
-* No action-row wrapper, because the card already has one to sit in. It shows a word at rest where
-* a chat reply shows only the icon: under a reply the icon is the whole affordance and the text is
-* the subject, but a card's row is a line of worded buttons, and an icon alone among them reads as
-* decoration. Both resting words are carried on the button so the reset after "Copied" returns to
-* them rather than to the chat reply's.
+* It sits in the card's heading, not its footer. It copies the whole finding, so the heading is
+* the thing it acts on. And it is a different kind of act from Investigate, Dismiss and Mark
+* answered, which change the review: those keep the footer to themselves, so the footer holds
+* only actions that change something.
+*
+* Icon only at rest, like a chat reply's. It used to show a word because it sat in a row of worded
+* buttons, where an icon alone read as decoration. In the heading it is on its own, and a word
+* would give a utility the weight of the title beside it. What it does is said by a hover
+* tooltip (`data-tip`, the same CSS tooltip the pills use, not a native `title`) and by its
+* accessible name. "Copied" and "Copy failed" still show as words, because a picture cannot say
+* that the click did something. The resting name is kept on the button so the reset returns to
+* "Copy finding" rather than to a chat reply's "Copy message".
 */
 function renderFindingCopySource(source, kind) {
+	const name = kind === "issue" ? "Copy finding" : "Copy question";
 	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"`, {
-		label: "Copy",
-		aria: `Copy ${kind}`
+		label: "",
+		aria: name,
+		tip: name
 	});
 }
 /** Where a finding's button keeps its resting word and name. Absent on a chat reply's. */
@@ -766,7 +777,8 @@ function button(handle, idle) {
 	const { icon } = STATES$1.idle;
 	const label = idle?.label ?? STATES$1.idle.label;
 	const aria = idle?.aria ?? STATES$1.idle.aria;
-	const control = `<button type="button" class="${MESSAGE_COPY_CLASS}" ${handle}${idle ? ` ${IDLE_LABEL_ATTR}="${escapeHtml$1(label)}" ${IDLE_ARIA_ATTR}="${escapeHtml$1(aria)}"` : ""} aria-label="${escapeHtml$1(aria)}"><span class="${MESSAGE_COPY_CLASS}-icon" aria-hidden="true">${svg(icon)}</span><span class="${MESSAGE_COPY_CLASS}-label"${label ? "" : " hidden"}>${escapeHtml$1(label)}</span></button>`;
+	const remembered = idle ? ` ${IDLE_LABEL_ATTR}="${escapeHtml$1(label)}" ${IDLE_ARIA_ATTR}="${escapeHtml$1(aria)}"` : "";
+	const control = `<button type="button" class="${idle ? `${MESSAGE_COPY_CLASS} ${FINDING_COPY_CLASS}` : MESSAGE_COPY_CLASS}" ${handle}${remembered}${idle ? ` data-tip="${escapeHtml$1(idle.tip)}"` : ""} aria-label="${escapeHtml$1(aria)}"><span class="${MESSAGE_COPY_CLASS}-icon" aria-hidden="true">${svg(icon)}</span><span class="${MESSAGE_COPY_CLASS}-label"${label ? "" : " hidden"}>${escapeHtml$1(label)}</span></button>`;
 	return idle ? control : `<div class="${MESSAGE_ACTIONS_CLASS}">${control}</div>`;
 }
 function svg(paths) {
@@ -2027,7 +2039,7 @@ function renderReconciliationNote(note) {
 * stuck. Outlined rather than filled — see sidebar.css.
 *
 * Text only, no leading icon. Every other action in the sidebar (Jump to file, Mark answered,
-* Explain, Analyse, Copy) is a bare label, and an emoji here read as decoration on a control
+* Explain, Analyse) is a bare label, and an emoji here read as decoration on a control
 * whose neighbours have none rather than as emphasis.
 */
 function renderInvestigateButton(attribute, id, label) {
@@ -2147,7 +2159,10 @@ function renderIssuesTab(session, investigatingId) {
 		return `
     <div class="pr-helper-issue-card pr-helper-severity-${issue.severity}${issue.reconciliationStatus ? ` pr-helper-issue-${issue.reconciliationStatus}` : ""}${issue.investigationImpact ? ` pr-helper-issue-investigation-${issue.investigationImpact}` : ""}${isDismissed && !stale ? " pr-helper-issue-card-dismissed" : ""}" data-finding-card="${escapeHtml$1(issue.id)}">
       <div class="pr-helper-issue-header">
-        <p class="pr-helper-issue-title">${renderInlineText(issue.title)}</p>
+        <div class="pr-helper-issue-title-row">
+          <p class="pr-helper-issue-title">${renderInlineText(issue.title)}</p>
+          ${renderFindingCopySource(issueMarkdown(issue), "issue")}
+        </div>
         <div class="pr-helper-issue-badges">
           ${renderFindingStatus(issue.reconciliationStatus)}
           ${renderDismissedBadge(isDismissed)}
@@ -2167,7 +2182,6 @@ function renderIssuesTab(session, investigatingId) {
       ${renderRelatedFiles(issue.relatedFiles)}
       ${renderInvestigationProvenance("issue", issue.id, records, now)}
       <div class="pr-helper-issue-actions">
-        ${renderFindingCopySource(issueMarkdown(issue), "issue")}
         ${renderInvestigationActions("issue", issue.id, records, investigatingId)}
         ${renderDismissToggle(issue, isDismissed, stale, dismissals.get(issue.id))}
       </div>
@@ -2236,7 +2250,10 @@ function renderQuestionCard(q, sections, answered, records, now = Date.now(), in
 	return `
     <div class="pr-helper-question-card${answered ? " pr-helper-question-card-answered" : ""}${q.investigationImpact ? ` pr-helper-issue-investigation-${q.investigationImpact}` : ""}" data-finding-card="${escapeHtml$1(q.id)}"${q.sectionId ? ` data-section-id="${escapeHtml$1(q.sectionId)}"` : ""}>
       <div class="pr-helper-issue-header">
-        <p class="pr-helper-issue-title">${renderInlineText(q.question)}</p>
+        <div class="pr-helper-issue-title-row">
+          <p class="pr-helper-issue-title">${renderInlineText(q.question)}</p>
+          ${renderFindingCopySource(questionMarkdown(q), "question")}
+        </div>
         <div class="pr-helper-issue-badges">
           ${renderQuestionStatus(q.reconciliationStatus)}
           ${renderInvestigationImpact(q.investigationImpact)}
@@ -2259,7 +2276,6 @@ function renderQuestionCard(q, sections, answered, records, now = Date.now(), in
       <div class="pr-helper-question-actions">
         ${jumpLink}
         <div class="pr-helper-question-actions-group">
-          ${renderFindingCopySource(questionMarkdown(q), "question")}
           ${renderInvestigationActions("question", q.id, records, investigatingId)}
           ${answerToggle}
         </div>
@@ -4015,6 +4031,20 @@ function renderUpdateProgress(job) {
 </div>`;
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/pr_state_view.ts
+/**
+* The pull request's state, beside the review's status and never instead of it: `Reviewed` is the
+* reviewer's decision, `Merged` is the upstream's. Open says nothing, because it is what a review
+* of a pull request usually is, and a label that never varies teaches nothing.
+*
+* `besideNumber` is whether the pull request's number sits right before the pill. There, "Closed"
+* is enough; anywhere else it would read as the branch closing, so the pill names the PR itself.
+*/
+function renderPrState(state, besideNumber = false) {
+	if (state !== "merged" && state !== "closed") return "";
+	return `<span class="prb-card-pr-state" data-pr-state="${state}">${state === "merged" ? "Merged" : besideNumber ? "Closed" : "PR closed"}</span>`;
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/page.ts
 function renderWorkspace(view) {
 	const { session, changeSet } = view;
@@ -4394,6 +4424,7 @@ function renderReviewBar(view, freshness) {
   <span class="prb-change-ref" title="${escapeHtml$1(changeSet.repoPath)}">
     ${changeSet.isLinkedWorktree ? `<span class="prb-checkout">${escapeHtml$1(changeSet.checkoutName)}</span>` : ""}
     ${view.prContext ? `<a class="prb-pr-chip" href="${escapeHtml$1(view.prContext.prContext.pullRequest.url)}" target="_blank" rel="noopener noreferrer" title="${escapeHtml$1(view.prContext.prContext.pullRequest.title ?? "")}">#${view.prContext.prContext.pullRequest.number}</a>` : ""}
+    ${view.prState ? `<span class="prb-pr-state-slot" data-pr-state-slot${view.prContext ? " data-beside-number" : ""}>${renderPrState(view.prState.state, view.prContext !== void 0)}</span>` : ""}
     ${renderSubject(changeSet)}
     ${changeSet.includesUncommitted ? "<span class=\"prb-file-status\">+ working tree</span>" : ""}
     ${renderAuthor(view)}
@@ -5912,6 +5943,20 @@ function storedRequestKeys() {
 function prStatesByRequest(states = readPrStates()) {
 	return new Map(Object.entries(states).map(([key, entry]) => [key, entry.state]));
 }
+/**
+* One review's pull request state, for its page, filed under the same key the list looks up. Null
+* for a review with no request or one never observed: unknown, not open. Only ever what the watcher
+* last saw; nothing here reaches the forge.
+*/
+function prStateOf$1(workspace, states = readPrStates()) {
+	const request = requestOf(workspace);
+	if (!request) return null;
+	return states[requestKey(workspace.repoPath, request.number)]?.state ?? null;
+}
+/** Whether a review is of a pull request at all, whatever its state. See `requestOf`. */
+function hasRequest(workspace) {
+	return requestOf(workspace) !== null;
+}
 function defaultDeps() {
 	return {
 		candidates: watchedCandidates,
@@ -6417,15 +6462,6 @@ function statusOf(review) {
 	if (review.completedAt) return "complete";
 	return review.lastViewedAt ? "in-progress" : "new";
 }
-/**
-* The pull request's state, beside the review's status and never instead of it: `Reviewed` is the
-* reviewer's decision, `Merged` is the upstream's. Open says nothing, because it is what a review
-* of a pull request usually is, and a label that never varies teaches nothing.
-*/
-function renderPrState(state) {
-	if (state !== "merged" && state !== "closed") return "";
-	return `<span class="prb-card-pr-state" data-pr-state="${state}">${state === "merged" ? "Merged" : "PR closed"}</span>`;
-}
 function renderStatus(status) {
 	return `<span class="prb-card-status" data-status="${status}">${statusLabel(status)}</span>`;
 }
@@ -8636,11 +8672,6 @@ function addMany(res, ids) {
 		refused
 	});
 }
-/** Compacted here rather than in `queue.ts`, which must not learn to read the review store. */
-function compactQueue(stored) {
-	const live = stored.filter((queued) => loadWorkspace(queued) !== null);
-	return live.length === stored.length ? stored : writeQueue(live) ?? stored;
-}
 /**
 * Delete one review, or everything on one card.
 *
@@ -8881,6 +8912,7 @@ async function sendWorkspacePage(res, workspace) {
 		freshness: freshness ?? void 0,
 		availability: standing?.availability ?? null,
 		remote: await remoteOrNull(workspace),
+		...hasRequest(workspace) ? { prState: { state: prStateOf$1(workspace) } } : {},
 		dismissedFreshnessSha: workspace.dismissedFreshnessSha,
 		lastPrRefresh: workspace.lastPrRefresh,
 		lastUpdate: notice,
@@ -8999,10 +9031,12 @@ async function handleFreshness(res, workspace, quiet) {
 	const standing = await standingOrNull(workspace);
 	const report = standing?.freshness ?? null;
 	const remote = await remoteOrNull(workspace);
+	const prState = prStateOf$1(workspace);
 	const revision = workspaceRevision(workspace) ?? null;
 	if (!report) return send(res, 200, {
 		report: null,
 		remote,
+		prState,
 		kiss,
 		kissDismissedSha: workspace.dismissedKissFreshnessSha ?? null,
 		revision
@@ -9018,6 +9052,7 @@ async function handleFreshness(res, workspace, quiet) {
 		report,
 		availability: standing?.availability ?? null,
 		remote,
+		prState,
 		dismissedSha: workspace.dismissedFreshnessSha ?? null,
 		kiss,
 		kissDismissedSha: workspace.dismissedKissFreshnessSha ?? null,

package/package.json

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

package/static/shell.css

@@ -562,6 +562,10 @@ body[data-server-stopped] .prb-shell { opacity: 0.55; pointer-events: none; }
   text-decoration: none;
 }
 .prb-pr-chip:hover { background: var(--brand); color: #fff; }
+/* Where the pull request's Merged or Closed pill goes, right after the chip. No box of its own, so an
+   open pull request's empty slot leaves no extra gap in the row and a filled one lays the pill out
+   as the chip's sibling. The pill is the Reviews card's, `.prb-card-pr-state`. */
+.prb-pr-state-slot { display: contents; }
 
 /* Beside the branch, after it, because the branch is what the reviewer came for and this is who
    it belongs to. One line, ellipsed rather than wrapped: the header is a single row and a long

package/static/sidebar.css

@@ -876,6 +876,64 @@
 
 .pr-helper-issue-title { font-size: 14px; font-weight: 600; margin: 0; text-align: left; width: 100%; }
 
+/* The title and the one control that acts on the whole finding, side by side. The title takes the
+   room and wraps; the copy icon keeps its size at the top-right, level with the title's first line
+   however many lines the title wraps to. */
+.pr-helper-issue-title-row {
+  display: flex;
+  align-items: flex-start;
+  gap: 8px;
+  width: 100%;
+}
+
+.pr-helper-issue-title-row > .pr-helper-issue-title { flex: 1; min-width: 0; }
+
+/* Full strength rather than a chat reply's 0.65: at 0.65 its grey falls under 3:1 on the white card.
+   Pulled up a pixel so the 14px glyph sits on the 14px title's first line rather than below it.
+   Positioned so its tooltip can anchor to it: unlike a pill's, which spans its whole row, this one
+   belongs to one small control at the card's edge. */
+.pr-helper-finding-copy {
+  position: relative;
+  flex-shrink: 0;
+  margin-top: -1px;
+  opacity: 1;
+  font-size: 12px;
+}
+
+/* The tooltip grows leftward from the right edge, back over the card rather than out of the pane,
+   and stays on one line. The same look, delay and text reset as `.pr-helper-tip`, which it cannot
+   reuse: that one pins itself to its positioned row, and this control is its own anchor. */
+.pr-helper-finding-copy::after {
+  content: attr(data-tip);
+  position: absolute;
+  top: calc(100% + 6px);
+  right: 0;
+  z-index: 3;
+  padding: 5px 8px;
+  border-radius: 6px;
+  background: #24292f;
+  color: #ffffff;
+  font-size: 11px;
+  font-weight: 400;
+  font-family: inherit;
+  line-height: 1.4;
+  white-space: nowrap;
+  opacity: 0;
+  pointer-events: none;
+  transition: opacity 0.1s;
+}
+
+.pr-helper-finding-copy:hover::after,
+.pr-helper-finding-copy:focus-visible::after {
+  opacity: 1;
+  transition-delay: 0.35s;
+}
+
+/* The button is saying "Copied" or "Copy failed" in words right now; a tooltip offering to copy
+   would argue with it. */
+.pr-helper-finding-copy.copied::after,
+.pr-helper-finding-copy.failed::after { display: none; }
+
 .pr-helper-issue-desc {
   font-size: 13px;
   line-height: 1.5;
@@ -4187,20 +4245,6 @@
   margin-top: 12px;
 }
 
-/* Copying the finding is a different kind of act from deciding about it, so it takes the far edge
-   rather than joining Investigate and Dismiss. On a question card it sits in the group instead,
-   because the far edge there already belongs to Jump to file. */
-.pr-helper-issue-actions > .pr-helper-msg-copy { margin-right: auto; }
-
-/* Full strength on a card. Under a chat reply the control is a quiet afterthought to the text, and
-   its 0.65 is right there; here it is a worded button among worded buttons, and at 0.65 its grey
-   falls under 3:1 on the white card. The size is its neighbours'. */
-.pr-helper-issue-actions .pr-helper-msg-copy,
-.pr-helper-question-actions .pr-helper-msg-copy {
-  opacity: 1;
-  font-size: 12px;
-}
-
 /* margin-left:auto rather than relying on the row's space-between: the jump link beside it is
    conditional (a question without a file has none), and space-between alone would swing this
    group to the left edge whenever it is the row's only child. Wraps because three controls do