prreviewbuddy 0.48.0 → 0.50.0

package/dist/server.js

@@ -1,4 +1,4 @@
-import { $t as mutateWorkspace, A as createPrState, An as describeAuthorship, At as allJobs, B as toneOf, Bt as purposeOf, C as readEvents, Cn as doneVerb, Ct as forgeFor, D as announceJob, Dn as questionsOutstanding, Dt as checkReview, E as telemetryUploadConsent, En as issuesOutstanding, Et as checkCodeFreshness, F as markAllRead, Ft as saveJob, G as extractJsonText, Gn as agentFor, Gt as groupByLineage, H as withUsageRecorded, Hn as AGENT_IDS, I as markRead, In as resolveModel, J as readMarker, Jn as AgentUnavailableError, Jt as latestKissRun, K as ReviewBeingDeletedError, Kt as indexKeptIds, L as pruneNotifications, Ln as clearAgentPreference, Lt as isLiveJob, M as eventName, Mt as fail, N as headline, Nn as isCommitReview, Nt as isTerminal, O as branchOf, On as reviewerDispositions, Ot as followedBranch, P as listNotifications, Pn as git, Pt as loadJob, Q as compactQueue, R as readNotification, Rn as readAgentPreference, Rt as phasesFor, Sn as readQueue, T as recordTelemetryUploadConsent, Tn as isQuestionOutstanding, Tt as fillFileUrlTemplate, U as EXPLAIN_SIMPLY_PROMPT, Ut as isClaimed, V as unreadCount$1, Vn as STORE_ROOT, W as asString, Wt as followedRefName, Xt as lineagePosition, Y as removeWorktree, Z as relativeTime$1, Zt as loadWorkspace, _ as agentEnvOf, _n as dequeueOnComplete, _t as parseWorkspacePort, a as wasBlocked, bn as queueRemove, bt as writePortPreference, c as RefreshUnavailableError, cn as workspaceStamps, d as refreshPrContext, dn as isLifecycle, dt as BUILD_VERSION, et as readIndexToken, f as isUnchanged, fn as lifecycleAction, gn as sectionOf, gt as MIN_WORKSPACE_PORT, h as liveUpdateFor, hn as parseLifecycle, ht as MAX_WORKSPACE_PORT, i as liveJobsFor, it as readServerRecord, j as createPrUpdate, jt as endedAt, k as subjectOf, kn as processDiscussion, kt as reviewedShaOf, l as mergeUpdateInto, ln as LIFECYCLES, m as UpdateAlreadyRunningError, mn as offersLifecycle, mt as DEFAULT_WORKSPACE_PORT, n as deleteReview, nn as recentWorkspaces, nt as clearServerRecord, o as discardJob, on as touchWorkspace, ot as serverEntryExists, p as askCheckout, pn as lifecycleChip, pt as feedbackUrl, q as MANAGED_ROOT, qn as AgentCancelledError, qt as indexReviewGroups, r as lineageIds, rn as reviewedCommit, s as FETCH_TIMEOUT_MS, sn as workspaceRevision, st as spawnServer, t as deleteLineage, tn as previousKissResult, u as updateReview, un as activityLabel, ut as writeServerRecord, vn as queueAdd, vt as resolveWorkspacePort, w as record, wn as isIssueOutstanding, wt as remoteHost, x as runJob, xn as queueStamp, xt as claimRequestUrl, y as startKissJob, yn as queueAddMany, yt as statedWorkspacePort, z as removeNotification, zn as writeAgentPreference, zt as progressSteps } from "./delete_review-D52wkt66.js";
+import { $ as compactQueue, A as createPrState, An as processDiscussion, At as reviewedShaOf, B as toneOf, Bn as writeAgentPreference, Bt as progressSteps, C as readEvents, Cn as readQueue, D as announceJob, Dn as issuesOutstanding, Dt as checkCodeFreshness, E as telemetryUploadConsent, En as isQuestionOutstanding, Et as fillFileUrlTemplate, F as markAllRead, Fn as git, Ft as loadJob, G as extractJsonText, Gt as followedRefName, H as withUsageRecorded, Hn as STORE_ROOT, I as markRead, It as saveJob, J as readMarker, Jn as AgentCancelledError, Jt as indexReviewGroups, K as ReviewBeingDeletedError, Kn as agentFor, Kt as groupByLineage, L as pruneNotifications, Ln as resolveModel, M as eventName, Mt as endedAt, N as headline, Nt as fail, O as branchOf, On as questionsOutstanding, Ot as checkReview, P as listNotifications, Pn as isCommitReview, Pt as isTerminal, Q as activateQueued, Qt as loadWorkspace, R as readNotification, Rn as clearAgentPreference, Rt as isLiveJob, Sn as queueStamp, St as claimRequestUrl, T as recordTelemetryUploadConsent, Tn as isIssueOutstanding, Tt as remoteHost, U as EXPLAIN_SIMPLY_PROMPT, Un as AGENT_IDS, V as unreadCount$1, Vt as purposeOf, W as asString, Wt as isClaimed, Y as removeWorktree, Yn as AgentUnavailableError, Yt as latestKissRun, Z as relativeTime$1, Zt as lineagePosition, _ as agentEnvOf, _n as parseLifecycle, _t as MIN_WORKSPACE_PORT, a as wasBlocked, at as readServerRecord, bn as queueAddMany, bt as statedWorkspacePort, c as RefreshUnavailableError, cn as workspaceRevision, ct as spawnServer, d as refreshPrContext, dn as activityLabel, dt as writeServerRecord, en as mutateWorkspace, f as isUnchanged, fn as isLifecycle, ft as BUILD_VERSION, gn as offersLifecycle, gt as MAX_WORKSPACE_PORT, h as liveUpdateFor, hn as lifecyclePill, ht as DEFAULT_WORKSPACE_PORT, i as liveJobsFor, in as reviewedCommit, j as createPrUpdate, jn as describeAuthorship, jt as allJobs, k as subjectOf, kn as reviewerDispositions, kt as followedBranch, l as mergeUpdateInto, ln as workspaceStamps, m as UpdateAlreadyRunningError, mn as lifecycleChip, mt as feedbackUrl, n as deleteReview, nn as previousKissResult, o as discardJob, p as askCheckout, pn as lifecycleAction, q as MANAGED_ROOT, qt as indexKeptIds, r as lineageIds, rn as recentWorkspaces, rt as clearServerRecord, s as FETCH_TIMEOUT_MS, sn as touchWorkspace, st as serverEntryExists, t as deleteLineage, tt as readIndexToken, u as updateReview, un as LIFECYCLES, vn as sectionOf, vt as parseWorkspacePort, w as record, wn as doneVerb, wt as forgeFor, x as runJob, xn as queueRemove, xt as writePortPreference, y as startKissJob, yn as queueAdd, yt as resolveWorkspacePort, z as removeNotification, zn as readAgentPreference, zt as phasesFor } from "./delete_review-BIXqUnpx.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";
@@ -253,6 +253,14 @@ function saveItemLabel(saved) {
 	return saved ? "Remove from saved" : "Save review";
 }
 /**
+* The queue item, on a card and on the review's own page. "review queue" rather than "queue",
+* because a menu is where somebody meets the idea for the first time. Here because the browser
+* relabels it after a change, on both pages.
+*/
+function queueItemLabel(queued) {
+	return queued ? "Remove from review queue" : "Add to review queue";
+}
+/**
 * What `Continue reviewing` opens, and whether it exists at all.
 *
 * One rule rather than an expression in the template, because "is there anything to continue" and
@@ -363,6 +371,7 @@ var PATHS = {
 	reviews: "<path d=\"M8 6h13M8 12h13M8 18h13M3 6h.01M3 12h.01M3 18h.01\"/>",
 	notifications: "<path d=\"M18 8a6 6 0 0 0-12 0c0 7-3 9-3 9h18s-3-2-3-9\"/><path d=\"M13.7 21a2 2 0 0 1-3.4 0\"/>",
 	settings: "<circle cx=\"12\" cy=\"12\" r=\"3\"/><path d=\"M19.4 15a1.7 1.7 0 0 0 .3 1.8l.1.1a2 2 0 1 1-2.8 2.8l-.1-.1a1.7 1.7 0 0 0-1.8-.3 1.7 1.7 0 0 0-1 1.5V21a2 2 0 1 1-4 0v-.1a1.7 1.7 0 0 0-1.1-1.5 1.7 1.7 0 0 0-1.8.3l-.1.1a2 2 0 1 1-2.8-2.8l.1-.1a1.7 1.7 0 0 0 .3-1.8 1.7 1.7 0 0 0-1.5-1H3a2 2 0 1 1 0-4h.1a1.7 1.7 0 0 0 1.5-1.1 1.7 1.7 0 0 0-.3-1.8l-.1-.1a2 2 0 1 1 2.8-2.8l.1.1a1.7 1.7 0 0 0 1.8.3H9a1.7 1.7 0 0 0 1-1.5V3a2 2 0 1 1 4 0v.1a1.7 1.7 0 0 0 1 1.5 1.7 1.7 0 0 0 1.8-.3l.1-.1a2 2 0 1 1 2.8 2.8l-.1.1a1.7 1.7 0 0 0-.3 1.8V9a1.7 1.7 0 0 0 1.5 1H21a2 2 0 1 1 0 4h-.1a1.7 1.7 0 0 0-1.5 1z\"/>",
+	help: "<circle cx=\"12\" cy=\"12\" r=\"10\"/><path d=\"M9.1 9a3 3 0 0 1 5.8 1c0 2-3 3-3 3\"/><path d=\"M12 17h.01\"/>",
 	feedback: "<path d=\"m22 2-7 20-4-9-9-4z\"/><path d=\"M22 2 11 13\"/>",
 	stop: "<path d=\"M12 2v10\"/><path d=\"M18.4 6.6a9 9 0 1 1-12.8 0\"/>",
 	restart: "<path d=\"M21 12a9 9 0 1 1-2.64-6.36\"/><path d=\"M21 3v6h-6\"/>"
@@ -436,6 +445,12 @@ var WORKSPACE_PLACES = [
 		href: "/settings",
 		icon: "settings",
 		label: "Settings"
+	},
+	{
+		place: "help",
+		href: "/help",
+		icon: "help",
+		label: "Help"
 	}
 ];
 /**
@@ -4698,9 +4713,30 @@ function renderReviewBody(view, sections) {
     ${updating(view) ? renderUpdatingDoc(view) : renderReviewDoc(view, sections)}
   </main>
   ${result ? renderAsk(view, sections, updating(view)) : ""}
+  ${result ? renderAskTab() : ""}
 </div>`;
 }
 /**
+* Buddy AI's own way in: a tab on the window's right edge, where its pane opens.
+*
+* Out of the header, which says where the review stands; the assistant is a tool beside the review,
+* and the edge it lives on is the edge this sits on. Hidden by the stylesheet whenever the pane is
+* showing, where the chevron in the pane's own header closes it again. The pane's left border is
+* not free for it: the resize handle runs its full height.
+*
+* Horizontal, and only as wide as its words: the product reads left to right everywhere else, and
+* a vertical label was a pattern it would have used exactly once.
+*
+* Only with a standard review, like the pane. A control that opens nothing is worse than none.
+*/
+function renderAskTab() {
+	return `<button type="button" class="prb-ask-tab" data-ask-toggle aria-controls="prb-ask" aria-expanded="false">${ASK_TAB_ICON}<span class="prb-ask-tab-label">Buddy AI</span></button>`;
+}
+/** Four-pointed, stroked like the rail's icons, so it reads as one of the product's own marks. */
+var ASK_TAB_ICON = "<svg class=\"prb-ask-tab-icon\" viewBox=\"0 0 24 24\" width=\"14\" height=\"14\" focusable=\"false\" aria-hidden=\"true\" fill=\"none\" stroke=\"currentColor\" stroke-width=\"1.75\" stroke-linejoin=\"round\"><path d=\"M12 3l1.8 5.6a2 2 0 0 0 1.3 1.3L21 12l-5.9 2.1a2 2 0 0 0-1.3 1.3L12 21l-1.8-5.6a2 2 0 0 0-1.3-1.3L3 12l5.9-2.1a2 2 0 0 0 1.3-1.3z\"/></svg>";
+/** Two chevrons pointing where the pane goes. */
+var ASK_HIDE_ICON = "<svg viewBox=\"0 0 24 24\" width=\"15\" height=\"15\" focusable=\"false\" aria-hidden=\"true\" fill=\"none\" stroke=\"currentColor\" stroke-width=\"2\" stroke-linecap=\"round\" stroke-linejoin=\"round\"><path d=\"m6 17 5-5-5-5M13 17l5-5-5-5\"/></svg>";
+/**
 * The document while an update runs: its progress and nothing else.
 *
 * Deliberately not a `.prb-view`. Views are what the rail and the hash switch between, and a page
@@ -4982,6 +5018,43 @@ function renderSubject(changeSet) {
     <span>${against}</span>`;
 }
 /**
+* Which commits this review is of, opened from the bar beside what it compares.
+*
+* Beside `.prb-change-ref` rather than inside it, because that element clips to keep the bar one
+* line and a panel inside it would be cut off. The pill's own label is a fact (how many commits),
+* so the bar gains information even closed. Facts only, and about this review: the general
+* explanation is the Help page's, linked when there is a server to answer the link.
+*
+* The count comes from the stored commit list, or from the authors' counts for a review made before
+* the list was recorded, which then says how many without listing them. Neither: no count at all,
+* never a zero.
+*/
+function renderRange(changeSet, live) {
+	const commits = changeSet.commits;
+	const count = commits?.length ?? changeSet.authors?.reduce((sum, author) => sum + author.commits, 0);
+	const counted = count !== void 0 && count > 0 ? count : void 0;
+	const plural = (n) => `${n} commit${n === 1 ? "" : "s"}`;
+	const sha = (value) => `<code>${escapeHtml$1(value.slice(0, 7))}</code>`;
+	const lines = [];
+	if (changeSet.targetType === "commit") {
+		const parent = changeSet.parentSha;
+		const against = !parent ? "the empty tree" : `its ${(changeSet.parentCount ?? 1) > 1 ? "first parent" : "parent"} ${sha(parent)}`;
+		lines.push(`The change commit ${sha(changeSet.data.pullRequest.head.sha)} made, compared against ${against}.`);
+	} else {
+		lines.push(changeSet.mergeBase ? `Compared against <code>${escapeHtml$1(changeSet.baseRef)}</code>, from where this branch split off at ${sha(changeSet.mergeBase)}.` : `Compared against <code>${escapeHtml$1(changeSet.baseRef)}</code>.`);
+		if (commits && commits.length > 0) {
+			const more = commits.length > 10 ? ` and ${commits.length - 10} more` : "";
+			lines.push(`Changes across ${plural(commits.length)}: ${commits.slice(0, 10).map(sha).join(" ")}${more}.`);
+		} else if (counted) lines.push(`Changes across ${plural(counted)}.`);
+	}
+	lines.push(changeSet.includesUncommitted ? "This review also read uncommitted changes in the working tree." : "Uncommitted changes are never part of a review.");
+	if (live) lines.push("<a href=\"/help#what-gets-reviewed\">How review ranges work</a>");
+	return `<details class="prb-range" data-range-details>
+    <summary class="prb-range-summary">${changeSet.targetType === "commit" ? "1 commit" : counted ? plural(counted) : "Range"}</summary>
+    <div class="prb-range-panel">${lines.map((line) => `<p>${line}</p>`).join("")}</div>
+  </details>`;
+}
+/**
 * The review's own bar, below the global strip.
 *
 * Everything here is about this review and nothing else is: the strip above carries the workspace's
@@ -5002,47 +5075,75 @@ function renderReviewBar(view, freshness) {
     ${changeSet.includesUncommitted ? "<span class=\"prb-file-status\">+ working tree</span>" : ""}
     ${renderAuthor(view)}
   </span>
+  ${renderRange(changeSet, Boolean(view.token))}
   <div class="prb-header-stats">
     <span>${fileCount} file${fileCount === 1 ? "" : "s"}</span>
     <span><span class="prb-add">+${additions}</span> <span class="prb-del">&minus;${deletions}</span></span>
     ${freshness || view.remote ? (() => {
 		const { label, state } = freshnessLabel(freshness ?? null, view.remote ?? null);
-		return `<span class="prb-fresh-status" data-fresh-status data-state="${state}">${escapeHtml$1(label)}</span>${view.token ? "<button type=\"button\" class=\"prb-fresh-check\" data-fresh-check>Check for updates</button>" : ""}`;
+		return `<span class="prb-fresh-status" data-fresh-status data-state="${state}">${escapeHtml$1(label)}</span>`;
 	})() : ""}
-    ${view.token ? `<button type="button" class="prb-complete" data-complete aria-pressed="${view.complete === true}">${view.complete ? "Reviewed" : "Mark reviewed"}</button>` : ""}
-    ${view.id ? renderIdMenu(view.id, Boolean(view.token), view.saved === true) : ""}
-    ${view.session.result ? "<button type=\"button\" class=\"prb-ask-toggle\" data-ask-toggle aria-expanded=\"false\">Buddy AI</button>" : ""}
+    ${view.token ? renderStatePills(view) : ""}
+    ${view.id ? renderIdMenu(view, freshness) : ""}
   </div>
 </div>`;
 }
 /**
-* This review's ID, where somebody looking at this review would think to find it.
+* Where this review stands, as pills that say and never do.
 *
-* In the header because the header answers "what am I looking at", and an ID is the most literal
-* possible answer to that: it is the name `prreviewbuddy review --review <review> --delete` takes.
-* It was on the index only, so the one page devoted to a single review was the only place that
-* could not tell you which review it was, and the help text's advice amounted to reading it out of
-* the address bar.
+* Two, because they answer two questions. Reviewed or In progress is what the reviewer has done;
+* Waiting or Done is what happens next with the branch. A review can be Reviewed and Waiting, so a
+* single pill would have to drop one of them. Active draws nothing, as an open pull request does
+* not: it is where every review starts, and a pill that never varies teaches nothing.
 *
-* The same `...` and the same menu markup as the index, deliberately. One gesture, in both places
-* it can be wanted, behaving identically. That includes deleting, with the same confirmation in the
-* same panel and the same sentence. The list is where you land afterwards, since the page you were
-* reading no longer exists. It deletes this review only, as a history row on the list does.
+* Never "New" here, because being on this page is what ends it.
+*/
+function renderStatePills(view) {
+	const status = view.complete ? "complete" : "in-progress";
+	const pill = lifecyclePill(view.lifecycle, view.activity);
+	return `<span class="prb-card-status" data-status="${status}" data-page-status>${statusLabel(status)}</span>
+    <span class="prb-card-status prb-page-lifecycle" data-page-lifecycle${pill?.changed ? " data-changed" : ""}${pill ? "" : " hidden"}>${escapeHtml$1(pill?.text ?? "")}</span>`;
+}
+/**
+* Everything that can be done to this review, from the page you are reading it on.
+*
+* The header only says where the review stands; this is where that is changed. It holds what the
+* Reviews list's card menu holds, in the same words from the same registries, so nobody has to go
+* back to the list to park or queue the review they are in. Check for updates leads because it is
+* the one thing here the list does not offer, and it is here rather than in the header because the
+* page already checks by itself, every minute while visible and on coming back to the tab.
 *
-* No delete where there is no server, which is the offline preview: the item would be a lie there.
-* Save follows the same rule, since saving is a write to the store the preview does not have. It
-* is the index's star, reachable from the page you are reading, so its words are the star's
-* `aria-pressed` pair spelled out: what pressing it will do, never the state it is in.
+* Waiting, Done and the queue belong to the branch, so they are offered on its newest review only,
+* as on the list, where the older reviews' rows carry none. The lineage notice already points at
+* the newest one.
+*
+* Delete keeps the index's confirmation, in the same panel with the same sentence. The list is
+* where you land afterwards, since the page you were reading no longer exists.
+*
+* Nothing that writes where there is no server, which is the offline preview: each item would be a
+* lie there. Copy review ID stays, because it is the name `prreviewbuddy review --review <review>
+* --delete` takes, and the page devoted to one review should be able to say which one it is.
 */
