@yemi33/minions 0.1.312 → 0.1.314

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.314 (2026-04-03)
+
+### Fixes
+-  show reviewer names in dashboard Signed Off By column
+
+### Other
+- perf: CC message handling — debounce localStorage, cap array, batch scroll
+
 ## 0.1.312 (2026-04-03)
 
 ### Fixes

package/dashboard/js/command-center.js

@@ -72,13 +72,20 @@ function ccRestoreMessages() {
   } catch {}
 }
 
+let _ccSaveDebounce = null;
 function ccSaveState() {
-  try {
-    if (_ccSessionId) localStorage.setItem('cc-session-id', _ccSessionId);
-    // Keep last 30 messages for display
-    const toSave = _ccMessages.slice(-30);
-    localStorage.setItem('cc-messages', JSON.stringify(toSave));
-  } catch { /* localStorage might be full */ }
+  // Trim in-memory array to prevent unbounded growth
+  if (_ccMessages.length > 100) _ccMessages = _ccMessages.slice(-100);
+  // Debounce localStorage writes — no need to serialize on every message during streaming
+  if (_ccSaveDebounce) return;
+  _ccSaveDebounce = setTimeout(() => {
+    _ccSaveDebounce = null;
+    try {
+      if (_ccSessionId) localStorage.setItem('cc-session-id', _ccSessionId);
+      const toSave = _ccMessages.slice(-30);
+      localStorage.setItem('cc-messages', JSON.stringify(toSave));
+    } catch { /* localStorage might be full */ }
+  }, 500);
 }
 
 function ccUpdateSessionIndicator() {
@@ -104,7 +111,7 @@ function ccAddMessage(role, html, skipSave) {
     (isUser ? 'background:var(--blue);color:#fff;align-self:flex-end' : 'background:var(--surface2);color:var(--text);align-self:flex-start;border:1px solid var(--border);position:relative');
   div.innerHTML = (isAssistant && !html.includes('color:var(--red)') && !html.includes('cc-queued-pill') ? llmCopyBtn() : '') + html;
   el.appendChild(div);
-  el.scrollTop = el.scrollHeight;
+  requestAnimationFrame(() => { el.scrollTop = el.scrollHeight; });
   if (!skipSave) {
     _ccMessages.push({ role, html });
     ccSaveState();

package/dashboard/js/render-prs.js

@@ -24,7 +24,7 @@ function prRow(pr) {
     '<td><span class="pr-agent">' + escHtml(pr.agent || '—') + '</span></td>' +
     '<td><span class="pr-branch">' + escHtml(pr.branch || '—') + '</span></td>' +
     '<td><span class="pr-badge ' + reviewClass + '">' + escHtml(reviewLabel) + '</span></td>' +
-    '<td>' + (sq.reviewer && sq.status !== 'waiting' ? '<span class="pr-agent" title="' + escHtml(sq.note || '') + '">' + escHtml(sq.reviewer) + '</span>' : sq.reviewer && sq.status === 'waiting' ? '<span class="pr-agent" style="color:var(--muted)" title="Vote pending confirmation">' + escHtml(sq.reviewer) + '…</span>' : '<span style="color:var(--muted);font-size:11px">—</span>') + '</td>' +
+    '<td>' + (sq.reviewer && sq.status !== 'waiting' ? '<span class="pr-agent" title="' + escHtml(sq.note || '') + '">' + escHtml(sq.reviewer) + '</span>' : sq.reviewer && sq.status === 'waiting' ? '<span class="pr-agent" style="color:var(--muted)" title="Vote pending confirmation">' + escHtml(sq.reviewer) + '…</span>' : pr.reviewedBy && pr.reviewedBy.length ? '<span class="pr-agent">' + escHtml(pr.reviewedBy.join(', ')) + '</span>' : '<span style="color:var(--muted);font-size:11px">—</span>') + '</td>' +
     '<td><span class="pr-badge ' + buildClass + '">' + escHtml(buildLabel) + '</span></td>' +
     '<td><span class="pr-badge ' + statusClass + '">' + escHtml(statusLabel) + '</span></td>' +
     '<td><span class="pr-date">' + escHtml(pr.created || '—') + '</span></td>' +

package/engine/ado.js

@@ -151,6 +151,19 @@ async function pollPrStatus(config) {
       else newReviewStatus = 'pending';
     }
 
+    // Store human reviewer names who approved or requested changes
+    const reviewedBy = reviewers
+      .filter(r => r.vote >= 5 || r.vote === -10)
+      .map(r => r.displayName)
+      .filter(Boolean);
+    // Fallback: if PR was merged and no decisive votes, use completedBy
+    if (!reviewedBy.length && newStatus === 'merged' && prData.closedBy?.displayName) {
+      reviewedBy.push(prData.closedBy.displayName);
+    }
+    if (JSON.stringify(pr.reviewedBy || []) !== JSON.stringify(reviewedBy)) {
+      pr.reviewedBy = reviewedBy; updated = true;
+    }
+
     if (pr.reviewStatus !== newReviewStatus) {
       log('info', `PR ${pr.id} reviewStatus: ${pr.reviewStatus} → ${newReviewStatus}`);
       pr.reviewStatus = newReviewStatus;

package/engine/github.js

@@ -154,6 +154,19 @@ async function pollPrStatus(config) {
       }
       const states = [...latestByUser.values()];
 
+      // Store human reviewer names who approved or requested changes
+      const reviewedBy = [...latestByUser.entries()]
+        .filter(([, state]) => state === 'APPROVED' || state === 'CHANGES_REQUESTED')
+        .map(([user]) => user)
+        .filter(Boolean);
+      // Fallback: if PR was merged and no decisive reviews, use merged_by
+      if (!reviewedBy.length && prData.merged && prData.merged_by?.login) {
+        reviewedBy.push(prData.merged_by.login);
+      }
+      if (JSON.stringify(pr.reviewedBy || []) !== JSON.stringify(reviewedBy)) {
+        pr.reviewedBy = reviewedBy; updated = true;
+      }
+
       let newReviewStatus = pr.reviewStatus || 'pending';
       if (states.some(s => s === 'CHANGES_REQUESTED')) newReviewStatus = 'changes-requested';
       else if (states.some(s => s === 'APPROVED')) newReviewStatus = 'approved';

package/package.json

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