@link-assistant/hive-mind 1.23.6 → 1.23.7

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # @link-assistant/hive-mind
 
+## 1.23.7
+
+### Patch Changes
+
+- d951635: Fix --auto-restart-until-mergable false positive on empty CI checks
+
+  The `--auto-restart-until-mergable` mode was incorrectly posting "Ready to merge" when CI checks hadn't started yet. This was caused by JavaScript's vacuous truth: `[].every(fn)` returns `true`, so an empty checks array would pass all validation.
+
+  Fix: Return `pending` status when no CI checks exist yet, instead of `success`.
+
 ## 1.23.6
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "1.23.6",
+  "version": "1.23.7",
   "description": "AI-powered issue solver and hive mind for collaborative problem solving",
   "main": "src/hive.mjs",
   "type": "module",

package/src/github-merge.lib.mjs

@@ -340,6 +340,22 @@ export async function checkPRCIStatus(owner, repo, prNumber, verbose = false) {
       })),
     ];
 
+    // Issue #1304: If no checks exist yet, treat as pending
+    // This handles the race condition where CI hasn't started yet after a commit is pushed.
+    // An empty array would otherwise pass all checks due to JavaScript's vacuous truth
+    // ([].every(fn) returns true for any fn).
+    if (allChecks.length === 0) {
+      if (verbose) {
+        console.log(`[VERBOSE] /merge: PR #${prNumber} has no CI checks yet - treating as pending`);
+      }
+      return {
+        status: 'pending',
+        checks: [],
+        allPassed: false,
+        hasPending: true,
+      };
+    }
+
     const hasPending = allChecks.some(c => c.status !== 'completed' || c.conclusion === null);
     const allPassed = !hasPending && allChecks.every(c => c.conclusion === 'success' || c.conclusion === 'skipped' || c.conclusion === 'neutral');
     const hasFailed = allChecks.some(c => c.conclusion === 'failure' || c.conclusion === 'cancelled' || c.conclusion === 'timed_out');