patchrelay 0.12.4 → 0.12.6

package/dist/build-info.json

@@ -1,6 +1,6 @@
 {
   "service": "patchrelay",
-  "version": "0.12.4",
-  "commit": "dea4ce43848b",
-  "builtAt": "2026-03-24T18:27:01.637Z"
+  "version": "0.12.6",
+  "commit": "62c29649ea46",
+  "builtAt": "2026-03-24T21:48:34.334Z"
 }

package/dist/db.js

@@ -260,7 +260,7 @@ export class PatchRelayDatabase {
     }
     listRunningRuns() {
         const rows = this.connection
-            .prepare("SELECT * FROM runs WHERE status = 'running'")
+            .prepare("SELECT * FROM runs WHERE status IN ('running', 'queued')")
             .all();
         return rows.map(mapRunRow);
     }

package/dist/github-webhook-handler.js

@@ -226,32 +226,32 @@ export class GitHubWebhookHandler {
     async emitLinearActivity(issue, newState, event) {
         if (!issue.agentSessionId)
             return;
-        const linear = await this.linearProvider.forProject(issue.projectId);
-        if (!linear?.createAgentActivity)
-            return;
-        const messages = {
-            pr_open: `PR #${event.prNumber ?? ""} opened.${event.prUrl ? ` ${event.prUrl}` : ""}`,
-            awaiting_queue: "PR approved. Preparing merge.",
-            changes_requested: `Review requested changes.${event.reviewerName ? ` Reviewer: ${event.reviewerName}` : ""}`,
-            repairing_ci: `CI check failed${event.checkName ? `: ${event.checkName}` : ""}. Starting repair.`,
-            repairing_queue: "Merge conflict with base branch. Starting repair.",
-            done: `PR merged and deployed.${event.prNumber ? ` PR #${event.prNumber}` : ""}`,
-            failed: "PR was closed without merging.",
-        };
-        const body = messages[newState];
-        if (!body)
-            return;
-        const type = newState === "failed" || newState === "repairing_ci" || newState === "repairing_queue"
-            ? "error"
-            : "response";
         try {
+            const linear = await this.linearProvider.forProject(issue.projectId);
+            if (!linear?.createAgentActivity)
+                return;
+            const messages = {
+                pr_open: `PR #${event.prNumber ?? ""} opened.${event.prUrl ? ` ${event.prUrl}` : ""}`,
+                awaiting_queue: "PR approved. Preparing merge.",
+                changes_requested: `Review requested changes.${event.reviewerName ? ` Reviewer: ${event.reviewerName}` : ""}`,
+                repairing_ci: `CI check failed${event.checkName ? `: ${event.checkName}` : ""}. Starting repair.`,
+                repairing_queue: "Merge conflict with base branch. Starting repair.",
+                done: `PR merged and deployed.${event.prNumber ? ` PR #${event.prNumber}` : ""}`,
+                failed: "PR was closed without merging.",
+            };
+            const body = messages[newState];
+            if (!body)
+                return;
+            const type = newState === "failed" || newState === "repairing_ci" || newState === "repairing_queue"
+                ? "error"
+                : "response";
             await linear.createAgentActivity({
                 agentSessionId: issue.agentSessionId,
                 content: { type, body },
             });
         }
         catch {
-            // Non-blocking — don't fail the webhook for a Linear activity error
+            // Non-blocking — don't crash the webhook handler for a Linear activity error
         }
     }
 }

package/dist/run-orchestrator.js

@@ -1,5 +1,6 @@
 import { existsSync, readFileSync } from "node:fs";
 import path from "node:path";
+import { ACTIVE_RUN_STATES } from "./factory-state.js";
 import { buildHookEnv, runProjectHook } from "./hook-runner.js";
 import { buildRunningSessionPlan, buildCompletedSessionPlan, buildFailedSessionPlan, } from "./agent-session-plan.js";
 import { buildStageReport, countEventMethods, extractTurnId, resolveRunCompletionStatus, summarizeCurrentThread, } from "./run-reporting.js";
