patchrelay 0.17.0 → 0.17.1

package/dist/agent-session-plan.js

@@ -93,7 +93,6 @@ export function buildAgentSessionPlan(params) {
         case "implementing":
             return setStatuses(planForRunType("implementation", params), ["completed", "inProgress", "pending", "pending"]);
         case "pr_open":
-        case "awaiting_review":
             return setStatuses(implementationPlan(), ["completed", "completed", "inProgress", "pending"]);
         case "changes_requested":
             return setStatuses(reviewFixPlan(), ["completed", "inProgress", "pending", "pending"]);
@@ -164,7 +163,7 @@ export function buildCompletedSessionPlan(runType) {
     if (runType === "ci_repair" || runType === "queue_repair") {
         return buildAgentSessionPlan({ factoryState: "awaiting_queue" });
     }
-    return buildAgentSessionPlan({ factoryState: "awaiting_review" });
+    return buildAgentSessionPlan({ factoryState: "pr_open" });
 }
 export function buildAwaitingHandoffSessionPlan(runType) {
     return buildCompletedSessionPlan(runType);

package/dist/build-info.json

@@ -1,6 +1,6 @@
 {
   "service": "patchrelay",
-  "version": "0.17.0",
-  "commit": "972d51f6e0da",
-  "builtAt": "2026-03-25T12:55:22.635Z"
+  "version": "0.17.1",
+  "commit": "ff6a8d2fcbef",
+  "builtAt": "2026-03-25T14:45:25.511Z"
 }

package/dist/cli/watch/IssueRow.js

@@ -5,7 +5,6 @@ const STATE_COLORS = {
     preparing: "blue",
     implementing: "yellow",
     pr_open: "cyan",
-    awaiting_review: "cyan",
     changes_requested: "magenta",
     repairing_ci: "magenta",
     awaiting_queue: "green",

package/dist/db/migrations.js

@@ -133,6 +133,10 @@ export function runPatchRelayMigrations(connection) {
     addColumnIfMissing(connection, "issues", "pending_merge_prep", "INTEGER NOT NULL DEFAULT 0");
     // Add merge_prep_attempts for retry budget / escalation
     addColumnIfMissing(connection, "issues", "merge_prep_attempts", "INTEGER NOT NULL DEFAULT 0");
+    // Add review_fix_attempts counter
+    addColumnIfMissing(connection, "issues", "review_fix_attempts", "INTEGER NOT NULL DEFAULT 0");
+    // Collapse awaiting_review into pr_open (state normalization)
+    connection.prepare("UPDATE issues SET factory_state = 'pr_open' WHERE factory_state = 'awaiting_review'").run();
 }
 function addColumnIfMissing(connection, table, column, definition) {
     const cols = connection.prepare(`PRAGMA table_info(${table})`).all();

package/dist/db.js

@@ -149,6 +149,10 @@ export class PatchRelayDatabase {
                 sets.push("queue_repair_attempts = @queueRepairAttempts");
                 values.queueRepairAttempts = params.queueRepairAttempts;
             }
+            if (params.reviewFixAttempts !== undefined) {
+                sets.push("review_fix_attempts = @reviewFixAttempts");
+                values.reviewFixAttempts = params.reviewFixAttempts;
+            }
             if (params.mergePrepAttempts !== undefined) {
                 sets.push("merge_prep_attempts = @mergePrepAttempts");
                 values.mergePrepAttempts = params.mergePrepAttempts;
@@ -389,6 +393,7 @@ function mapIssueRow(row) {
         ...(row.pr_check_status !== null && row.pr_check_status !== undefined ? { prCheckStatus: String(row.pr_check_status) } : {}),
         ciRepairAttempts: Number(row.ci_repair_attempts ?? 0),
         queueRepairAttempts: Number(row.queue_repair_attempts ?? 0),
+        reviewFixAttempts: Number(row.review_fix_attempts ?? 0),
         mergePrepAttempts: Number(row.merge_prep_attempts ?? 0),
         pendingMergePrep: Boolean(row.pending_merge_prep),
     };

package/dist/factory-state.js

@@ -5,10 +5,12 @@ export const ACTIVE_RUN_STATES = new Set([
     "changes_requested",
     "repairing_queue",
 ]);
-/** Which factory states are terminal (no further transitions possible). */
+/** Which factory states are terminal (no further transitions possible except pr_merged → done). */
 export const TERMINAL_STATES = new Set([
     "done",
     "escalated",
+    "failed",
+    "awaiting_input",
 ]);
 // ─── Semantic guards ─────────────────────────────────────────────
 //

package/dist/github-webhook-handler.js

@@ -158,8 +158,11 @@ export class GitHubWebhookHandler {
                 }
             }
         }
-        // Reset repair counters on new push
-        if (event.triggerEvent === "pr_synchronize") {
+        // Re-read issue after all upserts so reactive run logic sees current state
+        const freshIssue = this.db.getIssue(issue.projectId, issue.linearIssueId) ?? issue;
+        // Reset repair counters on new push — but only when no repair run is active,
+        // since Codex pushes during repair and resetting mid-run would bypass budgets.
+        if (event.triggerEvent === "pr_synchronize" && !freshIssue.activeRunId) {
             this.db.upsertIssue({
                 projectId: issue.projectId,
                 linearIssueId: issue.linearIssueId,
@@ -167,8 +170,6 @@ export class GitHubWebhookHandler {
                 queueRepairAttempts: 0,
             });
         }
-        // Re-read issue after all upserts so reactive run logic sees current state
-        const freshIssue = this.db.getIssue(issue.projectId, issue.linearIssueId) ?? issue;
         this.logger.info({ issueKey: issue.issueKey, branchName: event.branchName, triggerEvent: event.triggerEvent, prNumber: event.prNumber }, "GitHub webhook: updated issue PR state");
         this.feed?.publish({
             level: event.triggerEvent.includes("failed") ? "warn" : "info",

package/dist/linear-session-reporting.js

@@ -13,7 +13,6 @@ function describeNextState(state, prNumber) {
     const prLabel = prNumber ? `PR #${prNumber}` : "the pull request";
     switch (state) {
         case "pr_open":
-        case "awaiting_review":
             return `${prLabel} is ready for review.`;
         case "awaiting_queue":
             return `${prLabel} is approved and back in the merge flow.`;
@@ -149,7 +148,6 @@ export function buildMergePrepEscalationActivity(attempts) {
 }
 export function summarizeIssueStateForLinear(issue) {
     switch (issue.factoryState) {
-        case "awaiting_review":
         case "pr_open":
             return issue.prNumber ? `PR #${issue.prNumber} is awaiting review.` : "Awaiting review.";
         case "awaiting_queue":

package/dist/merge-queue.js

@@ -103,7 +103,6 @@ export class MergeQueue {
                 pendingRunType: "queue_repair",
                 pendingRunContextJson: JSON.stringify({ failureReason: "merge_conflict" }),
                 pendingMergePrep: false,
-                mergePrepAttempts: 0,
             });
             this.enqueueIssue(issue.projectId, issue.linearIssueId);
             this.feed?.publish({

package/dist/run-orchestrator.js

@@ -9,8 +9,9 @@ import { buildAgentSessionExternalUrls } from "./agent-session-presentation.js";
 import { WorktreeManager } from "./worktree-manager.js";
 import { resolveAuthoritativeLinearStopState } from "./linear-workflow.js";
 import { execCommand } from "./utils.js";
-const DEFAULT_CI_REPAIR_BUDGET = 2;
-const DEFAULT_QUEUE_REPAIR_BUDGET = 2;
+const DEFAULT_CI_REPAIR_BUDGET = 3;
+const DEFAULT_QUEUE_REPAIR_BUDGET = 3;
+const DEFAULT_REVIEW_FIX_BUDGET = 3;
 function slugify(value) {
     return value.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "").slice(0, 60);
 }
@@ -114,6 +115,10 @@ export class RunOrchestrator {
             this.escalate(issue, runType, `Queue repair budget exhausted (${DEFAULT_QUEUE_REPAIR_BUDGET} attempts)`);
             return;
         }
+        if (runType === "review_fix" && issue.reviewFixAttempts >= DEFAULT_REVIEW_FIX_BUDGET) {
+            this.escalate(issue, runType, `Review fix budget exhausted (${DEFAULT_REVIEW_FIX_BUDGET} attempts)`);
+            return;
+        }
         // Increment repair counters
         if (runType === "ci_repair") {
             this.db.upsertIssue({ projectId: issue.projectId, linearIssueId: issue.linearIssueId, ciRepairAttempts: issue.ciRepairAttempts + 1 });
@@ -121,6 +126,9 @@ export class RunOrchestrator {
         if (runType === "queue_repair") {
             this.db.upsertIssue({ projectId: issue.projectId, linearIssueId: issue.linearIssueId, queueRepairAttempts: issue.queueRepairAttempts + 1 });
         }
+        if (runType === "review_fix") {
+            this.db.upsertIssue({ projectId: issue.projectId, linearIssueId: issue.linearIssueId, reviewFixAttempts: issue.reviewFixAttempts + 1 });
+        }
         // Build prompt
         const prompt = buildRunPrompt(issue, runType, project.repoPath, context);
         // Resolve workspace
@@ -296,22 +304,9 @@ export class RunOrchestrator {
         // Complete the run
         const trackedIssue = this.db.issueToTrackedIssue(issue);
         const report = buildStageReport(run, trackedIssue, thread, countEventMethods(this.db.listThreadEvents(run.id)));
-        // Determine post-run state. When a re-run finds the PR already exists
-        // and makes no changes, no pr_opened webhook arrives — the state would
-        // stay in the active-run state forever. Advance based on PR metadata.
+        // Determine post-run state based on current PR metadata.
         const freshIssue = this.db.getIssue(run.projectId, run.linearIssueId) ?? issue;
-        let postRunState;
-        if (ACTIVE_RUN_STATES.has(freshIssue.factoryState) && freshIssue.prNumber) {
-            if (freshIssue.prReviewState === "approved") {
-                postRunState = "awaiting_queue";
-            }
-            else if (freshIssue.prState === "merged") {
-                postRunState = "done";
-            }
-            else {
-                postRunState = "awaiting_review";
-            }
-        }
+        const postRunState = resolvePostRunState(freshIssue);
         this.db.transaction(() => {
             this.db.finishRun(run.id, {
                 status: "completed",
@@ -544,6 +539,8 @@ export class RunOrchestrator {
         if (latestTurn?.status === "completed") {
             const trackedIssue = this.db.issueToTrackedIssue(issue);
             const report = buildStageReport(run, trackedIssue, thread, countEventMethods(this.db.listThreadEvents(run.id)));
+            const freshIssue = this.db.getIssue(run.projectId, run.linearIssueId) ?? issue;
+            const postRunState = resolvePostRunState(freshIssue);
             this.db.transaction(() => {
                 this.db.finishRun(run.id, {
                     status: "completed",
@@ -556,8 +553,13 @@ export class RunOrchestrator {
                     projectId: run.projectId,
                     linearIssueId: run.linearIssueId,
                     activeRunId: null,
+                    ...(postRunState ? { factoryState: postRunState } : {}),
+                    ...(postRunState === "awaiting_queue" ? { pendingMergePrep: true } : {}),
                 });
             });
+            if (postRunState === "awaiting_queue") {
+                this.enqueueIssue(run.projectId, run.linearIssueId);
+            }
         }
     }
     // ─── Internal helpers ─────────────────────────────────────────────
@@ -660,3 +662,17 @@ export class RunOrchestrator {
         throw new Error(`Failed to read thread ${threadId}`);
     }
 }
+/**
+ * Determine post-run factory state from current PR metadata.
+ * Used by both the normal completion path and reconciliation.
+ */
+function resolvePostRunState(issue) {
+    if (ACTIVE_RUN_STATES.has(issue.factoryState) && issue.prNumber) {
+        if (issue.prReviewState === "approved")
+            return "awaiting_queue";
+        if (issue.prState === "merged")
+            return "done";
+        return "pr_open";
+    }
+    return undefined;
+}

package/package.json

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