@ravshansbox/pi-next-issue 1.0.4 → 1.0.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ravshansbox/pi-next-issue",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "description": "Pi extension that works through GitHub issues in the background",
   "license": "MIT",
   "type": "module",

package/src/github.ts

@@ -1,6 +1,6 @@
 import { setTimeout as sleep } from 'node:timers/promises';
 import { must, run } from './exec.ts';
-import { LABELS, STOP_LABELS, type ChecksResult, type Issue } from './runner.ts';
+import { LABELS, STOP_LABELS, type ChecksResult, type Issue, type MergeResult } from './runner.ts';
 
 export type Repo = { root: string; slug: string };
 
@@ -247,21 +247,31 @@ export async function merge(
   pr: number,
   timeoutMs: number,
   signal: AbortSignal,
-): Promise<boolean> {
+): Promise<MergeResult> {
+  const before = await gh(repo, ['pr', 'view', String(pr), '--json', 'state', '--jq', '.state']);
+  if (before === 'MERGED') {
+    return { merged: true };
+  }
   const result = await run('gh', ['pr', 'merge', String(pr), '--squash', '--repo', repo.slug], {
     cwd: repo.root,
   });
   if (result.code !== 0) {
-    return false;
+    return {
+      merged: false,
+      reason: result.stderr.trim().slice(-2000) || `gh pr merge failed with code ${result.code}`,
+    };
   }
   const deadline = Date.now() + timeoutMs;
   for (;;) {
     const state = await gh(repo, ['pr', 'view', String(pr), '--json', 'state', '--jq', '.state']);
     if (state === 'MERGED') {
-      return true;
+      return { merged: true };
+    }
+    if (state === 'CLOSED') {
+      return { merged: false, reason: 'the pull request was closed' };
     }
-    if (state === 'CLOSED' || Date.now() >= deadline) {
-      return false;
+    if (Date.now() >= deadline) {
+      return { merged: false, reason: 'the merge did not finish in time' };
     }
     await sleep(Math.min(POLL_MS, deadline - Date.now()), undefined, { signal });
   }

package/src/runner.ts

@@ -52,7 +52,7 @@ export type Ports = {
   checks(pr: number, cwd: string, signal: AbortSignal): Promise<ChecksResult>;
   failedLogs(pr: number): Promise<string>;
   commentOnPr(pr: number, body: string): Promise<void>;
-  merge(pr: number, signal: AbortSignal): Promise<boolean>;
+  merge(pr: number, signal: AbortSignal): Promise<MergeResult>;
   cleanup(issue: number, branch: string, cwd: string): Promise<void>;
   work(task: AgentTask): Promise<string>;
   review(task: AgentTask): Promise<Verdict | undefined>;
@@ -64,6 +64,8 @@ export type Ports = {
   unlock(): Promise<void>;
 };
 
+export type MergeResult = { merged: true } | { merged: false; reason: string };
+
 export type Outcome = 'merged' | 'needs-human';
 
 export type Status = 'idle' | 'running' | 'stopping' | 'finished' | 'stopped' | 'failed';
@@ -330,8 +332,9 @@ export class Runner {
         }
         case 'merge': {
           this.stage('merge');
-          if (!(await ports.merge(pr(), this.controller.signal))) {
-            return needsHuman(`pull request #${pr()} could not be merged`);
+          const result = await ports.merge(pr(), this.controller.signal);
+          if (!result.merged) {
+            return needsHuman(`pull request #${pr()} could not be merged: ${result.reason}`);
           }
           await ports.setLabel(number, LABELS.done, [LABELS.inProgress]);
           await ports.cleanup(number, state.branch, cwd);