@yemi33/minions 0.1.148 → 0.1.150

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## 0.1.150 (2026-04-01)
+
+### Dashboard
+- dashboard/js/render-plans.js
+
+## 0.1.149 (2026-04-01)
+
+### Engine
+- engine.js
+- engine/meeting.js
+- engine/shared.js
+
+### Other
+- test/unit.test.js
+
 ## 0.1.148 (2026-04-01)
 
 ### Engine

package/dashboard/js/render-plans.js

@@ -172,19 +172,15 @@ function renderPlans(plans) {
     const isArchived = p.archived;
 
     // For .md plans with a linked PRD, use the PRD's status as the authoritative intent
-    // (p.status for .md is 'draft'/'converted'/'active', not the PRD lifecycle status)
     let prdJsonStatus = p.status || 'active';
     if (prdFile && p.format !== 'prd') {
       const linkedPrd = plans.find(pp => pp.file === prdFile && pp.format === 'prd');
       if (linkedPrd) prdJsonStatus = linkedPrd.status || prdJsonStatus;
-      // If the linked PRD was archived, treat the .md plan as completed
       else if (!linkedPrd) {
         const archivedPrd = archivedPlans.find(pp => pp.file === prdFile && pp.format === 'prd');
         if (archivedPrd) prdJsonStatus = 'completed';
       }
     }
-    // 'converted' means plan-to-PRD succeeded — treat as 'approved' for status derivation
-    if (prdJsonStatus === 'converted') prdJsonStatus = prdFile ? 'approved' : 'completed';
 
     // Single source of truth: derive status from work items
     const effectiveStatus = isArchived ? 'completed' : derivePlanStatus(prdFile, p.file, prdJsonStatus, allWi);
@@ -192,14 +188,13 @@ function renderPlans(plans) {
     const statusLabelsMap = {
       'completed': 'Completed', 'in-progress': 'In Progress', 'paused': 'Paused',
       'awaiting-approval': 'Awaiting Approval', 'approved': 'Approved', 'rejected': 'Rejected',
-      'revision-requested': 'Revision Requested', 'has-failures': 'Has Failures', 'active': 'Active',
-      'converted': 'Converted to PRD', 'draft': 'Draft'
+      'revision-requested': 'Revision Requested', 'has-failures': 'Has Failures', 'active': 'Active'
     };
     const label = statusLabelsMap[effectiveStatus] || effectiveStatus;
     const needsAction = (effectiveStatus === 'awaiting-approval' || effectiveStatus === 'paused') && !isArchived;
     const isRevision = effectiveStatus === 'revision-requested';
     const isCompleted = effectiveStatus === 'completed';
-    const isDraft = (p.format === 'draft' || rawStatus === 'draft') && !isCompleted;
+    const isDraft = p.format === 'draft' && !isCompleted;
     // For .md drafts: show Execute only if no PRD exists yet (not already executed)
 
     let actions = '';
@@ -571,7 +566,10 @@ async function planArchive(file, btn) {
     const d = await res.json().catch(() => ({}));
     if (res.ok && d.ok) {
       try { closeModal(); } catch { /* may not be open */ }
-      showToast('cmd-toast', 'Archived' + (file.endsWith('.json') ? ' PRD and source plan' : ''), true);
+      var msg = 'Archived';
+      if (d.archivedSource) msg += ' PRD + source plan (' + d.archivedSource + ')';
+      if (d.cancelledItems) msg += ', cancelled ' + d.cancelledItems + ' pending item(s)';
+      showToast('cmd-toast', msg, true);
       refreshPlans();
       refresh();
     } else {

package/engine/meeting.js

@@ -88,11 +88,11 @@ function discoverMeetingWork(config) {
       const key = `meeting-${meeting.id}-r${round}-${concluder}`;
       if (activeKeys.has(key)) continue;
 
-      const humanNotes = (meeting.humanNotes || []).map(n => '- ' + n).join('\n');
-      const allFindings = Object.entries(meeting.findings || {}).map(([agent, f]) =>
+      const humanNotes = (Array.isArray(meeting.humanNotes) ? meeting.humanNotes : []).map(n => '- ' + n).join('\n');
+      const allFindings = Object.entries(typeof meeting.findings === 'object' && meeting.findings ? meeting.findings : {}).map(([agent, f]) =>
         `### ${agents[agent]?.name || agent}\n\n${f.content || '(no findings)'}`
       ).join('\n\n---\n\n');
-      const allDebate = Object.entries(meeting.debate || {}).map(([agent, d]) =>
+      const allDebate = Object.entries(typeof meeting.debate === 'object' && meeting.debate ? meeting.debate : {}).map(([agent, d]) =>
         `### ${agents[agent]?.name || agent}\n\n${d.content || '(no response)'}`
       ).join('\n\n---\n\n');
 
@@ -184,6 +184,10 @@ function discoverMeetingWork(config) {
 function collectMeetingFindings(meetingId, agentId, roundName, output) {
   const meeting = getMeeting(meetingId);
   if (!meeting) return;
+  if (meeting.status === 'completed' || meeting.status === 'archived') {
+    log('info', `Ignoring late findings from ${agentId} for completed meeting ${meetingId}`);
+    return;
+  }
 
   const { text } = shared.parseStreamJsonOutput(output, { maxTextLength: 50000 });
   const rawContent = (text || '').trim();
@@ -256,11 +260,35 @@ function addMeetingNote(meetingId, note) {
   return meeting;
 }
 
+function _killMeetingDispatches(meetingId) {
+  try {
+    const DISPATCH_PATH = path.join(__dirname, '..', 'engine', 'dispatch.json');
+    const dispatch = safeJson(DISPATCH_PATH) || {};
+    const toKill = (dispatch.active || []).filter(d => d.meta?.meetingId === meetingId);
+    if (toKill.length === 0) return 0;
+    // Remove from active and move to completed
+    shared.mutateJsonFileLocked(DISPATCH_PATH, (dp) => {
+      dp.active = (dp.active || []).filter(d => d.meta?.meetingId !== meetingId);
+      dp.completed = dp.completed || [];
+      for (const d of toKill) {
+        dp.completed.push({ ...d, result: 'error', reason: 'Meeting ended/advanced by human', completed_at: new Date().toISOString() });
+      }
+      if (dp.completed.length > 100) dp.completed = dp.completed.slice(-100);
+      return dp;
+    }, { defaultValue: { pending: [], active: [], completed: [] } });
+    log('info', `Killed ${toKill.length} active meeting dispatch(es) for ${meetingId}`);
+    return toKill.length;
+  } catch (e) { log('warn', 'kill meeting dispatches: ' + e.message); return 0; }
+}
+
 function advanceMeetingRound(meetingId) {
   const meeting = getMeeting(meetingId);
-  if (!meeting || meeting.status === 'completed') return null;
+  if (!meeting || meeting.status === 'completed' || meeting.status === 'archived') return null;
+  _killMeetingDispatches(meetingId);
   if (meeting.status === 'investigating') { meeting.status = 'debating'; meeting.round = 2; }
   else if (meeting.status === 'debating') { meeting.status = 'concluding'; meeting.round = 3; }
+  else if (meeting.status === 'concluding') { meeting.status = 'completed'; meeting.completedAt = new Date().toISOString(); }
+  else return meeting; // no change
   meeting.roundStartedAt = new Date().toISOString();
   saveMeeting(meeting);
   return meeting;
@@ -269,6 +297,7 @@ function advanceMeetingRound(meetingId) {
 function endMeeting(meetingId) {
   const meeting = getMeeting(meetingId);
   if (!meeting) return null;
+  _killMeetingDispatches(meetingId);
   meeting.status = 'completed';
   meeting.completedAt = new Date().toISOString();
   saveMeeting(meeting);
@@ -294,6 +323,7 @@ function unarchiveMeeting(id) {
 }
 
 function deleteMeeting(id) {
+  _killMeetingDispatches(id);
   const filePath = path.join(MEETINGS_DIR, id + '.json');
   if (!fs.existsSync(filePath)) return false;
   fs.unlinkSync(filePath);

package/engine/shared.js

@@ -258,6 +258,7 @@ function gitEnv() {
  * Single source of truth — used by llm.js, consolidation.js, and lifecycle.js.
  */
 function parseStreamJsonOutput(raw, { maxTextLength = 0 } = {}) {
+  if (typeof raw !== 'string') raw = '';
   let text = '';
   let usage = null;
   let sessionId = null;

package/engine.js

@@ -1496,9 +1496,9 @@ function discoverFromWorkItems(config, project) {
           '',
           'A previous agent run timed out but left a checkpoint. Continue from where it left off.',
           '',
-          cpData.completed && cpData.completed.length > 0 ? `### Completed\n${cpData.completed.map(s => '- ' + s).join('\n')}` : '',
-          cpData.remaining && cpData.remaining.length > 0 ? `### Remaining\n${cpData.remaining.map(s => '- ' + s).join('\n')}` : '',
-          cpData.blockers && cpData.blockers.length > 0 ? `### Blockers\n${cpData.blockers.map(s => '- ' + s).join('\n')}` : '',
+          Array.isArray(cpData.completed) && cpData.completed.length > 0 ? `### Completed\n${cpData.completed.map(s => '- ' + s).join('\n')}` : '',
+          Array.isArray(cpData.remaining) && cpData.remaining.length > 0 ? `### Remaining\n${cpData.remaining.map(s => '- ' + s).join('\n')}` : '',
+          Array.isArray(cpData.blockers) && cpData.blockers.length > 0 ? `### Blockers\n${cpData.blockers.map(s => '- ' + s).join('\n')}` : '',
           cpData.branch_state ? `### Branch State\n${cpData.branch_state}` : '',
         ].filter(Boolean).join('\n');
         vars.checkpoint_context = cpSummary;
@@ -1914,9 +1914,9 @@ function discoverCentralWorkItems(config) {
             '',
             'A previous agent run timed out but left a checkpoint. Continue from where it left off.',
             '',
-            cpData.completed && cpData.completed.length > 0 ? `### Completed\n${cpData.completed.map(s => '- ' + s).join('\n')}` : '',
-            cpData.remaining && cpData.remaining.length > 0 ? `### Remaining\n${cpData.remaining.map(s => '- ' + s).join('\n')}` : '',
-            cpData.blockers && cpData.blockers.length > 0 ? `### Blockers\n${cpData.blockers.map(s => '- ' + s).join('\n')}` : '',
+            Array.isArray(cpData.completed) && cpData.completed.length > 0 ? `### Completed\n${cpData.completed.map(s => '- ' + s).join('\n')}` : '',
+            Array.isArray(cpData.remaining) && cpData.remaining.length > 0 ? `### Remaining\n${cpData.remaining.map(s => '- ' + s).join('\n')}` : '',
+            Array.isArray(cpData.blockers) && cpData.blockers.length > 0 ? `### Blockers\n${cpData.blockers.map(s => '- ' + s).join('\n')}` : '',
             cpData.branch_state ? `### Branch State\n${cpData.branch_state}` : '',
           ].filter(Boolean).join('\n');
           vars.checkpoint_context = cpSummary;
@@ -2125,7 +2125,7 @@ function discoverWork(config) {
   for (const item of allWork) {
     addToDispatch(item);
     if (item.meta?.source === 'pr-human-feedback') {
-      clearPendingHumanFeedbackFlag(item.meta.project, item.meta.pr?.id);
+      clearPendingHumanFeedbackFlag(item.meta?.project, item.meta?.pr?.id);
     }
   }
 

package/package.json

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