@yemi33/minions 0.1.2157 → 0.1.2159

package/dashboard/js/detail-panel.js

@@ -118,7 +118,7 @@ function renderDetailContent(detail, tab) {
     el.innerHTML =
       '<div style="display:flex;gap:6px;margin-bottom:8px">' +
         '<button class="pr-pager-btn" id="charter-edit-btn" style="font-size:var(--text-sm);padding:2px 10px" onclick="_toggleCharterEdit()">Edit</button>' +
-        '<button class="pr-pager-btn" id="charter-save-btn" style="font-size:var(--text-sm);padding:2px 10px;color:var(--green);border-color:var(--green);display:none" onclick="_saveCharter()">Save</button>' +
+        '<button class="modal-copy is-success" id="charter-save-btn" style="display:none" onclick="_saveCharter()">Save</button>' +
         '<button class="pr-pager-btn" id="charter-cancel-btn" style="font-size:var(--text-sm);padding:2px 10px;display:none" onclick="_cancelCharterEdit()">Cancel</button>' +
       '</div>' +
       '<div id="charter-view" class="section">' + renderMd(charterContent || 'No charter found. Click Edit to create one.') + '</div>' +

package/dashboard/js/fre.js

@@ -120,17 +120,10 @@ function renderFre(statusOrProjects) {
     'flex-shrink:0',
   ].join(';');
 
-  const btnPrimary = [
-    'padding:6px 14px',
-    'background:var(--blue)',
-    'color:#fff',
-    'border:none',
-    'border-radius:var(--radius-sm)',
-    'cursor:pointer',
-    'font-size:var(--text-md)',
-    'font-weight:600',
-    'margin-top:8px',
-  ].join(';');
+  // W-mq57qetc0009ed48: blue-primary CTA inline blob removed; "+ Add Project"
+  // now uses the shared .btn-primary-solid primitive. Only the layout-only
+  // margin-top:8px stays inline.
+  const btnPrimaryMarginTop = 'margin-top:8px';
 
   const btnGhost = [
     'padding:6px 14px',
@@ -187,7 +180,7 @@ function renderFre(statusOrProjects) {
             '<div style="color:var(--muted);font-size:var(--text-md);line-height:1.5">' +
               'Projects map a local git worktree to a remote repo. Without a project, agents have nowhere to run.' +
             '</div>' +
-            '<button onclick="addProject()" style="' + btnPrimary + '">+ Add Project</button>' +
+            '<button onclick="addProject()" class="btn-primary-solid" style="' + btnPrimaryMarginTop + '">+ Add Project</button>' +
           '</div>' +
         '</div>' +
 

package/dashboard/js/qa.js

@@ -229,8 +229,8 @@ async function loadQaRunbooks() {
         '</div>' +
       '</div>' +
       '<div class="qa-runbook-actions" style="display:flex;gap:var(--space-2)">' +
-        '<button class="qa-btn-primary qa-runbook-run-btn" data-runbook-id="' + escHtml(id) + '" onclick="qaRunRunbook(this.dataset.runbookId)">Run</button>' +
-        '<button class="qa-btn-ghost qa-btn-danger qa-runbook-delete-btn" data-runbook-id="' + escHtml(id) + '" onclick="qaDeleteRunbook(this.dataset.runbookId)">Delete</button>' +
+        '<button class="btn-primary-solid qa-runbook-run-btn" data-runbook-id="' + escHtml(id) + '" onclick="qaRunRunbook(this.dataset.runbookId)">Run</button>' +
+        '<button class="btn-ghost qa-btn-danger qa-runbook-delete-btn" data-runbook-id="' + escHtml(id) + '" onclick="qaDeleteRunbook(this.dataset.runbookId)">Delete</button>' +
       '</div>' +
       '</div>';
   }).join('');
@@ -350,7 +350,7 @@ function _qaRenderRunRow(run) {
   // to click.
   let deleteBtn = '';
   if (_qaIsTerminalStatus(status)) {
-    deleteBtn = '<button class="qa-btn-ghost qa-run-delete-btn" ' +
+    deleteBtn = '<button class="btn-ghost qa-run-delete-btn" ' +
       'data-run-id="' + escHtml(id) + '" ' +
       'onclick="qaDeleteRun(this.dataset.runId)">Delete</button>';
   }
@@ -731,14 +731,14 @@ function _qaRenderSessionCard(s) {
   //   - terminal          → no actions
   let actions = '';
   if (state === 'awaiting-approval') {
-    actions += '<button class="qa-btn-ghost" onclick="qaSessionReviewDraft(\'' + escHtml(id) + '\')">Review draft</button>';
-    actions += '<button class="qa-btn-primary" onclick="qaSessionApprove(\'' + escHtml(id) + '\')">Approve &amp; run</button>';
-    actions += '<button class="qa-btn-ghost" onclick="qaSessionEditPrompt(\'' + escHtml(id) + '\')">Edit</button>';
-    actions += '<button class="qa-btn-ghost" onclick="qaSessionCancel(\'' + escHtml(id) + '\')">Cancel</button>';
+    actions += '<button class="btn-ghost" onclick="qaSessionReviewDraft(\'' + escHtml(id) + '\')">Review draft</button>';
+    actions += '<button class="btn-primary-solid" onclick="qaSessionApprove(\'' + escHtml(id) + '\')">Approve &amp; run</button>';
+    actions += '<button class="btn-ghost" onclick="qaSessionEditPrompt(\'' + escHtml(id) + '\')">Edit</button>';
+    actions += '<button class="btn-ghost" onclick="qaSessionCancel(\'' + escHtml(id) + '\')">Cancel</button>';
   }
   if (!isTerminal) {
-    actions += '<button class="qa-btn-ghost" onclick="qaSessionDismiss(\'' + escHtml(id) + '\')">Dismiss</button>';
-    actions += '<button class="qa-btn-ghost qa-btn-danger" onclick="qaSessionKill(\'' + escHtml(id) + '\')">Kill spawn</button>';
+    actions += '<button class="btn-ghost" onclick="qaSessionDismiss(\'' + escHtml(id) + '\')">Dismiss</button>';
+    actions += '<button class="btn-ghost qa-btn-danger" onclick="qaSessionKill(\'' + escHtml(id) + '\')">Kill spawn</button>';
   }
 
   let meta = '';
@@ -976,7 +976,7 @@ function _qaBuildModalShell(sessionId) {
   header.appendChild(title);
   const closeBtn = document.createElement('button');
   closeBtn.type = 'button';
-  closeBtn.className = 'qa-btn-ghost qa-modal-close';
+  closeBtn.className = 'btn-ghost qa-modal-close';
   closeBtn.setAttribute('aria-label', 'Close');
   closeBtn.textContent = '×';
   closeBtn.addEventListener('click', _qaModalClose);
@@ -1080,7 +1080,7 @@ function _qaModalRenderBody(payload) {
     if (!payload.truncated) {
       const approve = document.createElement('button');
       approve.type = 'button';
-      approve.className = 'qa-btn-primary';
+      approve.className = 'btn-primary-solid';
       approve.textContent = 'Approve & run';
       approve.addEventListener('click', () => {
         _qaModalClose();
@@ -1090,7 +1090,7 @@ function _qaModalRenderBody(payload) {
     }
     const editBtn = document.createElement('button');
     editBtn.type = 'button';
-    editBtn.className = 'qa-btn-ghost';
+    editBtn.className = 'btn-ghost';
     editBtn.textContent = 'Request edit';
     editBtn.addEventListener('click', () => {
       _qaModalClose();
@@ -1099,7 +1099,7 @@ function _qaModalRenderBody(payload) {
     footer.appendChild(editBtn);
     const closeBtn = document.createElement('button');
     closeBtn.type = 'button';
-    closeBtn.className = 'qa-btn-ghost';
+    closeBtn.className = 'btn-ghost';
     closeBtn.textContent = 'Close';
     closeBtn.addEventListener('click', _qaModalClose);
     footer.appendChild(closeBtn);

package/dashboard/js/render-agents.js

@@ -143,7 +143,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 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:var(--text-md)">Retry</button>' +
+        '<button data-agent-id="' + escapeHtml(id) + '" onclick="openAgentDetail(this.dataset.agentId)" class="btn-primary-lg">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:var(--text-md);color:var(--text)">Close</button>' +
       '</div>';
   }

package/dashboard/js/render-inbox.js

@@ -270,7 +270,7 @@ function openQuickNoteModal() {
         '<textarea id="note-content" rows="6" 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 note... Agents will see this after consolidation."></textarea></label>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px">' +
         '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
-        '<button onclick="submitQuickNote()" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Save Note</button>' +
+        '<button onclick="submitQuickNote()" class="btn-primary-lg">Save Note</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');

package/dashboard/js/render-kb.js

@@ -222,7 +222,7 @@ 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(event)" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Save</button>' +
+        '<button onclick="submitKbEntry(event)" class="btn-primary-lg">Save</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');

package/dashboard/js/render-meetings.js

@@ -123,7 +123,7 @@ function renderMeetings(meetings, opts) {
           (m.status === 'archived'
             ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px" onclick="event.stopPropagation();_unarchiveMeeting(\'' + escHtml(m.id) + '\')">Unarchive</button>'
             : (m.status === 'completed' ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px" onclick="event.stopPropagation();_archiveMeeting(\'' + escHtml(m.id) + '\')">Archive</button>' : '')) +
-          '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--red);border-color:var(--red)" onclick="event.stopPropagation();_deleteMeeting(\'' + escHtml(m.id) + '\')">Delete</button>' +
+          '<button class="btn-destructive" onclick="event.stopPropagation();_deleteMeeting(\'' + escHtml(m.id) + '\')">Delete</button>' +
         '</div>' +
       '</div>' +
       (timeStr ? '<div style="margin-top:4px;font-size:var(--text-sm);color:var(--muted)">' + escHtml(timeStr) + '</div>' : '') +
@@ -246,17 +246,17 @@ function _renderMeetingDetail(m) {
       if (m.status === 'archived') {
         html += '<div style="display:flex;gap:8px;border-top:1px solid var(--border);padding-top:8px">' +
           '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px" onclick="_unarchiveMeeting(\'' + escHtml(m.id) + '\')">Unarchive</button>' +
-          '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_deleteMeeting(\'' + escHtml(m.id) + '\')">Delete</button>' +
+          '<button class="btn-destructive btn-destructive-md" onclick="_deleteMeeting(\'' + escHtml(m.id) + '\')">Delete</button>' +
         '</div>';
       } else if (m.status === 'completed') {
         const linkedPlan = _findLinkedPlan(m);
         html += '<div style="display:flex;gap:8px;flex-wrap:wrap;border-top:1px solid var(--border);padding-top:8px">' +
           (linkedPlan
             ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--blue);border-color:var(--blue)" onclick="_viewPlanWithBack(\'' + escHtml(linkedPlan.file) + '\',\'' + escHtml(m.id) + '\')">View Plan</button>' +
-              '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--green);border-color:var(--green)" onclick="_createPlanFromMeeting(\'' + escHtml(m.id) + '\',this)">New Plan</button>'
-            : '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--green);border-color:var(--green)" onclick="_createPlanFromMeeting(\'' + escHtml(m.id) + '\',this)">Create Plan from Meeting</button>') +
+              '<button class="btn-action btn-action-md" onclick="_createPlanFromMeeting(\'' + escHtml(m.id) + '\',this)">New Plan</button>'
+            : '<button class="btn-action btn-action-md" onclick="_createPlanFromMeeting(\'' + escHtml(m.id) + '\',this)">Create Plan from Meeting</button>') +
           '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px" onclick="_archiveMeeting(\'' + escHtml(m.id) + '\')">Archive</button>' +
-          '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_deleteMeeting(\'' + escHtml(m.id) + '\')">Delete</button>' +
+          '<button class="btn-destructive btn-destructive-md" onclick="_deleteMeeting(\'' + escHtml(m.id) + '\')">Delete</button>' +
         '</div>' +
         '<div id="meeting-plan-toast" class="cmd-toast cmd-toast-inline" style="margin-top:6px"></div>' +
         '<div style="font-size:var(--text-xs);color:var(--muted);margin-top:4px">Use the Q&amp;A below to discuss action items' + (linkedPlan ? '' : ', then create a plan to execute them') + '.</div>';
@@ -267,8 +267,8 @@ function _renderMeetingDetail(m) {
         '</div>' +
         '<div style="display:flex;gap:8px;margin-top:4px">' +
           '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--yellow);border-color:var(--yellow)" onclick="_advanceMeeting(\'' + escHtml(m.id) + '\',this)">Skip to Next Round</button>' +
-          '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_endMeeting(\'' + escHtml(m.id) + '\',this)">End Meeting</button>' +
-          '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_deleteMeeting(\'' + escHtml(m.id) + '\')">Delete</button>' +
+          '<button class="btn-destructive btn-destructive-md" onclick="_endMeeting(\'' + escHtml(m.id) + '\',this)">End Meeting</button>' +
+          '<button class="btn-destructive btn-destructive-md" onclick="_deleteMeeting(\'' + escHtml(m.id) + '\')">Delete</button>' +
         '</div>';
       }
 
@@ -347,7 +347,7 @@ 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(event)" 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)" class="btn-primary-lg">Start Meeting</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');

package/dashboard/js/render-other.js

@@ -5,8 +5,8 @@ function renderProjects(projects) {
   const visible = (projects || []).filter(p => typeof isDeleted === 'function' ? !isDeleted('project:' + p.name) : true);
   if (!visible.length) {
     list.innerHTML = '<span style="color:var(--muted);font-style:italic">No projects linked.</span>' +
-      '<span onclick="addProject()" style="background:var(--surface2);border:1px solid var(--border);border-radius:4px;padding:3px 10px;color:var(--green);font-weight:500;cursor:pointer;border-style:dashed;margin-left:8px">+ Add Project</span>' +
-      '<span onclick="openScanProjectsModal()" style="background:var(--surface2);border:1px solid var(--border);border-radius:4px;padding:3px 10px;color:var(--blue);font-weight:500;cursor:pointer;border-style:dashed;font-size:var(--text-sm)">Scan</span>';
+      '<span onclick="addProject()" class="empty-state-add" style="margin-left:8px">+ Add Project</span>' +
+      '<span onclick="openScanProjectsModal()" class="empty-state-add empty-state-add-blue">Scan</span>';
     return;
   }
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() (fields: project name, path, branch metadata)
@@ -17,8 +17,8 @@ function renderProjects(projects) {
       '<span onclick="event.stopPropagation();projectChipRemove(\'' + escHtml(p.name) + '\')" title="Remove project (cancels pending work, archives data dir)" style="color:var(--muted);font-weight:600;cursor:pointer;padding:0 2px;line-height:1" onmouseover="this.style.color=\'var(--red)\'" onmouseout="this.style.color=\'var(--muted)\'">&times;</span>' +
     '</span>'
   ).join('') +
-  '<span onclick="addProject()" style="background:var(--surface2);border:1px solid var(--border);border-radius:4px;padding:3px 10px;color:var(--muted);font-weight:500;cursor:pointer;border-style:dashed">+ Add</span>' +
-  '<span onclick="openScanProjectsModal()" style="background:var(--surface2);border:1px solid var(--border);border-radius:4px;padding:3px 10px;color:var(--blue);font-weight:500;cursor:pointer;border-style:dashed;font-size:var(--text-sm)">Scan</span>';
+  '<span onclick="addProject()" class="empty-state-add empty-state-add-muted">+ Add</span>' +
+  '<span onclick="openScanProjectsModal()" class="empty-state-add empty-state-add-blue">Scan</span>';
 
 }
 
@@ -422,7 +422,7 @@ async function openScanProjectsModal() {
         '<label style="width:60px;color:var(--text);font-size:var(--text-md)">Depth' +
           '<input id="scan-depth" type="number" value="3" min="1" max="6" 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);font-size:var(--text-md)">' +
         '</label>' +
-        '<button onclick="_runProjectScan()" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer;white-space:nowrap">Scan</button>' +
+        '<button onclick="_runProjectScan()" class="btn-primary-lg" style="white-space:nowrap">Scan</button>' +
       '</div>' +
       '<div id="scan-results" style="color:var(--muted);font-size:var(--text-md)">Click Scan to find git repos in the directory.</div>' +
       '<div class="cmd-toast" id="scan-toast" style="margin-top:0"></div>' +

package/dashboard/js/render-pinned.js

@@ -17,7 +17,7 @@ function renderPinned(entries) {
       ';border-radius:4px;cursor:pointer" onclick="if(shouldIgnoreSelectionClick(event))return;openPinnedView(' + i + ')">' +
       '<div style="display:flex;justify-content:space-between;align-items:center">' +
         '<strong style="font-size:var(--text-md)">' + escHtml(e.title) + '</strong>' +
-        '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--red);border-color:var(--red)" data-pin-title="' + escHtml(e.title) + '" onclick="event.stopPropagation();removePinnedNote(this.dataset.pinTitle)">Unpin</button>' +
+        '<button class="btn-destructive" data-pin-title="' + escHtml(e.title) + '" onclick="event.stopPropagation();removePinnedNote(this.dataset.pinTitle)">Unpin</button>' +
       '</div>' +
       '<div style="font-size:var(--text-sm);color:var(--muted);margin-top:4px">' + renderMd(e.content.slice(0, 200)) + (e.content.length > 200 ? '...' : '') + '</div>' +
     '</div>'
@@ -44,7 +44,7 @@ function openPinNoteModal() {
       '<label style="color:var(--text);font-size:var(--text-md)">Title<input id="pin-title" 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)" placeholder="e.g. API freeze until Friday"></label>' +
       '<label style="color:var(--text);font-size:var(--text-md)">Content<textarea id="pin-content" rows="4" 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="Context all agents should see..."></textarea></label>' +
       '<label style="color:var(--text);font-size:var(--text-md)">Level<select id="pin-level" 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)"><option value="info">Info</option><option value="warning">Warning</option><option value="critical">Critical</option></select></label>' +
-      '<div style="display:flex;justify-content:flex-end;gap:8px"><button onclick="closeModal()" class="pr-pager-btn">Cancel</button><button onclick="submitPinnedNote(event)" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Pin</button></div>' +
+      '<div style="display:flex;justify-content:flex-end;gap:8px"><button onclick="closeModal()" class="pr-pager-btn">Cancel</button><button onclick="submitPinnedNote(event)" class="btn-primary-lg">Pin</button></div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');
 }
@@ -194,7 +194,7 @@ function openPinnedEdit(idx) {
   cancelBtn.textContent = 'Cancel';
   cancelBtn.onclick = function() { openPinnedView(idx); };
   const saveBtn = document.createElement('button');
-  saveBtn.style.cssText = 'padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer';
+  saveBtn.className = 'btn-primary-lg';
   saveBtn.textContent = 'Save';
   saveBtn.onclick = function(ev) { submitPinnedEdit(originalTitle, ev); };
   actions.appendChild(cancelBtn);

package/dashboard/js/render-pipelines.js

@@ -399,12 +399,12 @@ function openPipelineDetail(id) {
     '<span style="font-size:var(--text-sm);color:var(--muted)">' + _renderPipelineTriggerLabel(p.trigger?.cron) + ' · ' + escHtml(_getPipelineStageLabel(p)) + '</span>' +
     '<div style="display:flex;gap:6px">' +
       (activeRun
-        ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_abortPipeline(\'' + escHtml(id) + '\',this)">Abort</button>' +
+        ? '<button class="btn-destructive btn-destructive-md" onclick="_abortPipeline(\'' + escHtml(id) + '\',this)">Abort</button>' +
           '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--yellow);border-color:var(--yellow)" onclick="_retriggerPipeline(\'' + escHtml(id) + '\',this)">Retrigger</button>'
-        : '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--green);border-color:var(--green)" onclick="_triggerPipeline(\'' + escHtml(id) + '\',this)">Run Now</button>') +
+        : '<button class="btn-action btn-action-md" onclick="_triggerPipeline(\'' + escHtml(id) + '\',this)">Run Now</button>') +
       '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--blue);border-color:var(--blue)" onclick="openEditPipelineModal(\'' + escHtml(id) + '\')">Edit</button>' +
       '<button class="pr-pager-btn' + (_pipelineToggleInFlight.has(id) ? ' disabled' : '') + '" style="font-size:var(--text-xs);padding:2px 8px" onclick="_togglePipelineEnabled(\'' + escHtml(id) + '\',' + !p.enabled + ',this)">' + (p.enabled !== false ? 'Disable' : 'Enable') + '</button>' +
-      '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_deletePipelineConfirm(\'' + escHtml(id) + '\')">Delete</button>' +
+      '<button class="btn-destructive btn-destructive-md" onclick="_deletePipelineConfirm(\'' + escHtml(id) + '\')">Delete</button>' +
     '</div>' +
   '</div>';
 
@@ -454,7 +454,7 @@ function openPipelineDetail(id) {
       _renderMonitoredResources(s.monitoredResources || []) +
       _renderArtifactLinks(stageRun.artifacts, id) +
       (stageRun.output ? '<div class="pipeline-stage-output" style="margin-top:6px;font-size:var(--text-base);max-height:150px;overflow-y:auto">' + renderMd(stageRun.output.slice(0, 500)) + '</div>' : '') +
-      (stageStatus === 'waiting-human' ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;color:var(--green);border-color:var(--green);margin-top:6px" onclick="_continuePipeline(\'' + escHtml(id) + '\',\'' + escHtml(s.id) + '\',this)">Continue</button>' : '') +
+      (stageStatus === 'waiting-human' ? '<button class="btn-action btn-action-md" style="margin-top:6px" onclick="_continuePipeline(\'' + escHtml(id) + '\',\'' + escHtml(s.id) + '\',this)">Continue</button>' : '') +
     '</div>';
   });
 
@@ -726,7 +726,7 @@ function openCreatePipelineModal() {
       '<label style="color:var(--text);font-size:var(--text-md)">Stages (JSON array)<textarea id="pl-stages" rows="10" style="' + inputStyle + ';resize:vertical;font-family:Consolas,monospace" placeholder=\'[{"id":"audit","type":"task","title":"Audit codebase","taskType":"explore"},{"id":"discuss","type":"meeting","title":"Discuss findings","dependsOn":["audit"],"participants":["all"]}]\'></textarea></label>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px;margin-top:4px">' +
         '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
-        '<button onclick="_submitCreatePipeline()" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Create Pipeline</button>' +
+        '<button onclick="_submitCreatePipeline()" class="btn-primary-lg">Create Pipeline</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');
@@ -828,7 +828,7 @@ function openEditPipelineModal(id) {
       '<label style="color:var(--text);font-size:var(--text-md)">Stages (JSON array)<textarea id="pl-stages" rows="10" style="' + inputStyle + ';resize:vertical;font-family:Consolas,monospace">' + escHtml(JSON.stringify(p.stages || [], null, 2)) + '</textarea></label>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px;margin-top:4px">' +
         '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
-        '<button onclick="_submitEditPipeline()" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Save</button>' +
+        '<button onclick="_submitEditPipeline()" class="btn-primary-lg">Save</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal-body').style.whiteSpace = 'normal';

package/dashboard/js/render-plans.js

@@ -22,7 +22,7 @@ function openCreatePlanModal() {
       '<div style="font-size:var(--text-base);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(event)" 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)" class="btn-primary-lg">Create Plan</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');
@@ -468,7 +468,7 @@ function planReexecuteModal(file, project) {
       '<div style="font-size:var(--text-base);color:var(--muted)">Leave blank to re-run without steering. The agent will read the existing PRD and rewrite it.</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="planReexecuteSubmit(\'' + escapeHtml(file) + '\',\'' + escapeHtml(project || '') + '\')" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Re-execute</button>' +
+        '<button onclick="planReexecuteSubmit(\'' + escapeHtml(file) + '\',\'' + escapeHtml(project || '') + '\')" class="btn-primary-lg">Re-execute</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');

package/dashboard/js/render-prd.js

@@ -105,15 +105,15 @@ function renderPrd(prd, prog) {
     let actions = '';
     if (prdFile && !headerStale) {
       if (effectiveStatus === 'awaiting-approval') {
-        actions = ' <button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--green);border-color:var(--green);margin-left:4px" onclick="planApprove(\'' + escHtml(prdFile) + '\',this)">Approve</button>';
+        actions = ' <button class="btn-action" style="margin-left:4px" onclick="planApprove(\'' + escHtml(prdFile) + '\',this)">Approve</button>';
       } else if (effectiveStatus === 'completed') {
         const hasVerifyWi = allWi.some(w => w.itemType === 'verify' && w.sourcePlan === prdFile);
-        actions = (hasVerifyWi ? '' : ' <button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--green);border-color:var(--green);margin-left:4px" onclick="triggerVerify(\'' + escHtml(prdFile) + '\',this)">Verify</button>') +
+        actions = (hasVerifyWi ? '' : ' <button class="btn-action" style="margin-left:4px" onclick="triggerVerify(\'' + escHtml(prdFile) + '\',this)">Verify</button>') +
           ' <button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;margin-left:4px" onclick="planArchive(\'' + escHtml(prdFile) + '\',this)">Archive</button>';
       } else if (effectiveStatus === 'dispatched') {
         actions = ' <button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--yellow);border-color:var(--yellow);margin-left:4px" onclick="planPause(\'' + escHtml(prdFile) + '\',this)">Pause</button>';
       } else if (effectiveStatus === 'paused') {
-        actions = ' <button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--green);border-color:var(--green);margin-left:4px" onclick="planApprove(\'' + escHtml(prdFile) + '\',this)">Resume</button>' +
+        actions = ' <button class="btn-action" style="margin-left:4px" onclick="planApprove(\'' + escHtml(prdFile) + '\',this)">Resume</button>' +
           ' <button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;margin-left:4px" onclick="planArchive(\'' + escHtml(prdFile) + '\',this)">Archive</button>';
       }
     }
@@ -823,7 +823,7 @@ async function prdItemEdit(source, itemId) {
       '<label style="font-size:var(--text-base);color:var(--muted)">Description</label>' +
       '<div style="display:flex;gap:6px">' +
         '<button id="prd-desc-edit-btn" type="button" class="pr-pager-btn" aria-label="Edit description" style="font-size:var(--text-sm);padding:2px 10px" onclick="_togglePrdDescEdit()">Edit</button>' +
-        '<button id="prd-desc-save-btn" type="button" class="pr-pager-btn" aria-label="Save description" style="font-size:var(--text-sm);padding:2px 10px;color:var(--green);border-color:var(--green);display:none" onclick="_savePrdDesc(\'' + escHtml(source) + '\',\'' + escHtml(itemId) + '\')">Save</button>' +
+        '<button id="prd-desc-save-btn" type="button" class="btn-action btn-action-lg" aria-label="Save description" style="display:none" onclick="_savePrdDesc(\'' + escHtml(source) + '\',\'' + escHtml(itemId) + '\')">Save</button>' +
         '<button id="prd-desc-cancel-btn" type="button" class="pr-pager-btn" aria-label="Cancel description edit" style="font-size:var(--text-sm);padding:2px 10px;display:none" onclick="_cancelPrdDescEdit()">Cancel</button>' +
       '</div>' +
     '</div>' +

package/dashboard/js/render-prs.js

@@ -204,7 +204,7 @@ function prRow(pr) {
     '<td><span class="pr-badge ' + statusClass + '" title="' + escapeHtml(statusLabel) + '">' + escapeHtml(statusLabel) + '</span></td>' +
     '<td>' + (observeBtn || '<span style="color:var(--muted);font-size:var(--text-base)">—</span>') + '</td>' +
     '<td><span class="pr-date" title="' + escapeHtml(createdLabel) + '">' + escapeHtml(createdLabel) + '</span></td>' +
-    '<td><button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 5px;color:var(--red);border-color:var(--red)" data-pr-id="' + escapeHtml(String(prId)) + '" onclick="event.stopPropagation();unlinkPr(this.dataset.prId)" title="Remove from tracking">x</button></td>' +
+    '<td><button class="btn-destructive" style="padding:1px 5px" data-pr-id="' + escapeHtml(String(prId)) + '" onclick="event.stopPropagation();unlinkPr(this.dataset.prId)" title="Remove from tracking">x</button></td>' +
     '</tr>';
 }
 
@@ -350,7 +350,7 @@ function openAddPrModal() {
       '<div style="font-size:var(--text-base);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(event)" 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)" class="btn-primary-lg">Link PR</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');

package/dashboard/js/render-schedules.js

@@ -352,10 +352,10 @@ function renderSchedules(schedules, opts) {
         '<td>' + enabledBadge + '</td>' +
         '<td><span class="pr-date">' + escHtml(lastRun) + '</span></td>' +
         '<td style="white-space:nowrap">' +
-          '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--green);border-color:var(--green);margin-right:4px" onclick="event.stopPropagation();runScheduleNow(\'' + escHtml(s.id) + '\',this)" title="Run now">Run now</button>' +
+          '<button class="btn-action" style="margin-right:4px" onclick="event.stopPropagation();runScheduleNow(\'' + escHtml(s.id) + '\',this)" title="Run now">Run now</button>' +
           '<button class="pr-pager-btn' + (_schedToggleInFlight.has(s.id) ? ' disabled' : '') + '" style="font-size:var(--text-xs);padding:1px 6px;color:' + (s.enabled ? 'var(--yellow)' : 'var(--green)') + ';border-color:' + (s.enabled ? 'var(--yellow)' : 'var(--green)') + ';margin-right:4px" onclick="event.stopPropagation();toggleScheduleEnabled(\'' + escHtml(s.id) + '\',' + !s.enabled + ')" title="' + (s.enabled ? 'Disable' : 'Enable') + '">' + (s.enabled ? '&#x23F8;' : '&#x25B6;') + '</button>' +
           '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--blue);border-color:var(--blue);margin-right:4px" onclick="event.stopPropagation();openEditScheduleModal(\'' + escHtml(s.id) + '\')" title="Edit">&#x270E;</button>' +
-          '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--red);border-color:var(--red)" onclick="event.stopPropagation();deleteSchedule(\'' + escHtml(s.id) + '\')" title="Delete">&#x2715;</button>' +
+          '<button class="btn-destructive" onclick="event.stopPropagation();deleteSchedule(\'' + escHtml(s.id) + '\')" title="Delete">&#x2715;</button>' +
         '</td>' +
       '</tr>';
     }
@@ -515,7 +515,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 + ',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>' +
+      '<button onclick="submitSchedule(' + isEdit + ',event)" class="btn-primary-lg">' + (isEdit ? 'Save' : 'Create') + '</button>' +
     '</div>' +
   '</div>';
 }

package/dashboard/js/render-watches.js

@@ -216,11 +216,11 @@ function renderWatches(watchesData, opts) {
     if (w.status === 'active') {
       html += '<button class="pr-pager-btn' + pauseBusyCls + '" style="font-size:var(--text-xs);padding:1px 6px;color:var(--yellow);border-color:var(--yellow);margin-right:4px" onclick="event.stopPropagation();toggleWatchPause(\'' + escHtml(w.id) + '\',true)" title="Pause">&#x23F8;</button>';
     } else if (w.status === 'paused') {
-      html += '<button class="pr-pager-btn' + pauseBusyCls + '" style="font-size:var(--text-xs);padding:1px 6px;color:var(--green);border-color:var(--green);margin-right:4px" onclick="event.stopPropagation();toggleWatchPause(\'' + escHtml(w.id) + '\',false)" title="Resume">&#x25B6;</button>';
+      html += '<button class="btn-action' + pauseBusyCls + '" style="margin-right:4px" onclick="event.stopPropagation();toggleWatchPause(\'' + escHtml(w.id) + '\',false)" title="Resume">&#x25B6;</button>';
     }
 
     // Delete button
-    html += '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--red);border-color:var(--red)" onclick="event.stopPropagation();deleteWatch(\'' + escHtml(w.id) + '\')" title="Delete">&#x2715;</button>';
+    html += '<button class="btn-destructive" onclick="event.stopPropagation();deleteWatch(\'' + escHtml(w.id) + '\')" title="Delete">&#x2715;</button>';
     html += '</td></tr>';
   }
 
@@ -548,7 +548,7 @@ function _addWatchStepRow(initial) {
   row.innerHTML =
     '<div style="display:flex;align-items:center;justify-content:space-between;gap:8px">' +
       '<select id="watch-step-' + idx + '-type" style="' + inputStyle + ';margin-top:0;flex:1" onchange="_updateWatchStepParamsHint(' + idx + ')">' + actionOpts + '</select>' +
-      '<button type="button" class="pr-pager-btn" style="font-size:var(--text-sm);padding:2px 8px;color:var(--red);border-color:var(--red)" onclick="_removeWatchStepRow(' + idx + ')">Remove</button>' +
+      '<button type="button" class="btn-destructive btn-destructive-lg" onclick="_removeWatchStepRow(' + idx + ')">Remove</button>' +
     '</div>' +
     '<label style="color:var(--text);font-size:var(--text-base)">Params (JSON) <span style="font-size:var(--text-sm);color:var(--muted)" id="watch-step-' + idx + '-params-hint"></span>' +
       '<textarea id="watch-step-' + idx + '-params" rows="3" placeholder=\'{"title": "Re-check {{target}} after merge"}\' style="' + inputStyle + ';font-family:monospace"></textarea>' +
@@ -612,7 +612,7 @@ function _addRequireRow(initial) {
     '<label style="color:var(--text);font-size:var(--text-base);margin:0">Target<input id="watch-require-' + idx + '-target" placeholder="e.g. PR-123, pipeline-X" style="' + inputStyle + '"></label>' +
     '<label style="color:var(--text);font-size:var(--text-base);margin:0">Target Type<select id="watch-require-' + idx + '-target-type" style="' + inputStyle + '" onchange="_updateRequireConditionOptions(' + idx + ')">' + ttOpts + '</select></label>' +
     '<label style="color:var(--text);font-size:var(--text-base);margin:0">Condition<select id="watch-require-' + idx + '-condition" style="' + inputStyle + '"></select></label>' +
-    '<button type="button" class="pr-pager-btn" style="font-size:var(--text-sm);padding:6px 8px;color:var(--red);border-color:var(--red)" onclick="_removeRequireRow(' + idx + ')">Remove</button>';
+    '<button type="button" class="btn-destructive btn-destructive-lg" style="padding:6px 8px" onclick="_removeRequireRow(' + idx + ')">Remove</button>';
   container.appendChild(row);
   if (initTarget) document.getElementById('watch-require-' + idx + '-target').value = initTarget;
   if (initTT) document.getElementById('watch-require-' + idx + '-target-type').value = initTT;
@@ -655,7 +655,7 @@ function _updateWatchConditionOptions() {
 
 function _renderCreateWatchModal() {
   document.getElementById('modal-title').innerHTML = 'Create Watch' +
-    ' <button class="pr-pager-btn" style="font-size:var(--text-sm);padding:2px 12px;color:var(--green);border-color:var(--green);margin-left:8px" onclick="submitWatch()">Create</button>';
+    ' <button class="btn-action btn-action-lg" style="padding:2px 12px;margin-left:8px" onclick="submitWatch()">Create</button>';
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() by _watchFormHtml() (fields: target/action registries, project names, agent ids/names)
   document.getElementById('modal-body').innerHTML = _watchFormHtml();
   document.getElementById('modal-body').style.whiteSpace = 'normal';

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

@@ -246,9 +246,9 @@ function wiRow(item) {
     '<td style="white-space:nowrap">' +
       ((item.status === 'pending' || item.status === 'failed') ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--blue);border-color:var(--blue);margin-right:4px" onclick="event.stopPropagation();editWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Edit work item">&#x270E;</button>' : '') +
       ((item.status === 'done' || item.status === 'failed') ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--muted);border-color:var(--border);margin-right:4px" onclick="event.stopPropagation();archiveWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Archive work item">&#x1F4E6;</button>' : '') +
-      ((item.status === 'done' || item.status === 'failed') && !item._humanFeedback ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--green);border-color:var(--green);margin-right:4px" onclick="event.stopPropagation();feedbackWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Give feedback">&#x1F44D;&#x1F44E;</button>' : (item._humanFeedback ? '<span style="font-size:var(--text-xs)" title="Feedback given">' + (item._humanFeedback.rating === 'up' ? '&#x1F44D;' : '&#x1F44E;') + '</span> ' : '')) +
+      ((item.status === 'done' || item.status === 'failed') && !item._humanFeedback ? '<button class="btn-action" style="margin-right:4px" onclick="event.stopPropagation();feedbackWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Give feedback">&#x1F44D;&#x1F44E;</button>' : (item._humanFeedback ? '<span style="font-size:var(--text-xs)" title="Feedback given">' + (item._humanFeedback.rating === 'up' ? '&#x1F44D;' : '&#x1F44E;') + '</span> ' : '')) +
       ((item.status === 'pending' || item.status === 'dispatched' || item.status === 'queued' || item.status === 'failed') ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--orange);border-color:var(--orange);margin-right:4px" onclick="event.stopPropagation();cancelWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Cancel work item and kill agent">&#x1F6AB;</button>' : '') +
-      '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--red);border-color:var(--red)" onclick="event.stopPropagation();deleteWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Delete work item and kill agent">&#x2715;</button>' +
+      '<button class="btn-destructive" onclick="event.stopPropagation();deleteWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Delete work item and kill agent">&#x2715;</button>' +
     '</td>' +
   '</tr>';
 }
@@ -380,7 +380,7 @@ async function editWorkItem(id, source) {
       '</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="submitWorkItemEdit(\'' + escapeHtml(id) + '\',\'' + escapeHtml(source || '') + '\',event)" style="padding:6px 16px;font-size:var(--text-md);background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Save</button>' +
+        '<button onclick="submitWorkItemEdit(\'' + escapeHtml(id) + '\',\'' + escapeHtml(source || '') + '\',event)" class="btn-primary-lg">Save</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');
@@ -601,7 +601,7 @@ function openCreateWorkItemModal() {
       '<label id="wi-new-skippr-row" style="color:var(--text);font-size:var(--text-md);display:flex;gap:8px;align-items:center;cursor:pointer"><input type="checkbox" id="wi-new-skippr"> Skip PR creation (push branch only)</label>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px;margin-top:4px">' +
         '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
-        '<button onclick="_submitCreateWorkItem(event)" style="padding:6px 16px;background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Create</button>' +
+        '<button onclick="_submitCreateWorkItem(event)" class="btn-primary-lg">Create</button>' +
       '</div>' +
     '</div>';
   document.getElementById('modal').classList.add('open');

package/dashboard/js/settings.js

@@ -488,8 +488,7 @@ async function openSettings() {
   if (!document.getElementById('modal-settings-reset')) {
     const resetBtn = document.createElement('button');
     resetBtn.id = 'modal-settings-reset';
-    resetBtn.className = 'modal-copy';
-    resetBtn.style.cssText = 'color:var(--red);border-color:var(--red)';
+    resetBtn.className = 'modal-copy is-danger';
     resetBtn.textContent = 'Reset';
     resetBtn.onclick = resetSettingsToDefaults;
     actions.insertBefore(resetBtn, actions.lastElementChild);
@@ -497,8 +496,7 @@ async function openSettings() {
   if (!document.getElementById('modal-settings-save')) {
     const saveBtn = document.createElement('button');
     saveBtn.id = 'modal-settings-save';
-    saveBtn.className = 'modal-copy';
-    saveBtn.style.cssText = 'color:var(--green);border-color:var(--green)';
+    saveBtn.className = 'modal-copy is-success';
     saveBtn.innerHTML = '<svg width="12" height="12" viewBox="0 0 16 16" fill="currentColor"><path d="M13.78 4.22a.75.75 0 010 1.06l-7.25 7.25a.75.75 0 01-1.06 0L2.22 9.28a.75.75 0 011.06-1.06L6 10.94l6.72-6.72a.75.75 0 011.06 0z"/></svg> Save';
     saveBtn.onclick = saveSettings;
     actions.insertBefore(saveBtn, actions.lastElementChild);

package/dashboard/layout.html

@@ -122,7 +122,7 @@
       <h3 id="modal-title">—</h3>
       <div class="modal-header-actions">
         <button class="modal-copy" id="modal-edit-btn" onclick="modalToggleEdit()" title="Edit" style="display:none"><svg width="12" height="12" viewBox="0 0 16 16" fill="currentColor"><path d="M11.013 1.427a1.75 1.75 0 012.474 0l1.086 1.086a1.75 1.75 0 010 2.474l-8.61 8.61c-.21.21-.47.364-.756.445l-3.251.93a.75.75 0 01-.927-.928l.929-3.25c.081-.286.235-.547.445-.758l8.61-8.61zM12.9 2.97L4.288 11.58l-.537 1.878 1.878-.537L14.242 4.31 12.9 2.97z"/></svg> Edit</button>
-        <button class="modal-copy" id="modal-save-btn" onclick="modalSaveEdit()" title="Save" style="display:none;color:var(--green);border-color:var(--green)"><svg width="12" height="12" viewBox="0 0 16 16" fill="currentColor"><path d="M13.78 4.22a.75.75 0 010 1.06l-7.25 7.25a.75.75 0 01-1.06 0L2.22 9.28a.75.75 0 011.06-1.06L6 10.94l6.72-6.72a.75.75 0 011.06 0z"/></svg> Save</button>
+        <button class="modal-copy is-success" id="modal-save-btn" onclick="modalSaveEdit()" title="Save" style="display:none"><svg width="12" height="12" viewBox="0 0 16 16" fill="currentColor"><path d="M13.78 4.22a.75.75 0 010 1.06l-7.25 7.25a.75.75 0 01-1.06 0L2.22 9.28a.75.75 0 011.06-1.06L6 10.94l6.72-6.72a.75.75 0 011.06 0z"/></svg> Save</button>
         <button class="modal-copy" id="modal-cancel-edit-btn" onclick="modalCancelEdit()" title="Cancel edit" style="display:none">Cancel</button>
         <button class="modal-copy pin-btn" id="modal-pin-btn" onclick="toggleModalPin()" title="Pin to top" style="display:none">Pin</button>
         <button class="modal-copy" id="modal-copy-btn" onclick="copyModalContent()" title="Copy to clipboard"><svg width="12" height="12" viewBox="0 0 16 16" fill="currentColor"><path d="M0 6.75C0 5.784.784 5 1.75 5h1.5a.75.75 0 010 1.5h-1.5a.25.25 0 00-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 00.25-.25v-1.5a.75.75 0 011.5 0v1.5A1.75 1.75 0 019.25 16h-7.5A1.75 1.75 0 010 14.25z"/><path d="M5 1.75C5 .784 5.784 0 6.75 0h7.5C15.216 0 16 .784 16 1.75v7.5A1.75 1.75 0 0114.25 11h-7.5A1.75 1.75 0 015 9.25zm1.75-.25a.25.25 0 00-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 00.25-.25v-7.5a.25.25 0 00-.25-.25z"/></svg> Copy</button>

package/dashboard/pages/qa.html

@@ -15,7 +15,7 @@
             </ul>
           </div>
           <div style="display:flex;gap:var(--space-3);align-items:center;margin-top:var(--space-2)">
-            <button type="button" id="qa-cc-pitch-open" class="qa-btn-primary" onclick="qaFocusCommandCenter()">Open Command Center →</button>
+            <button type="button" id="qa-cc-pitch-open" class="btn-primary-solid" onclick="qaFocusCommandCenter()">Open Command Center →</button>
             <a href="/state/docs/qa-runbook-lifecycle.md" target="_blank" rel="noopener" class="qa-cc-pitch-docs" style="color:var(--blue);font-size:var(--text-base)">Docs →</a>
           </div>
         </div>

package/dashboard/styles.css

@@ -488,6 +488,170 @@
   .btn-add.disabled { opacity: 0.4; cursor: not-allowed; pointer-events: none; }
   .btn-add.btn-add-lg { font-size: var(--text-sm); padding: 2px 8px; }
 
+  /* W-mq57qetc0009ed48 — green-bordered action button family WITHOUT a
+     leading "+" glyph (Approve / Verify / Resume / Save / Run Now /
+     Continue / Create / New Plan / Trigger Verify / etc). Sibling to
+     .btn-add — same green/surface2 visual identity, but semantically a
+     verb-action rather than a creator. Three padding+size variants cover
+     every callsite that was previously inline `style="font-size:…;
+     padding:…;color:var(--green);border-color:var(--green)"` on top of
+     .pr-pager-btn:
+       .btn-action            — compact text-xs / 1px 6px (inline PRD
+                                 chips, work-item feedback)
+       .btn-action.btn-action-md — text-xs / 2px 8px (meetings, pipelines,
+                                 watches resume, schedule run-now chip)
+       .btn-action.btn-action-lg — text-sm / 2px 10px (modal Save, watch
+                                 Create, charter Save, schedule modal Run
+                                 now)
+     Positional/layout-only inline styles (e.g. margin-left:8px, margin-
+     right:4px, margin-top:6px) stay at the callsite. Do NOT re-create the
+     green inline blob — extend this primitive instead. */
+  .btn-action {
+    background: var(--surface2);
+    border: 1px solid var(--green);
+    color: var(--green);
+    font-size: var(--text-xs);
+    padding: 1px 6px;
+    border-radius: var(--radius-sm);
+    cursor: pointer;
+    transition: background var(--transition-base);
+  }
+  .btn-action:hover { background: rgba(63,185,80,0.1); }
+  .btn-action:focus-visible { outline: 2px solid var(--green); outline-offset: 1px; }
+  .btn-action:disabled,
+  .btn-action.disabled { opacity: 0.4; cursor: not-allowed; pointer-events: none; }
+  .btn-action.btn-action-md { padding: 2px 8px; }
+  .btn-action.btn-action-lg { font-size: var(--text-sm); padding: 2px 10px; }
+
+  /* W-mq57qetc0009ed48 — red-bordered destructive button family (Delete /
+     Abort / Remove / Unpin / unlink-x). Visual sibling to .btn-action —
+     same surface2 background, same padding scale, same hover-tint pattern,
+     just swapped to var(--red). Three size variants:
+       .btn-destructive            — compact text-xs / 1px 6px (work-item
+                                     delete, watch delete, schedule delete,
+                                     unpin, unlink-x in PR row)
+       .btn-destructive.btn-destructive-md — text-xs / 2px 8px (pipelines
+                                     Abort + Delete)
+       .btn-destructive.btn-destructive-lg — text-sm / 2px 10px (watch
+                                     builder Remove step / Remove
+                                     requirement)
+     Positional / outlier inline overrides (padding:1px 5px on the PR
+     unlink-x chip, padding:6px 8px on the requirement-row Remove) stay
+     at the callsite. Do NOT re-create the red inline blob — extend this
+     primitive instead. */
+  .btn-destructive {
+    background: var(--surface2);
+    border: 1px solid var(--red);
+    color: var(--red);
+    font-size: var(--text-xs);
+    padding: 1px 6px;
+    border-radius: var(--radius-sm);
+    cursor: pointer;
+    transition: background var(--transition-base);
+  }
+  .btn-destructive:hover { background: rgba(248,81,73,0.1); }
+  .btn-destructive:focus-visible { outline: 2px solid var(--red); outline-offset: 1px; }
+  .btn-destructive:disabled,
+  .btn-destructive.disabled { opacity: 0.4; cursor: not-allowed; pointer-events: none; }
+  .btn-destructive.btn-destructive-md { padding: 2px 8px; }
+  .btn-destructive.btn-destructive-lg { font-size: var(--text-sm); padding: 2px 10px; }
+
+  /* W-mq57qetc0009ed48 — solid-blue primary CTA. Single source of truth
+     for the filled-blue rectangle that gates submission flows (QA "Start
+     QA Session", "+ New runbook", "Save" in the QA runbook editor, FRE
+     "+ Add Project"). Replaces the standalone .qa-btn-primary (now
+     removed — all callsites migrated) and the ad-hoc
+     `background:var(--blue);color:#fff;border:none;…` style blob in
+     fre.js. */
+  .btn-primary-solid {
+    background: var(--blue);
+    color: #fff;
+    border: none;
+    border-radius: var(--radius-sm);
+    padding: var(--space-3) var(--space-7);
+    font-size: var(--text-md);
+    font-weight: 600;
+    cursor: pointer;
+    transition: opacity var(--transition-base);
+  }
+  .btn-primary-solid:hover { opacity: 0.9; }
+  .btn-primary-solid:focus-visible { outline: 2px solid var(--blue); outline-offset: 2px; }
+  .btn-primary-solid:disabled,
+  .btn-primary-solid.disabled { opacity: 0.4; cursor: not-allowed; pointer-events: none; }
+
+  /* W-mq57qetc0009ed48 — solid-blue modal CTA. Lighter-weight sibling to
+     .btn-primary-solid: same filled-blue rectangle, padding:6px 16px, but
+     without the font-weight:600 used by qa/fre primary CTAs. Used for the
+     "Save" / "Create Plan" / "Re-execute" / "Save Note" buttons inside
+     dashboard modals (render-inbox, render-kb, render-plans) and the
+     "Retry" button in the agent-detail error state. Replaces the ad-hoc
+     `background:var(--blue);color:#fff;border:none;border-radius:sm;
+     padding:6px 16px` blob that was duplicated across five callsites. */
+  .btn-primary-lg {
+    background: var(--blue);
+    color: #fff;
+    border: none;
+    border-radius: var(--radius-sm);
+    padding: 6px 16px;
+    font-size: var(--text-md);
+    cursor: pointer;
+    transition: opacity var(--transition-base);
+  }
+  .btn-primary-lg:hover { opacity: 0.9; }
+  .btn-primary-lg:focus-visible { outline: 2px solid var(--blue); outline-offset: 2px; }
+  .btn-primary-lg:disabled,
+  .btn-primary-lg.disabled { opacity: 0.4; cursor: not-allowed; pointer-events: none; }
+
+  /* W-mq57qetc0009ed48 — muted ghost button. Transparent background,
+     bordered, muted text. Used for secondary actions next to a primary
+     CTA (QA Cancel / Edit / Dismiss / "− remove artifact row" / "+ Add
+     artifact"). Replaces the standalone .qa-btn-ghost (now removed —
+     all callsites migrated). */
+  .btn-ghost {
+    background: transparent;
+    color: var(--muted);
+    border: 1px solid var(--border);
+    border-radius: var(--radius-sm);
+    padding: var(--space-2) var(--space-5);
+    font-size: var(--text-base);
+    cursor: pointer;
+    transition: color var(--transition-base), background var(--transition-base);
+  }
+  .btn-ghost:hover { color: var(--text); background: var(--surface); }
+  .btn-ghost:focus-visible { outline: 2px solid var(--blue); outline-offset: 1px; }
+  .btn-ghost:disabled,
+  .btn-ghost.disabled { opacity: 0.4; cursor: not-allowed; pointer-events: none; }
+
+  /* W-mq57qetc0009ed48 — dashed empty-state chip ("+ Add Project" in
+     the projects empty-state). Same chip footprint as the project chips
+     in render-other.js, but with a dashed border + green text to read as
+     a placeholder/call-to-action. */
+  .empty-state-add {
+    display: inline-block;
+    background: var(--surface2);
+    border: 1px dashed var(--border);
+    color: var(--green);
+    font-weight: 500;
+    padding: 3px 10px;
+    border-radius: var(--radius-sm);
+    cursor: pointer;
+    transition: background var(--transition-base);
+  }
+  .empty-state-add:hover { background: rgba(63,185,80,0.1); }
+  .empty-state-add:focus-visible { outline: 2px solid var(--green); outline-offset: 1px; }
+  .empty-state-add:disabled,
+  .empty-state-add.disabled { opacity: 0.4; cursor: not-allowed; pointer-events: none; }
+  /* Muted variant used when the chip sits next to existing project chips
+     (so it reads as a secondary "add another" affordance rather than the
+     primary empty-state CTA). */
+  .empty-state-add.empty-state-add-muted { color: var(--muted); }
+  .empty-state-add.empty-state-add-muted:hover { background: var(--surface); }
+  /* Blue variant used for the "Scan" affordance that sits next to "+ Add
+     Project" in the projects empty-state — same dashed chip, but text-sm and
+     blue text instead of green. */
+  .empty-state-add.empty-state-add-blue { color: var(--blue); font-size: var(--text-sm); }
+  .empty-state-add.empty-state-add-blue:hover { background: rgba(88,166,255,0.1); }
+
   /* Badge — unified badge/pill base */
   .badge {
     font-size: var(--text-sm); font-weight: 600;
@@ -796,6 +960,8 @@
   .modal-copy { background: var(--surface2); border: 1px solid var(--border); color: var(--muted); font-size: var(--text-base); cursor: pointer; padding: var(--space-2) var(--space-5); border-radius: var(--radius-sm); display: flex; align-items: center; gap: var(--space-2); transition: all var(--transition-base); }
   .modal-copy:hover { color: var(--text); border-color: var(--text); }
   .modal-copy.copied { color: var(--green); border-color: var(--green); }
+  .modal-copy.is-success { color: var(--green); border-color: var(--green); }
+  .modal-copy.is-danger { color: var(--red); border-color: var(--red); }
   .modal-close { background: none; border: none; color: var(--muted); font-size: var(--text-2xl); cursor: pointer; padding: var(--space-2) var(--space-4); }
   .modal-close:hover { color: var(--text); }
 
@@ -1019,20 +1185,10 @@
   .qa-add-artifact-btn { align-self: flex-start; margin-top: var(--space-2); }
   .qa-form-actions { flex-direction: row; align-items: center; gap: var(--space-4); }
   .qa-form-msg { font-size: var(--text-base); margin-left: var(--space-3); }
-  .qa-btn-primary {
-    background: var(--blue); color: #fff; border: none;
-    border-radius: var(--radius-sm); padding: var(--space-3) var(--space-7);
-    font-size: var(--text-md); font-weight: 600; cursor: pointer;
-  }
-  .qa-btn-primary:hover { opacity: 0.9; }
-  .qa-btn-primary:disabled { opacity: 0.4; cursor: not-allowed; }
-  .qa-btn-ghost {
-    background: transparent; color: var(--muted);
-    border: 1px solid var(--border); border-radius: var(--radius-sm);
-    padding: var(--space-2) var(--space-5); font-size: var(--text-base);
-    cursor: pointer;
-  }
-  .qa-btn-ghost:hover { color: var(--text); background: var(--surface); }
+  /* .qa-btn-primary and .qa-btn-ghost were removed in W-mq57qetc0009ed48
+     (PR #80 follow-up). All callsites migrated to the shared .btn-primary-solid
+     and .btn-ghost primitives defined near .btn-add. .qa-btn-danger is kept
+     as a color-only modifier composable with .btn-ghost. */
 
   .qa-runbooks-list { display: flex; flex-direction: column; gap: var(--space-3); }
   .qa-runbook-row {

package/docs/completion-reports.md

@@ -86,6 +86,7 @@ Do **not** invent, regenerate, or share the nonce across dispatches — each spa
 | `followups` | array | Optional. PR-comment follow-up work items the agent dispatched via `POST /api/work-items` with `meta.pr_followup` set. Each entry: `{wi_id, title, reason, parent_comment_id}`. See [PR-comment follow-ups](#pr-comment-follow-ups). |
 | `meta.review` | object | Optional, review tasks only. Records project-local review-skill outcome — see [Review skill outcomes](#review-skill-outcomes). Aliased by the generalized `meta.skill` (W-mq1cczi90006b21f). |
 | `meta.skill` | object | Optional. Records project-local skill outcome for ANY playbook type that surfaces a `## Project skills` block (implement / fix / plan / review / etc.). See [Project skill outcomes](#project-skill-outcomes). |
+| `meta.descriptionAudit` | object | Optional, `fix` / `implement` dispatches that push commits. Records the PR description audit + screenshot-refresh outcome — see [PR description audit](#pr-description-audit). |
 
 ## Project skill outcomes
 
@@ -150,6 +151,44 @@ All `meta.review` fields are optional and backward-compatible — older agents t
 
 Dispatchers can suppress the block entirely by setting `meta.skipProjectReviewSkills: true` on the review work item — the playbook then renders identically to the pre-W-mq16xtdx 8-step contract. Use this for meta-reviews of the review skill itself; the engine still accepts `meta.review.skillSkipped` in the report regardless. (W-mq1cczi90006b21f generalized this to `meta.skipProjectSkills`, which suppresses the block on every dispatch type — both flags are honored.)
 
+## PR description audit
+
+W-mq5l3f2u000i9045. The `fix` and `implement` playbooks inline a `## PR description audit` section that the agent runs **after** pushing commits to the PR's source branch and **before** marking the work item done. The audit re-reads the PR description, compares it against the diff of the newly-pushed commits, and patches stale sections (feature lists, config tables, file/line counts, ASCII diagrams, embedded screenshots) so reviewers don't end up reading outdated prose.
+
+The audit is best-effort:
+
+- Screenshot recapture requires a runnable dev server (detected via `package.json` scripts named `dev`, `start`, or `serve`) and Playwright MCP. When either is missing or fails, the agent skips the recapture without failing the work item.
+- GitHub does not expose a documented REST endpoint for PR-comment image uploads; the agent skips the recapture on GitHub with `result: "screenshots-refreshed-skipped (gh-attachment-unavailable)"` instead of failing.
+- Raw PNGs are saved under `agents/<id>/screenshots/` only. **Screenshots NEVER land in the repo** — adding any code path that `git add`s a PNG is a release-blocker bug.
+- The audit never introduces screenshots to a PR that didn't already have any (refresh only), never modifies the PR title, never toggles draft state, and never posts a separate PR comment narrating the description change.
+
+Record the outcome under `meta.descriptionAudit`. All fields are optional and backward-compatible — older agents that never set them stay valid.
+
+```json
+{
+  "status": "success",
+  "meta": {
+    "descriptionAudit": {
+      "ran": true,
+      "result": "description-patched",
+      "oldScreenshots": [],
+      "newScreenshots": [],
+      "patchedSections": ["feature-bullets", "config-table"]
+    }
+  }
+}
+```
+
+| Field | Type | Notes |
+|---|---|---|
+| `meta.descriptionAudit.ran` | boolean | `true` if the agent executed the audit (whether or not it edited the description). `false` when the audit was suppressed (no commits pushed, no-op completion, `meta.skipDescriptionAudit` set, or the playbook skipped the audit for any reason). Omit `meta.descriptionAudit` entirely when the audit was not even considered. |
+| `meta.descriptionAudit.result` | string | One of: `"no-changes-needed"`, `"description-patched"`, `"screenshots-refreshed"`, `"description-patched+screenshots-refreshed"`, or `"skipped:<reason>"` (e.g. `"skipped:meta-flag"`, `"skipped:no-commits"`, `"skipped:noop"`). The `"screenshots-refreshed-skipped (gh-attachment-unavailable)"` shape is also valid when GitHub's attachment limitation forces a partial-skip. |
+| `meta.descriptionAudit.oldScreenshots` | string[] | URLs of image refs the agent replaced. Omit or `[]` when no screenshots changed. |
+| `meta.descriptionAudit.newScreenshots` | string[] | URLs of newly uploaded image attachments. Omit or `[]` when no screenshots changed. |
+| `meta.descriptionAudit.patchedSections` | string[] | Human-readable identifiers of the description sections that changed (e.g. `"feature-bullets"`, `"config-table"`, `"file-list"`, `"line-counts"`, `"ascii-diagram"`, `"screenshots"`). Omit or `[]` when no sections changed. |
+
+Dispatchers can suppress the audit entirely by setting `meta.skipDescriptionAudit: true` on the work item — the playbook then renders an explicit "audit suppressed" notice instead of the audit steps. Use this for skill-meta updates, doc-only fixes whose PR description text won't be affected by the diff, and follow-up dispatches that explicitly own the description themselves.
+
 ## `failure_class` enum
 
 Defined in `engine/shared.js` as `FAILURE_CLASS`. Use the canonical hyphenated string.

package/engine/playbook.js

@@ -307,6 +307,14 @@ const PLAYBOOK_OPTIONAL_VARS = new Set([
   // projects legitimately resolve it to ''.
   'project_review_skills_block',
   'skip_project_review_skills',
+  // W-mq5l3f2u000i9045 — opt-out flag for the PR description audit + screenshot
+  // refresh fragment inlined into fix.md / implement.md. Truthy (set when
+  // item.meta.skipDescriptionAudit is true) renders the "audit suppressed"
+  // notice; falsy/empty leaves the notice block out and the agent runs the
+  // standard audit. Optional because (a) every other playbook ignores the var
+  // and (b) the default state is the empty string, which the playbook's
+  // conditional block treats as "not skipped".
+  'skip_description_audit',
   // P-e6b3c2d8 — QA Session template vars. session_id / target_kind /
   // flows_raw / managed_spawn_name are required (declared in
   // PLAYBOOK_REQUIRED_VARS['qa-session-setup']); these target_* sub-fields
@@ -435,7 +443,12 @@ function isSafePlaybookType(playbookType) {
     !name.includes('..') &&
     !/[\\/]/.test(name) &&
     !path.isAbsolute(name) &&
-    !/^[a-zA-Z]:/.test(name);
+    !/^[a-zA-Z]:/.test(name) &&
+    // W-mq5l3f2u000i9045 — files named `_*.md` are shared fragments inlined
+    // into dispatching playbooks (e.g. `_pr-description-audit.md`). They are
+    // not standalone playbooks; reject `_…` types so a dispatch can never
+    // resolve to a fragment file.
+    !name.startsWith('_');
 }
 
 function resolvePlaybookPath(projectName, playbookType) {
@@ -1171,7 +1184,14 @@ function buildPrDispatch(agentId, config, project, pr, type, extraVars, taskLabe
     // (skipProjectReviewSkills is the PR-82 alias). Default OFF: review/fix
     // dispatches surface the project-local skills block.
     skip_project_review_skills: !!(meta && meta.skipProjectReviewSkills),
+    // W-mq1cczi90006b21f — generalized project-skills suppression flag.
+    // Honors either the new `meta.skipProjectSkills` or the PR-82
+    // `meta.skipProjectReviewSkills` alias so older work items keep working.
     skip_project_skills: !!(meta && (meta.skipProjectSkills || meta.skipProjectReviewSkills)),
+    // W-mq5l3f2u000i9045 — opt-out for the inlined PR description audit
+    // section in fix.md. Truthy renders the "audit suppressed" notice; falsy
+    // (the default) leaves the standard audit instructions in place.
+    skip_description_audit: !!(meta && meta.skipDescriptionAudit),
     ...extraVars,
     task_id: dispatchId,
   };

package/engine.js

@@ -6121,6 +6121,14 @@ function renderProjectWorkItemPromptForAgent(item, workType, agentId, config, pr
     managed_spawn_ttl_minutes: item.meta && Number.isFinite(Number(item.meta.managed_spawn_ttl_minutes))
       ? Math.floor(Number(item.meta.managed_spawn_ttl_minutes))
       : '',
+    // W-mq5l3f2u000i9045 — opt-out flag for the inlined PR description audit
+    // section in fix.md / implement.md. When the dispatcher sets
+    // item.meta.skipDescriptionAudit (skill-meta updates, doc-only fixes,
+    // follow-up dispatches that explicitly own the description), the playbook
+    // renders the "audit suppressed" notice instead of the standard audit
+    // steps. Default off — every fix/implement dispatch that pushes commits
+    // audits the PR description by default.
+    skip_description_audit: !!(item.meta && item.meta.skipDescriptionAudit),
     // W-mpeiwz6k0005bf34-c — opt-in qa-validate context. The dispatch handler
     // POST /api/qa/runbooks/run stamps meta.qaRunId + meta.qaRunbook (full
     // spec) + meta.qaTarget (managed-process snapshot) on the work item;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2157",
+  "version": "0.1.2159",
   "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/playbooks/_pr-description-audit.md

@@ -0,0 +1,142 @@
+# PR description audit + screenshot refresh (shared fragment)
+
+> **Naming convention.** Files named `_*.md` under `playbooks/` are **shared
+> fragments**. The engine renderer (`engine/playbook.js`) does not (yet)
+> implement an `<include>` directive, so each consuming playbook inlines this
+> fragment verbatim and the two copies are kept in sync by hand. If you change
+> this file, also update the inlined copies in `playbooks/fix.md` and
+> `playbooks/implement.md` (and any other playbook that pushes commits to a
+> tracked PR). The matching test in `test/unit/config-and-playbooks.test.js`
+> ("PR description audit fragment is inlined consistently") fails fast when the
+> copies drift.
+
+## When to run
+
+Run this audit **after** you push new commits to the PR's source branch and
+**before** you mark the work item done.
+
+Skip the audit entirely when:
+
+- This dispatch was invoked with `meta.skipDescriptionAudit: true` (the
+  engine renders an explicit "audit suppressed" notice when the flag is set).
+- You did not push any new commits this dispatch (no behavior change → no
+  description drift).
+- The dispatch produced a no-op completion (`noop: true`).
+
+When you skip, still report it: set `meta.descriptionAudit.ran = false` and
+`meta.descriptionAudit.result = "skipped:<reason>"` in the completion JSON.
+
+## Audit steps
+
+1. **Fetch the current PR description.**
+   - GitHub: `gh pr view <num> --repo <owner>/<repo> --json body --jq .body`
+   - Azure DevOps: `az repos pr show --id <num> --query description -o tsv`
+2. **Compute the diff for the commits you just pushed.**
+   - `git log --stat <baseline>..HEAD` where `<baseline>` is `origin/main` for
+     the first push or `origin/<branch>@{1}` (pre-push tip) for follow-up
+     pushes. `git log --oneline <baseline>..HEAD` if you only need the SHA list.
+3. **Compare description sections against the diff.** For each section,
+   evaluate whether the new commits change anything it claims. Focus
+   specifically on:
+   - Feature lists / "what changed" bullet points
+   - Config / option / matrix tables
+   - File or path lists
+   - Line counts, before/after metrics, perf numbers, table cell values
+   - ASCII diagrams that depict layout the diff touched
+   - Embedded markdown image refs (`![alt](url)`) — see the screenshot
+     subsection below
+4. **Decide the result.**
+   - Description is consistent with the diff → no-op. Record
+     `meta.descriptionAudit = { ran: true, result: "no-changes-needed" }`
+     in the completion report and stop here.
+   - Description needs targeted edits → build the patched body. **Preserve prose voice**. Edit only the stale parts. Do **not** rewrite. Do **not**
+     add new sections that weren't there before. Do **not** change the PR
+     title. Do **not** flip draft state, close/reopen, or post a separate PR
+     comment narrating the description edit.
+5. **Push the patched description.**
+   - GitHub: write the new body to a temp file, then
+     `gh pr edit <num> --repo <owner>/<repo> --body-file <file>`
+   - Azure DevOps:
+     `az repos pr update --id <num> --description "$(Get-Content -Raw <file>)"`
+     (PowerShell) or the POSIX equivalent. ADO's description field also
+     truncates around ~4 KB on the `--description` flag — if you hit that,
+     fall back to `az rest --method patch` against
+     `…/pullRequests/<id>?api-version=7.1` with `{ "description": "<body>" }`.
+6. **GET-verify the description post-patch** by re-running the fetch from
+   step 1 and confirming the body matches what you sent.
+
+## Screenshot handling
+
+This applies only when the **existing** description embeds markdown image
+refs. **Never introduce screenshots to a PR that didn't already have any** —
+this audit is a refresh, not an introduction.
+
+1. Extract every `![alt](url)` from the current description.
+2. For each image, decide whether it depicts a view this dispatch's diff
+   touched. Heuristics (any one is enough):
+   - `alt` text or filename mentions a component / route / view name that
+     appears in the changed files.
+   - The image URL path contains an issue/PR id matching this PR.
+   - The description text immediately around the image references files
+     present in the diff.
+3. If the view was touched **and** the project has a runnable dev server
+   (detect via `package.json` scripts named `dev`, `start`, or `serve`)
+   **and** Playwright MCP is available, recapture:
+   1. Spin up the dev server with a detached handoff (see
+      `shared-rules.md` → "Long-Running Commands"). Capture PID + log path +
+      URL + stop command.
+   2. Drive Playwright MCP to the relevant route and screenshot the same
+      view. Save raw PNGs to `agents/<id>/screenshots/` only. **NEVER `git add` a PNG** — adding any code path that commits a screenshot is a
+      release-blocker bug.
+   3. Upload as a PR attachment:
+      - **ADO**: `PUT /pullRequests/<id>/attachments/<filename>` with the
+        PNG bytes, then reference the returned URL in the description.
+      - **GitHub**: there is no documented REST endpoint for PR-comment
+        image uploads. The browser drag-drop path uses
+        `github.com/upload/asset`, which `gh` does not wrap. If you cannot
+        find a reliable scripted upload path, **skip** this screenshot,
+        leave the old image URL in place, and record
+        `meta.descriptionAudit.result = "screenshots-refreshed-skipped (gh-attachment-unavailable)"`.
+   4. Patch the description to swap old image URLs for new.
+   5. Stop the dev server using the recorded stop command.
+4. If the dev server fails to start, the route 404s, Playwright fails, or
+   the host's attachment API is unreachable → **skip that screenshot**
+   without failing the work item. Record the skip and the reason in
+   `meta.descriptionAudit`.
+
+## Out-of-scope guardrails
+
+- Don't add screenshots that weren't already there.
+- Don't rewrite description prose beyond the targeted stale edits.
+- Don't modify the PR title.
+- Don't toggle draft state, close/reopen the PR, or post a separate PR
+  comment narrating the description change.
+- Don't fail the work item on a screenshot or attachment failure — degrade
+  gracefully and record the skip.
+- Screenshots **never** land in the repo. Save raw PNGs only under
+  `agents/<id>/screenshots/`. No `git add` of any PNG, ever.
+
+## Completion-report shape
+
+When the audit ran (whether it edited the description or no-op'd), include
+this block in the JSON completion report (see
+`docs/completion-reports.md` → "PR description audit"):
+
+```json
+{
+  "meta": {
+    "descriptionAudit": {
+      "ran": true,
+      "result": "description-patched",
+      "oldScreenshots": [],
+      "newScreenshots": [],
+      "patchedSections": ["feature-bullets", "config-table"]
+    }
+  }
+}
+```
+
+Omit `meta.descriptionAudit` entirely when the audit did not run (no
+commits pushed, no-op completion, etc.). When `meta.skipDescriptionAudit`
+suppressed the audit, set `ran: false` and
+`result: "skipped:meta-flag"` so the audit history stays auditable.

package/playbooks/fix.md

@@ -99,6 +99,77 @@ Do NOT remove the worktree — the engine handles cleanup automatically.
 - content: Explain what was fixed, reference each review finding, include build/test status
 - Sign: `Fixed by Minions ({{agent_name}} — {{agent_role}} · {{agent_model}})`
 
+## PR description audit (mandatory unless `meta.skipDescriptionAudit`)
+
+<!-- Inlined from playbooks/_pr-description-audit.md — keep in sync (enforced by test/unit/config-and-playbooks.test.js). -->
+
+{{#skip_description_audit}}
+> **Audit suppressed.** This dispatch was invoked with `meta.skipDescriptionAudit: true`.
+> Record `meta.descriptionAudit = { ran: false, result: "skipped:meta-flag" }` in your completion report
+> and skip the audit steps below entirely.
+{{/skip_description_audit}}
+
+After you push commits to the PR's source branch and BEFORE you mark the work item done, audit the PR description and refresh it to match the new diff. Skip the audit when you pushed no commits this dispatch or when the dispatch is a no-op completion; record the skip in `meta.descriptionAudit`.
+
+**Audit steps**
+
+1. Fetch the current PR description.
+   - GitHub: `gh pr view <num> --repo <owner>/<repo> --json body --jq .body`
+   - Azure DevOps: `az repos pr show --id <num> --query description -o tsv`
+2. Compute the diff for the commits you just pushed: `git log --stat <baseline>..HEAD` where `<baseline>` is `origin/<branch>@{1}` for follow-up pushes (or `origin/main` for the very first push).
+3. For each section of the description, evaluate whether the new diff invalidates anything it claims. Focus on: feature lists / "what changed" bullets, config/option tables, file or path lists, line counts, before/after metrics, ASCII diagrams, and embedded markdown image refs.
+4. **No drift** → record `meta.descriptionAudit = { ran: true, result: "no-changes-needed" }` and stop.
+5. **Drift detected** → build a patched body. **Preserve prose voice.** Edit only the stale parts. Do NOT rewrite. Do NOT add new sections. Do NOT change the PR title. Do NOT toggle draft state. Do NOT post a separate PR comment narrating the description change.
+6. Push the patched description.
+   - GitHub: write the new body to a temp file, then `gh pr edit <num> --repo <owner>/<repo> --body-file <file>`.
+   - Azure DevOps: `az repos pr update --id <num> --description "$(Get-Content -Raw <file>)"`. If the body exceeds ~4 KB, fall back to `az rest --method patch` against `…/pullRequests/<id>?api-version=7.1` with `{ "description": "<body>" }`.
+7. GET-verify the description post-patch by re-running step 1 and confirming the body matches what you sent.
+
+**Screenshot handling (refresh-only — never introduce screenshots)**
+
+This applies only when the **existing** description embeds markdown image refs (`![alt](url)`). NEVER add screenshots to a PR that didn't already have any.
+
+1. Extract every `![alt](url)` from the current description.
+2. Decide which images depict views this dispatch's diff touched (heuristic: alt text / filename / surrounding prose mentions a component / route / view name present in the changed files).
+3. If the view was touched AND the project has a runnable dev server (detect via `package.json` scripts named `dev`, `start`, or `serve`) AND Playwright MCP is available, recapture:
+   - Spin up the dev server with a detached handoff per `shared-rules.md` → "Long-Running Commands". Record PID + log path + URL + stop command.
+   - Drive Playwright MCP to the relevant route and screenshot the view. Save raw PNGs to `agents/<id>/screenshots/` ONLY. **NEVER `git add` a PNG** — committing a screenshot is a release-blocker bug.
+   - Upload as a PR attachment:
+     - **ADO**: `PUT /pullRequests/<id>/attachments/<filename>` with the PNG bytes, then reference the returned URL in the description.
+     - **GitHub**: no documented REST endpoint exists for PR-comment image uploads (the browser drag-drop uses `github.com/upload/asset`, which `gh` does not wrap). If you cannot find a reliable scripted upload path, **skip** the recapture, leave the old image URL in place, and record `meta.descriptionAudit.result = "screenshots-refreshed-skipped (gh-attachment-unavailable)"`.
+   - Patch the description to swap old image URLs for new.
+   - Stop the dev server using the recorded stop command.
+4. If the dev server fails to start, the route 404s, Playwright fails, or the host's attachment API is unreachable → **skip that screenshot** without failing the work item. Record the skip and the reason in `meta.descriptionAudit`.
+
+**Out-of-scope guardrails**
+
+- Don't add screenshots that weren't already there. Refresh only.
+- Don't rewrite description prose beyond the targeted stale edits.
+- Don't modify the PR title; don't toggle draft state, close/reopen the PR.
+- Don't post a separate PR comment summarizing the description change.
+- Don't fail the work item on a screenshot or attachment failure — degrade gracefully and record the skip.
+- Screenshots NEVER land in the repo. `agents/<id>/screenshots/` only; never `git add` a PNG.
+
+**Completion-report shape**
+
+When the audit ran (whether it edited the description or no-op'd), include this in the completion JSON (full schema: `docs/completion-reports.md` → "PR description audit"):
+
+```json
+{
+  "meta": {
+    "descriptionAudit": {
+      "ran": true,
+      "result": "description-patched",
+      "oldScreenshots": [],
+      "newScreenshots": [],
+      "patchedSections": ["feature-bullets", "config-table"]
+    }
+  }
+}
+```
+
+Omit `meta.descriptionAudit` entirely when the audit did not run (no commits pushed, no-op completion). When `meta.skipDescriptionAudit` suppressed the audit, set `ran: false` and `result: "skipped:meta-flag"`.
+
 ## Resolve Review Comments
 
 After pushing, respond to each review comment/thread. **Every disposition requires a reply before you resolve, close, or mark-fixed the thread — no exceptions.** Resolving a thread silently is a process violation (see "Resolving Review Threads — No Silent Closures" in `shared-rules.md`).

package/playbooks/implement.md

@@ -101,6 +101,77 @@ Create the PR for implement tasks — the engine tracks review and completion fr
 
 Include build/test status and run instructions in the PR description. If the project has a runnable app, include the localhost URL.
 
+## PR description audit (mandatory unless `meta.skipDescriptionAudit`)
+
+<!-- Inlined from playbooks/_pr-description-audit.md — keep in sync (enforced by test/unit/config-and-playbooks.test.js). -->
+
+{{#skip_description_audit}}
+> **Audit suppressed.** This dispatch was invoked with `meta.skipDescriptionAudit: true`.
+> Record `meta.descriptionAudit = { ran: false, result: "skipped:meta-flag" }` in your completion report
+> and skip the audit steps below entirely.
+{{/skip_description_audit}}
+
+After you push commits to the PR's source branch and BEFORE you mark the work item done, audit the PR description and refresh it to match the new diff. Skip the audit when you pushed no commits this dispatch or when the dispatch is a no-op completion; record the skip in `meta.descriptionAudit`.
+
+**Audit steps**
+
+1. Fetch the current PR description.
+   - GitHub: `gh pr view <num> --repo <owner>/<repo> --json body --jq .body`
+   - Azure DevOps: `az repos pr show --id <num> --query description -o tsv`
+2. Compute the diff for the commits you just pushed: `git log --stat <baseline>..HEAD` where `<baseline>` is `origin/<branch>@{1}` for follow-up pushes (or `origin/main` for the very first push).
+3. For each section of the description, evaluate whether the new diff invalidates anything it claims. Focus on: feature lists / "what changed" bullets, config/option tables, file or path lists, line counts, before/after metrics, ASCII diagrams, and embedded markdown image refs.
+4. **No drift** → record `meta.descriptionAudit = { ran: true, result: "no-changes-needed" }` and stop.
+5. **Drift detected** → build a patched body. **Preserve prose voice.** Edit only the stale parts. Do NOT rewrite. Do NOT add new sections. Do NOT change the PR title. Do NOT toggle draft state. Do NOT post a separate PR comment narrating the description change.
+6. Push the patched description.
+   - GitHub: write the new body to a temp file, then `gh pr edit <num> --repo <owner>/<repo> --body-file <file>`.
+   - Azure DevOps: `az repos pr update --id <num> --description "$(Get-Content -Raw <file>)"`. If the body exceeds ~4 KB, fall back to `az rest --method patch` against `…/pullRequests/<id>?api-version=7.1` with `{ "description": "<body>" }`.
+7. GET-verify the description post-patch by re-running step 1 and confirming the body matches what you sent.
+
+**Screenshot handling (refresh-only — never introduce screenshots)**
+
+This applies only when the **existing** description embeds markdown image refs (`![alt](url)`). NEVER add screenshots to a PR that didn't already have any.
+
+1. Extract every `![alt](url)` from the current description.
+2. Decide which images depict views this dispatch's diff touched (heuristic: alt text / filename / surrounding prose mentions a component / route / view name present in the changed files).
+3. If the view was touched AND the project has a runnable dev server (detect via `package.json` scripts named `dev`, `start`, or `serve`) AND Playwright MCP is available, recapture:
+   - Spin up the dev server with a detached handoff per `shared-rules.md` → "Long-Running Commands". Record PID + log path + URL + stop command.
+   - Drive Playwright MCP to the relevant route and screenshot the view. Save raw PNGs to `agents/<id>/screenshots/` ONLY. **NEVER `git add` a PNG** — committing a screenshot is a release-blocker bug.
+   - Upload as a PR attachment:
+     - **ADO**: `PUT /pullRequests/<id>/attachments/<filename>` with the PNG bytes, then reference the returned URL in the description.
+     - **GitHub**: no documented REST endpoint exists for PR-comment image uploads (the browser drag-drop uses `github.com/upload/asset`, which `gh` does not wrap). If you cannot find a reliable scripted upload path, **skip** the recapture, leave the old image URL in place, and record `meta.descriptionAudit.result = "screenshots-refreshed-skipped (gh-attachment-unavailable)"`.
+   - Patch the description to swap old image URLs for new.
+   - Stop the dev server using the recorded stop command.
+4. If the dev server fails to start, the route 404s, Playwright fails, or the host's attachment API is unreachable → **skip that screenshot** without failing the work item. Record the skip and the reason in `meta.descriptionAudit`.
+
+**Out-of-scope guardrails**
+
+- Don't add screenshots that weren't already there. Refresh only.
+- Don't rewrite description prose beyond the targeted stale edits.
+- Don't modify the PR title; don't toggle draft state, close/reopen the PR.
+- Don't post a separate PR comment summarizing the description change.
+- Don't fail the work item on a screenshot or attachment failure — degrade gracefully and record the skip.
+- Screenshots NEVER land in the repo. `agents/<id>/screenshots/` only; never `git add` a PNG.
+
+**Completion-report shape**
+
+When the audit ran (whether it edited the description or no-op'd), include this in the completion JSON (full schema: `docs/completion-reports.md` → "PR description audit"):
+
+```json
+{
+  "meta": {
+    "descriptionAudit": {
+      "ran": true,
+      "result": "description-patched",
+      "oldScreenshots": [],
+      "newScreenshots": [],
+      "patchedSections": ["feature-bullets", "config-table"]
+    }
+  }
+}
+```
+
+Omit `meta.descriptionAudit` entirely when the audit did not run (no commits pushed, no-op completion). When `meta.skipDescriptionAudit` suppressed the audit, set `ran: false` and `result: "skipped:meta-flag"`.
+
 ## When to Stop
 
 Your task is complete when the requested implementation is delivered, the validation story is truthful and sufficient for review, the branch is pushed, and the PR exists. Include the PR URL in your final message so the engine can track it.