prreviewbuddy 0.50.0 → 0.52.2

package/dist/server.js

@@ -1,4 +1,4 @@
-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 { $ as compactQueue, $t as loadWorkspace, A as createPrState, An as reviewerDispositions, At as followedBranch, B as toneOf, Bn as resolveModel, Bt as phasesFor, C as readEvents, Cn as queueStamp, Ct as claimRequestUrl, D as announceJob, Dn as isQuestionOutstanding, Dt as fillFileUrlTemplate, E as telemetryUploadConsent, En as isIssueOutstanding, Et as remoteHost, F as markAllRead, Ft as isTerminal, G as extractJsonText, Gn as STORE_ROOT, Gt as isClaimed, H as withUsageRecorded, Hn as readAgentPreference, Ht as purposeOf, I as markRead, It as loadJob, J as readMarker, Jt as indexKeptIds, K as ReviewBeingDeletedError, Kn as AGENT_IDS, Kt as followedRefName, L as pruneNotifications, Ln as isCommitReview, Lt as saveJob, M as eventName, Mn as describeAuthorship, Mt as allJobs, N as headline, Nn as WITHHELD_REASON, Nt as endedAt, O as branchOf, On as issuesOutstanding, Ot as checkCodeFreshness, P as listNotifications, Pn as kissOutcome, Pt as fail, Q as activateQueued, Qn as AgentUnavailableError, Qt as lineagePosition, R as readNotification, Rn as git, Sn as queueRemove, St as writePortPreference, T as recordTelemetryUploadConsent, Tn as doneVerb, Tt as forgeFor, U as EXPLAIN_SIMPLY_PROMPT, Un as writeAgentPreference, V as unreadCount$1, Vn as clearAgentPreference, Vt as progressSteps, W as asString, Xt as latestKissRun, Y as removeWorktree, Yn as agentFor, Yt as indexReviewGroups, Z as relativeTime$1, Zn as AgentCancelledError, _ as agentEnvOf, _n as offersLifecycle, _t as MAX_WORKSPACE_PORT, a as wasBlocked, an as reviewedCommit, bn as queueAdd, bt as resolveWorkspacePort, c as RefreshUnavailableError, cn as touchWorkspace, ct as serverEntryExists, d as refreshPrContext, dn as LIFECYCLES, f as isUnchanged, fn as activityLabel, ft as writeServerRecord, gn as lifecyclePill, gt as DEFAULT_WORKSPACE_PORT, h as liveUpdateFor, hn as lifecycleChip, ht as feedbackUrl, i as liveJobsFor, in as recentWorkspaces, it as clearServerRecord, j as createPrUpdate, jn as processDiscussion, jt as reviewedShaOf, k as subjectOf, kn as questionsOutstanding, kt as checkReview, l as mergeUpdateInto, ln as workspaceRevision, lt as spawnServer, m as UpdateAlreadyRunningError, mn as lifecycleAction, n as deleteReview, nt as readIndexToken, o as discardJob, ot as readServerRecord, p as askCheckout, pn as isLifecycle, pt as BUILD_VERSION, q as MANAGED_ROOT, qt as groupByLineage, r as lineageIds, rn as previousKissResult, s as FETCH_TIMEOUT_MS, t as deleteLineage, tn as mutateWorkspace, tt as settleQueue, u as updateReview, un as workspaceStamps, vn as parseLifecycle, vt as MIN_WORKSPACE_PORT, w as record, wn as readQueue, x as runJob, xn as queueAddMany, xt as statedWorkspacePort, y as startKissJob, yn as sectionOf, yt as parseWorkspacePort, z as removeNotification, zt as isLiveJob } from "./delete_review-BiV1aBpx.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";
@@ -208,6 +208,30 @@ function resolveSectionId(run, sectionId) {
 	return run.result.sections.some((section) => section.id === sectionId) ? namespaced(run.id, sectionId) : sectionId;
 }
 //#endregion
