prreviewbuddy 0.91.1 → 0.92.0

package/dist/server.js

@@ -1,4 +1,4 @@
-import { $a as runnableAgents, $i as explanationOf, $n as authorPreview, $r as kissDecisionsOf, $t as recordRuntimeBestEffort, A as settleQueue, Ai as createWithMembers, An as moveIds, Ar as attemptIdOf, At as UpdateAlreadyRunningError, B as humanizeModelId, Ba as agentIdOf, Bi as queueStamp, Bn as existingPaths, Br as phasesFor, C as parsePullRequestLink, Ca as objectWithAnyOf, Ci as lifecyclePillAction, Cn as createPrState, Cr as followedBranch, Ct as RefreshUnavailableError, D as activateQueued, Da as resolveTarget, Di as sectionOf$1, Dn as markAllRead, Dr as fillFileUrlTemplate, E as relativeTime$1, Ea as kissOutcome, Ei as parseLifecycle, En as listNotifications, Er as forgeResolver, Et as mergeUpdateInto, Fa as shorthandOf, Fi as setMembership, Fn as retireNotification, Fr as pruneJobResidue, Ft as prepareReview, G as reviewTag, Ga as STORE_ROOT, Gi as issuesOutstanding, Gn as storedConversation, Gr as isClaimed, Gt as recordTelemetryUploadConsent, H as findingRef, Ha as readAgentPreference, Hi as doneVerb, Hn as adoptLegacyBaseline, Hr as purposeOf, Ht as record, Ia as git, Ii as updateCollections, In as toneOf, Ir as saveJob, It as settleReview, Ja as agentFor$1, Ji as reviewerDispositions, Jn as isUnchanged, Jr as followedRefName, Jt as BUILD_VERSION, K as reviewTagsIn, Ka as AGENT_IDS, Ki as questionsOutstanding, Kn as unforeseenFailure, Kt as telemetryUploadConsent, L as modelProvenanceOf, La as refExists, Li as queueAdd, Ln as unreadAround, Lt as reanalyseReview, Ma as resolveBase, Mi as nameProblemText, Mn as readNotification, Mr as fail$1, Mt as runningJobs, N as cliEntry, Na as AmbiguousRefError, Ni as readCollections, Nn as readStateOf, Nr as isTerminal, Nt as failureOf$1, O as compactQueue, Oa as originRemote, Oi as collectionsStamp, On as markRead, Or as RUN_RECORD_RETENTION_MS, Ot as updateReview, Pa as shortNameOf, Pi as renameCollection, Pn as removeNotification, Pr as loadJob, Pt as observeReview, Q as markAskRunning, Qa as parkedReason, Qi as explanationActionOf, Qn as UNESTABLISHED_MOTIVATION, Qr as kissDecisionRefusal, Qt as mapRuntimes, R as provenanceDetailLines, Ra as resolveAgent, Ri as queueAddMany, Rn as unreadCount$1, Rr as isLiveJob, Rt as runJob, S as extractPullRequestLinks, Sa as extractJsonText, Si as lifecycleName$1, Sn as createArchitectureNotification, Sr as checkReview, St as FETCH_TIMEOUT_MS, T as runBatch, Ta as WITHHELD_REASON, Ti as offersLifecycle, Tn as eventName, Tr as fetchHead, Tt as cancellableHere, U as findingRefs, Ua as writeAgentPreference, Ui as isIssueOutstanding, Un as prepareConversationWithin, V as findingAtRef, Va as clearAgentPreference, Vi as readQueue, Vn as ASK_REREAD_CEILING_MS, Vr as progressSteps, Vt as readEvents, W as refOf, Wi as isQuestionOutstanding, Wn as saveRefresh, X as isRequestId, Xa as detectAgents, Xi as processDiscussion, Xn as EXPLAIN_SIMPLY_PROMPT, Xr as indexKeptIds, Xt as feedbackUrl, Ya as agentIdentityOf, Yi as verdictClosure, Yn as agentEnvOf, Yr as groupByLineage, Za as isAgentId, Zi as recommendationsOf, Zn as chatConversationSection, Zr as indexReviewGroups, Zt as waitingHistory, _ as endRun, _a as statusLookup, _i as activityAttributes, _n as writeJsonAtomically, _r as requestJobStop, _t as parseWorkspacePort, a as lineageIds, aa as assessmentLookup, ai as mutateWorkspace, an as loadArchitecture, ar as escapeHtml$1, b as stopRun, ba as SEVERITIES, bi as lifecycleChip, bn as subjectOf, br as findExistingReview, bt as writePortPreference, c as discardJob, ca as findingAssessment, ci as previousKissResult, cn as repositoryKey, cr as ReviewBeingDeletedError, ct as serverEntryExists, d as startKissJob, da as gradeScale, dn as trackedFiles, dr as MANAGED_ROOT, ea as isExplanationAction, ei as latestKissRun, en as RUNTIME_WORDS, eo as versionFrom, er as hasAuthorContent, f as isDriving, fa as isGradeFor, fi as summarise, fn as ownersOf, fr as createWorktree, g as completeRun, ga as sourcePhrase, gi as LIFECYCLES, gn as isArchitectureId, gr as removeWorktree, gt as MIN_WORKSPACE_PORT, h as beginRun, ha as reassessIn, hi as workspaceStamps, hn as ancestorsOf, hr as readMarker, ht as MAX_WORKSPACE_PORT, i as deleteReview, ia as analysisRefOf, in as checkArchitecture, ir as CODE_COPY_CLASS, ja as isCommitReview, ji as deleteCollection, jn as pruneNotifications, jr as endedAt, jt as liveUpdateFor, ka as DetachedHeadError, ki as createCollection, kn as markReadForReview, kr as allJobs, kt as askCheckout, l as kissAgentId, la as findingIn, li as recentWorkspaces, ln as storeArchitecture, lr as isBusy, lt as spawnServer, m as RUN_KINDS, ma as promptGrade, mi as workspaceRevision, mn as REPOSITORY_SCOPE, mr as isWorktreeHeld, mt as DEFAULT_WORKSPACE_PORT, n as MCP_TOOL_NAMES, na as ANALYSIS_ONLY, ni as lineagePosition, nn as formatDuration, no as AgentCancelledError, nr as renderMarkdown, o as liveJobsFor, oa as closesFinding, on as partFiles, or as renderInlineText, ot as readServerRecord, p as track, pa as isOffered, pi as touchWorkspace, pn as QUOTE_LIMITS, pr as holdWorktree, pt as writeServerRecord, q as endAskRequest, qa as agentById, qi as resolvedByConversation, qn as refreshPrContext, qr as decideKissRecommendation, r as deleteLineage, ra as NO_STATUS, ri as loadWorkspace, rn as ARCHITECTURE_DIR, ro as AgentUnavailableError, rr as CODE_BLOCK_CLASS, rt as clearServerRecordOwnedBy, s as wasBlocked, sa as currentGradeIn, si as presentedKissRunOf, sn as readArchitecture, ta as withExplanation, ti as lineageKey, tn as describeRuntimes, tr as GITHUB_MARKDOWN, tt as readIndexToken, ua as findingStatus, ui as reviewedCommit, un as storedScopes, ur as withReviewLock, ut as stopChild, v as heldRuns, va as wasReassessed, vi as activityLabel, vn as announceJob, vt as resolveWorkspacePort, w as unsupportedReason, wa as describeAuthorship, wi as lifecyclePillLabel, wn as createPrUpdate, wr as reviewedShaOf, wt as cancelUpdate, x as batchIdentity, xa as asString, xi as lifecycleItemLabel, xn as categoryOf, xr as checkCodeFreshness, y as reportRun, ya as followUpFindingId, yi as isLifecycle, yn as branchOf, yt as statedWorkspacePort, z as agentDisplayName, za as resolveModel, zi as queueRemove, zn as withUsageRecorded, zr as isTerminalJobPhase } from "./mcp_tools-BR7_XprF.js";
+import { $a as runnableAgents, $i as explanationOf, $n as authorPreview, $r as kissDecisionsOf, $t as recordRuntimeBestEffort, A as settleQueue, Ai as createWithMembers, An as moveIds, Ar as attemptIdOf, At as UpdateAlreadyRunningError, B as humanizeModelId, Ba as agentIdOf, Bi as queueStamp, Bn as existingPaths, Br as phasesFor, C as parsePullRequestLink, Ca as objectWithAnyOf, Ci as lifecyclePillAction, Cn as createPrState, Cr as followedBranch, Ct as RefreshUnavailableError, D as activateQueued, Da as resolveTarget, Di as sectionOf$1, Dn as markAllRead, Dr as fillFileUrlTemplate, E as relativeTime$1, Ea as kissOutcome, Ei as parseLifecycle, En as listNotifications, Er as forgeResolver, Et as mergeUpdateInto, Fa as shorthandOf, Fi as setMembership, Fn as retireNotification, Fr as pruneJobResidue, Ft as prepareReview, G as reviewTag, Ga as STORE_ROOT, Gi as issuesOutstanding, Gn as storedConversation, Gr as isClaimed, Gt as recordTelemetryUploadConsent, H as findingRef, Ha as readAgentPreference, Hi as doneVerb, Hn as adoptLegacyBaseline, Hr as purposeOf, Ht as record, Ia as git, Ii as updateCollections, In as toneOf, Ir as saveJob, It as settleReview, Ja as agentFor$1, Ji as reviewerDispositions, Jn as isUnchanged, Jr as followedRefName, Jt as BUILD_VERSION, K as reviewTagsIn, Ka as AGENT_IDS, Ki as questionsOutstanding, Kn as unforeseenFailure, Kt as telemetryUploadConsent, L as modelProvenanceOf, La as refExists, Li as queueAdd, Ln as unreadAround, Lt as reanalyseReview, Ma as resolveBase, Mi as nameProblemText, Mn as readNotification, Mr as fail$1, Mt as runningJobs, N as cliEntry, Na as AmbiguousRefError, Ni as readCollections, Nn as readStateOf, Nr as isTerminal, Nt as failureOf$1, O as compactQueue, Oa as originRemote, Oi as collectionsStamp, On as markRead, Or as RUN_RECORD_RETENTION_MS, Ot as updateReview, Pa as shortNameOf, Pi as renameCollection, Pn as removeNotification, Pr as loadJob, Pt as observeReview, Q as markAskRunning, Qa as parkedReason, Qi as explanationActionOf, Qn as UNESTABLISHED_MOTIVATION, Qr as kissDecisionRefusal, Qt as mapRuntimes, R as provenanceDetailLines, Ra as resolveAgent, Ri as queueAddMany, Rn as unreadCount$1, Rr as isLiveJob, Rt as runJob, S as extractPullRequestLinks, Sa as extractJsonText, Si as lifecycleName$1, Sn as createArchitectureNotification, Sr as checkReview, St as FETCH_TIMEOUT_MS, T as runBatch, Ta as WITHHELD_REASON, Ti as offersLifecycle, Tn as eventName, Tr as fetchHead, Tt as cancellableHere, U as findingRefs, Ua as writeAgentPreference, Ui as isIssueOutstanding, Un as prepareConversationWithin, V as findingAtRef, Va as clearAgentPreference, Vi as readQueue, Vn as ASK_REREAD_CEILING_MS, Vr as progressSteps, Vt as readEvents, W as refOf, Wi as isQuestionOutstanding, Wn as saveRefresh, X as isRequestId, Xa as detectAgents, Xi as processDiscussion, Xn as EXPLAIN_SIMPLY_PROMPT, Xr as indexKeptIds, Xt as feedbackUrl, Ya as agentIdentityOf, Yi as verdictClosure, Yn as agentEnvOf, Yr as groupByLineage, Za as isAgentId, Zi as recommendationsOf, Zn as chatConversationSection, Zr as indexReviewGroups, Zt as waitingHistory, _ as endRun, _a as statusLookup, _i as activityAttributes, _n as writeJsonAtomically, _r as requestJobStop, _t as parseWorkspacePort, a as lineageIds, aa as assessmentLookup, ai as mutateWorkspace, an as loadArchitecture, ar as escapeHtml$1, b as stopRun, ba as SEVERITIES, bi as lifecycleChip, bn as subjectOf, br as findExistingReview, bt as writePortPreference, c as discardJob, ca as findingAssessment, ci as previousKissResult, cn as repositoryKey, cr as ReviewBeingDeletedError, ct as serverEntryExists, d as startKissJob, da as gradeScale, dn as trackedFiles, dr as MANAGED_ROOT, ea as isExplanationAction, ei as latestKissRun, en as RUNTIME_WORDS, eo as versionFrom, er as hasAuthorContent, f as isDriving, fa as isGradeFor, fi as summarise, fn as ownersOf, fr as createWorktree, g as completeRun, ga as sourcePhrase, gi as LIFECYCLES, gn as isArchitectureId, gr as removeWorktree, gt as MIN_WORKSPACE_PORT, h as beginRun, ha as reassessIn, hi as workspaceStamps, hn as ancestorsOf, hr as readMarker, ht as MAX_WORKSPACE_PORT, i as deleteReview, ia as analysisRefOf, in as checkArchitecture, ir as CODE_COPY_CLASS, ja as isCommitReview, ji as deleteCollection, jn as pruneNotifications, jr as endedAt, jt as liveUpdateFor, ka as DetachedHeadError, ki as createCollection, kn as markReadForReview, kr as allJobs, kt as askCheckout, l as kissAgentId, la as findingIn, li as recentWorkspaces, ln as storeArchitecture, lr as isBusy, lt as spawnServer, m as RUN_KINDS, ma as promptGrade, mi as workspaceRevision, mn as REPOSITORY_SCOPE, mr as isWorktreeHeld, mt as DEFAULT_WORKSPACE_PORT, n as MCP_TOOL_NAMES, na as ANALYSIS_ONLY, ni as lineagePosition, nn as formatDuration, no as AgentCancelledError, nr as renderMarkdown, o as liveJobsFor, oa as closesFinding, on as partFiles, or as renderInlineText, ot as readServerRecord, p as track, pa as isOffered, pi as touchWorkspace, pn as QUOTE_LIMITS, pr as holdWorktree, pt as writeServerRecord, q as endAskRequest, qa as agentById, qi as resolvedByConversation, qn as refreshPrContext, qr as decideKissRecommendation, r as deleteLineage, ra as NO_STATUS, ri as loadWorkspace, rn as ARCHITECTURE_DIR, ro as AgentUnavailableError, rr as CODE_BLOCK_CLASS, rt as clearServerRecordOwnedBy, s as wasBlocked, sa as currentGradeIn, si as presentedKissRunOf, sn as readArchitecture, ta as withExplanation, ti as lineageKey, tn as describeRuntimes, tr as GITHUB_MARKDOWN, tt as readIndexToken, ua as findingStatus, ui as reviewedCommit, un as storedScopes, ur as withReviewLock, ut as stopChild, v as heldRuns, va as wasReassessed, vi as activityLabel, vn as announceJob, vt as resolveWorkspacePort, w as unsupportedReason, wa as describeAuthorship, wi as lifecyclePillLabel, wn as createPrUpdate, wr as reviewedShaOf, wt as cancelUpdate, x as batchIdentity, xa as asString, xi as lifecycleItemLabel, xn as categoryOf, xr as checkCodeFreshness, y as reportRun, ya as followUpFindingId, yi as isLifecycle, yn as branchOf, yt as statedWorkspacePort, z as agentDisplayName, za as resolveModel, zi as queueRemove, zn as withUsageRecorded, zr as isTerminalJobPhase } from "./mcp_tools-BKPpd9dN.js";
 import { basename, delimiter, isAbsolute, join, relative, resolve } from "node:path";
 import { execFile, spawn } from "node:child_process";
 import { accessSync, constants, existsSync, mkdirSync, readFileSync, readdirSync, realpathSync, rmSync, statSync, writeFileSync } from "node:fs";