-function renderIdMenu(id, canWrite, saved) {
-	const safe = escapeHtml$1(id);
-	return `<div class="prb-menu" data-menu>
+function renderIdMenu(view, freshness) {
+	const safe = escapeHtml$1(view.id ?? "");
+	const canWrite = Boolean(view.token);
+	const latest = !view.lineage || view.lineage.position >= view.lineage.total;
+	const mark = view.complete ? "in-progress" : "complete";
+	const branchItems = canWrite && latest;
+	const items = [
+		canWrite && (freshness || view.remote) ? "<button type=\"button\" class=\"prb-menu-item\" data-fresh-check>Check for updates</button>" : "",
+		canWrite ? `<button type="button" class="prb-menu-item" data-mark-status="${mark}">${markItemLabel(mark)}</button>` : "",
+		...branchItems ? LIFECYCLES.map(({ id }) => `<button type="button" class="prb-menu-item" data-lifecycle-set="${id}"${offersLifecycle(id, view.lifecycle, view.activity) ? "" : " hidden"}>${escapeHtml$1(lifecycleAction(id))}</button>`) : [],
+		branchItems ? `<button type="button" class="prb-menu-item" data-queue-action="${view.queued ? "remove" : "add"}">${queueItemLabel(view.queued === true)}</button>` : "",
+		canWrite ? `<button type="button" class="prb-menu-item" data-save-toggle aria-pressed="${view.saved === true}">${saveItemLabel(view.saved === true)}</button>` : "",
+		`<button type="button" class="prb-menu-item" data-copy-id="${safe}">Copy review ID</button>`,
+		canWrite ? `<button type="button" class="prb-menu-item prb-menu-item-danger" data-delete="${safe}">Delete review</button>` : ""
+	].filter(Boolean);
+	return `<div class="prb-menu" data-menu data-review-menu${[view.lifecycle ? ` data-lifecycle-state="${view.lifecycle.state}" data-lifecycle-at="${view.lifecycle.at}"` : "", view.activity ? ` data-activity-at="${view.activity.at}" data-activity-kind="${view.activity.kind}"` : ""].join("")}>
     <button type="button" class="prb-menu-trigger" data-menu-trigger aria-expanded="false" aria-haspopup="true" aria-label="Actions for this review">&#8943;</button>
     <div class="prb-menu-panel" data-menu-panel hidden>
       <div data-menu-items>
-        ${canWrite ? `<button type="button" class="prb-menu-item" data-save-toggle aria-pressed="${saved}">${saveItemLabel(saved)}</button>` : ""}
-        <button type="button" class="prb-menu-item" data-copy-id="${safe}">Copy review ID</button>
-        ${canWrite ? `<button type="button" class="prb-menu-item prb-menu-item-danger" data-delete="${safe}">Delete review</button>` : ""}
+        ${items.join("\n        ")}
       </div>
       ${canWrite ? `<div class="prb-menu-confirm" data-menu-confirm hidden>
         <p class="prb-menu-question">Delete this review?</p>
@@ -5059,7 +5160,7 @@ function renderIdMenu(id, canWrite, saved) {
 * Everything below the review: where else to go, and what to do to the server.
 *
 * These are in the rail rather than the header because they are not about this review. The header
-* answers "what am I looking at" — branch, size, whether I am done with it, and Buddy AI — and
+* answers "what am I looking at" — branch, size, where the review stands — and
 * anything global sitting up there competed with that. Leaving the review is navigation, and it
 * belongs with the other navigation.
 *
@@ -5569,7 +5670,7 @@ function renderAsk(view, sections, paused = false) {
 	const pending = view.pendingAsk ?? null;
 	const pendingTurn = pending ? `<div class="prb-msg prb-msg-user">${escapeHtml$1(pending.question ?? "")}</div>
       <div class="prb-thinking" data-ask-pending><i></i><i></i><i></i><span>Reading the change…</span></div>` : "";
-	return `<aside class="prb-ask" aria-label="Ask Buddy AI">
+	return `<aside class="prb-ask" id="prb-ask" aria-label="Ask Buddy AI">
   <div
     class="prb-ask-resize"
     data-ask-resize
@@ -5579,9 +5680,12 @@ function renderAsk(view, sections, paused = false) {
     tabindex="0"
   ></div>
   <div class="prb-ask-header">
-    <h2>Ask Buddy AI</h2>
-    <span class="prb-ask-scope" data-ask-scope>whole change</span>
+    <hgroup class="prb-ask-heading">
+      <h2>Ask Buddy AI</h2>
+      <p class="prb-ask-scope">Asking about: <span data-ask-scope>the whole change</span></p>
+    </hgroup>
     ${renderFocusToggle("buddy", "Buddy AI")}
+    <button type="button" class="prb-ask-hide" data-ask-toggle aria-controls="prb-ask" aria-label="Hide Buddy AI">${ASK_HIDE_ICON}</button>
   </div>
   ${paused ? "" : renderPromptBank(view.prompts ?? [], view.promptsCollapsed === true)}
   <div class="prb-ask-log" data-ask-log>
@@ -7034,7 +7138,7 @@ function renderCardMenu(group, followed, queued, status, activity) {
 function renderMenu(item) {
 	const id = escapeHtml$1(item.id);
 	const markItem = item.mark === void 0 ? "" : `<button type="button" class="prb-menu-item" data-mark-status="${item.mark}" data-mark-id="${id}">${markItemLabel(item.mark)}</button>`;
-	const queueItem = item.queue === void 0 ? "" : `<button type="button" class="prb-menu-item" data-queue-action="${item.queue}" data-queue-id="${id}">${item.queue === "remove" ? "Remove from review queue" : "Add to review queue"}</button>`;
+	const queueItem = item.queue === void 0 ? "" : `<button type="button" class="prb-menu-item" data-queue-action="${item.queue}" data-queue-id="${id}">${queueItemLabel(item.queue === "remove")}</button>`;
 	const lifecycleItems = item.lifecycle === void 0 ? "" : LIFECYCLES.map(({ id: target }) => `<button type="button" class="prb-menu-item" data-lifecycle-set="${target}" data-lifecycle-id="${id}"${offersLifecycle(target, item.lifecycle.stored, item.lifecycle.activity) ? "" : " hidden"}>${escapeHtml$1(lifecycleAction(target))}</button>`).join("\n        ");
 	return `<div class="prb-menu" data-menu>
     <button type="button" class="prb-menu-trigger" data-menu-trigger aria-expanded="false" aria-haspopup="true" aria-label="${escapeHtml$1(item.label)}">&#8943;</button>
@@ -7843,6 +7947,87 @@ function defaultFetchDeps() {
 	};
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/help_page.ts
+/** The diagrams are text on purpose: they are the same ones `prreviewbuddy --help` prints. */
+var TOPICS = [{
+	id: "what-gets-reviewed",
+	title: "What gets reviewed",
+	body: `<p>A review is the combined change between a base and the branch you review, shown as one
+    diff. It is not just the latest commit. The base determines the split point: the commit where
+    your branch left it (git calls this the merge-base). PR Review Buddy reviews the changes on your
+    branch since that point.</p>
+<pre class="prb-help-diagram">main:      A---B---C
+                \\
+feature:         D---E</pre>
+    <p>Reviewing <code>feature</code> against <code>main</code> covers D and E together. C is on
+    main but not on your branch, so it is not part of the review, and the review does not show its
+    changes as removed. Commits already in the base are never reviewed again.</p>
+
+    <h3>Local main and origin/main</h3>
+    <p>Your local <code>main</code> can hold commits that are not on <code>origin/main</code> yet,
+    such as a merge you have not pushed. When the two are different commits, the one you choose
+    moves the split point.</p>
+<pre class="prb-help-diagram">origin/main:   A---B
+                    \\
+local main:          C---D
+                          \\
+feature:                   E</pre>
+    <ul>
+      <li>Against local <code>main</code>, the review covers E.</li>
+      <li>Against <code>origin/main</code>, it covers C, D and E together.</li>
+    </ul>
+    <p>When nobody names a base and the two differ, PR Review Buddy does not guess. It asks which
+    one you mean and says which commits each would review.</p>
+
+    <h3>Uncommitted changes</h3>
+    <p>Never included. A review is a fixed snapshot of your commits, made in a checkout of its own,
+    so work you have not committed is not part of it.</p>
+
+    <h3>Choosing a base</h3>
+    <p>The base is chosen when the review is made, in the terminal:</p>
+    <ul>
+      <li><code>prreviewbuddy review --base refs/heads/main</code> for everything on your branch since it left your local main</li>
+      <li><code>prreviewbuddy review --base origin/main</code> for everything on your branch not yet on origin/main</li>
+      <li><code>prreviewbuddy review --commit &lt;commit&gt;</code> for one commit, against its parent</li>
+    </ul>
+    <p>Every review shows its own range. The commit count beside the branch name at the top of a
+    review opens its split point and the commits it covers.</p>`
+}];
+function renderHelpPage(view) {
+	return `<!doctype html>
+<html lang="en"${themeAttribute(view.theme)}>
+<head>
+<meta charset="utf-8">
+<meta name="viewport" content="width=device-width, initial-scale=1">
+<title>Help · PR Review Buddy</title>
+<link rel="icon" type="image/png" href="/assets/icon128.png">
+<link rel="stylesheet" href="/assets/sidebar.css">
+<link rel="stylesheet" href="/assets/shell.css">
+</head>
+<body class="prb-index-body" data-way-back="${escapeHtml$1(wayBackSentence())}">
+${renderGlobalBar({
+		current: "help",
+		unread: view.unreadNotifications ?? 0,
+		driving: view.driving
+	})}
+${renderContextBar("Help", "")}
+<div class="prb-shell">
+${renderWorkspaceRail({
+		current: "help",
+		unread: view.unreadNotifications ?? 0
+	})}
+<main class="prb-index prb-help">
+  ${TOPICS.map((topic) => `<section class="prb-help-topic" id="${topic.id}">
+    <h2>${escapeHtml$1(topic.title)}</h2>
+    ${topic.body}
+  </section>`).join("")}
+</main>
+</div>
+<script src="/assets/help.js" type="module"><\/script>
+</body>
+</html>`;
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/notifications_page.ts
 function renderNotificationsPage(view) {
 	const unread = view.notifications.filter((notification) => notification.readAt === void 0).length;
@@ -8193,12 +8378,25 @@ function buildPrompt$2(workspace, question, sectionId, cwd) {
 		...recent.length > 0 ? [
 			"---",
 			"EARLIER IN THIS CONVERSATION:",
-			recent.map((entry) => `Q: ${entry.question}\nA: ${entry.answer}`).join("\n\n")
+			recent.map((entry) => `Q ${askedWhere(workspace, entry.sectionId)}: ${entry.question}\nA: ${entry.answer}`).join("\n\n")
 		] : [],
 		"---",
 		`QUESTION: ${question}`
 	].join("\n");
 }
+/**
+* Where an earlier question was asked, which the question alone does not say.
+*
+* "What is risky here?" in one theme followed by "and how does that compare?" in another reads, as
+* bare text, as two questions about the same thing. A theme that has since gone, because the
+* review was updated, is said to have gone rather than dropped: dropping it would present a
+* theme's question as one about the whole change.
+*/
+function askedWhere(workspace, sectionId) {
+	if (sectionId === null) return "(asked about the whole change)";
+	const section = workspace.session.result?.sections.find((s) => s.id === sectionId);
+	return section ? `(asked while reading the "${section.title}" theme)` : "(asked while reading a theme that is no longer in the review)";
+}
 //#endregion
 //#region ../../packages/review-harness/src/workspace/assistant_runs.ts
 var RUN_KINDS = ["ask", "diagram"];
@@ -9206,6 +9404,7 @@ async function handle(req, res, options) {
 	if (url.pathname === "/open") return handleOpen(req, res, url.searchParams, options.indexToken);
 	if (url.pathname === "/settings") return void handleSettings(req, res, url.searchParams, options.indexToken);
 	if (url.pathname === "/notifications") return void handleNotifications(req, res, url.searchParams, options.indexToken);
+	if (url.pathname === "/help") return void handleHelp(req, res, url.searchParams, options.indexToken);
 	if (url.pathname === "/notifications/open") return handleNotificationOpen(req, res, url.searchParams, options.indexToken);
 	if (url.pathname === "/notifications/recent") return handleRecentNotifications(req, res, url.searchParams, options.indexToken);
 	if (url.pathname === "/pin") return handlePin(req, res, options.indexToken);
@@ -9239,6 +9438,8 @@ async function handle(req, res, options) {
 	if (action === "/stop") return handleStop(req, res, workspace);
 	if (action === "/kiss") return handleKiss(req, res, workspace);
 	if (action === "/state") return handleState(req, res, workspace);
+	if (action === "/lifecycle") return handleOwnLifecycle(req, res, workspace, options.establishBaseline ?? establishBaseline);
+	if (action === "/queue") return handleOwnQueue(req, res, workspace);
 	if (action === "/delete") return handleDeleteOwn(req, res, workspace);
 	if (action === "/event") return handleEvent(req, res, workspace);
 	if (action === "/prefs") return handlePrefs(req, res);
@@ -9413,6 +9614,18 @@ async function handleSettings(req, res, params, indexToken) {
 	}), indexCookieHeader(indexToken));
 }
 /**
+* Help. Static, but still behind the index's credential like every other workspace place: its
+* rail and strip carry the unread count, which is this machine's state.
+*/
+function handleHelp(req, res, params, indexToken) {
+	if (!indexAuthorised(req, params, indexToken)) return refuseIndex(req, res);
+	sendHtml(res, 200, renderHelpPage({
+		unreadNotifications: unreadCount$1(),
+		driving: isDriving(),
+		theme: readPreferences().theme
+	}), indexCookieHeader(indexToken));
+}
+/**
 * Whether the caller is a browser navigating, rather than something reading the API.
 *
 * `Accept` is the only signal available and browsers are consistent about it: a navigation asks
@@ -9545,7 +9758,6 @@ async function handleStatus(req, res, indexToken) {
 			current.completedAt = complete ? now : void 0;
 			if (!complete) current.lastViewedAt = current.lastViewedAt ?? now;
 		})) continue;
-		dequeueOnComplete(id, complete);
 		changed += 1;
 	}
 	send(res, 200, {
@@ -9579,28 +9791,110 @@ async function handleLifecycle(req, res, indexToken, baseline) {
 		return send(res, 400, { error: `There is no lifecycle state called \`${String(state)}\`. Choose one of ${choices}.` });
 	}
 	const ids = Array.isArray(body?.ids) ? body.ids.filter((id) => typeof id === "string") : [];
-	const now = Date.now();
+	const { changed, at } = applyLifecycle(ids, state);
+	send(res, 200, {
+		ok: true,
+		changed,
+		at,
+		queue: readQueue()
+	});
+	baselineParked(ids, state, baseline);
+}
+/**
+* The write behind every lifecycle mark, from the list or from the review's own page. Waiting and
+* Done dequeue; see `handleLifecycle`.
+*/
+function applyLifecycle(ids, state) {
+	const at = Date.now();
 	let changed = 0;
 	for (const id of ids) {
 		if (!mutateWorkspace(id, (current) => {
 			current.lifecycle = state === "active" ? void 0 : {
 				state,
-				at: now
+				at
 			};
 		})) continue;
 		if (state !== "active") queueRemove(id);
 		changed += 1;
 	}
+	return {
+		changed,
+		at
+	};
+}
+/** After the answer is sent, so a first look at the forge never holds up the page that asked. */
+function baselineParked(ids, state, baseline) {
+	if (state !== "waiting") return;
+	const unseen = unobserved(ids);
+	if (unseen.size > 0) baseline(unseen);
+}
+/**
+* Lifecycle and queue, asked by the review's own page on its own token.
+*
+* Both belong to the branch, and the list only ever reads them from the newest review of it, so a
+* mark written on an older one would be a write nobody sees. The page offers neither there; this
+* says why to anything that asks anyway.
+*/
+function refuseUnlessLatest(res, workspace) {
+	const { position, total } = lineagePosition(workspace);
+	if (position >= total) return false;
+	send(res, 409, { error: "A newer review of this branch exists. Waiting, Done and the review queue are set on the newest one." });
+	return true;
+}
+/**
+* What the Reviews list would say about this branch: the newest analysis's creation, a push the
+* watcher saw, or a change of pull request state. Reads every record, so it is asked only where
+* it can change what is drawn.
+*/
+function branchActivity(workspace, all = recentWorkspaces(Number.MAX_SAFE_INTEGER)) {
+	const group = groupByLineage(all).find((candidate) => candidate.history.some((review) => review.id === workspace.id));
+	return group ? reviewActivity([group]).get(group.latest.id) : void 0;
+}
+/**
+* Where the review stands, for the page's own checks to repaint from. Another tab can mark or park
+* it, and the watcher can see the branch move while the page is open.
+*
+* Activity only for a parked review: it decides nothing for an Active one, and it costs a read of
+* every record (measured 27 ms warm, 525 ms cold, on 66 reviews), once a minute per open tab.
+*/
+function pageStanding(workspace) {
+	const lifecycle = parseLifecycle(workspace.lifecycle) ?? null;
+	return {
+		complete: workspace.completedAt !== void 0,
+		lifecycle,
+		queued: readQueue().includes(workspace.id),
+		activity: lifecycle ? branchActivity(workspace) ?? null : null
+	};
+}
+/** Where this review stands after a lifecycle or queue change, which is everything the page repaints. */
+function ownStanding(id) {
+	return {
+		lifecycle: parseLifecycle(loadWorkspace(id)?.lifecycle) ?? null,
+		queued: readQueue().includes(id)
+	};
+}
+async function handleOwnLifecycle(req, res, workspace, baseline) {
+	const state = (await readJson(req))?.state;
+	if (!isLifecycle(state)) {
+		const choices = LIFECYCLES.map((lifecycle) => lifecycle.id).join(", ");
+		return send(res, 400, { error: `There is no lifecycle state called \`${String(state)}\`. Choose one of ${choices}.` });
+	}
+	if (refuseUnlessLatest(res, workspace)) return;
+	applyLifecycle([workspace.id], state);
 	send(res, 200, {
 		ok: true,
-		changed,
-		at: now,
-		queue: readQueue()
+		...ownStanding(workspace.id)
 	});
-	if (state === "waiting") {
-		const unseen = unobserved(ids);
-		if (unseen.size > 0) baseline(unseen);
-	}
+	baselineParked([workspace.id], state, baseline);
+}
+async function handleOwnQueue(req, res, workspace) {
+	const action = (await readJson(req))?.action;
+	if (action !== "add" && action !== "remove") return send(res, 400, { error: "Unknown queue action." });
+	if (refuseUnlessLatest(res, workspace)) return;
+	const stored = applyQueue(workspace.id, action);
+	if (stored === null) return send(res, 500, { error: "The queue could not be saved." });
+	if (action === "add" && !stored.includes(workspace.id)) return send(res, 409, { error: `The review queue is full. It holds 50 reviews at most.` });
+	send(res, 200, ownStanding(workspace.id));
 }
 /**
 * Which of these reviews the watcher has never seen the head of.
@@ -9713,28 +10007,17 @@ async function handleQueue(req, res, indexToken) {
 		return addMany(res, body.ids.filter((one) => typeof one === "string"));
 	}
 	if (!id || !loadWorkspace(id)) return send(res, 404, { error: "No such review" });
+	const stored = applyQueue(id, action);
+	if (stored === null) return send(res, 500, { error: "The queue could not be saved." });
+	send(res, 200, { queue: stored });
+}
+/** One review in or out of the queue, from a card or from its own page. The queue as stored, or null. */
+function applyQueue(id, action) {
 	if (action === "add") compactQueue(readQueue());
 	const stored = action === "add" ? queueAdd(id) : queueRemove(id);
-	if (stored === null) return send(res, 500, { error: "The queue could not be saved." });
+	if (stored === null) return null;
 	if (action === "add") activateQueued([id], stored);
-	send(res, 200, { queue: compactQueue(stored) });
-}
-/**
-* 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.
-*/
-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;
-		});
-	}
+	return compactQueue(stored);
 }
 /**
 * Adding a selection, and saying which of it did not go in.
@@ -9976,6 +10259,7 @@ async function sendWorkspacePage(res, workspace) {
 	if (notice) mutateWorkspace(workspace.id, (current) => {
 		if (current.lastUpdate?.at === notice.at) current.lastUpdate.acknowledged = true;
 	});
+	const everyReview = recentWorkspaces(Number.MAX_SAFE_INTEGER);
 	const html = renderWorkspace({
 		session: workspace.session,
 		changeSet: workspace.changeSet,
@@ -10014,7 +10298,10 @@ async function sendWorkspacePage(res, workspace) {
 		changeMap: workspace.changeMap,
 		reviewedSha: reviewedCommit(workspace),
 		revision: workspaceRevision(workspace),
-		lineage: lineagePosition(workspace),
+		lineage: lineagePosition(workspace, everyReview),
+		lifecycle: parseLifecycle(workspace.lifecycle),
+		activity: branchActivity(workspace, everyReview),
+		queued: readQueue().includes(workspace.id),
 		job: workspace.jobId ? loadJob(workspace.jobId) : void 0,
 		updateJob: liveUpdateFor(workspace.id),
 		kissRun,
@@ -10134,7 +10421,8 @@ async function handleFreshness(res, workspace, quiet) {
 		prState,
 		kiss,
 		kissDismissedSha: workspace.dismissedKissFreshnessSha ?? null,
-		revision
+		revision,
+		standing: pageStanding(workspace)
 	});
 	if (!quiet) record({
 		event: "freshness_checked",
@@ -10151,7 +10439,8 @@ async function handleFreshness(res, workspace, quiet) {
 		dismissedSha: workspace.dismissedFreshnessSha ?? null,
 		kiss,
 		kissDismissedSha: workspace.dismissedKissFreshnessSha ?? null,
-		revision
+		revision,
+		standing: pageStanding(workspace)
 	});
 }
 /**
@@ -10609,7 +10898,8 @@ async function handleState(req, res, workspace) {
 	send(res, 200, {
 		ok: true,
 		completedAt: saved.completedAt ?? null,
-		pinned: saved.pinned === true
+		pinned: saved.pinned === true,
+		queued: readQueue().includes(saved.id)
 	});
 }
 /** What `/state` can change. A body naming none of these writes nothing. */
@@ -10627,10 +10917,7 @@ function applyState(workspace, body) {
 	if (Array.isArray(body?.dismissedIssues)) reviewer.dismissedIssues = body.dismissedIssues.filter((entry) => typeof entry?.id === "string" && typeof entry?.sessionId === "string");
 	if (typeof body?.dismissFreshnessSha === "string" || body?.dismissFreshnessSha === null) workspace.dismissedFreshnessSha = body.dismissFreshnessSha ?? void 0;
 	if (typeof body?.dismissKissFreshnessSha === "string" || body?.dismissKissFreshnessSha === null) workspace.dismissedKissFreshnessSha = body.dismissKissFreshnessSha ?? void 0;
-	if (typeof body?.complete === "boolean") {
-		workspace.completedAt = body.complete ? Date.now() : void 0;
-		dequeueOnComplete(workspace.id, body.complete);
-	}
+	if (typeof body?.complete === "boolean") workspace.completedAt = body.complete ? Date.now() : void 0;
 	if (typeof body?.pinned === "boolean") workspace.pinned = body.pinned;
 }
 /** 1 MB is far more than any question or id list, and far less than a memory problem. */