@yemi33/minions 0.1.646 → 0.1.648

package/CHANGELOG.md

@@ -1,8 +1,14 @@
 # Changelog
 
-## 0.1.646 (2026-04-09)
+## 0.1.648 (2026-04-09)
+
+### Fixes
+-  use full slug for plan matching to prevent false-positive collisions
+
+## 0.1.647 (2026-04-09)
 
 ### Features
+-  structured note IDs for inbox notes
 -  pipeline plan stage reconciliation and UX fixes
 -  track notes, plans, and PRDs as work item artifacts
 

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

@@ -454,7 +454,10 @@ function openWorkItemDetail(id) {
   if (arts.plan) artPills += '<span onclick="planView(\'' + escHtml(arts.plan) + '\')" style="' + pillStyle + '">📋 Plan</span> ';
   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) { artPills += '<span onclick="closeModal();switchPage(\'inbox\')" style="' + pillStyle + '">📝 ' + escHtml(n.replace(/\.md$/, '').slice(0, 30)) + '</span> '; });
+  if (arts.notes && arts.notes.length > 0) arts.notes.forEach(function(n) {
+    var noteLabel = typeof n === 'object' ? (n.id || n.file || 'note').slice(0, 30) : String(n).replace(/\.md$/, '').slice(0, 30);
+    artPills += '<span onclick="closeModal();switchPage(\'inbox\')" 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>');
 

package/engine/pipeline.js

@@ -305,7 +305,7 @@ function _findExistingPlanForMeeting(meetingIds, plansDir) {
     if (mtg?.title) {
       // Dashboard convention: "Meeting follow-up: {title}" → slug
       const dashSlug = ('meeting-follow-up-' + mtg.title).toLowerCase().replace(/[^a-z0-9]+/g, '-').replace(/^-|-$/g, '').slice(0, 50);
-      slugPrefixes.push(dashSlug.slice(0, 30));
+      slugPrefixes.push(dashSlug);
     }
   }
   let slugMatch = null;

package/engine/playbook.js

@@ -250,8 +250,11 @@ function renderPlaybook(type, vars) {
   const timeStamp = ts().slice(11, 16).replace(':', '');
   const inboxSlug = [vars.agent_id || 'agent', vars.task_id || '', dateStamp(), timeStamp].filter(Boolean).join('-');
   content += `\n\n---\n\n## REQUIRED: Write Learnings\n\n`;
+  const noteId = `NOTE-${shared.uid()}`;
   content += `After completing your task, write **one** findings file to:\n`;
   content += `\`${MINIONS_DIR}/notes/inbox/${inboxSlug}.md\`\n\n`;
+  content += `Start the file with this YAML frontmatter (required for tracking):\n`;
+  content += `\`\`\`\n---\nid: ${noteId}\nagent: ${vars.agent_id || 'agent'}\ndate: ${dateStamp()}\n---\n\`\`\`\n\n`;
   content += `**IMPORTANT: Write exactly ONE inbox file per task.** If the playbook above already specifies an inbox path, use THAT path instead and include your learnings in the same document. Do NOT create a second file — duplicates clog consolidation.\n\n`;
   content += `Include in your findings file:\n`;
   content += `- What you learned about the codebase\n`;

package/engine/shared.js

@@ -286,15 +286,30 @@ function uniquePath(filePath) {
  * @param {string} content - Markdown content to write
  * @returns {boolean} true if a write occurred, false if deduped/skipped
  */
+/**
+ * Extract note ID from frontmatter of an inbox file. Returns null if no ID found.
+ */
+function parseNoteId(content) {
+  if (!content) return null;
+  const m = content.match(/^---\n[\s\S]*?id:\s*(NOTE-\w+)[\s\S]*?---/);
+  return m ? m[1] : null;
+}
+
 function writeToInbox(agentId, slug, content, _inboxDir) {
   try {
     const inboxDir = _inboxDir || path.join(MINIONS_DIR, 'notes', 'inbox');
     const prefix = `${agentId}-${slug}-${dateStamp()}`;
     const existing = safeReadDir(inboxDir).find(f => f.startsWith(prefix));
     if (existing) return false;
+    const noteId = `NOTE-${uid()}`;
+    // Inject structured ID as YAML frontmatter if content doesn't already have it
+    const hasFrontmatter = content.trimStart().startsWith('---');
+    const tagged = hasFrontmatter
+      ? content.replace(/^---\n/, `---\nid: ${noteId}\n`)
+      : `---\nid: ${noteId}\nagent: ${agentId}\ndate: ${dateStamp()}\n---\n\n${content}`;
     const filePath = path.join(inboxDir, `${prefix}.md`);
-    safeWrite(filePath, content);
-    return true;
+    safeWrite(filePath, tagged);
+    return noteId;
   } catch (e) {
     log('warn', `writeToInbox failed: ${e.message}`);
     return false;
@@ -860,7 +875,7 @@ module.exports = {
   mutatePullRequests,
   uid,
   uniquePath,
-  writeToInbox,
+  writeToInbox, parseNoteId,
   exec,
   execAsync,
   execSilent,

package/engine.js

@@ -791,11 +791,18 @@ async function spawnAgent(dispatchItem, config) {
       try {
         const artWiPath = resolveWorkItemPath(dispatchItem.meta);
         if (artWiPath) {
-          // Collect inbox notes written by this agent today
+          // Collect inbox notes written by this agent today (with structured IDs if available)
           const _artToday = shared.dateStamp();
           const _artInboxDir = path.join(MINIONS_DIR, 'notes', 'inbox');
           let _artNotes = [];
-          try { _artNotes = shared.safeReadDir(_artInboxDir).filter(f => f.startsWith(agentId + '-') && f.includes(_artToday)); } catch {}
+          try {
+            const noteFiles = shared.safeReadDir(_artInboxDir).filter(f => f.startsWith(agentId + '-') && f.includes(_artToday));
+            for (const f of noteFiles) {
+              const content = shared.safeRead(path.join(_artInboxDir, f));
+              const noteId = shared.parseNoteId(content);
+              _artNotes.push({ file: f, id: noteId || f.replace(/\.md$/, '') });
+            }
+          } catch {}
 
           mutateJsonFileLocked(artWiPath, data => {
             if (!Array.isArray(data)) return data;
@@ -806,7 +813,7 @@ async function spawnAgent(dispatchItem, config) {
             if (dispatchItem.meta.branch) arts.branch = dispatchItem.meta.branch;
             if (wi._pr) arts.pr = wi._pr;
             if (wi._prUrl) arts.prUrl = wi._prUrl;
-            if (_artNotes.length > 0) arts.notes = _artNotes;
+            if (_artNotes.length > 0) arts.notes = _artNotes.map(n => ({ id: n.id, file: n.file }));
             // Track plan/PRD artifacts from dispatch metadata
             if (dispatchItem.meta.item?.planFile) arts.plan = dispatchItem.meta.item.planFile;
             if (dispatchItem.meta.item?._prdFilename) arts.prd = dispatchItem.meta.item._prdFilename;

package/package.json

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