@yemi33/minions 0.1.1676 → 0.1.1678

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.1678 (2026-05-02)
+
+### Fixes
+-  dashboard screen audit — bug fixes across 11 renderers
+
+## 0.1.1677 (2026-05-02)
+
+### Fixes
+-  process bot PR comments as actionable
+
 ## 0.1.1676 (2026-05-02)
 
 ### Other

package/dashboard/js/render-agents.js

@@ -39,7 +39,7 @@ function renderAgents(agents) {
   agentData = agents;
   const grid = document.getElementById('agents-grid');
   grid.innerHTML = agents.map(a => `
-    <div class="agent-card ${statusColor(a.status)}" onclick="if(shouldIgnoreSelectionClick(event))return;openAgentDetail('${escapeHtml(a.id)}')">
+    <div class="agent-card ${statusColor(a.status)}" data-agent-id="${escapeHtml(a.id)}" onclick="if(shouldIgnoreSelectionClick(event))return;openAgentDetail(this.dataset.agentId)">
       <div class="agent-card-header">
         <span class="agent-name"><span class="agent-emoji">${escapeHtml(a.emoji)}</span>${escapeHtml(a.name)}${_runtimeTagHtml(a.runtime)}</span>
         <span class="status-badge ${escapeHtml(a.status)}">${escapeHtml(a.status)}</span>
@@ -116,7 +116,7 @@ async function openAgentDetail(id) {
     document.getElementById('detail-content').innerHTML =
       '<div style="padding:24px;text-align:center">' +
         '<div style="color:var(--red);margin-bottom:12px">Error loading agent detail: ' + escapeHtml(e.message) + '</div>' +
-        '<button onclick="openAgentDetail(\'' + escapeHtml(id) + '\')" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer;font-size:12px">Retry</button>' +
+        '<button data-agent-id="' + escapeHtml(id) + '" onclick="openAgentDetail(this.dataset.agentId)" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer;font-size:12px">Retry</button>' +
         ' <button onclick="closeDetail()" style="padding:6px 16px;background:var(--surface2);border:1px solid var(--border);border-radius:var(--radius-sm);cursor:pointer;font-size:12px;color:var(--text)">Close</button>' +
       '</div>';
   }

package/dashboard/js/render-dispatch.js

@@ -18,12 +18,7 @@ function renderEngineStatus(engine) {
   // Detect stale engine — says running but heartbeat is old (>2 min)
   if (state === 'running' && engine?.heartbeat) {
     staleMs = Date.now() - engine.heartbeat;
-    if (staleMs > 120000) {
-      state = 'stale';
-    }
-  } else if (state === 'running' && !engine?.heartbeat) {
-    // Running but no heartbeat yet — engine just started or old version
-    state = 'running';
+    if (staleMs > 120000) state = 'stale';
   }
 
   badge.className = 'engine-badge ' + (state === 'stale' ? 'stopped' : state);
@@ -172,12 +167,12 @@ function renderDispatch(dispatch) {
         '</tr>';
       }).join('') + '</tbody></table>';
     if (completed.length > COMPLETED_PER_PAGE) {
-      completedEl.innerHTML += '<div class="pr-pager">' +
+      completedEl.insertAdjacentHTML('beforeend', '<div class="pr-pager">' +
         '<span class="pr-page-info">Showing ' + (compStart + 1) + ' to ' + Math.min(compStart + COMPLETED_PER_PAGE, completed.length) + ' of ' + completed.length + '</span>' +
         '<div class="pr-pager-btns">' +
           '<button class="pr-pager-btn ' + (_completedPage === 0 ? 'disabled' : '') + '" onclick="_completedPrev()">Prev</button>' +
           '<button class="pr-pager-btn ' + (_completedPage >= totalCompPages - 1 ? 'disabled' : '') + '" onclick="_completedNext()">Next</button>' +
-        '</div></div>';
+        '</div></div>');
     }
   } else {
     completedEl.innerHTML = '<p class="empty">No completed dispatches yet.</p>';
@@ -206,12 +201,12 @@ function renderEngineLog(log) {
     '</div>'
   ).join('');
   if (reversed.length > LOG_PER_PAGE) {
-    el.innerHTML += '<div class="pr-pager">' +
+    el.insertAdjacentHTML('beforeend', '<div class="pr-pager">' +
       '<span class="pr-page-info">Showing ' + (logStart + 1) + ' to ' + Math.min(logStart + LOG_PER_PAGE, reversed.length) + ' of ' + reversed.length + '</span>' +
       '<div class="pr-pager-btns">' +
         '<button class="pr-pager-btn ' + (_logPage === 0 ? 'disabled' : '') + '" onclick="_logPrev()">Prev</button>' +
         '<button class="pr-pager-btn ' + (_logPage >= totalLogPages - 1 ? 'disabled' : '') + '" onclick="_logNext()">Next</button>' +
-      '</div></div>';
+      '</div></div>');
   }
 }
 
@@ -228,7 +223,7 @@ async function showErrorDetails(agentId, reason, task) {
   document.getElementById('modal').classList.add('open');
 
   try {
-    const output = await fetch('/api/agent/' + agentId + '/output').then(r => r.text());
+    const output = await safeFetch('/api/agent/' + agentId + '/output').then(r => r.text());
     const lines = output.split('\n');
     const stderrIdx = lines.findIndex(l => l.startsWith('## stderr'));
     let summary = '';

package/dashboard/js/render-inbox.js

@@ -43,12 +43,12 @@ function renderInbox(inbox) {
     </div>`;
   }).join('');
   if (inbox.length > INBOX_PER_PAGE) {
-    list.innerHTML += '<div class="pr-pager">' +
+    list.insertAdjacentHTML('beforeend', '<div class="pr-pager">' +
       '<span class="pr-page-info">Showing ' + (inboxStart + 1) + ' to ' + Math.min(inboxStart + INBOX_PER_PAGE, inbox.length) + ' of ' + inbox.length + '</span>' +
       '<div class="pr-pager-btns">' +
         '<button class="pr-pager-btn ' + (_inboxPage === 0 ? 'disabled' : '') + '" onclick="_inboxPrev()">Prev</button>' +
         '<button class="pr-pager-btn ' + (_inboxPage >= totalInboxPages - 1 ? 'disabled' : '') + '" onclick="_inboxNext()">Next</button>' +
-      '</div></div>';
+      '</div></div>');
   }
   restoreNotifBadges();
 }

