prreviewbuddy 0.43.0 → 0.45.0

package/dist/server.js

@@ -1,4 +1,4 @@
-import { A as createPrState, An as AGENT_IDS, At as loadJob, B as toneOf, Bt as followedRefName, C as readEvents, Ct as checkReview, D as announceJob, Dn as writeAgentPreference, Dt as endedAt, E as telemetryUploadConsent, En as readAgentPreference, Et as allJobs, F as markAllRead, Fn as AgentCancelledError, Ft as progressSteps, G as MANAGED_ROOT, Gt as lineagePosition, H as withUsageRecorded, Ht as indexReviewGroups, I as markRead, In as AgentUnavailableError, It as purposeOf, Jt as mutateWorkspace, K as readMarker, Kt as loadWorkspace, L as pruneNotifications, M as eventName, N as headline, Nn as agentFor, Nt as isLiveJob, O as branchOf, Ot as fail, P as listNotifications, Pt as phasesFor, Q as clearServerRecord, Qt as reviewedCommit, R as readNotification, Sn as git, St as checkCodeFreshness, T as recordTelemetryUploadConsent, Tn as clearAgentPreference, Tt as reviewedShaOf, U as EXPLAIN_SIMPLY_PROMPT, Ut as latestKissRun, V as unreadCount$1, W as ReviewBeingDeletedError, X as readIndexToken, Xt as previousKissResult, Y as relativeTime$1, Zt as recentWorkspaces, _ as agentEnvOf, _n as processDiscussion, _t as claimRequestUrl, a as wasBlocked, an as queueAdd, bt as remoteHost, c as RefreshUnavailableError, cn as queueStamp, d as refreshPrContext, dn as doneVerb, dt as MAX_WORKSPACE_PORT, et as readServerRecord, f as isUnchanged, fn as isIssueOutstanding, ft as MIN_WORKSPACE_PORT, gn as reviewerDispositions, gt as writePortPreference, h as liveUpdateFor, hn as questionsOutstanding, ht as statedWorkspacePort, i as liveJobsFor, in as dequeueOnComplete, j as createPrUpdate, jt as saveJob, k as subjectOf, kn as STORE_ROOT, kt as isTerminal, l as mergeUpdateInto, ln as readQueue, lt as feedbackUrl, m as UpdateAlreadyRunningError, mn as issuesOutstanding, mt as resolveWorkspacePort, n as deleteReview, nn as workspaceRevision, nt as serverEntryExists, o as discardJob, on as queueAddMany, ot as writeServerRecord, p as askCheckout, pn as isQuestionOutstanding, pt as parseWorkspacePort, q as removeWorktree, r as lineageIds, rn as workspaceStamps, rt as spawnServer, s as FETCH_TIMEOUT_MS, sn as queueRemove, st as BUILD_VERSION, t as deleteLineage, tn as touchWorkspace, u as updateReview, un as writeQueue, ut as DEFAULT_WORKSPACE_PORT, vn as describeAuthorship, w as record, wn as resolveModel, wt as followedBranch, x as runJob, xn as isCommitReview, xt as fillFileUrlTemplate, y as startKissJob, yt as forgeFor, z as removeNotification, zt as isClaimed } from "./delete_review-DPRL5IPt.js";
+import { $t as recentWorkspaces, A as createPrState, An as STORE_ROOT, At as fail, B as toneOf, C as readEvents, Cn as git, Ct as fillFileUrlTemplate, D as announceJob, Dn as readAgentPreference, Dt as reviewedShaOf, E as telemetryUploadConsent, En as clearAgentPreference, Et as followedBranch, F as markAllRead, Ft as isLiveJob, G as MANAGED_ROOT, Gt as latestKissRun, H as withUsageRecorded, Ht as followedRefName, I as markRead, In as AgentCancelledError, It as phasesFor, Jt as loadWorkspace, K as readMarker, L as pruneNotifications, Ln as AgentUnavailableError, Lt as progressSteps, M as eventName, Mt as loadJob, N as headline, Nt as saveJob, O as branchOf, On as writeAgentPreference, Ot as allJobs, P as listNotifications, Pn as agentFor, Q as readIndexToken, Qt as previousKissResult, R as readNotification, Rt as purposeOf, Sn as isCommitReview, St as remoteHost, T as recordTelemetryUploadConsent, Tn as resolveModel, Tt as checkReview, U as EXPLAIN_SIMPLY_PROMPT, V as unreadCount$1, Vt as isClaimed, W as ReviewBeingDeletedError, Wt as indexReviewGroups, X as compactQueue, Xt as mutateWorkspace, Y as relativeTime$1, _ as agentEnvOf, _n as reviewerDispositions, _t as statedWorkspacePort, a as wasBlocked, an as workspaceStamps, at as spawnServer, c as RefreshUnavailableError, cn as queueAddMany, ct as writeServerRecord, d as refreshPrContext, dn as readQueue, dt as feedbackUrl, en as reviewedCommit, et as clearServerRecord, f as isUnchanged, fn as doneVerb, ft as DEFAULT_WORKSPACE_PORT, gn as questionsOutstanding, gt as resolveWorkspacePort, h as liveUpdateFor, hn as issuesOutstanding, ht as parseWorkspacePort, i as liveJobsFor, in as workspaceRevision, it as serverEntryExists, j as createPrUpdate, jn as AGENT_IDS, jt as isTerminal, k as subjectOf, kt as endedAt, l as mergeUpdateInto, ln as queueRemove, lt as BUILD_VERSION, m as UpdateAlreadyRunningError, mn as isQuestionOutstanding, mt as MIN_WORKSPACE_PORT, n as deleteReview, nt as readServerRecord, o as discardJob, on as dequeueOnComplete, p as askCheckout, pn as isIssueOutstanding, pt as MAX_WORKSPACE_PORT, q as removeWorktree, qt as lineagePosition, r as lineageIds, rn as touchWorkspace, s as FETCH_TIMEOUT_MS, sn as queueAdd, t as deleteLineage, u as updateReview, un as queueStamp, vn as processDiscussion, vt as writePortPreference, w as record, wt as checkCodeFreshness, x as runJob, xt as forgeFor, y as startKissJob, yn as describeAuthorship, yt as claimRequestUrl, z as removeNotification } from "./delete_review-BWMmO81Q.js";
 import { basename, dirname, isAbsolute, join, relative, resolve } from "node:path";
 import { existsSync, mkdirSync, readFileSync, readdirSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { createHash, timingSafeEqual } from "node:crypto";
@@ -744,19 +744,30 @@ var STATES$1 = {
 function renderMessageCopySource(source) {
 	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"`);
 }
+/** The class that places a finding's copy control in its card's heading, and gives it its tooltip. */
+var FINDING_COPY_CLASS = "pr-helper-finding-copy";
 /**
 * The same control on an issue or question card, carrying that finding as Markdown.
 *
-* No action-row wrapper, because the card already has one to sit in. It shows a word at rest where
-* a chat reply shows only the icon: under a reply the icon is the whole affordance and the text is
-* the subject, but a card's row is a line of worded buttons, and an icon alone among them reads as
-* decoration. Both resting words are carried on the button so the reset after "Copied" returns to
-* them rather than to the chat reply's.
+* It sits in the card's heading, not its footer. It copies the whole finding, so the heading is
+* the thing it acts on. And it is a different kind of act from Investigate, Dismiss and Mark
+* answered, which change the review: those keep the footer to themselves, so the footer holds
+* only actions that change something.
+*
+* Icon only at rest, like a chat reply's. It used to show a word because it sat in a row of worded
+* buttons, where an icon alone read as decoration. In the heading it is on its own, and a word
+* would give a utility the weight of the title beside it. What it does is said by a hover
+* tooltip (`data-tip`, the same CSS tooltip the pills use, not a native `title`) and by its
+* accessible name. "Copied" and "Copy failed" still show as words, because a picture cannot say
+* that the click did something. The resting name is kept on the button so the reset returns to
+* "Copy finding" rather than to a chat reply's "Copy message".
 */
 function renderFindingCopySource(source, kind) {
+	const name = kind === "issue" ? "Copy finding" : "Copy question";
 	return button(`${MESSAGE_SOURCE_ATTR}="${escapeHtml$1(source)}"`, {
-		label: "Copy",
-		aria: `Copy ${kind}`
+		label: "",
+		aria: name,
+		tip: name
 	});
 }
 /** Where a finding's button keeps its resting word and name. Absent on a chat reply's. */
@@ -766,13 +777,42 @@ function button(handle, idle) {
 	const { icon } = STATES$1.idle;
 	const label = idle?.label ?? STATES$1.idle.label;
 	const aria = idle?.aria ?? STATES$1.idle.aria;
-	const control = `<button type="button" class="${MESSAGE_COPY_CLASS}" ${handle}${idle ? ` ${IDLE_LABEL_ATTR}="${escapeHtml$1(label)}" ${IDLE_ARIA_ATTR}="${escapeHtml$1(aria)}"` : ""} aria-label="${escapeHtml$1(aria)}"><span class="${MESSAGE_COPY_CLASS}-icon" aria-hidden="true">${svg(icon)}</span><span class="${MESSAGE_COPY_CLASS}-label"${label ? "" : " hidden"}>${escapeHtml$1(label)}</span></button>`;
+	const remembered = idle ? ` ${IDLE_LABEL_ATTR}="${escapeHtml$1(label)}" ${IDLE_ARIA_ATTR}="${escapeHtml$1(aria)}"` : "";
+	const control = `<button type="button" class="${idle ? `${MESSAGE_COPY_CLASS} ${FINDING_COPY_CLASS}` : MESSAGE_COPY_CLASS}" ${handle}${remembered}${idle ? ` data-tip="${escapeHtml$1(idle.tip)}"` : ""} aria-label="${escapeHtml$1(aria)}"><span class="${MESSAGE_COPY_CLASS}-icon" aria-hidden="true">${svg(icon)}</span><span class="${MESSAGE_COPY_CLASS}-label"${label ? "" : " hidden"}>${escapeHtml$1(label)}</span></button>`;
 	return idle ? control : `<div class="${MESSAGE_ACTIONS_CLASS}">${control}</div>`;
 }
 function svg(paths) {
 	return `<svg viewBox="0 0 16 16" width="14" height="14" fill="currentColor">${paths}</svg>`;
 }
 //#endregion
+//#region ../../packages/review-harness/src/domain/ui/components/ask_buddy_menu.ts
+/**
+* Ask Buddy on a finding card: a sparkle beside Copy that opens the assistant's actions for the
+* whole finding.
+*
+* The selection menu has offered these actions for a long time, but only to someone who thought to
+* highlight text first, and few did. This puts the same actions on the card itself. The selection
+* menu stays the precision tool for asking about an excerpt.
+*
+* The markup is the page's existing `[data-menu]` menu (the header's `...` uses it), so it opens,
+* 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.
+*/
+var ASK_BUDDY_LABEL = "Ask Buddy";
+/** Four-point sparkle, the conventional mark for "ask the AI". currentColor, like Copy beside it. */
+var SPARKLE = `<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16" width="14" height="14" fill="currentColor" aria-hidden="true"><path d="M7.53 1.28a.5.5 0 0 1 .94 0l.9 2.6a3 3 0 0 0 1.85 1.85l2.6.9a.5.5 0 0 1 0 .94l-2.6.9a3 3 0 0 0-1.85 1.85l-.9 2.6a.5.5 0 0 1-.94 0l-.9-2.6a3 3 0 0 0-1.85-1.85l-2.6-.9a.5.5 0 0 1 0-.94l2.6-.9a3 3 0 0 0 1.85-1.85z"/><path d="M13 11.2a.3.3 0 0 1 .56 0l.3.86a1 1 0 0 0 .62.62l.86.3a.3.3 0 0 1 0 .56l-.86.3a1 1 0 0 0-.62.62l-.3.86a.3.3 0 0 1-.56 0l-.3-.86a1 1 0 0 0-.62-.62l-.86-.3a.3.3 0 0 1 0-.56l.86-.3a1 1 0 0 0 .62-.62z"/></svg>`;
+/** `kind` goes into the accessible name, so a screen reader hears which finding it is about. */
+function renderAskBuddyMenu(kind) {
+	const name = `${ASK_BUDDY_LABEL} about this ${kind}`;
+	return `<div class="prb-menu prb-ask-buddy" data-menu data-ask-buddy-menu>
+    <button type="button" class="prb-menu-trigger prb-ask-buddy-trigger" data-menu-trigger aria-haspopup="menu" aria-expanded="false" aria-label="${escapeHtml$1(name)}" data-tip="${escapeHtml$1(ASK_BUDDY_LABEL)}">${SPARKLE}</button>
+    <div class="prb-menu-panel prb-ask-buddy-panel" data-menu-panel role="menu" aria-label="${escapeHtml$1(name)}" hidden></div>
+  </div>`;
+}
+//#endregion
 //#region ../../packages/review-harness/src/domain/ui/components/finding_copy.ts
 /** An issue: its title in bold, the description, then the location when it has one. */
 function issueMarkdown(issue) {
@@ -2027,7 +2067,7 @@ function renderReconciliationNote(note) {
 * stuck. Outlined rather than filled — see sidebar.css.
 *
 * Text only, no leading icon. Every other action in the sidebar (Jump to file, Mark answered,
-* Explain, Analyse, Copy) is a bare label, and an emoji here read as decoration on a control
+* Explain, Analyse) is a bare label, and an emoji here read as decoration on a control
 * whose neighbours have none rather than as emphasis.
 */
 function renderInvestigateButton(attribute, id, label) {
@@ -2127,7 +2167,7 @@ function renderClosedFindings(summary, cards, kind, nothingOpen) {
       <div class="pr-helper-closed-findings-list">${cards}</div>
     </details>`;
 }
-function renderIssuesTab(session, investigatingId) {
+function renderIssuesTab(session, investigatingId, options) {
 	if (!session) return emptyState("Generate an analysis to see issues.");
 	const outcome = outcomeOf(session);
 	if (outcome.kind === "none") return emptyState("Generate an analysis to see issues.");
@@ -2147,7 +2187,11 @@ function renderIssuesTab(session, investigatingId) {
 		return `
     <div class="pr-helper-issue-card pr-helper-severity-${issue.severity}${issue.reconciliationStatus ? ` pr-helper-issue-${issue.reconciliationStatus}` : ""}${issue.investigationImpact ? ` pr-helper-issue-investigation-${issue.investigationImpact}` : ""}${isDismissed && !stale ? " pr-helper-issue-card-dismissed" : ""}" data-finding-card="${escapeHtml$1(issue.id)}">
       <div class="pr-helper-issue-header">
-        <p class="pr-helper-issue-title">${renderInlineText(issue.title)}</p>
+        <div class="pr-helper-issue-title-row">
+          <p class="pr-helper-issue-title">${renderInlineText(issue.title)}</p>
+          ${renderFindingCopySource(issueMarkdown(issue), "issue")}
+          ${options?.askBuddy ? renderAskBuddyMenu("issue") : ""}
+        </div>
         <div class="pr-helper-issue-badges">
           ${renderFindingStatus(issue.reconciliationStatus)}
           ${renderDismissedBadge(isDismissed)}
@@ -2167,7 +2211,6 @@ function renderIssuesTab(session, investigatingId) {
       ${renderRelatedFiles(issue.relatedFiles)}
       ${renderInvestigationProvenance("issue", issue.id, records, now)}
       <div class="pr-helper-issue-actions">
-        ${renderFindingCopySource(issueMarkdown(issue), "issue")}
         ${renderInvestigationActions("issue", issue.id, records, investigatingId)}
         ${renderDismissToggle(issue, isDismissed, stale, dismissals.get(issue.id))}
       </div>
@@ -2236,7 +2279,11 @@ function renderQuestionCard(q, sections, answered, records, now = Date.now(), in
 	return `
     <div class="pr-helper-question-card${answered ? " pr-helper-question-card-answered" : ""}${q.investigationImpact ? ` pr-helper-issue-investigation-${q.investigationImpact}` : ""}" data-finding-card="${escapeHtml$1(q.id)}"${q.sectionId ? ` data-section-id="${escapeHtml$1(q.sectionId)}"` : ""}>
       <div class="pr-helper-issue-header">
-        <p class="pr-helper-issue-title">${renderInlineText(q.question)}</p>
+        <div class="pr-helper-issue-title-row">
+          <p class="pr-helper-issue-title">${renderInlineText(q.question)}</p>
+          ${renderFindingCopySource(questionMarkdown(q), "question")}
+          ${options?.askBuddy ? renderAskBuddyMenu("question") : ""}
+        </div>
         <div class="pr-helper-issue-badges">
           ${renderQuestionStatus(q.reconciliationStatus)}
           ${renderInvestigationImpact(q.investigationImpact)}
@@ -2259,7 +2306,6 @@ function renderQuestionCard(q, sections, answered, records, now = Date.now(), in
       <div class="pr-helper-question-actions">
         ${jumpLink}
         <div class="pr-helper-question-actions-group">
-          ${renderFindingCopySource(questionMarkdown(q), "question")}
           ${renderInvestigationActions("question", q.id, records, investigatingId)}
           ${answerToggle}
         </div>
@@ -2334,6 +2380,70 @@ function renderQuestionsTab(session, investigatingId, options) {
     `).join("")}${ungrouped.map(card).join("")}${closed}`;
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/doc_head.ts
+/**
+* The title row a document pane opens with, and the provenance disclosure at the right of it.
+*
+* Two things sit in this row and they are not the same kind of thing. The title names the pane;
+* Analysis details answers "how was this made, and does it describe the code I am looking at" --
+* when it completed, which commit, which audience, which model, what it spent, which rubric. That
+* is a question about the whole analysis, so it belongs level with the pane's name rather than
+* below the output: a reader who meets it after the recommendations has already decided how much
+* to trust them.
+*
+* A control, not a strip. Full width would give provenance the weight of content, which it is not,
+* and it is the reason the disclosure opens into a panel anchored under the button rather than
+* pushing the report down the page. Opening it must not move the thing the reader was reading.
+*
+* This row lives inside the light document pane, never in the dark page header above it. The
+* header names the change; these details describe one run against it, and the two would be read as
+* one fact if they shared a surface. The `prb-doc-` prefix is load-bearing for that: it is what
+* `pane_palette.test.ts` looks for when it checks that nothing in the pane borrows a shell colour.
+*/
+/**
+* The panel's contents are plain sentences, escaped here, because every caller is passing facts
+* rather than markup and one of them is a model's own words about a file it cited.
+*/
+function renderAnalysisDetails(items) {
+	if (items.length === 0) return "";
+	return `<details class="prb-doc-provenance" data-analysis-details>
+  <summary class="prb-doc-provenance-summary">Analysis details</summary>
+  <div class="prb-doc-provenance-panel">
+    <ul>${items.map((item) => `<li>${escapeHtml$1(item)}</li>`).join("")}</ul>
+  </div>
+</details>`;
+}
+/**
+* `aside` is markup rather than a string of text: it is a rendered control, and the only one so
+* far is the disclosure above. Empty when a pane has no provenance to show, which is every state
+* before a run has produced something, and the row then draws as the bare title it used to be.
+*/
+function renderDocHead(title, aside = "") {
+	return `<div class="prb-doc-head">
+  <h3 class="prb-doc-title">${escapeHtml$1(title)}</h3>
+  ${aside}
+</div>`;
+}
+/**
+* Expand a surface to fill the workspace, or bring it back.
+*
+* Icon-only, like Copy on a card, because it sits in a title row and a word there would compete
+* with the title. Both icons are in the markup and the stylesheet shows the one `aria-pressed`
+* calls for, so the client only has to flip the attribute (see `paintFocus` in focus_mode.ts),
+* which also rewrites the name to say what pressing it will do.
+*
+* `name` is the surface as a person would say it, and goes into the accessible name.
+*/
+var EXPAND_ICON = "<path d=\"M15 3h6v6M9 21H3v-6M21 3l-7 7M3 21l7-7\"/>";
+var COLLAPSE_ICON = "<path d=\"M4 14h6v6M20 10h-6V4M14 10l7-7M3 21l7-7\"/>";
+function focusIcon(paths, className) {
+	return `<svg class="${className}" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="15" height="15" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true">${paths}</svg>`;
+}
+function renderFocusToggle(surface, name) {
+	const label = `Expand ${name}`;
+	return `<button type="button" class="prb-focus-toggle" data-focus-toggle="${surface}" data-focus-name="${escapeHtml$1(name)}" aria-pressed="false" aria-label="${escapeHtml$1(label)}" title="${escapeHtml$1(label)}">${focusIcon(EXPAND_ICON, "prb-focus-expand")}${focusIcon(COLLAPSE_ICON, "prb-focus-collapse")}</button>`;
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/guide_view.ts
 /**
 * The workspace's Guide: one theme at a time.
@@ -2384,7 +2494,7 @@ function renderWorkspaceGuide(session, followUpInFlight = null) {
 		return `<div class="prb-guide-detail" data-guide-detail="${escapeHtml$1(section.id)}"${hidden}>${card}</div>`;
 	}).join("");
 	return `
-    <div class="pr-helper-guide prb-guide">
+    <div class="pr-helper-guide prb-guide" data-guide-start="${escapeHtml$1(selected.id)}">
       ${renderGuideProgress(sections, completed)}
       ${renderThemePicker(sections, selected.id)}
       <div class="prb-guide-details">${details}</div>
@@ -2404,7 +2514,7 @@ function renderGuideProgress(sections, completed) {
 	const done = sections.filter((section) => completed.has(section.id)).length;
 	const left = remainingMinutes(sections, completed);
 	const time = left > 0 ? ` &middot; ~${left} min` : "";
-	return `<p class="prb-guide-progress">${done > 0 ? `${done} of ${total} theme${total === 1 ? "" : "s"} completed` : `${total} theme${total === 1 ? "" : "s"}`}${time}</p>`;
+	return `<div class="prb-guide-head"><p class="prb-guide-progress">${done > 0 ? `${done} of ${total} theme${total === 1 ? "" : "s"} completed` : `${total} theme${total === 1 ? "" : "s"}`}${time}</p>${renderFocusToggle("guide", "Review path")}</div>`;
 }
 /**
 * The same theme list the rail draws, for the width where the rail is not drawn.
@@ -2418,51 +2528,6 @@ function renderThemePicker(sections, selectedId) {
 	return `<div class="prb-guide-picker">${sections.map((section, index) => `<button type="button" class="prb-guide-pick" data-goto-section="${escapeHtml$1(section.id)}" data-theme-index="${index}" data-recommended-start="${section.recommendedStartingPoint}" aria-current="${section.id === selectedId}"><span class="prb-guide-pick-order">${index + 1}</span>${escapeHtml$1(section.title)}</button>`).join("")}</div>`;
 }
 //#endregion
-//#region ../../packages/review-harness/src/workspace/doc_head.ts
-/**
-* The title row a document pane opens with, and the provenance disclosure at the right of it.
-*
-* Two things sit in this row and they are not the same kind of thing. The title names the pane;
-* Analysis details answers "how was this made, and does it describe the code I am looking at" --
-* when it completed, which commit, which audience, which model, what it spent, which rubric. That
-* is a question about the whole analysis, so it belongs level with the pane's name rather than
-* below the output: a reader who meets it after the recommendations has already decided how much
-* to trust them.
-*
-* A control, not a strip. Full width would give provenance the weight of content, which it is not,
-* and it is the reason the disclosure opens into a panel anchored under the button rather than
-* pushing the report down the page. Opening it must not move the thing the reader was reading.
-*
-* This row lives inside the light document pane, never in the dark page header above it. The
-* header names the change; these details describe one run against it, and the two would be read as
-* one fact if they shared a surface. The `prb-doc-` prefix is load-bearing for that: it is what
-* `pane_palette.test.ts` looks for when it checks that nothing in the pane borrows a shell colour.
-*/
-/**
-* The panel's contents are plain sentences, escaped here, because every caller is passing facts
-* rather than markup and one of them is a model's own words about a file it cited.
-*/
-function renderAnalysisDetails(items) {
-	if (items.length === 0) return "";
-	return `<details class="prb-doc-provenance" data-analysis-details>
-  <summary class="prb-doc-provenance-summary">Analysis details</summary>
-  <div class="prb-doc-provenance-panel">
-    <ul>${items.map((item) => `<li>${escapeHtml$1(item)}</li>`).join("")}</ul>
-  </div>
-</details>`;
-}
-/**
-* `aside` is markup rather than a string of text: it is a rendered control, and the only one so
-* far is the disclosure above. Empty when a pane has no provenance to show, which is every state
-* before a run has produced something, and the row then draws as the bare title it used to be.
-*/
-function renderDocHead(title, aside = "") {
-	return `<div class="prb-doc-head">
-  <h3 class="prb-doc-title">${escapeHtml$1(title)}</h3>
-  ${aside}
-</div>`;
-}
-//#endregion
 //#region ../../packages/review-harness/src/workspace/diagram_view.ts
 /**
 * The diagrams a review holds, drawn as cards in one scrolling pane.
@@ -4216,7 +4281,7 @@ function renderKissViews(view) {
 function renderFilesView(view, sections) {
 	if (!(view.session.result !== null || (view.job ? changeIsRead(view.job) : false))) return "";
 	return `<section class="prb-view" data-view="files" hidden>
-        <h3 class="prb-doc-title">Files</h3>
+        ${renderDocHead("Files", renderFocusToggle("files", "Files"))}
         ${renderFiles(view, sections)}
       </section>`;
 }
@@ -4234,13 +4299,16 @@ function renderStandardViews(view, sections) {
         ${renderDiagrams(view.diagrams ?? [], sections, view.reviewedSha ?? null)}
       </section>
       <section class="prb-view" data-view="issues" hidden>
-        <h3 class="prb-doc-title">Issues</h3>
+        ${renderDocHead("Issues", renderFocusToggle("issues", "Issues"))}
         ${renderAttributionAge(view)}
-        ${renderIssuesTab(session, null)}
+        ${renderIssuesTab(session, null, { askBuddy: true })}
       </section>
       <section class="prb-view" data-view="questions" hidden>
-        <h3 class="prb-doc-title">Questions</h3>
-        ${renderQuestionsTab(session, null, { suppressFileJump: true })}
+        ${renderDocHead("Questions", renderFocusToggle("questions", "Questions"))}
+        ${renderQuestionsTab(session, null, {
+		suppressFileJump: true,
+		askBuddy: true
+	})}
       </section>
       <section class="prb-view" data-view="prcontext" hidden>
         ${renderPrContextView(view.prContext ?? null, {
@@ -4975,8 +5043,9 @@ function renderAsk(view, sections, paused = false) {
   <div class="prb-ask-header">
     <h2>Ask Buddy AI</h2>
     <span class="prb-ask-scope" data-ask-scope>whole change</span>
+    ${renderFocusToggle("buddy", "Buddy AI")}
   </div>
-  ${paused ? "" : renderPromptBank(view.prompts ?? [])}
+  ${paused ? "" : renderPromptBank(view.prompts ?? [], view.promptsCollapsed === true)}
   <div class="prb-ask-log" data-ask-log>
     ${transcript.length > 0 || pending ? transcript.map((exchange) => `<div class="prb-msg prb-msg-user">${escapeHtml$1(exchange.question)}</div>
       <div class="prb-msg prb-msg-assistant prb-md">${renderMarkdown(exchange.answer)}${renderMessageCopySource(exchange.answer)}</div>`).join("") + pendingTurn : `<div class="prb-ask-empty"${paused ? " inert" : ""}>
@@ -5001,13 +5070,16 @@ function renderAsk(view, sections, paused = false) {
 * The list is always present, even when empty: an empty bank with an Add button explains what
 * the feature is, where a hidden one leaves it undiscoverable until someone reads a changelog.
 */
-function renderPromptBank(prompts) {
-	return `<div class="prb-bank" data-prompt-bank data-prompts="${escapeHtml$1(JSON.stringify(prompts))}">
+function renderPromptBank(prompts, collapsed = false) {
+	return `<div class="prb-bank" data-prompt-bank data-prompts="${escapeHtml$1(JSON.stringify(prompts))}"${collapsed ? " data-collapsed" : ""}>
     <div class="prb-bank-head">
-      <span>Your prompts</span>
+      <button type="button" class="prb-bank-toggle" data-prompt-toggle aria-expanded="${!collapsed}" aria-controls="prb-bank-list">
+        <svg class="prb-bank-chevron" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16" width="10" height="10" fill="currentColor" aria-hidden="true"><path d="M6 4l4 4-4 4z"/></svg>
+        <span>Your prompts</span><span class="prb-bank-count" data-prompt-count>${prompts.length}</span>
+      </button>
       <button type="button" class="prb-bank-add" data-prompt-add>Add</button>
     </div>
-    <div class="prb-bank-list" data-prompt-list>${prompts.map(renderPromptChip).join("")}</div>
+    <div class="prb-bank-list" id="prb-bank-list" data-prompt-list${collapsed ? " hidden" : ""}>${prompts.map(renderPromptChip).join("")}</div>
   </div>`;
 }
 function renderPromptChip(prompt) {
@@ -6239,7 +6311,7 @@ function renderCard(group, now, analysis, queued, moved, prState) {
         ${renderStatus(status)}
         ${renderPrState(prState)}
       </div>
-      ${review.summary ? `<p class="prb-card-summary">${escapeHtml$1(review.summary)}</p>` : ""}
+      ${review.summary ? `<p class="prb-card-summary">${renderInlineText(review.summary)}</p>` : ""}
       <p class="prb-card-where">
         <span class="prb-card-repo">${escapeHtml$1(lastSegment(review.repoPath))}</span>
         ${review.prNumber === void 0 ? "" : `<span class="prb-card-pr">#${review.prNumber}</span>`}
@@ -7918,10 +7990,11 @@ function readPreferences() {
 	try {
 		const parsed = JSON.parse(readFileSync(PREFERENCES_PATH, "utf8"));
 		if (!parsed || typeof parsed !== "object") return {};
-		const { askWidth, dismissedNotices } = parsed;
+		const { askWidth, dismissedNotices, promptsCollapsed } = parsed;
 		return {
 			...typeof askWidth === "number" && Number.isFinite(askWidth) ? { askWidth: clampAskWidth(askWidth) } : {},
-			...Array.isArray(dismissedNotices) ? { dismissedNotices: dismissedNotices.filter((key) => typeof key === "string" && NOTICE_KEY.test(key)) } : {}
+			...Array.isArray(dismissedNotices) ? { dismissedNotices: dismissedNotices.filter((key) => typeof key === "string" && NOTICE_KEY.test(key)) } : {},
+			...typeof promptsCollapsed === "boolean" ? { promptsCollapsed } : {}
 		};
 	} catch {
 		return {};
@@ -7938,6 +8011,16 @@ function writeAskWidth(width) {
 		writeFileSync(PREFERENCES_PATH, `${JSON.stringify(next, null, 2)}\n`);
 	} catch {}
 }
+/** Remember whether "Your prompts" is folded. Read-modify-write, like every other field here. */
+function writePromptsCollapsed(collapsed) {
+	try {
+		mkdirSync(STORE_ROOT, { recursive: true });
+		writeFileSync(PREFERENCES_PATH, `${JSON.stringify({
+			...readPreferences(),
+			promptsCollapsed: collapsed
+		}, null, 2)}\n`);
+	} catch {}
+}
 /** Remember that the reviewer has read a notice. Refuses anything not shaped like a key. */
 function dismissNotice(key) {
 	if (!NOTICE_KEY.test(key)) return false;
@@ -7973,6 +8056,44 @@ function clampAskWidth(width) {
 /** Long enough for a paragraph of standing instructions, short enough not to crowd the question. */
 var MAX_PROMPT_CHARS = 2e3;
 /**
+* What a reviewer starts with, before they have saved anything.
+*
+* Drawn from what reviewers were typing by hand most often. Ordinary saved prompts in every
+* respect: no flag marks them as defaults, so they edit, delete and reorder exactly as a prompt the
+* reviewer wrote does. Whoever reads a bank decides when these apply. In the workspace that is
+* only when no bank has ever been written, so a deleted default stays deleted.
+*
+* The ids are fixed rather than generated, so reading an unwritten bank twice gives the same bank.
+* Nothing depends on their spelling beyond that.
+*/
+var DEFAULT_PROMPTS = Object.freeze([
+	{
+		id: "default-assumptions",
+		name: "What does it assume?",
+		prompt: "What assumptions does this change make? Say which are stated, which are implied, and what happens if one turns out to be wrong."
+	},
+	{
+		id: "default-explain-educational",
+		name: "Teach me this",
+		prompt: "Explain this in an educational way: the concepts it relies on, why it is built this way, and what I should take away from it."
+	},
+	{
+		id: "default-what-could-go-wrong",
+		name: "What could go wrong?",
+		prompt: "What could go wrong here? Think about edge cases, failure modes and the existing callers this could break."
+	},
+	{
+		id: "default-walkthrough",
+		name: "Walk me through it",
+		prompt: "Walk me through the implementation step by step, in the order the code runs."
+	},
+	{
+		id: "default-verify-manually",
+		name: "What should I verify?",
+		prompt: "What should I verify manually before approving this, and how would I check each one?"
+	}
+].map((prompt) => Object.freeze(prompt)));
+/**
 * Everything reaching this function came from a client, so nothing here is trusted.
 *
 * Entries are repaired where the repair is unambiguous — trimming, truncating — and dropped where
@@ -8014,9 +8135,29 @@ function sanitisePromptBank(value) {
 * a workspace that will not open.
 */
 var PROMPTS_PATH = join(STORE_ROOT, "prompts.json");
+/**
+* The bank as stored or, when nothing has ever been stored, the defaults.
+*
+* Only a file that does not exist counts as a new bank. Once the file exists it is the reviewer's,
+* even when it is empty: an empty list is a reviewer who deleted every prompt, and handing the
+* defaults back to them would be undoing what they did. A corrupt file is not "new" either. It
+* reads as empty, as it always has, rather than being covered over with prompts the reviewer
+* never chose.
+*
+* The defaults are returned, not written. The first add, edit or delete posts the whole bank,
+* defaults included, and that write is what turns them into the reviewer's own. A reader that
+* wrote here would create the file on a page load, which the offline preview must not do.
+*/
 function readPromptBank() {
+	let raw;
+	try {
+		raw = readFileSync(PROMPTS_PATH, "utf8");
+	} catch (error) {
+		if (error.code === "ENOENT") return DEFAULT_PROMPTS.map((prompt) => ({ ...prompt }));
+		return [];
+	}
 	try {
-		return sanitisePromptBank(JSON.parse(readFileSync(PROMPTS_PATH, "utf8")));
+		return sanitisePromptBank(JSON.parse(raw));
 	} catch {
 		return [];
 	}
@@ -8656,11 +8797,6 @@ function addMany(res, ids) {
 		refused
 	});
 }
-/** Compacted here rather than in `queue.ts`, which must not learn to read the review store. */
-function compactQueue(stored) {
-	const live = stored.filter((queued) => loadWorkspace(queued) !== null);
-	return live.length === stored.length ? stored : writeQueue(live) ?? stored;
-}
 /**
 * Delete one review, or everything on one card.
 *
@@ -8886,6 +9022,7 @@ async function sendWorkspacePage(res, workspace) {
 		repoPath: workspace.repoPath,
 		askWidth: readPreferences().askWidth,
 		dismissedNotices: readPreferences().dismissedNotices,
+		promptsCollapsed: readPreferences().promptsCollapsed === true,
 		transcript: workspace.transcript,
 		pendingAsk: reportRun(workspace.id, "ask"),
 		pendingDiagram: reportRun(workspace.id, "diagram"),
@@ -9470,6 +9607,7 @@ async function handlePrefs(req, res) {
 	const askWidth = body?.askWidth;
 	if (typeof askWidth === "number") writeAskWidth(askWidth);
 	else if (askWidth === null) clearAskWidth();
+	if (typeof body?.promptsCollapsed === "boolean") writePromptsCollapsed(body.promptsCollapsed);
 	if (typeof body?.dismissNotice === "string" && !dismissNotice(body.dismissNotice)) return send(res, 400, { error: "Not a notice this page can dismiss." });
 	send(res, 200, { ok: true });
 }
@@ -9531,7 +9669,8 @@ var ASK_SOURCES = [
 	"suggestion",
 	"free_text",
 	"explain_simply",
-	"selection"
+	"selection",
+	"finding"
 ];
 function askSource(value) {
 	return ASK_SOURCES.includes(value) ? value : "free_text";

package/package.json

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