@yemi33/minions 0.1.1569 → 0.1.1571

package/CHANGELOG.md

@@ -1,8 +1,14 @@
 # Changelog
 
-## 0.1.1569 (2026-04-27)
+## 0.1.1571 (2026-04-27)
 
 ### Fixes
+- break work-items.json watcher tick storm
+
+## 0.1.1570 (2026-04-27)
+
+### Fixes
+-  gate retry redispatches
 -  avoid duplicate review retry loops
 
 ## 0.1.1568 (2026-04-27)

package/engine/timeout.js

@@ -398,6 +398,10 @@ function checkTimeouts(config) {
     allWiPaths.push(projectWorkItemsPath(project));
   }
   for (const wiPath of allWiPaths) {
+    // skipWriteIfUnchanged is critical — checkTimeouts runs every tick, and
+    // without this gate the file's mtime updates on every call (no real
+    // change), tripping the cli.js work-items.json watcher → triggering tick
+    // → calling checkTimeouts → ... a 5-6s loop of "File change detected".
     mutateJsonFileLocked(wiPath, (items) => {
       if (!items || !Array.isArray(items)) return items;
       let changed = false;
@@ -438,7 +442,7 @@ function checkTimeouts(config) {
         }
       }
       return items;
-    }, { defaultValue: [] });
+    }, { defaultValue: [], skipWriteIfUnchanged: true });
   }
 }
 

package/engine.js

@@ -2360,6 +2360,19 @@ function discoverFromWorkItems(config, project) {
         skipped.gated++; continue;
       }
     }
+    // Min retry gap (#1770): even when isRetry bypasses the completed-dedup, a
+    // tight retry loop on the same WI burns _retryCount inside one tick window.
+    // Honor a configurable minimum interval since the last retry so an idempotent
+    // agent (e.g. review bailing because a duplicate review already exists)
+    // cannot flip a successfully-posted review into status=failed.
+    if (isRetry && item._lastRetryAt) {
+      const minGap = ENGINE_DEFAULTS.minRetryGapMs;
+      const lastAt = Date.parse(item._lastRetryAt);
+      if (Number.isFinite(lastAt) && (Date.now() - lastAt) < minGap) {
+        if (item._pendingReason !== 'retry_cooldown') { item._pendingReason = 'retry_cooldown'; needsWrite = true; }
+        skipped.gated++; continue;
+      }
+    }
     if (isOnCooldown(key, cooldownMs)) {
       if (item._pendingReason !== 'cooldown') { item._pendingReason = 'cooldown'; needsWrite = true; }
       skipped.gated++; continue;
@@ -2804,6 +2817,14 @@ function discoverCentralWorkItems(config) {
         continue;
       }
     }
+    // Min retry gap (#1770): symmetric with discoverFromWorkItems — even when
+    // isRetry bypasses the completed-dedup, enforce a min interval since
+    // _lastRetryAt so a tight retry loop can't burn _retryCount in one tick.
+    if (isRetry && item._lastRetryAt) {
+      const minGap = ENGINE_DEFAULTS.minRetryGapMs;
+      const lastAt = Date.parse(item._lastRetryAt);
+      if (Number.isFinite(lastAt) && (Date.now() - lastAt) < minGap) continue;
+    }
     if (isOnCooldown(key, 0)) continue;
 
     const workType = item.type || 'implement';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1569",
+  "version": "0.1.1571",
   "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"