@yemi33/minions 0.1.1841 → 0.1.1842

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.1842 (2026-05-10)
+
+### Fixes
+- retry rename on Windows EPERM/EBUSY in removeProject archive
+
 ## 0.1.1841 (2026-05-10)
 
 ### Other

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-10T03:24:41.388Z"
+  "cachedAt": "2026-05-10T03:52:39.031Z"
 }

package/engine/projects.js

@@ -15,6 +15,23 @@ function _sameProjectName(a, b) {
   return String(a || '').toLowerCase() === String(b || '').toLowerCase();
 }
 
+// Windows EPERM/EBUSY can hit fs.renameSync transiently when antivirus,
+// indexer, or another scanner has the directory open. Retry with backoff
+// before giving up — same pattern used by shared.safeWrite for atomic
+// JSON writes (engine/shared.js:424).
+function _renameWithRetry(src, dest) {
+  for (let attempt = 0; attempt < 5; attempt++) {
+    try {
+      fs.renameSync(src, dest);
+      return;
+    } catch (e) {
+      const transient = e.code === 'EPERM' || e.code === 'EBUSY' || e.code === 'ENOTEMPTY';
+      if (!transient || attempt === 4) throw e;
+      shared.sleepMs(50 * (attempt + 1)); // 50, 100, 150, 200ms
+    }
+  }
+}
+
 function _projectRefMatches(projectRef, removedProject, projects) {
   if (!projectRef) return false;
   const refs = (projectRef && typeof projectRef === 'object')
@@ -267,9 +284,16 @@ function removeProject(target, options = {}) {
         let archived = path.join(archiveRoot, project.name + '-' + stamp);
         let n = 1;
         while (fs.existsSync(archived)) archived = path.join(archiveRoot, project.name + '-' + stamp + '-' + (++n));
-        fs.renameSync(dataDir, archived);
+        _renameWithRetry(dataDir, archived);
         summary.archivedTo = path.relative(MINIONS_DIR, archived).replace(/\\/g, '/');
-      } catch (e) { summary.warnings.push('archive data dir: ' + e.message); }
+      } catch (e) {
+        // Surface the failure: callers (CLI, dashboard, tests) need to know
+        // the data dir is still in place. Previously this was a silent warning
+        // and `archivedTo` stayed null, making flakes look like missing-set
+        // assertions instead of real archive failures.
+        summary.archiveError = e.code ? `${e.code}: ${e.message}` : e.message;
+        summary.warnings.push('archive data dir: ' + e.message);
+      }
     }
   }
 

package/package.json

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