+//#region ../../packages/review-harness/src/domain/ai_review/ask_scope.ts
+/**
+* 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. `undefined` is a question recorded before
+* scopes were, and gets no label: calling it "the whole change" would state something nobody
+* recorded. A theme that has since gone, because the review was updated, is said to have gone.
+*/
+function askedWhere(sections, sectionId) {
+	if (sectionId === void 0) return null;
+	if (sectionId === null) return "(asked about the whole change)";
+	const section = 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)";
+}
+/**
+* The theme the reviewer has open, named rather than used to filter what the model sees: they
+* routinely ask "does this interact with X", where X is in another theme.
+*/
+function readingThemeLine(sections, sectionId) {
+	const section = sectionId === null ? void 0 : sections.find((s) => s.id === sectionId);
+	return section ? `The reviewer is currently reading the "${section.title}" theme. Answer with that in view, but do not ignore the rest of the change if it bears on the question.` : null;
+}
+//#endregion
 //#region ../../packages/review-harness/src/domain/utils/html.ts
 function escapeHtml$1(value) {
 	return value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&#39;");
@@ -218,6 +242,47 @@ function renderInlineText(value) {
 //#endregion
 //#region ../../packages/review-harness/src/workspace/reviews_index.ts
 /**
+* The sections of the list, top to bottom.
+*
+* Pinned is not a lifecycle state. It is where a card is drawn, on top of a lifecycle it keeps: a
+* pinned review can be Active, Waiting or Done, and unpinning it puts it back in the section that
+* says. That is why it lives here, beside the other placement rules, and not in `LIFECYCLES`.
+*/
+var INDEX_SECTIONS = [
+	"pinned",
+	"active",
+	"waiting",
+	"done"
+];
+/**
+* Which section a card is drawn in: Pinned when pinned, and otherwise where its lifecycle says.
+*
+* Pinning controls prominence in the general list and never overrides an ordering the reviewer
+* asked for. Under such a filter a pinned card stays in its lifecycle section, in the order the
+* filter gives, and only its star says it is pinned.
+*/
+function indexSectionOf(pinned, lifecycle, activity, filter) {
+	return pinned && !ordersExplicitly(filter) ? "pinned" : sectionOf(lifecycle, activity);
+}
+/**
+* Whether a filter replaces the activity order with one the reviewer set. Queue is the only one:
+* the list is the queue, in its order.
+* `orderIndex` and `indexSectionOf` both ask this, so the ordering and the placement cannot disagree.
+*/
+function ordersExplicitly(filter) {
+	return filter === "queue";
+}
+/**
+* The lifecycle chip on a card, or absent.
+*
+* In a lifecycle section the heading already names the lifecycle, so the chip only says what moved
+* (`lifecycleChip`). In Pinned nothing names it, so the chip does, in the words the review page's
+* pill uses: the list and the page give one answer about the same review.
+*/
+function cardLifecycleChip(pinned, lifecycle, activity) {
+	return pinned ? lifecyclePill(lifecycle, activity)?.text : lifecycleChip(lifecycle, activity);
+}
+/**
 * "45 reviews · 13 branches", and "13 reviews" when those are the same thirteen things.
 *
 * The two numbers are different questions and only both worth asking once they differ. A page of
@@ -245,12 +310,12 @@ function markItemLabel(mark) {
 	return mark === "complete" ? "Mark reviewed" : "Mark in progress";
 }
 /**
-* The review page's Save item, which says what pressing it will do. The words beside the index's
-* star and its Saved filter, so saving from either place reads as the same act. Here because the
-* browser relabels the item after the change, for the reason `markItemLabel` is here.
+* The review page's Pin item, which says what pressing it will do. The words of the index's star,
+* so pinning from either place reads as the same act. Here because the browser relabels the item
+* after the change, for the reason `markItemLabel` is here.
 */
-function saveItemLabel(saved) {
-	return saved ? "Remove from saved" : "Save review";
+function pinItemLabel(pinned) {
+	return pinned ? "Unpin review" : "Pin review";
 }
 /**
 * The queue item, on a card and on the review's own page. "review queue" rather than "queue",
@@ -261,16 +326,6 @@ 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
-* "what is first" are the same question and must not be able to answer differently -- a button that
-* is present but navigates nowhere is worse than no button.
-*/
-function continueTarget(queue) {
-	return queue[0] ?? null;
-}
-/**
 * The Queue pill, which carries a number only once there is one.
 *
 * "Queue 0" is a count of something that is not there, and reads as a fault. Bare "Queue" reads as
@@ -335,6 +390,11 @@ function anythingRunning(states) {
 	return false;
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/fetch_updates_view.ts
+/** A card's and a review page's item. Named for what it reaches: a remote branch, not the list. */
+var FETCH_REVIEW_LABEL = "Fetch remote updates";
+var FETCH_SELECTED_LABEL = "Fetch selected";
+//#endregion
 //#region ../../packages/review-harness/src/workspace/way_back.ts
 var CLI = {
 	command: "prreviewbuddy open",
@@ -636,7 +696,7 @@ function renderWorkspaceRail(options) {
 * with the reason for it.
 */
 function renderHeaderNav(_options) {
-	return `<nav class="prb-header-nav" aria-label="Workspace"><a class="prb-feedback" href="${escapeHtml$1(feedbackUrl())}" target="_blank" rel="noopener noreferrer">Send feedback</a></nav>`;
+	return `<nav class="prb-header-nav" aria-label="Workspace"><a class="prb-feedback" href="${escapeHtml$1(feedbackUrl())}" target="_blank" rel="noopener noreferrer"><span class="prb-feedback-icon" aria-hidden="true">${iconSvg("feedback")}</span><span class="prb-feedback-label">Send feedback</span></a></nav>`;
 }
 //#endregion
 //#region ../../packages/review-harness/src/domain/utils/file_path.ts
@@ -773,9 +833,10 @@ var FINDING_COPY_CLASS = "pr-helper-finding-copy";
 * 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".
+* accessible name. After a click it shows a green check and says "Copied" (or "Copy failed") in
+* the bubble below it, the words used everywhere else, but not inline: beside it is Ask Buddy. The
+* resting name and tip are 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";
@@ -788,11 +849,12 @@ function renderFindingCopySource(source, kind) {
 /** Where a finding's button keeps its resting word and name. Absent on a chat reply's. */
 var IDLE_LABEL_ATTR = "data-copy-idle-label";
 var IDLE_ARIA_ATTR = "data-copy-idle-aria";
+var IDLE_TIP_ATTR = "data-copy-idle-tip";
 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 remembered = idle ? ` ${IDLE_LABEL_ATTR}="${escapeHtml$1(label)}" ${IDLE_ARIA_ATTR}="${escapeHtml$1(aria)}"` : "";
+	const remembered = idle ? ` ${IDLE_LABEL_ATTR}="${escapeHtml$1(label)}" ${IDLE_ARIA_ATTR}="${escapeHtml$1(aria)}" ${IDLE_TIP_ATTR}="${escapeHtml$1(idle.tip)}"` : "";
 	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>`;
 }
@@ -813,8 +875,7 @@ function svg(paths) {
 * closes and positions exactly as that one does. The panel is empty here and filled when it opens,
 * because it lists the reviewer's saved prompts and those change without the page re-rendering.
 *
-* Workspace only. The extension renders these cards too, and there is no Buddy pane behind a card
-* there for this to open, so `ReviewTabOptions.askBuddy` gates it and the extension never sets it.
+* The host decides where the answer appears; `ReviewTabOptions.askBuddy` says the host has wired it.
 */
 var ASK_BUDDY_LABEL = "Ask Buddy";
 /** Four-point sparkle, the conventional mark for "ask the AI". currentColor, like Copy beside it. */
@@ -3617,10 +3678,11 @@ ${renderRunControl(view.canRun, "Run KISS review again")}`;
 function renderResultBody(run, provenance = "inline") {
 	const result = run.result;
 	if (!result) return "";
-	return `${result.recommendations.length === 0 ? `<div class="prb-kiss-outcome" data-review-outcome="empty">
+	const outcome = kissOutcome(result, run.audit);
+	return `${outcome === "proportionate" ? `<div class="prb-kiss-outcome" data-review-outcome="empty">
     <h4 class="prb-kiss-outcome-title">Appropriately simple</h4>
     <p class="prb-kiss-outcome-body">This change appears appropriately simple for the current requirements. No material simplifications were identified.</p>
-  </div>` : ""}
+  </div>` : outcome === "incomplete" ? renderIncomplete(run) : ""}
 <div class="prb-kiss-assessment pr-helper-markdown">${renderMarkdown(result.assessment)}</div>
 <div class="prb-kiss-summary pr-helper-markdown">${renderMarkdown(result.summary)}</div>
 ${result.aggregateEstimate ? `<p class="prb-kiss-estimate"><span class="prb-kiss-estimate-label">Rough estimate</span> ${escapeHtml$1(result.aggregateEstimate)}</p>` : ""}
@@ -3649,15 +3711,38 @@ function renderKissRecommendations(view) {
 	const result = presented?.result;
 	if (!result) return `${RECS_HEAD}${renderNoRunYet(view)}`;
 	const earlier = earlierNote(view, presented);
-	if (result.recommendations.length === 0) return `${RECS_HEAD}${earlier}
+	const outcome = kissOutcome(result, presented.audit);
+	if (outcome === "incomplete") return `${RECS_HEAD}${earlier}${renderIncomplete(presented)}`;
+	if (outcome === "proportionate") return `${RECS_HEAD}${earlier}
     <div class="prb-kiss-outcome" data-review-outcome="empty">
       <h4 class="prb-kiss-outcome-title">Nothing to simplify</h4>
       <p class="prb-kiss-outcome-body">The analysis completed and did not identify a materially simpler way to meet these requirements.</p>
     </div>`;
-	return `${RECS_HEAD}${earlier}
+	return `${RECS_HEAD}${earlier}${renderWithheldNote(presented)}
 ${result.recommendations.map((recommendation) => renderRecommendation(recommendation)).join("")}`;
 }
 /**
+* A run whose every recommendation was withheld: incomplete, and said to be.
+*
+* Never "appropriately simple" and never "nothing to simplify". The analysis did recommend
+* something; it just did not return enough of it to show, and the one conclusion that cannot be
+* drawn from that is that the change is proportionate. Its own outcome value, so a test can tell it
+* from both a clean answer and a failed run.
+*/
+function renderIncomplete(run) {
+	const withheld = run.audit.withheldRecommendations ?? 0;
+	return `<div class="prb-kiss-outcome" data-review-outcome="incomplete">
+    <h4 class="prb-kiss-outcome-title">Recommendations came back incomplete</h4>
+    <p class="prb-kiss-outcome-body">The analysis returned ${withheld === 1 ? "a recommendation" : `${withheld} recommendations`}, but ${withheld === 1 ? "it was" : "each was"} ${escapeHtml$1(WITHHELD_REASON)}, so ${withheld === 1 ? "it cannot" : "none can"} be shown. This is not a finding that the change is appropriately simple. Running KISS again usually produces a complete answer.</p>
+  </div>`;
+}
+/** Said on the list itself when some were withheld, since the correction box is on the other pane. */
+function renderWithheldNote(run) {
+	const withheld = run.audit.withheldRecommendations ?? 0;
+	if (withheld === 0) return "";
+	return `<p class="prb-kiss-note" data-kiss-withheld>${withheld === 1 ? "1 more recommendation was" : `${withheld} more recommendations were`} withheld as incomplete: ${escapeHtml$1(WITHHELD_REASON)}.</p>`;
+}
+/**
 * Which run these came from, said only when it is not the one the section is headed by.
 *
 * Silent in the ordinary case, because a permanent "this is the current run" strip is the noise
@@ -3871,14 +3956,30 @@ function renderAudit(run) {
 	const notes = [];
 	if (run.audit.unknownPaths.length > 0) notes.push(`${run.audit.unknownPaths.length} cited ${run.audit.unknownPaths.length === 1 ? "path does" : "paths do"} not exist in the reviewed version of the repository, and ${run.audit.unknownPaths.length === 1 ? "it was" : "they were"} withdrawn: ${run.audit.unknownPaths.join(", ")}.`);
 	if (run.audit.unlocatedRecommendations > 0) notes.push(`${run.audit.unlocatedRecommendations} recommendation${run.audit.unlocatedRecommendations === 1 ? "" : "s"} ${run.audit.unlocatedRecommendations === 1 ? "is" : "are"} shown without a file to open, because nothing ${run.audit.unlocatedRecommendations === 1 ? "it" : "they"} cited exists in the reviewed version of the repository.`);
+	const withheld = run.audit.withheldRecommendations ?? 0;
+	if (withheld > 0 && run.result && kissOutcome(run.result, run.audit) !== "incomplete") notes.push(`${withheld} recommendation${withheld === 1 ? " was" : "s were"} withheld as incomplete: ${WITHHELD_REASON}.`);
 	if (notes.length === 0) return "";
 	return `<div class="prb-kiss-audit" role="note">
   <p class="prb-kiss-audit-title">What was corrected</p>
   ${notes.map((note) => `<p>${escapeHtml$1(note)}</p>`).join("")}
 </div>`;
 }
+/**
+* One recommendation, as a decision aid rather than an analysis record.
+*
+* The card carries what a reviewer needs to understand and weigh the recommendation: the title,
+* the case in a sentence or two, the simpler approach, what it gains, the trade-off that could
+* change the decision, and where to start. Everything that exists to audit or challenge it sits in
+* one disclosure beneath, closed until asked for.
+*
+* Reads both schemas. A record written under schema 1 has no gist, and its `complexity` stands in
+* as the lede, because it is the only field of that shape that names what the card is about; it is
+* then not repeated in the disclosure. Stored runs are history and are not rewritten to fit.
+*/
 function renderRecommendation(recommendation) {
 	const worthiness = recommendation.commentWorthiness;
+	const lede = recommendation.gist || recommendation.complexity || "";
+	const [start, ...rest] = orderedLocations(recommendation.locations);
 	return `<article class="prb-kiss-rec" data-payoff="${recommendation.payoff}"${worthiness ? ` data-worthiness="${worthiness}"` : ""}>
   <header class="prb-kiss-rec-head">
     <h4 class="prb-kiss-rec-title">${escapeHtml$1(recommendation.title)}</h4>
@@ -3887,20 +3988,56 @@ function renderRecommendation(recommendation) {
       ${worthiness ? `<span class="prb-kiss-worthiness" data-worthiness="${worthiness}">${escapeHtml$1(WORTHINESS[worthiness])}</span>` : ""}
     </div>
   </header>
-  ${field("What was introduced", recommendation.complexity)}
-  ${field("Evidence in this change", recommendation.evidence)}
-  ${field("Why the requirements do not call for it", recommendation.whyNotJustified)}
-  ${field("A simpler direction", recommendation.simplerDirection)}
-  ${field("Payoff and trade-offs", recommendation.payoffAndTradeoffs)}
-  ${recommendation.reductionEstimate ? `<p class="prb-kiss-estimate"><span class="prb-kiss-estimate-label">Rough estimate</span> ${escapeHtml$1(recommendation.reductionEstimate)}</p>` : ""}
+  ${lede ? `<div class="prb-kiss-gist pr-helper-markdown" data-kiss-gist>${renderMarkdown(lede)}</div>` : ""}
+  ${field("Simpler approach", recommendation.simplerDirection)}
+  ${renderGains(recommendation)}
+  ${recommendation.caveat ? `<p class="prb-kiss-caveat" data-kiss-caveat><span class="prb-kiss-caveat-label">Trade-off</span> ${escapeHtml$1(recommendation.caveat)}</p>` : ""}
+  ${start ? `<div class="prb-kiss-locations">
+    <p class="prb-kiss-locations-label">Start here</p>
+    ${renderLocationList([start])}
+  </div>` : ""}
+  ${renderEvidence(recommendation, rest)}
   ${recommendation.draftComment ? `<details class="prb-kiss-draft">
     <summary class="prb-kiss-draft-title">Draft comment</summary>
     <blockquote class="pr-helper-markdown">${renderMarkdown(recommendation.draftComment)}</blockquote>
     ${renderMessageCopySource(recommendation.draftComment)}
   </details>` : ""}
-  ${renderLocations(recommendation.locations)}
 </article>`;
 }
+/**
+* What acting on it gains, on one line: the estimate first, then the gains.
+*
+* The estimate says it is one in words beside it, so a phrase like "~35–50 lines" cannot be taken
+* for a measurement, and the gains beside it are not read as estimates too.
+*/
+function renderGains(recommendation) {
+	const items = [...recommendation.reductionEstimate ? [`<span class="prb-kiss-gain" data-estimate>${escapeHtml$1(recommendation.reductionEstimate)} <span class="prb-kiss-gain-note">(estimate)</span></span>`] : [], ...(recommendation.gains ?? []).map((gain) => `<span class="prb-kiss-gain">${escapeHtml$1(gain)}</span>`)];
+	if (items.length === 0) return "";
+	return `<p class="prb-kiss-gains" data-kiss-gains>${items.join("<span class=\"prb-kiss-gain-sep\" aria-hidden=\"true\"> · </span>")}</p>`;
+}
+/**
+* The audit trail, in one disclosure: enough to check or challenge the recommendation.
+*
+* Closed by default. A reviewer should never have to open it to understand what is being
+* recommended, only to verify it. The other locations live here too, because the card names the
+* one place to start and a list of files up front is a decision the reader has not asked to make.
+*/
+function renderEvidence(recommendation, rest) {
+	const body = [
+		field("Evidence in this change", recommendation.evidence),
+		field("Why the requirements do not call for it", recommendation.whyNotJustified),
+		field("Trade-offs", recommendation.payoffAndTradeoffs),
+		rest.length > 0 ? `<div class="prb-kiss-locations">
+    <p class="prb-kiss-locations-label">Also relevant</p>
+    ${renderLocationList(rest)}
+  </div>` : ""
+	].filter(Boolean);
+	if (body.length === 0) return "";
+	return `<details class="prb-kiss-evidence" data-kiss-evidence>
+    <summary class="prb-kiss-evidence-title">Evidence &amp; trade-offs${escapeHtml$1(rest.length > 0 ? ` · ${rest.length} more ${rest.length === 1 ? "file" : "files"}` : "")}</summary>
+    ${body.join("\n    ")}
+  </details>`;
+}
 function field(label, value) {
 	if (!value) return "";
 	return `<div class="prb-kiss-field">
@@ -3909,7 +4046,8 @@ function field(label, value) {
   </div>`;
 }
 /**
-* Where a recommendation points, primary first. The whole of KISS's file navigation.
+* Where an item of complexity worth keeping points, primary first. A recommendation draws its own
+* "Start here" on the card and keeps the rest in its evidence disclosure; see `renderRecommendation`.
 *
 * `data-file` and `data-line` and nothing else: the workspace already opens anything carrying
 * those, so a KISS location behaves exactly as a standard finding's does and there is no second
@@ -3945,7 +4083,7 @@ function field(label, value) {
 */
 function renderLocations(locations) {
 	if (locations.length === 0) return "";
-	const [first, ...rest] = [...locations].sort((a, b) => Number(b.primary) - Number(a.primary));
+	const [first, ...rest] = orderedLocations(locations);
 	if (rest.length === 0) return `<div class="prb-kiss-locations">${renderLocationList([first])}</div>`;
 	return `<div class="prb-kiss-locations">
   <p class="prb-kiss-locations-label">Start here</p>
@@ -3954,6 +4092,10 @@ function renderLocations(locations) {
   ${renderLocationList(rest)}
 </div>`;
 }
+/** Primary first, the rest in the order the model gave them. */
+function orderedLocations(locations) {
+	return [...locations].sort((a, b) => Number(b.primary) - Number(a.primary));
+}
 function renderLocationList(locations) {
 	return `<ul class="prb-kiss-locations-list">${locations.map((location) => {
 		const { dir, name } = splitFilePath(location.path);
@@ -5132,10 +5274,11 @@ function renderIdMenu(view, freshness) {
 	const branchItems = canWrite && latest;
 	const items = [
 		canWrite && (freshness || view.remote) ? "<button type=\"button\" class=\"prb-menu-item\" data-fresh-check>Check for updates</button>" : "",
+		branchItems ? `<button type="button" class="prb-menu-item" data-fetch-remote hidden>${escapeHtml$1(FETCH_REVIEW_LABEL)}</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>` : "",
+		canWrite ? `<button type="button" class="prb-menu-item" data-pin-toggle aria-pressed="${view.pinned === true}">${pinItemLabel(view.pinned === 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);
@@ -6811,10 +6954,6 @@ function startWatching(makeDeps = defaultDeps) {
 	};
 }
 //#endregion
-//#region ../../packages/review-harness/src/workspace/fetch_updates_view.ts
-var FETCH_UPDATES_LABEL = "Fetch updates";
-var FETCH_SELECTED_LABEL = "Fetch selected";
-//#endregion
 //#region ../../packages/review-harness/src/workspace/reviews_page.ts
 function renderReviewsPage(view) {
 	const now = view.now ?? Date.now();
@@ -6822,20 +6961,19 @@ function renderReviewsPage(view) {
 		at: group.latest.createdAt,
 		kind: "created"
 	};
-	const ordered = [...view.groups].sort((a, b) => Number(b.latest.pinned) - Number(a.latest.pinned) || activityOf(b).at - activityOf(a).at);
+	const ordered = [...view.groups].sort((a, b) => activityOf(b).at - activityOf(a).at);
 	const sections = {
+		pinned: [],
 		active: [],
 		waiting: [],
 		done: []
 	};
-	for (const group of ordered) sections[sectionOf(group.latest.lifecycle, activityOf(group))].push(group);
+	for (const group of ordered) sections[indexSectionOf(group.latest.pinned === true, group.latest.lifecycle, activityOf(group), "all")].push(group);
 	const rank = new Map(ordered.map((group, index) => [group, index]));
 	const renderCards = (groups) => groups.map((group) => renderCard(group, now, activityOf(group), rank.get(group), view.analyses?.get(group.latest.id), positions.get(group.latest.id), group.history.map((review) => view.moved?.get(review.id)).find((kind) => kind !== void 0), prStateOf(group.latest, view.prStates))).join("");
 	const storedReviews = view.groups.reduce((total, group) => total + group.history.length, 0);
 	const queue = view.queue ?? [];
 	const positions = new Map(queue.map((id, position) => [id, position]));
-	const continues = continueTarget(queue);
-	const front = continues === null ? void 0 : findReview(view.groups, continues);
 	return `<!doctype html>
 <html lang="en"${themeAttribute(view.theme)}>
 <head>
@@ -6859,20 +6997,14 @@ ${renderWorkspaceRail({
 		unread: view.unreadNotifications ?? 0
 	})}
 <main class="prb-index">
-  ${front === void 0 ? "" : renderContinue(front)}
   ${ordered.length === 0 ? "" : `<div class="prb-index-summary">
     <p class="prb-index-count" data-count>${escapeHtml$1(countLabel(storedReviews, ordered.length))}</p>
     <button type="button" class="prb-index-select" data-select-mode aria-pressed="false">Select</button>
   </div>`}
-  <div class="prb-cards" data-cards="active">
-    ${renderCards(sections.active)}
-  </div>
-  ${renderLoadMore("active")}
+  ${INDEX_SECTIONS.map((section) => renderSection(section, sections[section], renderCards)).join("\n  ")}
   ${ordered.length === 0 ? `<p class="prb-index-empty">No reviews stored yet. Ask Claude Code to review a change and one will appear here.</p>` : `<p class="prb-index-none" data-no-matches hidden>Nothing matches that.</p>`}
-  ${renderSection("waiting", "Waiting", sections.waiting, true, renderCards)}
-  ${renderSection("done", "Done", sections.done, false, renderCards)}
   ${ordered.length === 0 ? "" : renderBulkBar()}
-  <p class="prb-index-note">Reviews are kept for thirty days, on this machine only. Saved reviews, Done reviews and anything in your review queue are kept for ninety days instead, counting from the last time you opened or marked them. Every review has a fixed address, so a link to one is worth keeping and still opens after a restart. Stopping the server deletes nothing; it only stops anything being served until the next review starts it again.</p>
+  <p class="prb-index-note">Reviews are kept for thirty days, on this machine only. Pinned reviews, Done reviews and anything in your review queue are kept for ninety days instead, counting from the last time you opened or marked them. Every review has a fixed address, so a link to one is worth keeping and still opens after a restart. Stopping the server deletes nothing; it only stops anything being served until the next review starts it again.</p>
 </main>
 </div>
 <script src="/assets/reviews.js" type="module"><\/script>
@@ -6896,13 +7028,11 @@ function renderControls(queued, authors, anyRequest) {
       <select data-status-filter aria-label="Filter by status">
         <option value="all">All</option>
         <option value="queue" data-queue-option>${escapeHtml$1(queueOptionLabel(queued))}</option>
-        <option value="saved">Saved</option>
         <option value="in-progress">In progress</option>
         <option value="complete">Reviewed</option>
       </select>
     </label>
     ${anyRequest ? renderPrFilter() : ""}
-    <button type="button" class="prb-index-fetch-button" data-fetch-updates disabled>${escapeHtml$1(FETCH_UPDATES_LABEL)}</button>
   </div>`;
 }
 /**
@@ -6934,25 +7064,6 @@ function renderAuthorFilter(authors) {
     </select>`;
 }
 /**
-* The way back into the batch, above the list.
-*
-* A link rather than a button, and carrying the review's own token, because that is what every
-* other way into a review on this page is -- the list is the only thing holding every credential.
-*
-* It names the front of the queue rather than saying only "Continue reviewing", so somebody
-* returning after an hour can tell whether it is about to open what they expect before they click.
-*/
-function renderContinue(review) {
-	return `<a class="prb-index-continue" href="${escapeHtml$1(`/r/${encodeURIComponent(review.id)}?t=${encodeURIComponent(review.token)}`)}">
-    <span class="prb-index-continue-label">Continue reviewing</span>
-    <span class="prb-index-continue-title">${escapeHtml$1(review.title)}</span>
-  </a>`;
-}
-/** The review a queued id refers to. The caller has already guaranteed it is one this page shows. */
-function findReview(groups, id) {
-	for (const group of groups) for (const review of group.history) if (review.id === id) return review;
-}
-/**
 * One section's Load more, which the browser shows once that section holds more than it is showing.
 *
 * One per section rather than one pager for the page, so revealing older Done reviews never costs
@@ -6963,15 +7074,17 @@ function renderLoadMore(section) {
 	return `<button type="button" class="prb-index-more" data-load-more="${section}" hidden>Load more</button>`;
 }
 /**
-* Waiting or Done, as a disclosure under the working list.
+* One section, as a disclosure headed with its name and count.
 *
-* Active has no heading: it is what the page has always been, and the count line above it already
-* says what it holds. Waiting starts open, because it is work that will come back; Done starts
-* closed, because it is the archive. Rendered empty and hidden rather than left out, so a card the
-* browser moves here after a mark has somewhere to go.
+* Every section is headed, Active included. Active used to go without, because the count line
+* directly above it said what it held; with Pinned drawn first, the count line sits over Pinned and
+* would be heading the wrong list. Everything starts open but Done, which is the archive. Rendered
+* empty and hidden rather than left out, so a card the browser moves here after a mark or a pin has
+* somewhere to go.
 */
-function renderSection(section, title, groups, open, renderCards) {
-	return `<details class="prb-index-section" data-section="${section}"${open ? " open" : ""}${groups.length === 0 ? " hidden" : ""}>
+function renderSection(section, groups, renderCards) {
+	const title = section === "pinned" ? "Pinned" : LIFECYCLES.find((lifecycle) => lifecycle.id === section).name;
+	return `<details class="prb-index-section" data-section="${section}"${section !== "done" ? " open" : ""}${groups.length === 0 ? " hidden" : ""}>
     <summary class="prb-index-section-head">${title} · <span data-section-count>${groups.length}</span></summary>
     <div class="prb-cards" data-cards="${section}">
       ${renderCards(groups)}
@@ -6998,7 +7111,7 @@ function renderBulkBar() {
     <p class="prb-bulk-count" data-bulk-count aria-live="polite"></p>
     <div class="prb-bulk-actions" data-bulk-actions>
       <button type="button" class="prb-bulk-button" data-bulk-select-all>${escapeHtml$1(selectAllLabel(0))}</button>
-      <button type="button" class="prb-bulk-button" data-bulk-fetch disabled>${escapeHtml$1(FETCH_SELECTED_LABEL)}</button>
+      <button type="button" class="prb-bulk-button" data-bulk-fetch hidden>${escapeHtml$1(FETCH_SELECTED_LABEL)}</button>
       <button type="button" class="prb-bulk-button" data-bulk-queue disabled>Add to review queue</button>
       <button type="button" class="prb-bulk-button" data-bulk-mark="complete">Mark reviewed</button>
       <button type="button" class="prb-bulk-button" data-bulk-mark="in-progress">Mark in progress</button>
@@ -7039,12 +7152,12 @@ function renderBulkBar() {
 function renderCard(group, now, activity, rank, analysis, queued, moved, prState) {
 	const review = group.latest;
 	const status = statusOf(review);
-	const chip = lifecycleChip(review.lifecycle, activity);
+	const chip = cardLifecycleChip(review.pinned === true, review.lifecycle, activity);
 	const href = `/r/${encodeURIComponent(review.id)}?t=${encodeURIComponent(review.token)}`;
 	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}"`}${prState === void 0 ? "" : ` data-pr-state="${prState}"`}${review.authorHandle ? ` data-author="${escapeHtml$1(review.authorHandle)}"` : ""} data-lifecycle="${review.lifecycle?.state ?? "active"}"${review.lifecycle ? ` data-lifecycle-at="${review.lifecycle.at}"` : ""} data-activity-at="${activity.at}" data-activity-kind="${activity.kind}" data-rank="${rank}" data-reviews="${group.history.length}" data-lineage="${escapeHtml$1(group.history.map((entry) => entry.id).join(" "))}" 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>
+    <button type="button" class="prb-card-pin" data-pin="${escapeHtml$1(review.id)}" aria-pressed="${review.pinned}" aria-label="${review.pinned ? "Unpin" : "Pin"} this review">${review.pinned ? "&#9733;" : "&#9734;"}</button>
     ${renderCardMenu(group, followed, queued !== void 0, status, activity)}
     <a class="prb-card-link" href="${escapeHtml$1(href)}">
       <div class="prb-card-head">
@@ -7107,7 +7220,8 @@ function renderCardMenu(group, followed, queued, status, activity) {
 		copy: "Copy review ID",
 		queue: queued ? "remove" : "add",
 		mark,
-		lifecycle
+		lifecycle,
+		fetch: true
 	});
 	const all = `all ${count} reviews following ${followed}`;
 	return renderMenu({
@@ -7119,6 +7233,7 @@ function renderCardMenu(group, followed, queued, status, activity) {
 		queue: queued ? "remove" : "add",
 		mark,
 		lifecycle,
+		fetch: true,
 		copy: "Copy latest review ID"
 	});
 }
@@ -7137,6 +7252,7 @@ function renderCardMenu(group, followed, queued, status, activity) {
 */
 function renderMenu(item) {
 	const id = escapeHtml$1(item.id);
+	const fetchItem = item.fetch ? `<button type="button" class="prb-menu-item" data-fetch-review="${id}" hidden>${escapeHtml$1(FETCH_REVIEW_LABEL)}</button>` : "";
 	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}">${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        ");
@@ -7144,6 +7260,7 @@ function renderMenu(item) {
     <button type="button" class="prb-menu-trigger" data-menu-trigger aria-expanded="false" aria-haspopup="true" aria-label="${escapeHtml$1(item.label)}">&#8943;</button>
     <div class="prb-menu-panel" data-menu-panel hidden>
       <div data-menu-items>
+        ${fetchItem}
         ${markItem}
         ${lifecycleItems}
         ${queueItem}
@@ -7902,6 +8019,10 @@ async function fetchTargetIndex(deps = defaultFetchDeps()) {
 	}
 	return targets;
 }
+/** Whether one review is a fetch target: the review page's question about itself. */
+async function isFetchTarget(reviewId, deps = defaultFetchDeps()) {
+	return (await fetchTargets(deps, /* @__PURE__ */ new Set([reviewId]))).eligible.has(reviewId);
+}
 var inFlight = /* @__PURE__ */ new Map();
 var queue = Promise.resolve();
 /**
@@ -8362,7 +8483,8 @@ function extractAnswer(raw) {
 * and wrongly.
 */
 function buildPrompt$2(workspace, question, sectionId, cwd) {
-	const section = workspace.session.result?.sections.find((s) => s.id === sectionId);
+	const sections = workspace.session.result?.sections ?? [];
+	const reading = readingThemeLine(sections, sectionId);
 	const recent = workspace.transcript.slice(-4);
 	return [
 		SYSTEM_PROMPT$1,
@@ -8374,29 +8496,19 @@ function buildPrompt$2(workspace, question, sectionId, cwd) {
 		"---",
 		"DIFF:",
 		workspace.diff,
-		...section ? ["---", `The reviewer is currently reading the "${section.title}" theme. Answer with that in view, but do not ignore the rest of the change if it bears on the question.`] : [],
+		...reading ? ["---", reading] : [],
 		...recent.length > 0 ? [
 			"---",
 			"EARLIER IN THIS CONVERSATION:",
-			recent.map((entry) => `Q ${askedWhere(workspace, entry.sectionId)}: ${entry.question}\nA: ${entry.answer}`).join("\n\n")
+			recent.map((entry) => {
+				const where = askedWhere(sections, entry.sectionId);
+				return `Q${where ? ` ${where}` : ""}: ${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"];
@@ -9429,6 +9541,7 @@ async function handle(req, res, options) {
 	if (action === "/job") return send(res, 200, jobReport(workspace));
 	if (action === "/retry") return handleRetry(res, workspace);
 	if (action === "/freshness") return handleFreshness(res, workspace, url.searchParams.get("quiet") === "1");
+	if (action === "/fetch") return handleFetchOwn(req, res, workspace);
 	if (action === "/refresh") return handleRefreshPr(res, workspace);
 	if (action === "/update") return handleUpdate(res, workspace);
 	if (action === "/review") return send(res, 200, workspace.session.result ?? {});
@@ -9503,10 +9616,10 @@ function handleIndex(req, res, params, indexToken) {
 	const cookie = indexCookieHeader(indexToken);
 	if (onward) return redirect(res, onward, cookie);
 	const states = analysisStates();
-	const queue = readQueue();
 	const reviews = recentWorkspaces(Number.MAX_SAFE_INTEGER);
-	const kept = indexKeptIds(reviews, queue);
 	const everyGroup = groupByLineage(reviews);
+	const queue = settleQueue(everyGroup);
+	const kept = indexKeptIds(reviews, queue);
 	const activity = reviewActivity(everyGroup);
 	const groups = indexReviewGroups(MAX_LISTED_BRANCHES, MAX_KEPT_BRANCHES, kept, (group) => activity.get(group.latest.id)?.at ?? group.latest.createdAt, everyGroup);
 	const rendered = new Set(groups.flatMap((group) => group.history.map((review) => review.id)));
@@ -9967,6 +10080,19 @@ async function handleFetchUpdates(req, res, indexToken) {
 	send(res, 200, await runFetchUpdates(new Set(ids)));
 }
 /**
+* Fetch remote updates, from one review's own page, on its own token.
+*
+* The list's fetch scoped to this review: `GET` says whether it applies (the page draws the item
+* only if so, asked after load so rendering never waits on git), `POST` runs it. The review page's
+* Check for updates stays read-only; this is the one explicit act on that page that reaches the
+* network, and it writes remote-tracking refs and nothing else.
+*/
+async function handleFetchOwn(req, res, workspace) {
+	if (req.method === "GET") return send(res, 200, { fetchable: await isFetchTarget(workspace.id) });
+	if (req.method !== "POST") return send(res, 405, { error: "Method not allowed." });
+	send(res, 200, await runFetchUpdates(/* @__PURE__ */ new Set([workspace.id])));
+}
+/**
 * Pin or unpin a review.
 *
 * Authorised by the index credential rather than the review's own, because pinning is a fact about
@@ -10280,7 +10406,7 @@ async function sendWorkspacePage(res, workspace) {
 		pendingDiagram: reportRun(workspace.id, "diagram"),
 		prompts: readPromptBank(),
 		complete: workspace.completedAt !== void 0,
-		saved: workspace.pinned === true,
+		pinned: workspace.pinned === true,
 		unreadNotifications: unreadCount$1(),
 		driving: isDriving(),
 		prContext: workspace.prContext ? {