@yemi33/minions 0.1.149 → 0.1.151

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## 0.1.151 (2026-04-02)
+
+### Engine
+- engine.js
+- engine/lifecycle.js
+
+### Dashboard
+- dashboard/js/command-center.js
+- dashboard/js/render-meetings.js
+
+### Playbooks
+- work-item.md
+
+## 0.1.150 (2026-04-01)
+
+### Dashboard
+- dashboard/js/render-plans.js
+
 ## 0.1.149 (2026-04-01)
 
 ### Engine

package/dashboard/js/command-center.js

@@ -37,6 +37,28 @@ function ccRestoreMessages() {
   for (const msg of _ccMessages) {
     ccAddMessage(msg.role, msg.html, true);
   }
+  // Restore "thinking" indicator if CC was mid-request when page refreshed
+  try {
+    const sendingState = JSON.parse(localStorage.getItem('cc-sending') || 'null');
+    if (sendingState?.sending && (Date.now() - sendingState.startedAt) < 300000) {
+      _ccSending = true;
+      const elapsed = Date.now() - sendingState.startedAt;
+      const thinking = document.createElement('div');
+      thinking.id = 'cc-thinking';
+      thinking.style.cssText = 'padding:8px 12px;border-radius:8px;font-size:11px;color:var(--muted);align-self:flex-start;display:flex;align-items:center;gap:8px';
+      thinking.innerHTML = '<span class="dot-pulse" style="display:inline-flex;gap:3px"><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite"></span><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite;animation-delay:0.2s"></span><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite;animation-delay:0.4s"></span></span> <span id="cc-thinking-text">Still working...</span> <span id="cc-thinking-time" style="font-size:10px;color:var(--border)">' + Math.floor(elapsed / 1000) + 's</span>' +
+        ' <button onclick="ccNewSession()" style="font-size:9px;padding:2px 8px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--red);cursor:pointer">Reset</button>';
+      el.appendChild(thinking);
+      el.scrollTop = el.scrollHeight;
+      // Update timer
+      const startTime = sendingState.startedAt;
+      const restoreTimer = setInterval(function() {
+        var timeEl = document.getElementById('cc-thinking-time');
+        if (!timeEl || !_ccSending) { clearInterval(restoreTimer); return; }
+        timeEl.textContent = Math.floor((Date.now() - startTime) / 1000) + 's';
+      }, 1000);
+    }
+  } catch {}
 }
 
 function ccSaveState() {
@@ -109,6 +131,7 @@ async function ccSend() {
 
 async function _ccDoSend(message, skipUserMsg) {
   _ccSending = true;
+  try { localStorage.setItem('cc-sending', JSON.stringify({ sending: true, startedAt: Date.now() })); } catch {}
 
   if (!skipUserMsg) ccAddMessage('user', escHtml(message));
 
@@ -160,7 +183,9 @@ async function _ccDoSend(message, skipUserMsg) {
     thinking.remove();
 
     if (data.error) {
-      ccAddMessage('assistant', '<span style="color:var(--red)">' + escHtml(data.error) + '</span>');
+      const isBusy = data.error.includes('busy');
+      ccAddMessage('assistant', '<span style="color:var(--red)">' + escHtml(data.error) + '</span>' +
+        (isBusy ? ' <button onclick="ccNewSession()" style="margin-top:4px;padding:3px 10px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--blue);cursor:pointer;font-size:10px">Reset CC</button>' : ''));
       return;
     }
 
@@ -201,6 +226,7 @@ async function _ccDoSend(message, skipUserMsg) {
       '<button id="' + retryId + '" onclick="ccRetryLast()" style="margin-top:6px;padding:4px 12px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--blue);cursor:pointer;font-size:11px">Retry</button>');
   } finally {
     _ccSending = false;
+    try { localStorage.removeItem('cc-sending'); } catch {}
     // Show notification badge on CC button if drawer is closed
     if (!_ccOpen) showNotifBadge(document.getElementById('cc-toggle-btn'));
   }

package/dashboard/js/render-meetings.js

@@ -173,11 +173,11 @@ function _renderMeetingDetail(m) {
       } else {
         html += '<div style="display:flex;gap:8px;border-top:1px solid var(--border);padding-top:8px">' +
           '<input id="meeting-note-input" type="text" placeholder="Add context for all agents..." style="flex:1;padding:6px 8px;background:var(--bg);border:1px solid var(--border);border-radius:var(--radius-sm);color:var(--text);font-size:12px" onkeydown="if(event.key===\'Enter\')_submitMeetingNote(\'' + escHtml(m.id) + '\')">' +
-          '<button onclick="_submitMeetingNote(\'' + escHtml(m.id) + '\')" style="padding:6px 12px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer;font-size:11px">Add Note</button>' +
+          '<button onclick="_submitMeetingNote(\'' + escHtml(m.id) + '\',this)" style="padding:6px 12px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer;font-size:11px">Add Note</button>' +
         '</div>' +
         '<div style="display:flex;gap:8px;margin-top:4px">' +
-          '<button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;color:var(--yellow);border-color:var(--yellow)" onclick="_advanceMeeting(\'' + escHtml(m.id) + '\')">Skip to Next Round</button>' +
-          '<button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_endMeeting(\'' + escHtml(m.id) + '\')">End Meeting</button>' +
+          '<button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;color:var(--yellow);border-color:var(--yellow)" onclick="_advanceMeeting(\'' + escHtml(m.id) + '\',this)">Skip to Next Round</button>' +
+          '<button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_endMeeting(\'' + escHtml(m.id) + '\',this)">End Meeting</button>' +
           '<button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_deleteMeeting(\'' + escHtml(m.id) + '\')">Delete</button>' +
         '</div>';
       }
@@ -269,45 +269,58 @@ async function _submitCreateMeeting() {
   } catch (e) { alert('Error: ' + e.message); openCreateMeetingModal(); }
 }
 
-async function _submitMeetingNote(id) {
+async function _submitMeetingNote(id, btn) {
   const input = document.getElementById('meeting-note-input');
   if (!input?.value?.trim()) return;
   const note = input.value.trim();
   input.value = '';
+  if (btn) { btn.textContent = 'Adding...'; btn.style.pointerEvents = 'none'; btn.style.opacity = '0.6'; }
   try {
     const res = await fetch('/api/meetings/note', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id, note })
     });
-    if (res.ok) openMeetingDetail(id);
+    if (res.ok) { showToast('cmd-toast', 'Note added', true); }
     else { input.value = note; alert('Failed to add note'); }
   } catch (e) { input.value = note; alert('Error: ' + e.message); }
+  if (btn) { btn.textContent = 'Add Note'; btn.style.pointerEvents = ''; btn.style.opacity = ''; }
 }
 
