patchrelay 0.36.2 → 0.36.3

package/dist/build-info.json

@@ -1,6 +1,6 @@
 {
   "service": "patchrelay",
-  "version": "0.36.2",
-  "commit": "45b497cfbf8e",
-  "builtAt": "2026-04-08T23:32:38.896Z"
+  "version": "0.36.3",
+  "commit": "e132192dca38",
+  "builtAt": "2026-04-08T23:42:01.882Z"
 }

package/dist/cli/cluster-health.js

@@ -327,6 +327,21 @@ async function evaluateGitHubIssueHealth(snapshot, config, runCommand, reviewQui
             },
         };
     }
+    if (gateCheckStatus === "success"
+        && reviewDecision === "CHANGES_REQUESTED"
+        && mergeConflictDetected
+        && issue.factoryState !== "changes_requested"
+        && issue.activeRunId === undefined
+        && ageMs >= RECONCILIATION_GRACE_MS) {
+        return {
+            ciEntry,
+            finding: {
+                status: "fail",
+                scope: "github:branch-upkeep",
+                message: "PR is still dirty after requested changes, but no branch-upkeep run is active",
+            },
+        };
+    }
     if (gateCheckStatus === "success"
         && reviewDecision === "CHANGES_REQUESTED"
         && latestBlockingReviewHeadSha === pr.headRefOid
@@ -423,6 +438,9 @@ function deriveCiOwner(params) {
             : "downstream";
     }
     if (params.reviewDecision === "CHANGES_REQUESTED") {
+        if (params.mergeConflictDetected) {
+            return params.factoryState === "changes_requested" ? "patchrelay" : "unknown";
+        }
         if (params.factoryState === "changes_requested")
             return "patchrelay";
         if (params.reviewQuillAttempt)
@@ -451,6 +469,9 @@ function describeCiOwnership(params) {
         && params.latestBlockingReviewHeadSha
         && params.currentHeadSha !== params.latestBlockingReviewHeadSha);
     if (params.owner === "patchrelay") {
+        if (params.mergeConflictDetected) {
+            return "PatchRelay owns the next branch-upkeep move";
+        }
         return params.gateCheckStatus === "failure"
             ? "PatchRelay owns the next CI repair move"
             : "PatchRelay owns the next requested-changes move";
@@ -479,6 +500,11 @@ function describeCiOwnership(params) {
             : "Waiting on external GitHub automation";
     }
     if (params.reviewDecision === "CHANGES_REQUESTED") {
+        if (params.mergeConflictDetected) {
+            return headAdvancedPastBlockingReview
+                ? "PR is still dirty after a newer pushed head and no branch-upkeep run is active"
+                : "PR is still dirty on the current blocked head and no branch-upkeep run is active";
+        }
         return blockingReviewTargetsCurrentHead
             ? "Requested changes still block the same head and no fix run is active"
             : "Waiting on review after a newer pushed head";

package/dist/idle-reconciliation.js

@@ -16,6 +16,22 @@ function isReviewDecisionChangesRequested(value) {
 function isReviewDecisionReviewRequired(value) {
     return value?.trim().toUpperCase() === "REVIEW_REQUIRED";
 }
+function buildBranchUpkeepContext(prNumber, baseBranch, mergeStateStatus, headSha) {
+    const promptContext = [
+        `The requested code change may already be present, but GitHub still reports PR #${prNumber} as ${mergeStateStatus ?? "DIRTY"} 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 {
+        branchUpkeepRequired: true,
+        reviewFixMode: "branch_upkeep",
+        wakeReason: "branch_upkeep",
+        promptContext,
+        ...(mergeStateStatus ? { mergeStateStatus } : {}),
+        ...(headSha ? { failingHeadSha: headSha } : {}),
+        baseBranch,
+    };
+}
 function hasCompletedReviewQuillVerdict(entries) {
     return (entries ?? []).some((entry) => entry.__typename === "CheckRun"
         && entry.name === "review-quill/verdict"
@@ -475,6 +491,23 @@ export class IdleIssueReconciler {
                 mergeConflictDetected,
                 downstreamOwned,
             });
+            if (reactiveIntent?.runType === "branch_upkeep" && mergeConflictDetected) {
+                this.logger.info({ issueKey: issue.issueKey, prNumber: issue.prNumber, mergeable: pr.mergeable, mergeStateStatus: pr.mergeStateStatus }, "Reconciliation: PR still needs branch upkeep after requested changes");
+                this.advanceIdleIssue(issue, reactiveIntent.compatibilityFactoryState, {
+                    pendingRunType: reactiveIntent.runType,
+                    pendingRunContext: buildBranchUpkeepContext(issue.prNumber, project.github?.baseBranch ?? "main", pr.mergeStateStatus, pr.headRefOid),
+                });
+                this.feed?.publish({
+                    level: "warn",
+                    kind: "github",
+                    issueKey: issue.issueKey,
+                    projectId: issue.projectId,
+                    stage: reactiveIntent.compatibilityFactoryState,
+                    status: "branch_upkeep_queued",
+                    summary: `PR #${issue.prNumber} is still dirty after requested changes, dispatching branch upkeep`,
+                });
+                return;
+            }
             if (reactiveIntent?.runType === "queue_repair" && mergeConflictDetected) {
                 this.logger.info({ issueKey: issue.issueKey, prNumber: issue.prNumber, mergeable: pr.mergeable }, "Reconciliation: PR needs queue repair from fresh GitHub truth");
                 this.advanceIdleIssue(issue, reactiveIntent.compatibilityFactoryState, {
@@ -510,7 +543,7 @@ export class IdleIssueReconciler {
                 return;
             }
             if (mergeConflictDetected) {
-                this.logger.debug({ issueKey: issue.issueKey, prNumber: issue.prNumber, mergeable: pr.mergeable, mergeStateStatus: pr.mergeStateStatus }, "Reconciliation: PR is dirty but not yet queue-admitted; leaving PatchRelay in review state");
+                this.logger.debug({ issueKey: issue.issueKey, prNumber: issue.prNumber, mergeable: pr.mergeable, mergeStateStatus: pr.mergeStateStatus }, "Reconciliation: PR is dirty but no automation owner was derived");
             }
         }
         catch (error) {

package/dist/issue-session.js

@@ -59,6 +59,13 @@ export function deriveIssueSessionReactiveIntent(params) {
         };
     }
     if (params.prReviewState === "changes_requested") {
+        if (params.mergeConflictDetected) {
+            return {
+                runType: "branch_upkeep",
+                wakeReason: "branch_upkeep",
+                compatibilityFactoryState: "changes_requested",
+            };
+        }
         return {
             runType: "review_fix",
             wakeReason: "review_changes_requested",

package/package.json

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