@yemi33/minions 0.1.331 → 0.1.333

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.333 (2026-04-03)
+
+### Fixes
+-  guarantee test cleanup via finally block in test harness
+
+## 0.1.332 (2026-04-03)
+
+### Fixes
+-  retry doesn't revert completed work items + restore skipPr guard
+
 ## 0.1.331 (2026-04-03)
 
 ### Fixes

package/engine/dispatch.js

@@ -137,7 +137,7 @@ function completeDispatch(id, result = DISPATCH_RESULT.SUCCESS, reason = '', res
             const items = safeJson(wiPath);
             if (!items || !Array.isArray(items)) throw new Error('work items unreadable');
             const wi = items.find(i => i.id === item.meta.item.id);
-            if (wi && wi.status !== WI_STATUS.PAUSED) {
+            if (wi && wi.status !== WI_STATUS.PAUSED && wi.status !== WI_STATUS.DONE && !wi.completedAt) {
               wi._retryCount = retries + 1;
               wi.status = WI_STATUS.PENDING;
               wi._lastRetryReason = reason || '';

package/engine/lifecycle.js

@@ -1175,9 +1175,14 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
           const items = safeJson(wiPath) || [];
           const wi = items.find(i => i.id === meta.item.id);
           if (wi) {
-            wi._retryCount = retries + 1; wi.status = WI_STATUS.PENDING; delete wi.dispatched_at; delete wi.dispatched_to;
-            if (type === WORK_TYPE.DECOMPOSE) delete wi._decomposing; // clear so item can retry decomposition
-            shared.safeWrite(wiPath, items);
+            // Don't revert if already completed by another code path
+            if (wi.status === WI_STATUS.DONE || wi.completedAt) {
+              log('info', `Skip retry for ${meta.item.id} — already completed`);
+            } else {
+              wi._retryCount = retries + 1; wi.status = WI_STATUS.PENDING; delete wi.dispatched_at; delete wi.dispatched_to;
+              if (type === WORK_TYPE.DECOMPOSE) delete wi._decomposing;
+              shared.safeWrite(wiPath, items);
+            }
           }
         }
       } catch (err) { log('warn', `Retry update: ${err.message}`); }
@@ -1261,7 +1266,7 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
   }
 
   // Detect implement tasks that completed without creating a PR
-  if (isSuccess && (type === WORK_TYPE.IMPLEMENT || type === WORK_TYPE.IMPLEMENT_LARGE || type === WORK_TYPE.FIX) && prsCreatedCount === 0 && meta?.item?.id) {
+  if (isSuccess && (type === WORK_TYPE.IMPLEMENT || type === WORK_TYPE.IMPLEMENT_LARGE || type === WORK_TYPE.FIX) && prsCreatedCount === 0 && meta?.item?.id && !meta?.item?.skipPr && meta?.project?.localPath) {
     // Check if a PR already exists linked to this work item (from a previous attempt)
     const projects = shared.getProjects(config);
     const existingPrFound = Object.values(getPrLinks()).includes(meta.item.id);

package/package.json

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