prreviewbuddy 0.91.1 → 0.93.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 versionFrom, $i as isExplanationAction, $n as hasAuthorContent, $r as latestKissRun, $t as waitingHistory, A as settleQueue, Aa as isCommitReview, Ai as deleteCollection, An as markRead, Ar as endedAt, At as askCheckout, B as humanizeModelId, Ba as clearAgentPreference, Bi as readQueue, Bn as unreadCount$1, Br as progressSteps, Bt as runJob, C as parsePullRequestLink, Ca as describeAuthorship, Ci as lifecyclePillLabel, Cn as categoryOf, Cr as reviewedShaOf, Ct as RefreshUnavailableError, D as activateQueued, Da as originRemote, Di as collectionsStamp, Dn as eventName, Dr as RUN_RECORD_RETENTION_MS, E as relativeTime$1, Ea as resolveTarget, Ei as sectionOf$1, En as createPrUpdate, Er as fillFileUrlTemplate, Et as mergeUpdateInto, Fa as git, Fi as updateCollections, Fn as readStateOf, Fr as saveJob, Ft as failureOf$1, G as reviewTag, Ga as AGENT_IDS, Gi as questionsOutstanding, Gn as saveRefresh, H as findingRef, Ha as writeAgentPreference, Hi as isIssueOutstanding, Hn as existingPaths, Ia as refExists, Ii as queueAdd, In as removeNotification, It as observeReview, Ja as agentIdentityOf, Ji as verdictClosure, Jn as agentEnvOf, Jr as groupByLineage, Jt as telemetryUploadConsent, K as reviewTagsIn, Ka as agentById, Ki as resolvedByConversation, Kn as storedConversation, Kr as decideKissRecommendation, L as modelProvenanceOf, La as resolveAgent, Li as queueAddMany, Ln as retireNotification, Lr as isLiveJob, Lt as prepareReview, Ma as AmbiguousRefError, Mi as readCollections, Mn as moveIds, Mr as isTerminal, Mt as UpdateAlreadyRunningError, N as cliEntry, Na as shortNameOf, Ni as renameCollection, Nn as pruneNotifications, Nr as loadJob, Nt as liveUpdateFor, O as compactQueue, Oa as DetachedHeadError, Oi as createCollection, On as listNotifications, Or as allJobs, Ot as recheckConversation, Pa as shorthandOf, Pi as setMembership, Pn as readNotification, Pr as pruneJobResidue, Pt as runningJobs, Q as markAskRunning, Qa as runnableAgents, Qi as explanationOf, Qn as authorPreview, Qr as kissDecisionsOf, Qt as feedbackUrl, R as provenanceDetailLines, Ra as resolveModel, Ri as queueRemove, Rn as toneOf, Rr as isTerminalJobPhase, Rt as settleReview, S as extractPullRequestLinks, Sa as objectWithAnyOf, Si as lifecyclePillAction, Sn as subjectOf, Sr as followedBranch, St as FETCH_TIMEOUT_MS, T as runBatch, Ta as kissOutcome, Ti as parseLifecycle, Tn as createPrState, Tr as forgeResolver, Tt as cancellableHere, U as findingRefs, Ui as isQuestionOutstanding, Un as ASK_REREAD_CEILING_MS, Ut as readEvents, V as findingAtRef, Va as readAgentPreference, Vi as doneVerb, Vn as withUsageRecorded, Vr as purposeOf, W as refOf, Wa as STORE_ROOT, Wi as issuesOutstanding, Wn as prepareConversationWithin, Wr as isClaimed, Wt as record, X as isRequestId, Xa as isAgentId, Xi as recommendationsOf, Xn as chatConversationSection, Xr as indexReviewGroups, Xt as BUILD_VERSION, Ya as detectAgents, Yi as processDiscussion, Yn as EXPLAIN_SIMPLY_PROMPT, Yr as indexKeptIds, Za as parkedReason, Zi as explanationActionOf, Zn as UNESTABLISHED_MOTIVATION, Zr as kissDecisionRefusal, _ as endRun, _a as wasReassessed, _i as activityLabel, _n as ancestorsOf, _t as parseWorkspacePort, a as lineageIds, aa as closesFinding, an as ARCHITECTURE_DIR, ar as renderInlineText, b as stopRun, ba as asString, bi as lifecycleItemLabel, bn as announceJob, br as checkCodeFreshness, bt as writePortPreference, c as discardJob, ca as findingIn, ci as recentWorkspaces, cn as partFiles, cr as isBusy, ct as serverEntryExists, d as startKissJob, da as isGradeFor, di as summarise, dn as storeArchitecture, dr as createWorktree, ea as withExplanation, ei as lineageKey, en as mapRuntimes, er as GITHUB_MARKDOWN, f as isDriving, fa as isOffered, fi as touchWorkspace, fn as storedScopes, fr as holdWorktree, g as completeRun, ga as statusLookup, gi as activityAttributes, gn as REPOSITORY_SCOPE, gr as requestJobStop, gt as MIN_WORKSPACE_PORT, h as beginRun, ha as sourcePhrase, hi as LIFECYCLES, hn as QUOTE_LIMITS, hr as removeWorktree, ht as MAX_WORKSPACE_PORT, i as deleteReview, ia as assessmentLookup, ii as mutateWorkspace, in as formatDuration, ir as escapeHtml$1, ja as resolveBase, ji as nameProblemText, jn as markReadForReview, jr as fail$1, jt as findingsChanged, ki as createWithMembers, kn as markAllRead, kr as attemptIdOf, kt as updateReview, l as kissAgentId, la as findingStatus, li as reviewedCommit, ln as readArchitecture, lr as withReviewLock, lt as spawnServer, m as RUN_KINDS, ma as reassessIn, mi as workspaceStamps, mn as ownersOf, mr as readMarker, mt as DEFAULT_WORKSPACE_PORT, n as MCP_TOOL_NAMES, na as NO_STATUS, ni as loadWorkspace, nn as RUNTIME_WORDS, no as AgentUnavailableError, nr as CODE_BLOCK_CLASS, o as liveJobsFor, oa as currentGradeIn, oi as presentedKissRunOf, on as checkArchitecture, ot as readServerRecord, p as track, pa as promptGrade, pi as workspaceRevision, pn as trackedFiles, pr as isWorktreeHeld, pt as writeServerRecord, q as endAskRequest, qa as agentFor$1, qi as reviewerDispositions, qn as isUnchanged, qr as followedRefName, qt as recordTelemetryUploadConsent, r as deleteLineage, ra as analysisRefOf, rn as describeRuntimes, rr as CODE_COPY_CLASS, rt as clearServerRecordOwnedBy, s as wasBlocked, sa as findingAssessment, si as previousKissResult, sn as loadArchitecture, sr as ReviewBeingDeletedError, ta as ANALYSIS_ONLY, ti as lineagePosition, tn as recordRuntimeBestEffort, to as AgentCancelledError, tr as renderMarkdown, tt as readIndexToken, ua as gradeScale, un as repositoryKey, ur as MANAGED_ROOT, ut as stopChild, v as heldRuns, va as followUpFindingId, vi as isLifecycle, vn as isArchitectureId, vt as resolveWorkspacePort, w as unsupportedReason, wa as WITHHELD_REASON, wi as offersLifecycle, wn as createArchitectureNotification, wr as fetchHead, wt as cancelUpdate, x as batchIdentity, xa as extractJsonText, xi as lifecycleName$1, xn as branchOf, xr as checkReview, y as reportRun, ya as SEVERITIES, yi as lifecycleChip, yn as writeJsonAtomically, yr as findExistingReview, yt as statedWorkspacePort, z as agentDisplayName, za as agentIdOf, zi as queueStamp, zn as unreadAround, zr as phasesFor, zt as reanalyseReview } from "./mcp_tools-DbFR9Z90.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.
@@ -4980,13 +5015,12 @@ ${renderDecisions(processed)}
 ${processed.all.length === 0 ? "<p class=\"prb-pr-empty\">This pull request has no comments yet.</p>" : buckets}`;
 }
 /**
-* Re-reading the threads, and what the last re-read found.
+* Re-checking the discussion, and what the last re-read found.
 *
-* The narrowest useful action in the product, and the copy has to keep it narrow. This corrects
-* what the review already claims about other people's threads — who raised one, whether it is
-* still open — and it cannot produce a finding, because deciding whether a new comment changes
-* anything is judgement and judgement is a fresh review. A reviewer who pressed this expecting a
-* re-review would be worse off than one who never saw the button.
+* Two things, said in the order they happen: the threads are fetched and every "already raised"
+* note corrected, then the findings are reassessed against whatever in the discussion is new to
+* them. The copy keeps it narrow in the one way that still matters: the code is the commit the
+* review read, so this is not a re-review and a comment claiming a fix cannot settle it here.
 */
 function renderRefresh(refresh) {
 	if (!refresh?.enabled) return "";
@@ -4994,10 +5028,12 @@ function renderRefresh(refresh) {
 	return `<div class="prb-pr-refresh">
   <button type="button" data-pr-recheck>Re-check the conversation</button>
   <span class="prb-pr-refresh-note">
