@yemi33/minions 0.1.2 → 0.1.4

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## 0.1.4 (2026-03-24)
+
+### Engine
+- engine.js
+- engine/ado.js
+- engine/github.js
+- engine/lifecycle.js
+
+## 0.1.3 (2026-03-24)
+
+### Engine
+- engine/lifecycle.js
+
+### Dashboard
+- dashboard.html
+- dashboard.js
+
 ## 0.1.1 (2026-03-23)
 
 ### Engine

package/dashboard.html

@@ -3675,9 +3675,17 @@ function renderPlans(plans) {
             : (statusLabels[status] || status);
     const needsAction = (status === 'awaiting-approval' || status === 'paused' || isPrdAwaitingApproval || isPrdPaused) && !isArchived;
     const isRevision = status === 'revision-requested' && !isArchived;
+    const isCompleted = status === 'completed';
+    const isDraft = (p.format === 'draft' || status === 'draft') && !isCompleted;
+    const isAwaitingApproval = status === 'awaiting-approval';
+    const isPaused = status === 'paused';
+    const isApproved = status === 'approved' || status === 'active';
+    // For .md drafts: show Execute only if no PRD exists yet (not already executed)
+    const prdFile = planToPrdFile[p.file] || (p.file.endsWith('.json') ? p.file : '');
 
     let actions = '';
-    if (needsAction) {
+    const resumeVisible = ((isPrdBlocked || isAwaitingApproval || isPaused) && prdFile && !isArchived);
+    if (needsAction && !resumeVisible) {
       actions = '<div class="plan-card-actions" onclick="event.stopPropagation()">' +
         '<button class="plan-btn approve" onclick="planApprove(\'' + escHtml(p.file) + '\')">Approve</button>' +
         '<button class="plan-btn" style="color:var(--blue);border-color:var(--blue)" onclick="planDiscuss(\'' + escHtml(p.file) + '\')">Discuss &amp; Revise</button>' +
@@ -3694,13 +3702,6 @@ function renderPlans(plans) {
       actions = '<div class="plan-card-meta" style="margin-top:6px;color:var(--purple,#a855f7)">Revision in progress: ' + escHtml((p.revisionFeedback || '').slice(0, 100)) + '</div>';
     }
 
-    const isCompleted = status === 'completed';
-    const isDraft = (p.format === 'draft' || status === 'draft') && !isCompleted;
-    const isAwaitingApproval = status === 'awaiting-approval';
-    const isPaused = status === 'paused';
-    const isApproved = status === 'approved' || status === 'active';
-    // For .md drafts: show Execute only if no PRD exists yet (not already executed)
-    const prdFile = planToPrdFile[p.file] || (p.file.endsWith('.json') ? p.file : '');
     const executeBtn = isDraft && !isWorking && !isPrdBlocked && !isArchived && !prdFile ? '<button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;color:var(--green);font-weight:600" ' +
       'onclick="event.stopPropagation();planExecute(\'' + escHtml(p.file) + '\',\'' + escHtml(p.project) + '\',this)">Execute</button>' : '';
     // Pause/Resume: target the PRD .json file if it exists, otherwise the plan itself
@@ -4061,7 +4062,8 @@ async function planView(file) {
     const prdCompleted = wi.some(w => w.type === 'plan-to-prd' && w.status === 'done' && w.planFile === normalizedFile);
     // Check if a PRD already exists for this plan (via plans list sourcePlan linkage)
     const hasPrd = (window._lastStatus?.plans || []).some(p => p.sourcePlan === normalizedFile && p.format === 'prd');
-    const modalExecuteBtn = isMdPlan && !hasActiveWork && !prdCompleted && !hasPrd ? '<button class="pr-pager-btn" style="font-size:10px;padding:2px 10px;color:var(--green);font-weight:600" ' +
+    const modalShowResume = isPaused;
+    const modalExecuteBtn = isMdPlan && !modalShowResume && !hasActiveWork && !prdCompleted && !hasPrd ? '<button class="pr-pager-btn" style="font-size:10px;padding:2px 10px;color:var(--green);font-weight:600" ' +
       'onclick="planExecute(\'' + escHtml(normalizedFile) + '\',\'\',this)">Execute</button>' : '';
     const modalCompletedLabel = prdCompleted && !hasActiveWork ? '<span style="font-size:10px;color:var(--green);font-weight:600">Completed</span>' : '';
     const modalInProgressLabel = hasActiveWork ? '<span style="font-size:10px;color:var(--blue)">In Progress</span>' : '';

package/dashboard.js

@@ -1477,7 +1477,7 @@ If nothing to do, return: { "duplicates": [], "reclassify": [], "remove": [] }`;
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 
-  // POST /api/plans/pause — pause a plan (stops new item materialization + pauses work items)
+  // POST /api/plans/pause — pause a plan (stops new item materialization + resets active items to pending)
   if (req.method === 'POST' && req.url === '/api/plans/pause') {
     try {
       const body = await readBody(req);
@@ -1488,25 +1488,17 @@ If nothing to do, return: { "duplicates": [], "reclassify": [], "remove": [] }`;
       plan.pausedAt = new Date().toISOString();
       safeWrite(planPath, plan);
 
-      // Propagate pause to materialized work items across all projects
-      // But skip items that already have active PRs — those are past the point of pausing
-      let paused = 0;
+      // Propagate pause to materialized work items across all projects:
+      // kill any active agent process and reset non-completed items back to pending.
+      let reset = 0;
       const wiPaths = [path.join(MINIONS_DIR, 'work-items.json')];
-      const allPrItemIds = new Set();
       for (const proj of PROJECTS) {
         wiPaths.push(shared.projectWorkItemsPath(proj));
-        try {
-          const prs = safeJson(shared.projectPrPath(proj)) || [];
-          for (const pr of prs) {
-            if (pr.status === 'active' && pr.prdItems?.length) {
-              pr.prdItems.forEach(id => allPrItemIds.add(id));
-            }
-          }
-        } catch {}
       }
       const dispatchPath = path.join(MINIONS_DIR, 'engine', 'dispatch.json');
       const dispatch = JSON.parse(safeRead(dispatchPath) || '{}');
-      const killedAgents = [];
+      const killedAgents = new Set();
+      const resetItemIds = new Set();
 
       for (const wiPath of wiPaths) {
         try {
@@ -1515,17 +1507,12 @@ If nothing to do, return: { "duplicates": [], "reclassify": [], "remove": [] }`;
           let changed = false;
           for (const w of items) {
             if (w.sourcePlan !== body.file) continue;
-            // Don't pause if this item already has an active PR
-            if (allPrItemIds.has(w.id) || allPrItemIds.has(w.sourcePlanItem)) continue;
+            // Keep completed items as-is, reset everything else to pending.
+            if (w.status === 'done' || w.status === 'implemented' || w.status === 'complete' || w.status === 'in-pr') continue;
 
-            if (w.status === 'pending') {
-              w.status = 'paused';
-              w._pausedBy = 'prd-pause';
-              paused++;
-              changed = true;
-            } else if (w.status === 'dispatched') {
-              // Kill the agent working on this item
-              const activeEntry = (dispatch.active || []).find(d => d.meta?.dispatchKey?.includes(w.id));
+            if (w.status === 'dispatched') {
+              // Kill the agent working on this item, if any.
+              const activeEntry = (dispatch.active || []).find(d => d.meta?.item?.id === w.id || d.meta?.dispatchKey?.includes(w.id));
               if (activeEntry) {
                 const statusPath = path.join(MINIONS_DIR, 'agents', activeEntry.agent, 'status.json');
                 try {
@@ -1542,30 +1529,41 @@ If nothing to do, return: { "duplicates": [], "reclassify": [], "remove": [] }`;
                   delete agentStatus.dispatched;
                   safeWrite(statusPath, agentStatus);
                 } catch {}
-                killedAgents.push(activeEntry.agent);
+                killedAgents.add(activeEntry.agent);
               }
-              w.status = 'paused';
-              w._pausedBy = 'prd-pause';
-              paused++;
-              changed = true;
             }
+
+            if (w.status !== 'pending') reset++;
+            w.status = 'pending';
+            delete w._pausedBy;
+            delete w._resumedAt;
+            delete w.dispatched_at;
+            delete w.dispatched_to;
+            delete w.failReason;
+            delete w.failedAt;
+            changed = true;
+            if (w.id) resetItemIds.add(w.id);
           }
           if (changed) safeWrite(wiPath, items);
         } catch {}
       }
 
