patchrelay 0.38.0 → 0.38.2

package/dist/linear-progress-reporter.js

@@ -1,185 +1,11 @@
-import { sanitizeOperatorFacingCommand, sanitizeOperatorFacingText } from "./presentation-text.js";
-const PROGRESS_THROTTLE_MS = 5_000;
-const MAX_PROGRESS_TEXT_LENGTH = 220;
 export class LinearProgressReporter {
-    db;
-    emitActivity;
-    progressThrottle = new Map();
-    workingOnPublishedRuns = new Set();
-    agentMessageBuffers = new Map();
-    agentMessageProgressPublished = new Set();
-    constructor(db, emitActivity) {
-        this.db = db;
-        this.emitActivity = emitActivity;
+    constructor(_db, _emitActivity) { }
+    maybeEmitProgress(_notification, _run) {
+        // Keep routine Codex progress in local/operator surfaces rather than
+        // turning every planning or reasoning update into Linear thread chatter.
+        return;
     }
-    maybeEmitProgress(notification, run) {
-        const issue = this.db.issues.getIssue(run.projectId, run.linearIssueId);
-        if (!issue)
-            return;
-        const agentSentence = this.consumeAgentMessageSentence(notification, run);
-        const workingOn = resolveWorkingOnActivity(notification, agentSentence?.sentence);
-        if (workingOn && !this.workingOnPublishedRuns.has(run.id)) {
-            this.workingOnPublishedRuns.add(run.id);
-            void this.emitActivity(issue, workingOn);
-        }
-        const progress = resolveEphemeralProgressActivity(notification, agentSentence?.sentence);
-        if (!progress)
-            return;
-        if (!progress.bypassThrottle) {
-            const now = Date.now();
-            const lastEmit = this.progressThrottle.get(run.id) ?? 0;
-            if (now - lastEmit < PROGRESS_THROTTLE_MS)
-                return;
-            this.progressThrottle.set(run.id, now);
-        }
-        void this.emitActivity(issue, progress.activity, { ephemeral: true });
+    clearProgress(_runId) {
+        return;
     }
-    clearProgress(runId) {
-        this.progressThrottle.delete(runId);
-        this.workingOnPublishedRuns.delete(runId);
-        for (const key of this.agentMessageBuffers.keys()) {
-            if (key.startsWith(`${runId}:`)) {
-                this.agentMessageBuffers.delete(key);
-            }
-        }
-        for (const key of this.agentMessageProgressPublished) {
-            if (key.startsWith(`${runId}:`)) {
-                this.agentMessageProgressPublished.delete(key);
-            }
-        }
-    }
-    consumeAgentMessageSentence(notification, run) {
-        const messageKey = resolveAgentMessageKey(notification, run);
-        if (!messageKey)
-            return undefined;
-        if (this.agentMessageProgressPublished.has(messageKey))
-            return undefined;
-        const delta = resolveAgentMessageDelta(notification);
-        if (delta) {
-            const previous = this.agentMessageBuffers.get(messageKey) ?? "";
-            const next = `${previous}${delta}`;
-            this.agentMessageBuffers.set(messageKey, next);
-            const sentence = extractFirstCompletedSentence(next);
-            if (!sentence)
-                return undefined;
-            this.agentMessageProgressPublished.add(messageKey);
-            return { sentence };
-        }
-        const completedText = resolveCompletedAgentMessageText(notification);
-        if (!completedText)
-            return undefined;
-        const sentence = extractFirstSentence(completedText);
-        if (!sentence)
-            return undefined;
-        this.agentMessageProgressPublished.add(messageKey);
-        return { sentence };
-    }
-}
-function resolveWorkingOnActivity(notification, agentSentence) {
-    const summary = resolveWorkingOnSummary(notification) ?? agentSentence;
-    if (!summary)
-        return undefined;
-    return { type: "response", body: `Working on: ${summary}` };
-}
-function resolveEphemeralProgressActivity(notification, agentSentence) {
-    if (notification.method === "item/started") {
-        const item = notification.params.item;
-        if (!item)
-            return undefined;
-        const type = typeof item.type === "string" ? item.type : undefined;
-        if (type === "commandExecution") {
-            const cmd = item.command;
-            const cmdStr = Array.isArray(cmd)
-                ? sanitizeOperatorFacingCommand(cmd.map((part) => String(part)).join(" "))
-                : sanitizeOperatorFacingCommand(typeof cmd === "string" ? cmd : undefined);
-            return { activity: { type: "action", action: "Running", parameter: truncateProgressText(cmdStr ?? "command", 120) } };
-        }
-        if (type === "mcpToolCall") {
-            const server = typeof item.server === "string" ? item.server : "";
-            const tool = typeof item.tool === "string" ? item.tool : "";
-            return { activity: { type: "action", action: "Using", parameter: `${server}/${tool}` } };
-        }
-        if (type === "dynamicToolCall") {
-            const tool = typeof item.tool === "string" ? item.tool : "tool";
-            return { activity: { type: "action", action: "Using", parameter: tool } };
-        }
-    }
-    if (agentSentence) {
-        return {
-            activity: { type: "thought", body: agentSentence },
-            bypassThrottle: true,
-        };
-    }
-    return undefined;
-}
-function resolveWorkingOnSummary(notification) {
-    if (notification.method !== "turn/plan/updated") {
-        return undefined;
-    }
-    const plan = notification.params.plan;
-    if (!Array.isArray(plan))
-        return undefined;
-    const ranked = plan
-        .map((entry) => entry)
-        .filter((entry) => typeof entry.step === "string" && entry.step.trim().length > 0)
-        .sort((a, b) => rankPlanStatus(a.status) - rankPlanStatus(b.status));
-    const first = ranked[0];
-    return summarizeProgressSentence(typeof first?.step === "string" ? first.step : undefined);
-}
-function rankPlanStatus(status) {
-    return status === "inProgress" ? 0
-        : status === "pending" ? 1
-            : status === "completed" ? 2
-                : 3;
-}
-function resolveAgentMessageKey(notification, run) {
-    if (notification.method === "item/agentMessage/delta") {
-        const itemId = typeof notification.params.itemId === "string" ? notification.params.itemId : undefined;
-        return itemId ? `${run.id}:${itemId}` : undefined;
-    }
-    if (notification.method === "item/completed") {
-        const item = notification.params.item;
-        const itemId = typeof item?.id === "string" ? item.id : undefined;
-        const itemType = typeof item?.type === "string" ? item.type : undefined;
-        return itemId && itemType === "agentMessage" ? `${run.id}:${itemId}` : undefined;
-    }
-    return undefined;
-}
-function resolveAgentMessageDelta(notification) {
-    if (notification.method !== "item/agentMessage/delta") {
-        return undefined;
-    }
-    return typeof notification.params.delta === "string" ? notification.params.delta : undefined;
-}
-function resolveCompletedAgentMessageText(notification) {
-    if (notification.method !== "item/completed") {
-        return undefined;
-    }
-    const item = notification.params.item;
-    if (!item || item.type !== "agentMessage")
-        return undefined;
-    return typeof item.text === "string" ? item.text : undefined;
-}
-function extractFirstSentence(text) {
-    const sanitized = sanitizeOperatorFacingText(text)?.replace(/\s+/g, " ").trim();
-    if (!sanitized)
-        return undefined;
-    const match = sanitized.match(/^(.+?[.!?])(?:\s|$)/);
-    return truncateProgressText((match?.[1] ?? sanitized).trim(), MAX_PROGRESS_TEXT_LENGTH);
-}
-function extractFirstCompletedSentence(text) {
-    const sanitized = sanitizeOperatorFacingText(text)?.replace(/\s+/g, " ").trim();
-    if (!sanitized)
-        return undefined;
-    const match = sanitized.match(/^(.+?[.!?])(?:\s|$)/);
-    return match?.[1] ? truncateProgressText(match[1].trim(), MAX_PROGRESS_TEXT_LENGTH) : undefined;
-}
-function summarizeProgressSentence(text) {
-    const summary = extractFirstSentence(text);
-    if (!summary)
-        return undefined;
-    return summary.endsWith(".") || summary.endsWith("!") || summary.endsWith("?") ? summary : `${summary}.`;
-}
-function truncateProgressText(text, maxLength) {
-    return text.length <= maxLength ? text : `${text.slice(0, maxLength - 3).trimEnd()}...`;
 }

package/dist/linear-status-comment-sync.js

@@ -1,8 +1,7 @@
 import { extractCompletionCheck } from "./completion-check.js";
+import { isClosedPrState } from "./pr-state.js";
 import { deriveIssueStatusNote } from "./status-note.js";
 import { derivePatchRelayWaitingReason } from "./waiting-reason.js";
-import { isClosedPrState } from "./pr-state.js";
-import { isUndelegatedPausedIssue } from "./paused-issue-state.js";
 export async function syncVisibleStatusComment(params) {
     const { db, issue, linear, logger, trackedIssue, options } = params;
     try {
@@ -25,23 +24,8 @@ export async function syncVisibleStatusComment(params) {
         logger.warn({ issueKey: issue.issueKey, error: msg }, "Failed to sync Linear status comment");
     }
 }
-export function shouldSyncVisibleIssueComment(issue, hasAgentSession) {
-    if (!hasAgentSession) {
-        return true;
-    }
-    if (issue.sessionState === "waiting_input" || issue.sessionState === "failed"
-        || issue.factoryState === "awaiting_input" || issue.factoryState === "failed" || issue.factoryState === "escalated") {
-        return true;
-    }
-    if (isUndelegatedPausedIssue(issue)) {
-        return true;
-    }
-    if ((issue.sessionState === "done" || issue.factoryState === "done")
-        && ((issue.prNumber === undefined && !issue.prUrl)
-            || isClosedPrState(issue.prState))) {
-        return true;
-    }
-    return false;
+export function shouldSyncVisibleIssueComment(_issue, _hasAgentSession) {
+    return true;
 }
 function renderStatusComment(db, issue, trackedIssue, options) {
     const activeRun = issue.activeRunId ? db.runs.getRunById(issue.activeRunId) : undefined;

package/dist/manual-issue-actions.js

@@ -0,0 +1,37 @@
+import { hasOpenPr } from "./pr-state.js";
+export function resolveRetryTarget(params) {
+    if (params.prState === "merged") {
+        return { runType: "none", factoryState: "done" };
+    }
+    if (hasOpenPr(params.prNumber, params.prState) && params.lastGitHubFailureSource === "queue_eviction") {
+        return { runType: "queue_repair", factoryState: "repairing_queue" };
+    }
+    if (hasOpenPr(params.prNumber, params.prState)
+        && (params.prCheckStatus === "failed" || params.prCheckStatus === "failure" || params.lastGitHubFailureSource === "branch_ci")) {
+        return { runType: "ci_repair", factoryState: "repairing_ci" };
+    }
+    if (hasOpenPr(params.prNumber, params.prState) && params.prReviewState === "changes_requested") {
+        return {
+            runType: params.pendingRunType === "branch_upkeep" || params.lastRunType === "branch_upkeep"
+                ? "branch_upkeep"
+                : "review_fix",
+            factoryState: "changes_requested",
+        };
+    }
+    if (hasOpenPr(params.prNumber, params.prState)) {
+        return { runType: "implementation", factoryState: "implementing" };
+    }
+    return { runType: "implementation", factoryState: "delegated" };
+}
+export function buildManualRetryAttemptReset(runType) {
+    if (runType === "ci_repair") {
+        return { ciRepairAttempts: 0 };
+    }
+    if (runType === "queue_repair") {
+        return { queueRepairAttempts: 0 };
+    }
+    if (runType === "review_fix" || runType === "branch_upkeep") {
+        return { reviewFixAttempts: 0 };
+    }
+    return {};
+}

package/dist/presentation-text.js

@@ -5,6 +5,16 @@ function unwrapShellWrappedCommand(text) {
         .replace(/^(?:\/bin\/bash|bash|\/bin\/sh|sh)\s+-lc\s+'([^`\n]+)'$/g, "$1")
         .replace(/^(?:\/bin\/bash|bash|\/bin\/sh|sh)\s+-lc\s+"([^`\n]+)"$/g, "$1");
 }
+function stripLocalMarkdownLinks(text) {
+    return text
+        .replace(/\[([^\]]+)\]\(<\/[^)>]+>\)/g, "`$1`")
+        .replace(/\[([^\]]+)\]\((\/[^)\s]+)\)/g, "`$1`");
+}
+function stripLocalAbsolutePaths(text) {
+    return text
+        .replace(/`\/(?:home|Users|private|tmp|var\/folders)\/[^`\n]+`/g, "`local path omitted`")
+        .replace(/(^|[\s(])\/(?:home|Users|private|tmp|var\/folders)\/[^\s)]+/g, "$1`local path omitted`");
+}
 export function sanitizeOperatorFacingCommand(command) {
     const trimmed = command?.trim();
     if (!trimmed) {
@@ -17,5 +27,5 @@ export function sanitizeOperatorFacingText(text) {
     if (!trimmed) {
         return undefined;
     }
-    return unwrapShellWrappedCommand(trimmed);
+    return stripLocalAbsolutePaths(stripLocalMarkdownLinks(unwrapShellWrappedCommand(trimmed)));
 }

package/dist/prompting/patchrelay.js

@@ -184,12 +184,12 @@ function buildRequestedChangesContext(runType, context) {
     const mode = resolveRequestedChangesMode(runType, context);
     const lines = [];
     if (mode === "branch_upkeep") {
-        lines.push("## Branch Upkeep After Requested Changes", "", "The requested review changes may already be addressed, but GitHub still shows the PR branch as behind or dirty against the base branch.", "Update the existing PR branch onto the latest base branch, resolve conflicts carefully, rerun the narrowest relevant verification, and push a newer head.", "Do not open a new PR.", "");
+        lines.push("## Branch Upkeep After Requested Changes", "", "Goal: restore merge readiness on the existing PR branch without regressing review or CI readiness.", "The requested review changes may already be addressed, but GitHub still shows the PR branch as behind or dirty against the base branch.", "Update the existing PR branch onto the latest base branch, resolve conflicts carefully, rerun the narrowest relevant verification, and push a newer head.", "Do not open a new PR.", "", "1. Refresh the latest remote branch and base branch state first.", "2. Rebase or merge onto the latest base branch and resolve conflicts in a way that preserves the branch's current intent and prior fixes.", "3. Audit the conflicted areas for semantic regressions, not just textual conflicts.", "4. Run focused verification for the touched areas and enough surrounding checks to regain confidence that the branch is still review-ready.", "5. Commit and push a newer head on the existing PR branch.", "6. Do not stop at 'conflicts resolved' if the resulting branch is no longer likely to pass review or CI.", "");
     }
     else {
         const reviewer = typeof context?.reviewerName === "string" ? context.reviewerName : undefined;
         const reviewBody = typeof context?.reviewBody === "string" ? context.reviewBody.trim() : "";
-        lines.push("## Review Changes Requested", "", reviewer ? `Reviewer: ${reviewer}` : "", reviewBody ? `Review summary:\n${reviewBody}` : "", "", "1. Start with the structured review context below. Treat the inline review comments as the primary repair checklist for this turn.", "2. Check the current diff (`git diff origin/main`) — a prior rebase may have already resolved some concerns.", "3. For each review point: if already resolved on the current head, note why. If not, fix it.", "4. If the structured review context looks incomplete, inspect the latest GitHub review threads directly before deciding you are done.", "5. Run verification, commit, and push a newer head on the existing PR branch.", "6. Do not try to hand the same head back to review. If you cannot produce a new pushed head, stop and surface the blocker clearly.", "7. GitHub review happens after the new head is pushed and CI is green. Do not use `gh pr edit --add-reviewer` as part of this workflow.", "");
+        lines.push("## Review Changes Requested", "", "Goal: restore review readiness on the existing PR branch, not merely patch the latest cited line.", "Treat the reviewer comments as evidence of what still makes the branch unready. Your job is to return the branch to a state that is likely to pass the next full review.", "", reviewer ? `Reviewer: ${reviewer}` : "", reviewBody ? `Review summary:\n${reviewBody}` : "", "", "1. Start with the structured review context below, then inspect the current diff (`git diff origin/main`) and current code before deciding what still needs work.", "2. Infer the underlying concern or invariant behind the review feedback. Do not assume each comment is an isolated chore.", "3. For each review point: if already resolved on the current head, note why. If not, fix it. Then inspect adjacent code paths and flows that could fail for the same reason.", "4. If the structured review context looks incomplete, inspect the latest GitHub review threads directly before deciding you are done.", "5. Verify the branch as a whole for the relevant concern class: current review issue, nearby regressions, relevant tests, and compatibility with the latest base branch.", "6. Only finish when you believe the branch is review-ready again. If you cannot get it there, stop and surface the blocker clearly.", "7. Commit and push a newer head on the existing PR branch. Do not try to hand the same head back to review.", "8. GitHub review happens after the new head is pushed and CI is green. Do not use `gh pr edit --add-reviewer` as part of this workflow.", "");
         appendStructuredReviewContext(lines, context);
     }
     return lines.join("\n").trim();
@@ -201,7 +201,8 @@ function buildCiRepairContext(context) {
     return [
         "## CI Repair",
         "",
-        "A full CI iteration has settled failed on your PR. Start from the specific failing check/job/step below on the latest remote PR branch tip, fix that concrete failure first, then push to the same PR branch.",
+        "Goal: restore CI readiness on the existing PR branch so the next full CI run is likely to pass.",
+        "A full CI iteration has settled failed on your PR. Start from the specific failing check/job/step below on the latest remote PR branch tip, but do not stop at a narrow patch if the same root cause is likely to fail other checks in the suite.",
         snapshot?.gateCheckName ? `Gate check: ${String(snapshot.gateCheckName)}` : "",
         snapshot?.gateCheckStatus ? `Gate status: ${String(snapshot.gateCheckStatus)}` : "",
         snapshot?.settledAt ? `Settled at: ${String(snapshot.settledAt)}` : "",
@@ -219,10 +220,11 @@ function buildCiRepairContext(context) {
             : "",
         "",
         "Fetch the latest remote branch state first. If the branch moved since this failure, restart from the new tip instead of pushing older work.",
-        "Read the latest logs for the named failing check, fix that root cause, and only broaden scope when the logs show direct fallout from the same issue.",
+        "Read the latest logs for the named failing check, identify the root cause, and check whether that same cause is likely to affect other jobs or checks.",
+        "Fix the root cause, not just the first visible symptom.",
         "Do not change workflows, dependency installation, or unrelated tests unless the failing logs clearly point there.",
-        "Run focused verification for the named failure, then commit and push.",
-        "Do not open a new PR. Keep working on the existing branch until CI goes green or the situation is clearly stuck.",
+        "Run the narrowest local verification that gives real confidence for the suite, then commit and push.",
+        "Do not open a new PR. Keep working on the existing branch until the branch is likely to pass CI again or the situation is clearly stuck.",
         "Do not change test expectations unless the test is genuinely wrong.",
     ].filter(Boolean).join("\n");
 }

package/dist/reactive-pr-state.js

@@ -0,0 +1,65 @@
+import { readRemotePrState } from "./remote-pr-state.js";
+export function isRequestedChangesRunType(runType) {
+    return runType === "review_fix" || runType === "branch_upkeep";
+}
+export function normalizeRemotePrState(value) {
+    const normalized = value?.trim().toUpperCase();
+    if (normalized === "OPEN")
+        return "open";
+    if (normalized === "CLOSED")
+        return "closed";
+    if (normalized === "MERGED")
+        return "merged";
+    return undefined;
+}
+export function normalizeRemoteReviewDecision(value) {
+    const normalized = value?.trim().toUpperCase();
+    if (normalized === "APPROVED")
+        return "approved";
+    if (normalized === "CHANGES_REQUESTED")
+        return "changes_requested";
+    if (normalized === "REVIEW_REQUIRED")
+        return "commented";
+    return undefined;
+}
+export function isDirtyMergeStateStatus(value) {
+    return value?.trim().toUpperCase() === "DIRTY";
+}
+export function buildReviewFixBranchUpkeepContext(prNumber, baseBranch, pr, context) {
+    const promptContext = [
+        `The requested code change may already be present, but GitHub still reports PR #${prNumber} as ${String(pr.mergeStateStatus)} against latest ${baseBranch}.`,
+        `This turn is branch upkeep on the existing PR branch: update onto latest ${baseBranch}, resolve any conflicts, rerun the narrowest relevant verification, and push a newer head.`,
+        "Do not stop just because the requested code change is already present. Review can only move forward after a new pushed head.",
+    ].join(" ");
+    return {
+        ...(context ?? {}),
+        branchUpkeepRequired: true,
+        reviewFixMode: "branch_upkeep",
+        wakeReason: "branch_upkeep",
+        promptContext,
+        ...(pr.mergeStateStatus ? { mergeStateStatus: pr.mergeStateStatus } : {}),
+        ...(pr.headRefOid ? { failingHeadSha: pr.headRefOid } : {}),
+        baseBranch,
+    };
+}
+export async function readReactivePrSnapshot(config, projectId, prNumber) {
+    const project = config.projects.find((entry) => entry.id === projectId);
+    const repoFullName = project?.github?.repoFullName;
+    if (!repoFullName) {
+        return undefined;
+    }
+    const pr = await readRemotePrState(repoFullName, prNumber);
+    if (!pr) {
+        return undefined;
+    }
+    return {
+        projectId,
+        repoFullName,
+        baseBranch: project?.github?.baseBranch ?? "main",
+        pr,
+        prState: normalizeRemotePrState(pr.state),
+        reviewState: normalizeRemoteReviewDecision(pr.reviewDecision),
+        headSha: pr.headRefOid,
+        gateCheckName: project?.gateChecks?.find((entry) => entry.trim())?.trim() ?? "verify",
+    };
+}

