prreviewbuddy 0.50.1 → 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";
@@ -242,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
@@ -269,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",
@@ -285,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
@@ -359,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",
@@ -660,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
@@ -3642,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>` : ""}
@@ -3674,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
@@ -3896,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>
@@ -3912,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">
@@ -3934,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
@@ -3970,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>
@@ -3979,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);
@@ -5157,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);
@@ -6836,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();
@@ -6847,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>
@@ -6884,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>
@@ -6921,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>`;
 }
 /**
@@ -6959,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
@@ -6988,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)}
@@ -7023,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>
@@ -7064,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">
@@ -7132,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({
@@ -7144,6 +7233,7 @@ function renderCardMenu(group, followed, queued, status, activity) {
 		queue: queued ? "remove" : "add",
 		mark,
 		lifecycle,
+		fetch: true,
 		copy: "Copy latest review ID"
 	});
 }
@@ -7162,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        ");
@@ -7169,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}
@@ -7927,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();
 /**
@@ -9445,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 ?? {});
@@ -9519,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)));
@@ -9983,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
@@ -10296,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 ? {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prreviewbuddy",
-  "version": "0.50.1",
+  "version": "0.52.2",
   "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/{chunk-BeL0x-wt.js → chunk-B5CvMZtt.js}

@@ -1 +1 @@
-function e(e,t){e.addEventListener(`pageshow`,e=>{e.persisted&&t()})}function t(e){return`${e} more`}function n(e){return e===0?`Nothing unread`:`${e} unread`}function r(e){return e===`restarting`?`Workspace server restarting`:e===`unreachable`?`Workspace server not running`:`Workspace server running`}function i(e,t){let n=e.querySelector(`[data-server-status]`);if(!n)return;n.dataset.serverStatus=t;let i=n.querySelector(`[data-server-status-label]`);i&&(i.textContent=r(t))}function a(e,t){let n=e.querySelector(`[data-server-status]`);!n||n.dataset.serverStatus===`restarting`||i(e,t?`running`:`unreachable`)}var o=6e3,s=5e3;function c(e=document){let t=e.querySelector(`.prb-toasts`);if(t)return t;let n=e.createElement(`div`);return n.className=`prb-toasts`,n.setAttribute(`role`,`status`),n.setAttribute(`aria-live`,`polite`),e.body.append(n),n}function l(e,t,n=window){return u(e,{kind:t.kind,text:t.text},n)}function u(e,t,n){let r=e.ownerDocument,i=r.createElement(`div`);i.className=`prb-toast`,i.dataset.kind=t.kind;let a=r.createElement(`span`);a.className=`prb-toast-text`,a.textContent=t.text;let s=r.createElement(`a`);s.className=`prb-toast-open`,t.href!==void 0&&t.link!==void 0&&(s.href=t.href,s.textContent=t.link);let c=r.createElement(`button`);for(c.type=`button`,c.className=`prb-toast-close`,c.setAttribute(`aria-label`,`Dismiss: ${t.text}`),c.textContent=`×`,i.append(a,...t.href===void 0?[]:[s],c),e.append(i);e.children.length>3;)e.firstElementChild?.remove();let l=n.setTimeout(()=>i.remove(),o),u=()=>n.clearTimeout(l),d=()=>{n.clearTimeout(l),l=n.setTimeout(()=>i.remove(),o)};return i.addEventListener(`mouseenter`,u),i.addEventListener(`mouseleave`,d),i.addEventListener(`focusin`,u),i.addEventListener(`focusout`,d),c.addEventListener(`click`,()=>{n.clearTimeout(l),i.remove()}),i}function d(e,t,n=window){return u(e,{kind:t.tone,text:t.headline,href:`/notifications/open?id=${encodeURIComponent(t.id)}`,link:`Open`},n)}function f(e,n,r=window){return u(e,{kind:`more`,text:t(n),href:`/notifications`,link:`View all`},r)}function p(e=document,t=window){let n=Number(e.body.dataset.notificationsSince);if(!Number.isFinite(n)||n<=0)return;let r=c(e),i=async()=>{try{let i=await t.fetch(`/notifications/recent?after=${n}`);if(a(e,!0),i.status===403)return;if(i.ok){let e=await i.json();typeof e.now==`number`&&(n=e.now);for(let n of[...e.notifications??[]].reverse())d(r,n,t);typeof e.more==`number`&&e.more>0&&f(r,e.more,t)}}catch{a(e,!1)}t.setTimeout(()=>void i(),s)};t.setTimeout(()=>void i(),s)}function m(e,t){if(typeof t==`number`)for(let r of e.querySelectorAll(`[data-unread-count]`))r.dataset.unreadStyle===`rail`&&(r.textContent=t===0?``:n(t)),r.hidden=t===0}function h(e,t){m(e,t.unread),typeof t.driving==`boolean`&&g(e,t.driving)}function g(e,t){let n=e.querySelector(`[data-restart]`),r=e.querySelector(`[data-restart-note]`);n&&(n.disabled=t),r&&(r.hidden=!t)}var _=`prb-server-restarted`;function v(e=document,t=window){let n=null;try{n=t.sessionStorage.getItem(_),t.sessionStorage.removeItem(_)}catch{}n!==null&&l(c(e),{kind:`ready`,text:`Server restarted. Workspace is back online.`},t)}function y(e){v();let t=document.querySelector(`[data-restart]`),n=document.querySelector(`[data-restart-note]`);t&&t.addEventListener(`click`,()=>{t.disabled=!0,b(e,t,n)})}async function b(e,t,n){let r;try{let i=await fetch(e,{method:`POST`});if(i.status===409){g(document,!0);return}if(!i.ok)return void w(t,n,await C(i));let a=await i.json();r=typeof a.startedAt==`number`?a.startedAt:null}catch{w(t,n,`The workspace server could not be reached.`);return}if(document.body.dataset.restarting=`true`,i(document,`restarting`),await x(r)){try{window.sessionStorage.setItem(_,`1`)}catch{}window.location.reload();return}if(delete document.body.dataset.restarting,await S()){i(document,`running`),w(t,n,`The server did not restart, and is still the one that was running.`);return}i(document,`unreachable`),w(t,n,`Server restart failed. Run prreviewbuddy open to start the workspace again.`)}async function x(e){let t=Date.now()+2e4;for(;Date.now()<t;){await new Promise(e=>setTimeout(e,300));try{let t=await fetch(`/health`,{cache:`no-store`});if(!t.ok)continue;let n=await t.json();if(typeof n.startedAt==`number`&&n.startedAt!==e)return!0}catch{}}return!1}async function S(){try{return(await fetch(`/health`,{cache:`no-store`})).ok}catch{return!1}}async function C(e){try{return(await e.json()).error??`The server could not be restarted.`}catch{return`The server could not be restarted.`}}function w(e,t,n){e.disabled=!1,t&&(t.textContent=n,t.hidden=!1)}export{a,e as c,p as i,h as n,i as o,m as r,n as s,y as t};
+function e(e,t){e.addEventListener(`pageshow`,e=>{e.persisted&&t()})}function t(e){return`${e} more`}function n(e){return e===0?`Nothing unread`:`${e} unread`}function r(e){return e===`restarting`?`Workspace server restarting`:e===`unreachable`?`Workspace server not running`:`Workspace server running`}function i(e,t){let n=e.querySelector(`[data-server-status]`);if(!n)return;n.dataset.serverStatus=t;let i=n.querySelector(`[data-server-status-label]`);i&&(i.textContent=r(t))}function a(e,t){let n=e.querySelector(`[data-server-status]`);!n||n.dataset.serverStatus===`restarting`||i(e,t?`running`:`unreachable`)}var o=6e3,s=5e3;function c(e=document){let t=e.querySelector(`.prb-toasts`);if(t)return t;let n=e.createElement(`div`);return n.className=`prb-toasts`,n.setAttribute(`role`,`status`),n.setAttribute(`aria-live`,`polite`),e.body.append(n),n}function l(e,t,n=window){return u(e,{kind:t.kind,text:t.text},n)}function u(e,t,n){let r=e.ownerDocument,i=r.createElement(`div`);i.className=`prb-toast`,i.dataset.kind=t.kind;let a=r.createElement(`span`);a.className=`prb-toast-text`,a.textContent=t.text;let s=r.createElement(`a`);s.className=`prb-toast-open`,t.href!==void 0&&t.link!==void 0&&(s.href=t.href,s.textContent=t.link);let c=r.createElement(`button`);for(c.type=`button`,c.className=`prb-toast-close`,c.setAttribute(`aria-label`,`Dismiss: ${t.text}`),c.textContent=`×`,i.append(a,...t.href===void 0?[]:[s],c),e.append(i);e.children.length>3;)e.firstElementChild?.remove();let l=n.setTimeout(()=>i.remove(),o),u=()=>n.clearTimeout(l),d=()=>{n.clearTimeout(l),l=n.setTimeout(()=>i.remove(),o)};return i.addEventListener(`mouseenter`,u),i.addEventListener(`mouseleave`,d),i.addEventListener(`focusin`,u),i.addEventListener(`focusout`,d),c.addEventListener(`click`,()=>{n.clearTimeout(l),i.remove()}),i}function d(e,t,n=window){return u(e,{kind:t.tone,text:t.headline,href:`/notifications/open?id=${encodeURIComponent(t.id)}`,link:`Open`},n)}function f(e,n,r=window){return u(e,{kind:`more`,text:t(n),href:`/notifications`,link:`View all`},r)}function p(e=document,t=window){let n=Number(e.body.dataset.notificationsSince);if(!Number.isFinite(n)||n<=0)return;let r=c(e),i=async()=>{try{let i=await t.fetch(`/notifications/recent?after=${n}`);if(a(e,!0),i.status===403)return;if(i.ok){let e=await i.json();typeof e.now==`number`&&(n=e.now);for(let n of[...e.notifications??[]].reverse())d(r,n,t);typeof e.more==`number`&&e.more>0&&f(r,e.more,t)}}catch{a(e,!1)}t.setTimeout(()=>void i(),s)};t.setTimeout(()=>void i(),s)}function m(e,t){if(typeof t==`number`)for(let r of e.querySelectorAll(`[data-unread-count]`))r.dataset.unreadStyle===`rail`&&(r.textContent=t===0?``:n(t)),r.hidden=t===0}function h(e,t){m(e,t.unread),typeof t.driving==`boolean`&&g(e,t.driving)}function g(e,t){let n=e.querySelector(`[data-restart]`),r=e.querySelector(`[data-restart-note]`);n&&(n.disabled=t),r&&(r.hidden=!t)}var _=`prb-server-restarted`;function v(e=document,t=window){let n=null;try{n=t.sessionStorage.getItem(_),t.sessionStorage.removeItem(_)}catch{}n!==null&&l(c(e),{kind:`ready`,text:`Server restarted. Workspace is back online.`},t)}function y(e){v();let t=document.querySelector(`[data-restart]`),n=document.querySelector(`[data-restart-note]`);t&&t.addEventListener(`click`,()=>{t.disabled=!0,b(e,t,n)})}async function b(e,t,n){let r;try{let i=await fetch(e,{method:`POST`});if(i.status===409){g(document,!0);return}if(!i.ok)return void w(t,n,await C(i));let a=await i.json();r=typeof a.startedAt==`number`?a.startedAt:null}catch{w(t,n,`The workspace server could not be reached.`);return}if(document.body.dataset.restarting=`true`,i(document,`restarting`),await x(r)){try{window.sessionStorage.setItem(_,`1`)}catch{}window.location.reload();return}if(delete document.body.dataset.restarting,await S()){i(document,`running`),w(t,n,`The server did not restart, and is still the one that was running.`);return}i(document,`unreachable`),w(t,n,`Server restart failed. Run prreviewbuddy open to start the workspace again.`)}async function x(e){let t=Date.now()+2e4;for(;Date.now()<t;){await new Promise(e=>setTimeout(e,300));try{let t=await fetch(`/health`,{cache:`no-store`});if(!t.ok)continue;let n=await t.json();if(typeof n.startedAt==`number`&&n.startedAt!==e)return!0}catch{}}return!1}async function S(){try{return(await fetch(`/health`,{cache:`no-store`})).ok}catch{return!1}}async function C(e){try{return(await e.json()).error??`The server could not be restarted.`}catch{return`The server could not be restarted.`}}function w(e,t,n){e.disabled=!1,t&&(t.textContent=n,t.hidden=!1)}export{p as a,i as c,l as i,n as l,h as n,c as o,m as r,a as s,y as t,e as u};