@@ -1573,6 +1573,7 @@ function tallyEntries(entries) {
 		bySection: {},
 		verify: 0,
 		decide: 0,
+		fix: 0,
 		step: 0
 	};
 	for (const entry of entries) {
@@ -1602,6 +1603,7 @@ function issuesAttentionLine(tally) {
 	const steps = [
 		tally.verify ? count(tally.verify, "verification step", "verification steps") : "",
 		tally.decide ? count(tally.decide, "decision", "decisions") : "",
+		tally.fix ? count(tally.fix, "fix", "fixes") : "",
 		tally.step ? count(tally.step, "next step", "next steps") : ""
 	].filter(Boolean);
 	return [count(total, "outstanding issue", "outstanding issues"), ...steps].join(" · ");
@@ -3443,8 +3445,13 @@ function renderIssuesTab(session, investigatingId, options) {
       ${renderFileLocation(issue.file, "pr-helper-issue-loc", issue.line)}
       ${renderRelatedFiles(issue.relatedFiles)}
       ${renderInvestigationProvenance("issue", issue.id, records, now)}`;
+		const actions = `<div class="pr-helper-issue-actions">
+        ${renderInvestigationActions("issue", issue.id, records, investigatingId)}
+        ${renderDismissToggle(issue, isDismissed, stale, dismissals.get(issue.id), dispositions.isClosedByVerdict(issue))}
+      </div>`;
 		const body = framed ? `${fixed ? "" : renderConsequence(impact)}${explanation}
       ${renderReconciliationNote(issue.reconciliationNote)}
+      <div class="pr-helper-card-footer">
       <details class="pr-helper-issue-technical">
         <summary class="pr-helper-issue-technical-summary">Technical details</summary>
         <div class="pr-helper-issue-technical-body">
@@ -3452,7 +3459,10 @@ function renderIssuesTab(session, investigatingId, options) {
           <p class="pr-helper-issue-technical-title">${renderInlineText(issue.title)}</p>
           ${technical}
         </div>
-      </details>` : technical;
+      </details>
+      ${actions}
+      </div>` : `${technical}
+      ${actions}`;
 		const next = impact?.next ? ` data-issue-next="${impact.next.kind ?? "step"}"` : "";
 		return `
     <div class="pr-helper-issue-card ${cardGradeClass("issue", assessment.current)}${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)}" data-issue-section="${sectionOf(issue)}"${next}>
@@ -3476,10 +3486,6 @@ function renderIssuesTab(session, investigatingId, options) {
       ${renderStaleDismissal(stale)}
       ${renderAssessmentRegion("issue", assessment, grading, statusOf("issue", issue.id))}
       ${body}
-      <div class="pr-helper-issue-actions">
-        ${renderInvestigationActions("issue", issue.id, records, investigatingId)}
-        ${renderDismissToggle(issue, isDismissed, stale, dismissals.get(issue.id), dispositions.isClosedByVerdict(issue))}
-      </div>
     </div>
   `;
 	};
@@ -3521,7 +3527,8 @@ function renderConsequence(impact) {
 }
 var STEP_LABELS = {
 	verify: "To verify:",
-	decide: "To decide:"
+	decide: "To decide:",
+	fix: "To fix:"
 };
 function renderConsequenceStep(step) {
 	if (!step) return "";
@@ -3616,13 +3623,47 @@ function renderQuestionCard(q, sections, answered, records, now = Date.now(), in
 	const answerToggle = `<button type="button" class="pr-helper-question-answer-toggle" data-question-answered="${escapeHtml$1(q.id)}" data-answered="${answered}"${settled ? " data-settled=\"true\"" : ""}${verdictClosed ? " data-verdict-closed=\"true\"" : ""}>${answered ? "Mark as unanswered" : "Mark answered"}</button>`;
 	const investigationAnswer = q.investigationAnswer?.trim() ? `<h5>Answer from investigation</h5><p class="pr-helper-issue-desc pr-helper-investigation-answer">${renderInlineText(q.investigationAnswer)}</p>` : "";
 	const reconciliationAnswer = q.reconciliationAnswer?.trim() ? `<h5>${escapeHtml$1(reconciledAnswerHeading(q.reconciledAgainst))}</h5><p class="pr-helper-issue-desc pr-helper-investigation-answer">${renderInlineText(q.reconciliationAnswer)}</p>` : "";
+	const why = q.whyThisMatters ? `<h5>Why this matters</h5><p class="pr-helper-issue-desc">${renderInlineText(q.whyThisMatters)}</p>` : "";
+	const files = q.file || q.relatedFiles?.length ? `<h5>Files</h5>${renderFileLocation(q.file, "pr-helper-issue-loc", void 0)}${renderRelatedFiles(q.relatedFiles)}` : "";
+	const provenance = renderInvestigationProvenance("question", q.id, records, now);
+	const framing = q.framing;
+	const actions = `<div class="pr-helper-question-actions">
+        ${jumpLink}
+        <div class="pr-helper-question-actions-group">
+          ${renderInvestigationActions("question", q.id, records, investigatingId)}
+          ${answerToggle}
+        </div>
+      </div>`;
+	const answers = `${reconciliationAnswer}
+      ${renderReconciliationNote(q.reconciliationNote)}
+      ${investigationAnswer}`;
+	const body = framing ? `${renderQuestionFraming(framing)}${renderCardExplanation(explanation, options)}
+      ${answers}
+      <div class="pr-helper-card-footer">
+      <details class="pr-helper-issue-technical">
+        <summary class="pr-helper-issue-technical-summary">Technical context</summary>
+        <div class="pr-helper-issue-technical-body">
+          <p class="pr-helper-issue-technical-title">${renderInlineText(q.question)}</p>
+          ${why}
+          ${renderInvestigationNote(q.investigationNote)}
+          ${files}
+          ${provenance}
+        </div>
+      </details>
+      ${actions}
+      </div>` : `${why}${renderCardExplanation(explanation, options)}
+      ${answers}
+      ${renderInvestigationNote(q.investigationNote)}
+      ${files}
+      ${provenance}
+      ${actions}`;
 	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)}"${questionHome(q) ? ` data-section-id="${escapeHtml$1(questionHome(q))}"` : ""}>
       <div class="pr-helper-issue-header">
         <div class="pr-helper-issue-title-row">
           ${options?.feedbackSelect ? renderFeedbackPick("question", q.question, questionFeedbackMarkdown(q), ref) : ""}
           ${renderFindingRef(ref)}
-          <p class="pr-helper-issue-title">${renderInlineText(q.question)}</p>
+          <p class="pr-helper-issue-title">${renderInlineText(framing?.question ?? q.question)}</p>
           ${renderFindingCopySource(questionMarkdown(q, assessment), "question")}
           ${options?.askBuddy ? renderAskBuddyMenu("question") : ""}
         </div>
@@ -3639,23 +3680,17 @@ function renderQuestionCard(q, sections, answered, records, now = Date.now(), in
         </div>
       </div>
       ${renderAssessmentRegion("question", assessment, grading, status)}
-      ${q.whyThisMatters ? `<h5>Why this matters</h5><p class="pr-helper-issue-desc">${renderInlineText(q.whyThisMatters)}</p>` : ""}${renderCardExplanation(explanation, options)}
-      ${reconciliationAnswer}
-      ${renderReconciliationNote(q.reconciliationNote)}
-      ${investigationAnswer}
-      ${renderInvestigationNote(q.investigationNote)}
-      ${q.file || q.relatedFiles?.length ? `<h5>Files</h5>${renderFileLocation(q.file, "pr-helper-issue-loc", void 0)}${renderRelatedFiles(q.relatedFiles)}` : ""}
-      ${renderInvestigationProvenance("question", q.id, records, now)}
-      <div class="pr-helper-question-actions">
-        ${jumpLink}
-        <div class="pr-helper-question-actions-group">
-          ${renderInvestigationActions("question", q.id, records, investigatingId)}
-          ${answerToggle}
-        </div>
-      </div>
+      ${body}
     </div>
   `;
 }
+/** What the question means, read first: why the answer matters and what is needed. */
+function renderQuestionFraming(framing) {
+	return `<div class="pr-helper-issue-impact">
+        <p class="pr-helper-issue-scenario">${renderInlineText(framing.why)}</p>
+        ${framing.needed ? `<p class="pr-helper-issue-next pr-helper-question-needed"><span class="pr-helper-issue-next-label">Needed:</span> ${renderInlineText(framing.needed)}</p>` : ""}
+      </div>`;
+}
 /**
 * The counter's contents, split out so a caller that toggles one question can refresh it without
 * re-rendering the tab.
@@ -6957,20 +6992,28 @@ function renderStandardViews(view, sections) {
 * When the "already raised" claims below were last true.
 *
 * Every attribution on this view carries a thread's state — "still unresolved", "since marked
-* resolved" — and that state was frozen the moment the conversation was imported. Nothing
-* re-checks it, and nothing here can: the daemon reaches no forge. So a card can go on telling
-* the reviewer that Sergei's thread is open an hour after Sergei closed it.
+* resolved" — and that state was frozen the moment the conversation was imported. Nothing on this
+* view re-checks it; the PR context view does, on request. Until then a card can go on telling the
+* reviewer that Sergei's thread is open an hour after Sergei closed it.
 *
 * Every other kind of staleness in this product degrades into vagueness. This one degrades into a
 * confident, specific, wrong claim about what a named colleague currently thinks, which is worth
-* one line above the list rather than an explanation the reviewer has to go and find.
+* one line above the list: its age always in view, the explanation one disclosure away, and the way
+* to the re-check beside it.
 *
 * Only where a conversation was attached: there are no such claims to qualify otherwise.
 */
 function renderAttributionAge(view) {
 	if (!view.prContext) return "";
 	const imported = relativeTime$1(view.prContext.prContext.discussion.fetchedAt);
-	return `<p class="prb-doc-note">Any &ldquo;already raised&rdquo; note below describes the pull request as it stood ${escapeHtml$1(imported)}, when its discussion was fetched. That includes whether each thread was still open. If someone has replied or resolved a thread since then, this page has no way of knowing. Use <button type="button" class="prb-doc-note-link" data-view-nav="prcontext">Re-check the conversation</button> on the PR context view to bring it up to date.</p>`;
+	return `<div class="prb-discussion-age">
+          <details class="prb-discussion-age-detail">
+            <summary class="prb-discussion-age-summary">PR discussion checked ${escapeHtml$1(imported)}</summary>
+            <p class="prb-discussion-age-body">Any &ldquo;already raised&rdquo; note below, and whether each thread was still open, describes the pull request as it stood when its discussion was fetched. A reply or a resolved thread since then is not reflected here until the conversation is re-checked.</p>
+          </details>
+          <span class="prb-discussion-age-sep" aria-hidden="true">&middot;</span>
+          <button type="button" class="prb-doc-note-link" data-view-nav="prcontext">Re-check in PR context</button>
+        </div>`;
 }
 /**
 * The link out to this file on the pull request.
@@ -12690,11 +12733,21 @@ feature:                   E</pre>
     confirm</b> where the behaviour is right only if it was meant, and <b>Maintenance</b> for what it
     costs later. Within each group the most severe come first. Severity and confidence are shown on
     each card, separately from its group.</p>
-    <p>A card leads with what the issue means and when it happens, then the step that would settle
-    it, marked <b>To verify</b> or <b>To decide</b>. The technical account, where it is in the code and
-    its history are under <b>Technical details</b> on the card. Folding a group is remembered for the
-    next review. Issues from reviews made before this have no grouping and are under <b>Other
-    issues</b>, as they were.</p>
+    <p>A card leads with what the issue means and when it happens, then the step it calls for, when
+    there is one: <b>To verify</b> settles whether it happens, <b>To decide</b> asks whether it was
+    meant, and <b>To fix</b> names the change that would correct it. The technical account, where it
+    is in the code and its history are under <b>Technical details</b> on the card. Folding a group is
+    remembered for the next review.</p>
+    <p>Issues from older reviews have no summary yet and are under <b>Other issues</b>. The next
+    Update that finds new commits or new discussion writes one for each issue still outstanding,
+    from the same evidence; it does not re-check the issue. An Update that finds nothing new changes
+    nothing.</p>
+    <p>Questions read the same way: each leads with the question in plain terms, why the answer
+    matters and what is needed, with the original question and its files under <b>Technical
+    context</b>. Any answer so far stays on the card.</p>
+    <p>When the review came with a pull request discussion, the line above the issues says when it
+    was last checked. Its “already raised” notes describe the discussion as it was then; <b>Re-check
+    in PR context</b> takes you to bring it up to date.</p>
     <p>Resolved and dismissed issues are in the closed list at the bottom. One the new commits fixed
     leads with what the code now does, when the update could see it.</p>`
 	},

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prreviewbuddy",
-  "version": "0.91.1",
+  "version": "0.92.0",
   "description": "Understand code changes before you approve them. Local branch and pull request reviews with Claude Code or Codex.",
   "license": "UNLICENSED",
   "homepage": "https://prreviewbuddy.com",

package/static/shell.css

@@ -2386,10 +2386,6 @@ a.prb-rail-item { text-decoration: none; }
   line-height: var(--lh-body);
 }
 
-/* A caveat above a list, in the light pane. Pane literals, not shell tokens — see
-   pane_palette.test.ts for why this is a rule and not a preference. */
-/* Full width, because it introduces the cards below it and a narrower block reads as a floating
-   aside rather than as the heading note it is. */
 /* The control named in the note, as the control rather than as bold text. It goes to the view that
    holds it, because a sentence that tells the reviewer where something is and cannot take them
    there is asking them to do the navigating twice. */
@@ -2405,16 +2401,41 @@ a.prb-rail-item { text-decoration: none; }
 }
 .prb-doc-note-link:hover { color: var(--pane-ink); }
 
-.prb-doc-note {
-  margin: 0 0 14px;
-  padding: 8px 11px;
-  border-left: 3px solid var(--pane-line);
-  border-radius: 0 6px 6px 0;
-  background: rgba(15, 23, 42, 0.035);
+/* The age of the "already raised" claims, as one quiet line above the Issues orientation. */
+.prb-discussion-age {
+  display: flex;
+  flex-wrap: wrap;
+  align-items: baseline;
+  column-gap: 6px;
+  margin: 0 0 12px;
   color: var(--pane-ink-muted);
   font-size: var(--t-meta);
   line-height: var(--lh-body);
 }
+.prb-discussion-age-summary {
+  cursor: pointer;
+  list-style: none;
+}
+.prb-discussion-age-summary::-webkit-details-marker { display: none; }
+.prb-discussion-age-summary::before {
+  content: '';
+  display: inline-block;
+  margin-right: 6px;
+  vertical-align: 1px;
+  border-left: 5px solid currentColor;
+  border-top: 4px solid transparent;
+  border-bottom: 4px solid transparent;
+}
+.prb-discussion-age-detail[open] > .prb-discussion-age-summary::before { transform: rotate(90deg); }
+.prb-discussion-age-summary:hover { color: var(--pane-ink); }
+/* Open, the explanation takes the row, and the re-check follows it on its own line. */
+.prb-discussion-age-detail[open] { flex-basis: 100%; }
+.prb-discussion-age-detail[open] ~ .prb-discussion-age-sep { display: none; }
+.prb-discussion-age-body {
+  margin: 6px 0 4px;
+  padding-left: 11px;
+  border-left: 2px solid var(--pane-line);
+}
 
 .prb-pr-asof {
   flex-basis: 100%;

package/static/sidebar.css

@@ -1068,7 +1068,7 @@
 .pr-helper-issue-card {
   border: 1px solid var(--color-border-default, #d0d7de);
   border-radius: 8px;
-  padding: 16px;
+  padding: 12px 14px;
   margin-bottom: 12px;
   /* Stated rather than inherited: the surface behind this is now the grey canvas. */
   background: #ffffff;
@@ -1437,7 +1437,7 @@
 .pr-helper-question-card {
   border: 1px solid var(--color-border-default, #d0d7de);
   border-radius: 8px;
-  padding: 16px;
+  padding: 12px 14px;
   margin-bottom: 12px;
   /* Stated rather than inherited: the surface behind this is now the grey canvas. */
   background: #ffffff;
@@ -1447,6 +1447,9 @@
    card would throw away the reviewer's own record that they resolved it. The check mark is the
    whole "done" signal; nothing else about the card recedes. */
 .pr-helper-question-card-answered { box-shadow: inset 3px 0 0 #1a7f37; }
+/* What an answer still needs is moot once the question is settled, and a tick moves the card here
+   without a redraw, so the drawer is what decides it. */
+.pr-helper-closed-findings .pr-helper-question-needed { display: none; }
 
 .pr-helper-question-card h5 {
   font-size: 10px;
@@ -1598,14 +1601,15 @@
 }
 
 .pr-helper-issues-orientation-text {
-  font-size: 13px;
+  font-size: 14px;
+  color: var(--color-fg-default, #1f2328);
   line-height: 1.5;
   margin: 0 0 4px;
   overflow-wrap: anywhere;
 }
 
 .pr-helper-issues-attention {
-  font-size: 12px;
+  font-size: 13px;
   font-weight: 600;
   color: var(--color-fg-muted, #57606a);
   margin: 0;
@@ -1713,6 +1717,21 @@
    drawer, smaller and muted, because it belongs to one card rather than to the pane. */
 .pr-helper-issue-technical { margin-top: 8px; }
 
+/* A framed card's disclosure and actions share a row: closed, the summary sits left and the actions
+   right; opened, the disclosure takes the row and the actions wrap below its body. */
+.pr-helper-card-footer {
+  display: flex;
+  flex-wrap: wrap;
+  align-items: center;
+  column-gap: 12px;
+  row-gap: 8px;
+  margin-top: 8px;
+}
+.pr-helper-card-footer > .pr-helper-issue-technical { flex: 1 1 auto; min-width: 0; margin-top: 0; }
+.pr-helper-card-footer > .pr-helper-issue-technical[open] { flex-basis: 100%; }
+.pr-helper-card-footer > .pr-helper-issue-actions,
+.pr-helper-card-footer > .pr-helper-question-actions { margin-top: 0; margin-left: auto; }
+
 .pr-helper-issue-technical-summary {
   display: inline-flex;
   align-items: center;
@@ -1769,7 +1788,7 @@
   justify-content: space-between;
   flex-wrap: wrap;
   gap: 12px;
-  margin-top: 12px;
+  margin-top: 8px;
 }
 
 /* Deliberately not a filled button, matching .pr-helper-guide-start-cta: this is "here's the
@@ -5000,7 +5019,7 @@
   align-items: center;
   justify-content: flex-end;
   gap: 12px;
-  margin-top: 12px;
+  margin-top: 8px;
 }
 
 /* margin-left:auto rather than relying on the row's space-between: the jump link beside it is