@yemi33/minions 0.1.429 → 0.1.431

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.429 (2026-04-06)
+## 0.1.431 (2026-04-06)
 
 ### Features
 -  version check interval on settings page
@@ -11,6 +11,8 @@
 - 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
 -  Notes & KB sidebar notifies on inbox changes and notes.md edits

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/engine/lifecycle.js

@@ -1358,7 +1358,21 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
   if (type === WORK_TYPE.REVIEW) updatePrAfterReview(agentId, meta?.pr, meta?.project);
   if (type === WORK_TYPE.FIX) updatePrAfterFix(meta?.pr, meta?.project, meta?.source);
   checkForLearnings(agentId, config.agents[agentId], dispatchItem.task);
-  if (effectiveSuccess) extractSkillsFromOutput(stdout, agentId, dispatchItem, config);
+  if (effectiveSuccess) {
+    extractSkillsFromOutput(stdout, agentId, dispatchItem, config);
+    // Also scan inbox notes for skill blocks — agents often write skills to inbox, not stdout
+    try {
+      const today = dateStamp();
+      const inboxDir = path.join(MINIONS_DIR, 'notes', 'inbox');
+      const inboxFiles = shared.safeReadDir(inboxDir).filter(f => f.includes(agentId) && f.includes(today));
+      for (const f of inboxFiles) {
+        const content = shared.safeRead(path.join(inboxDir, f));
+        if (content && content.includes('```skill')) {
+          extractSkillsFromOutput(content, agentId, dispatchItem, config);
+        }
+      }
+    } catch {}
+  }
   const finalResult = effectiveSuccess ? DISPATCH_RESULT.SUCCESS : DISPATCH_RESULT.ERROR;
   updateAgentHistory(agentId, dispatchItem, finalResult);
   // Don't count auto-retries as errors in metrics — only count final outcomes

package/package.json

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