prreviewbuddy 0.90.0 → 0.91.1

package/dist/server.js

@@ -1,4 +1,4 @@
-import { $a as versionFrom, $i as explanationOf, $n as authorPreview, $r as kissDecisionsOf, $t as recordRuntimeBestEffort, A as settleQueue, Aa as isCommitReview, Ai as createWithMembers, An as moveIds, Ar as attemptIdOf, At as UpdateAlreadyRunningError, B as humanizeModelId, Ba as clearAgentPreference, Bi as queueStamp, Bn as existingPaths, Br as phasesFor, C as parsePullRequestLink, Ca as describeAuthorship, Ci as lifecyclePillAction, Cn as createPrState, Cr as followedBranch, Ct as RefreshUnavailableError, D as activateQueued, Da as originRemote, Di as sectionOf, Dn as markAllRead, Dr as fillFileUrlTemplate, E as relativeTime$1, Ea as resolveTarget, Ei as parseLifecycle, En as listNotifications, Er as forgeResolver, Et as mergeUpdateInto, Fa as git, Fi as setMembership, Fn as retireNotification, Fr as pruneJobResidue, Ft as prepareReview, G as reviewTag, Ga as AGENT_IDS, Gi as issuesOutstanding, Gn as storedConversation, Gr as isClaimed, Gt as recordTelemetryUploadConsent, H as findingRef, Ha as writeAgentPreference, Hi as doneVerb, Hn as adoptLegacyBaseline, Hr as purposeOf, Ht as record, Ia as refExists, Ii as updateCollections, In as toneOf, Ir as saveJob, It as settleReview, Ja as agentIdentityOf, Ji as reviewerDispositions, Jn as isUnchanged, Jr as followedRefName, Jt as BUILD_VERSION, K as reviewTagsIn, Ka as agentById, Ki as questionsOutstanding, Kn as unforeseenFailure, Kt as telemetryUploadConsent, L as modelProvenanceOf, La as resolveAgent, Li as queueAdd, Ln as unreadAround, Lt as reanalyseReview, Ma as AmbiguousRefError, Mi as nameProblemText, Mn as readNotification, Mr as fail$1, Mt as runningJobs, N as cliEntry, Na as shortNameOf, Ni as readCollections, Nn as readStateOf, Nr as isTerminal, Nt as failureOf$1, O as compactQueue, Oa as DetachedHeadError, Oi as collectionsStamp, On as markRead, Or as RUN_RECORD_RETENTION_MS, Ot as updateReview, Pa as shorthandOf, Pi as renameCollection, Pn as removeNotification, Pr as loadJob, Pt as observeReview, Q as markAskRunning, Qa as runnableAgents, Qi as explanationActionOf, Qn as UNESTABLISHED_MOTIVATION, Qr as kissDecisionRefusal, Qt as mapRuntimes, R as provenanceDetailLines, Ra as resolveModel, Ri as queueAddMany, Rn as unreadCount$1, Rr as isLiveJob, Rt as runJob, S as extractPullRequestLinks, Sa as objectWithAnyOf, Si as lifecycleName$1, Sn as createArchitectureNotification, Sr as checkReview, St as FETCH_TIMEOUT_MS, T as runBatch, Ta as kissOutcome, Ti as offersLifecycle, Tn as eventName, Tr as fetchHead, Tt as cancellableHere, U as findingRefs, Ui as isIssueOutstanding, Un as prepareConversationWithin, V as findingAtRef, Va as readAgentPreference, Vi as readQueue, Vn as ASK_REREAD_CEILING_MS, Vr as progressSteps, Vt as readEvents, W as refOf, Wa as STORE_ROOT, Wi as isQuestionOutstanding, Wn as saveRefresh, X as isRequestId, Xa as isAgentId, Xi as processDiscussion, Xn as EXPLAIN_SIMPLY_PROMPT, Xr as indexKeptIds, Xt as feedbackUrl, Ya as detectAgents, Yi as verdictClosure, Yn as agentEnvOf, Yr as groupByLineage, Za as parkedReason, 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 asString, 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, 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 resolveBase, ji as deleteCollection, jn as pruneNotifications, jr as endedAt, jt as liveUpdateFor, 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 AgentUnavailableError, 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 agentFor$1, qi as resolvedByConversation, qn as refreshPrContext, qr as decideKissRecommendation, r as deleteLineage, ra as NO_STATUS, ri as loadWorkspace, rn as ARCHITECTURE_DIR, 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, to as AgentCancelledError, 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 WITHHELD_REASON, wi as lifecyclePillLabel, wn as createPrUpdate, wr as reviewedShaOf, wt as cancelUpdate, x as batchIdentity, xa as extractJsonText, 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 agentIdOf, zi as queueRemove, zn as withUsageRecorded, zr as isTerminalJobPhase } from "./mcp_tools-CSj5vcSE.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-BR7_XprF.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";
@@ -182,6 +182,7 @@ function composeReview(session) {
 		schemaVersion: 1,
 		summary: primary.summary,
 		...primary.motivation ? { motivation: primary.motivation } : {},
+		...primary.orientation ? { orientation: primary.orientation } : {},
 		recommendation: primary.recommendation,
 		nextSteps: primary.nextSteps,
 		overallRisk,
@@ -569,7 +570,7 @@ function isCollectionId(value) {
 * filter gives, and only its star says it is pinned.
 */
 function indexSectionOf(pinned, lifecycle, activity, filter) {
-	return pinned && !ordersExplicitly(filter) ? "pinned" : sectionOf(lifecycle, activity);
+	return pinned && !ordersExplicitly(filter) ? "pinned" : sectionOf$1(lifecycle, activity);
 }
 /**
 * Where a review's own card is drawn, or absent when it is drawn only inside its collections.
@@ -595,7 +596,7 @@ function homeSection(pinned, lifecycle, activity, collected, filter) {
 */
 function collectionPlacement(pinned, lifecycle, activity, filter) {
 	if (pinned || ordersExplicitly(filter)) return void 0;
-	return sectionOf(lifecycle, activity) === "done" ? "completed" : "active";
+	return sectionOf$1(lifecycle, activity) === "done" ? "completed" : "active";
 }
 /** A collection heading's count: what still needs attention, and the members drawn in Pinned. */
 function collectionCountLabel(active, pinned) {
@@ -902,7 +903,7 @@ function svg(paths) {
 * Buddy is asked about can never disagree. That is also why no severity, theme or status appears:
 * those sort the findings for the reviewer and are not part of what the author is told.
 */
-function count(n, noun) {
+function count$1(n, noun) {
 	return `${n} ${noun}${n === 1 ? "" : "s"}`;
 }
 /**
@@ -919,8 +920,8 @@ function feedbackSelectionParts(issues, questions) {
 	const total = issues + questions;
 	return {
 		total: total === 0 ? "Nothing chosen" : `${total} chosen`,
-		issues: issues ? count(issues, "issue") : "",
-		questions: questions ? count(questions, "question") : ""
+		issues: issues ? count$1(issues, "issue") : "",
+		questions: questions ? count$1(questions, "question") : ""
 	};
 }
 /** "5 chosen · 2 issues · 3 questions", "1 chosen · 1 question", "Nothing chosen". */
@@ -1522,6 +1523,99 @@ function splitFilePath(path) {
 	};
 }
 //#endregion
+//#region ../../packages/review-harness/src/domain/ui/components/impact_groups.ts
+/** The pane's folds: each section, and the closed drawer under them. */
+var ISSUE_FOLDS = [
+	"failure",
+	"risk",
+	"intent",
+	"maintenance",
+	"other",
+	"closed"
+];
+/** Display order: what is wrong, what may be, what only the reviewer can judge, what costs later. */
+var ISSUE_SECTIONS = [
+	"failure",
+	"risk",
+	"intent",
+	"maintenance",
+	"other"
+];
+var ISSUE_SECTION_TITLES = {
+	failure: "Failures",
+	risk: "Possible risks",
+	intent: "Intent to confirm",
+	maintenance: "Maintenance",
+	other: "Other issues"
+};
+function sectionOf(issue) {
+	return issue.consequence?.group ?? "other";
+}
+/**
+* Issues by section, each section by severity as the reviewer now holds it, ties in the order the
+* analysis gave them. Severity orders within a section and never moves an issue between sections.
+*/
+function groupIssues(issues, gradeOf) {
+	const rank = (issue) => {
+		const index = SEVERITIES.indexOf(gradeOf(issue) ?? "");
+		return index < 0 ? SEVERITIES.length : index;
+	};
+	return ISSUE_SECTIONS.map((section) => ({
+		section,
+		issues: issues.map((issue, index) => ({
+			issue,
+			index
+		})).filter(({ issue }) => sectionOf(issue) === section).sort((a, b) => rank(a.issue) - rank(b.issue) || a.index - b.index).map(({ issue }) => issue)
+	})).filter((group) => group.issues.length > 0);
+}
+function tallyEntries(entries) {
+	const tally = {
+		bySection: {},
+		verify: 0,
+		decide: 0,
+		step: 0
+	};
+	for (const entry of entries) {
+		tally.bySection[entry.section] = (tally.bySection[entry.section] ?? 0) + 1;
+		if (entry.next) tally[entry.next] += 1;
+	}
+	return tally;
+}
+function tallyIssues(issues) {
+	return tallyEntries(issues.map((issue) => {
+		const step = issue.consequence?.next;
+		return {
+			section: sectionOf(issue),
+			...step ? { next: step.kind ?? "step" } : {}
+		};
+	}));
+}
+var count = (n, one, many) => `${n} ${n === 1 ? one : many}`;
+/**
+* One factual line over the sections: how many issues are outstanding, and how many of them name
+* a step, by the kind the analysis gave it. The per-section counts are the nav's, so they are not
+* repeated here. Always "issues": Questions keep their own tab and their own count.
+*/
+function issuesAttentionLine(tally) {
+	const total = Object.values(tally.bySection).reduce((sum, n) => sum + (n ?? 0), 0);
+	if (total === 0) return "No issues are outstanding from this analysis.";
+	const steps = [
+		tally.verify ? count(tally.verify, "verification step", "verification steps") : "",
+		tally.decide ? count(tally.decide, "decision", "decisions") : "",
+		tally.step ? count(tally.step, "next step", "next steps") : ""
+	].filter(Boolean);
+	return [count(total, "outstanding issue", "outstanding issues"), ...steps].join(" · ");
+}
+/**
+* Buttons to each section with its count. Buttons, not `#` links: the hash routes the review page.
+* Drawn only when there are two sections to choose between; a single one is already in view.
+*/
+function renderImpactNavItems(tally) {
+	const present = ISSUE_SECTIONS.filter((section) => (tally.bySection[section] ?? 0) > 0);
+	if (present.length < 2) return "";
+	return present.map((section) => `<button type="button" class="pr-helper-issue-nav-item" data-issue-nav="${section}">${escapeHtml$1(ISSUE_SECTION_TITLES[section])}<span class="pr-helper-issue-nav-count">${tally.bySection[section]}</span></button>`).join("");
+}
+//#endregion
 //#region ../../packages/review-harness/src/domain/ai_review/review_outcome.ts
 /**
 * The one reading of a session, for every surface.
@@ -3292,16 +3386,6 @@ function renderInvestigationNote(note) {
 	return `<p class="pr-helper-investigation-note">${renderInlineText(note)}</p>`;
 }
 /**
-* The bar above the two groups, worded the way the Questions tab words its own.
-*
-* Only drawn once a reconciliation has closed something. On a first analysis every finding is
-* open, "15 of 15 still open" is a sentence with no information in it, and a review that has
-* never been updated should not grow a progress bar to tell the reviewer they have made none.
-*/
-function findingsProgressBody(total, open, verb = "resolved") {
-	return open === 0 ? `<span class="pr-helper-questions-progress-check" aria-hidden="true">\u2713</span><span class="pr-helper-questions-progress-count">All ${total} issues ${verb}</span>` : `<span class="pr-helper-questions-progress-count">${open} of ${total} still open</span>`;
-}
-/**
 * Work that is done, kept and closed.
 *
 * Collapsed rather than dropped: the reviewer asked whether these were fixed, and "they are gone
@@ -3309,8 +3393,8 @@ function findingsProgressBody(total, open, verb = "resolved") {
 * ten resolved findings above five open ones is the layout that made the count feel wrong in the
 * first place.
 */
-function renderClosedFindings(summary, cards, kind, nothingOpen, waiting) {
-	return `<details class="pr-helper-closed-findings" data-closed-findings="${kind}"${nothingOpen ? " data-nothing-open" : ""}>
+function renderClosedFindings(summary, cards, kind, nothingOpen, waiting, open = false) {
+	return `<details class="pr-helper-closed-findings" data-closed-findings="${kind}"${nothingOpen ? " data-nothing-open" : ""}${open ? " open" : ""}>
       <summary class="pr-helper-closed-findings-summary" data-closed-findings-summary>${escapeHtml$1(summary)}${renderWaitingSuggestions(waiting)}</summary>
       <div class="pr-helper-closed-findings-list">${cards}</div>
     </details>`;
@@ -3346,13 +3430,37 @@ function renderIssuesTab(session, investigatingId, options) {
 		const isDismissed = dispositions.isDismissed(issue);
 		const stale = dispositions.isStale(issue);
 		const assessment = assess("issue", issue);
+		const impact = issue.consequence;
+		const fixed = issue.reconciliationStatus === "resolved" ? issue.resolvedBehaviour?.trim() : void 0;
+		const framed = Boolean(impact || fixed);
+		const headline = fixed ?? impact?.title ?? issue.title;
+		const explanation = renderCardExplanation(explanationOf(session, "issue", issue.id), options);
+		const technical = `<p class="pr-helper-issue-desc">${renderInlineText(issue.description)}</p>${framed ? "" : explanation}
+      ${framed ? "" : renderReconciliationNote(issue.reconciliationNote)}
+      ${renderStatusHistory(issue.statusHistory, now)}
+      ${renderInvestigationNote(issue.investigationNote)}
+      ${renderExistingDiscussion(issue.existingDiscussion)}
+      ${renderFileLocation(issue.file, "pr-helper-issue-loc", issue.line)}
+      ${renderRelatedFiles(issue.relatedFiles)}
+      ${renderInvestigationProvenance("issue", issue.id, records, now)}`;
+		const body = framed ? `${fixed ? "" : renderConsequence(impact)}${explanation}
+      ${renderReconciliationNote(issue.reconciliationNote)}
+      <details class="pr-helper-issue-technical">
+        <summary class="pr-helper-issue-technical-summary">Technical details</summary>
+        <div class="pr-helper-issue-technical-body">
+          ${fixed && impact ? renderOriginalConsequence(impact) : ""}
+          <p class="pr-helper-issue-technical-title">${renderInlineText(issue.title)}</p>
+          ${technical}
+        </div>
+      </details>` : technical;
+		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)}">
+    <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}>
       <div class="pr-helper-issue-header">
         <div class="pr-helper-issue-title-row">
           ${options?.feedbackSelect ? renderFeedbackPick("issue", issue.title, issueFeedbackMarkdown(issue), ref) : ""}
           ${renderFindingRef(ref)}
-          <p class="pr-helper-issue-title">${renderInlineText(issue.title)}</p>
+          <p class="pr-helper-issue-title">${renderInlineText(headline)}</p>
           ${renderFindingCopySource(issueMarkdown(issue, assessment), "issue")}
           ${options?.askBuddy ? renderAskBuddyMenu("issue") : ""}
         </div>
@@ -3367,14 +3475,7 @@ function renderIssuesTab(session, investigatingId, options) {
       </div>
       ${renderStaleDismissal(stale)}
       ${renderAssessmentRegion("issue", assessment, grading, statusOf("issue", issue.id))}
-      <p class="pr-helper-issue-desc">${renderInlineText(issue.description)}</p>${renderCardExplanation(explanationOf(session, "issue", issue.id), options)}
-      ${renderReconciliationNote(issue.reconciliationNote)}
-      ${renderStatusHistory(issue.statusHistory, now)}
-      ${renderInvestigationNote(issue.investigationNote)}
-      ${renderExistingDiscussion(issue.existingDiscussion)}
-      ${renderFileLocation(issue.file, "pr-helper-issue-loc", issue.line)}
-      ${renderRelatedFiles(issue.relatedFiles)}
-      ${renderInvestigationProvenance("issue", issue.id, records, now)}
+      ${body}
       <div class="pr-helper-issue-actions">
         ${renderInvestigationActions("issue", issue.id, records, investigatingId)}
         ${renderDismissToggle(issue, isDismissed, stale, dismissals.get(issue.id), dispositions.isClosedByVerdict(issue))}
@@ -3383,11 +3484,56 @@ function renderIssuesTab(session, investigatingId, options) {
   `;
 	};
 	const counts = issuesOutstanding(result.issues, dispositions);
-	if (counts.done === 0) return result.issues.map(card).join("");
-	const open = result.issues.filter((issue) => isIssueOutstanding(issue, dispositions)).map(card).join("");
+	const outstanding = result.issues.filter((issue) => isIssueOutstanding(issue, dispositions));
 	const closedIssues = result.issues.filter((issue) => !isIssueOutstanding(issue, dispositions));
-	const closed = closedIssues.map(card).join("");
-	return `<div class="pr-helper-findings-progress" role="status">${findingsProgressBody(counts.total, counts.open, doneVerb(counts))}</div>${open}${renderClosedFindings(closedFindingsSummary(counts), closed, "issues", counts.open === 0, closedIssues.filter((issue) => assess("issue", issue).pending || statusOf("issue", issue.id).pending).length)}`;
+	const folds = options?.issueFolds ?? {};
+	const tally = tallyIssues(outstanding);
+	const grouped = new Map(groupIssues(outstanding, (issue) => assess("issue", issue).current).map((group) => [group.section, group.issues]));
+	const present = new Set(result.issues.map(sectionOf));
+	const headless = present.size === 1 && present.has("other");
+	const groups = ISSUE_SECTIONS.filter((section) => present.has(section)).map((section) => {
+		const issues = grouped.get(section) ?? [];
+		if (headless) return `<div class="pr-helper-issue-group" data-impact-group="${section}"${issues.length ? "" : " hidden"}>
+      <div class="pr-helper-issue-group-list">${issues.map(card).join("")}</div>
+    </div>`;
+		return `<details class="pr-helper-issue-group" data-impact-group="${section}"${folds[section] === false ? "" : " open"}${issues.length ? "" : " hidden"}>
+      <summary class="pr-helper-issue-group-summary"><h4 class="pr-helper-issue-group-title">${escapeHtml$1(ISSUE_SECTION_TITLES[section])}</h4><span class="pr-helper-issue-group-count" data-issue-group-count>${issues.length}</span></summary>
+      <div class="pr-helper-issue-group-list">${issues.map(card).join("")}</div>
+    </details>`;
+	}).join("");
+	const nav = renderImpactNavItems(tally);
+	const orientation = result.orientation?.trim();
+	const head = `<div class="pr-helper-issues-orientation">
+      ${orientation ? `<p class="pr-helper-issues-orientation-text">${renderInlineText(orientation)}</p>` : ""}
+      <p class="pr-helper-issues-attention" role="status" data-issues-attention>${escapeHtml$1(issuesAttentionLine(tally))}</p>
+      <nav class="pr-helper-issue-nav" aria-label="Issues by kind" data-issue-nav-list${nav ? "" : " hidden"}>${nav}</nav>
+    </div>`;
+	if (closedIssues.length === 0) return `${head}${groups}`;
+	return `${head}${groups}${renderClosedFindings(closedFindingsSummary(counts), closedIssues.map(card).join(""), "issues", counts.open === 0, closedIssues.filter((issue) => assess("issue", issue).pending || statusOf("issue", issue.id).pending).length, folds.closed === true)}`;
+}
+/** What the issue means, read first: the scenario and the step that would settle it. */
+function renderConsequence(impact) {
+	if (!impact) return "";
+	return `<div class="pr-helper-issue-impact">
+        <p class="pr-helper-issue-scenario">${renderInlineText(impact.scenario)}</p>
+        ${renderConsequenceStep(impact.next)}
+      </div>`;
+}
+var STEP_LABELS = {
+	verify: "To verify:",
+	decide: "To decide:"
+};
+function renderConsequenceStep(step) {
+	if (!step) return "";
+	return `<p class="pr-helper-issue-next"><span class="pr-helper-issue-next-label">${step.kind ? STEP_LABELS[step.kind] : "Next step:"}</span> ${renderInlineText(step.text)}</p>`;
+}
+/** A fixed issue's original consequence, kept with the technical account once the fix heads the card. */
+function renderOriginalConsequence(impact) {
+	return `<div class="pr-helper-issue-original">
+            <h5>Before the fix</h5>
+            <p class="pr-helper-issue-original-title">${renderInlineText(impact.title)}</p>
+            <p class="pr-helper-issue-scenario">${renderInlineText(impact.scenario)}</p>
+          </div>`;
 }
 /**
 * What to call the closed pile, by what is actually in it.
@@ -6783,7 +6929,8 @@ function renderStandardViews(view, sections) {
 		askBuddy: true,
 		feedbackSelect: true,
 		reassess: true,
-		findingRefs: true
+		findingRefs: true,
+		...view.reviewLayout?.issueGroups ? { issueFolds: view.reviewLayout.issueGroups } : {}
 	})}
       </section>
       <section class="prb-view" data-view="questions" hidden>
@@ -10136,7 +10283,7 @@ function renderReviewsPage(view) {
 		const collected = (memberOf.get(group.key) ?? []).length > 0;
 		const home = homeSection(group.latest.pinned === true, group.latest.lifecycle, activityOf(group), collected, "all");
 		if (home === void 0) collectedHome.add(group);
-		sections[home ?? sectionOf(group.latest.lifecycle, activityOf(group))].push(group);
+		sections[home ?? sectionOf$1(group.latest.lifecycle, activityOf(group))].push(group);
 	}
 	const storedReviews = view.groups.reduce((total, group) => total + group.history.length, 0);
 	const renderSectionAt = (section) => renderSection(section, sections[section], (groups) => renderCards(groups, "home", (group) => collectedHome.has(group)), sectionOpen(section, view.sectionsCollapsed), sections[section].filter((group) => !inProgressOnly.has(group.latest.id) && !collectedHome.has(group)).length);
@@ -11000,10 +11147,11 @@ function isRunsSection(value) {
 /**
 * How the reviewer last laid out a review page, kept so the next review opens the same way.
 *
-* Three things, each the layout as it last stood rather than a log of what was pressed: whether
+* Four things, each the layout as it last stood rather than a log of what was pressed: whether
 * Buddy AI was open or closed, whether a surface was expanded (Buddy AI itself, or the document,
-* whichever pane that is on the next review), and which parts of the rail were folded. A field
-* nobody has changed has no key, so the page's own defaults still decide for a new reviewer.
+* whichever pane that is on the next review), which parts of the rail were folded, and which
+* sections of the Issues pane were. A field nobody has changed has no key, so the page's own
+* defaults still decide for a new reviewer.
 *
 * Pure, so the page and the server read it with the same rules. Stored in `ui.json`; see
 * `preferences.ts`.
@@ -11014,35 +11162,41 @@ var RAIL_FOLDS = [
 	"prb-rail-path",
 	"prb-rail-kiss"
 ];
-function readRail(value) {
+function readFolds(value, known) {
 	if (!value || typeof value !== "object") return void 0;
-	const rail = {};
-	for (const fold of RAIL_FOLDS) {
+	const folds = {};
+	for (const fold of known) {
 		const open = value[fold];
-		if (typeof open === "boolean") rail[fold] = open;
+		if (typeof open === "boolean") folds[fold] = open;
 	}
-	return Object.keys(rail).length > 0 ? rail : void 0;
+	return Object.keys(folds).length > 0 ? folds : void 0;
 }
+var readRail = (value) => readFolds(value, RAIL_FOLDS);
+var readIssueGroups = (value) => readFolds(value, ISSUE_FOLDS);
 /** A stored layout, keeping only what this page knows how to draw. Anything else reads as unset. */
 function readReviewLayout(value) {
 	if (!value || typeof value !== "object") return {};
-	const { ask, expanded, rail } = value;
+	const { ask, expanded, rail, issueGroups } = value;
 	const folds = readRail(rail);
+	const groups = readIssueGroups(issueGroups);
 	return {
 		...ask === "open" || ask === "closed" ? { ask } : {},
 		...expanded === "buddy" || expanded === "document" ? { expanded } : {},
-		...folds ? { rail: folds } : {}
+		...folds ? { rail: folds } : {},
+		...groups ? { issueGroups: groups } : {}
 	};
 }
 /** A patch from the page, or null when it holds nothing this page could have sent. */
 function readReviewLayoutPatch(value) {
 	if (!value || typeof value !== "object") return null;
-	const { ask, expanded, rail } = value;
+	const { ask, expanded, rail, issueGroups } = value;
 	const folds = readRail(rail);
+	const groups = readIssueGroups(issueGroups);
 	const patch = {
 		...ask === "open" || ask === "closed" ? { ask } : {},
 		...expanded === "buddy" || expanded === "document" || expanded === null ? { expanded } : {},
-		...folds ? { rail: folds } : {}
+		...folds ? { rail: folds } : {},
+		...groups ? { issueGroups: groups } : {}
 	};
 	return Object.keys(patch).length > 0 ? patch : null;
 }
@@ -11056,6 +11210,10 @@ function applyReviewLayoutPatch(layout, patch) {
 		...patch.rail ? { rail: {
 			...layout.rail,
 			...patch.rail
+		} } : {},
+		...patch.issueGroups ? { issueGroups: {
+			...layout.issueGroups,
+			...patch.issueGroups
 		} } : {}
 	};
 }
@@ -12522,6 +12680,23 @@ feature:                   E</pre>
     reads to answer your questions about the change, and what Update review starts from. After
     that it is removed. The review itself stays, and a later question makes a new checkout for
     itself.</p>`
+	},
+	{
+		id: "reading-issues",
+		title: "Reading issues",
+		body: `<p>Issues opens with a sentence on what the change touches and how many issues are
+    outstanding. Each issue is grouped by the decision it asks of you: <b>Failures</b> the code shows
+    going wrong under a stated condition, <b>Possible risks</b> still to be verified, <b>Intent to
+    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>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>`
 	},
 	{
 		id: "updating-and-reanalysing",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prreviewbuddy",
-  "version": "0.90.0",
+  "version": "0.91.1",
   "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/sidebar.css

@@ -1521,8 +1521,7 @@
    Deliberately lighter than the Guide's progress box (no bar, no "next up" line): the Guide is
    a workflow you walk in order, so it earns a full dashboard, whereas this is a flat list where
    the only useful fact is how many are still open. */
-.pr-helper-questions-progress,
-.pr-helper-findings-progress {
+.pr-helper-questions-progress {
   display: flex;
   align-items: center;
   gap: 8px;
@@ -1590,6 +1589,180 @@
 .pr-helper-closed-findings-list .pr-helper-issue-card:hover,
 .pr-helper-closed-findings-list .pr-helper-question-card:hover { opacity: 1; }
 
+/* ── Issues by kind ──
+   The pane opens on what the change touches and how much needs attention, then one section per
+   kind of decision. A section is a disclosure so a reader can fold away what they have dealt
+   with; its heading keeps the Questions tab's group style so the two tabs read as one product. */
+.pr-helper-issues-orientation {
+  margin-bottom: 16px;
+}
+
+.pr-helper-issues-orientation-text {
+  font-size: 13px;
+  line-height: 1.5;
+  margin: 0 0 4px;
+  overflow-wrap: anywhere;
+}
+
+.pr-helper-issues-attention {
+  font-size: 12px;
+  font-weight: 600;
+  color: var(--color-fg-muted, #57606a);
+  margin: 0;
+}
+
+.pr-helper-issue-nav {
+  display: flex;
+  flex-wrap: wrap;
+  gap: 6px;
+  margin-top: 10px;
+}
+
+.pr-helper-issue-nav[hidden] { display: none; }
+
+.pr-helper-issue-nav-item {
+  display: inline-flex;
+  align-items: center;
+  gap: 6px;
+  font: inherit;
+  font-size: 12px;
+  font-weight: 600;
+  padding: 3px 10px;
+  border: 1px solid var(--color-border-default, #d0d7de);
+  border-radius: 100px;
+  background: transparent;
+  color: var(--color-fg-default, #1f2328);
+  cursor: pointer;
+}
+
+.pr-helper-issue-nav-item:hover { border-color: #9333ea; }
+.pr-helper-issue-nav-item:focus-visible { outline: 2px solid #9333ea; outline-offset: 2px; }
+
+.pr-helper-issue-nav-count {
+  color: var(--color-fg-muted, #57606a);
+  font-variant-numeric: tabular-nums;
+}
+
+.pr-helper-issue-group { margin-bottom: 8px; }
+.pr-helper-issue-group[hidden] { display: none; }
+
+.pr-helper-issue-group-summary {
+  display: flex;
+  align-items: center;
+  gap: 6px;
+  cursor: pointer;
+  list-style: none;
+  padding: 4px 0;
+  margin-bottom: 6px;
+  color: #9333ea;
+  scroll-margin-top: 12px;
+}
+
+.pr-helper-issue-group-summary::-webkit-details-marker { display: none; }
+
+.pr-helper-issue-group-summary::before {
+  content: '';
+  width: 0;
+  height: 0;
+  border-left: 5px solid currentColor;
+  border-top: 4px solid transparent;
+  border-bottom: 4px solid transparent;
+  transition: transform 0.15s ease;
+}
+
+.pr-helper-issue-group[open] > .pr-helper-issue-group-summary::before { transform: rotate(90deg); }
+.pr-helper-issue-group-summary:focus-visible { outline: 2px solid #9333ea; outline-offset: 2px; border-radius: 2px; }
+
+.pr-helper-issue-group-title {
+  font-size: 11px;
+  font-weight: 700;
+  text-transform: uppercase;
+  letter-spacing: 0.04em;
+  margin: 0;
+}
+
+.pr-helper-issue-group-count {
+  font-size: 11px;
+  font-weight: 600;
+  color: var(--color-fg-muted, #57606a);
+}
+
+/* The consequence, read first. The title row is shared with every card; these sit under it. */
+.pr-helper-issue-card .pr-helper-issue-title { overflow-wrap: anywhere; }
+
+.pr-helper-issue-impact { margin: 0 0 8px; }
+
+.pr-helper-issue-scenario {
+  font-size: 13px;
+  line-height: 1.5;
+  margin: 0 0 6px;
+  text-align: left;
+  overflow-wrap: anywhere;
+}
+
+.pr-helper-issue-next {
+  font-size: 13px;
+  line-height: 1.5;
+  margin: 0;
+  overflow-wrap: anywhere;
+}
+
+.pr-helper-issue-next-label { font-weight: 600; }
+
+/* The technical account, one click away and never open on arrival. Same chevron as the closed
+   drawer, smaller and muted, because it belongs to one card rather than to the pane. */
+.pr-helper-issue-technical { margin-top: 8px; }
+
+.pr-helper-issue-technical-summary {
+  display: inline-flex;
+  align-items: center;
+  gap: 6px;
+  cursor: pointer;
+  list-style: none;
+  font-size: 12px;
+  font-weight: 600;
+  color: var(--color-fg-muted, #57606a);
+  padding: 2px 0;
+}
+
+.pr-helper-issue-technical-summary::-webkit-details-marker { display: none; }
+
+.pr-helper-issue-technical-summary::before {
+  content: '';
+  width: 0;
+  height: 0;
+  border-left: 5px solid currentColor;
+  border-top: 4px solid transparent;
+  border-bottom: 4px solid transparent;
+  transition: transform 0.15s ease;
+}
+
+.pr-helper-issue-technical[open] > .pr-helper-issue-technical-summary::before { transform: rotate(90deg); }
+.pr-helper-issue-technical-summary:hover { color: var(--color-fg-default, #1f2328); }
+.pr-helper-issue-technical-summary:focus-visible { outline: 2px solid #9333ea; outline-offset: 2px; border-radius: 2px; }
+
+.pr-helper-issue-technical-body {
+  margin-top: 8px;
+  padding-left: 11px;
+  border-left: 2px solid var(--color-border-muted, #d8dee4);
+  overflow-wrap: anywhere;
+}
+
+.pr-helper-issue-technical-title {
+  font-size: 13px;
+  font-weight: 600;
+  margin: 0 0 4px;
+}
+
+.pr-helper-issue-original { margin-bottom: 10px; }
+.pr-helper-issue-original h5 { margin: 0 0 4px; }
+
+.pr-helper-issue-original-title {
+  font-size: 13px;
+  font-weight: 600;
+  margin: 0 0 4px;
+}
+
 .pr-helper-question-actions {
   display: flex;
   align-items: center;