patchrelay 0.30.0 → 0.31.0

package/dist/service.js

@@ -1,4 +1,5 @@
 import { resolveGitHubAppCredentials, createGitHubAppTokenManager, ensureGhWrapper, } from "./github-app-token.js";
+import { parseGitHubFailureContext, summarizeGitHubFailureContext } from "./github-failure-context.js";
 import { GitHubWebhookHandler } from "./github-webhook-handler.js";
 import { IssueQueryService } from "./issue-query-service.js";
 import { DatabaseBackedLinearClientProvider } from "./linear-client.js";
@@ -216,6 +217,10 @@ export class PatchRelayService {
           i.current_linear_state, i.factory_state, i.updated_at,
           i.pending_run_type,
           i.pr_number, i.pr_review_state, i.pr_check_status,
+          i.last_github_failure_source,
+          i.last_github_failure_head_sha,
+          i.last_github_failure_check_name,
+          i.last_github_failure_context_json,
           active_run.run_type AS active_run_type,
           latest_run.run_type AS latest_run_type,
           latest_run.status AS latest_run_status,
@@ -229,7 +234,10 @@ export class PatchRelayService {
              AND blockers.linear_issue_id = d.blocker_linear_issue_id
             WHERE d.project_id = i.project_id
               AND d.linear_issue_id = i.linear_issue_id
-              AND LOWER(TRIM(COALESCE(blockers.current_linear_state, d.blocker_current_linear_state, ''))) != 'done'
+              AND (
+                COALESCE(blockers.current_linear_state_type, d.blocker_current_linear_state_type, '') != 'completed'
+                AND LOWER(TRIM(COALESCE(blockers.current_linear_state, d.blocker_current_linear_state, ''))) != 'done'
+              )
           ) AS blocked_by_count,
           (
             SELECT json_group_array(COALESCE(blockers.issue_key, d.blocker_issue_key, d.blocker_linear_issue_id))
@@ -239,7 +247,10 @@ export class PatchRelayService {
              AND blockers.linear_issue_id = d.blocker_linear_issue_id
             WHERE d.project_id = i.project_id
               AND d.linear_issue_id = i.linear_issue_id
-              AND LOWER(TRIM(COALESCE(blockers.current_linear_state, d.blocker_current_linear_state, ''))) != 'done'
+              AND (
+                COALESCE(blockers.current_linear_state_type, d.blocker_current_linear_state_type, '') != 'completed'
+                AND LOWER(TRIM(COALESCE(blockers.current_linear_state, d.blocker_current_linear_state, ''))) != 'done'
+              )
           ) AS blocked_by_keys_json
         FROM issues i
         LEFT JOIN runs active_run ON active_run.id = i.active_run_id
@@ -251,13 +262,22 @@ export class PatchRelayService {
         ORDER BY i.updated_at DESC, i.issue_key ASC`)
             .all();
         return rows.map((row) => {
+            const failureContext = parseGitHubFailureContext(typeof row.last_github_failure_context_json === "string" ? row.last_github_failure_context_json : undefined);
             const statusNote = extractStatusNote(typeof row.latest_run_summary_json === "string" ? row.latest_run_summary_json : undefined, typeof row.latest_run_report_json === "string" ? row.latest_run_report_json : undefined);
             const blockedByKeys = parseStringArray(typeof row.blocked_by_keys_json === "string" ? row.blocked_by_keys_json : undefined);
             const blockedByCount = Number(row.blocked_by_count ?? 0);
             const readyForExecution = row.pending_run_type !== null && row.pending_run_type !== undefined && row.active_run_type === null && blockedByCount === 0;
+            const failureSummary = summarizeGitHubFailureContext(failureContext);
+            const derivedStatusNote = blockedByCount > 0
+                ? `Blocked by ${blockedByKeys.join(", ")}`
+                : failureSummary && (row.factory_state === "repairing_ci"
+                    || row.factory_state === "repairing_queue"
+                    || row.factory_state === "failed")
+                    ? failureSummary
+                    : statusNote;
             const statusNoteWithBlockers = blockedByCount > 0
                 ? `Blocked by ${blockedByKeys.join(", ")}`
-                : statusNote;
+                : derivedStatusNote;
             return {
                 ...(row.issue_key !== null ? { issueKey: String(row.issue_key) } : {}),
                 ...(row.title !== null ? { title: String(row.title) } : {}),
@@ -275,6 +295,11 @@ export class PatchRelayService {
                 ...(row.pr_number !== null ? { prNumber: Number(row.pr_number) } : {}),
                 ...(row.pr_review_state !== null ? { prReviewState: String(row.pr_review_state) } : {}),
                 ...(row.pr_check_status !== null ? { prCheckStatus: String(row.pr_check_status) } : {}),
+                ...(row.last_github_failure_source !== null ? { latestFailureSource: String(row.last_github_failure_source) } : {}),
+                ...(row.last_github_failure_head_sha !== null ? { latestFailureHeadSha: String(row.last_github_failure_head_sha) } : {}),
+                ...(row.last_github_failure_check_name !== null ? { latestFailureCheckName: String(row.last_github_failure_check_name) } : {}),
+                ...(failureContext?.stepName ? { latestFailureStepName: failureContext.stepName } : {}),
+                ...(failureContext?.summary ? { latestFailureSummary: failureContext.summary } : {}),
                 updatedAt: String(row.updated_at),
             };
         });

package/dist/webhook-handler.js

@@ -180,6 +180,7 @@ export class WebhookHandler {
             ...(hydratedIssue.priority != null ? { priority: hydratedIssue.priority } : {}),
             ...(hydratedIssue.estimate != null ? { estimate: hydratedIssue.estimate } : {}),
             ...(hydratedIssue.stateName ? { currentLinearState: hydratedIssue.stateName } : {}),
+            ...(hydratedIssue.stateType ? { currentLinearStateType: hydratedIssue.stateType } : {}),
             ...(pendingRunType ? { pendingRunType, factoryState: "delegated" } : {}),
             ...(clearPendingImplementation ? { pendingRunType: null } : {}),
             ...((pendingRunType || existingIssue?.pendingRunType === "implementation") && pendingRunContextJson
@@ -203,27 +204,30 @@ export class WebhookHandler {
     }
     async syncIssueDependencies(projectId, issue) {
         let source = issue;
-        if (source.blockedBy.length === 0 && source.blocks.length === 0) {
+        if (!source.relationsKnown) {
             const linear = await this.linearProvider.forProject(projectId);
             if (linear) {
                 try {
                     source = mergeIssueMetadata(source, await linear.getIssue(issue.id));
                 }
                 catch {
-                    // Fall back to webhook payload data when live hydration is unavailable.
+                    // Preserve existing dependency rows when webhook relation data is incomplete.
                 }
             }
         }
-        this.db.replaceIssueDependencies({
-            projectId,
-            linearIssueId: source.id,
-            blockers: source.blockedBy.map((blocker) => ({
-                blockerLinearIssueId: blocker.id,
-                ...(blocker.identifier ? { blockerIssueKey: blocker.identifier } : {}),
-                ...(blocker.title ? { blockerTitle: blocker.title } : {}),
-                ...(blocker.stateName ? { blockerCurrentLinearState: blocker.stateName } : {}),
-            })),
-        });
+        if (source.relationsKnown) {
+            this.db.replaceIssueDependencies({
+                projectId,
+                linearIssueId: source.id,
+                blockers: source.blockedBy.map((blocker) => ({
+                    blockerLinearIssueId: blocker.id,
+                    ...(blocker.identifier ? { blockerIssueKey: blocker.identifier } : {}),
+                    ...(blocker.title ? { blockerTitle: blocker.title } : {}),
+                    ...(blocker.stateName ? { blockerCurrentLinearState: blocker.stateName } : {}),
+                    ...(blocker.stateType ? { blockerCurrentLinearStateType: blocker.stateType } : {}),
+                })),
+            });
+        }
         return source;
     }
     reconcileDependentReadiness(projectId, blockerLinearIssueId) {
@@ -559,10 +563,12 @@ function mergeIssueMetadata(issue, liveIssue) {
         ...(issue.teamKey ? {} : liveIssue.teamKey ? { teamKey: liveIssue.teamKey } : {}),
         ...(issue.stateId ? {} : liveIssue.stateId ? { stateId: liveIssue.stateId } : {}),
         ...(issue.stateName ? {} : liveIssue.stateName ? { stateName: liveIssue.stateName } : {}),
+        ...(issue.stateType ? {} : liveIssue.stateType ? { stateType: liveIssue.stateType } : {}),
         ...(issue.delegateId ? {} : liveIssue.delegateId ? { delegateId: liveIssue.delegateId } : {}),
         ...(issue.delegateName ? {} : liveIssue.delegateName ? { delegateName: liveIssue.delegateName } : {}),
+        relationsKnown: issue.relationsKnown || liveIssue.blockedBy !== undefined || liveIssue.blocks !== undefined,
         labelNames: issue.labelNames.length > 0 ? issue.labelNames : (liveIssue.labels ?? []).map((l) => l.name),
-        blockedBy: issue.blockedBy.length > 0 ? issue.blockedBy : (liveIssue.blockedBy ?? []),
-        blocks: issue.blocks.length > 0 ? issue.blocks : (liveIssue.blocks ?? []),
+        blockedBy: issue.relationsKnown ? issue.blockedBy : (liveIssue.blockedBy ?? issue.blockedBy),
+        blocks: issue.relationsKnown ? issue.blocks : (liveIssue.blocks ?? issue.blocks),
     };
 }

package/dist/webhooks.js

@@ -227,6 +227,7 @@ function extractIssueMetadata(payload) {
         ...(delegateName ? { delegateName } : {}),
         ...(priority != null ? { priority } : {}),
         ...(estimate != null ? { estimate } : {}),
+        relationsKnown: false,
         labelNames: extractLabelNames(issueRecord),
         blockedBy: [],
         blocks: [],

package/package.json

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