@yemi33/minions 0.1.709 → 0.1.711

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.711 (2026-04-09)
+
+### Fixes
+-  modal derivePlanStatus was called with wrong args, plans not cached
+
+## 0.1.710 (2026-04-09)
+
+### Fixes
+-  modal shows Execute for completed plans, missing status labels
+
 ## 0.1.709 (2026-04-09)
 
 ### Fixes

package/dashboard/js/render-plans.js

@@ -56,6 +56,7 @@ async function _submitCreatePlan() {
 async function refreshPlans() {
   try {
     const plans = await fetch('/api/plans').then(r => r.json());
+    window._lastPlans = plans; // Store globally so plan modal can access
     renderPlans(plans);
   } catch (e) { console.error('plans refresh:', e.message); }
 }
@@ -434,15 +435,18 @@ function _renderPlanModal(normalizedFile, raw, lastMod) {
   const isMdPlan = normalizedFile.endsWith('.md');
   let planStatus = '';
   try { if (normalizedFile.endsWith('.json')) planStatus = JSON.parse(raw).status || ''; } catch {}
-  // Modal buttons mirror card logic — derive effectiveStatus the same way
-  const allPlans = window._lastStatus?.plans || [];
+  // Modal buttons mirror card logic — use cached plans data (from /api/plans, not /api/status)
+  const allPlans = window._lastPlans || [];
   const cardPlan = allPlans.find(p => p.file === normalizedFile || p.sourcePlan === normalizedFile || (p.file?.endsWith('.json') && p.file === normalizedFile));
   // Also check for linked PRD by searching all plans for a PRD whose source_plan matches this file
   const linkedPrd = allPlans.find(p => p.sourcePlan === normalizedFile && p.format === 'prd');
-  const effectiveStatus = cardPlan ? derivePlanStatus(cardPlan) : (linkedPrd ? derivePlanStatus(linkedPrd) : (planStatus || 'active'));
   const prdFile = cardPlan?.file?.endsWith('.json') ? cardPlan.file : (linkedPrd?.file || '');
+  const prdJsonStatus = linkedPrd?.status || (cardPlan?.file?.endsWith('.json') ? cardPlan.status : '') || planStatus || '';
+  const allWi = window._lastWorkItems || [];
+  const effectiveStatus = (prdFile || normalizedFile) ? derivePlanStatus(prdFile, normalizedFile, prdJsonStatus, allWi) : (planStatus || 'active');
   const isArchived = !!(cardPlan?.archived || linkedPrd?.archived);
-  const isDraft = isMdPlan && !prdFile;
+  const isCompleted = effectiveStatus === 'completed';
+  const isDraft = isMdPlan && !prdFile && !isCompleted;
   const isNeedsAction = (effectiveStatus === 'awaiting-approval' || effectiveStatus === 'paused') && !isArchived;
 
   const bs = 'font-size:10px;padding:2px 10px'; // button style
@@ -467,6 +471,14 @@ function _renderPlanModal(normalizedFile, raw, lastMod) {
   if (effectiveStatus === 'dispatched' && prdFile && !isArchived) {
     modalActions += '<button class="pr-pager-btn" style="' + bs + ';color:var(--yellow)" onclick="planPause(\'' + escHtml(prdFile) + '\',this)">Pause</button> ';
   }
+  // Completed label
+  if (isCompleted && !isArchived) {
+    modalActions += '<span style="' + bs + ';color:var(--green);font-weight:600">Completed</span> ';
+  }
+  // In progress label
+  if (effectiveStatus === 'dispatched') {
+    modalActions += '<span style="' + bs + ';color:var(--blue)">In Progress</span> ';
+  }
   // Verify / Verified badge
   const modalVerifyWi = (window._lastWorkItems || []).find(w => w.itemType === 'verify' && w.sourcePlan === (prdFile || normalizedFile));
   if (effectiveStatus === 'completed' && prdFile && !isArchived && !modalVerifyWi) {

package/package.json

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