@yemi33/minions 0.1.671 → 0.1.672

package/CHANGELOG.md

@@ -1,11 +1,12 @@
 # Changelog
 
-## 0.1.671 (2026-04-09)
+## 0.1.672 (2026-04-09)
 
 ### Features
 -  populate _artifacts on work items for detail modal
 
 ### Fixes
+-  cache dir listings in artifact population, make notes clickable
 - surface blocking tool call state in dashboard (closes #619) (#634)
 - interrupt blocking tool calls immediately on steering (closes #627) (#632)
 - reduce Bash blocking grace and errored task dedup window (closes #593) (#630)

package/dashboard/js/render-work-items.js

@@ -457,8 +457,9 @@ function openWorkItemDetail(id) {
   if (arts.prd) artPills += '<span onclick="planView(\'' + escHtml(arts.prd) + '\')" style="' + pillStyle + '">📄 PRD</span> ';
   if (arts.sourcePlan) artPills += '<span onclick="planView(\'' + escHtml(arts.sourcePlan) + '\')" style="' + pillStyle + '">📋 Source Plan</span> ';
   if (arts.notes && arts.notes.length > 0) arts.notes.forEach(function(n) {
-    var noteLabel = (n && typeof n === 'object') ? (n.id || n.file || 'note').slice(0, 30) : String(n || 'note').replace(/\.md$/, '').slice(0, 30);
-    artPills += '<span onclick="closeModal();switchPage(\'inbox\')" style="' + pillStyle + '">📝 ' + escHtml(noteLabel) + '</span> ';
+    var noteFile = (n && typeof n === 'object') ? (n.file || n) : String(n || '');
+    var noteLabel = noteFile.replace(/\.md$/, '').slice(0, 30);
+    artPills += '<span onclick="openInboxNote(\'' + escHtml(noteFile) + '\')" style="' + pillStyle + '">📝 ' + escHtml(noteLabel) + '</span> ';
   });
   if (arts.skills && arts.skills.length > 0) arts.skills.forEach(function(s) { artPills += '<span onclick="openSkill(\'' + escHtml(s) + '\',\'minions\',\'\')" style="' + pillStyle + '">⚙ ' + escHtml(s) + '</span> '; });
   if (artPills) html += field('Artifacts', '<div style="display:flex;flex-wrap:wrap;gap:4px">' + artPills + '</div>');
@@ -501,4 +502,16 @@ function viewAgentOutput(logPath) {
     });
 }
 
-window.MinionsWork = { wiRow, renderWorkItems, editWorkItem, submitWorkItemEdit, deleteWorkItem, archiveWorkItem, toggleWorkItemArchive, retryWorkItem, wiPrev, wiNext, feedbackWorkItem, submitFeedback, openCreateWorkItemModal, openWorkItemDetail, openAllWorkItems, viewAgentOutput };
+function openInboxNote(filename) {
+  // Find in inboxData by filename, or open directly via fetch
+  var idx = (inboxData || []).findIndex(function(item) { return item.name === filename; });
+  if (idx >= 0) {
+    openModal(idx);
+  } else {
+    // Fallback: switch to inbox page
+    closeModal();
+    switchPage('inbox');
+  }
+}
+
+window.MinionsWork = { wiRow, renderWorkItems, editWorkItem, submitWorkItemEdit, deleteWorkItem, archiveWorkItem, toggleWorkItemArchive, retryWorkItem, wiPrev, wiNext, feedbackWorkItem, submitFeedback, openCreateWorkItemModal, openWorkItemDetail, openAllWorkItems, viewAgentOutput, openInboxNote };

package/engine/queries.js

@@ -707,21 +707,23 @@ function getWorkItems(config) {
   }
 
   // Populate _artifacts for the work item detail modal
+  // Cache directory listings to avoid N × readdirSync
+  const _agentDirCache = {};
+  const _inboxFiles = safeReadDir(INBOX_DIR);
   for (const item of allItems) {
     const arts = {};
     const agentId = item.dispatched_to || item.agent;
     if (agentId) {
-      // Output log — find most recent matching dispatch output
-      const agentDir = path.join(MINIONS_DIR, 'agents', agentId);
-      try {
-        const files = safeReadDir(agentDir).filter(f => f.startsWith('output-') && f.includes(item.id?.slice(-8) || '___') && f.endsWith('.log'));
-        if (files.length > 0) arts.outputLog = agentId + '/' + files[files.length - 1];
-      } catch {}
-      // Inbox notes created by this agent for this task
-      try {
-        const inboxFiles = safeReadDir(INBOX_DIR).filter(f => f.includes(agentId) && (f.includes(item.id || '___') || f.includes(item.id?.slice(-8) || '___')));
-        if (inboxFiles.length > 0) arts.notes = inboxFiles;
-      } catch {}
+      const idSuffix = item.id?.slice(-8) || '___';
+      // Output log
+      if (!_agentDirCache[agentId]) {
+        _agentDirCache[agentId] = safeReadDir(path.join(MINIONS_DIR, 'agents', agentId)).filter(f => f.startsWith('output-') && f.endsWith('.log'));
+      }
+      const matchLog = _agentDirCache[agentId].filter(f => f.includes(idSuffix));
+      if (matchLog.length > 0) arts.outputLog = agentId + '/' + matchLog[matchLog.length - 1];
+      // Inbox notes
+      const matchNotes = _inboxFiles.filter(f => f.includes(agentId) && (f.includes(item.id || '___') || f.includes(idSuffix)));
+      if (matchNotes.length > 0) arts.notes = matchNotes;
     }
     if (item.branch) arts.branch = item.branch;
     if (item.sourcePlan) arts.sourcePlan = item.sourcePlan;

package/package.json

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