prreviewbuddy 0.30.0 → 0.31.0

package/dist/main.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { $t as writeAgentPreference, A as removeWorktree, B as workspaceUrl, C as recordTelemetryUploadConsent, Dt as saveWorkspace, Et as reviewedRepositories, G as DEFAULT_WORKSPACE_PORT, H as BUILD_VERSION, I as ensureServer, J as parseWorkspacePort, Jt as git, K as MAX_WORKSPACE_PORT, Kt as resolveTarget, M as relativeTime, N as readIndexToken, O as MANAGED_ROOT, Ot as summarise, P as bootstrapUrl, Q as forgeResolver, Qt as readAgentPreference, R as reviewsUrl, S as record$1, St as positionInLineage, U as PACKAGE_NAME, X as statedWorkspacePort, Xt as resolveModel, Yt as resolveAgent, Z as writePortPreference, a as wasBlocked, at as isTerminal, b as startJob, bt as loadWorkspace, c as updateReview, ct as PHASES, dt as purposeOf, en as CONFIG_PATH, f as UpdateAlreadyRunningError, ft as clearClaim, g as recordKissRun, ht as groupByLineage, i as liveJobsFor, in as agentById, j as modelHelpLines, k as readMarker, m as runningJobs, mt as followedRefName, n as deleteReview$1, nn as AGENT_IDS, nt as allJobs, on as detectAgents, ot as loadJob, p as liveUpdateFor, pt as isClaimed, q as MIN_WORKSPACE_PORT, qt as displayRef, rn as DEFAULT_AGENT_ID, tn as STORE_ROOT, tt as checkFreshness, ut as progressSteps, v as reanalyseReview, vt as lineageKeyFor, w as telemetryUploadConsent, wt as recentWorkspaces, xt as matchingWorkspaceIds, y as runJob, z as stopServer } from "./delete_review-B_2J2Wn9.js";
+import { $t as writeAgentPreference, A as removeWorktree, B as workspaceUrl, C as recordTelemetryUploadConsent, Dt as saveWorkspace, Et as reviewedRepositories, G as DEFAULT_WORKSPACE_PORT, H as BUILD_VERSION, I as ensureServer, J as parseWorkspacePort, Jt as git, K as MAX_WORKSPACE_PORT, Kt as resolveTarget, M as relativeTime, N as readIndexToken, O as MANAGED_ROOT, Ot as summarise, P as bootstrapUrl, Q as forgeResolver, Qt as readAgentPreference, R as reviewsUrl, S as record$1, St as positionInLineage, U as PACKAGE_NAME, X as statedWorkspacePort, Xt as resolveModel, Yt as resolveAgent, Z as writePortPreference, a as wasBlocked, at as isTerminal, b as startJob, bt as loadWorkspace, c as updateReview, ct as PHASES, dt as purposeOf, en as CONFIG_PATH, f as UpdateAlreadyRunningError, ft as clearClaim, g as recordKissRun, ht as groupByLineage, i as liveJobsFor, in as agentById, j as modelHelpLines, k as readMarker, m as runningJobs, mt as followedRefName, n as deleteReview$1, nn as AGENT_IDS, nt as allJobs, on as detectAgents, ot as loadJob, p as liveUpdateFor, pt as isClaimed, q as MIN_WORKSPACE_PORT, qt as displayRef, rn as DEFAULT_AGENT_ID, tn as STORE_ROOT, tt as checkReview, ut as progressSteps, v as reanalyseReview, vt as lineageKeyFor, w as telemetryUploadConsent, wt as recentWorkspaces, xt as matchingWorkspaceIds, y as runJob, z as stopServer } from "./delete_review-C2wax0ED.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";
@@ -15,6 +15,7 @@ var VALUED = {
 		"--model",
 		"--branch",
 		"--pr",
+		"--commit",
 		"--review",
 		"--audience",
 		"--lens",
@@ -60,10 +61,11 @@ var POSITIONALS = {
 	config: 3,
 	uninstall: 0
 };
-/** The three flags that say what is being acted on, and the field each becomes. */
+/** The flags that say what is being acted on, and the field each becomes. */
 var TARGETS = {
 	"--branch": "branchRef",
 	"--pr": "prUrl",
+	"--commit": "commitRef",
 	"--review": "reviewId"
 };
 /**
@@ -302,9 +304,11 @@ function refuseCombination(values, flags) {
 		}
 	}
 	if (values["--pr"] && values["--base"]) return "A pull request is compared against the branch it is asking to merge into, so `--base` would stop the diff being the pull request's. To compare its branch against something else, review the branch: `prreviewbuddy review --branch <branch> --base <ref>`.";
+	if (values["--commit"] && values["--base"]) return "A commit is reviewed as the change it made, which is its diff against its own parent, so `--base` would make the findings about a different change under a page still naming `--commit`. To compare a branch against a base of your choosing, review the branch: `prreviewbuddy review --branch <branch> --base <ref>`.";
+	if (values["--commit"] && flags.has("--update")) return `\`--update\` ${OPERATIONS["--update"][1]}, and \`--commit\` names one commit, which cannot move, so there is nothing to bring up to date. To analyse the same commit again, run \`prreviewbuddy review --commit ${values["--commit"]} --reanalyse\`.`;
 	if (flags.has("--delete")) {
 		if (!values["--review"]) {
-			const named = values["--branch"] ?? values["--pr"];
+			const named = values["--branch"] ?? values["--pr"] ?? values["--commit"];
 			return (named ? `\`--delete\` removes one review, and \`${named}\` can name several: reviewing a target twice with \`--fresh\` leaves two. ` : "`--delete` removes one review, and the branch you are standing on can have several: reviewing it twice with `--fresh` leaves two. ") + "Say which with `--review <review>`, which `prreviewbuddy open` prints beside each review.";
 		}
 		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.";
@@ -626,11 +630,20 @@ async function pick(ask, options) {
 */
 async function findExistingReview(target) {
 	const groups = groupByLineage(recentWorkspaces(Number.MAX_SAFE_INTEGER));
-	const wanted = lineageKeyFor(target.originRepoPath, target.branch, target.ref);
+	const wanted = target.commit ? lineageKeyFor({
+		kind: "commit",
+		repoPath: target.originRepoPath,
+		sha: target.commit.sha
+	}) : lineageKeyFor({
+		kind: "branch",
+		repoPath: target.originRepoPath,
+		branch: target.branch,
+		followedRef: target.ref
+	});
 	const group = groups.find((entry) => entry.key === wanted);
 	if (!group) return null;
 	const requested = target.pullRequest?.number;
-	const history = group.history.filter((review) => target.base ? madeAgainst(target.base)(review) : true).filter((review) => requested ? review.prNumber === requested : true);
+	const history = target.commit ? group.history : group.history.filter((review) => target.base ? madeAgainst(target.base)(review) : true).filter((review) => requested ? review.prNumber === requested : true);
 	if (history.length === 0) return null;
 	const summary = history[0];
 	const found = {
@@ -640,6 +653,7 @@ async function findExistingReview(target) {
 		branch: summary.branch,
 		baseRef: summary.baseRef,
 		...summary.headResolvedRef ? { headResolvedRef: summary.headResolvedRef } : {},
+		...summary.targetType === "commit" ? { targetType: "commit" } : {},
 		createdAt: summary.createdAt,
 		commitSha: summary.commitSha ?? null,
 		commitsBehind: null,
@@ -651,7 +665,13 @@ async function findExistingReview(target) {
 	if (!workspace) return found;
 	const inProgress = isRunning(workspace.jobId);
 	try {
-		const code = (await checkFreshness(workspace)).code;
+		const { freshness, availability } = await checkReview(workspace);
+		if (availability) return {
+			...found,
+			inProgress,
+			commitAvailable: availability.state === "available"
+		};
+		const code = freshness.code;
 		const baselineGone = code.rewritten || code.reason === "missing-commit";
 		return {
 			...found,
@@ -722,6 +742,7 @@ function describeSubject(existing) {
 * reviewer's own working tree.
 */
 function describeMovement(existing) {
+	if (existing.targetType === "commit") return existing.commitAvailable === false ? "That commit is no longer in this repository, so the review can be read but not analysed again." : "A commit does not move, so the review still describes exactly that commit.";
 	const subject = followedRefName(existing.branch, existing.headResolvedRef) ?? existing.branch;
 	if (existing.baselineGone) return `${subject} has been rebuilt since, so that commit is no longer on it.`;
 	if (existing.commitsBehind === null) return `How far ${subject} has moved since could not be worked out.`;
@@ -803,7 +824,7 @@ var RESET = "\x1B[0m";
 function header(target, colour, purpose = "review") {
 	const meta = `${target.sha.slice(0, 7)} · isolated checkout · ${target.originRepoPath}`;
 	return [
-		`${purpose === "update" ? "Updating" : "Reviewing"} ${target.branch}`,
+		`${purpose === "update" ? "Updating" : "Reviewing"} ${target.commit ? `commit ${target.branch}` : target.branch}`,
 		colour ? `${DIM}${meta}${RESET}` : meta,
 		""
 	];
@@ -948,6 +969,7 @@ your own checkout is never read or written while it works.
 
   prreviewbuddy review                    review the current branch
   prreviewbuddy review --branch <branch>  review another branch, without checking it out
+  prreviewbuddy review --commit <commit>  review the change one commit made, against its parent
   prreviewbuddy review --pr <url>         review a pull request
   prreviewbuddy review --pr <url> <url> ...
                                           review several pull requests, each on its own, two at a
@@ -1256,7 +1278,8 @@ function startedReview(url, _target) {
 * never produced a result is offering to reconcile something against itself.
 */
 function existingReview(existing, url) {
-	const subject = followedRefName(existing.branch, existing.headResolvedRef) ?? existing.branch;
+	const commit = existing.targetType === "commit" ? (existing.commitSha ?? existing.branch).slice(0, 7) : null;
+	const subject = commit ? `commit ${commit}` : followedRefName(existing.branch, existing.headResolvedRef) ?? existing.branch;
 	if (existing.inProgress) return [
 		url,
 		"",
@@ -1272,7 +1295,7 @@ function existingReview(existing, url) {
 		`A review of ${subject} already exists, so here it is rather than a second one.`,
 		made,
 		"",
-		existing.baselineGone ? "This branch has been rebuilt since, so the commit the review was made from is no longer on it. There is nothing left to update it against, so this one needs starting fresh: `prreviewbuddy review --fresh`." : "To update the review in place, keeping your transcript and your place in it, run `prreviewbuddy review --update`. To start fresh alongside it, run `prreviewbuddy review --fresh`."
+		commit ? `To analyse the same commit again, keeping this analysis beside the new one, run \`prreviewbuddy review --commit ${commit} --reanalyse\`. To start a separate review of it, run \`prreviewbuddy review --commit ${commit} --fresh\`.` : existing.baselineGone ? "This branch has been rebuilt since, so the commit the review was made from is no longer on it. There is nothing left to update it against, so this one needs starting fresh: `prreviewbuddy review --fresh`." : "To update the review in place, keeping your transcript and your place in it, run `prreviewbuddy review --update`. To start fresh alongside it, run `prreviewbuddy review --fresh`."
 	].join("\n");
 }
 function reviewFinished(url, summary, conversation) {
@@ -1478,6 +1501,14 @@ function noReviewOfTarget(name, make) {
 	return `There is no review of ${name} to act on. Run \`${make}\` to make one.`;
 }
 /**
+* `--update` reaching a commit review by id, where the parser could not see what the id names.
+* The `--commit --update` spelling is refused earlier, in `args.ts`, in the same words.
+*/
+function commitReviewCannotUpdate(sha) {
+	const short = sha.slice(0, 7);
+	return `This review is of commit ${short}, and a commit cannot move, so there is nothing to bring up to date. To analyse the same commit again, run \`prreviewbuddy review --commit ${short} --reanalyse\`.`;
+}
+/**
 * The update's own sentence, which the harness already wrote for the workspace strip, plus the
 * link. Not rephrased here: an update that stopped early has a specific reason and the reviewer
 * gets the same reason whichever surface asked for it.
@@ -2200,6 +2231,7 @@ async function review(command, repoPath) {
 		repoPath,
 		...command.branchRef ? { branchRef: command.branchRef } : {},
 		...command.prUrl ? { prUrl: command.prUrl } : {},
+		...command.commitRef ? { commitRef: command.commitRef } : {},
 		...command.base ? { base: command.base } : {},
 		resolveRequest: forgeResolver
 	});
@@ -2207,7 +2239,7 @@ async function review(command, repoPath) {
 		const existing = await findExistingReview(target);
 		const workspace = existing ? loadWorkspace(existing.id) : null;
 		if (!workspace) {
-			err(noReviewOfTarget(target.branch, makeCommandFor(command)));
+			err(noReviewOfTarget(target.commit ? `commit ${target.branch}` : target.branch, makeCommandFor(command)));
 			return 1;
 		}
 		return operate(command, workspace);
@@ -2345,6 +2377,10 @@ function selectionOf(command) {
 * store, so the two lines after it here are the whole difference between this and the HTTP route.
 */
 async function operate(command, workspace) {
+	if (command.operation === "update" && workspace.changeSet.targetType === "commit") {
+		err(commitReviewCannotUpdate(workspace.changeSet.data.pullRequest.head.sha));
+		return 1;
+	}
 	if (command.operation === "reanalyse") {
 		const agent = await chooseFor(command, workspace);
 		if (!agent) return 1;
@@ -2419,6 +2455,10 @@ function follow(jobId) {
 * the whole of what there is to say.
 */
 function actedOn(workspace) {
+	if (workspace.changeSet.targetType === "commit") return {
+		name: `commit ${workspace.changeSet.data.pullRequest.head.sha.slice(0, 7)}`,
+		createdAt: workspace.createdAt
+	};
 	const { headRef, headResolvedRef } = workspace.changeSet;
 	return {
 		name: followedRefName(headRef, headResolvedRef) ?? headRef,
@@ -2437,6 +2477,7 @@ function actedOn(workspace) {
 function makeCommandFor(command) {
 	if (command.branchRef) return `prreviewbuddy review --branch ${command.branchRef}`;
 	if (command.prUrl) return `prreviewbuddy review --pr ${command.prUrl}`;
+	if (command.commitRef) return `prreviewbuddy review --commit ${command.commitRef}`;
 	return "prreviewbuddy review";
 }
 function failureOf(workspaceId) {

package/dist/server.js

@@ -1,4 +1,4 @@
-import { $ as fillFileUrlTemplate, $t as writeAgentPreference, A as removeWorktree, At as workspaceRevision, Bt as isQuestionOutstanding, C as recordTelemetryUploadConsent, Ct as previousKissResult, D as ReviewBeingDeletedError, Dt as saveWorkspace, E as EXPLAIN_SIMPLY_PROMPT, F as clearServerRecord, Ft as queueStamp, G as DEFAULT_WORKSPACE_PORT, Gt as describeAuthorship, H as BUILD_VERSION, Ht as questionsOutstanding, It as readQueue, J as parseWorkspacePort, Jt as git, K as MAX_WORKSPACE_PORT, L as readServerRecord, Lt as writeQueue, M as relativeTime$1, Mt as dequeueOnComplete, N as readIndexToken, Nt as queueAdd, O as MANAGED_ROOT, Pt as queueRemove, Qt as readAgentPreference, Rt as doneVerb, S as record, T as withUsageRecorded, Tt as reviewedCommit, Ut as reviewerDispositions, V as writeServerRecord, Vt as issuesOutstanding, W as feedbackUrl, Wt as processDiscussion, X as statedWorkspacePort, Xt as resolveModel, Y as resolveWorkspacePort, Z as writePortPreference, Zt as clearAgentPreference, _ as startKissJob, _t as latestKissRun, a as wasBlocked, an as agentFor, at as isTerminal, bt as loadWorkspace, c as updateReview, cn as AgentUnavailableError, d as askCheckout, dt as purposeOf, et as checkCodeFreshness, f as UpdateAlreadyRunningError, gt as indexReviewGroups, h as agentEnvOf, i as liveJobsFor, it as fail, jt as workspaceStamps, k as readMarker, kt as touchWorkspace, l as refreshPrContext, lt as phasesFor, mt as followedRefName, n as deleteReview, nn as AGENT_IDS, nt as allJobs, o as discardJob, ot as loadJob, p as liveUpdateFor, pt as isClaimed, q as MIN_WORKSPACE_PORT, r as lineageIds, rt as endedAt, s as RefreshUnavailableError, sn as AgentCancelledError, st as saveJob, t as deleteLineage, tn as STORE_ROOT, tt as checkFreshness, u as isUnchanged, ut as progressSteps, w as telemetryUploadConsent, wt as recentWorkspaces, x as readEvents, y as runJob, yt as lineagePosition, zt as isIssueOutstanding } from "./delete_review-B_2J2Wn9.js";
+import { $ as fillFileUrlTemplate, $t as writeAgentPreference, A as removeWorktree, At as workspaceRevision, Bt as isQuestionOutstanding, C as recordTelemetryUploadConsent, Ct as previousKissResult, D as ReviewBeingDeletedError, Dt as saveWorkspace, E as EXPLAIN_SIMPLY_PROMPT, F as clearServerRecord, Ft as queueStamp, G as DEFAULT_WORKSPACE_PORT, Gt as describeAuthorship, H as BUILD_VERSION, Ht as questionsOutstanding, It as readQueue, J as parseWorkspacePort, Jt as git, K as MAX_WORKSPACE_PORT, L as readServerRecord, Lt as writeQueue, M as relativeTime$1, Mt as dequeueOnComplete, N as readIndexToken, Nt as queueAdd, O as MANAGED_ROOT, Pt as queueRemove, Qt as readAgentPreference, Rt as doneVerb, S as record, T as withUsageRecorded, Tt as reviewedCommit, Ut as reviewerDispositions, V as writeServerRecord, Vt as issuesOutstanding, W as feedbackUrl, Wt as processDiscussion, X as statedWorkspacePort, Xt as resolveModel, Y as resolveWorkspacePort, Z as writePortPreference, Zt as clearAgentPreference, _ as startKissJob, _t as latestKissRun, a as wasBlocked, an as agentFor, at as isTerminal, bt as loadWorkspace, c as updateReview, cn as AgentUnavailableError, d as askCheckout, dt as purposeOf, et as checkCodeFreshness, f as UpdateAlreadyRunningError, gt as indexReviewGroups, h as agentEnvOf, i as liveJobsFor, it as fail, jt as workspaceStamps, k as readMarker, kt as touchWorkspace, l as refreshPrContext, lt as phasesFor, mt as followedRefName, n as deleteReview, nn as AGENT_IDS, nt as allJobs, o as discardJob, ot as loadJob, p as liveUpdateFor, pt as isClaimed, q as MIN_WORKSPACE_PORT, r as lineageIds, rt as endedAt, s as RefreshUnavailableError, sn as AgentCancelledError, st as saveJob, t as deleteLineage, tn as STORE_ROOT, tt as checkReview, u as isUnchanged, ut as progressSteps, w as telemetryUploadConsent, wt as recentWorkspaces, x as readEvents, y as runJob, yt as lineagePosition, zt as isIssueOutstanding } from "./delete_review-C2wax0ED.js";
 import { isAbsolute, join, relative, resolve } from "node:path";
 import { existsSync, mkdirSync, readFileSync, readdirSync, statSync, writeFileSync } from "node:fs";
 import { createHash, timingSafeEqual } from "node:crypto";
@@ -256,7 +256,16 @@ var PATHS = {
 };
 /** The icon slot for one rail row. Hidden from assistive technology: the label beside it says it. */
 function railIcon(icon) {
-	return `<span class="prb-rail-icon" aria-hidden="true"><svg viewBox="0 0 24 24" focusable="false" fill="none" stroke="currentColor" stroke-width="1.75" stroke-linecap="round" stroke-linejoin="round">${PATHS[icon]}</svg></span>`;
+	return `<span class="prb-rail-icon" aria-hidden="true">${iconSvg(icon)}</span>`;
+}
+/**
+* The same drawing without the rail's slot, for somewhere else that means the same thing.
+*
+* The Files tab counts a file's issues and questions with these, so the icon a reader learned from
+* the rail's Issues and Questions rows is the icon they meet on the row. Sized by the caller's CSS.
+*/
+function iconSvg(icon) {
+	return `<svg viewBox="0 0 24 24" focusable="false" aria-hidden="true" fill="none" stroke="currentColor" stroke-width="1.75" stroke-linecap="round" stroke-linejoin="round">${PATHS[icon]}</svg>`;
 }
 //#endregion
 //#region ../../packages/review-harness/src/domain/utils/file_path.ts
@@ -3024,6 +3033,10 @@ function freshnessLabel(report) {
 		label: "Check for updates",
 		state: "unknown"
 	};
+	if (report.code.state === "fixed") return {
+		label: "Freshness: Fixed",
+		state: "fixed"
+	};
 	if (report.code.state === "changed") return {
 		label: report.code.rewritten ? "Code rewritten" : "Code changed",
 		state: "changed"
@@ -3062,9 +3075,10 @@ function renderFreshness(report, options = {}) {
 	if (!report) return "";
 	const alarming = report.code.state === "changed";
 	const lost = report.code.reason === "missing-commit";
+	const unavailable = options.availability?.state === "missing";
 	const dismissed = options.dismissedSha !== void 0 && options.dismissedSha === report.code.latestSha;
-	if (!options.open && (!(alarming || lost) || dismissed)) return "";
-	const status = codeStatus(report.code);
+	if (!options.open && !unavailable && (!(alarming || lost) || dismissed)) return "";
+	const status = codeStatus(report.code, options.availability ?? null);
 	const facts = [...status.facts, `checked ${escapeHtml$1(relativeTime$1(report.checkedAt))}`];
 	return `<div class="prb-fresh" data-state="${alarming || lost ? "changed" : "quiet"}" role="status">
   <div class="prb-fresh-head">
@@ -3142,7 +3156,13 @@ function short(sha) {
 function reviewed(sha) {
 	return sha ? [`reviewed ${short(sha)}`] : [];
 }
-function codeStatus(code) {
+function codeStatus(code, availability) {
+	if (code.state === "fixed") return {
+		state: "fixed",
+		title: "Freshness: Fixed",
+		facts: reviewed(code.reviewedSha),
+		consequence: availability?.state === "missing" ? "That commit is no longer in this repository. The review still reads in full, but it cannot be analysed again until the commit is back." : "This review is of one commit, and a commit does not change, so these findings keep describing it."
+	};
 	if (code.state === "current") return {
 		state: "current",
 		title: "Review up to date",
@@ -3379,9 +3399,10 @@ function renderWorkspace(view) {
 <body data-api-base="${escapeHtml$1(view.apiBase)}"${view.token ? ` data-token="${escapeHtml$1(view.token)}"` : ""}${view.editor ? ` data-editor-label="${escapeHtml$1(view.editor.label)}" data-editor-template="${escapeHtml$1(view.editor.template)}"` : ""}${view.repoPath ? ` data-repo-path="${escapeHtml$1(view.repoPath)}"` : ""}${view.revision === void 0 ? "" : ` data-workspace-revision="${escapeHtml$1(String(view.revision))}"`} data-session-id="${escapeHtml$1(view.session.id)}" data-explain-prompt="${escapeHtml$1(EXPLAIN_SIMPLY_PROMPT)}" data-way-back="${escapeHtml$1(wayBackSentence())}" data-default-view="${session.result ? "overview" : showsProgress(view) ? "progress" : "kiss"}">
 ${renderHeader(view, freshness)}
 ${showsProgress(view) ? "" : renderJobState(view.job ?? null)}
-<div class="prb-fresh-region" data-fresh-region${freshness ? ` data-report="${escapeHtml$1(JSON.stringify(freshness))}"` : ""}${view.dismissedFreshnessSha ? ` data-dismissed-sha="${escapeHtml$1(view.dismissedFreshnessSha)}"` : ""}>${renderFreshness(freshness ?? null, {
+<div class="prb-fresh-region" data-fresh-region${freshness ? ` data-report="${escapeHtml$1(JSON.stringify(freshness))}"` : ""}${view.availability ? ` data-availability="${escapeHtml$1(JSON.stringify(view.availability))}"` : ""}${view.dismissedFreshnessSha ? ` data-dismissed-sha="${escapeHtml$1(view.dismissedFreshnessSha)}"` : ""}>${renderFreshness(freshness ?? null, {
 		dismissedSha: view.dismissedFreshnessSha,
-		actions: view.token !== void 0
+		actions: view.token !== void 0,
+		availability: view.availability ?? null
 	})}</div>
 ${renderUpdateProgress(view.updateJob)}
 ${renderUpdateOutcome(view.lastUpdate)}
@@ -3661,6 +3682,25 @@ function lineageLeft(review) {
 	if (review.unansweredQuestions > 0) parts.push(`${review.unansweredQuestions} unanswered question${review.unansweredQuestions === 1 ? "" : "s"}`);
 	return parts.length === 0 ? "" : `<span class="prb-lineage-left">${escapeHtml$1(parts.join(", "))}</span>`;
 }
+/**
+* What the review is of, and what it was compared against.
+*
+* A commit review says `Commit` in words, because a short sha where a branch name usually sits
+* reads as a branch nobody has. A merge names the first parent, because which parent the diff was
+* taken against is the one fact about a merge's diff a reader cannot work out from the page.
+*/
+function renderSubject(changeSet) {
+	const arrow = "<span class=\"prb-arrow\" aria-label=\"compared against\">&rarr;</span>";
+	if (changeSet.targetType !== "commit") return `<strong>${escapeHtml$1(changeSet.headRef)}</strong>
+    ${arrow}
+    <span>${escapeHtml$1(changeSet.baseRef)}</span>`;
+	const sha = changeSet.data.pullRequest.head.sha;
+	const parent = changeSet.parentSha;
+	const against = !parent ? "the empty tree" : `${(changeSet.parentCount ?? 1) > 1 ? "first parent" : "parent"} <code>${escapeHtml$1(parent.slice(0, 7))}</code>`;
+	return `<strong>Commit <code>${escapeHtml$1(sha.slice(0, 7))}</code></strong>
+    ${arrow}
+    <span>${against}</span>`;
+}
 function renderHeader(view, freshness) {
 	const { changeSet } = view;
 	const additions = changeSet.data.files.reduce((sum, f) => sum + f.additions, 0);
@@ -3671,9 +3711,7 @@ function renderHeader(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>` : ""}
-    <strong>${escapeHtml$1(changeSet.headRef)}</strong>
-    <span class="prb-arrow" aria-label="compared against">&rarr;</span>
-    <span>${escapeHtml$1(changeSet.baseRef)}</span>
+    ${renderSubject(changeSet)}
     ${changeSet.includesUncommitted ? "<span class=\"prb-file-status\">+ working tree</span>" : ""}
     ${renderAuthor(view)}
   </span>
@@ -4046,7 +4084,7 @@ function renderDisclosure(view) {
 function renderFiles(view, sections) {
 	const files = view.changeSet.data.files;
 	const byPath = new Map(files.map((file) => [file.name, file]));
-	const references = collectReferences(view, sections);
+	const references = collectReferences(view);
 	const claimed = /* @__PURE__ */ new Set();
 	const groups = sections.map((section) => {
 		const sectionFiles = section.files.map((file) => byPath.get(file.path)).filter((file) => {
@@ -4102,7 +4140,7 @@ function referencedGroup(view, sections, references) {
 */
 function referencedRow(view, file, titles, refs) {
 	const { dir, name } = splitFilePath(file.path);
-	const badges = [...refs?.issues ? [`<span class="prb-file-ref prb-file-ref-issue">${refs.issues} issue${refs.issues === 1 ? "" : "s"}</span>`] : [], ...refs?.questions ? [`<span class="prb-file-ref">${refs.questions} question${refs.questions === 1 ? "" : "s"}</span>`] : []].join("");
+	const counts = fileCounts(refs);
 	const named = file.sectionIds.map((id) => titles.get(id)).filter((title) => Boolean(title));
 	const by = named.length > 0 ? `<span class="prb-file-referenced-by">Referenced by: ${escapeHtml$1(named.join(", "))}</span>` : "";
 	return `<div class="prb-file prb-file-expandable prb-file-referenced" data-file="${escapeHtml$1(file.path)}" role="button" tabindex="0" aria-expanded="false" data-search="${escapeHtml$1(file.path.toLowerCase())}" title="${escapeHtml$1(file.path)}">
@@ -4111,48 +4149,48 @@ function referencedRow(view, file, titles, refs) {
       <span class="pr-helper-path-name">${escapeHtml$1(name)}</span>
       ${by}
     </span>
-    ${badges ? `<span class="prb-file-badges">${badges}</span>` : ""}
+    ${counts}
+    <span class="prb-file-spacer" aria-hidden="true"></span>
   </div>`;
 }
 /**
+* How many findings point at a file, drawn the way the rail draws them: its issue and question
+* icons, each with a number.
+*
+* A column of its own before the line counts, rendered on every row whether or not it holds
+* anything, so the numbers line up down the list. Each count is announced as a sentence ("1 issue")
+* rather than a native tooltip, and the digits are hidden so they are not read twice.
+*/
+function fileCounts(refs) {
+	const count = (n, icon, noun) => n > 0 ? `<span class="prb-file-count prb-file-count-${icon}" role="img" aria-label="${n} ${noun}${n === 1 ? "" : "s"}">${iconSvg(icon)}<span aria-hidden="true">${n}</span></span>` : "";
+	return `<span class="prb-file-counts">${count(refs?.issues ?? 0, "issues", "issue")}${count(refs?.questions ?? 0, "questions", "question")}</span>`;
+}
+/**
 * Which findings point at each file.
 *
 * Worth showing on the row because it is the one thing the file list cannot say for itself: a file
 * with four issues against it and a file with none look identical by path, status and line count.
 */
-function collectReferences(view, sections) {
+function collectReferences(view) {
 	const result = /* @__PURE__ */ new Map();
-	const owner = /* @__PURE__ */ new Map();
-	for (const section of sections) for (const file of section.files) if (!owner.has(file.path)) owner.set(file.path, section.id);
-	const titles = new Map(sections.map((section) => [section.id, `${section.order}. ${section.title}`]));
-	const entry = (path) => {
-		const existing = result.get(path);
-		if (existing) return existing;
-		const created = {
-			issues: 0,
-			questions: 0,
-			otherSections: []
-		};
-		result.set(path, created);
-		return created;
-	};
-	const note = (path, sectionId, kind) => {
-		if (!path) return;
-		const record = entry(path);
-		record[kind] += 1;
-		const title = sectionId && sectionId !== owner.get(path) ? titles.get(sectionId) : void 0;
-		if (title && !record.otherSections.includes(title)) record.otherSections.push(title);
-	};
-	const noteRelated = (paths, kind) => {
-		for (const path of paths ?? []) entry(path)[kind] += 1;
+	const note = (paths, kind) => {
+		for (const path of paths) {
+			if (!path) continue;
+			const record = result.get(path) ?? {
+				issues: 0,
+				questions: 0
+			};
+			record[kind] += 1;
+			result.set(path, record);
+		}
 	};
 	for (const issue of view.session.result?.issues ?? []) {
-		note(issue.file, issue.sectionId, "issues");
-		noteRelated(issue.relatedFiles, "issues");
+		note([issue.file], "issues");
+		note(issue.relatedFiles ?? [], "issues");
 	}
 	for (const question of view.session.result?.questions ?? []) {
-		note(question.file, question.sectionId, "questions");
-		noteRelated(question.relatedFiles, "questions");
+		note([question.file], "questions");
+		note(question.relatedFiles ?? [], "questions");
 	}
 	return result;
 }
@@ -4165,25 +4203,21 @@ function fileGroup(view, heading, files, references) {
 }
 function fileRow(view, file, refs) {
 	const { dir, name } = splitFilePath(file.name);
-	const badges = [
-		...refs?.issues ? [`<span class="prb-file-ref prb-file-ref-issue">${refs.issues} issue${refs.issues === 1 ? "" : "s"}</span>`] : [],
-		...refs?.questions ? [`<span class="prb-file-ref">${refs.questions} question${refs.questions === 1 ? "" : "s"}</span>`] : [],
-		...(refs?.otherSections ?? []).map((title) => `<span class="prb-file-ref">also ${escapeHtml$1(title)}</span>`)
-	].join("");
+	const counts = fileCounts(refs);
 	const readable = !(file.additions === 0 && file.deletions === 0 && !file.patch);
 	const openEditor = !readable && view.editor ? `<button type="button" class="prb-file-open" data-open-editor="${escapeHtml$1(file.name)}" data-open-line="1" title="Open in ${escapeHtml$1(view.editor.label)}">Open in ${escapeHtml$1(view.editor.label)}</button>` : "";
 	const link = readable ? null : prFileLink(view, file.name);
 	const openPr = link ? `<a class="prb-file-open prb-file-pr" href="${escapeHtml$1(link.href)}" target="_blank" rel="noopener noreferrer" title="${escapeHtml$1(link.label)}">Pull request</a>` : "";
 	const attributes = readable ? ` data-file="${escapeHtml$1(file.name)}" role="button" tabindex="0" aria-expanded="false"` : "";
 	return `<div class="prb-file${readable ? " prb-file-expandable" : ""}"${attributes} data-search="${escapeHtml$1(file.name.toLowerCase())}" title="${escapeHtml$1(file.name)}">
-    <span class="prb-file-status">${escapeHtml$1(file.status)}</span>
+    <span class="prb-file-status" data-status="${escapeHtml$1(file.status)}">${escapeHtml$1(file.status)}</span>
     <span class="prb-file-path">
       ${dir ? `<span class="pr-helper-path-dir">${escapeHtml$1(dir)}</span>` : ""}
       <span class="pr-helper-path-name">${escapeHtml$1(name)}</span>
     </span>
     ${openPr}
     ${openEditor}
-    ${badges ? `<span class="prb-file-badges">${badges}</span>` : ""}
+    ${counts}
     <span class="prb-file-stat"><span class="prb-add">+${file.additions}</span> <span class="prb-del">&minus;${file.deletions}</span></span>
   </div>`;
 }
@@ -4561,7 +4595,7 @@ function renderCard(group, now, analysis, queued) {
 	const review = group.latest;
 	const status = statusOf(review);
 	const href = `/r/${encodeURIComponent(review.id)}?t=${encodeURIComponent(review.token)}`;
-	const followed = followedRefName(review.branch, review.headResolvedRef) ?? review.branch;
+	const followed = review.targetType === "commit" ? `commit ${(review.commitSha ?? review.branch).slice(0, 7)}` : followedRefName(review.branch, review.headResolvedRef) ?? review.branch;
 	return `<article class="prb-card" data-card data-id="${escapeHtml$1(review.id)}" data-status="${status}" data-pinned="${review.pinned}"${queued === void 0 ? "" : ` data-queue="${queued}"`}${review.authorHandle ? ` data-author="${escapeHtml$1(review.authorHandle)}"` : ""} data-reviews="${group.history.length}" data-search-text="${escapeHtml$1(groupSearchText(group))}">
     <input type="checkbox" class="prb-card-select" data-select="${escapeHtml$1(review.id)}" aria-label="Select ${escapeHtml$1(followed)}">
     <button type="button" class="prb-card-pin" data-pin="${escapeHtml$1(review.id)}" aria-pressed="${review.pinned}" aria-label="${review.pinned ? "Unsave" : "Save"} this review">${review.pinned ? "&#9733;" : "&#9734;"}</button>
@@ -4824,6 +4858,7 @@ function searchText(review) {
 		review.repoPath,
 		review.branch,
 		followedRefName(review.branch, review.headResolvedRef) ?? "",
+		review.targetType === "commit" ? review.commitSha ?? "" : "",
 		review.title,
 		review.summary,
 		review.author ?? "",
@@ -6571,7 +6606,8 @@ function recordWorkspaceOpened(workspace) {
 	if (event) record(event);
 }
 async function sendWorkspacePage(res, workspace) {
-	const freshness = await freshnessOrNull(workspace);
+	const standing = await standingOrNull(workspace);
+	const freshness = standing?.freshness ?? null;
 	const kissFreshness = await kissFreshnessOrNull(workspace);
 	const kissRun = latestKissRun(workspace);
 	const notice = workspace.lastUpdate?.acknowledged ? void 0 : workspace.lastUpdate;
@@ -6600,6 +6636,7 @@ async function sendWorkspacePage(res, workspace) {
 			processed: processDiscussion(workspace.prContext.discussion)
 		} : void 0,
 		freshness: freshness ?? void 0,
+		availability: standing?.availability ?? null,
 		dismissedFreshnessSha: workspace.dismissedFreshnessSha,
 		lastPrRefresh: workspace.lastPrRefresh,
 		lastUpdate: notice,
@@ -6710,7 +6747,8 @@ function jobReport(workspace) {
 */
 async function handleFreshness(res, workspace) {
 	const kiss = await kissFreshnessOrNull(workspace);
-	const report = await freshnessOrNull(workspace);
+	const standing = await standingOrNull(workspace);
+	const report = standing?.freshness ?? null;
 	const revision = workspaceRevision(workspace) ?? null;
 	if (!report) return send(res, 200, {
 		report: null,
@@ -6727,6 +6765,7 @@ async function handleFreshness(res, workspace) {
 	});
 	send(res, 200, {
 		report,
+		availability: standing?.availability ?? null,
 		dismissedSha: workspace.dismissedFreshnessSha ?? null,
 		kiss,
 		kissDismissedSha: workspace.dismissedKissFreshnessSha ?? null,
@@ -6895,9 +6934,9 @@ async function handleKiss(req, res, workspace) {
 * to serve a review that is sitting complete on disk. The page renders without the banner, which
 * reads as "not checked" rather than as "nothing has changed".
 */
-async function freshnessOrNull(workspace) {
+async function standingOrNull(workspace) {
 	try {
-		return await checkFreshness(workspace);
+		return await checkReview(workspace);
 	} catch {
 		return null;
 	}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prreviewbuddy",
-  "version": "0.30.0",
+  "version": "0.31.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

@@ -177,11 +177,16 @@ body > * { flex: 0 0 auto; }
   border-bottom: 1px solid var(--border);
 }
 
+/* One line, always. The header is a fixed height, and anything that wrapped inside it (`Commit
+   2e8bf41`, `parent 39ad3d3`) broke across two lines in a bar with room for one. On a narrow
+   window this clips rather than wraps; the author's name already ellipsizes inside it. */
 .prb-change-ref {
   display: flex;
   align-items: center;
   gap: 8px;
   min-width: 0;
+  white-space: nowrap;
+  overflow: hidden;
   font-family: ui-monospace, SFMono-Regular, 'SF Mono', Menlo, monospace;
   font-size: 12.5px;
   color: var(--text-secondary);
@@ -197,6 +202,9 @@ body > * { flex: 0 0 auto; }
   font-size: 12.5px;
   color: var(--text-secondary);
 }
+/* Each control on one line, for the same reason as the change ref. Not the `...` menu: its panel is
+   a fixed-width box hanging below the header, and its items have to wrap inside it. */
+.prb-header-stats > :not(.prb-menu) { white-space: nowrap; }
 .prb-header-stats .prb-add { color: var(--success); }
 .prb-header-stats .prb-del { color: var(--danger); }
 
@@ -846,17 +854,18 @@ a.prb-rail-item { text-decoration: none; }
 /* Directory above, filename below in purple, which is how every other file citation in the
    product is drawn. The classes are the extension's own so the two cannot drift apart. */
 .prb-file-path { flex: 1 1 auto; min-width: 0; line-height: 1.35; }
-/* The pills are a column of their own, sitting between the buttons and the line counts rather
-   than trailing the path. Read down the list they answer one question at a glance: which files
-   have something to say about them. Trailing the path they moved with every filename length. */
-.prb-file-badges {
-  display: flex;
+/* A file's issue and question counts: the rail's own icons, each with a number, in a column before
+   the line counts. A fixed width that is rendered on every row, empty or not, so the numbers read
+   down the list as a column and never take width from the filename. */
+.prb-file-counts {
   flex: none;
-  flex-wrap: wrap;
-  align-items: center;
+  display: flex;
   justify-content: flex-end;
-  gap: 6px;
-  min-width: 0;
+  align-items: center;
+  gap: 10px;
+  min-width: 64px;
+  font-size: 11.5px;
+  font-variant-numeric: tabular-nums;
 }
 
 /* What the path alone cannot say: a file with four issues against it and one with none look
@@ -867,22 +876,14 @@ a.prb-rail-item { text-decoration: none; }
    --surface / --border / --text-* tokens are the shell's, so a chip styled with them renders as a
    dark box in the middle of a light list — legible in neither theme, because it belongs to the
    other one. Use the pane's own literals, as .prb-file-status and .prb-file-name already do. */
-.prb-file-ref {
-  flex: none;
-  padding: 1px 6px;
-  border-radius: 999px;
-  border: 1px solid rgba(15, 23, 42, 0.15);
-  background: rgba(15, 23, 42, 0.04);
+.prb-file-count {
+  display: inline-flex;
+  align-items: center;
+  gap: 3px;
   color: #57606a;
-  font-family: inherit;
-  font-size: 10.5px;
-  letter-spacing: 0.01em;
-}
-.prb-file-ref-issue {
-  border-color: rgba(207, 34, 46, 0.3);
-  background: rgba(207, 34, 46, 0.07);
-  color: #cf222e;
 }
+.prb-file-count svg { width: 13px; height: 13px; flex: none; }
+.prb-file-count-issues { color: #cf222e; }
 
 /* Files the review leant on that this change does not touch. Pane literals throughout, per the
    note above.
@@ -971,10 +972,17 @@ a.prb-rail-item { text-decoration: none; }
   font-variant-numeric: tabular-nums;
   font-size: 11.5px;
 }
+/* The line-count column's width, on a referenced row that has no line counts, so its finding counts
+   still line up with the changed files above it. Kept in step with `.prb-file-stat`'s min-width. */
+.prb-file-spacer { flex: none; min-width: 74px; }
 .prb-file-stat .prb-add { color: #1a7f37; }
 .prb-file-stat .prb-del { color: #cf222e; }
+/* One width for every status, so the path beside it starts at the same edge on every row. Sized for
+   the longest word, UNCHANGED, plus the padding, whichever box-sizing is in force. */
 .prb-file-status {
   flex: none;
+  min-width: calc(10ch + 10px);
+  text-align: center;
   padding: 0 5px;
   border-radius: 3px;
   background: rgba(175, 184, 193, 0.24);
@@ -983,6 +991,16 @@ a.prb-rail-item { text-decoration: none; }
   text-transform: uppercase;
   letter-spacing: 0.04em;
 }
+/* One colour per kind of change, so a new file and an edited one are told apart without reading
+   the word. Added and removed take the pane's own `+` and `−` colours, which the row's line counts
+   already use for the same meaning. Pane literals, per the palette note above. `unchanged` keeps
+   the grey. */
+.prb-file-status[data-status='added'] { background: rgba(26, 127, 55, 0.12); color: #1a7f37; }
+.prb-file-status[data-status='removed'] { background: rgba(207, 34, 46, 0.1); color: #cf222e; }
+.prb-file-status[data-status='modified'],
+.prb-file-status[data-status='changed'] { background: rgba(212, 167, 44, 0.18); color: #9a6700; }
+.prb-file-status[data-status='renamed'],
+.prb-file-status[data-status='copied'] { background: rgba(9, 105, 218, 0.1); color: #0969da; }
 
 /* ── Ask panel ── */
 .prb-ask {