@@ -275,6 +276,22 @@ 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.
+        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 = "pr_open";
+            }
+        }
         this.db.transaction(() => {
             this.db.finishRun(run.id, {
                 status: "completed",
@@ -287,8 +304,14 @@ export class RunOrchestrator {
                 projectId: run.projectId,
                 linearIssueId: run.linearIssueId,
                 activeRunId: null,
+                ...(postRunState ? { factoryState: postRunState } : {}),
+                ...(postRunState === "awaiting_queue" ? { pendingMergePrep: true } : {}),
             });
         });
+        // If we advanced to awaiting_queue, enqueue for merge prep
+        if (postRunState === "awaiting_queue") {
+            this.enqueueIssue(run.projectId, run.linearIssueId);
+        }
         this.feed?.publish({
             level: "info",
             kind: "turn",
@@ -328,24 +351,49 @@ export class RunOrchestrator {
         }
     }
     async reconcileRun(run) {
-        if (!run.threadId) {
-            this.failRunAndClear(run, "Run has no thread ID during reconciliation");
-            return;
-        }
         const issue = this.db.getIssue(run.projectId, run.linearIssueId);
         if (!issue)
             return;
-        // Read Codex state
+        // Zombie run: claimed in DB but Codex never started (no thread).
+        // This happens when the service crashes between claiming the run
+        // and starting the Codex turn. Re-enqueue instead of failing.
+        if (!run.threadId) {
+            this.logger.warn({ issueKey: issue.issueKey, runId: run.id, runType: run.runType }, "Zombie run detected (no thread) — clearing and re-enqueueing");
+            this.db.transaction(() => {
+                this.db.finishRun(run.id, { status: "failed", failureReason: "Zombie: never started (no thread after restart)" });
+                this.db.upsertIssue({
+                    projectId: run.projectId,
+                    linearIssueId: run.linearIssueId,
+                    activeRunId: null,
+                    pendingRunType: run.runType,
+                    pendingRunContextJson: null,
+                });
+            });
+            this.enqueueIssue(run.projectId, run.linearIssueId);
+            return;
+        }
+        // Read Codex state — thread may not exist after app-server restart.
         let thread;
         try {
             thread = await this.readThreadWithRetry(run.threadId);
         }
         catch {
-            this.failRunAndClear(run, "Codex thread not found during reconciliation");
+            this.logger.warn({ issueKey: issue.issueKey, runId: run.id, runType: run.runType, threadId: run.threadId }, "Stale thread during reconciliation — clearing and re-enqueueing");
+            this.db.transaction(() => {
+                this.db.finishRun(run.id, { status: "failed", failureReason: "Stale thread after restart" });
+                this.db.upsertIssue({
+                    projectId: run.projectId,
+                    linearIssueId: run.linearIssueId,
+                    activeRunId: null,
+                    pendingRunType: run.runType,
+                    pendingRunContextJson: null,
+                });
+            });
+            this.enqueueIssue(run.projectId, run.linearIssueId);
             return;
         }
-        // Check Linear state
-        const linear = await this.linearProvider.forProject(run.projectId);
+        // Check Linear state (non-fatal — token refresh may fail)
+        const linear = await this.linearProvider.forProject(run.projectId).catch(() => undefined);
         if (linear) {
             const linearIssue = await linear.getIssue(run.linearIssueId).catch(() => undefined);
             if (linearIssue) {
@@ -429,10 +477,10 @@ export class RunOrchestrator {
     async emitLinearActivity(issue, type, body, options) {
         if (!issue.agentSessionId)
             return;
-        const linear = await this.linearProvider.forProject(issue.projectId);
-        if (!linear)
-            return;
         try {
+            const linear = await this.linearProvider.forProject(issue.projectId);
+            if (!linear)
+                return;
             await linear.createAgentActivity({
                 agentSessionId: issue.agentSessionId,
                 content: { type, body },
@@ -446,10 +494,10 @@ export class RunOrchestrator {
     async updateLinearPlan(issue, plan) {
         if (!issue.agentSessionId)
             return;
-        const linear = await this.linearProvider.forProject(issue.projectId);
-        if (!linear?.updateAgentSession)
-            return;
         try {
+            const linear = await this.linearProvider.forProject(issue.projectId);
+            if (!linear?.updateAgentSession)
+                return;
             await linear.updateAgentSession({ agentSessionId: issue.agentSessionId, plan });
         }
         catch (error) {

package/dist/service-runtime.js

@@ -14,6 +14,7 @@ export class ServiceRuntime {
     webhookQueue;
     issueQueue;
     ready = false;
+    linearConnected = false;
     startupError;
     reconcileTimer;
     reconcileInProgress = false;
@@ -54,10 +55,14 @@ export class ServiceRuntime {
     enqueueIssue(projectId, issueId) {
         this.issueQueue.enqueue({ projectId, issueId });
     }
+    setLinearConnected(connected) {
+        this.linearConnected = connected;
+    }
     getReadiness() {
         return {
-            ready: this.ready && this.codex.isStarted(),
+            ready: this.ready && this.codex.isStarted() && this.linearConnected,
             codexStarted: this.codex.isStarted(),
+            linearConnected: this.linearConnected,
             ...(this.startupError ? { startupError: this.startupError } : {}),
         };
     }

package/dist/service.js

@@ -73,10 +73,37 @@ export class PatchRelayService {
             this.githubAppTokenManager = createGitHubAppTokenManager(ghAppCredentials, logger);
         }
         this.codex.on("notification", (notification) => {
-            void this.orchestrator.handleCodexNotification(notification);
+            this.orchestrator.handleCodexNotification(notification).catch((error) => {
+                const msg = error instanceof Error ? error.message : String(error);
+                logger.error({ method: notification.method, error: msg }, "Unhandled error in Codex notification handler");
+            });
         });
     }
     async start() {
+        // Verify Linear connectivity for all configured projects before starting.
+        // Auth errors do not prevent startup (the OAuth callback must be reachable
+        // for `patchrelay connect`), but the service reports NOT READY until at
+        // least one project has a working Linear token.
+        let anyLinearConnected = false;
+        for (const project of this.config.projects) {
+            try {
+                const client = await this.linearProvider.forProject(project.id);
+                if (client) {
+                    anyLinearConnected = true;
+                }
+                else {
+                    this.logger.warn({ projectId: project.id }, "No Linear installation linked — run 'patchrelay connect' to authorize");
+                }
+            }
+            catch (error) {
+                const msg = error instanceof Error ? error.message : String(error);
+                this.logger.error({ projectId: project.id, error: msg }, "Linear auth failed — run 'patchrelay connect' to refresh the token. Runs for this project will fail until re-authorized.");
+            }
+        }
+        this.runtime.setLinearConnected(anyLinearConnected);
+        if (!anyLinearConnected && this.config.projects.length > 0) {
+            this.logger.error("No projects have working Linear auth — service is NOT READY. Run 'patchrelay connect' to authorize.");
+        }
         if (this.githubAppTokenManager) {
             await ensureGhWrapper(this.logger);
             await this.githubAppTokenManager.start();
@@ -96,7 +123,11 @@ export class PatchRelayService {
         return await this.oauthService.createStart(params);
     }
     async completeLinearOAuth(params) {
-        return await this.oauthService.complete(params);
+        const result = await this.oauthService.complete(params);
+        // A successful OAuth completion means at least one project now has
+        // working Linear auth — update readiness.
+        this.runtime.setLinearConnected(true);
+        return result;
     }
     getLinearOAuthStateStatus(state) {
         return this.oauthService.getStateStatus(state);

package/package.json

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