patchrelay 0.60.0 → 0.62.0

package/dist/build-info.json

@@ -1,6 +1,6 @@
 {
   "service": "patchrelay",
-  "version": "0.60.0",
-  "commit": "116496455e0c",
-  "builtAt": "2026-05-04T22:16:11.428Z"
+  "version": "0.62.0",
+  "commit": "8ec3fef80b54",
+  "builtAt": "2026-05-04T22:29:13.205Z"
 }

package/dist/cli/cluster-health.js

@@ -342,6 +342,21 @@ async function evaluateGitHubIssueHealth(snapshot, config, runCommand, reviewQui
         && issue.factoryState !== "awaiting_queue"
         && issue.activeRunId === undefined
         && ageMs >= RECONCILIATION_GRACE_MS) {
+        // Plan §6.1: when the PR is also approved, this is the
+        // "In Review · stuck at admission" condition — the lander would
+        // accept the verdict but branch CI is red and (post-§4.3) we no
+        // longer auto-repair. Keep the same scope/status pair so existing
+        // dashboards continue to surface it; just sharpen the message.
+        if (reviewDecision === "APPROVED") {
+            return {
+                ciEntry,
+                finding: {
+                    status: "fail",
+                    scope: "github:ci",
+                    message: "In Review · stuck at admission — PR is approved but gate CI is red and no CI repair is running",
+                },
+            };
+        }
         return {
             ciEntry,
             finding: {

package/dist/db/issue-store.js

@@ -402,6 +402,22 @@ export class IssueStore {
             .all();
         return rows.map(mapIssueRow);
     }
+    // Issues that are approved by review-quill but stuck in In Review
+    // because branch CI is failing — the merge-steward never admits them.
+    // Plan §6.2: surface this as IN_REVIEW_STUCK so an operator notices
+    // before the issue goes silent for hours.
+    listApprovedRedCiIssues() {
+        const rows = this.connection
+            .prepare(`SELECT * FROM issues
+         WHERE factory_state = 'pr_open'
+         AND active_run_id IS NULL
+         AND pending_run_type IS NULL
+         AND pr_number IS NOT NULL
+         AND pr_review_state = 'approved'
+         AND pr_check_status = 'failure'`)
+            .all();
+        return rows.map(mapIssueRow);
+    }
     replaceIssueDependencies(params) {
         const now = isoNow();
         this.connection

package/dist/prompting/patchrelay.js

@@ -271,7 +271,7 @@ function buildStructuredReviewContext(context) {
         lines.push("No inline review comments were captured for this review.");
         return lines.join("\n");
     }
-    lines.push(`Inline review comments captured: ${reviewComments.length}`, "Resolve each comment below or verify it is already fixed on the current head before you stop.", "A requested-changes turn is only complete if you push a newer PR head or deliberately escalate because you are blocked.", "");
+    lines.push(`Inline review comments captured: ${reviewComments.length}`, "Resolve each comment below or verify it is already fixed on the current head before you stop.", "Complete the turn either by pushing a newer PR head with the fix, or — if your reviewer-pass produces only comments, test wording, or PR-body changes — by editing the PR body via `gh pr edit` instead of pushing. Do not push a commit that produces a patch-id-equivalent diff just to make the fix unmistakable.", "If you are blocked, deliberately escalate instead of pushing.", "");
     for (const comment of reviewComments) {
         const location = comment.path
             ? `${comment.path}${comment.line !== undefined ? `:${comment.line}` : ""}${comment.side ? ` (${comment.side})` : ""}`
@@ -293,12 +293,12 @@ function buildRequestedChangesContext(runType, context) {
     const mode = resolveRequestedChangesMode(runType, context);
     const lines = [];
     if (mode === "branch_upkeep") {
-        lines.push("Branch upkeep is required on the existing PR branch.", "Goal: restore merge readiness on the current branch and push a newer head without regressing review or CI readiness.");
+        lines.push("Branch upkeep is required on the existing PR branch.", "Goal: restore merge readiness on the current branch. Push a newer head only when the work actually changes the diff against the base; do not republish a patch-id-equivalent head.");
     }
     else {
         const reviewer = typeof context?.reviewerName === "string" ? context.reviewerName : undefined;
         const reviewBody = typeof context?.reviewBody === "string" ? context.reviewBody.trim() : "";
-        lines.push("Requested changes on the existing PR branch.", "Goal: restore review readiness and push a newer head on the current PR branch.", "Address the real concern behind the feedback and verify nearby invariants in the touched flow before you publish.", reviewer ? `Reviewer: ${reviewer}` : "", reviewBody ? `Review summary:\n${reviewBody}` : "");
+        lines.push("Requested changes on the existing PR branch.", "Goal: restore review readiness on the current PR branch. Push a newer head only when the fix actually changes the diff; if the reviewer-pass produces only comments, test wording, or PR-body changes, edit the PR body via `gh pr edit` instead.", "Address the real concern behind the feedback and verify nearby invariants in the touched flow before you publish.", reviewer ? `Reviewer: ${reviewer}` : "", reviewBody ? `Review summary:\n${reviewBody}` : "");
         appendStructuredReviewContext(lines, context);
     }
     return lines.join("\n").trim();

package/dist/queue-health-monitor.js

@@ -2,6 +2,11 @@ import { resolveMergeQueueProtocol } from "./merge-queue-protocol.js";
 import { execCommand } from "./utils.js";
 const QUEUE_HEALTH_GRACE_MS = 120_000;
 const QUEUE_HEALTH_PROBE_FAILURE_COOLDOWN_MS = 300_000;
+// Plan §6.2: an approved PR with red branch CI for >= this long is
+// stuck at admission — operator notice is needed before the issue
+// goes silent for hours.
+const IN_REVIEW_STUCK_THRESHOLD_MS = 30 * 60 * 1000;
+const IN_REVIEW_STUCK_FEED_COOLDOWN_MS = 30 * 60 * 1000;
 function isDuplicateProbe(issue, context) {
     const signature = typeof context?.failureSignature === "string" ? context.failureSignature : undefined;
     const headSha = typeof context?.failureHeadSha === "string" ? context.failureHeadSha : undefined;
@@ -17,6 +22,7 @@ export class QueueHealthMonitor {
     logger;
     feed;
     probeFailureFeedTimes = new Map();
+    inReviewStuckFeedTimes = new Map();
     constructor(db, config, advancer, logger, feed) {
         this.db = db;
         this.config = config;
@@ -28,6 +34,42 @@ export class QueueHealthMonitor {
         for (const issue of this.db.issues.listAwaitingQueueIssues()) {
             await this.probeQueuedIssue(issue);
         }
+        for (const issue of this.db.issues.listApprovedRedCiIssues()) {
+            this.probeInReviewStuckIssue(issue);
+        }
+    }
+    // Plan §6.2: emit IN_REVIEW_STUCK when an approved PR has a red gate
+    // for more than 30 minutes. Consequence of plan §4.3 — branch CI
+    // failures while approved no longer trigger ci_repair, so the
+    // condition is otherwise invisible to the operator.
+    probeInReviewStuckIssue(issue) {
+        if (!issue.prNumber)
+            return;
+        const project = this.config.projects.find((p) => p.id === issue.projectId);
+        if (!project)
+            return;
+        const reference = issue.lastGitHubFailureAt ?? issue.updatedAt;
+        const stuckMs = Date.now() - Date.parse(reference);
+        if (stuckMs < IN_REVIEW_STUCK_THRESHOLD_MS)
+            return;
+        const feedKey = `${issue.projectId}::${issue.linearIssueId}`;
+        const lastFedAt = this.inReviewStuckFeedTimes.get(feedKey) ?? 0;
+        if (Date.now() - lastFedAt < IN_REVIEW_STUCK_FEED_COOLDOWN_MS)
+            return;
+        this.inReviewStuckFeedTimes.set(feedKey, Date.now());
+        const minutes = Math.round(stuckMs / 60_000);
+        const failedCheck = issue.lastGitHubFailureCheckName ?? "branch CI";
+        this.logger.warn({ issueKey: issue.issueKey, prNumber: issue.prNumber, stuckMs, failedCheck }, "Queue health: approved PR is stuck at admission with red branch CI");
+        this.feed?.publish({
+            level: "warn",
+            kind: "github",
+            issueKey: issue.issueKey,
+            projectId: issue.projectId,
+            stage: "pr_open",
+            status: "in_review_stuck",
+            summary: `In Review · stuck at admission — PR #${issue.prNumber} has been approved with red ${failedCheck} for ${minutes} min`,
+            detail: issue.lastGitHubFailureCheckUrl ?? undefined,
+        });
     }
     async probeQueuedIssue(issue) {
         if (!issue.prNumber)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "patchrelay",
-  "version": "0.60.0",
+  "version": "0.62.0",
   "license": "MIT",
   "type": "module",
   "repository": {