package/dist/reactive-run-policy.js

@@ -1,47 +1,4 @@
-import { execCommand } from "./utils.js";
-function isRequestedChangesRunType(runType) {
-    return runType === "review_fix" || runType === "branch_upkeep";
-}
-function normalizeRemotePrState(value) {
-    const normalized = value?.trim().toUpperCase();
-    if (normalized === "OPEN")
-        return "open";
-    if (normalized === "CLOSED")
-        return "closed";
-    if (normalized === "MERGED")
-        return "merged";
-    return undefined;
-}
-function normalizeRemoteReviewDecision(value) {
-    const normalized = value?.trim().toUpperCase();
-    if (normalized === "APPROVED")
-        return "approved";
-    if (normalized === "CHANGES_REQUESTED")
-        return "changes_requested";
-    if (normalized === "REVIEW_REQUIRED")
-        return "commented";
-    return undefined;
-}
-function isDirtyMergeStateStatus(value) {
-    return value?.trim().toUpperCase() === "DIRTY";
-}
-function buildReviewFixBranchUpkeepContext(prNumber, baseBranch, pr, context) {
-    const promptContext = [
-        `The requested code change may already be present, but GitHub still reports PR #${prNumber} as ${String(pr.mergeStateStatus)} against latest ${baseBranch}.`,
-        `This turn is branch upkeep on the existing PR branch: update onto latest ${baseBranch}, resolve any conflicts, rerun the narrowest relevant verification, and push a newer head.`,
-        "Do not stop just because the requested code change is already present. Review can only move forward after a new pushed head.",
-    ].join(" ");
-    return {
-        ...(context ?? {}),
-        branchUpkeepRequired: true,
-        reviewFixMode: "branch_upkeep",
-        wakeReason: "branch_upkeep",
-        promptContext,
-        ...(pr.mergeStateStatus ? { mergeStateStatus: pr.mergeStateStatus } : {}),
-        ...(pr.headRefOid ? { failingHeadSha: pr.headRefOid } : {}),
-        baseBranch,
-    };
-}
+import { buildReviewFixBranchUpkeepContext, isDirtyMergeStateStatus, isRequestedChangesRunType, readReactivePrSnapshot, } from "./reactive-pr-state.js";
 export class ReactiveRunPolicy {
     config;
     db;
@@ -60,15 +17,11 @@ export class ReactiveRunPolicy {
         if (!issue.prNumber || issue.prState !== "open" || !issue.lastGitHubFailureHeadSha) {
             return undefined;
         }
-        const project = this.config.projects.find((entry) => entry.id === run.projectId);
-        if (!project?.github?.repoFullName) {
-            return undefined;
-        }
         try {
-            const pr = await this.loadRemotePrState(project.github.repoFullName, issue.prNumber);
-            if (!pr || pr.state?.toUpperCase() !== "OPEN")
+            const snapshot = await readReactivePrSnapshot(this.config, run.projectId, issue.prNumber);
+            if (!snapshot || snapshot.prState !== "open")
                 return undefined;
-            if (!pr.headRefOid || pr.headRefOid !== issue.lastGitHubFailureHeadSha)
+            if (!snapshot.headSha || snapshot.headSha !== issue.lastGitHubFailureHeadSha)
                 return undefined;
             return `Repair finished but PR #${issue.prNumber} is still on failing head ${issue.lastGitHubFailureHeadSha.slice(0, 8)}`;
         }
@@ -91,18 +44,14 @@ export class ReactiveRunPolicy {
         if (!run.sourceHeadSha) {
             return `Requested-changes run finished for PR #${issue.prNumber} without a recorded starting head SHA. PatchRelay cannot verify that a new head was published.`;
         }
-        const project = this.config.projects.find((entry) => entry.id === run.projectId);
-        if (!project?.github?.repoFullName) {
-            return undefined;
-        }
         try {
-            const pr = await this.loadRemotePrState(project.github.repoFullName, issue.prNumber);
-            if (!pr || pr.state?.toUpperCase() !== "OPEN")
+            const snapshot = await readReactivePrSnapshot(this.config, run.projectId, issue.prNumber);
+            if (!snapshot || snapshot.prState !== "open")
                 return undefined;
-            if (!pr.headRefOid) {
+            if (!snapshot.headSha) {
                 return `Requested-changes run finished for PR #${issue.prNumber} but GitHub did not report a current head SHA.`;
             }
-            if (pr.headRefOid === run.sourceHeadSha) {
+            if (snapshot.headSha === run.sourceHeadSha) {
                 return `Requested-changes run finished for PR #${issue.prNumber} without pushing a new head; PatchRelay must not hand the same SHA back to review.`;
             }
             return undefined;
@@ -123,28 +72,20 @@ export class ReactiveRunPolicy {
         if (!issue.prNumber) {
             return this.db.issues.getIssue(run.projectId, run.linearIssueId) ?? issue;
         }
-        const project = this.config.projects.find((entry) => entry.id === run.projectId);
-        const repoFullName = project?.github?.repoFullName;
-        if (!repoFullName) {
-            return this.db.issues.getIssue(run.projectId, run.linearIssueId) ?? issue;
-        }
         try {
-            const pr = await this.loadRemotePrState(repoFullName, issue.prNumber);
-            if (!pr) {
+            const snapshot = await readReactivePrSnapshot(this.config, run.projectId, issue.prNumber);
+            if (!snapshot) {
                 return this.db.issues.getIssue(run.projectId, run.linearIssueId) ?? issue;
             }
-            const nextPrState = normalizeRemotePrState(pr.state);
-            const nextReviewState = normalizeRemoteReviewDecision(pr.reviewDecision);
-            const gateCheckName = project?.gateChecks?.find((entry) => entry.trim())?.trim() ?? "verify";
-            const headAdvanced = Boolean(pr.headRefOid && pr.headRefOid !== issue.lastGitHubFailureHeadSha);
+            const headAdvanced = Boolean(snapshot.headSha && snapshot.headSha !== issue.lastGitHubFailureHeadSha);
             const reviewFixHeadAdvanced = isRequestedChangesRunType(run.runType)
-                && Boolean(pr.headRefOid && run.sourceHeadSha && pr.headRefOid !== run.sourceHeadSha);
+                && Boolean(snapshot.headSha && run.sourceHeadSha && snapshot.headSha !== run.sourceHeadSha);
             this.upsertIssueIfLeaseHeld(run.projectId, run.linearIssueId, {
                 projectId: run.projectId,
                 linearIssueId: run.linearIssueId,
-                ...(nextPrState ? { prState: nextPrState } : {}),
-                ...(pr.headRefOid ? { prHeadSha: pr.headRefOid } : {}),
-                ...(nextReviewState ? { prReviewState: nextReviewState } : {}),
+                ...(snapshot.prState ? { prState: snapshot.prState } : {}),
+                ...(snapshot.headSha ? { prHeadSha: snapshot.headSha } : {}),
+                ...(snapshot.reviewState ? { prReviewState: snapshot.reviewState } : {}),
                 ...((headAdvanced || reviewFixHeadAdvanced)
                     ? {
                         prCheckStatus: "pending",
@@ -158,8 +99,8 @@ export class ReactiveRunPolicy {
                         lastQueueIncidentJson: null,
                         lastAttemptedFailureHeadSha: null,
                         lastAttemptedFailureSignature: null,
-                        lastGitHubCiSnapshotHeadSha: pr.headRefOid ?? null,
-                        lastGitHubCiSnapshotGateCheckName: gateCheckName,
+                        lastGitHubCiSnapshotHeadSha: snapshot.headSha ?? null,
+                        lastGitHubCiSnapshotGateCheckName: snapshot.gateCheckName,
                         lastGitHubCiSnapshotGateCheckStatus: "pending",
                         lastGitHubCiSnapshotJson: null,
                         lastGitHubCiSnapshotSettledAt: null,
@@ -183,31 +124,24 @@ export class ReactiveRunPolicy {
         if (!issue.prNumber || issue.prState !== "open" || issue.prReviewState !== "changes_requested") {
             return context;
         }
-        const project = this.config.projects.find((entry) => entry.id === issue.projectId);
-        const repoFullName = project?.github?.repoFullName;
-        if (!repoFullName) {
-            return context;
-        }
         try {
-            const pr = await this.loadRemotePrState(repoFullName, issue.prNumber);
-            if (!pr)
+            const snapshot = await readReactivePrSnapshot(this.config, issue.projectId, issue.prNumber);
+            if (!snapshot)
                 return context;
-            const nextPrState = normalizeRemotePrState(pr.state);
-            const nextReviewState = normalizeRemoteReviewDecision(pr.reviewDecision);
             this.upsertIssueIfLeaseHeld(issue.projectId, issue.linearIssueId, {
                 projectId: issue.projectId,
                 linearIssueId: issue.linearIssueId,
-                ...(nextPrState ? { prState: nextPrState } : {}),
-                ...(pr.headRefOid ? { prHeadSha: pr.headRefOid } : {}),
-                ...(nextReviewState ? { prReviewState: nextReviewState } : {}),
+                ...(snapshot.prState ? { prState: snapshot.prState } : {}),
+                ...(snapshot.headSha ? { prHeadSha: snapshot.headSha } : {}),
+                ...(snapshot.reviewState ? { prReviewState: snapshot.reviewState } : {}),
             }, "review-fix wake refresh");
-            if (nextPrState !== "open")
+            if (snapshot.prState !== "open")
                 return context;
-            if (nextReviewState && nextReviewState !== "changes_requested")
+            if (snapshot.reviewState && snapshot.reviewState !== "changes_requested")
                 return context;
-            if (!isDirtyMergeStateStatus(pr.mergeStateStatus))
+            if (!isDirtyMergeStateStatus(snapshot.pr.mergeStateStatus))
                 return context;
-            return buildReviewFixBranchUpkeepContext(issue.prNumber, project?.github?.baseBranch ?? "main", pr, context);
+            return buildReviewFixBranchUpkeepContext(issue.prNumber, snapshot.baseBranch, snapshot.pr, context);
         }
         catch (error) {
             this.logger.debug({
@@ -228,34 +162,27 @@ export class ReactiveRunPolicy {
         if (issue.prReviewState !== "changes_requested") {
             return undefined;
         }
-        const project = this.config.projects.find((entry) => entry.id === run.projectId);
-        const repoFullName = project?.github?.repoFullName;
-        if (!repoFullName) {
-            return undefined;
-        }
         try {
-            const pr = await this.loadRemotePrState(repoFullName, issue.prNumber);
-            if (!pr)
+            const snapshot = await readReactivePrSnapshot(this.config, run.projectId, issue.prNumber);
+            if (!snapshot)
                 return undefined;
-            const nextPrState = normalizeRemotePrState(pr.state);
-            const nextReviewState = normalizeRemoteReviewDecision(pr.reviewDecision);
             this.upsertIssueIfLeaseHeld(issue.projectId, issue.linearIssueId, {
                 projectId: issue.projectId,
                 linearIssueId: issue.linearIssueId,
-                ...(nextPrState ? { prState: nextPrState } : {}),
-                ...(pr.headRefOid ? { prHeadSha: pr.headRefOid } : {}),
-                ...(nextReviewState ? { prReviewState: nextReviewState } : {}),
+                ...(snapshot.prState ? { prState: snapshot.prState } : {}),
+                ...(snapshot.headSha ? { prHeadSha: snapshot.headSha } : {}),
+                ...(snapshot.reviewState ? { prReviewState: snapshot.reviewState } : {}),
             }, "post-run follow-up refresh");
-            if (nextPrState !== "open")
+            if (snapshot.prState !== "open")
                 return undefined;
-            if (nextReviewState && nextReviewState !== "changes_requested")
+            if (snapshot.reviewState && snapshot.reviewState !== "changes_requested")
                 return undefined;
-            if (!isDirtyMergeStateStatus(pr.mergeStateStatus))
+            if (!isDirtyMergeStateStatus(snapshot.pr.mergeStateStatus))
                 return undefined;
             return {
                 pendingRunType: "branch_upkeep",
                 factoryState: "changes_requested",
-                context: buildReviewFixBranchUpkeepContext(issue.prNumber, project?.github?.baseBranch ?? "main", pr),
+                context: buildReviewFixBranchUpkeepContext(issue.prNumber, snapshot.baseBranch, snapshot.pr),
                 summary: `PR #${issue.prNumber} is still dirty after review fix; queued branch upkeep`,
             };
         }
@@ -268,16 +195,6 @@ export class ReactiveRunPolicy {
             return undefined;
         }
     }
-    async loadRemotePrState(repoFullName, prNumber) {
-        const { stdout, exitCode } = await execCommand("gh", [
-            "pr", "view", String(prNumber),
-            "--repo", repoFullName,
-            "--json", "headRefOid,state,reviewDecision,mergeStateStatus",
-        ], { timeoutMs: 10_000 });
-        if (exitCode !== 0)
-            return undefined;
-        return JSON.parse(stdout);
-    }
     upsertIssueIfLeaseHeld(projectId, linearIssueId, params, context) {
         const updated = this.withHeldLease(projectId, linearIssueId, (lease) => this.db.issueSessions.upsertIssueWithLease(lease, params));
         if (updated === undefined) {