@yemi33/minions 0.1.1154 → 0.1.1155

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.1154 (2026-04-18)
+## 0.1.1155 (2026-04-18)
 
 ### Features
 - redact ADO tokens and JWTs from engine/log.json writes (#1297)
@@ -9,6 +9,7 @@
 - seed realActivityMap at spawn time, stamp pid in live-output (#1200)
 
 ### Fixes
+-  prevent archived plans from re-triggering via orphaned .backup restore
 - pinned context optimistic save (closes #1295) (#1316)
 -  re-check throttle state per-PR iteration to avoid stale fixThrottled
 - preserve buildErrorLog through transient states, persist poll time (#1273)
@@ -28,7 +29,6 @@
 -  reassign pending items from unspawned temp agents to idle named agents (#1204) (#1212)
 - guard undefined agent in pending dispatch loop (closes #1206) (#1210)
 -  improve fallback meeting conclusion
--  remove command center chevron
 
 ### Other
 - refactor: hoist fixThrottled before PR loop and drop underscore prefix

package/engine/lifecycle.js

@@ -355,8 +355,12 @@ function archivePlan(planFile, plan, projects, config) {
     // Remove .backup sidecar — if left behind, safeJson() would restore the pre-completion
     // snapshot (status: approved, no _completionNotified) on engine restart, re-triggering
     // plan completion and spawning duplicate verify tasks for already-archived plans.
+    // On Windows, the unlink can fail due to file locking; overwrite with archived status
+    // as a fallback so a restored backup is inert even if deletion fails.
     const backupPath = planPath + '.backup';
-    if (fs.existsSync(backupPath)) fs.unlinkSync(backupPath);
+    try { fs.unlinkSync(backupPath); } catch {
+      try { fs.writeFileSync(backupPath, JSON.stringify({ status: 'archived' })); } catch { }
+    }
   } catch (err) {
     log('warn', `Failed to archive PRD ${planFile}: ${err.message}`);
   }

package/engine.js

@@ -3130,6 +3130,13 @@ async function discoverWork(config) {
       if (fs.existsSync(prdDir)) {
         for (const f of fs.readdirSync(prdDir).filter(f => f.endsWith('.json'))) {
           if (completedPlanCache.has(f)) continue;
+          if (fs.existsSync(path.join(prdDir, 'archive', f))) {
+            // Orphaned backup restore — plan is already archived. Purge the ghost copy.
+            try { fs.unlinkSync(path.join(prdDir, f)); } catch { }
+            try { fs.unlinkSync(path.join(prdDir, f + '.backup')); } catch { }
+            completedPlanCache.add(f);
+            continue;
+          }
           const plan = safeJson(path.join(prdDir, f));
           if (!plan?.missing_features || plan.status === 'completed') {
             if (plan?.status === 'completed') completedPlanCache.add(f);
@@ -3309,6 +3316,13 @@ async function tickInner() {
       const prdFiles = safeReadDir(PRD_DIR).filter(f => f.endsWith('.json'));
       for (const file of prdFiles) {
         if (completedPlanCache.has(file)) continue;
+        if (fs.existsSync(path.join(PRD_DIR, 'archive', file))) {
+          // Orphaned backup restore — plan is already archived. Purge the ghost copy.
+          try { fs.unlinkSync(path.join(PRD_DIR, file)); } catch { }
+          try { fs.unlinkSync(path.join(PRD_DIR, file + '.backup')); } catch { }
+          completedPlanCache.add(file);
+          continue;
+        }
         const plan = safeJson(path.join(PRD_DIR, file));
         if (plan && plan.missing_features && plan.status !== 'completed') {
           const completed = checkPlanCompletion({ item: { sourcePlan: file } }, config);

package/package.json

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