@yemi33/minions 0.1.313 → 0.1.314

package/CHANGELOG.md

@@ -1,6 +1,9 @@
 # Changelog
 
-## 0.1.313 (2026-04-03)
+## 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

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.313",
+  "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"