@yemi33/minions 0.1.654 → 0.1.656

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## 0.1.656 (2026-04-09)
+
+### Fixes
+-  bust require cache for package.json in engine start
+-  PRD discovery loop, github.js duplicate const, transcript null guards
+
+## 0.1.655 (2026-04-09)
+
+### Fixes
+-  skip PR reconciliation for projects with no PRs or work items
+
 ## 0.1.654 (2026-04-09)
 
 ### Fixes

package/dashboard/js/render-meetings.js

@@ -197,7 +197,7 @@ function _renderMeetingDetail(m) {
 
       // Wire up doc-chat Q&A panel for the meeting transcript
       const transcript = (m.transcript || []).map(t =>
-        '### ' + t.agent + ' (' + t.type + ', Round ' + t.round + ')\n\n' + (t.content || '')
+        '### ' + (t.agent || 'agent') + ' (' + (t.type || '') + ', Round ' + (t.round || '?') + ')\n\n' + (t.content || '')
       ).join('\n\n---\n\n');
       const meetingDoc = '# Meeting: ' + m.title + '\n\n**Agenda:** ' + m.agenda + '\n\n' + transcript;
       _modalDocContext = { title: 'Meeting: ' + m.title, content: meetingDoc, selection: '' };
@@ -418,7 +418,7 @@ async function _createPlanFromMeeting(id, btn) {
 
     // Use doc-chat to generate a structured plan from the meeting
     const transcript = (m.transcript || []).map(function(t) {
-      return '### ' + t.agent + ' (' + t.type + ', Round ' + t.round + ')\n\n' + (t.content || '');
+      return '### ' + (t.agent || 'agent') + ' (' + (t.type || '') + ', Round ' + (t.round || '?') + ')\n\n' + (t.content || '');
     }).join('\n\n---\n\n');
     const meetingDoc = '# Meeting: ' + m.title + '\n\n**Agenda:** ' + m.agenda + '\n\n' + transcript;
 

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

@@ -455,7 +455,7 @@ 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 = typeof n === 'object' ? (n.id || n.file || 'note').slice(0, 30) : String(n).replace(/\.md$/, '').slice(0, 30);
+    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> ';
   });
   if (arts.skills && arts.skills.length > 0) arts.skills.forEach(function(s) { artPills += '<span onclick="openSkill(\'' + escHtml(s) + '\',\'minions\',\'\')" style="' + pillStyle + '">⚙ ' + escHtml(s) + '</span> '; });
