@yemi33/minions 0.1.2120 → 0.1.2121

package/engine/shared.js

@@ -2242,6 +2242,7 @@ const ENGINE_DEFAULTS = {
   autoFixHumanComments: true, // auto-dispatch fix agents for actionable human PR comments
   autoConsolidateMemory: false, // opt-in: periodically spawn engine/kb-sweep-runner.js from the tick loop (4h cadence). Inbox→notes consolidation already runs every tick via consolidateInbox; this flag only controls the KB sweep.
   prNoOpFixPauseAttempts: 2, // pause one PR automation cause after repeated no-op fixes for unchanged evidence
+  quarantineAutoRecoveryMax: 2, // #2996 follow-up: cap on auto-flipping WORKTREE_DIRTY/WORKTREE_DIVERGENT failures back to pending (the quarantine is self-healing so the next dispatch starts clean; the cap prevents infinite loops if quarantine itself keeps failing).
   completionReportRetentionDays: 90, // retain completion report sidecars beyond capped dispatch history
   completionReportMaxFiles: 5000, // hard cap for completion report sidecars during cleanup
   // P-bfa2c-cors-wildcard: extra Origins permitted to receive an

package/engine.js

@@ -5807,6 +5807,52 @@ function discoverFromWorkItems(config, project) {
       }
     }
 
+    // #2996 follow-up: auto-recover from WORKTREE_DIRTY / WORKTREE_DIVERGENT quarantines.
+    // The quarantine is self-healing (dir renamed away, ref backed up, branch reset to
+    // origin), so the next dispatch starts from a clean worktree. Auto-flip to pending up
+    // to ENGINE_DEFAULTS.quarantineAutoRecoveryMax times so WIs without downstream deps and
+    // no human watching don't rot in `failed`. _failureClass is stamped on the WI only by
+    // the force-demote path (engine/dispatch.js); the standard non-retryable path leaves
+    // it on the dispatch record but embeds the class name in failReason — so detection
+    // covers both shapes.
+    if (item.status === WI_STATUS.FAILED && !isItemCompleted(item)) {
+      const fr = String(item.failReason || '');
+      const isQuarantineFail = item._failureClass === FAILURE_CLASS.WORKTREE_DIRTY
+        || item._failureClass === FAILURE_CLASS.WORKTREE_DIVERGENT
+        || /\bWORKTREE_DIRTY\b/.test(fr)
+        || /\bWORKTREE_DIVERGENT\b/.test(fr);
+      if (isQuarantineFail) {
+        item._quarantineRecoveryCount = (item._quarantineRecoveryCount || 0) + 1;
+        const cap = ENGINE_DEFAULTS.quarantineAutoRecoveryMax || 2;
+        if (item._quarantineRecoveryCount <= cap) {
+          const prevReason = item.failReason;
+          item.status = WI_STATUS.PENDING;
+          delete item.failReason;
+          delete item.failedAt;
+          delete item._failureClass;
+          delete item._lastDispatchResult;
+          delete item.dispatched_at;
+          delete item.dispatched_to;
+          delete item._pendingReason;
+          log('info', `Quarantine auto-recovery: ${item.id} → pending (attempt ${item._quarantineRecoveryCount}/${cap}, was: ${prevReason})`);
+          needsWrite = true;
+          // Intentionally do NOT `continue` — fall through to the normal dispatch path
+          // (self-heal of stale completed dedupe + cooldown + dispatch evaluation) just
+          // like the dependency-failed recovery block above. The dedupe + cooldown gates
+          // still throttle how often the next dispatch fires.
+        } else {
+          // Cap hit: leave failed, log once via the gave-up flag so the warn doesn't
+          // fire every tick. _retriesByAgent is untouched — quarantine failures are
+          // environmental, not agent failures.
+          if (!item._quarantineRecoveryGaveUp) {
+            log('warn', `Quarantine auto-recovery: ${item.id} hit cap (${cap}) — leaving failed for human intervention`);
+            item._quarantineRecoveryGaveUp = true;
+            needsWrite = true;
+          }
+        }
+      }
+    }
+
     if (item.status !== WI_STATUS.QUEUED && item.status !== WI_STATUS.PENDING) continue;
 
     // Dependency gate: skip items whose depends_on are not yet met; propagate failure

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2120",
+  "version": "0.1.2121",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"