@yemi33/minions 0.1.770 → 0.1.772

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.772 (2026-04-10)
+
+### Fixes
+-  filter CI bot noise from PR human comment detection
+
+## 0.1.771 (2026-04-10)
+
+### Fixes
+-  clarify timeout.js error reason — "Exited with error" vs "Completed"
+
 ## 0.1.770 (2026-04-10)
 
 ### Features

package/engine/ado.js

@@ -459,6 +459,11 @@ async function pollPrHumanComments(config) {
       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
+        // 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;
 
         const entry = {
           threadId: thread.id,

package/engine/github.js

@@ -441,10 +441,17 @@ async function pollPrHumanComments(config) {
       ...(Array.isArray(reviewComments) ? reviewComments : []).map(c => ({ ...c, _type: 'review' }))
     ];
 
-    // Filter out bot comments and minions's own comments
+    // Filter out bot comments, minions's own comments, and CI noise
     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 login = (c.user?.login || '').toLowerCase();
+      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
       return true;
     });
 

package/engine/timeout.js

@@ -206,7 +206,7 @@ function checkTimeouts(config) {
           safeWrite(outputLogPath, `# Output for dispatch ${item.id}\n# Exit code: ${isSuccess ? 0 : 1}\n# Completed: ${ts()}\n# Detected via output scan\n\n## Result\n${text || '(no text)'}\n`);
         } catch (e) { log('warn', 'parse output result: ' + e.message); }
 
-        completeDispatch(item.id, isSuccess ? DISPATCH_RESULT.SUCCESS : DISPATCH_RESULT.ERROR, 'Completed (detected from output)');
+        completeDispatch(item.id, isSuccess ? DISPATCH_RESULT.SUCCESS : DISPATCH_RESULT.ERROR, isSuccess ? 'Completed (detected from output)' : 'Exited with error (detected from output)');
 
         // Run post-completion hooks via shared helper (async — fire and forget in timeout context)
         const fullLogForHooks = safeRead(liveLogPath) || liveLog;

package/package.json

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