@yemi33/minions 0.1.1677 → 0.1.1679

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.1679 (2026-05-02)
+
+### Fixes
+-  preserve PR context for linked fix work items
+
+## 0.1.1678 (2026-05-02)
+
+### Fixes
+-  dashboard screen audit — bug fixes across 11 renderers
+
 ## 0.1.1677 (2026-05-02)
 
 ### Fixes

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

@@ -110,6 +110,22 @@ function normalizePrMetadata(metadata) {
   };
 }
 
+function getWorkItemPrRef(input) {
+  if (!input || typeof input !== 'object') return null;
+  return input.targetPr || input.pr || input.prId || input.prNumber || input.pullRequest || input.sourcePr || input.prUrl || null;
+}
+
+function copyWorkItemPrFields(item, input, pr = null) {
+  const prRef = getWorkItemPrRef(input);
+  if (!prRef && !pr) return;
+  const prNumber = pr ? shared.getPrNumber(pr) : shared.getPrNumber(prRef);
+  item.targetPr = pr?.id || prRef;
+  item.pr_id = pr?.id || (typeof prRef === 'string' ? prRef : '');
+  if (prNumber != null) item.prNumber = prNumber;
+  if (pr?.branch || input.prBranch) item.prBranch = pr?.branch || input.prBranch;
+  if (pr?.title || input.prTitle) item.prTitle = pr?.title || input.prTitle;
+  if (pr?.url || input.prUrl) item.prUrl = pr?.url || input.prUrl;
+}
 function linkPullRequestForTracking({ url, title, project: projectName, autoObserve, context, workItemId }, config = CONFIG, options = {}) {
   if (!url) {
     const err = new Error('url required');
@@ -1395,6 +1411,8 @@ async function executeCCActions(actions) {
           const workType = routing.normalizeWorkType(action.workType || (action.type !== 'dispatch' ? action.type : WORK_TYPE.IMPLEMENT), WORK_TYPE.IMPLEMENT);
           const id = 'W-' + shared.uid();
           const project = action.project || '';
+          const prRef = getWorkItemPrRef(action);
+          let linkedPr = null;
 
           // Strict project resolution. Silent fallback to PROJECTS[0] when the model named an unknown
           // project caused work items to land in the wrong repo. Now: unknown name → error; ambiguous
@@ -1408,6 +1426,18 @@ async function executeCCActions(actions) {
               results.push({ type: action.type, error: `Project "${project}" not found. Known projects: ${known}` });
               break;
             }
+          } else if (prRef) {
+            const allPrs = getPullRequests().filter(p => !p._ghost);
+            linkedPr = shared.findPrRecord(allPrs, prRef) || null;
+            if (linkedPr?._project && linkedPr._project !== 'central') {
+              targetProject = PROJECTS.find(p => p.name?.toLowerCase() === String(linkedPr._project).toLowerCase()) || null;
+            }
+            if (!targetProject && PROJECTS.length > 1) {
+              results.push({ type: action.type, error: `project field is required when ${PROJECTS.length} projects are configured: ${PROJECTS.map(p => p.name).join(', ')}` });
+              break;
+            } else if (!targetProject && PROJECTS.length === 1) {
+              targetProject = PROJECTS[0];
+            }
           } else if (PROJECTS.length > 1) {
             results.push({ type: action.type, error: `project field is required when ${PROJECTS.length} projects are configured: ${PROJECTS.map(p => p.name).join(', ')}` });
             break;
@@ -1416,6 +1446,16 @@ async function executeCCActions(actions) {
           }
           // PROJECTS.length === 0 → targetProject stays null, falls back to root work-items.json (existing behavior).
 
+          if (prRef && !linkedPr && targetProject) {
+            const projectPrs = shared.safeJson(shared.projectPrPath(targetProject)) || [];
+            shared.normalizePrRecords(projectPrs, targetProject);
+            linkedPr = shared.findPrRecord(projectPrs, prRef, targetProject) || null;
+          }
+          if (prRef && (workType === WORK_TYPE.FIX || workType === WORK_TYPE.REVIEW || workType === WORK_TYPE.TEST) && !linkedPr) {
+            results.push({ type: action.type, error: `PR not found: ${prRef}` });
+            break;
+          }
+
           const wiPath = targetProject ? shared.projectWorkItemsPath(targetProject) : path.join(MINIONS_DIR, 'work-items.json');
 
           // Promote `agent` (singular) → `agents` (array). Models emit either shape and the prior code
@@ -1438,14 +1478,16 @@ async function executeCCActions(actions) {
           const isOneShot = action.oneShot === true || (action.oneShot !== false && ccOneShotTypes.has(workType));
           shared.mutateJsonFileLocked(wiPath, items => {
             if (!Array.isArray(items)) items = [];
-            items.push({
+            const item = {
               id, title: action.title, type: workType,
               priority: action.priority || 'medium', description: action.description || '',
               status: WI_STATUS.PENDING, created: new Date().toISOString(),
-              createdBy: 'command-center', project,
+              createdBy: 'command-center', project: targetProject?.name || project,
               ...(agentHints.length ? { preferred_agent: agentHints[0], agents: agentHints } : {}),
               ...(isOneShot ? { oneShot: true } : {}),
-            });
+            };
+            copyWorkItemPrFields(item, action, linkedPr);
+            items.push(item);
             return items;
           }, { defaultValue: [] });
           results.push({ type: action.type, id, ok: true });
@@ -2740,6 +2782,7 @@ const server = http.createServer(async (req, res) => {
       if (body.acceptanceCriteria) item.acceptanceCriteria = body.acceptanceCriteria;
       if (body.skipPr) item.skipPr = true;
       if (body.oneShot) item.oneShot = true;
+      copyWorkItemPrFields(item, body);
       let dupId = null;
       mutateJsonFileLocked(wiPath, (items) => {
         if (!Array.isArray(items)) items = [];

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-02T05:19:51.665Z"
+  "cachedAt": "2026-05-02T05:46:43.671Z"
 }

package/engine/playbook.js

@@ -628,9 +628,13 @@ function selectPlaybook(workType, item) {
   if (workType === WORK_TYPE.IMPLEMENT || workType === WORK_TYPE.IMPLEMENT_LARGE) {
     return 'implement';
   }
-  if (workType === WORK_TYPE.REVIEW && !item?._pr && !item?.pr_id) {
+  const hasPrContext = !!(item?._pr || item?.pr_id || item?.targetPr || item?.sourcePr || item?.pr);
+  if (workType === WORK_TYPE.REVIEW && !hasPrContext) {
     return 'work-item';
   }
+  if (workType === WORK_TYPE.FIX && hasPrContext) {
+    return 'fix';
+  }
   const typeSpecificPlaybooks = ['explore', 'review', 'test', 'plan-to-prd', 'plan', 'ask', 'verify', 'decompose', 'docs', 'meeting-investigate', 'meeting-debate', 'meeting-conclude'];
   return typeSpecificPlaybooks.includes(workType) ? workType : 'work-item';
 }

package/engine.js

@@ -2592,6 +2592,14 @@ function renderProjectWorkItemPromptForAgent(item, workType, agentId, config, pr
     worktree_path: path.resolve(root, config.engine?.worktreeRoot || '../worktrees', `${branchName}`),
     commit_message: item.commitMessage || `feat: ${item.title || item.id}`,
     notes_content: '',
+    pr_id: item.pr_id || item._pr || item.targetPr || item.sourcePr || item.pr || '',
+    pr_number: item.prNumber || item.pr_number || '',
+    pr_title: item.pr_title || item.prTitle || '',
+    pr_branch: item.pr_branch || item.prBranch || '',
+    pr_author: item.pr_author || item.prAuthor || '',
+    pr_url: item.pr_url || item.prUrl || '',
+    reviewer: item.reviewer || 'Reviewer',
+    review_note: item.review_note || item.reviewNote || item.description || item.title || 'See PR thread comments',
   };
   const cpResult = buildWorkItemDispatchVars(item, vars, config, {
     worktreePath: vars.worktree_path || root,
@@ -2612,6 +2620,38 @@ function renderProjectWorkItemPromptForAgent(item, workType, agentId, config, pr
   };
 }
 
+function getWorkItemPrRef(item) {
+  if (!item || typeof item !== 'object') return null;
+  return item.targetPr || item.pr || item.pr_id || item.sourcePr || item.pullRequest || item.prUrl || item.prNumber || null;
+}
+
+function resolveWorkItemPrRecord(item, project) {
+  if (!project) return null;
+  const prRef = getWorkItemPrRef(item);
+  if (!prRef) return null;
+  const prs = safeJson(projectPrPath(project)) || [];
+  shared.normalizePrRecords(prs, project);
+  return shared.findPrRecord(prs, prRef, project);
+}
+
+function withWorkItemPrContext(item, pr) {
+  if (!pr) return item;
+  const prNumber = shared.getPrNumber(pr);
+  return {
+    ...item,
+    _pr: pr.id,
+    pr_id: pr.id,
+    targetPr: item.targetPr || pr.id,
+    prNumber: prNumber ?? item.prNumber,
+    pr_number: prNumber ?? item.pr_number,
+    pr_title: pr.title || item.pr_title || item.prTitle || '',
+    pr_branch: pr.branch || item.pr_branch || item.prBranch || '',
+    pr_author: pr.agent || item.pr_author || item.prAuthor || '',
+    pr_url: pr.url || item.pr_url || item.prUrl || '',
+    reviewer: item.reviewer || 'Reviewer',
+    review_note: item.review_note || item.reviewNote || item.description || item.title || 'See PR thread comments',
+  };
+}
 function projectFromDispatchMeta(metaProject, config) {
   if (!metaProject) return null;
   const projects = getProjects(config);
@@ -2775,8 +2815,17 @@ function discoverFromWorkItems(config, project) {
       skipped.noAgent++; continue;
     }
 
+    const linkedPr = resolveWorkItemPrRecord(item, project);
+    const promptItem = linkedPr ? withWorkItemPrContext(item, linkedPr) : item;
+    const prBranch = linkedPr?.branch || '';
+    const isPrTargeted = !!(linkedPr && (workType === WORK_TYPE.FIX || workType === WORK_TYPE.REVIEW || workType === WORK_TYPE.TEST));
+    if (!linkedPr && getWorkItemPrRef(item) && (workType === WORK_TYPE.FIX || workType === WORK_TYPE.REVIEW || workType === WORK_TYPE.TEST)) {
+      if (item._pendingReason !== 'pr_not_found') { item._pendingReason = 'pr_not_found'; needsWrite = true; }
+      log('warn', `Work item ${item.id} references PR ${getWorkItemPrRef(item)} but no tracked PR record was found`);
+      continue;
+    }
     const isShared = item.branchStrategy === 'shared-branch' && item.featureBranch;
-    const branchName = isShared ? item.featureBranch : (item.branch || `work/${item.id}`);
+    const branchName = isPrTargeted && prBranch ? prBranch : (isShared ? item.featureBranch : (item.branch || `work/${item.id}`));
     const deferredAgentResolution = agentId === routing.ANY_AGENT;
 
     // Branch mutex: skip if target branch is locked by an active dispatch
@@ -2789,7 +2838,7 @@ function discoverFromWorkItems(config, project) {
     }
 
     const promptAgentId = deferredAgentResolution ? reservedAgentId : agentId;
-    const promptResult = renderProjectWorkItemPromptForAgent(item, workType, promptAgentId, config, project, root, branchName);
+    const promptResult = renderProjectWorkItemPromptForAgent(promptItem, workType, promptAgentId, config, project, root, branchName);
     if (promptResult.needsReview) {
       log('warn', `Work item ${item.id} exceeded 3 checkpoint-resumes — marking as failed for manual intervention`);
       item.status = WI_STATUS.FAILED;
@@ -2827,7 +2876,7 @@ function discoverFromWorkItems(config, project) {
       agentRole: config.agents[agentId]?.role || tempAgents.get(agentId)?.role || 'Agent',
       task: `[${project?.name || 'project'}] ${item.title || item.description?.slice(0, 80) || item.id}`,
       prompt,
-      meta: { dispatchKey: key, source: 'work-item', branch: branchName, branchStrategy: item.branchStrategy || 'parallel', useExistingBranch: !!(item.branchStrategy === 'shared-branch' && item.featureBranch), item, project: { name: project?.name, localPath: project?.localPath }, deferAgentResolution: deferredAgentResolution }
+      meta: { dispatchKey: key, source: 'work-item', branch: branchName, branchStrategy: item.branchStrategy || 'parallel', useExistingBranch: !!(isPrTargeted || (item.branchStrategy === 'shared-branch' && item.featureBranch)), item: promptItem, project: { name: project?.name, localPath: project?.localPath }, deferAgentResolution: deferredAgentResolution, ...(linkedPr ? { pr: linkedPr } : {}) }
     });
 
     setCooldown(key);

package/package.json

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

package/prompts/cc-system.md

@@ -80,8 +80,8 @@ I'll dispatch dallas to fix that bug.
 - `knowledge`: `title`, `content`, and `category` REQUIRED. Valid categories: architecture, conventions, project-notes, build-reports, reviews.
 
 Core action types:
-- **dispatch**: title (REQUIRED), workType, priority (low/medium/high), agents[] or agent (optional — both shapes accepted), project (REQUIRED when multi-project), description
-  workTypes: `explore` (research/report only, NO PR), `ask` (answer/report, NO PR), `implement` (new code, PR REQUIRED), `fix` (bug fix, PR REQUIRED), `review` (code review, NO PR), `test` (tests, PR if new), `verify` (merge/build/maintenance, NO PR)
+- **dispatch**: title (REQUIRED), workType, priority (low/medium/high), agents[] or agent (optional — both shapes accepted), project (REQUIRED when multi-project unless `pr` resolves to a tracked PR), description, pr (optional PR number/id/url for work that targets an existing PR)
+  workTypes: `explore` (research/report only, NO PR), `ask` (answer/report, NO PR), `implement` (new code, PR REQUIRED), `fix` (standalone bug fix creates a PR; include `pr` when fixing review comments/build failures on an existing PR), `review` (code review, NO PR), `test` (tests, PR if new), `verify` (merge/build/maintenance, NO PR)
   If the user wants a design/architecture artifact committed through a PR, dispatch `implement` or `docs` rather than `explore`.
   When the user names a specific agent ("assign this to lambert"), put exactly that one name in `agents` (e.g. `"agents": ["lambert"]`). A single-agent assignment is hard-pinned by the server — it will queue for that agent only and skip the routing table. Use multi-agent arrays only when the user names multiple agents or asks for fan-out.
 - **build-and-test**: pr, project (optional), agent (optional) — Run the build-and-test playbook against a PR. The agent will checkout the PR branch, run the project's build/test commands, and report results. Use when the user asks to "run tests on PR X" or "build PR X" or after a fix to verify nothing regressed.