@yemi33/minions 0.1.728 → 0.1.729

package/CHANGELOG.md

@@ -1,8 +1,10 @@
 # Changelog
 
-## 0.1.728 (2026-04-09)
+## 0.1.729 (2026-04-09)
 
 ### Features
+- playbook improvements — soften CAPS, add context-awareness, restructure decompose output
+- pipeline auto-detect — skip wait/plan stages when artifacts exist
 - replace raw status strings with WI_STATUS constants in dashboard.js
 - Soften playbook CAPS emphasis and add context-awareness guidance
 -  Add structured completion protocol to PR-producing playbooks
@@ -13,6 +15,7 @@
 - make post-verify plan archiving opt-in via autoArchive config
 
 ### Fixes
+-  resolve 3 failing unit tests on PR-661
 -  repair 3 pre-existing test failures in source-pattern tests
 -  merge master, sync dashboard-build jsFiles, bump HTML size limit
 -  resolve 4 pre-existing test failures breaking CI

package/dashboard/js/render-pipelines.js

@@ -401,9 +401,10 @@ function openPipelineDetail(id) {
   document.getElementById('modal-body').innerHTML = html;
   document.getElementById('modal').classList.add('open');
 
-  // Live-poll while modal is open and pipeline has an active run
-  if (activeRun) {
-    _pipelinePollId = id;
+  // Live-poll while modal is open — always poll (not just active runs)
+  // so the modal updates when stages advance, pipelines get triggered, etc.
+  _pipelinePollId = id;
+  if (!_pipelinePollInterval) {
     _pipelinePollInterval = setInterval(function() {
       if (!document.getElementById('modal')?.classList?.contains('open') || _pipelinePollId !== id) {
         _stopPipelinePoll(); return;
@@ -426,12 +427,29 @@ function openPipelineDetail(id) {
 }
 var _pipelinePollHash = '';
 
+/**
+ * Fetch fresh pipeline data and re-render the detail modal immediately.
+ * Used after actions (continue, trigger, abort) to avoid waiting for the 4s poll.
+ */
+async function _refreshPipelineDetail(id) {
+  try {
+    var res = await fetch('/api/pipelines');
+    var d = await res.json();
+    var fresh = (d.pipelines || []).find(function(x) { return x.id === id; });
+    if (fresh) {
+      _pipelinesData = _pipelinesData.map(function(x) { return x.id === id ? fresh : x; });
+      _pipelinePollHash = JSON.stringify(fresh.runs || []);
+      openPipelineDetail(id);
+    }
+  } catch (e) { /* silent — next poll will catch up */ }
+}
+
 async function _triggerPipeline(id, btn) {
   if (btn) { btn.textContent = 'Starting...'; btn.style.pointerEvents = 'none'; btn.style.opacity = '0.6'; }
   try {
     var res = await fetch('/api/pipelines/trigger', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ id: id }) });
     var d = await res.json();
-    if (res.ok) { showToast('cmd-toast', 'Pipeline triggered: ' + (d.runId || ''), true); try { closeModal(); } catch {} refresh(); }
+    if (res.ok) { showToast('cmd-toast', 'Pipeline triggered: ' + (d.runId || ''), true); refresh(); await _refreshPipelineDetail(id); }
     else { if (btn) { btn.textContent = 'Run Now'; btn.style.pointerEvents = ''; btn.style.opacity = ''; } alert('Failed: ' + (d.error || 'unknown')); }
   } catch (e) { if (btn) { btn.textContent = 'Run Now'; btn.style.pointerEvents = ''; btn.style.opacity = ''; } alert('Error: ' + e.message); }
 }
@@ -444,20 +462,8 @@ async function _abortPipeline(id, btn) {
     if (res.ok) {
       var d = await res.json().catch(function() { return {}; });
       showToast('cmd-toast', 'Pipeline aborted — ' + (d.cancelledWorkItems || 0) + ' work items cancelled', true);
-      // Replace abort button with Run Now
-      if (btn) {
-        btn.textContent = 'Run Now';
-        btn.style.color = 'var(--green)';
-        btn.style.borderColor = 'var(--green)';
-        btn.style.pointerEvents = '';
-        btn.style.opacity = '';
-        btn.onclick = function() { _triggerPipeline(id, btn); };
-        // Remove the retrigger button next to it (no longer needed)
-        var next = btn.nextElementSibling;
-        if (next && next.textContent.trim() === 'Retrigger') next.remove();
-      }
       refresh();
-      openPipelineDetail(id);
+      await _refreshPipelineDetail(id);
     } else { var d = await res.json().catch(function() { return {}; }); alert('Abort failed: ' + (d.error || 'unknown')); if (btn) { btn.textContent = 'Abort'; btn.style.pointerEvents = ''; btn.style.opacity = ''; } }
   } catch (e) { alert('Error: ' + e.message); if (btn) { btn.textContent = 'Abort'; btn.style.pointerEvents = ''; btn.style.opacity = ''; } }
 }
@@ -470,8 +476,8 @@ async function _retriggerPipeline(id, btn) {
     var d = await res.json();
     if (res.ok) {
       showToast('cmd-toast', 'Pipeline retriggered: ' + (d.runId || ''), true);
-      if (btn) { btn.textContent = '\u2713 Retriggered'; btn.style.color = 'var(--green)'; }
-      setTimeout(function() { openPipelineDetail(id); }, 1500);
+      refresh();
+      await _refreshPipelineDetail(id);
     } else { alert('Retrigger failed: ' + (d.error || 'unknown')); if (btn) { btn.textContent = 'Retrigger'; btn.style.pointerEvents = ''; btn.style.opacity = ''; } }
   } catch (e) { alert('Error: ' + e.message); if (btn) { btn.textContent = 'Retrigger'; btn.style.pointerEvents = ''; btn.style.opacity = ''; } }
 }
@@ -493,7 +499,8 @@ async function _continuePipeline(id, stageId, btn) {
     if (res.ok) {
       showToast('cmd-toast', 'Stage continued — dispatching next tick', true);
       if (btn) { btn.textContent = '\u2713 Continued'; btn.style.color = 'var(--green)'; btn.style.borderColor = 'var(--green)'; btn.style.opacity = '1'; }
-      setTimeout(function() { openPipelineDetail(id); }, 2000);
+      // Immediate refresh — no waiting for 4s poll
+      await _refreshPipelineDetail(id);
     } else {
       var d = await res.json().catch(function() { return {}; }); alert('Failed: ' + (d.error || 'unknown'));
       if (btn) { btn.textContent = 'Continue'; btn.style.pointerEvents = ''; btn.style.opacity = ''; }

package/engine/pipeline.js

@@ -320,6 +320,17 @@ function _findExistingPlanForMeeting(meetingIds, plansDir) {
   return slugMatch;
 }
 
+// Check if a PRD already exists for a given plan file (plan already converted)
+function _findExistingPrdForPlan(planFile, prdDir) {
+  if (!fs.existsSync(prdDir)) return null;
+  const prdFiles = safeReadDir(prdDir).filter(f => f.endsWith('.json'));
+  for (const pf of prdFiles) {
+    const prd = safeJson(path.join(prdDir, pf));
+    if (prd?.source_plan === planFile) return pf;
+  }
+  return null;
+}
+
 async function executePlanStage(stage, stageState, run, config) {
   const plansDir = path.join(__dirname, '..', 'plans');
   if (!fs.existsSync(plansDir)) fs.mkdirSync(plansDir, { recursive: true });
@@ -334,6 +345,18 @@ async function executePlanStage(stage, stageState, run, config) {
     const existingPlanFile = _findExistingPlanForMeeting(meetingIds, plansDir);
     if (existingPlanFile) {
       log('info', `Pipeline ${run.pipelineId}: reconciling plan stage — adopting existing plan "${existingPlanFile}"`);
+
+      // Check if a PRD already exists for this plan (skip plan-to-prd entirely)
+      const prdDir = path.join(__dirname, '..', 'prd');
+      const existingPrdFile = _findExistingPrdForPlan(existingPlanFile, prdDir);
+      if (existingPrdFile) {
+        log('info', `Pipeline ${run.pipelineId}: PRD "${existingPrdFile}" already exists for plan "${existingPlanFile}" — skipping plan-to-prd`);
+        return {
+          status: PIPELINE_STATUS.RUNNING,
+          artifacts: { plans: [existingPlanFile], workItems: [], prds: [existingPrdFile], prs: [] },
+        };
+      }
+
       // Adopt or create plan-to-prd WI atomically under lock
       let adoptedWiId = wiId;
       mutateWorkItems(wiPath, workItems => {
@@ -563,12 +586,17 @@ function isStageComplete(stage, stageState, run, config) {
 
   switch (stage.type) {
     case STAGE_TYPE.TASK: {
+      // Check root + all project work-items.json (WIs may be moved to project paths)
       const wiPath = path.join(__dirname, '..', 'work-items.json');
       const workItems = safeJson(wiPath) || [];
+      const allProjectWi = shared.getProjects(config).reduce((acc, p) => {
+        return acc.concat(safeJson(shared.projectWorkItemsPath(p)) || []);
+      }, []);
+      const all = [...workItems, ...allProjectWi];
       const ids = artifacts.workItems || [];
       if (ids.length === 0) return false;
       return ids.every(id => {
-        const wi = workItems.find(w => w.id === id);
+        const wi = all.find(w => w.id === id);
         return !wi || wi.status === WI_STATUS.DONE || wi.status === WI_STATUS.FAILED; // missing = treat as done
       });
     }
@@ -725,8 +753,10 @@ async function discoverPipelineWork(config) {
           if (stage.type === STAGE_TYPE.TASK) {
             const wiPath = path.join(__dirname, '..', 'work-items.json');
             const workItems = safeJson(wiPath) || [];
+            const projWi = shared.getProjects(config).reduce((acc, p) => acc.concat(safeJson(shared.projectWorkItemsPath(p)) || []), []);
+            const allWi = [...workItems, ...projWi];
             output = (stageState.artifacts?.workItems || []).map(id => {
-              const wi = workItems.find(w => w.id === id);
+              const wi = allWi.find(w => w.id === id);
               return wi?.resultSummary || wi?.title || id;
             }).join('\n');
           } else if (stage.type === STAGE_TYPE.MEETING) {
@@ -771,7 +801,34 @@ async function discoverPipelineWork(config) {
         }
       }
 
-      if (stageState.status === PIPELINE_STATUS.WAITING_HUMAN) { allComplete = false; continue; }
+      if (stageState.status === PIPELINE_STATUS.WAITING_HUMAN) {
+        // Auto-complete wait stages when the preceding meeting already produced a plan
+        // Common pattern: meeting → wait → plan — if plan exists, nothing to wait for
+        if (stage.type === STAGE_TYPE.WAIT) {
+          const nextPlanStage = stages.find(s =>
+            s.type === STAGE_TYPE.PLAN && (s.dependsOn || []).includes(stage.id)
+          );
+          if (nextPlanStage) {
+            const meetingIds = _findMeetingsInRun(activeRun);
+            if (meetingIds.length > 0) {
+              const plansDir = path.join(__dirname, '..', 'plans');
+              if (fs.existsSync(plansDir)) {
+                const existingPlan = _findExistingPlanForMeeting(meetingIds, plansDir);
+                if (existingPlan) {
+                  updateRunStage(pipeline.id, activeRun.runId, stage.id, {
+                    status: PIPELINE_STATUS.COMPLETED, completedAt: ts(),
+                    output: `Auto-completed: plan "${existingPlan}" already exists for meeting`,
+                  });
+                  stageState.status = PIPELINE_STATUS.COMPLETED;
+                  log('info', `Pipeline ${pipeline.id}: wait stage ${stage.id} auto-completed — plan "${existingPlan}" already exists`);
+                  continue; // re-evaluate as completed on next iteration
+                }
+              }
+            }
+          }
+        }
+        allComplete = false; continue;
+      }
 
       // Check if pending stage is ready to start
       if (stageState.status === PIPELINE_STATUS.PENDING) {
@@ -852,4 +909,5 @@ module.exports = {
   getPipelineRuns, getActiveRun, startRun, updateRunStage, completeRun,
   discoverPipelineWork,
   evaluateCondition, // exported for testing
+  _findMeetingsInRun, _findExistingPlanForMeeting, _findExistingPrdForPlan, // exported for testing
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.728",
+  "version": "0.1.729",
   "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/playbooks/decompose.md

@@ -48,10 +48,13 @@ Write the decomposition result as a JSON code block in your response:
     {
       "id": "{{item_id}}-a",
       "name": "Short descriptive name",
+      "objective": "One-sentence goal: what this sub-task achieves.",
       "description": "What to build, where, and how. Be specific enough that an engineer can implement without further exploration.",
       "estimated_complexity": "small|medium",
       "depends_on": [],
-      "acceptance_criteria": ["Criterion 1", "Criterion 2"]
+      "acceptance_criteria": ["Criterion 1", "Criterion 2"],
+      "expected_output": "What done looks like — the artifact produced (e.g., a PR adding X, a new file at Y, a config change).",
+      "scope_boundaries": ["Out-of-scope item 1", "Out-of-scope item 2"]
     }
   ]
 }

package/playbooks/shared-rules.md

@@ -4,6 +4,8 @@ Your context window may be compacted or summarized mid-task by Claude's automati
 
 ## Engine Rules (apply to all tasks)
 
+**Context compaction:** Your context window may be compacted mid-task by Claude's infrastructure. If you notice your earlier conversation history appears truncated or summarized, this is normal and expected. Do not interpret compaction as a signal to stop early or wrap up. Continue working toward your task objective — all relevant instructions and state remain available.
+
 - Do NOT write to `agents/*/status.json` — the engine manages your status automatically.
 - Do NOT remove worktrees — the engine handles cleanup automatically.
 - Do NOT checkout branches in the main working tree — use worktrees or `git diff`/`git show`.