@yemi33/minions 0.1.2274 → 0.1.2275

package/engine/queries.js

@@ -1977,6 +1977,18 @@ function getWorkItems(config, opts) {
   // Use snapshot — sync access; cold start before any async warm returns [].
   // Best-effort enrichment for work item _artifacts.notes, not correctness-critical.
   const _kbEntries = getKnowledgeBaseEntriesSnapshot();
+  // PERF (dashboard event-loop freeze fix): the notes-linkage match below is
+  // `entry contains agentId AND entry contains itemId`. Filtering the FULL
+  // _kbEntries (can be ~10k) and _archiveFiles (~2k) per work item is O(items ×
+  // files) — on a busy instance that's millions of String.includes() per
+  // /api/work-items call, run synchronously, which hard-freezes the single
+  // dashboard event loop. Bucket the two large collections by agentId ONCE per
+  // distinct agent (lazily, like _agentDirCache above), then each item only
+  // scans its agent's much-smaller bucket for the itemId. Exact same match
+  // semantics; O(agents × files) one-time + O(items × bucket) instead of
+  // O(items × files). _inboxFiles stays a direct filter (tiny — a handful).
+  const _kbByAgent = {};      // agentId → kb entries whose source includes agentId
+  const _archiveByAgent = {}; // agentId → archive filenames that include agentId
   // P-34fa5d79 — pre-build the wiId → notes map so each item's _notes lookup
   // is O(1) instead of re-scanning inbox+archive per item.
   const _notesByWi = _buildNotesByWiMap();
@@ -1996,14 +2008,25 @@ function getWorkItems(config, opts) {
       // Notes: inbox → KB (via source field) → archive (fallback if KB was swept)
       const itemId = item.id || '___';
       const matchInbox = _inboxFiles.filter(f => f.includes(agentId) && f.includes(itemId));
-      const matchKb = _kbEntries.filter(kb => kb.source && kb.source.includes(agentId) && kb.source.includes(itemId));
+      // Lazily bucket KB by agentId (computed once per distinct agent), then
+      // filter the small bucket by itemId — same result as filtering all of
+      // _kbEntries by (agentId AND itemId), without the per-item full scan.
+      if (_kbByAgent[agentId] === undefined) {
+        _kbByAgent[agentId] = _kbEntries.filter(kb => kb.source && kb.source.includes(agentId));
+      }
+      const matchKb = _kbByAgent[agentId].filter(kb => kb.source.includes(itemId));
       const allNotes = [
         ...matchInbox,
         ...matchKb.map(kb => 'kb:' + kb.cat + '/' + kb.file),
       ];
-      // Archive fallback — only if nothing found in inbox or KB
+      // Archive fallback — only if nothing found in inbox or KB. Same agentId
+      // bucketing as KB so the (potentially large) archive list isn't fully
+      // re-scanned per item.
       if (allNotes.length === 0) {
-        const matchArchive = _archiveFiles.filter(f => f.includes(agentId) && f.includes(itemId));
+        if (_archiveByAgent[agentId] === undefined) {
+          _archiveByAgent[agentId] = _archiveFiles.filter(f => f.includes(agentId));
+        }
+        const matchArchive = _archiveByAgent[agentId].filter(f => f.includes(itemId));
         for (const f of matchArchive) allNotes.push('archive:' + f);
       }
       if (allNotes.length > 0) arts.notes = _mergeArtifactNotes(arts.notes, allNotes);

package/package.json

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