package/dashboard/js/render-kb.js

@@ -134,12 +134,12 @@ function renderKnowledgeBase() {
     '</div>';
   }).join('');
   if (items.length > KB_PER_PAGE) {
-    listEl.innerHTML += '<div class="pr-pager">' +
+    listEl.insertAdjacentHTML('beforeend', '<div class="pr-pager">' +
       '<span class="pr-page-info">Showing ' + (kbStart + 1) + ' to ' + Math.min(kbStart + KB_PER_PAGE, items.length) + ' of ' + items.length + '</span>' +
       '<div class="pr-pager-btns">' +
         '<button class="pr-pager-btn ' + (_kbPage === 0 ? 'disabled' : '') + '" onclick="_kbPrev()">Prev</button>' +
         '<button class="pr-pager-btn ' + (_kbPage >= totalKbPages - 1 ? 'disabled' : '') + '" onclick="_kbNext()">Next</button>' +
-      '</div></div>';
+      '</div></div>');
   }
   restoreNotifBadges();
 }
@@ -242,18 +242,18 @@ function openCreateKbModal() {
         '<textarea id="kb-new-content" rows="8" style="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);resize:vertical;font-family:inherit" placeholder="Write your knowledge entry..."></textarea></label>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px">' +
         '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
-        '<button onclick="submitKbEntry()" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Save</button>' +
+        '<button onclick="submitKbEntry(event)" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Save</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');
 }
 
