@yemi33/minions 0.1.775 → 0.1.777

package/CHANGELOG.md

@@ -1,12 +1,18 @@
 # Changelog
 
-## 0.1.775 (2026-04-10)
+## 0.1.777 (2026-04-10)
+
+### Fixes
+-  pass reviewer's actual output to fix agent, not just task label
+
+## 0.1.776 (2026-04-10)
 
 ### Features
 -  auto-rebase dependent PRs after dependency merge
 -  show PR merged/active status on PRD item links
 
 ### Fixes
+-  better agent comment filtering — detect by patterns, not just signature
 -  filter CI bot noise from PR human comment detection
 
 ### Other

package/engine/ado.js

@@ -458,12 +458,18 @@ async function pollPrHumanComments(config) {
     for (const thread of threads) {
       for (const comment of (thread.comments || [])) {
         if (!comment.content || comment.commentType === 'system') continue;
-        if (/\bMinions\s*\(/i.test(comment.content)) continue; // skip minions's own comments
+        const content = comment.content;
+        // Minions agent signatures
+        if (/\bMinions\s*\(/i.test(content)) continue;
+        if (/\b(Review|Fixed|Explored|Verified|Tested)\s+by\s+Minions\b/i.test(content)) continue;
+        if (/\[minions\]/i.test(content)) continue;
         // Skip bot/service account comments
         const authorName = (comment.author?.displayName || '').toLowerCase();
         if (/\b(bot|service|build|pipeline|codecov|sonar)\b/i.test(authorName)) continue;
-        // Skip automated status comments (coverage, build reports)
-        if (/^#{1,3}\s*(Coverage|Build|Test|Deploy|Pipeline)\s*(Report|Status|Result|Summary)/i.test(comment.content)) continue;
+        // Short comments from same author that look agent-generated
+        if (content.length < 500 && /\b(fixed|addressed|resolved|pushed|updated|rebased)\b/i.test(content) && /\b(review|feedback|comment|finding)\b/i.test(content)) continue;
+        // Automated status comments (coverage, build reports)
+        if (/^#{1,3}\s*(Coverage|Build|Test|Deploy|Pipeline)\s*(Report|Status|Result|Summary)/i.test(content)) continue;
 
         const entry = {
           threadId: thread.id,

package/engine/github.js

@@ -441,17 +441,25 @@ async function pollPrHumanComments(config) {
       ...(Array.isArray(reviewComments) ? reviewComments : []).map(c => ({ ...c, _type: 'review' }))
     ];
 
-    // Filter out bot comments, minions's own comments, and CI noise
+    // Filter out bot comments, minions agent comments, and CI noise
+    // Agents post as the PAT owner — detect by signature patterns, not account type
+    const prAuthorLogin = (prData?.user?.login || '').toLowerCase();
     const humanComments = allComments.filter(c => {
       if (c.user?.type === 'Bot') return false;
-      if (/\bMinions\s*\(/i.test(c.body || '')) return false;
-      // Common CI bot usernames
+      const body = c.body || '';
+      // Minions agent signatures (review, fix, explore, etc.)
+      if (/\bMinions\s*\(/i.test(body)) return false;
+      if (/\b(Review|Fixed|Explored|Verified|Tested)\s+by\s+Minions\b/i.test(body)) return false;
+      if (/\[minions\]/i.test(body)) return false;
+      // Comments from the PR author account are likely agent-generated (agents use same PAT)
+      // Only skip if the comment looks automated (short, or has agent patterns)
       const login = (c.user?.login || '').toLowerCase();
+      if (login === prAuthorLogin && body.length < 500 && /\b(fixed|addressed|resolved|pushed|updated|rebased)\b/i.test(body)) return false;
+      // Common CI bot usernames
       if (/\b(bot|codecov|sonar|dependabot|renovate|github-actions|azure-pipelines)\b/i.test(login)) return false;
       // Automated status comments (coverage reports, build badges, etc.)
-      const body = c.body || '';
       if (/^#{1,3}\s*(Coverage|Build|Test|Deploy|Pipeline)\s*(Report|Status|Result|Summary)/i.test(body)) return false;
-      if (/!\[.*\]\(https?:\/\/.*badge/i.test(body)) return false; // badge images
+      if (/!\[.*\]\(https?:\/\/.*badge/i.test(body)) return false;
       return true;
     });
 

package/engine/lifecycle.js

@@ -720,7 +720,7 @@ function syncPrsFromOutput(output, agentId, meta, config) {
 
 // ─── Post-Completion Hooks ──────────────────────────────────────────────────
 
-async function updatePrAfterReview(agentId, pr, project, config) {
+async function updatePrAfterReview(agentId, pr, project, config, resultSummary) {
 
   if (!pr?.id) return;
 
@@ -755,7 +755,7 @@ async function updatePrAfterReview(agentId, pr, project, config) {
     target.minionsReview = {
       reviewer: reviewerName,
       reviewedAt: ts(),
-      note: completedEntry?.task || ''
+      note: resultSummary || completedEntry?.task || ''
     };
     updatedTarget = { ...pr, ...target };
     return prs;
@@ -1592,7 +1592,7 @@ async function runPostCompletionHooks(dispatchItem, agentId, code, stdout, confi
     }
   }
 
-  if (type === WORK_TYPE.REVIEW) await updatePrAfterReview(agentId, meta?.pr, meta?.project, config);
+  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);
   checkForLearnings(agentId, config.agents[agentId], dispatchItem.task);
   if (effectiveSuccess) {

package/package.json

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