@yemi33/minions 0.1.414 → 0.1.416

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.1.416 (2026-04-06)
+
+### Features
+- Fix notes.md race condition and null guards in consolidation.js
+- Replace 5 raw status strings with WI_STATUS constants
+- Fix null crashes in lifecycle.js syncPrsFromOutput and createReviewFeedbackForAuthor
+
 ## 0.1.414 (2026-04-06)
 
 ### Fixes

package/engine/consolidation.js

@@ -38,7 +38,7 @@ function consolidateInbox(config) {
 
   const items = files.map(f => ({
     name: f,
-    content: safeRead(path.join(INBOX_DIR, f))
+    content: safeRead(path.join(INBOX_DIR, f)) || ''
   }));
 
   const existingNotes = getNotes() || '';
@@ -220,30 +220,33 @@ function consolidateWithLLM(items, existingNotes, files, config) {
       }
 
       const entry = '\n\n---\n\n' + digest;
-      const current = getNotes();
-      let newContent = current + entry;
-
-      if (newContent.length > 50000) {
-        // Truncate on section boundary — scan backward for last \n# before byte limit
-        // Never cut mid-section to preserve readability
-        const limit = 50000;
-        const lastSectionBoundary = newContent.lastIndexOf('\n---\n\n### ', limit);
-        if (lastSectionBoundary > 0) {
-          newContent = newContent.slice(0, lastSectionBoundary);
-          log('info', `Pruned notes.md at section boundary (pos ${lastSectionBoundary}) to stay under ${limit} bytes`);
-        } else {
-          // Fallback: use the old section-count approach
-          const sections = newContent.split('\n---\n\n### ');
-          if (sections.length > 10) {
-            const header = sections[0];
-            const recent = sections.slice(-8);
-            newContent = header + '\n---\n\n### ' + recent.join('\n---\n\n### ');
-            log('info', `Pruned notes.md: removed ${sections.length - 9} old sections`);
+      // Wrap read-modify-write in file lock to prevent race with concurrent consolidation or manual edits
+      shared.withFileLock(NOTES_PATH + '.lock', () => {
+        const current = getNotes() || '';
+        let newContent = current + entry;
+
+        if (newContent.length > 50000) {
+          // Truncate on section boundary — scan backward for last \n# before byte limit
+          // Never cut mid-section to preserve readability
+          const limit = 50000;
+          const lastSectionBoundary = newContent.lastIndexOf('\n---\n\n### ', limit);
+          if (lastSectionBoundary > 0) {
+            newContent = newContent.slice(0, lastSectionBoundary);
+            log('info', `Pruned notes.md at section boundary (pos ${lastSectionBoundary}) to stay under ${limit} bytes`);
+          } else {
+            // Fallback: use the old section-count approach
+            const sections = newContent.split('\n---\n\n### ');
+            if (sections.length > 10) {
+              const header = sections[0];
+              const recent = sections.slice(-8);
+              newContent = header + '\n---\n\n### ' + recent.join('\n---\n\n### ');
+              log('info', `Pruned notes.md: removed ${sections.length - 9} old sections`);
+            }
           }
         }
-      }
 
-      safeWrite(NOTES_PATH, newContent);
+        safeWrite(NOTES_PATH, newContent);
+      });
       classifyToKnowledgeBase(items);
       archiveInboxFiles(files);
       log('info', `LLM consolidation complete: ${files.length} notes processed by Haiku`);
@@ -297,10 +300,10 @@ function consolidateWithRegex(items, files) {
       if (!trimmed || sectionPattern.test(trimmed)) continue;
       let insight = null;
       const numMatch = trimmed.match(numberedPattern);
-      if (numMatch) insight = `**${numMatch[1].trim()}**: ${numMatch[2].trim()}`;
+      if (numMatch && numMatch[1] && numMatch[2]) insight = `**${numMatch[1].trim()}**: ${numMatch[2].trim()}`;
       if (!insight) {
         const bulMatch = trimmed.match(bulletPattern);
-        if (bulMatch) insight = `**${bulMatch[1].trim()}**: ${bulMatch[2].trim()}`;
+        if (bulMatch && bulMatch[1] && bulMatch[2]) insight = `**${bulMatch[1].trim()}**: ${bulMatch[2].trim()}`;
       }
       if (!insight && importantKeywords.test(trimmed) && !trimmed.startsWith('#') && trimmed.length > 30 && trimmed.length < 500) {
         insight = trimmed;
@@ -363,19 +366,22 @@ function consolidateWithRegex(items, files) {
   const dupCount = allInsights.length - deduped.length;
   if (dupCount > 0) entry += `_Deduplication: ${dupCount} duplicate(s) removed._\n`;
 
-  const current = getNotes();
-  let newContent = current + entry;
-  if (newContent.length > 50000) {
-    const limit = 50000;
-    const lastBoundary = newContent.lastIndexOf('\n---\n\n### ', limit);
-    if (lastBoundary > 0) {
-      newContent = newContent.slice(0, lastBoundary);
-    } else {
-      const sections = newContent.split('\n---\n\n### ');
-      if (sections.length > 10) { newContent = sections[0] + '\n---\n\n### ' + sections.slice(-8).join('\n---\n\n### '); }
+  // Wrap read-modify-write in file lock to prevent race with concurrent consolidation or manual edits
+  shared.withFileLock(NOTES_PATH + '.lock', () => {
+    const current = getNotes() || '';
+    let newContent = current + entry;
+    if (newContent.length > 50000) {
+      const limit = 50000;
+      const lastBoundary = newContent.lastIndexOf('\n---\n\n### ', limit);
+      if (lastBoundary > 0) {
+        newContent = newContent.slice(0, lastBoundary);
+      } else {
+        const sections = newContent.split('\n---\n\n### ');
+        if (sections.length > 10) { newContent = sections[0] + '\n---\n\n### ' + sections.slice(-8).join('\n---\n\n### '); }
+      }
     }
-  }
-  safeWrite(NOTES_PATH, newContent);
+    safeWrite(NOTES_PATH, newContent);
+  });
   classifyToKnowledgeBase(items);
   archiveInboxFiles(files);
   log('info', `Regex fallback: consolidated ${files.length} notes \u2192 ${deduped.length} insights into notes.md`);

package/engine/dispatch.js

@@ -116,7 +116,7 @@ function completeDispatch(id, result = DISPATCH_RESULT.SUCCESS, reason = '', res
       const maxRetries = ENGINE_DEFAULTS.maxRetries;
       if (retryableFailure && retries < maxRetries) {
         log('info', `Dispatch error for ${item.meta.item.id} — auto-retry ${retries + 1}/${maxRetries}`);
-        lifecycle().updateWorkItemStatus(item.meta, 'pending', '');
+        lifecycle().updateWorkItemStatus(item.meta, WI_STATUS.PENDING, '');
         // Remove this dispatch key from completed so dedupe doesn't block immediate redispatch.
         if (item.meta?.dispatchKey) {
           try {
@@ -150,7 +150,7 @@ function completeDispatch(id, result = DISPATCH_RESULT.SUCCESS, reason = '', res
         const finalReason = !retryableFailure
           ? `Non-retryable failure: ${reason || 'Unknown error'}`
           : (reason || `Failed after ${maxRetries} retries`);
-        lifecycle().updateWorkItemStatus(item.meta, 'failed', finalReason);
+        lifecycle().updateWorkItemStatus(item.meta, WI_STATUS.FAILED, finalReason);
         // Alert: find items blocked by this failure and write inbox note
         try {
           const config = getConfig();

package/engine/lifecycle.js

@@ -617,6 +617,7 @@ function syncPrsFromOutput(output, agentId, meta, config) {
   const inboxFiles = getInboxFiles().filter(f => f.includes(agentId) && f.includes(today));
   for (const f of inboxFiles) {
     const content = safeRead(path.join(INBOX_DIR, f));
+    if (!content) continue;
     const prHeaderPattern = /\*\*PR[:\*]*\*?\s*[#-]*\s*(?:(?:visualstudio\.com|dev\.azure\.com)[^\s"]*?pullrequest\/(\d+)|github\.com\/[^\s"]*?\/pull\/(\d+))/gi;
     while ((match = prHeaderPattern.exec(content)) !== null) prMatches.add(match[1] || match[2]);
   }
@@ -991,7 +992,7 @@ function createReviewFeedbackForAuthor(reviewerAgentId, pr, config) {
   const inboxFiles = getInboxFiles();
   const reviewFiles = inboxFiles.filter(f => f.includes(reviewerAgentId) && f.includes(today));
   if (reviewFiles.length === 0) return;
-  const reviewContent = reviewFiles.map(f => safeRead(path.join(INBOX_DIR, f))).join('\n\n');
+  const reviewContent = reviewFiles.map(f => safeRead(path.join(INBOX_DIR, f))).filter(Boolean).join('\n\n');
   const feedbackFile = `feedback-${authorAgentId}-from-${reviewerAgentId}-${pr.id}-${today}.md`;
   const feedbackPath = shared.uniquePath(path.join(INBOX_DIR, feedbackFile));
   const content = `# Review Feedback for ${config.agents[authorAgentId]?.name || authorAgentId}\n\n` +

package/engine/queries.js

@@ -10,7 +10,8 @@ const os = require('os');
 const shared = require('./shared');
 
 const { safeRead, safeReadDir, safeJson, safeWrite, getProjects,
-  projectWorkItemsPath, projectPrPath, parseSkillFrontmatter, KB_CATEGORIES } = shared;
+  projectWorkItemsPath, projectPrPath, parseSkillFrontmatter, KB_CATEGORIES,
+  WI_STATUS } = shared;
 
 // ── Paths ───────────────────────────────────────────────────────────────────
 
@@ -179,7 +180,7 @@ function getAgentStatus(agentId) {
     const latestInFlight = allItems
       .filter(w =>
         (w.dispatched_to || '').toLowerCase() === String(agentId).toLowerCase() &&
-        w.status === 'dispatched'
+        w.status === WI_STATUS.DISPATCHED
       )
       .sort((a, b) => (b.dispatched_at || '').localeCompare(a.dispatched_at || ''))[0];
     if (latestInFlight) {

package/package.json

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