@yemi33/minions 0.1.1 → 0.1.3

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 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/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 {}

package/package.json

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

package/routing.md

@@ -0,0 +1,32 @@
+# Work Routing
+
+How the engine decides who handles what. Parsed by engine.js — keep the table format exact.
+
+## Routing Table
+
+<!-- FORMAT: | work_type | preferred_agent | fallback | -->
+| Work Type | Preferred | Fallback |
+|-----------|-----------|----------|
+| implement | dallas | ralph |
+| implement:large | rebecca | dallas |
+| review | ripley | lambert |
+| fix | _author_ | dallas |
+| plan | ripley | rebecca |
+| plan-to-prd | lambert | rebecca |
+| explore | ripley | rebecca |
+| test | dallas | ralph |
+| ask | ripley | rebecca |
+| verify | dallas | ralph |
+
+Notes:
+- `_author_` means route to the PR author
+- `implement:large` is for items with `estimated_complexity: "large"`
+- Engine falls back to any idle agent if both preferred and fallback are busy
+
+## Rules
+
+1. **Eager by default** — spawn all agents who can start work, not one at a time
+2. **No self-review** — author cannot review their own PR
+3. **Exploration gates implementation** — when exploring, finish before implementing
+4. **Implementation informs PRD** — Lambert reads build summaries before writing PRD
+5. **All rules in `notes.md` apply** — engine injects them into every playbook