-async function _advanceMeeting(id) {
+async function _advanceMeeting(id, btn) {
   if (!confirm('Skip to next round? Agents that haven\'t finished will be skipped.')) return;
-  showToast('cmd-toast', 'Advancing to next round...', true);
+  if (btn) { btn.textContent = 'Advancing...'; btn.style.pointerEvents = 'none'; btn.style.opacity = '0.6'; }
   try {
-    await fetch('/api/meetings/advance', {
+    const res = await fetch('/api/meetings/advance', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id })
     });
-    wakeEngine();
-    openMeetingDetail(id);
+    if (res.ok) {
+      showToast('cmd-toast', 'Advanced to next round', true);
+      wakeEngine();
+    } else {
+      const d = await res.json().catch(function() { return {}; });
+      alert('Advance failed: ' + (d.error || 'unknown'));
+    }
   } catch (e) { alert('Error: ' + e.message); }
+  if (btn) { btn.textContent = 'Skip to Next Round'; btn.style.pointerEvents = ''; btn.style.opacity = ''; }
 }
 
-async function _endMeeting(id) {
+async function _endMeeting(id, btn) {
   if (!confirm('End this meeting? Current round will be stopped.')) return;
-  try { closeModal(); } catch { /* expected */ }
-  showToast('cmd-toast', 'Meeting ended', true);
+  if (btn) { btn.textContent = 'Ending...'; btn.style.pointerEvents = 'none'; btn.style.opacity = '0.6'; }
   try {
-    await fetch('/api/meetings/end', {
+    const res = await fetch('/api/meetings/end', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id })
     });
-    refresh();
-  } catch (e) { alert('Error: ' + e.message); refresh(); }
+    if (res.ok) {
+      showToast('cmd-toast', 'Meeting ended', true);
+    } else {
+      const d = await res.json().catch(function() { return {}; });
+      alert('End failed: ' + (d.error || 'unknown'));
+      if (btn) { btn.textContent = 'End Meeting'; btn.style.pointerEvents = ''; btn.style.opacity = ''; }
+    }
+  } catch (e) { alert('Error: ' + e.message); if (btn) { btn.textContent = 'End Meeting'; btn.style.pointerEvents = ''; btn.style.opacity = ''; } }
 }
 
 async function _archiveMeeting(id) {

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/lifecycle.js

@@ -1221,24 +1221,24 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
 
   if (isSuccess && meta?.item?.id && !skipDoneStatus) updateWorkItemStatus(meta, 'done', '');
 
-  // Auto-dispatch evaluate work item after implement completes successfully
+  // Auto-dispatch review work item after implement completes successfully
   if (isSuccess && !skipDoneStatus && type === 'implement' && meta?.item?.id) {
-    const evalLoop = config.engine?.evalLoop ?? shared.ENGINE_DEFAULTS.evalLoop;
-    if (evalLoop) {
+    const autoReview = config.engine?.autoReview ?? shared.ENGINE_DEFAULTS.autoReview;
+    if (autoReview) {
       try {
         const wiPath = resolveWiPath(meta);
         if (wiPath) {
           const items = safeJson(wiPath) || [];
-          // Dedup: skip if an evaluate item already exists for this parent
-          const existing = items.find(i => i._evalParentId === meta.item.id && i.type === 'evaluate');
+          // Dedup: skip if a review item already exists for this parent
+          const existing = items.find(i => i._evalParentId === meta.item.id && i.type === 'review');
           if (existing) {
-            log('info', `Eval loop: evaluate item ${existing.id} already exists for ${meta.item.id}, skipping`);
+            log('info', `Eval loop: review item ${existing.id} already exists for ${meta.item.id}, skipping`);
           } else {
             const parentItem = items.find(i => i.id === meta.item.id);
             const evalItem = {
               id: 'W-' + shared.uid(),
-              title: `Evaluate: ${meta.item.title || meta.item.id}`,
-              type: 'evaluate',
+              title: `Review: ${meta.item.title || meta.item.id}`,
+              type: 'review',
               priority: meta.item.priority || 'high',
               status: 'pending',
               created: ts(),
@@ -1263,14 +1263,14 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
     }
   }
 
-  // Evaluate completion: parse verdict and handle eval→fix iteration loop
-  if (isSuccess && type === 'evaluate' && meta?.item?._evalParentId) {
+  // Review completion: parse verdict and handle review→fix iteration loop
+  if (isSuccess && type === 'review' && meta?.item?._evalParentId) {
     try {
       const verdict = parseEvalVerdict(resultSummary || stdout);
-      const evalLoop = config.engine?.evalLoop ?? shared.ENGINE_DEFAULTS.evalLoop;
+      const autoReview = config.engine?.autoReview ?? shared.ENGINE_DEFAULTS.autoReview;
       const maxIter = config.engine?.evalMaxIterations ?? shared.ENGINE_DEFAULTS.evalMaxIterations;
 
-      if (verdict && !verdict.pass && evalLoop) {
+      if (verdict && !verdict.pass && autoReview) {
         const wiPath = resolveWiPath(meta);
         if (wiPath) {
           const items = safeJson(wiPath) || [];

package/engine.js

@@ -1256,8 +1256,8 @@ function discoverFromPrs(config, project) {
       if (!agentId) continue;
 
       const item = buildPrDispatch(agentId, config, project, pr, 'review', {
-        pr_id: pr.id, pr_number: prNumber, pr_title: pr.title || '', pr_branch: pr.branch || '',
-        pr_author: pr.agent || '', pr_url: pr.url || '',
+        pr_id: pr.id, pr_number: prNumber, pr_title: pr.title ? ': ' + pr.title : '', pr_branch: pr.branch || '',
+        branch_name: pr.branch || '', pr_author: pr.agent || '', pr_url: pr.url || '',
       }, `Review PR ${pr.id}: ${pr.title}`, { dispatchKey: key, source: 'pr', pr, branch: pr.branch, project: projMeta });
       if (item) { newWork.push(item); setCooldown(key); }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.149",
+  "version": "0.1.151",
   "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/work-item.md

@@ -40,7 +40,7 @@ Keep branch names lowercase, use hyphens, max 60 chars.
    - title: `feat({{item_id}}): <description>`
 8. **Post implementation notes** as a PR thread comment:
    {{pr_comment_instructions}}
-9. **Add PR to tracker** — append to `{{project_path}}/.minions/pull-requests.json`:
+9. **Add PR to tracker** — append to `{{team_root}}/projects/{{project_name}}/pull-requests.json`:
    ```json
    { "id": "PR-<number>", "title": "...", "agent": "{{agent_name}}", "branch": "...", "reviewStatus": "pending", "status": "active", "created": "<date>", "url": "<pr-url>", "prdItems": ["{{item_id}}"] }
    ```