-      // Remove killed agents from dispatch active
-      if (killedAgents.length > 0) {
-        const killedSet = new Set(killedAgents);
+      // Remove dispatch active entries for reset items or killed agents.
+      if (resetItemIds.size > 0 || killedAgents.size > 0) {
         mutateJsonFileLocked(dispatchPath, (dp) => {
           dp.active = Array.isArray(dp.active) ? dp.active : [];
-          dp.active = dp.active.filter(d => !killedSet.has(d.agent));
+          dp.active = dp.active.filter(d => {
+            const itemId = d.meta?.item?.id;
+            if (itemId && resetItemIds.has(itemId)) return false;
+            if (killedAgents.has(d.agent)) return false;
+            return true;
+          });
           return dp;
         }, { defaultValue: { pending: [], active: [], completed: [] } });
       }
 
       invalidateStatusCache();
-      return jsonReply(res, 200, { ok: true, status: 'paused', pausedWorkItems: paused });
+      return jsonReply(res, 200, { ok: true, status: 'paused', resetWorkItems: reset });
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 

package/engine/ado.js

@@ -339,6 +339,11 @@ async function reconcilePrs(config) {
         // PR already tracked — write link to pr-links.json if we can extract an ID
         if (confirmedItemId) {
           addPrLink(prId, confirmedItemId);
+          const existing = existingPrs.find(p => p.id === prId);
+          if (existing && !(existing.prdItems || []).includes(confirmedItemId)) {
+            existing.prdItems = Array.isArray(existing.prdItems) ? existing.prdItems : [];
+            existing.prdItems.push(confirmedItemId);
+          }
           projectUpdated++;
         }
         continue;
@@ -354,6 +359,7 @@ async function reconcilePrs(config) {
         status: 'active',
         created: (adoPr.creationDate || '').slice(0, 10) || e.dateStamp(),
         url: prUrl,
+        prdItems: confirmedItemId ? [confirmedItemId] : [],
       });
       if (confirmedItemId) addPrLink(prId, confirmedItemId);
       existingIds.add(prId);

package/engine/github.js

@@ -5,7 +5,7 @@
  */
 
 const shared = require('./shared');
-const { exec, getProjects, projectPrPath, projectWorkItemsPath, safeJson, safeWrite, MINIONS_DIR } = shared;
+const { exec, getProjects, projectPrPath, projectWorkItemsPath, safeJson, safeWrite, MINIONS_DIR, addPrLink } = shared;
 const { getPrs } = require('./queries');
 const path = require('path');
 
@@ -285,14 +285,24 @@ async function reconcilePrs(config) {
 
     for (const ghPr of ghPrs) {
       const prId = `PR-${ghPr.number}`;
-      if (existingIds.has(prId)) continue;
-
       const branch = ghPr.head?.ref || '';
       const wiMatch = branch.match(/(P-[a-f0-9]{6,})/i) || branch.match(/(PL-W\d+)/i);
       const linkedItemId = wiMatch ? wiMatch[1] : null;
       const linkedItem = linkedItemId ? allItems.find(i => i.id === linkedItemId) : null;
       const confirmedItemId = linkedItem ? linkedItemId : null;
 
+      if (existingIds.has(prId)) {
+        if (confirmedItemId) {
+          addPrLink(prId, confirmedItemId);
+          const existing = existingPrs.find(p => p.id === prId);
+          if (existing && !(existing.prdItems || []).includes(confirmedItemId)) {
+            existing.prdItems = Array.isArray(existing.prdItems) ? existing.prdItems : [];
+            existing.prdItems.push(confirmedItemId);
+          }
+        }
+        continue;
+      }
+
       const prUrl = project.prUrlBase ? project.prUrlBase + ghPr.number : ghPr.html_url || '';
 
       existingPrs.push({
@@ -306,6 +316,7 @@ async function reconcilePrs(config) {
         url: prUrl,
         prdItems: confirmedItemId ? [confirmedItemId] : [],
       });
+      if (confirmedItemId) addPrLink(prId, confirmedItemId);
       existingIds.add(prId);
       projectAdded++;
 

package/engine/lifecycle.js

@@ -519,7 +519,7 @@ function syncPrsFromOutput(output, agentId, meta, config) {
           const resultText = parsed.result;
           const createdPattern = /(?:created|opened|submitted|new PR|PR created)[^\n]*?(?:(?:visualstudio\.com|dev\.azure\.com)[^\s"]*?pullrequest\/(\d+)|github\.com\/[^\s"]*?\/pull\/(\d+))/gi;
           while ((match = createdPattern.exec(resultText)) !== null) prMatches.add(match[1] || match[2]);
-          const createdIdPattern = /(?:created|opened|submitted|new)\s+PR[# -]*(\d{5,})/gi;
+          const createdIdPattern = /(?:created|opened|submitted|new)\s+PR[# -]*(\d{1,})/gi;
           while ((match = createdIdPattern.exec(resultText)) !== null) prMatches.add(match[1]);
         }
       } catch {}
@@ -586,6 +586,7 @@ function syncPrsFromOutput(output, agentId, meta, config) {
       status: 'active',
       created: e.dateStamp(),
       url: targetProject.prUrlBase ? targetProject.prUrlBase + prId : '',
+      prdItems: meta?.item?.id ? [meta.item.id] : [],
       sourcePlan: meta?.item?.sourcePlan || '',
       itemType: meta?.item?.itemType || ''
     });

package/engine.js

@@ -1211,17 +1211,22 @@ function writeInboxAlert(slug, content) {
   } catch {}
 }
 
-// Reconciles work items against known PRs via exact prdItems match only.
-// reconcilePrs (ado.js) and syncPrsFromOutput (lifecycle.js) are responsible for
-// correctly populating prdItems on PRs — this function just reads that linkage.
+// Reconciles work items against known PRs.
+// Primary linkage comes from prdItems in pull-requests.json; fallback linkage
+// uses engine/pr-links.json so matching does not depend on branch/title parsing.
 // onlyIds: if provided, only items whose ID is in this Set are eligible.
 function reconcileItemsWithPrs(items, allPrs, { onlyIds } = {}) {
+  const prLinks = shared.getPrLinks();
   let reconciled = 0;
   for (const wi of items) {
     if (wi.status !== 'pending' || wi._pr) continue;
     if (onlyIds && !onlyIds.has(wi.id)) continue;
 
-    const exactPr = allPrs.find(pr => (pr.prdItems || []).includes(wi.id));
+    let exactPr = allPrs.find(pr => (pr.prdItems || []).includes(wi.id));
+    if (!exactPr) {
+      const linkedPrId = Object.keys(prLinks).find(prId => prLinks[prId] === wi.id);
+      if (linkedPrId) exactPr = allPrs.find(pr => pr.id === linkedPrId) || { id: linkedPrId };
+    }
     if (exactPr) {
       wi.status = 'done';
       wi._pr = exactPr.id;

package/package.json

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