-async function submitKbEntry() {
-  var btn = event?.target; if (btn) { btn.disabled = true; btn.textContent = 'Creating...'; }
+async function submitKbEntry(e) {
+  var btn = (e || window.event)?.target; if (btn) { btn.disabled = true; btn.textContent = 'Saving...'; }
   const category = document.getElementById('kb-new-category').value;
   const title = document.getElementById('kb-new-title').value;
   const content = document.getElementById('kb-new-content').value;
-  if (!title || !content) { if (btn) { btn.disabled = false; btn.textContent = 'Create'; } alert('Title and content are required'); return; }
+  if (!title || !content) { if (btn) { btn.disabled = false; btn.textContent = 'Save'; } alert('Title and content are required'); return; }
   try {
     showToast('cmd-toast', 'KB entry created', true);
     const res = await fetch('/api/knowledge', {
@@ -261,8 +261,8 @@ async function submitKbEntry() {
       body: JSON.stringify({ category, title, content })
     });
     if (res.ok) { closeModal(); refreshKnowledgeBase(); }
-    else { if (btn) { btn.disabled = false; btn.textContent = 'Create'; } const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'KB create failed: ' + (d.error || 'unknown'), false); }
-  } catch (e) { if (btn) { btn.disabled = false; btn.textContent = 'Create'; } showToast('cmd-toast', 'Error: ' + e.message, false); }
+    else { if (btn) { btn.disabled = false; btn.textContent = 'Save'; } const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'KB create failed: ' + (d.error || 'unknown'), false); }
+  } catch (err) { if (btn) { btn.disabled = false; btn.textContent = 'Save'; } showToast('cmd-toast', 'Error: ' + err.message, false); }
 }
 
 async function kbOpenItem(category, file) {

package/dashboard/js/render-meetings.js

@@ -26,7 +26,7 @@ function renderMeetings(meetings) {
   const visible = _showArchived ? meetings : active;
   if (visible.length === 0) {
     el.innerHTML = '<p class="empty">No active meetings.</p>';
-    if (archived.length) el.innerHTML += '<div style="text-align:center;margin-top:8px"><button class="pr-pager-btn" style="font-size:10px" onclick="_toggleArchivedMeetings()">Show ' + archived.length + ' archived</button></div>';
+    if (archived.length) el.insertAdjacentHTML('beforeend', '<div style="text-align:center;margin-top:8px"><button class="pr-pager-btn" style="font-size:10px" onclick="_toggleArchivedMeetings()">Show ' + archived.length + ' archived</button></div>');
     return;
   }
 
@@ -66,17 +66,17 @@ function renderMeetings(meetings) {
   }).join('');
 
   if (visible.length > MTG_PER_PAGE) {
-    el.innerHTML += '<div class="pr-pager">' +
+    el.insertAdjacentHTML('beforeend', '<div class="pr-pager">' +
       '<span class="pr-page-info">Showing ' + (start + 1) + ' to ' + Math.min(start + MTG_PER_PAGE, visible.length) + ' of ' + visible.length + '</span>' +
       '<div class="pr-pager-btns">' +
         '<button class="pr-pager-btn ' + (_mtgPage === 0 ? 'disabled' : '') + '" onclick="_mtgPrev()">Prev</button>' +
         '<button class="pr-pager-btn ' + (_mtgPage >= totalPages - 1 ? 'disabled' : '') + '" onclick="_mtgNext()">Next</button>' +
-      '</div></div>';
+      '</div></div>');
   }
 
   if (archived.length > 0) {
-    el.innerHTML += '<div style="text-align:center;margin-top:8px"><button class="pr-pager-btn" style="font-size:10px" onclick="_toggleArchivedMeetings()">' +
-      (_showArchived ? 'Hide' : 'Show') + ' ' + archived.length + ' archived</button></div>';
+    el.insertAdjacentHTML('beforeend', '<div style="text-align:center;margin-top:8px"><button class="pr-pager-btn" style="font-size:10px" onclick="_toggleArchivedMeetings()">' +
+      (_showArchived ? 'Hide' : 'Show') + ' ' + archived.length + ' archived</button></div>');
   }
   restoreNotifBadges();
 }
