@overdeck/desktop 0.45.7 → 0.45.8

package/server/{review-artifacts-DTTdAYsU.js → review-artifacts-BGc69epg.js}

@@ -1,9 +1,9 @@
-import { Qn as extractNumberSync, er as init_issue_id } from "./config-yaml-DbdN9Fuq.js";
-import { a as init_activity_logger, n as emitActivityEntrySync } from "./activity-logger-Dm95Ua07.js";
-import { a as upsertMergeSetSync, i as init_merge_set, n as ensureMergeSetForIssueSync, o as withRepoArtifactUrlSync, s as withRepoStateSync } from "./merge-set-CUY9msG9.js";
-import { i as findPlanSync, o as init_io } from "./io-DtAQ_2hp.js";
-import { a as queryBeadsForIssue, r as init_beads_query } from "./beads-query-CtqWakL6.js";
-import { n as getForgeAdapter } from "./forge-BnTExkbK.js";
+import { Qn as extractNumberSync, er as init_issue_id } from "./config-yaml-DPAieL_G.js";
+import { a as init_activity_logger, n as emitActivityEntrySync } from "./activity-logger-pcbf-e30.js";
+import { a as upsertMergeSetSync, i as init_merge_set, n as ensureMergeSetForIssueSync, o as withRepoArtifactUrlSync, s as withRepoStateSync } from "./merge-set-TRxJwR-w.js";
+import { i as findPlanSync, o as init_io } from "./io-Wy8YpFeP.js";
+import { a as queryBeadsForIssue, r as init_beads_query } from "./beads-query-BrVNWoSL.js";
+import { n as getForgeAdapter } from "./forge-BixzckB0.js";
 import { Data, Effect } from "effect";
 import { existsSync } from "node:fs";
 import { join } from "node:path";
@@ -167,4 +167,4 @@ const createReviewArtifactsForIssue = (issueId, workspacePath) => Effect.tryProm
 //#endregion
 export { createReviewArtifactsForIssue };
 
-//# sourceMappingURL=review-artifacts-DTTdAYsU.js.map
+//# sourceMappingURL=review-artifacts-BGc69epg.js.map

package/server/{review-context-De14xPtc.js → review-context-CCq56O_Y.js}

@@ -1,11 +1,11 @@
 import { n as __esmMin } from "./rolldown-runtime-DNCzzKnI.js";
-import { cr as FsError, vr as init_errors } from "./config-yaml-DbdN9Fuq.js";
+import { cr as FsError, vr as init_errors } from "./config-yaml-DPAieL_G.js";
 import { a as PAN_DIRNAME, p as init_types } from "./types-CE1m_daD.js";
-import { i as findPlanSync, o as init_io, u as readPlan } from "./io-DtAQ_2hp.js";
-import { d as getDevrootPathSync, f as init_config } from "./config-DkEhJM0h.js";
-import { a as init_lifecycle_io, i as findVBriefByIssueSync } from "./lifecycle-io-WDGwa1vV.js";
-import { n as init_review_verdict_feedback } from "./review-verdict-feedback-Cfc-THvz.js";
-import { n as init_github_pr_lookup, r as lookupPullRequestNumberForBranch } from "./github-pr-lookup-CPri6JR1.js";
+import { i as findPlanSync, o as init_io, u as readPlan } from "./io-Wy8YpFeP.js";
+import { d as getDevrootPathSync, f as init_config } from "./config-oWoKf5jO.js";
+import { a as init_lifecycle_io, i as findVBriefByIssueSync } from "./lifecycle-io-DSrv9xGh.js";
+import { n as init_review_verdict_feedback } from "./review-verdict-feedback-wqnikhMK.js";
+import { n as init_github_pr_lookup, r as lookupPullRequestNumberForBranch } from "./github-pr-lookup-t8X8R0c0.js";
 import { Effect } from "effect";
 import { existsSync } from "fs";
 import { join } from "path";
