@yemi33/minions 0.1.326 → 0.1.327

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.326 (2026-04-03)
+## 0.1.327 (2026-04-03)
 
 ### Fixes
+-  eliminate all legacy done-status writers, add CANCELLED to WI_STATUS
 -  stop writing legacy done aliases, add migration, keep read compat
 
 ## 0.1.325 (2026-04-03)

package/engine/cleanup.js

@@ -441,7 +441,7 @@ function runCleanup(config, verbose = false) {
       let migrated = 0;
       for (const item of items) {
         if (LEGACY_DONE_ALIASES.has(item.status)) {
-          item.status = 'done';
+          item.status = shared.WI_STATUS.DONE;
           delete item._pendingReason;
           migrated++;
         }
@@ -459,7 +459,7 @@ function runCleanup(config, verbose = false) {
     let migrated = 0;
     for (const item of centralItems) {
       if (LEGACY_DONE_ALIASES.has(item.status)) {
-        item.status = 'done';
+        item.status = shared.WI_STATUS.DONE;
         delete item._pendingReason;
         migrated++;
       }
@@ -486,7 +486,7 @@ function runCleanup(config, verbose = false) {
       let migrated = 0;
       for (const feat of prd.missing_features) {
         if (LEGACY_DONE_ALIASES.has(feat.status)) {
-          feat.status = 'done';
+          feat.status = shared.WI_STATUS.DONE;
           migrated++;
         }
       }

package/engine/lifecycle.js

@@ -806,9 +806,9 @@ async function handlePostMerge(pr, project, config, newStatus) {
         const items = safeJson(wiPath);
         if (!items) continue;
         const item = items.find(i => i.id === mergedItemId);
-        if (item && item.status !== 'done') {
+        if (item && item.status !== WI_STATUS.DONE) {
           log('info', `Post-merge: marking work item ${mergedItemId} as done (was ${item.status}) for ${pr.id}`);
-          item.status = 'done';
+          item.status = WI_STATUS.DONE;
           item.completedAt = ts();
           item._mergedVia = pr.id;
           shared.safeWrite(wiPath, items);
@@ -1070,7 +1070,7 @@ function handleDecompositionResult(stdout, meta, config) {
     if (!parent) continue;
 
     // Mark parent as decomposed
-    parent.status = 'decomposed';
+    parent.status = WI_STATUS.DECOMPOSED;
     parent._decomposed = true;
     delete parent._decomposing;
     parent._subItemIds = subItems.map(s => s.id);

package/engine/shared.js

@@ -393,7 +393,8 @@ const ENGINE_DEFAULTS = {
 
 const WI_STATUS = {
   PENDING: 'pending', DISPATCHED: 'dispatched', DONE: 'done', FAILED: 'failed',
-  PAUSED: 'paused', QUEUED: 'queued', NEEDS_REVIEW: 'needs-human-review', DECOMPOSED: 'decomposed',
+  PAUSED: 'paused', QUEUED: 'queued', NEEDS_REVIEW: 'needs-human-review',
+  DECOMPOSED: 'decomposed', CANCELLED: 'cancelled',
 };
 // Read-side: accept legacy aliases for backward compat with old data/clients.
 // Write-side: only WI_STATUS.DONE is written (cleanup.js migrates old values on each run).

package/engine.js

@@ -1182,7 +1182,7 @@ function materializePlansAsWorkItems(config) {
         for (const wi of existingItems) {
           if (wi.status !== WI_STATUS.PENDING || wi.sourcePlan !== file) continue;
           if (!currentPrdIds.has(wi.id)) {
-            wi.status = 'cancelled';
+            wi.status = WI_STATUS.CANCELLED;
             wi.cancelledAt = ts();
             wi.cancelReason = `PRD item removed from ${file}`;
             cancelled++;
@@ -1504,7 +1504,7 @@ function discoverFromWorkItems(config, project) {
         const cpCount = (item._checkpointCount || 0) + 1;
         if (cpCount > 3) {
           log('warn', `Work item ${item.id} exceeded 3 checkpoint-resumes — marking as needs-human-review`);
-          item.status = 'needs-human-review';
+          item.status = WI_STATUS.NEEDS_REVIEW;
           item._checkpointCount = cpCount;
           needsWrite = true;
           continue;
@@ -1918,7 +1918,7 @@ function discoverCentralWorkItems(config) {
           const cpCount = (item._checkpointCount || 0) + 1;
           if (cpCount > 3) {
             log('warn', `Work item ${item.id} exceeded 3 checkpoint-resumes — marking as needs-human-review`);
-            item.status = 'needs-human-review';
+            item.status = WI_STATUS.NEEDS_REVIEW;
             item._checkpointCount = cpCount;
             continue;
           }
@@ -2425,7 +2425,7 @@ async function tickInner() {
               const wi = items.find(i => i.id === item.meta.item.id);
               if (wi && wi.status === WI_STATUS.DISPATCHED) {
                 // completeDispatch didn't update the work item — re-queue manually
-                wi.status = 'pending';
+                wi.status = WI_STATUS.PENDING;
                 wi._retryCount = (wi._retryCount || 0) + 1;
                 wi._lastRetryReason = 'spawnAgent returned null';
                 wi._lastRetryAt = ts();

package/package.json

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