@yemi33/minions 0.1.430 → 0.1.431

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.430 (2026-04-06)
+## 0.1.431 (2026-04-06)
 
 ### Features
 -  version check interval on settings page
@@ -11,6 +11,7 @@
 - Convert remaining lifecycle.js safeWrite calls to mutateJsonFileLocked
 
 ### Fixes
+-  PR dedup prefers merged/abandoned over active regardless of order
 -  extract skills from inbox notes, not just agent stdout
 -  deduplicate PR entries during poll write-back
 -  agent inbox notes include work item ID and time in filename

package/engine/ado.js

@@ -108,12 +108,16 @@ async function forEachActivePr(config, token, callback) {
           if (idx >= 0) currentPrs[idx] = updatedPr;
           // Don't push if not found — it was deleted by another writer, respect that
         }
-        // Remove duplicates (keep last occurrence which has latest status)
-        const seen = new Set();
-        return currentPrs.filter((p, i, arr) => {
-          const lastIdx = arr.findLastIndex(x => x.id === p.id);
-          return lastIdx === i;
-        });
+        // Remove duplicates — prefer merged/abandoned over active
+        const bestById = new Map();
+        const statusRank = { merged: 3, abandoned: 2, closed: 2, active: 1 };
+        for (const p of currentPrs) {
+          const existing = bestById.get(p.id);
+          if (!existing || (statusRank[p.status] || 0) > (statusRank[existing.status] || 0)) {
+            bestById.set(p.id, p);
+          }
+        }
+        return [...bestById.values()];
       }, { defaultValue: [] });
       totalUpdated += projectUpdated;
     }

package/engine/github.js

@@ -77,12 +77,16 @@ async function forEachActiveGhPr(config, callback) {
           const idx = currentPrs.findIndex(p => p.id === updatedPr.id);
           if (idx >= 0) currentPrs[idx] = updatedPr;
         }
-        // Remove duplicates (keep last occurrence which has latest status)
-        const seen = new Set();
-        return currentPrs.filter((p, i, arr) => {
-          const lastIdx = arr.findLastIndex(x => x.id === p.id);
-          return lastIdx === i;
-        });
+        // Remove duplicates — prefer merged/abandoned over active
+        const bestById = new Map();
+        const statusRank = { merged: 3, abandoned: 2, closed: 2, active: 1 };
+        for (const p of currentPrs) {
+          const existing = bestById.get(p.id);
+          if (!existing || (statusRank[p.status] || 0) > (statusRank[existing.status] || 0)) {
+            bestById.set(p.id, p);
+          }
+        }
+        return [...bestById.values()];
       }, { defaultValue: [] });
       totalUpdated += projectUpdated;
     }

package/package.json

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