@yemi33/minions 0.1.125 → 0.1.127

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## 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
+- engine.js
+- engine/shared.js
+
+### Dashboard
+- dashboard/js/detail-panel.js
+- dashboard/js/render-meetings.js
+- dashboard/js/render-work-items.js
+- dashboard/js/settings.js
+
 ## 0.1.125 (2026-04-01)
 
 ### Dashboard

package/dashboard/js/detail-panel.js

@@ -135,6 +135,7 @@ function _cancelCharterEdit() {
 }
 
 async function _saveCharter() {
+  if (!currentAgentId) { alert('No agent selected'); return; }
   const content = document.getElementById('charter-editor').value;
   const btn = document.getElementById('charter-save-btn');
   btn.textContent = 'Saving...'; btn.style.pointerEvents = 'none';

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-meetings.js

@@ -305,7 +305,7 @@ async function _unarchiveMeeting(id) {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id })
     });
-    if (!res.ok) { const d = await res.json().catch(() => ({})); alert('Delete failed: ' + (d.error || 'unknown')); refresh(); }
+    if (!res.ok) { const d = await res.json().catch(() => ({})); alert('Unarchive failed: ' + (d.error || 'unknown')); refresh(); }
   } catch (e) { alert('Error: ' + e.message); refresh(); }
 }
 

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

@@ -161,7 +161,8 @@ async function submitWorkItemEdit(id, source) {
 async function deleteWorkItem(id, source) {
   if (!confirm('Delete work item ' + id + '? This will kill any running agent and remove all dispatch history.')) return;
   markDeleted('wi:' + id);
-  document.querySelectorAll('tr').forEach(function(r) { if (r.textContent.includes(id)) r.remove(); });
+  var wiTable = document.getElementById('work-items-content');
+  (wiTable || document).querySelectorAll('tr').forEach(function(r) { if (r.textContent.includes(id)) r.remove(); });
   try {
     const res = await fetch('/api/work-items/delete', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -173,7 +174,8 @@ async function deleteWorkItem(id, source) {
 
 async function archiveWorkItem(id, source) {
   markDeleted('wi:' + id);
-  document.querySelectorAll('tr').forEach(function(r) { if (r.textContent.includes(id)) r.remove(); });
+  var wiTable = document.getElementById('work-items-content');
+  (wiTable || document).querySelectorAll('tr').forEach(function(r) { if (r.textContent.includes(id)) r.remove(); });
   showToast('cmd-toast', 'Archived ' + id, true);
   try {
     const res = await fetch('/api/work-items/archive', {

package/dashboard/js/settings.js

@@ -39,6 +39,7 @@ async function openSettings() {
     '</div>' +
     '<div style="display:flex;flex-direction:column;gap:6px;margin-bottom:16px">' +
       settingsToggle('Auto-approve Plans', 'set-autoApprovePlans', !!e.autoApprovePlans, 'PRDs are approved automatically without human review') +
+      settingsToggle('Auto-review PRs', 'set-autoReview', e.autoReview !== false, 'Automatically dispatch review agents for new PRs') +
       settingsToggle('Auto-decompose', 'set-autoDecompose', e.autoDecompose !== false, 'Large implement items are auto-split into sub-tasks') +
       settingsToggle('Allow Temp Agents', 'set-allowTempAgents', !!e.allowTempAgents, 'Spawn ephemeral agents when all permanent agents are busy') +
     '</div>' +
@@ -121,6 +122,7 @@ async function saveSettings() {
       restartGracePeriod: document.getElementById('set-restartGracePeriod').value,
       meetingRoundTimeout: document.getElementById('set-meetingRoundTimeout').value,
       autoApprovePlans: document.getElementById('set-autoApprovePlans').checked,
+      autoReview: document.getElementById('set-autoReview').checked,
       autoDecompose: document.getElementById('set-autoDecompose').checked,
       allowTempAgents: document.getElementById('set-allowTempAgents').checked,
     };

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/shared.js

@@ -337,6 +337,7 @@ const ENGINE_DEFAULTS = {
   allowTempAgents: false, // opt-in: spawn ephemeral agents when all permanent agents are busy
   autoDecompose: true, // auto-decompose implement:large items into sub-tasks
   autoApprovePlans: false, // auto-approve PRDs without waiting for human approval
+  autoReview: true, // auto-dispatch review agents for new PRs (disable for manual review workflow)
   meetingRoundTimeout: 600000, // 10min per meeting round before auto-advance
 };
 

package/engine.js

@@ -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' });
@@ -1245,7 +1247,8 @@ function discoverFromPrs(config, project) {
     const reviewStatus = pr.reviewStatus || 'pending';
 
     // PRs needing review: pending or waiting (review dispatched but no verdict yet)
-    const needsReview = reviewStatus === 'pending' || reviewStatus === 'waiting';
+    const autoReview = config.engine?.autoReview !== false;
+    const needsReview = autoReview && (reviewStatus === 'pending' || reviewStatus === 'waiting');
     if (needsReview) {
       const key = `review-${project?.name || 'default'}-${pr.id}`;
       if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) continue;

package/package.json

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