@@ -254,14 +254,14 @@ function openCreateMeetingModal() {
       '<div style="color:var(--text);font-size:var(--text-md)">Participants<div style="display:flex;flex-direction:column;gap:4px;margin-top:4px" id="mtg-participants">' + agentOpts + '</div></div>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px;margin-top:4px">' +
         '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
-        '<button onclick="_submitCreateMeeting()" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Start Meeting</button>' +
+        '<button onclick="_submitCreateMeeting(event)" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Start Meeting</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');
 }
 
-async function _submitCreateMeeting() {
-  var btn = event?.target; if (btn) { btn.disabled = true; btn.textContent = 'Starting...'; }
+async function _submitCreateMeeting(e) {
+  var btn = (e || window.event)?.target; if (btn) { btn.disabled = true; btn.textContent = 'Starting...'; }
   const title = document.getElementById('mtg-title')?.value?.trim();
   const agenda = document.getElementById('mtg-agenda')?.value?.trim();
   if (!title || !agenda) { if (btn) { btn.disabled = false; btn.textContent = 'Start Meeting'; } alert('Title and agenda required'); return; }

package/dashboard/js/render-pinned.js

@@ -4,7 +4,7 @@ function renderPinned(entries) {
   entries = (entries || []).filter(function(e) { return !isDeleted('pin:' + e.title); });
   const el = document.getElementById('pinned-content');
   if (!el) return;
-  if (!entries || entries.length === 0) {
+  if (entries.length === 0) {
     el.innerHTML = '<p class="empty">No pinned notes. Pin important context that all agents should see.</p>';
     return;
   }
@@ -40,7 +40,7 @@ async function submitPinnedNote(e) {
   const title = document.getElementById('pin-title').value;
   const content = document.getElementById('pin-content').value;
   const level = document.getElementById('pin-level').value;
-  if (!title || !content) { if (btn) { btn.disabled = false; btn.textContent = 'Pin Note'; } alert('Title and content required'); return; }
+  if (!title || !content) { if (btn) { btn.disabled = false; btn.textContent = 'Pin'; } alert('Title and content required'); return; }
   try { closeModal(); } catch { /* may not be open */ }
 
   // Optimistic render: append the new entry to the pinned list and re-render immediately

package/dashboard/js/render-plans.js

@@ -21,15 +21,15 @@ function openCreatePlanModal() {
       '<div style="font-size:11px;color:var(--muted)">After creating, click Execute on the plan card to have an agent convert it into a PRD with work items.</div>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px;margin-top:4px">' +
         '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
-        '<button onclick="_submitCreatePlan()" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Create Plan</button>' +
+        '<button onclick="_submitCreatePlan(event)" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Create Plan</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');
   setTimeout(() => document.getElementById('plan-new-title')?.focus(), 100);
 }
 
-async function _submitCreatePlan() {
-  var btn = event?.target; if (btn) { btn.disabled = true; btn.textContent = 'Creating...'; }
+async function _submitCreatePlan(e) {
+  var btn = (e || window.event)?.target; if (btn) { btn.disabled = true; btn.textContent = 'Creating...'; }
   const title = document.getElementById('plan-new-title')?.value?.trim();
   const content = document.getElementById('plan-new-content')?.value?.trim();
   if (!title) { if (btn) { btn.disabled = false; btn.textContent = 'Create Plan'; } alert('Title is required'); return; }
@@ -144,16 +144,13 @@ function renderPlans(plans) {
     }
   }
 
-  // Link .md plans to their PRD .json — if a PRD is being worked on, the source plan is too
-  // Convention: plan-w025-2026-03-15.md → officeagent-2026-03-15.json (same date, different prefix)
+  // Link .md plans to their PRD .json — if a PRD is being worked on, the source plan is too.
+  // The .md → .json mapping is materialized in planToPrdFile below; here we only need to
+  // know whether any PRD is active to mark all candidate source .md plans as working.
   const workingJsons = new Set([...workingPlanFiles].filter(f => f.endsWith('.json')));
   if (workingJsons.size > 0) {
     for (const p of plans) {
-      if (p.format === 'draft' && p.file.endsWith('.md')) {
-        // A .md plan is "working" if any PRD .json has active dispatches
-        // (since the .md is the source that generated those PRDs)
-        if (workingJsons.size > 0) workingPlanFiles.add(p.file);
-      }
+      if (p.format === 'draft' && p.file.endsWith('.md')) workingPlanFiles.add(p.file);
     }
   }
 

package/dashboard/js/render-prs.js

@@ -134,15 +134,15 @@ function openAddPrModal() {
       '<div style="font-size:11px;color:var(--muted);margin-top:-4px;padding-left:24px">Off = context only (e.g. teammate\'s PR). On = agents actively monitor and fix issues.</div>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px;margin-top:8px">' +
         '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
-        '<button onclick="_submitLinkPr()" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Link PR</button>' +
+        '<button onclick="_submitLinkPr(event)" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Link PR</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');
   setTimeout(() => document.getElementById('pr-link-url')?.focus(), 100);
 }
 
-async function _submitLinkPr() {
-  var btn = event?.target; if (btn) { btn.disabled = true; btn.textContent = 'Linking...'; }
+async function _submitLinkPr(e) {
+  var btn = (e || window.event)?.target; if (btn) { btn.disabled = true; btn.textContent = 'Linking...'; }
   const url = document.getElementById('pr-link-url')?.value?.trim();
   if (!url) { if (btn) { btn.disabled = false; btn.textContent = 'Link PR'; } alert('PR URL is required'); return; }
   const title = document.getElementById('pr-link-title')?.value?.trim() || '';
@@ -166,7 +166,8 @@ async function unlinkPr(id) {
   if (!confirm('Remove ' + id + ' from tracking?')) return;
   showToast('cmd-toast', id + ' removed', true);
   markDeleted('pr:' + id);
-  const row = document.querySelector('[data-pr-id="' + id + '"]')?.closest('tr');
+  const escId = window.CSS && CSS.escape ? CSS.escape(id) : id;
+  const row = document.querySelector('[data-pr-id="' + escId + '"]')?.closest('tr');
   if (row) row.remove();
   try {
     const res = await fetch('/api/pull-requests/delete', {

package/dashboard/js/render-schedules.js

@@ -292,7 +292,7 @@ function renderSchedules(schedules) {
       const lastRun = s._lastRun ? timeAgo(s._lastRun) : 'never';
       const typeBadge = '<span class="dispatch-type ' + escHtml(s.type || 'implement') + '">' + escHtml(s.type || 'implement') + '</span>';
       const humanCron = _cronToHuman(s.cron || '');
-      html += '<tr style="cursor:pointer" onclick="if(shouldIgnoreSelectionClick(event))return;openScheduleDetail(\'' + escHtml(s.id) + '\')">' +
+      html += '<tr data-sched-id="' + escHtml(s.id || '') + '" style="cursor:pointer" onclick="if(shouldIgnoreSelectionClick(event))return;openScheduleDetail(\'' + escHtml(s.id) + '\')">' +
         '<td><span class="pr-id">' + escHtml(s.id || '') + '</span></td>' +
         '<td style="max-width:200px;overflow:hidden;text-overflow:ellipsis;white-space:nowrap" title="' + escHtml(s.title || '') + '">' + escHtml(s.title || '') + '</td>' +
         '<td><span title="' + escHtml(s.cron || '') + '" style="font-size:11px;color:var(--blue)">' + escHtml(humanCron) + '</span></td>' +
@@ -459,7 +459,7 @@ function _scheduleFormHtml(sched, isEdit) {
     '</label>' +
     '<div style="display:flex;justify-content:flex-end;gap:8px;margin-top:8px">' +
       '<button onclick="closeModal()" class="pr-pager-btn" style="padding:6px 16px;font-size:var(--text-md)">Cancel</button>' +
-      '<button onclick="submitSchedule(' + isEdit + ')" style="padding:6px 16px;font-size:var(--text-md);background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">' + (isEdit ? 'Save' : 'Create') + '</button>' +
+      '<button onclick="submitSchedule(' + isEdit + ',event)" style="padding:6px 16px;font-size:var(--text-md);background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">' + (isEdit ? 'Save' : 'Create') + '</button>' +
     '</div>' +
   '</div>';
 }
@@ -487,8 +487,8 @@ function openEditScheduleModal(id) {
   _updateCronPreview();
 }
 
-async function submitSchedule(isEdit) {
-  var btn = event?.target; if (btn) { btn.disabled = true; btn.textContent = isEdit ? 'Saving...' : 'Creating...'; }
+async function submitSchedule(isEdit, e) {
+  var btn = (e || window.event)?.target; if (btn) { btn.disabled = true; btn.textContent = isEdit ? 'Saving...' : 'Creating...'; }
   _showScheduleError('');
   const title = document.getElementById('sched-edit-title').value.trim();
   const cron = window._schedComputedCron || '';
@@ -505,7 +505,7 @@ async function submitSchedule(isEdit) {
     id = _generateScheduleId(title);
   }
 
-  function _resetSchedBtn() { if (btn) { btn.disabled = false; btn.textContent = isEdit ? 'Save Changes' : 'Create Schedule'; } }
+  function _resetSchedBtn() { if (btn) { btn.disabled = false; btn.textContent = isEdit ? 'Save' : 'Create'; } }
   if (!title) { _resetSchedBtn(); _showScheduleError('Title is required'); return; }
   if (!cron) { _resetSchedBtn(); _showScheduleError('Schedule is required \u2014 select days and time, or use natural language'); return; }
 
@@ -523,9 +523,15 @@ async function submitSchedule(isEdit) {
   } catch (e) { _resetSchedBtn(); _showScheduleError('Error: ' + e.message); }
 }
 
+function _findSchedRow(id) {
+  var sel = 'tr[data-sched-id="' + (window.CSS && CSS.escape ? CSS.escape(id) : id) + '"]';
+  return document.querySelector(sel);
+}
+
 async function toggleScheduleEnabled(id, enabled) {
   // Optimistic toggle — swap badge text immediately
-  document.querySelectorAll('tr').forEach(function(r) { if (r.textContent.includes(id)) { var badge = r.querySelector('.status-badge'); if (badge) badge.textContent = enabled ? 'ENABLED' : 'DISABLED'; } });
+  var row = _findSchedRow(id);
+  if (row) { var badge = row.querySelector('.status-badge'); if (badge) badge.textContent = enabled ? 'ENABLED' : 'DISABLED'; }
   try {
     const res = await fetch('/api/schedules/update', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -542,7 +548,8 @@ async function deleteSchedule(id) {
   if (!confirm('Delete scheduled task "' + id + '"?')) return;
   showToast('cmd-toast', 'Schedule deleted', true);
   markDeleted('sched:' + id);
-  document.querySelectorAll('tr').forEach(function(r) { if (r.textContent.includes(id)) r.remove(); });
+  var row = _findSchedRow(id);
+  if (row) row.remove();
   try {
     const res = await fetch('/api/schedules/delete', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },

package/dashboard/js/render-watches.js

@@ -184,8 +184,10 @@ function toggleWatchPause(id, pause) {
     method: 'POST',
     headers: { 'Content-Type': 'application/json' },
     body: JSON.stringify({ id: id, status: newStatus })
-  }).then(function(res) { return res.json(); }).then(function(data) {
-    if (data.error) showToast('cmd-toast', 'Error: ' + data.error, false);
+  }).then(async function(res) {
+    var data = await res.json().catch(function() { return {}; });
+    if (!res.ok || data.error) showToast('cmd-toast', 'Error: ' + (data.error || ('HTTP ' + res.status)), false);
+    else if (typeof refresh === 'function') refresh();
   }).catch(function(err) {
     showToast('cmd-toast', 'Error: ' + err.message, false);
   });
@@ -297,12 +299,14 @@ function submitWatch() {
       stopAfter: stopAfter,
       onNotMet: onNotMet || null,
     })
-  }).then(function(res) { return res.json(); }).then(function(data) {
-    if (data.error) {
-      showToast('cmd-toast', 'Error: ' + data.error, false);
+  }).then(async function(res) {
+    var data = await res.json().catch(function() { return {}; });
+    if (!res.ok || data.error) {
+      showToast('cmd-toast', 'Error: ' + (data.error || ('HTTP ' + res.status)), false);
     } else {
-      showToast('cmd-toast', 'Watch created: ' + data.watch.id, true);
+      showToast('cmd-toast', 'Watch created: ' + (data.watch && data.watch.id || ''), true);
       closeModal();
+      if (typeof refresh === 'function') refresh();
     }
   }).catch(function(err) {
     showToast('cmd-toast', 'Error: ' + err.message, false);

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

@@ -40,7 +40,7 @@ function wiRow(item) {
     : (item.branchStrategy === 'shared-branch' && item.status === 'done')
       ? '<span style="font-size:9px;color:var(--muted)" title="Part of shared branch — aggregate PR created at verify stage">shared branch</span>'
       : '<span style="color:var(--muted)">—</span>';
-  return '<tr style="cursor:pointer" onclick="if(shouldIgnoreSelectionClick(event))return;openWorkItemDetail(\'' + escapeHtml(item.id) + '\')">' +
+  return '<tr data-wi-id="' + escapeHtml(item.id) + '" style="cursor:pointer" onclick="if(shouldIgnoreSelectionClick(event))return;openWorkItemDetail(\'' + escapeHtml(item.id) + '\')">' +
     '<td><span class="pr-id">' + escapeHtml(item.id || '') + '</span></td>' +
     '<td style="max-width:220px;overflow:hidden;text-overflow:ellipsis;white-space:nowrap" title="' + escapeHtml((item.title || '').slice(0, 200)) + '">' + escapeHtml(item.title || '') + '</td>' +
     '<td><span style="font-size:10px;color:var(--muted)">' + escapeHtml(item._source || '') + '</span>' +
@@ -195,12 +195,16 @@ async function submitWorkItemEdit(id, source, e) {
   } catch (e) { alert('Update error: ' + e.message); editWorkItem(id, source); }
 }
 
+function _removeWiRow(id) {
+  var sel = 'tr[data-wi-id="' + (window.CSS && CSS.escape ? CSS.escape(id) : id) + '"]';
+  document.querySelectorAll(sel).forEach(function(r) { r.remove(); });
+}
+
 async function deleteWorkItem(id, source) {
   if (!confirm('Delete work item ' + id + '? This will kill any running agent and remove all dispatch history.')) return;
   showToast('cmd-toast', 'Work item deleted', true);
   markDeleted('wi:' + id);
-  var wiTable = document.getElementById('work-items-content');
-  (wiTable || document).querySelectorAll('tr').forEach(function(r) { if (r.textContent.includes(id)) r.remove(); });
+  _removeWiRow(id);
   try {
     const res = await fetch('/api/work-items/delete', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -214,8 +218,7 @@ async function cancelWorkItem(id, source) {
   if (!confirm('Cancel work item ' + id + '? This will kill any running agent and mark it cancelled.')) return;
   showToast('cmd-toast', 'Work item cancelled', true);
   markDeleted('wi:' + id);
-  var wiTable = document.getElementById('work-items-content');
-  (wiTable || document).querySelectorAll('tr').forEach(function(r) { if (r.textContent.includes(id)) r.remove(); });
+  _removeWiRow(id);
   try {
     const res = await fetch('/api/work-items/cancel', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -228,8 +231,7 @@ async function cancelWorkItem(id, source) {
 async function archiveWorkItem(id, source) {
   showToast('cmd-toast', 'Archived ' + id, true);
   markDeleted('wi:' + id);
-  var wiTable = document.getElementById('work-items-content');
-  (wiTable || document).querySelectorAll('tr').forEach(function(r) { if (r.textContent.includes(id)) r.remove(); });
+  _removeWiRow(id);
   try {
     const res = await fetch('/api/work-items/archive', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },

package/engine/ado.js

@@ -795,10 +795,9 @@ async function pollPrHumanComments(config) {
       for (const comment of (thread.comments || [])) {
         if (!comment.content || comment.commentType === 'system') continue;
         const content = comment.content;
-        // Skip bots, CI noise, and ignored authors
+        // Skip explicitly ignored authors and CI-report bodies, but do not ignore bot authors by default.
         const authorName = (comment.author?.displayName || '').toLowerCase();
         if (ignoredAuthors.some(a => authorName.includes(a))) continue;
-        if (/\b(bot|service|build|pipeline|codecov|sonar)\b/i.test(authorName)) continue;
         if (/^#{1,3}\s*(Coverage|Build|Test|Deploy|Pipeline)\s*(Report|Status|Result|Summary)/i.test(content)) continue;
         // Detect agent comments (included in context, but don't trigger fix)
         const isAgent = /\bMinions\s*\(/i.test(content) || /\bby\s+Minions\b/i.test(content) || /\[minions\]/i.test(content);

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-02T04:37:01.667Z"
+  "cachedAt": "2026-05-02T05:22:33.108Z"
 }

package/engine/github.js

@@ -561,14 +561,12 @@ async function pollPrHumanComments(config) {
       ...(Array.isArray(reviewComments) ? reviewComments : []).map(c => ({ ...c, _type: 'review' }))
     ];
 
-    // Separate: agent comments (included in context, don't trigger fix) vs human comments (trigger fix)
-    // All non-bot, non-CI comments go into context. Only non-agent comments trigger pendingFix.
+    // Separate: agent comments (included in context, don't trigger fix) vs actionable comments (trigger fix).
+    // Bot-authored comments are actionable unless explicitly ignored or clearly CI report noise.
     const ignoredAuthors = new Set((config.engine?.ignoredCommentAuthors || []).map(a => a.toLowerCase()));
-    function _isBot(c) {
-      if (c.user?.type === 'Bot') return true;
+    function _isIgnoredComment(c) {
       const login = (c.user?.login || '').toLowerCase();
       if (ignoredAuthors.has(login)) return true;
-      if (/\b(bot|codecov|sonar|dependabot|renovate|github-actions|azure-pipelines)\b/i.test(login)) return true;
       const body = c.body || '';
       if (/^#{1,3}\s*(Coverage|Build|Test|Deploy|Pipeline)\s*(Report|Status|Result|Summary)/i.test(body)) return true;
       if (/!\[.*\]\(https?:\/\/.*badge/i.test(body)) return true;
@@ -581,16 +579,16 @@ async function pollPrHumanComments(config) {
       if (/\[minions\]/i.test(body)) return true;
       return false;
     }
-    const humanComments = allComments.filter(c => !_isBot(c));
+    const actionableComments = allComments.filter(c => !_isIgnoredComment(c));
 
     const cutoffStr = pr.humanFeedback?.lastProcessedCommentDate || pr.created || '1970-01-01';
     const cutoffMs = new Date(cutoffStr).getTime() || 0;
 
-    // Collect ALL human comments for full context, track new ones for triggering
+    // Collect ALL actionable comments for full context, track new ones for triggering
     const allCommentEntries = [];
     const newComments = [];
 
-    for (const c of humanComments) {
+    for (const c of actionableComments) {
       const date = c.created_at || c.updated_at || '';
       const isAgent = _isAgentComment(c);
       const entry = {

package/package.json

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