@yemi33/minions 0.1.915 → 0.1.917

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.915 (2026-04-13)
+## 0.1.917 (2026-04-13)
 
 ### Features
+- CC tab unread dot + reopened badge on work items
 - fix dep re-merge failure on retry with existing commits (#977)
 - audit and harden log buffering implementation (#971)
 - replace magic string 'active' with PR_STATUS.ACTIVE in lifecycle.js (#969)
@@ -17,6 +18,7 @@
 - add red dot notification on CC tab when response completes (#934) (#946)
 
 ### Fixes
+- PRD item status stuck at dispatched when fix completes (#989)
 - dep merge ancestor pruning + pre-flight simulation (#958) (#979)
 - add MAX_TURNS failure class and fix enum count test (#983)
 - prevent Create Plan from meeting saving doc-chat context bleed (#980)

package/dashboard/js/command-center.js

@@ -90,6 +90,8 @@ function toggleCommandCenter() {
     // Ensure at least one tab exists
     if (_ccTabs.length === 0) ccNewTab(true);
     clearNotifBadge(document.getElementById('cc-toggle-btn'));
+    var activeTabOnOpen = _ccActiveTab();
+    if (activeTabOnOpen) activeTabOnOpen._unread = false;
     ccRenderTabBar();
     document.getElementById('cc-input').focus();
     ccRestoreMessages();
@@ -129,6 +131,7 @@ function ccSwitchTab(id) {
   if (!tab) return;
   // If there is an active request, keep it running but switch view
   _ccActiveTabId = id;
+  tab._unread = false;
   var el = document.getElementById('cc-messages');
   el.innerHTML = '';
   // Re-render messages from the tab's data
@@ -247,6 +250,7 @@ function ccRenderTabBar() {
     var isActive = t.id === _ccActiveTabId;
     html += '<div class="cc-tab' + (isActive ? ' active' : '') + (t._sending ? ' working' : '') + '" onclick="ccSwitchTab(\'' + t.id + '\')" title="' + escHtml(t.title) + '">';
     html += '<span class="cc-tab-text">' + escHtml(t.title) + '</span>';
+    if (t._unread) html += '<span class="notif-badge done"></span>';
     html += '<span class="cc-tab-close" onclick="event.stopPropagation();ccCloseTab(\'' + t.id + '\')">&times;</span>';
     html += '</div>';
   }
@@ -661,6 +665,8 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
   } finally {
     if (activeTab) { activeTab._sending = false; activeTab._abortController = null; activeTab._429retries = 0; delete activeTab._streamedText; delete activeTab._toolsUsed; delete activeTab._sendStartedAt; }
     _ccSending = (_ccTabs.some(function(t) { return t._sending; }));
+    // Mark tab unread if response completed on a background tab or while drawer is closed
+    if (activeTab && !_wasAborted && (activeTab.id !== _ccActiveTabId || !_ccOpen)) activeTab._unread = true;
     ccRenderTabBar();
     try { clearInterval(phaseTimer); } catch { /* may not be defined if error before reader */ }
     try { localStorage.removeItem('cc-sending'); } catch {}

package/dashboard/js/render-work-items.js

@@ -48,6 +48,7 @@ function wiRow(item) {
     '<td>' + typeBadge(item.type) + '</td>' +
     '<td>' + priBadge(item.priority) + '</td>' +
     '<td>' + statusBadge(item.status || 'pending') +
+      (item._reopened ? ' <span style="font-size:9px;color:var(--purple);margin-left:4px" title="This item was reopened from a previously completed state">reopened</span>' : '') +
       (item._pendingReason && item.status === 'pending' && item._pendingReason !== 'already_dispatched' ? ' <span style="font-size:9px;color:var(--muted);margin-left:4px" title="Pending reason: ' + escHtml(item._pendingReason) + '">' + escHtml(item._pendingReason.replace(/_/g, ' ')) + '</span>' : '') +
       (item._pendingReason === 'already_dispatched' && item.status === 'pending' ? ' <span style="font-size:9px;color:var(--blue);margin-left:4px" title="In dispatch queue, waiting to be assigned">queued</span>' : '') +
       (item._skipReason && item.status === 'pending' ? ' <span style="font-size:9px;color:var(--yellow);margin-left:4px" title="Dispatch blocked: ' + escHtml(item._skipReason) + (item._blockedBy ? ' (by ' + escHtml(item._blockedBy) + ')' : '') + '">' + escHtml(item._skipReason.replace(/_/g, ' ')) + (item._blockedBy ? ' <span style="color:var(--muted)">(' + escHtml(item._blockedBy) + ')</span>' : '') + '</span>' : '') +
@@ -431,7 +432,8 @@ function openWorkItemDetail(id) {
 
   let html = '<div style="display:flex;flex-direction:column;gap:4px;font-size:13px">';
   html += '<div style="display:flex;gap:8px;align-items:center;margin-bottom:8px">' +
-    badge(statusCls, item.status || 'pending') + ' ' +
+    badge(statusCls, item.status || 'pending') +
+    (item._reopened ? ' <span style="font-size:9px;color:var(--purple);margin-left:4px" title="This item was reopened from a previously completed state">reopened</span>' : '') + ' ' +
     '<span class="dispatch-type ' + (item.type || 'implement') + '">' + escHtml(item.type || 'implement') + '</span>' +
     '<span class="prd-item-priority ' + (item.priority || '') + '">' + escHtml(item.priority || 'medium') + '</span>' +
     '</div>';

package/engine/lifecycle.js

@@ -617,6 +617,8 @@ function updateWorkItemStatus(meta, status, reason) {
 }
 
 const _VALID_PRD_STATUSES = new Set([...Object.values(WI_STATUS), 'missing']);
+// (#984) PRD statuses that are stale when the work item is actually done
+const _STALE_PRD_STATUSES = new Set([WI_STATUS.DISPATCHED, WI_STATUS.FAILED, WI_STATUS.PENDING]);
 function syncPrdItemStatus(itemId, status, sourcePlan) {
   if (!itemId) return;
   if (!_VALID_PRD_STATUSES.has(status)) return;
@@ -637,11 +639,12 @@ function syncPrdItemStatus(itemId, status, sourcePlan) {
   } catch (err) { log('warn', `PRD status sync: ${err.message}`); }
 }
 
-// ─── PRD Backward-Scan Reconciliation (#929) ────────────────────────────────
-// Proactive counterpart to syncPrdItemStatus. Scans all active PRDs and promotes
-// "missing" items to "updated" when a done work item already exists for that ID.
-// This catches cases where a PRD regeneration incorrectly sets items to "missing"
-// and no subsequent work item state change triggers the reactive sync.
+// ─── PRD Backward-Scan Reconciliation (#929, #984) ─────────────────────────
+// Proactive counterpart to syncPrdItemStatus. Scans all active PRDs and:
+// 1. Promotes "missing" items to "updated" when a done work item already exists (#929)
+// 2. Promotes stale "dispatched"/"failed"/"pending" items to "done" when the work item
+//    is actually done (#984) — catches cases where fix work items complete with a
+//    different ID than the original PRD feature, leaving the PRD status stale.
 
 function reconcilePrdStatuses(config) {
   if (!fs.existsSync(PRD_DIR)) return;
@@ -674,6 +677,14 @@ function reconcilePrdStatuses(config) {
           modified = true;
           log('info', `PRD backward-scan: promoted ${feature.id} from missing→updated in ${file} (done work item exists)`);
         }
+        // (#984) Stale status: PRD item stuck at dispatched/failed/pending while WI is done —
+        // happens when fix work items complete with a different ID than the original PRD feature
+        else if (_STALE_PRD_STATUSES.has(feature.status) && doneWiById.has(feature.id)) {
+          const prev = feature.status;
+          feature.status = WI_STATUS.DONE;
+          modified = true;
+          log('info', `PRD backward-scan: promoted ${feature.id} from ${prev}→done in ${file} (done work item exists)`);
+        }
       }
 
       if (modified) {
@@ -1790,7 +1801,23 @@ async function runPostCompletionHooks(dispatchItem, agentId, code, stdout, confi
   }
 
   if (type === WORK_TYPE.REVIEW) await updatePrAfterReview(agentId, meta?.pr, meta?.project, config, resultSummary);
-  if (type === WORK_TYPE.FIX) updatePrAfterFix(meta?.pr, meta?.project, meta?.source);
+  if (type === WORK_TYPE.FIX) {
+    updatePrAfterFix(meta?.pr, meta?.project, meta?.source);
+    // (#984) Sync PRD status for PR-linked features: fix work items have a different ID
+    // than the original PRD feature, so syncPrdItemStatus(fixWiId, ...) finds nothing.
+    // Use the PR's prdItems to propagate done status when the original work item is done.
+    if (effectiveSuccess && meta?.pr?.prdItems?.length) {
+      try {
+        const allWis = queries.getWorkItems(config);
+        for (const prdItemId of meta.pr.prdItems) {
+          const wi = allWis.find(w => w.id === prdItemId);
+          if (wi && DONE_STATUSES.has(wi.status) && wi.sourcePlan) {
+            syncPrdItemStatus(prdItemId, WI_STATUS.DONE, wi.sourcePlan);
+          }
+        }
+      } catch (err) { log('warn', `PRD sync after fix: ${err.message}`); }
+    }
+  }
   checkForLearnings(agentId, config.agents[agentId], dispatchItem.task);
   if (effectiveSuccess) {
     extractSkillsFromOutput(stdout, agentId, dispatchItem, config);

package/package.json

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