@yemi33/minions 0.1.330 → 0.1.332

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.332 (2026-04-03)
+
+### Fixes
+-  retry doesn't revert completed work items + restore skipPr guard
+
+## 0.1.331 (2026-04-03)
+
+### Fixes
+-  standardize PR created field to ISO format
+
 ## 0.1.330 (2026-04-03)
 
 ### Fixes

package/dashboard/js/render-prs.js

@@ -27,7 +27,7 @@ function prRow(pr) {
     '<td>' + (sq.reviewer && sq.status !== 'waiting' ? '<span class="pr-agent" title="' + escHtml(sq.note || '') + '">' + escHtml(sq.reviewer) + '</span>' : sq.reviewer && sq.status === 'waiting' ? '<span class="pr-agent" style="color:var(--muted)" title="Vote pending confirmation">' + escHtml(sq.reviewer) + '…</span>' : pr.reviewedBy && pr.reviewedBy.length ? '<span class="pr-agent">' + escHtml(pr.reviewedBy.join(', ')) + '</span>' : '<span style="color:var(--muted);font-size:11px">—</span>') + '</td>' +
     '<td><span class="pr-badge ' + buildClass + '">' + escHtml(buildLabel) + '</span></td>' +
     '<td><span class="pr-badge ' + statusClass + '">' + escHtml(statusLabel) + '</span></td>' +
-    '<td><span class="pr-date">' + escHtml(pr.created || '—') + '</span></td>' +
+    '<td><span class="pr-date">' + escHtml((pr.created || '—').slice(0, 10)) + '</span></td>' +
     '</tr>';
 }
 

package/dashboard.js

@@ -3458,7 +3458,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         branch: '',
         reviewStatus: autoObserve ? 'pending' : 'none',
         status: autoObserve ? 'active' : 'linked',
-        created: new Date().toISOString().slice(0, 10),
+        created: new Date().toISOString(),
         url,
         prdItems: [],
         _manual: true,

package/engine/ado.js

@@ -5,7 +5,7 @@
 
 const path = require('path');
 const shared = require('./shared');
-const { exec, getAdoOrgBase, addPrLink, log, dateStamp, PR_STATUS } = shared;
+const { exec, getAdoOrgBase, addPrLink, log, ts, dateStamp, PR_STATUS } = shared;
 const { getPrs } = require('./queries');
 
 // Lazy require to avoid circular dependency — only needed for engine().handlePostMerge
@@ -389,7 +389,7 @@ async function reconcilePrs(config) {
         branch,
         reviewStatus: 'pending',
         status: 'active',
-        created: (adoPr.creationDate || '').slice(0, 10) || dateStamp(),
+        created: adoPr.creationDate || ts(),
         url: prUrl,
         prdItems: confirmedItemId ? [confirmedItemId] : [],
       });

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/github.js

@@ -5,7 +5,7 @@
  */
 
 const shared = require('./shared');
-const { exec, getProjects, projectPrPath, projectWorkItemsPath, safeJson, safeWrite, MINIONS_DIR, addPrLink, getPrLinks, log, dateStamp, PR_STATUS } = shared;
+const { exec, getProjects, projectPrPath, projectWorkItemsPath, safeJson, safeWrite, MINIONS_DIR, addPrLink, getPrLinks, log, ts, dateStamp, PR_STATUS } = shared;
 const { getPrs } = require('./queries');
 const path = require('path');
 
@@ -373,7 +373,7 @@ async function reconcilePrs(config) {
         branch,
         reviewStatus: 'pending',
         status: 'active',
-        created: (ghPr.created_at || '').slice(0, 10) || dateStamp(),
+        created: ghPr.created_at || ts(),
         url: prUrl,
         prdItems: confirmedItemId ? [confirmedItemId] : [],
       });

package/engine/lifecycle.js

@@ -669,7 +669,7 @@ function syncPrsFromOutput(output, agentId, meta, config) {
         branch: meta?.branch || '',
         reviewStatus: 'pending',
         status: PR_STATUS.ACTIVE,
-        created: dateStamp(),
+        created: ts(),
         url: extractPrUrl(prId),
         prdItems: meta?.item?.id ? [meta.item.id] : [],
         sourcePlan: meta?.item?.sourcePlan || '',
@@ -681,6 +681,10 @@ function syncPrsFromOutput(output, agentId, meta, config) {
   for (const [prPath, { name, entries }] of newPrsByPath) {
     mutateJsonFileLocked(prPath, (data) => {
       const prs = Array.isArray(data) ? data : [];
+      // Normalize legacy YYYY-MM-DD created dates to ISO
+      for (const p of prs) {
+        if (p.created && p.created.length === 10) p.created = p.created + 'T00:00:00.000Z';
+      }
       for (const { prId, fullId, entry } of entries) {
         if (prs.some(p => p.id === fullId || String(p.id) === String(prId))) continue;
         prs.push(entry);
@@ -1171,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}`); }
@@ -1257,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.330",
+  "version": "0.1.332",
   "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"