patchrelay 0.61.0 → 0.63.0

package/dist/build-info.json

@@ -1,6 +1,6 @@
 {
   "service": "patchrelay",
-  "version": "0.61.0",
-  "commit": "d6c22120c20c",
-  "builtAt": "2026-05-04T22:20:57.540Z"
+  "version": "0.63.0",
+  "commit": "e265b8a2a40d",
+  "builtAt": "2026-05-04T22:32:17.463Z"
 }

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/merge-queue-protocol.js

@@ -1,6 +1,13 @@
 export const DEFAULT_MERGE_QUEUE_LABEL = "queue";
 export const DEFAULT_MERGE_QUEUE_CHECK_NAME = "merge-steward/queue";
 export const DEFAULT_PRIORITY_QUEUE_LABEL = "queue:priority";
+// Plan §2.4 defaults. Wired here so the resolver is the single source
+// of truth for bus-contract values; consumers (review-quill subscribe,
+// Linear-status sync, etc.) never read `project.github.*` directly.
+export const DEFAULT_SPEC_READY_CHECK_NAME = "merge-steward/spec-ready";
+export const DEFAULT_SPEC_BRANCH_PATTERN = "mq-spec-*";
+export const DEFAULT_NO_CACHE_LABEL = "review:no-cache";
+export const DEFAULT_QUEUED_FOR_DEPLOY_LABEL = "queued-for-deploy";
 export function resolveMergeQueueProtocol(project) {
     return {
         repoFullName: project?.github?.repoFullName,
@@ -8,5 +15,9 @@ export function resolveMergeQueueProtocol(project) {
         admissionLabel: project?.github?.mergeQueueLabel ?? DEFAULT_MERGE_QUEUE_LABEL,
         evictionCheckName: project?.github?.mergeQueueCheckName ?? DEFAULT_MERGE_QUEUE_CHECK_NAME,
         priorityLabel: project?.github?.priorityQueueLabel ?? DEFAULT_PRIORITY_QUEUE_LABEL,
+        specReadyCheckName: project?.github?.specReadyCheckName ?? DEFAULT_SPEC_READY_CHECK_NAME,
+        specBranchPattern: project?.github?.specBranchPattern ?? DEFAULT_SPEC_BRANCH_PATTERN,
+        noCacheLabel: project?.github?.noCacheLabel ?? DEFAULT_NO_CACHE_LABEL,
+        queuedForDeployLabel: project?.github?.queuedForDeployLabel ?? DEFAULT_QUEUED_FOR_DEPLOY_LABEL,
     };
 }

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.61.0",
+  "version": "0.63.0",
   "license": "MIT",
   "type": "module",
   "repository": {