-    Fetches every thread again and updates what this review says about them. If someone has
-    replied or resolved a thread since the review ran, the "Already raised" notes on the Issues
-    view will be corrected to match, and any note pointing at a thread that has been deleted will
-    be removed. Your code is not looked at again, so no findings are added, changed or removed.
+    Fetches the description and every thread again, corrects the "Already raised" notes, then
+    reassesses the issues and questions against anything new. A reply can answer a question,
+    resolve or reopen an issue, or suggest a different severity, and a concern it raises becomes a
+    new issue only once it is confirmed in the reviewed code. A comment saying something is fixed
+    is not taken on trust: the issue stays open until an update sees the fix in the code. Newer
+    commits are left for Update review.
   </span>
   ${last ? `<p class="prb-pr-refresh-last" data-ok="${last.ok}">
            Last re-checked ${escapeHtml$1(relativeTime$1(last.at))}. ${escapeHtml$1(last.message)}
@@ -6015,6 +6051,7 @@ function renderFreshAlternative(command) {
 */
 function renderUpdateOutcome(outcome) {
 	if (!outcome) return "";
+	if (outcome.scope === "conversation") return renderRecheckOutcome(outcome);
 	const state = outcome.ok ? "done" : outcome.stop === "refused" ? "refused" : outcome.stop === "cancelled" ? "cancelled" : "stopped";
 	const title = outcome.ok ? "Review updated" : outcome.stop === "refused" ? "There was nothing to update against" : outcome.stop === "cancelled" ? "Update cancelled" : "The review was not updated";
 	const facts = outcome.ok ? updateFacts(outcome) : null;
@@ -6027,6 +6064,72 @@ function renderUpdateOutcome(outcome) {
 </div>`;
 }
 /**
+* What a Re-check of the PR discussion did, in the same notice an update reports through.
+*
+* Five outcomes, each with its own headline, because the one the reviewer most needs to tell apart
+* is the partial: the discussion came in, so the "already raised" lines are current, but the
+* findings were not judged against it. That one says why and, when trying again could help, offers it.
+*/
+function renderRecheckOutcome(outcome) {
+	const retry = outcome.stop === "analysis-failed" && outcome.conversationOk === true;
+	const { state, title, body } = recheckWords(outcome);
+	const facts = outcome.ok && outcome.stop === null ? recheckFacts(outcome) : null;
+	return `<div class="prb-update-result" data-state="${state}" data-update-result data-recheck-result role="status">
+  <div class="prb-update-result-body">
+    ${facts ? `<p class="prb-update-facts"><strong>${escapeHtml$1(title)}</strong>${facts.map((fact) => `<span class="prb-update-fact">${escapeHtml$1(fact)}</span>`).join("")}</p>` : `<p><strong>${escapeHtml$1(title)}</strong> ${escapeHtml$1(body)}</p>`}
+    ${retry ? "<p class=\"prb-update-footnote\"><button type=\"button\" class=\"prb-fresh-quiet\" data-pr-recheck>Try again</button></p>" : ""}
+  </div>
+  <button type="button" class="prb-fresh-dismiss" data-update-close>Close</button>
+</div>`;
+}
+function recheckWords(outcome) {
+	if (outcome.stop === "cancelled") return {
+		state: "cancelled",
+		title: "Re-check cancelled",
+		body: "The findings and the discussion shown are exactly as they were."
+	};
+	if (outcome.conversationOk !== true) return {
+		state: "refused",
+		title: "The PR discussion was not re-checked",
+		body: outcome.message
+	};
+	if (outcome.stop === "already-current") return {
+		state: "done",
+		title: "No findings changed",
+		body: outcome.message
+	};
+	if (outcome.stop !== null) return {
+		state: "stopped",
+		title: "Discussion refreshed, findings not reassessed",
+		body: `The latest comments are in, and the notes about who raised what are current. ${outcome.message}`
+	};
+	if (outcome.counts && !findingsChanged(outcome.counts)) return {
+		state: "done",
+		title: "No findings changed",
+		body: "The new comments were read against every open issue and question, and each one stands as it was."
+	};
+	return {
+		state: "done",
+		title: "Findings reassessed against the PR discussion",
+		body: outcome.message
+	};
+}
+/** What a Re-check changed, largest news first, in the words an update uses. No commit: none moved. */
+function recheckFacts(outcome) {
+	const counts = outcome.counts;
+	if (!counts || !findingsChanged(counts)) return null;
+	const plural = (n, one, many) => `${n} ${n === 1 ? one : many}`;
+	return [
+		counts.questionsAnswered ? `${plural(counts.questionsAnswered, "question", "questions")} answered` : "",
+		counts.resolved ? `${plural(counts.resolved, "issue", "issues")} resolved by the discussion` : "",
+		counts.reopened ? `${plural(counts.reopened, "issue", "issues")} reopened` : "",
+		counts.affected ? `${counts.affected} to look at again` : "",
+		counts.newIssues ? plural(counts.newIssues, "new issue", "new issues") : "",
+		counts.gradeChanges ? `${plural(counts.gradeChanges, "grade change", "grade changes")} suggested` : "",
+		counts.decisionsChallenged ? `${plural(counts.decisionsChallenged, "accepted answer", "accepted answers")} put back to you` : ""
+	].filter(Boolean);
+}
+/**
 * A successful update as a row of facts: the commit it reached, then what changed, largest news first.
 *
 * Read at a glance rather than parsed. The sentence it replaces ("Updated to 7bc233f: 2 fixed,
@@ -6411,14 +6514,14 @@ function comparisonNote(code, hasPullRequest) {
 * button is the subject of the sentence now, so there is one name for one thing and the reviewer
 * presses the words they just read.
 *
-* Safe to run from here for the reason it always was: re-reading threads can only correct the
-* state of claims that already exist, never produce a finding.
+* It reads no new code: the findings are reassessed against the discussion at the reviewed commit,
+* and newer commits stay the Update button's.
 */
 function conversationNote(report, actions) {
 	if (!report.conversation.attached) return "";
 	const when = escapeHtml$1(relativeTime$1(report.conversation.importedAt));
 	if (!actions) return `The PR conversation was last checked ${when}.`;
-	return `<button type="button" class="prb-fresh-quiet" data-pr-recheck>Re-check the conversation</button> to refresh comments and resolved threads without re-running the code review. Last checked ${when}.`;
+	return `<button type="button" class="prb-fresh-quiet" data-pr-recheck>Re-check the conversation</button> to fetch new comments and reassess issues and questions against them, at the reviewed commit. Last checked ${when}.`;
 }
 //#endregion
 //#region ../../packages/review-harness/src/workspace/progress_view.ts
