@yemi33/minions 0.1.126 → 0.1.128

package/CHANGELOG.md

@@ -1,5 +1,30 @@
 # Changelog
 
+## 0.1.128 (2026-04-01)
+
+### Engine
+- engine.js
+- engine/dispatch.js
+- engine/lifecycle.js
+- engine/timeout.js
+
+### Dashboard
+- dashboard.js
+- dashboard/js/render-work-items.js
+
+### Other
+- test/unit.test.js
+
+## 0.1.127 (2026-04-01)
+
+### Engine
+- engine.js
+- engine/meeting.js
+
+### Dashboard
+- dashboard/js/render-kb.js
+- dashboard/js/render-schedules.js
+
 ## 0.1.126 (2026-04-01)
 
 ### Engine

package/dashboard/js/render-kb.js

@@ -147,15 +147,14 @@ async function submitKbEntry() {
   const title = document.getElementById('kb-new-title').value;
   const content = document.getElementById('kb-new-content').value;
   if (!title || !content) { alert('Title and content are required'); return; }
-  try { closeModal(); } catch { /* may not be open */ }
-  showToast('cmd-toast', 'KB entry created', true);
   try {
     const res = await fetch('/api/knowledge', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ category, title, content })
     });
-    if (res.ok) { refreshKnowledgeBase(); } else { const d = await res.json().catch(() => ({})); alert('KB create failed: ' + (d.error || 'unknown')); openNewKbModal(); }
-  } catch (e) { alert('Error: ' + e.message); openNewKbModal(); }
+    if (res.ok) { closeModal(); refreshKnowledgeBase(); showToast('cmd-toast', 'KB entry created', true); }
+    else { const d = await res.json().catch(() => ({})); alert('KB create failed: ' + (d.error || 'unknown')); }
+  } catch (e) { alert('Error: ' + e.message); }
 }
 
 async function kbOpenItem(category, file) {

package/dashboard/js/render-schedules.js

@@ -366,8 +366,8 @@ function _scheduleFormHtml(sched, isEdit) {
   const priorities = ['high', 'medium', 'low'];
   const typeOpts = types.map(t => '<option value="' + t + '"' + ((sched.type || 'implement') === t ? ' selected' : '') + '>' + t + '</option>').join('');
   const priOpts = priorities.map(p => '<option value="' + p + '"' + ((sched.priority || 'medium') === p ? ' selected' : '') + '>' + p + '</option>').join('');
-  const projOpts = '<option value="">Any</option>' + cmdProjects.map(p => '<option value="' + escHtml(p.name) + '"' + (sched.project === p.name ? ' selected' : '') + '>' + escHtml(p.name) + '</option>').join('');
-  const agentOpts = '<option value="">Auto</option>' + cmdAgents.map(a => '<option value="' + escHtml(a.id) + '"' + (sched.agent === a.id ? ' selected' : '') + '>' + escHtml(a.name) + '</option>').join('');
+  const projOpts = '<option value="">Any</option>' + (cmdProjects || []).map(p => '<option value="' + escHtml(p.name) + '"' + (sched.project === p.name ? ' selected' : '') + '>' + escHtml(p.name) + '</option>').join('');
+  const agentOpts = '<option value="">Auto</option>' + (cmdAgents || []).map(a => '<option value="' + escHtml(a.id) + '"' + (sched.agent === a.id ? ' selected' : '') + '>' + escHtml(a.name) + '</option>').join('');
 
   const inputStyle = 'display:block;width:100%;margin-top:4px;padding:6px 8px;background:var(--bg);border:1px solid var(--border);border-radius:var(--radius-sm);color:var(--text);font-size:var(--text-md);font-family:inherit';
   const pillStyle = 'display:inline-block;padding:4px 10px;margin:2px;border:1px solid var(--border);border-radius:12px;cursor:pointer;font-size:11px;color:var(--text);background:var(--bg);user-select:none;transition:all 0.15s';

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

@@ -94,7 +94,7 @@ function editWorkItem(id, source) {
   if (!item) return;
   const types = ['implement', 'fix', 'review', 'plan', 'verify', 'investigate', 'refactor', 'test', 'docs'];
   const priorities = ['critical', 'high', 'medium', 'low'];
-  const agentOpts = cmdAgents.map(a => '<option value="' + escHtml(a.id) + '"' + (item.agent === a.id ? ' selected' : '') + '>' + escHtml(a.name) + '</option>').join('');
+  const agentOpts = (cmdAgents || []).map(a => '<option value="' + escHtml(a.id) + '"' + (item.agent === a.id ? ' selected' : '') + '>' + escHtml(a.name) + '</option>').join('');
   const typeOpts = types.map(t => '<option value="' + t + '"' + ((item.type || 'implement') === t ? ' selected' : '') + '>' + t + '</option>').join('');
   const priOpts = priorities.map(p => '<option value="' + p + '"' + ((item.priority || 'medium') === p ? ' selected' : '') + '>' + p + '</option>').join('');
 

package/dashboard.js

@@ -3057,7 +3057,10 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     try {
       const body = await readBody(req);
       const configPath = path.join(MINIONS_DIR, 'config.json');
-      const config = safeJson(configPath);
+      const config = safeJson(configPath) || {};
+      if (!config.engine) config.engine = {};
+      if (!config.claude) config.claude = {};
+      if (!config.agents) config.agents = {};
 
       if (body.engine) {
         // Validate and apply engine settings

package/engine/dispatch.js

@@ -86,8 +86,8 @@ function completeDispatch(id, result = 'success', reason = '', resultSummary = '
     if (reason) item.reason = reason;
     if (resultSummary) item.resultSummary = resultSummary;
     delete item.prompt;
-    if (dispatch.completed.length >= 100) {
-      dispatch.completed = dispatch.completed.slice(-99);
+    if (dispatch.completed.length > 100) {
+      dispatch.completed = dispatch.completed.slice(-100);
     }
     dispatch.completed.push(item);
     return dispatch;

package/engine/lifecycle.js

@@ -1161,7 +1161,7 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
           return d.includes(branchSlug) && fs.statSync(path.join(worktreeRoot, d)).isDirectory();
         });
         // Only remove if no other active dispatch uses this branch
-        const dispatch = e.getDispatch();
+        const dispatch = getDispatch();
         const otherActive = ((dispatch.active || []).concat(dispatch.pending || [])).some(d =>
           d.id !== dispatchItem.id && d.meta?.branch && shared.sanitizeBranch && shared.sanitizeBranch(d.meta.branch) === branchSlug
         );
@@ -1208,10 +1208,10 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
             wi._retryCount = retries + 1;
             delete wi.dispatched_at;
             delete wi.dispatched_to;
-            e.log('info', `Auto-retry ${retries + 1}/3 for ${meta.item.id} (no PR created)`);
+            log('info', `Auto-retry ${retries + 1}/3 for ${meta.item.id} (no PR created)`);
           } else {
             wi.status = 'failed';
-            e.log('warn', `${meta.item.id} failed after 3 retries — no PR created`);
+            log('warn', `${meta.item.id} failed after 3 retries — no PR created`);
           }
           shared.safeWrite(wiPath, items);
         }

package/engine/meeting.js

@@ -179,6 +179,7 @@ function discoverMeetingWork(config) {
  * Called from runPostCompletionHooks when type === 'meeting'.
  */
 function collectMeetingFindings(meetingId, agentId, roundName, output) {
+  const e = engine();
   const meeting = getMeeting(meetingId);
   if (!meeting) return;
 

package/engine/timeout.js

@@ -130,7 +130,7 @@ function checkTimeouts(config) {
     // Optimization: only read file if recent activity (avoids reading stale 1MB logs)
     let completedViaOutput = false;
     try {
-      if (silentMs > 600000) throw 'skip'; // No point reading a file silent for >10min
+      if (silentMs > 600000) throw new Error('skip'); // No point reading a file silent for >10min
       const liveLog = safeRead(liveLogPath);
       if (liveLog && liveLog.includes('"type":"result"')) {
         completedViaOutput = true;

package/engine.js

@@ -1036,7 +1036,7 @@ function materializePlansAsWorkItems(config) {
         plan.status = 'approved';
         plan.approvedAt = new Date().toISOString();
         plan.approvedBy = 'auto-mode';
-        safeWrite(prdPath, plan);
+        safeWrite(path.join(PRD_DIR, file), plan);
         log('info', `Auto-approved plan: ${file}`);
       } else {
         continue; // Skip — waiting for human approval
@@ -1185,8 +1185,10 @@ function materializePlansAsWorkItems(config) {
       // Pre-create shared feature branch if branch_strategy is shared-branch
       if (plan.branch_strategy === 'shared-branch' && plan.feature_branch) {
         try {
-          const root = path.resolve(project.localPath);
-          const mainBranch = project.mainBranch || 'main';
+          const firstProject = itemsByProject.values().next().value?.project;
+          if (!firstProject?.localPath) throw new Error('no project with localPath');
+          const root = path.resolve(firstProject.localPath);
+          const mainBranch = firstProject.mainBranch || 'main';
           const branch = sanitizeBranch(plan.feature_branch);
           // Create branch from main (idempotent — ignores if exists)
           exec(`git branch "${branch}" "${mainBranch}" 2>/dev/null || true`, { cwd: root, stdio: 'pipe' });

package/package.json

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