@@ -490,7 +490,7 @@ function viewAgentOutput(logPath) {
   document.getElementById('modal-body').style.whiteSpace = 'pre-wrap';
   document.getElementById('modal').classList.add('open');
   fetch('/api/agent-output?file=' + encodeURIComponent(logPath))
-    .then(function(r) { return r.text(); })
+    .then(function(r) { if (!r.ok) throw new Error('HTTP ' + r.status); return r.text(); })
     .then(function(content) {
       document.getElementById('modal-body').textContent = content;
     })

package/engine/cli.js

@@ -85,9 +85,20 @@ const commands = {
     }
 
     // Record version + git commit so dashboard can detect stale engine code
+    // Bust require cache so npm updates are detected after minions restart
     let codeVersion = null;
-    try { codeVersion = require('../package.json').version; } catch {}
-    if (!codeVersion) { try { codeVersion = require('@yemi33/minions/package.json').version; } catch {} }
+    try {
+      const pkgPath = require.resolve('../package.json');
+      delete require.cache[pkgPath];
+      codeVersion = require('../package.json').version;
+    } catch {}
+    if (!codeVersion) {
+      try {
+        const pkgPath = require.resolve('@yemi33/minions/package.json');
+        delete require.cache[pkgPath];
+        codeVersion = require('@yemi33/minions/package.json').version;
+      } catch {}
+    }
     let codeCommit = null;
     try { codeCommit = require('child_process').execSync('git rev-parse --short HEAD', { cwd: path.resolve(__dirname, '..'), encoding: 'utf8', timeout: 5000, windowsHide: true }).trim(); } catch {}
     safeWrite(CONTROL_PATH, { state: 'running', pid: process.pid, started_at: e.ts(), codeVersion, codeCommit });

package/engine/github.js

@@ -511,6 +511,16 @@ async function reconcilePrs(config) {
     // Skip projects in backoff (inaccessible repo)
     if (isSlugInBackoff(slug)) continue;
 
+    // Skip projects with no tracked PRs and no work items — nothing to reconcile
+    const existingPrs = getPrs(project);
+    if (existingPrs.length === 0) {
+      try {
+        const wiPath = projectWorkItemsPath(project);
+        const wis = safeJson(wiPath) || [];
+        if (wis.length === 0) continue;
+      } catch { continue; }
+    }
+
     // Fetch open PRs
     const prsData = await ghApi('/pulls?state=open&per_page=100', slug);
     if (!prsData || !Array.isArray(prsData)) {
@@ -527,8 +537,8 @@ async function reconcilePrs(config) {
     if (ghPrs.length === 0) continue;
 
     const prPath = projectPrPath(project);
-    const existingPrs = safeJson(prPath) || [];
-    const existingIds = new Set(existingPrs.map(p => p.id));
+    const currentPrs = safeJson(prPath) || [];
+    const existingIds = new Set(currentPrs.map(p => p.id));
     let projectAdded = 0;
 
     // Load work items to match branches
@@ -549,7 +559,7 @@ async function reconcilePrs(config) {
       if (existingIds.has(prId)) {
         if (confirmedItemId) {
           addPrLink(prId, confirmedItemId);
-          const existing = existingPrs.find(p => p.id === prId);
+          const existing = currentPrs.find(p => p.id === prId);
           if (existing && !(existing.prdItems || []).includes(confirmedItemId)) {
             existing.prdItems = Array.isArray(existing.prdItems) ? existing.prdItems : [];
             existing.prdItems.push(confirmedItemId);
@@ -563,7 +573,7 @@ async function reconcilePrs(config) {
 
       const prUrl = project.prUrlBase ? project.prUrlBase + ghPr.number : ghPr.html_url || '';
 
-      existingPrs.push({
+      currentPrs.push({
         id: prId,
         title: (ghPr.title || `PR #${ghPr.number}`).slice(0, 120),
         agent: (linkedItem?.dispatched_to || ghPr.user?.login || 'unknown').toLowerCase(),
@@ -584,7 +594,7 @@ async function reconcilePrs(config) {
     // Backfill prdItems from pr-links for any PR with empty array
     const prLinks = getPrLinks();
     let backfilled = 0;
-    for (const pr of existingPrs) {
+    for (const pr of currentPrs) {
       const linked = prLinks[pr.id];
       if (linked && !(pr.prdItems || []).includes(linked)) {
         pr.prdItems = Array.isArray(pr.prdItems) ? pr.prdItems : [];
@@ -594,13 +604,13 @@ async function reconcilePrs(config) {
     }
 
     if (projectAdded > 0 || backfilled > 0) {
-      mutateJsonFileLocked(prPath, (currentPrs) => {
-        for (const pr of existingPrs) {
-          const idx = currentPrs.findIndex(p => p.id === pr.id);
-          if (idx >= 0) currentPrs[idx] = pr;
-          else currentPrs.push(pr);
+      mutateJsonFileLocked(prPath, (lockedPrs) => {
+        for (const pr of currentPrs) {
+          const idx = lockedPrs.findIndex(p => p.id === pr.id);
+          if (idx >= 0) lockedPrs[idx] = pr;
+          else lockedPrs.push(pr);
         }
-        return currentPrs;
+        return lockedPrs;
       }, { defaultValue: [] });
       totalAdded += projectAdded;
     }

package/engine/pipeline.js

@@ -603,21 +603,21 @@ function isStageComplete(stage, stageState, run, config) {
       const plans = artifacts.plans || [];
       const discoveredPrds = [];
       const discoveredWiIds = [];
+      const prdFiles = fs.existsSync(prdDir) ? safeReadDir(prdDir).filter(f => f.endsWith('.json')) : [];
       for (const planFile of plans) {
-        const prdFiles = fs.existsSync(prdDir) ? safeReadDir(prdDir).filter(f => f.endsWith('.json')) : [];
         for (const pf of prdFiles) {
           const prd = safeJson(path.join(prdDir, pf));
           if (prd?.source_plan === planFile && !(artifacts.prds || []).includes(pf) && !discoveredPrds.includes(pf)) {
             discoveredPrds.push(pf);
           }
         }
-        const allPrds = [...(artifacts.prds || []), ...discoveredPrds];
-        for (const prdFile of allPrds) {
-          const prdItems = all.filter(w => w.sourcePlan === prdFile && w.type !== WORK_TYPE.PLAN_TO_PRD);
-          for (const wi of prdItems) {
-            if (!(artifacts.workItems || []).includes(wi.id) && !discoveredWiIds.includes(wi.id)) {
-              discoveredWiIds.push(wi.id);
-            }
+      }
+      const allPrds = [...(artifacts.prds || []), ...discoveredPrds];
+      for (const prdFile of allPrds) {
+        const prdItems = all.filter(w => w.sourcePlan === prdFile && w.type !== WORK_TYPE.PLAN_TO_PRD);
+        for (const wi of prdItems) {
+          if (!(artifacts.workItems || []).includes(wi.id) && !discoveredWiIds.includes(wi.id)) {
+            discoveredWiIds.push(wi.id);
           }
         }
       }

package/package.json

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