@@ -6516,7 +6619,7 @@ function renderProgressView(job, fileCount, cancellable = false, history) {
 * and can be carried on; an update has the previous review, which is why the two are worded apart.
 */
 function renderElapsed(job, cancel, history) {
-	const button = !cancel ? "" : cancel === "Cancel review" ? `<button type="button" class="prb-progress-cancel" data-review-cancel="${escapeHtml$1(job.id)}">Cancel review</button>` : `<button type="button" class="prb-progress-cancel" data-update-cancel="${escapeHtml$1(job.id)}">Cancel update</button>`;
+	const button = !cancel ? "" : cancel === "Cancel review" ? `<button type="button" class="prb-progress-cancel" data-review-cancel="${escapeHtml$1(job.id)}">Cancel review</button>` : `<button type="button" class="prb-progress-cancel" data-update-cancel="${escapeHtml$1(job.id)}">${cancel}</button>`;
 	return `<div class="prb-progress-foot">
     <div class="prb-progress-runtime">
       <p class="prb-progress-elapsed" data-started-at="${job.startedAt}">${escapeHtml$1(runningFor(job.startedAt))}</p>${renderHistory$1(history)}
@@ -6564,11 +6667,14 @@ function renderHistory$1(history) {
 */
 function renderUpdateProgress(job, cancellable = false, history) {
 	if (!isLiveJob(job)) return "";
-	return `<div class="prb-update-progress" data-update-phase="${escapeHtml$1(job.phase)}" role="status" aria-live="polite">
-  <h3 class="prb-doc-title"><span class="prb-spinner" aria-hidden="true"></span>Updating this review</h3>
-  <p class="prb-progress-lead">The previous review will be replaced when reassessment finishes.</p>
+	const recheck = job.scope === "conversation";
+	const title = recheck ? "Re-checking the PR discussion" : "Updating this review";
+	const lead = recheck ? "Issues and questions will be updated from the discussion when reassessment finishes." : "The previous review will be replaced when reassessment finishes.";
+	return `<div class="prb-update-progress" data-update-phase="${escapeHtml$1(job.phase)}"${recheck ? " data-update-scope=\"conversation\"" : ""} role="status" aria-live="polite">
+  <h3 class="prb-doc-title"><span class="prb-spinner" aria-hidden="true"></span>${title}</h3>
+  <p class="prb-progress-lead">${lead}</p>
   ${renderSteps(job)}
-  ${renderElapsed(job, cancellable ? "Cancel update" : null, history)}
+  ${renderElapsed(job, cancellable ? recheck ? "Cancel re-check" : "Cancel update" : null, history)}
 </div>`;
 }
 //#endregion
@@ -6957,20 +7063,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">The issues and questions below, their &ldquo;already raised&rdquo; notes and whether each thread was still open describe 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, which also reassesses the issues and questions against it.</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 +12804,28 @@ 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. The issues, the questions and their “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>Re-checking fetches the latest comments and reassesses every issue and question against
+    anything new in them. A reply can answer a question, resolve or reopen an issue, or suggest a
+    different severity or importance, and a concern raised in a comment becomes a new issue once it
+    is confirmed in the reviewed code. A thread marked resolved, or a comment saying something is
+    fixed, does not resolve an issue on its own: it stays open until an Update sees the fix in the
+    code. Re-checking reads no new commits; Update review does. When the comments only change how
+    much one finding matters, <b>Reassess</b> on its card asks about that one alone.</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>`
 	},
@@ -12707,7 +12838,7 @@ feature:                   E</pre>
       <li><b>Update</b> brings the review up to date with commits added since it was made. Its
       findings are checked against the new commits rather than thrown away, and each says whether
       the new commits affected it. In the workspace, Update review appears on a review whose
-      branch has moved on.</li>
+      branch has moved on. It also reads the pull request discussion, as Re-check does.</li>
       <li><b>Reanalyse</b> analyses the same commit again, for a fresh judgement of the same change,
       for example with a different model.</li>
       <li><b>Fresh review</b> reviews the target again as a new review, and keeps the existing one
@@ -18283,10 +18414,15 @@ async function sendWorkspacePage(res, workspace) {
 	const updateJob = liveUpdateFor(workspace.id);
 	const reviewJob = workspace.jobId ? loadJob(workspace.jobId) : void 0;
 	const kissJob = kissRun ? loadJob(kissRun.jobId) : void 0;
-	const notice = workspace.lastUpdate?.acknowledged ? void 0 : workspace.lastUpdate;
-	if (notice) mutateWorkspace(workspace.id, (current) => {
-		if (current.lastUpdate?.at === notice.at) current.lastUpdate.acknowledged = true;
-	});
+	const unseen = [workspace.lastUpdate, workspace.lastRecheck].filter((outcome) => outcome !== void 0 && !outcome.acknowledged);
+	const notice = unseen.sort((a, b) => b.at - a.at)[0];
+	if (notice) {
+		const read = new Set(unseen.map((outcome) => outcome.at));
+		mutateWorkspace(workspace.id, (current) => {
+			if (current.lastUpdate && read.has(current.lastUpdate.at)) current.lastUpdate.acknowledged = true;
+			if (current.lastRecheck && read.has(current.lastRecheck.at)) current.lastRecheck.acknowledged = true;
+		});
+	}
 	const everyReview = recentWorkspaces(Number.MAX_SAFE_INTEGER);
 	const html = renderWorkspace({
 		session: workspace.session,
@@ -18515,33 +18651,78 @@ async function kissFreshnessOrNull(workspace) {
 	}
 }
 /**
-* Re-read the pull request conversation, at the reviewer's request.
+* Re-check the pull request discussion, at the reviewer's request: re-read it, then reconcile the
+* findings with whatever in it is new to them, at the commit the review read.
+*
+* Answers 200 with `ok: false` rather than an error status when it does not work, for the reason the
+* update route does: a failure is an outcome the page has to explain in a sentence. 409 when an update
+* or another Re-check holds the review, which the reloaded page then shows running.
 *
-* Answers 200 with `ok: false` rather than an error status when it does not work. The page's job
-* here is to tell the reviewer what happened in a sentence, and a failed re-read is an outcome —
-* the conversation on disk is untouched and the review is exactly as usable as it was a moment
-* ago. An HTTP error would make the page choose between a stack trace and silence.
+* Saved in two ways, because two different things can come of it. A reassessment that was applied
+* goes on the way an update does, onto the review as it is on disk. Anything short of that changed
+* only the conversation and the "already raised" lines, and goes on the way a re-read always has,
+* through `saveRefresh`, which will not put an older conversation over a newer one.
 */
 async function handleRefreshPr(res, workspace) {
 	const reattributing = JSON.stringify(workspace.session.result);
-	adoptLegacyBaseline(workspace);
-	const startedAt = Date.now();
+	const begun = Date.now();
 	let outcome;
 	try {
-		outcome = await refreshPrContext(workspace);
-	} catch {
-		outcome = unforeseenFailure(startedAt);
+		outcome = await track(recheckConversation(workspace, { source: "workspace" }));
+	} catch (error) {
+		if (error instanceof UpdateAlreadyRunningError) return send(res, 409, { error: error.message });
+		outcome = {
+			at: Date.now(),
+			ok: false,
+			message: error instanceof RefreshUnavailableError || error instanceof ReviewBeingDeletedError ? error.message : "The findings could not be reassessed. Nothing about them has changed.",
+			stop: "analysis-failed",
+			fetchFailed: false,
+			moved: false,
+			...rereadSince(workspace, begun) ? { conversationOk: workspace.lastPrRefresh.ok } : {},
+			fromSha: null,
+			toSha: null,
+			scope: "conversation"
+		};
 	}
-	saveRefresh(workspace, outcome, reattributing);
+	const applied = outcome.ok && outcome.stop === null;
+	const cancelled = outcome.stop === "cancelled";
+	workspace.lastRecheck = outcome;
+	const reread = !applied && !cancelled ? rereadSince(workspace, begun) : void 0;
+	if (reread) saveRefresh(workspace, reread, reattributing);
+	const saved = mutateWorkspace(workspace.id, (fresh) => applied ? {
+		...mergeUpdateInto(fresh, workspace),
+		lastRecheck: outcome
+	} : {
+		...fresh,
+		lastRecheck: outcome
+	});
+	const refresh = reread ?? (applied ? workspace.lastPrRefresh : void 0);
 	record({
 		event: "pr_conversation_rechecked",
 		reviewId: workspace.id,
-		ok: outcome.ok,
-		changed: outcome.changes ? !isUnchanged(outcome.changes) : false,
-		attributionsRestated: outcome.attributionsRestated ?? 0,
-		attributionsWithdrawn: outcome.attributionsWithdrawn ?? 0
+		ok: outcome.conversationOk === true,
+		changed: refresh?.changes ? !isUnchanged(refresh.changes) : false,
+		attributionsRestated: refresh?.attributionsRestated ?? 0,
+		attributionsWithdrawn: refresh?.attributionsWithdrawn ?? 0,
+		reassessment: reassessmentOf(outcome),
+		resolved: outcome.counts?.resolved ?? 0,
+		questionsAnswered: outcome.counts?.questionsAnswered ?? 0,
+		newIssues: outcome.counts?.newIssues ?? 0
 	});
-	send(res, 200, outcome);
+	send(res, 200, saved?.lastRecheck ?? outcome);
+}
+/** The re-read this request made, if it made one: the copy's record, begun no earlier than the request. */
+function rereadSince(workspace, begun) {
+	const refresh = workspace.lastPrRefresh;
+	return refresh && (refresh.startedAt ?? refresh.at) >= begun ? refresh : void 0;
+}
+/** What a Re-check did to the findings, as the closed set the log records. */
+function reassessmentOf(outcome) {
+	if (outcome.stop === "cancelled") return "cancelled";
+	if (outcome.stop === "already-current") return "not-needed";
+	if (outcome.stop === "analysis-failed") return "failed";
+	if (outcome.stop !== null) return "skipped";
+	return outcome.counts && findingsChanged(outcome.counts) ? "changed" : "unchanged";
 }
 /**
 * Bring the whole review up to date with the checkout, at the reviewer's request.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prreviewbuddy",
-  "version": "0.91.1",
+  "version": "0.93.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/{chunk-DMxtqzNj.js → chunk-Cg4cN8Vk.js}

@@ -1 +1 @@
-import{bt as e}from"./chunk-DRtin0n-.js";var t=`button:not([disabled])`;function n(e){return e.querySelector(`[data-menu-panel]`)}function r(e){return e.querySelector(`[data-menu-trigger]`)}function i(e){let t=n(e);return t?[...t.querySelectorAll(`input:not([disabled]), textarea:not([disabled]), button:not([disabled])`)].filter(e=>!e.closest(`[hidden]`)):[]}function a(e){let r=n(e);return r?[...r.querySelectorAll(t)].filter(e=>!e.closest(`[hidden]`)):[]}var o=`data-menu-placement`,s=`data-menu-align`;function c(e){let t=e.ownerDocument.defaultView,n=0,r=t?.innerHeight??1/0,i=0,a=t?.innerWidth??1/0,o=e=>e!==`visible`&&e!==``;for(let s=e.parentElement;s&&t;s=s.parentElement){let e=t.getComputedStyle(s);if(!o(e.overflowY)&&!o(e.overflowX))continue;let c=s.getBoundingClientRect();o(e.overflowY)&&(n=Math.max(n,c.top),r=Math.min(r,c.bottom)),o(e.overflowX)&&(i=Math.max(i,c.left),a=Math.min(a,c.right))}return{top:n,bottom:r,left:i,right:a}}var l=8,u=120;function d(e){let t=n(e),i=r(e);if(!t||!i)return;t.removeAttribute(o),t.removeAttribute(s),t.style.removeProperty(`max-height`),t.style.removeProperty(`overflow-y`),t.style.removeProperty(`box-sizing`),t.scrollTop=0;let a=t.getBoundingClientRect(),d=i.getBoundingClientRect(),f=c(e);a.right>f.right-l&&d.right-a.width>=f.left+l&&t.setAttribute(s,`end`);let p=Math.max(0,a.top-d.bottom),m=f.bottom-d.bottom-p-l,h=d.top-f.top-p-l,g=a.height,_=g<=m?`below`:g<=h||h>m?`above`:`below`;_===`above`&&t.setAttribute(o,`above`);let v=_===`above`?h:m;g<=v||(t.style.boxSizing=`border-box`,t.style.maxHeight=`${Math.max(Math.floor(v),u)}px`,t.style.overflowY=`auto`)}function f(e){return!e.metaKey&&!e.ctrlKey&&!e.altKey&&!e.shiftKey?e.key===`j`?`ArrowDown`:e.key===`k`?`ArrowUp`:e.key:e.key}function p(t,o={}){let s=null,c=null,l=o.within?`${o.within} [data-menu]`:`[data-menu]`,u=e=>e&&(!o.within||e.closest(o.within))?e:null,p=(e,t={})=>{let i=n(e);if(!i||i.hidden)return;i.hidden=!0;let a=r(e);a?.setAttribute(`aria-expanded`,`false`);let l=s===e?c:null;s===e&&(s=null,c=null),o.onClose?.(e),t.returnFocus&&(l??(()=>a?.focus()))()},m=()=>{for(let e of t.querySelectorAll(l))p(e)},h=(e,t,i=null)=>{let l=n(e);l&&(m(),c=i,o.onOpen?.(e),l.hidden=!1,d(e),r(e)?.setAttribute(`aria-expanded`,`true`),s=e,t&&a(e)[0]?.focus())};return t.addEventListener(`click`,e=>{let t=e.target,r=u(t?.closest?.(`[data-menu]`)??null);if(!r){m();return}if(t?.closest(`[data-menu-trigger]`)){let t=e.detail===0;n(r)?.hidden?h(r,t):p(r);return}s&&s!==r&&p(s)}),t.addEventListener(`keydown`,t=>{if(s&&!s.isConnected&&(s=null),!s)return;let n=s;if(t.key===`Escape`){t.preventDefault(),p(n,{returnFocus:!0});return}if(!n.contains(t.target))return;if(t.key===`Tab`){if(n.hasAttribute(`data-menu-form`)){let e=i(n),r=e.indexOf(t.target);if(e.length>0){t.preventDefault(),e[(r+(t.shiftKey?-1:1)+e.length)%e.length].focus();return}}p(n);return}if(e(t.target))return;let r=a(n);if(r.length===0)return;let o=r.indexOf(t.target),c=f(t),l=c===`ArrowDown`?r[(o+1)%r.length]:c===`ArrowUp`?r[(o-1+r.length)%r.length]:c===`Home`?r[0]:c===`End`?r[r.length-1]:null;l&&(t.preventDefault(),l.focus(),l.scrollIntoView({block:`nearest`}))}),{open:(e,t={})=>h(e,t.focusFirst===!0,t.returnFocus??null),close:p,closeAll:m}}function m(e){return e<3e4?1e3:e<12e4?3e3:e<6e5?8e3:15e3}export{p as n,m as t};
+import{bt as e}from"./chunk-DRtin0n-.js";var t=`button:not([disabled])`;function n(e){return e.querySelector(`[data-menu-panel]`)}function r(e){return e.querySelector(`[data-menu-trigger]`)}function i(e){let t=n(e);return t?[...t.querySelectorAll(`input:not([disabled]), textarea:not([disabled]), button:not([disabled])`)].filter(e=>!e.closest(`[hidden]`)):[]}function a(e){let r=n(e);return r?[...r.querySelectorAll(t)].filter(e=>!e.closest(`[hidden]`)):[]}var o=`data-menu-placement`,s=`data-menu-align`;function c(e){let t=e.ownerDocument.defaultView,n=0,r=t?.innerHeight??1/0,i=0,a=t?.innerWidth??1/0,o=e=>e!==`visible`&&e!==``;for(let s=e.parentElement;s&&t;s=s.parentElement){let e=t.getComputedStyle(s);if(!o(e.overflowY)&&!o(e.overflowX))continue;let c=s.getBoundingClientRect();o(e.overflowY)&&(n=Math.max(n,c.top),r=Math.min(r,c.bottom)),o(e.overflowX)&&(i=Math.max(i,c.left),a=Math.min(a,c.right))}return{top:n,bottom:r,left:i,right:a}}var l=8,u=120;function d(e){let t=n(e),i=r(e);if(!t||!i)return;t.removeAttribute(o),t.removeAttribute(s),t.style.removeProperty(`max-height`),t.style.removeProperty(`overflow-y`),t.style.removeProperty(`box-sizing`),t.scrollTop=0;let a=t.getBoundingClientRect(),d=i.getBoundingClientRect(),f=c(e);a.right>f.right-l&&d.right-a.width>=f.left+l&&t.setAttribute(s,`end`);let p=Math.max(0,a.top-d.bottom),m=f.bottom-d.bottom-p-l,h=d.top-f.top-p-l,g=a.height,_=g<=m?`below`:g<=h||h>m?`above`:`below`;_===`above`&&t.setAttribute(o,`above`);let v=_===`above`?h:m;g<=v||(t.style.boxSizing=`border-box`,t.style.maxHeight=`${Math.max(Math.floor(v),u)}px`,t.style.overflowY=`auto`)}function f(e){return!e.metaKey&&!e.ctrlKey&&!e.altKey&&!e.shiftKey?e.key===`j`?`ArrowDown`:e.key===`k`?`ArrowUp`:e.key:e.key}function p(t,o={}){let s=null,c=null,l=o.within?`${o.within} [data-menu]`:`[data-menu]`,u=e=>e&&(!o.within||e.closest(o.within))?e:null,p=(e,t={})=>{let i=n(e);if(!i||i.hidden)return;i.hidden=!0;let a=r(e);a?.setAttribute(`aria-expanded`,`false`);let l=s===e?c:null;s===e&&(s=null,c=null),o.onClose?.(e),t.returnFocus&&(l??(()=>a?.focus()))()},m=()=>{for(let e of t.querySelectorAll(l))p(e)},h=(e,t,i=null)=>{let l=n(e);l&&(m(),c=i,o.onOpen?.(e),l.hidden=!1,d(e),r(e)?.setAttribute(`aria-expanded`,`true`),s=e,t&&a(e)[0]?.focus())};return t.addEventListener(`click`,e=>{let t=e.target,r=u(t?.closest?.(`[data-menu]`)??null);if(!r){m();return}if(t?.closest(`[data-menu-trigger]`)){let t=e.detail===0;n(r)?.hidden?h(r,t):p(r);return}s&&s!==r&&p(s)}),t.addEventListener(`keydown`,t=>{if(s&&!s.isConnected&&(s=null),!s)return;let n=s;if(t.key===`Escape`){t.preventDefault(),p(n,{returnFocus:!0});return}if(!n.contains(t.target))return;if(t.key===`Tab`){if(n.hasAttribute(`data-menu-form`)){let e=i(n),r=e.indexOf(t.target);if(e.length>0){t.preventDefault(),e[(r+(t.shiftKey?-1:1)+e.length)%e.length].focus();return}}p(n);return}if(e(t.target))return;let r=a(n);if(r.length===0)return;let o=r.indexOf(t.target),c=f(t),l=c===`ArrowDown`?r[(o+1)%r.length]:c===`ArrowUp`?r[(o-1+r.length)%r.length]:c===`Home`?r[0]:c===`End`?r[r.length-1]:null;l&&(t.preventDefault(),l.focus(),l.scrollIntoView({block:`nearest`}))}),{open:(e,t={})=>h(e,t.focusFirst===!0,t.returnFocus??null),close:p,closeAll:m}}function m(e){return e===`done`||e===`failed`}function h(e,t){let n=e=>e.toLowerCase().replace(/\bthe\b/g,``).replace(/[^a-z]/g,``);return n(e)===n(t)}function g(e){return e<3e4?1e3:e<12e4?3e3:e<6e5?8e3:15e3}export{p as i,m as n,h as r,g as t};