@yemi33/minions 0.1.2333 → 0.1.2335

package/engine/projects.js

@@ -125,7 +125,10 @@ function _requeueProjectlessCentralWorkItems(itemIds) {
  * @param {object} [options]
  * @param {'archive'|'keep'|'purge'} [options.dataMode='archive']
  *   archive: move projects/<name>/ to projects/.archived/<name>-YYYYMMDD/
- *   keep:    leave projects/<name>/ in place
+ *   keep:    leave projects/<name>/ in place, but tombstone work-items.json /
+ *            pull-requests.json into projects/<name>/.dropped/ so dropScope()
+ *            (step 9) isn't immediately undone by the *-store.js resync-on-
+ *            read path re-hydrating the still-present JSON (W-mr96oira000g979a)
  *   purge:   rm -rf projects/<name>/
  * @returns {object} summary { ok, project, cancelledItems, killedAgents,
  *   drainedDispatches, cleanedWorktrees, disabledSchedules, archivedTo,
@@ -145,6 +148,7 @@ function removeProject(target, options = {}) {
     pipelineRefs: [],
     archivedTo: null,
     purgedDataDir: false,
+    tombstonedDataFiles: false,
     // P-bfa2d-remove-project-cleanup — soft-delete tag counts for entries in
     // the three central state files that survived the hard-cancel passes
     // (steps 1, 2). See step 7.5 for the orphan-tag policy rationale.
@@ -427,6 +431,35 @@ function removeProject(target, options = {}) {
         summary.archiveError = e.code ? `${e.code}: ${e.message}` : e.message;
         summary.warnings.push('archive data dir: ' + e.message);
       }
+    } else if (dataMode === 'keep') {
+      // W-mr96oira000g979a — 'keep' leaves projects/<name>/ (and therefore
+      // work-items.json / pull-requests.json) at its normal, live-resolving
+      // path. dropScope() below DELETEs the SQL rows for this scope, but the
+      // *-store.js resync-on-read path (_resyncScopeIfJsonDiverged) treats a
+      // scope with an empty SQL table + a still-present JSON file as a
+      // fresh-install first-touch and re-hydrates straight from that JSON —
+      // silently undoing the drop the very next time anything reads this
+      // scope. Tombstone the two live-state files into a `.dropped/`
+      // subdirectory so `_filePathForScope(scope)` (and the read-side
+      // resync/fallback logic keyed off it) no longer resolves to them,
+      // while keeping the historical data on disk for manual inspection —
+      // exactly what 'keep' promises, minus the auto-resurrection.
+      try {
+        const droppedDir = path.join(dataDir, '.dropped');
+        const stamp = new Date().toISOString().replace(/[:.]/g, '-');
+        for (const fname of ['work-items.json', 'pull-requests.json']) {
+          const src = path.join(dataDir, fname);
+          if (!fs.existsSync(src)) continue;
+          fs.mkdirSync(droppedDir, { recursive: true });
+          let dest = path.join(droppedDir, `${fname}.${stamp}`);
+          let n = 1;
+          while (fs.existsSync(dest)) dest = path.join(droppedDir, `${fname}.${stamp}-${++n}`);
+          _renameWithRetry(src, dest);
+          summary.tombstonedDataFiles = true;
+        }
+      } catch (e) {
+        summary.warnings.push('tombstone kept data files: ' + e.message);
+      }
     }
   }
 

package/engine/queries.js

@@ -2461,7 +2461,7 @@ function getPrdInfo(config) {
     const t = planTimings[wi.sourcePlan];
     if (wi.dispatched_at) { const d = new Date(wi.dispatched_at).getTime(); if (!t.firstDispatched || d < t.firstDispatched) t.firstDispatched = d; }
     if (wi.completedAt) { const c = new Date(wi.completedAt).getTime(); if (!t.lastCompleted || c > t.lastCompleted) t.lastCompleted = c; }
-    if (wi.status !== 'done') t.allDone = false;
+    if (!DONE_STATUSES.has(wi.status)) t.allDone = false;
   }
 
   const progress = {

package/package.json

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