@@ -700,4 +700,4 @@ var init_review_context = __esmMin((() => {
 //#endregion
 export { getDiffStat as a, getDiffBase as i, formatTier1Summary as n, init_review_context as o, getChangedFiles as r, buildReviewContext as t };
 
-//# sourceMappingURL=review-context-De14xPtc.js.map
+//# sourceMappingURL=review-context-CCq56O_Y.js.map

package/server/{review-monitor-BNV3kjLq.js → review-monitor-BKG2ubt5.js}

@@ -1,5 +1,5 @@
 import { n as __esmMin } from "./rolldown-runtime-DNCzzKnI.js";
-import { x as init_tmux } from "./tmux-CBYZ5lKF.js";
+import { x as init_tmux } from "./tmux-ePNhMIMU.js";
 import "effect";
 import "fs";
 import "path";
@@ -18,4 +18,4 @@ var init_review_monitor = __esmMin((() => {
 //#endregion
 export { init_review_monitor as n, REVIEW_SUB_ROLES as t };
 
-//# sourceMappingURL=review-monitor-BNV3kjLq.js.map
+//# sourceMappingURL=review-monitor-BKG2ubt5.js.map

package/server/{review-status-DWAsTqfJ.js → review-status-BQjqQXZG.js}

@@ -1,18 +1,18 @@
 import { n as __esmMin } from "./rolldown-runtime-DNCzzKnI.js";
-import { At as queueIssueRecordCommit, Dn as getProjectSync, Et as getIssueWorkspacePath, Gn as resolveProjectFromIssueSync, Nn as init_projects, Nt as readIssueRecordSync, Ot as init_record, jt as readIssueRecord, zt as writeIssueRecordSync } from "./config-yaml-DbdN9Fuq.js";
-import { a as init_activity_logger, n as emitActivityEntrySync, r as emitActivityTtsSync } from "./activity-logger-Dm95Ua07.js";
-import { d as getOverdeckDatabaseSync, f as init_infra } from "./infra-CCTzpcuJ.js";
-import { d as init_cost_sync, i as getCostBreakdownByStageAndModelSync, o as getCostForIssueSync } from "./cost-sync-CfXu-3TF.js";
-import { r as notifyPipelineSync, t as init_pipeline_notifier } from "./pipeline-notifier-BbGQtvKM.js";
-import { c as init_review_status_sync, d as init_review_status_normalize, f as normalizeReviewStatusSync, l as markWorkspaceStuck$1, n as deleteReviewStatus, o as getReviewStatusFromDbSync, r as getAllReviewStatusesFromDb, s as getReviewStatusesFromDb, u as upsertReviewStatusSync } from "./review-status-sync-D-p7TQit.js";
+import { At as queueIssueRecordCommit, Dn as getProjectSync, Et as getIssueWorkspacePath, Gn as resolveProjectFromIssueSync, Nn as init_projects, Nt as readIssueRecordSync, Ot as init_record, jt as readIssueRecord, zt as writeIssueRecordSync } from "./config-yaml-DPAieL_G.js";
+import { a as init_activity_logger, n as emitActivityEntrySync, r as emitActivityTtsSync } from "./activity-logger-pcbf-e30.js";
+import { d as getOverdeckDatabaseSync, f as init_infra } from "./infra-CpKlYZA9.js";
+import { d as init_cost_sync, i as getCostBreakdownByStageAndModelSync, o as getCostForIssueSync } from "./cost-sync-8lkDeL9g.js";
+import { r as notifyPipelineSync, t as init_pipeline_notifier } from "./pipeline-notifier-aX_5EqQO.js";
+import { c as init_review_status_sync, d as init_review_status_normalize, f as normalizeReviewStatusSync, l as markWorkspaceStuck$1, n as deleteReviewStatus, o as getReviewStatusFromDbSync, r as getAllReviewStatusesFromDb, s as getReviewStatusesFromDb, u as upsertReviewStatusSync } from "./review-status-sync-BWHnrq-k.js";
 import { a as PAN_DIRNAME, p as init_types, r as PAN_CONTINUES_DIRNAME } from "./types-CE1m_daD.js";
-import { i as init_merge_set, r as getMergeSetSync } from "./merge-set-CUY9msG9.js";
+import { i as init_merge_set, r as getMergeSetSync } from "./merge-set-TRxJwR-w.js";
 import { Data, Effect } from "effect";
 import { join } from "path";
 import { homedir } from "os";
 import { hostname } from "node:os";
-import { existsSync, promises } from "node:fs";
-import { join as join$1 } from "node:path";
+import { existsSync, mkdirSync, promises, readFileSync, writeFileSync } from "node:fs";
+import { dirname as dirname$1, join as join$1 } from "node:path";
 //#region ../../lib/overdeck/agent-state-sync.ts
 function isoFromMillis(value) {
 	return value == null ? void 0 : new Date(value).toISOString();
@@ -190,6 +190,23 @@ var init_agent_state_sync = __esmMin((() => {
 	SELECT_AGENT_SQL = `SELECT ${AGENT_COLUMNS_FOR_DB.join(", ")} FROM agents`;
 }));
 //#endregion
+//#region ../../lib/cloister/review-status-source.ts
+function registerReviewStatusMapReader(fn) {
+	reader = fn;
+}
+function readReviewStatusMap() {
+	if (!reader) return null;
+	try {
+		return reader();
+	} catch {
+		return null;
+	}
+}
+var reader;
+var init_review_status_source = __esmMin((() => {
+	reader = null;
+}));
+//#endregion
 //#region ../../lib/pan-dir/record-lock.ts
 /**
 * Serialize async read-modify-write rebuilds for a single issue record.
@@ -363,13 +380,18 @@ async function buildIssueRecord(project, issueId, opts = {}) {
 * PAN-1908 / PAN-1919: rebuild and queue the per-issue permanent record for a
 * given issue. Fire-and-forget: failures are logged, never thrown, so
 * review-status writes stay synchronous and fast.
+*
+* PAN-2583: returns whether the durable write actually landed, so the caller
+* can fall back to a sandbox-writable verdict drop when it did not (a sandboxed
+* reviewer cannot write ${OVERDECK_HOME}/state, and swallowing that here was
+* how blocked review verdicts silently vanished).
 */
 async function updateIssueRecordForIssue(issueId, reviewStatus) {
 	try {
 		const resolved = resolveProjectFromIssueSync(issueId);
-		if (!resolved) return;
+		if (!resolved) return false;
 		const project = getProjectSync(resolved.projectKey);
-		if (!project) return;
+		if (!project) return false;
 		await withIssueRecordLock(issueId, async () => {
 			const record = await buildIssueRecord(project, issueId, { reviewStatus });
 			const fresh = readIssueRecordSync(project, issueId);
@@ -380,8 +402,10 @@ async function updateIssueRecordForIssue(issueId, reviewStatus) {
 			};
 			queueIssueRecordCommit(project, issueId, writeIssueRecordSync(project, issueId, record));
 		});
+		return true;
 	} catch (err) {
 		console.warn(`[pan-dir/records] Failed to update record for ${issueId}: ${err.message}`);
+		return false;
 	}
 }
 var init_records = __esmMin((() => {
@@ -396,7 +420,9 @@ var init_records = __esmMin((() => {
 //#endregion
 //#region ../../lib/overdeck/review-status-record-sync.ts
 function updateIssueRecordForReviewStatusSync(issueId, status) {
-	updateIssueRecordForIssue(issueId, status);
+	Promise.resolve(updateIssueRecordForIssue(issueId, status)).then((landed) => {
+		if (landed === false) writeWorkspaceVerdictFallbackSync(issueId, status);
+	}).catch(() => writeWorkspaceVerdictFallbackSync(issueId, status));
 }
 /** Resolve and read the per-issue journal record's pipeline block, or null. Best-effort. */
 function readPipelineSync(issueId) {
@@ -432,6 +458,78 @@ function enrichReviewNotesFromRecordSync(issueId, status) {
 	};
 }
 /**
+* The durable field subset shared by the journal record's pipeline block and the
+* workspace verdict fallback (PAN-2583). Derived/live columns (readyForMerge,
+* blockerReasons) are intentionally omitted — the reader recomputes them.
+*/
+function durableSubset(p) {
+	return {
+		reviewStatus: p.reviewStatus,
+		testStatus: p.testStatus,
+		mergeStatus: p.mergeStatus ?? void 0,
+		inspectStatus: p.inspectStatus ?? void 0,
+		verificationStatus: p.verificationStatus ?? void 0,
+		reviewNotes: p.reviewNotes,
+		testNotes: p.testNotes,
+		mergeNotes: p.mergeNotes,
+		inspectNotes: p.inspectNotes,
+		verificationNotes: p.verificationNotes,
+		scopeDrift: p.scopeDrift,
+		prUrl: p.prUrl,
+		prNumber: p.prNumber,
+		prHeadSha: p.prHeadSha,
+		reviewedAtCommit: p.reviewedAtCommit,
+		lastVerifiedCommit: p.lastVerifiedCommit,
+		reviewRequestedAt: p.reviewRequestedAt,
+		reviewerVerdicts: p.reviewerVerdicts,
+		autoMerge: p.autoMerge,
+		deaconIgnored: p.deaconIgnored,
+		deaconIgnoredAt: p.deaconIgnoredAt,
+		deaconIgnoredReason: p.deaconIgnoredReason,
+		closedOut: p.closedOut,
+		closedOutAt: p.closedOutAt
+	};
+}
+function workspaceVerdictFallbackPath(issueId) {
+	try {
+		const resolved = resolveProjectFromIssueSync(issueId);
+		if (!resolved) return null;
+		return join$1(resolved.projectPath, "workspaces", `feature-${issueId.toLowerCase()}`, ".overdeck", "pipeline-verdict.json");
+	} catch {
+		return null;
+	}
+}
+function writeWorkspaceVerdictFallbackSync(issueId, status) {
+	try {
+		const path = workspaceVerdictFallbackPath(issueId);
+		if (!path) {
+			console.error(`[review-status] PAN-2583: journal write failed for ${issueId} and no workspace could be resolved — the verdict is NOT durable`);
+			return;
+		}
+		mkdirSync(dirname$1(path), { recursive: true });
+		const payload = {
+			issueId: issueId.toUpperCase(),
+			updatedAt: status.updatedAt ?? (/* @__PURE__ */ new Date()).toISOString(),
+			pipeline: durableSubset(status)
+		};
+		writeFileSync(path, JSON.stringify(payload, null, 2));
+		console.warn(`[review-status] journal write failed for ${issueId} — durable verdict written to workspace fallback ${path}; the host sweeps it on the next status read (PAN-2583)`);
+	} catch (err) {
+		console.error(`[review-status] PAN-2583: could not persist verdict for ${issueId} anywhere durable: ${err.message}`);
+	}
+}
+function readWorkspaceVerdictFallbackSync(issueId) {
+	try {
+		const path = workspaceVerdictFallbackPath(issueId);
+		if (!path || !existsSync(path)) return null;
+		const parsed = JSON.parse(readFileSync(path, "utf-8"));
+		if (!parsed?.updatedAt || !parsed.pipeline) return null;
+		return parsed;
+	} catch {
+		return null;
+	}
+}
+/**
 * PAN-1988: read the journal record's durable verdict for an issue — the SOURCE OF TRUTH.
 * Returns the record `updatedAt` (used to decide whether the journal is newer than the DB
 * cache) plus the durable status fields (flags + feedback notes). Derived/live columns
@@ -440,37 +538,32 @@ function enrichReviewNotesFromRecordSync(issueId, status) {
 * This is what makes verdict writes host-owned: a sandboxed agent can always write the journal
 * (workspace-local) even when it cannot write `~/.overdeck/overdeck.db`. The host reconciles
 * the cache from this on read, so no agent has to escalate out of its sandbox to record a verdict.
+*
+* PAN-2583: since the records migration (PAN-2541) the journal itself lives in
+* ${OVERDECK_HOME}/state, which a sandboxed reviewer cannot write either. When such a
+* writer dropped its verdict into the workspace fallback instead, overlay it here —
+* a strictly newer fallback wins over the record, and the host's own next journal
+* write folds it back into the canonical record.
 */
 function readJournalStatusSync(issueId) {
 	const p = readPipelineSync(issueId);
+	const fallback = readWorkspaceVerdictFallbackSync(issueId);
+	const fallbackNewer = !!fallback && (!p || (p.updatedAt ?? "") < fallback.updatedAt);
+	if (fallback && fallbackNewer) {
+		const base = p ? durableSubset(p) : {};
+		const overlay = Object.fromEntries(Object.entries(fallback.pipeline).filter(([, value]) => value !== void 0));
+		return {
+			updatedAt: fallback.updatedAt,
+			durable: {
+				...base,
+				...overlay
+			}
+		};
+	}
 	if (!p) return null;
 	return {
 		updatedAt: p.updatedAt,
-		durable: {
-			reviewStatus: p.reviewStatus,
-			testStatus: p.testStatus,
-			mergeStatus: p.mergeStatus ?? void 0,
-			inspectStatus: p.inspectStatus ?? void 0,
-			verificationStatus: p.verificationStatus ?? void 0,
-			reviewNotes: p.reviewNotes,
-			testNotes: p.testNotes,
-			mergeNotes: p.mergeNotes,
-			inspectNotes: p.inspectNotes,
-			verificationNotes: p.verificationNotes,
-			scopeDrift: p.scopeDrift,
-			prUrl: p.prUrl,
-			prNumber: p.prNumber,
-			prHeadSha: p.prHeadSha,
-			reviewedAtCommit: p.reviewedAtCommit,
-			lastVerifiedCommit: p.lastVerifiedCommit,
-			reviewRequestedAt: p.reviewRequestedAt,
-			autoMerge: p.autoMerge,
-			deaconIgnored: p.deaconIgnored,
-			deaconIgnoredAt: p.deaconIgnoredAt,
-			deaconIgnoredReason: p.deaconIgnoredReason,
-			closedOut: p.closedOut,
-			closedOutAt: p.closedOutAt
-		}
+		durable: durableSubset(p)
 	};
 }
 var init_review_status_record_sync = __esmMin((() => {
@@ -526,7 +619,7 @@ function verificationSatisfied(status) {
 * journal→DB reconcile in getReviewStatusSync derive readyForMerge identically.
 */
 function reviewGatesPassedSync(s) {
-	return s.reviewStatus === "passed" && (s.testStatus === "passed" || s.testStatus === "skipped") && verificationSatisfied(s) && (s.uatStatus === void 0 || s.uatStatus === "passed") && (s.mergeStatus === "pending" || s.mergeStatus === "queued" || s.mergeStatus === void 0 || s.mergeStatus === null);
+	return (s.reviewStatus === "passed" || s.reviewStatus === "skipped") && (s.testStatus === "passed" || s.testStatus === "skipped") && verificationSatisfied(s) && (s.uatStatus === void 0 || s.uatStatus === "passed") && (s.mergeStatus === "pending" || s.mergeStatus === "queued" || s.mergeStatus === void 0 || s.mergeStatus === null);
 }
 /**
 * Authoritative "allowed to merge" predicate (PAN-1759). The flywheel
@@ -542,7 +635,7 @@ function mergeGateEligibility(status) {
 		eligible: false,
 		reason: "no review record"
 	};
-	if (status.reviewStatus !== "passed") return {
+	if (status.reviewStatus !== "passed" && status.reviewStatus !== "skipped") return {
 		eligible: false,
 		reason: `review is ${status.reviewStatus}`
 	};
@@ -627,6 +720,15 @@ function setReviewStatusSync(issueId, update, existing) {
 		});
 		return status;
 	}
+	if (update.reviewStatus === "reviewing" && (status.reviewStatus === "blocked" || status.reviewStatus === "failed") && update.reviewSpawnedAt === void 0) {
+		console.warn(`[review-status] Rejecting stale 'reviewing' write for ${issueId} — a terminal '${status.reviewStatus}' verdict is already recorded for this review cycle. Only a new dispatch (carrying reviewSpawnedAt) may re-enter 'reviewing'.`);
+		notifyPipelineSync({
+			type: "status_changed",
+			issueId,
+			status
+		});
+		return status;
+	}
 	if ((update.testStatus === "dispatch_failed" || update.testStatus === "failed") && status.testStatus === "passed") {
 		console.warn(`[review-status] Rejecting testStatus regression from 'passed' to '${update.testStatus}' for ${issueId}`);
 		delete update.testStatus;
@@ -636,6 +738,10 @@ function setReviewStatusSync(issueId, update, existing) {
 		...status,
 		...update
 	};
+	if (update.reviewerVerdicts && status.reviewerVerdicts) merged.reviewerVerdicts = {
+		...status.reviewerVerdicts,
+		...update.reviewerVerdicts
+	};
 	const history = [...status.history || []];
 	const now = (/* @__PURE__ */ new Date()).toISOString();
 	if (update.reviewStatus && update.reviewStatus !== status.reviewStatus) history.push({
@@ -672,7 +778,7 @@ function setReviewStatusSync(issueId, update, existing) {
 	});
 	if (readyForMerge && !status.readyForMerge && updated.prUrl) (async () => {
 		try {
-			const { isGitHubAppConfigured, reportCommitStatus } = await import("./github-app-CIhMNI4C.js");
+			const { isGitHubAppConfigured, reportCommitStatus } = await import("./github-app-B5PDHMyn.js");
 			if (!isGitHubAppConfigured()) return;
 			const prMatch = updated.prUrl.match(/github\.com\/([^/]+)\/([^/]+)\/pull/);
 			if (!prMatch) return;
@@ -893,7 +999,7 @@ function setReviewStatusSync(issueId, update, existing) {
 			eventType: "readyForMerge"
 		});
 	}
-	if (update.reviewStatus === "passed" && status.reviewStatus !== "passed" && updated.testStatus === "pending") if (updated.reviewedAtCommit && updated.lastVerifiedCommit && updated.reviewedAtCommit === updated.lastVerifiedCommit) {
+	if ((update.reviewStatus === "passed" || update.reviewStatus === "skipped") && status.reviewStatus !== "passed" && status.reviewStatus !== "skipped" && updated.testStatus === "pending") if (updated.reviewedAtCommit && updated.lastVerifiedCommit && updated.reviewedAtCommit === updated.lastVerifiedCommit) {
 		console.log(`[review-status] Skipping test role for ${issueId} — no code drift since verification (HEAD=${updated.reviewedAtCommit.slice(0, 8)})`);
 		emitActivityEntrySync({
 			source: "cloister",
@@ -950,7 +1056,7 @@ function resetPipelineVerdictsForWorkStartSync(issueId) {
 */
 async function deliverReviewVerdictFeedbackHostSide(issueId, status) {
 	try {
-		const { deliverReviewVerdictFeedback } = await import("./review-verdict-feedback-BbIfN5Vg.js");
+		const { deliverReviewVerdictFeedback } = await import("./review-verdict-feedback-cZh9Wi9G.js");
 		if ((await Effect.runPromise(deliverReviewVerdictFeedback({
 			issueId,
 			verdict: status.reviewStatus === "failed" ? "failed" : "blocked",
@@ -983,7 +1089,7 @@ function maybeAutoDispatchReviewHostSide(issueId, status) {
 }
 async function dispatchReviewHostSide(issueId, prUrl) {
 	try {
-		const { resolveProjectFromIssueSync } = await import("./projects-CKTA6Klk.js");
+		const { resolveProjectFromIssueSync } = await import("./projects-BNmVxnNt.js");
 		const resolved = resolveProjectFromIssueSync(issueId);
 		if (!resolved) return;
 		const { existsSync } = await import("fs");
@@ -999,7 +1105,7 @@ async function dispatchReviewHostSide(issueId, prUrl) {
 			});
 			branch = stdout.trim() || branch;
 		} catch {}
-		const { spawnReviewRoleForIssue } = await import("./review-agent-BRSSFv7e.js");
+		const { spawnReviewRoleForIssue } = await import("./review-agent-DgySDb-G.js");
 		if ((await Effect.runPromise(spawnReviewRoleForIssue({
 			issueId,
 			workspace,
@@ -1021,13 +1127,13 @@ function maybeRecoverTestVerdictHostSide(issueId, status) {
 }
 async function recoverTestVerdictHostSide(issueId) {
 	try {
-		const { resolveProjectFromIssueSync } = await import("./projects-CKTA6Klk.js");
+		const { resolveProjectFromIssueSync } = await import("./projects-BNmVxnNt.js");
 		const resolved = resolveProjectFromIssueSync(issueId);
 		if (!resolved) return;
 		const { existsSync } = await import("fs");
 		const workspace = join(resolved.projectPath, "workspaces", `feature-${issueId.toLowerCase()}`);
 		if (!existsSync(workspace)) return;
-		const { readTestVerdictArtifact } = await import("./test-verdict-CmxX3KEf.js");
+		const { readTestVerdictArtifact } = await import("./test-verdict-BkubHtti.js");
 		const artifact = readTestVerdictArtifact(workspace);
 		if (!artifact) return;
 		setReviewStatusSync(issueId, {
@@ -1041,13 +1147,13 @@ async function recoverTestVerdictHostSide(issueId) {
 }
 async function deliverTestFailureToWorkAgentHostSide(issueId, status) {
 	try {
-		const { resolveProjectFromIssueSync } = await import("./projects-CKTA6Klk.js");
+		const { resolveProjectFromIssueSync } = await import("./projects-BNmVxnNt.js");
 		const resolved = resolveProjectFromIssueSync(issueId);
 		const workspace = resolved ? join(resolved.projectPath, "workspaces", `feature-${issueId.toLowerCase()}`) : void 0;
 		const notes = status.testNotes;
 		let feedbackPath;
 		try {
-			const { writeFeedbackFile } = await import("./feedback-writer-BvHE1I-3.js");
+			const { writeFeedbackFile } = await import("./feedback-writer-thrdxYpK.js");
 			const r = await Effect.runPromise(writeFeedbackFile({
 				issueId,
 				workspacePath: workspace,
@@ -1059,10 +1165,10 @@ async function deliverTestFailureToWorkAgentHostSide(issueId, status) {
 			if (r.success) feedbackPath = r.filePath;
 		} catch {}
 		const message = `SPECIALIST FEEDBACK: test-agent reported FAILED for ${issueId}.\n\n${feedbackPath ? `MUST READ: ${feedbackPath}\n\n` : ""}${notes ? `${notes.slice(0, 400)}\n\n` : ""}Fix the failing tests, commit and push, then re-run pan done ${issueId}. Do NOT stop at the prompt.`;
-		const { resolveIssueFeedbackTarget, surfaceIssueFeedbackNeedsYou } = await import("./feedback-target-DENaINE6.js");
+		const { resolveIssueFeedbackTarget, surfaceIssueFeedbackNeedsYou } = await import("./feedback-target-C4DaDSZ8.js");
 		const target = await resolveIssueFeedbackTarget(issueId);
 		if ("agentId" in target) {
-			const { messageAgent } = await import("./agents-KaEsWZ8D.js");
+			const { messageAgent } = await import("./agents-BIggLvi_.js");
 			await messageAgent(target.agentId, message);
 			console.log(`[review-status] delivered test failure to ${target.agentId} for ${issueId} (host-side)`);
 		} else await surfaceIssueFeedbackNeedsYou(issueId, target.reason, {
@@ -1254,9 +1360,11 @@ var init_review_status = __esmMin((() => {
 	init_pipeline_notifier();
 	init_activity_logger();
 	init_review_status_sync();
+	init_review_status_source();
 	init_review_status_normalize();
 	init_review_status_record_sync();
 	init_review_dispatch_decision();
+	registerReviewStatusMapReader(() => getAllReviewStatusesFromDb());
 	DEFAULT_STATUS_FILE = join(homedir(), ".overdeck", "review-status.json");
 	reviewDispatchAttemptAt = /* @__PURE__ */ new Map();
 	REVIEW_AUTO_DISPATCH_THROTTLE_MS = 3e4;
@@ -1291,6 +1399,6 @@ var init_review_status = __esmMin((() => {
 	});
 }));
 //#endregion
-export { stateToOverdeckParamsForDb as A, init_record_lock as C, init_agent_state_sync as D, getOverdeckAgentStateSync as E, listOverdeckAgentStatesSync as O, needsReviewDispatch as S, AGENT_COLUMNS_FOR_DB as T, setDeaconIgnored as _, fixStuckReadyForMerge as a, verificationSatisfied as b, init_review_status as c, loadReviewStatusesForIssues as d, markWorkspaceStuck as f, setAutoMerge as g, reviewGatesPassedSync as h, fixStuckCommentedReviews as i, saveOverdeckAgentStateSync as k, loadReadyForMergeFlags as l, resetPipelineVerdictsForWorkStartSync as m, clearReviewStatus as n, getReviewStatus as o, mergeGateEligibility as p, clearStuckMergeStatuses as r, getReviewStatusSync as s, ReviewStatusError as t, loadReviewStatuses as u, setReviewStatus as v, withIssueRecordLock as w, init_review_dispatch_decision as x, setReviewStatusSync as y };
+export { listOverdeckAgentStatesSync as A, init_record_lock as C, AGENT_COLUMNS_FOR_DB as D, readReviewStatusMap as E, stateToOverdeckParamsForDb as M, getOverdeckAgentStateSync as O, needsReviewDispatch as S, init_review_status_source as T, setDeaconIgnored as _, fixStuckReadyForMerge as a, verificationSatisfied as b, init_review_status as c, loadReviewStatusesForIssues as d, markWorkspaceStuck as f, setAutoMerge as g, reviewGatesPassedSync as h, fixStuckCommentedReviews as i, saveOverdeckAgentStateSync as j, init_agent_state_sync as k, loadReadyForMergeFlags as l, resetPipelineVerdictsForWorkStartSync as m, clearReviewStatus as n, getReviewStatus as o, mergeGateEligibility as p, clearStuckMergeStatuses as r, getReviewStatusSync as s, ReviewStatusError as t, loadReviewStatuses as u, setReviewStatus as v, withIssueRecordLock as w, init_review_dispatch_decision as x, setReviewStatusSync as y };
 
-//# sourceMappingURL=review-status-DWAsTqfJ.js.map
+//# sourceMappingURL=review-status-BQjqQXZG.js.map

package/server/{review-status-mo3tyc8L.js → review-status-bBkqUj5i.js}

@@ -1,3 +1,3 @@
-import { c as init_review_status, f as markWorkspaceStuck, n as clearReviewStatus, p as mergeGateEligibility, s as getReviewStatusSync, u as loadReviewStatuses, y as setReviewStatusSync } from "./review-status-DWAsTqfJ.js";
+import { c as init_review_status, f as markWorkspaceStuck, n as clearReviewStatus, p as mergeGateEligibility, s as getReviewStatusSync, u as loadReviewStatuses, y as setReviewStatusSync } from "./review-status-BQjqQXZG.js";
 init_review_status();
 export { clearReviewStatus, getReviewStatusSync, loadReviewStatuses, markWorkspaceStuck, mergeGateEligibility, setReviewStatusSync };

package/server/{review-status-enrichment-C6sr9wOI.js → review-status-enrichment-CGlcApSn.js}

@@ -1,5 +1,5 @@
-import { Gn as resolveProjectFromIssueSync, Nn as init_projects } from "./config-yaml-DbdN9Fuq.js";
-import { O as listSessionNames, x as init_tmux } from "./tmux-CBYZ5lKF.js";
+import { Gn as resolveProjectFromIssueSync, Nn as init_projects } from "./config-yaml-DPAieL_G.js";
+import { O as listSessionNames, x as init_tmux } from "./tmux-ePNhMIMU.js";
 import { Data, Effect } from "effect";
 import { existsSync } from "fs";
 import { join } from "path";
@@ -124,4 +124,4 @@ const enrichReviewStatus = (issueId, status) => Effect.tryPromise({
 //#endregion
 export { enrichReviewStatusFromSessions as n, enrichReviewStatus as t };
 
-//# sourceMappingURL=review-status-enrichment-C6sr9wOI.js.map
+//# sourceMappingURL=review-status-enrichment-CGlcApSn.js.map

package/server/{review-status-sync-D-p7TQit.js → review-status-sync-BWHnrq-k.js}

@@ -1,10 +1,10 @@
 import { n as __esmMin } from "./rolldown-runtime-DNCzzKnI.js";
-import { d as getOverdeckDatabaseSync, f as init_infra } from "./infra-CCTzpcuJ.js";
+import { d as getOverdeckDatabaseSync, f as init_infra } from "./infra-CpKlYZA9.js";
 import { Effect } from "effect";
 //#region ../../lib/review-status-normalize.ts
 function normalizeReviewStatusSync(status) {
 	const shouldClearMergeNotes = status.mergeStatus === "verifying" || status.mergeStatus === "merged";
-	const shouldClearReadyForMerge = status.mergeStatus === "merged" || status.reviewStatus !== "passed" || status.testStatus !== "passed" && status.testStatus !== "skipped" || status.uatStatus !== void 0 && status.uatStatus !== "passed" || (status.blockerReasons?.length ?? 0) > 0;
+	const shouldClearReadyForMerge = status.mergeStatus === "merged" || status.reviewStatus !== "passed" && status.reviewStatus !== "skipped" || status.testStatus !== "passed" && status.testStatus !== "skipped" || status.uatStatus !== void 0 && status.uatStatus !== "passed" || (status.blockerReasons?.length ?? 0) > 0;
 	return {
 		...status,
 		...shouldClearMergeNotes ? { mergeNotes: void 0 } : {},
@@ -271,4 +271,4 @@ var init_review_status_sync = __esmMin((() => {
 //#endregion
 export { getMergeBlockerReconcileCandidatesSync as a, init_review_status_sync as c, init_review_status_normalize as d, normalizeReviewStatusSync as f, getMergeBlockerReconcileCandidates as i, markWorkspaceStuck as l, deleteReviewStatus as n, getReviewStatusFromDbSync as o, getAllReviewStatusesFromDb as r, getReviewStatusesFromDb as s, clearWorkspaceStuck as t, upsertReviewStatusSync as u };
 
-//# sourceMappingURL=review-status-sync-D-p7TQit.js.map
+//# sourceMappingURL=review-status-sync-BWHnrq-k.js.map

package/server/{review-status-sync-MvYGQLDD.js → review-status-sync-BxHBVH8s.js}

@@ -1,3 +1,3 @@
-import { c as init_review_status_sync, t as clearWorkspaceStuck } from "./review-status-sync-D-p7TQit.js";
+import { c as init_review_status_sync, t as clearWorkspaceStuck } from "./review-status-sync-BWHnrq-k.js";
 init_review_status_sync();
 export { clearWorkspaceStuck };

package/server/{review-verdict-feedback-BbIfN5Vg.js → review-verdict-feedback-cZh9Wi9G.js}

@@ -1,3 +1,3 @@
-import { n as init_review_verdict_feedback, t as deliverReviewVerdictFeedback } from "./review-verdict-feedback-Cfc-THvz.js";
+import { n as init_review_verdict_feedback, t as deliverReviewVerdictFeedback } from "./review-verdict-feedback-wqnikhMK.js";
 init_review_verdict_feedback();
 export { deliverReviewVerdictFeedback };

package/server/{review-verdict-feedback-Cfc-THvz.js → review-verdict-feedback-wqnikhMK.js}

@@ -1,11 +1,10 @@
 import { n as __esmMin } from "./rolldown-runtime-DNCzzKnI.js";
-import { Gn as resolveProjectFromIssueSync, Nn as init_projects } from "./config-yaml-DbdN9Fuq.js";
-import { L as sessionExists, x as init_tmux } from "./tmux-CBYZ5lKF.js";
-import { c as init_review_status, s as getReviewStatusSync } from "./review-status-DWAsTqfJ.js";
-import { E as resumeAgent, Pn as getAgentStateSync, Tn as clearAgentPausedSync, j as messageAgent, t as init_agents } from "./agents-D_x5BSZM.js";
+import { Gn as resolveProjectFromIssueSync, Nn as init_projects } from "./config-yaml-DPAieL_G.js";
+import { c as init_review_status, s as getReviewStatusSync } from "./review-status-BQjqQXZG.js";
+import { j as messageAgent, t as init_agents } from "./agents-DHvsjsuP.js";
 import { a as PAN_DIRNAME, p as init_types } from "./types-CE1m_daD.js";
-import { a as writeFeedbackFile, i as init_feedback_writer } from "./feedback-writer-Dxg8qM8i.js";
-import { n as resolveIssueFeedbackTarget, r as surfaceIssueFeedbackNeedsYou, t as init_feedback_target } from "./feedback-target-Dv4VnbQl.js";
+import { a as writeFeedbackFile, i as init_feedback_writer } from "./feedback-writer-DMaO0XXa.js";
+import { n as resolveIssueFeedbackTarget, r as surfaceIssueFeedbackNeedsYou, t as init_feedback_target } from "./feedback-target-oE4-Esxl.js";
 import { Effect } from "effect";
 import { existsSync } from "node:fs";
 import { join } from "node:path";
@@ -95,15 +94,20 @@ async function deliverReviewVerdictFeedbackPromise(opts) {
 	if (fileResult.success && fileResult.filePath) {
 		const message = `SPECIALIST FEEDBACK: review-agent reported ${opts.verdict.toUpperCase()} for ${issueId}.\n\nMUST READ: ${fileResult.filePath}\n\nUse your Read tool to open this file, read every line, then fix ALL review findings. Do NOT stop at the prompt.`;
 		try {
-			const target = await resolveIssueFeedbackTarget(issueId, {
-				itemId: opts.slotItemId,
-				revivePipelinePausedAgent
-			});
+			const target = await resolveIssueFeedbackTarget(issueId, { itemId: opts.slotItemId });
 			if ("agentId" in target) try {
 				await messageAgent(target.agentId, message);
 				agentMessageSent = true;
 			} catch (err) {
-				console.log(`[review-verdict-feedback] Could not message ${target.agentId}; feedback file remains available: ${err instanceof Error ? err.message : String(err)}`);
+				const reason = err instanceof Error ? err.message : String(err);
+				console.warn(`[review-verdict-feedback] Could not message ${target.agentId}; feedback file remains available: ${reason}`);
+				try {
+					await surfaceIssueFeedbackNeedsYou(issueId, `Feedback delivery to ${target.agentId} failed: ${reason}`, {
+						specialist: "review-agent",
+						feedbackPath: fileResult.filePath,
+						slotItemId: opts.slotItemId
+					});
+				} catch {}
 			}
 			else try {
 				await surfaceIssueFeedbackNeedsYou(issueId, target.reason, {
@@ -125,36 +129,12 @@ async function deliverReviewVerdictFeedbackPromise(opts) {
 		agentMessageSent
 	};
 }
-/**
-* PAN-2461: unpause + resume an agent that the PIPELINE paused (pausedReason
-* starts with "needs-you:") so feedback can be delivered to it. Operator pauses
-* are left untouched.
-*/
-async function revivePipelinePausedAgent(agentId, issueId) {
-	try {
-		const state = getAgentStateSync(agentId);
-		if (!state) return false;
-		if (state.paused !== true || !state.pausedReason?.startsWith("needs-you:")) return false;
-		console.log(`[review-verdict-feedback] ${agentId} is pipeline-paused (${state.pausedReason}) — unpausing to deliver feedback for ${issueId}`);
-		clearAgentPausedSync(agentId);
-		const result = await resumeAgent(agentId);
-		if (!result.success) {
-			console.warn(`[review-verdict-feedback] Failed to revive ${agentId}: ${result.error}`);
-			return false;
-		}
-		return Effect.runPromise(sessionExists(agentId));
-	} catch (err) {
-		console.warn(`[review-verdict-feedback] revivePipelinePausedAgent(${agentId}) failed: ${err instanceof Error ? err.message : String(err)}`);
-		return false;
-	}
-}
 var execFileAsync, PR_COMMENT_TIMEOUT_MS, deliverReviewVerdictFeedback;
 var init_review_verdict_feedback = __esmMin((() => {
 	init_agents();
 	init_projects();
 	init_review_status();
 	init_types();
-	init_tmux();
 	init_feedback_writer();
 	init_feedback_target();
 	execFileAsync = promisify(execFile);
@@ -164,4 +144,4 @@ var init_review_verdict_feedback = __esmMin((() => {
 //#endregion
 export { init_review_verdict_feedback as n, postPrComment as r, deliverReviewVerdictFeedback as t };
 
-//# sourceMappingURL=review-verdict-feedback-Cfc-THvz.js.map
+//# sourceMappingURL=review-verdict-feedback-wqnikhMK.js.map

package/server/{rollup-DgWlrSBc.js → rollup-BJE-Vc7H.js}

@@ -1,8 +1,8 @@
 import { n as __esmMin } from "./rolldown-runtime-DNCzzKnI.js";
-import { S as MemoryStatus, t as init_src } from "./src-AXK4mjue.js";
-import { a as resolveArchiveDir, f as resolveStatusFile, i as init_paths, l as resolveObservationsFile, n as ensureDir, r as ensureParentDir, u as resolvePendingDir } from "./paths-Bo-kA9ta.js";
-import { i as init_settings, n as getMemoryRollupPendingThreshold } from "./settings-DZq_bi-B.js";
-import { i as updateMemoryHealth, n as extractWithProviderPolicy, r as init_health, t as init_providers } from "./providers-uPI9uU7Q.js";
+import { S as MemoryStatus, t as init_src } from "./src-fgbU-jJ-.js";
+import { a as resolveArchiveDir, f as resolveStatusFile, i as init_paths, l as resolveObservationsFile, n as ensureDir, r as ensureParentDir, u as resolvePendingDir } from "./paths-TE1GOZDr.js";
+import { i as init_settings, n as getMemoryRollupPendingThreshold } from "./settings-D2wt4nku.js";
+import { i as updateMemoryHealth, n as extractWithProviderPolicy, r as init_health, t as init_providers } from "./providers-4-xOzHQu.js";
 import { Result, Schema } from "effect";
 import { dirname } from "path";
 import { randomUUID } from "crypto";
@@ -98,8 +98,8 @@ async function enqueueStatusRollupEvent(job) {
 		await configuredStatusRollupProcessor(job);
 		return;
 	}
-	const { initEventStore } = await import("./event-store-BeEE8pW5.js");
-	const { synthesizeStatusRollup, commitStatusRollup } = await import("./rollup-CxR6GMTe.js");
+	const { initEventStore } = await import("./event-store-DIk9wvnS.js");
+	const { synthesizeStatusRollup, commitStatusRollup } = await import("./rollup-DBNy1yFB.js");
 	await (await initEventStore()).appendAsync({
 		type: "memory.rollup_triggered",
 		timestamp: (/* @__PURE__ */ new Date()).toISOString(),
@@ -295,7 +295,7 @@ async function clearPendingTurns(projectId, issueId, pendingTurns) {
 	return cleared;
 }
 async function emitStatusUpdatedEvent(event) {
-	const { initEventStore } = await import("./event-store-BeEE8pW5.js");
+	const { initEventStore } = await import("./event-store-DIk9wvnS.js");
 	await (await initEventStore()).appendAsync({
 		type: "memory.status_updated",
 		timestamp: (/* @__PURE__ */ new Date()).toISOString(),
@@ -446,4 +446,4 @@ var init_rollup = __esmMin((() => {
 //#endregion
 export { readCurrentStatus as a, init_pending as c, readArchivedStatuses as i, maybeTriggerStatusRollup as l, commitStatusRollup as n, readRecentObservations as o, init_rollup as r, synthesizeStatusRollup as s, buildStatusRollupPrompt as t, writePendingTurn as u };
 
-//# sourceMappingURL=rollup-DgWlrSBc.js.map
+//# sourceMappingURL=rollup-BJE-Vc7H.js.map

package/server/{rollup-CxR6GMTe.js → rollup-DBNy1yFB.js}

@@ -1,3 +1,3 @@
-import { n as commitStatusRollup, r as init_rollup, s as synthesizeStatusRollup } from "./rollup-DgWlrSBc.js";
+import { n as commitStatusRollup, r as init_rollup, s as synthesizeStatusRollup } from "./rollup-BJE-Vc7H.js";
 init_rollup();
 export { commitStatusRollup, synthesizeStatusRollup };