@yemi33/minions 0.1.1562 → 0.1.1564

package/CHANGELOG.md

@@ -1,6 +1,14 @@
 # Changelog
 
-## 0.1.1562 (2026-04-27)
+## 0.1.1564 (2026-04-27)
+
+### Other
+- Harden skill creation guidance
+
+## 0.1.1563 (2026-04-27)
+
+### Fixes
+-  tighten destructive dashboard UX
 
 ### Other
 - chore: minions uninstall prints re-install command

package/README.md

@@ -457,7 +457,7 @@ All playbooks use `{{template_variables}}` filled from project config. Condition
 
 Every playbook has an explicit "When to Stop" section telling agents exactly what constitutes completion. Code-pushing playbooks enforce **build → test → repo checks → push** ordering. Per-type max-turns prevent runaway tool loops (explore=30, ask=20, implement=75, verify=100).
 
-Playbooks are fully customizable — edit them to match your workflow. System prompts for CC and plan advisor live in `prompts/` with `{{variable}}` substitution.
+Playbooks are fully customizable — edit the shared templates in `playbooks/` to change the repo-wide defaults. You can also create machine-local overrides in `projects/<name>/playbooks/<type>.md`; Minions will prefer those for that project, but they are treated as user data and remain gitignored. System prompts for CC and plan advisor live in `prompts/` with `{{variable}}` substitution.
 
 ## Health Monitoring
 
@@ -503,7 +503,7 @@ When a reviewer flags issues, the engine creates `feedback-<author>-from-<review
 `engine/metrics.json` tracks per agent: tasks completed, errors, PRs created/approved/rejected, reviews done. Visible in CLI (`status`) and dashboard with color-coded approval rates.
 
 ### 5. Skills
-Agents save repeatable workflows to `skills/<name>.md` with Claude Code-compatible frontmatter. Engine builds an index injected into all prompts. Skills can also be stored per-project at `<project>/.claude/skills/<name>/SKILL.md` (requires a PR). Visible in the dashboard Skills section.
+Agents save Minions-wide repeatable workflows as user-level Claude skills in `~/.claude/skills/<name>/SKILL.md`, so they are usable both inside Minions and in normal Claude windows. Repo-specific skills can also be stored per-project at `<project>/.claude/skills/<name>/SKILL.md` (requires a PR). Visible in the dashboard Skills section.
 
 See `docs/self-improvement.md` for the full breakdown.
 
@@ -730,5 +730,3 @@ Per-project engine state remains centralized:
     work-items.json      <- Per-project work queue
     pull-requests.json   <- PR tracker
 ```
-
-

package/dashboard/js/render-inbox.js

@@ -158,17 +158,17 @@ function modalCancelEdit() {
 
 async function deleteInboxItem(name) {
   if (!confirm('Delete "' + name + '" from inbox?')) return;
+  showToast('cmd-toast', 'Inbox item deleted', true);
   markDeleted('inbox:' + name);
   const card = document.querySelector('.inbox-item[data-file="notes/inbox/' + CSS.escape(name) + '"]');
   if (card) card.remove();
-  showToast('cmd-toast', 'Inbox item deleted', true);
   try {
     const res = await fetch('/api/inbox/delete', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ name })
     });
-    if (!res.ok) { const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'Delete failed: ' + (d.error || 'unknown'), false); refresh(); }
-  } catch (e) { showToast('cmd-toast', 'Delete error: ' + e.message, false); refresh(); }
+    if (!res.ok) { const d = await res.json().catch(() => ({})); clearDeleted('inbox:' + name); showToast('cmd-toast', 'Delete failed: ' + (d.error || 'unknown'), false); refresh(); }
+  } catch (e) { clearDeleted('inbox:' + name); showToast('cmd-toast', 'Delete error: ' + e.message, false); refresh(); }
 }
 
 async function openInboxInExplorer(name) {
@@ -223,11 +223,11 @@ async function submitQuickNote() {
 }
 
 async function doPromoteToKB(name, category) {
+  showToast('cmd-toast', 'Promoted to Knowledge Base', true);
   try { closeModal(); } catch { /* expected */ }
   markDeleted('inbox:' + name);
   const card = document.querySelector('.inbox-item[data-file="notes/inbox/' + CSS.escape(name) + '"]');
   if (card) card.remove();
-  showToast('cmd-toast', 'Promoted to Knowledge Base', true);
   try {
     const res = await fetch('/api/inbox/promote-kb', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -235,8 +235,8 @@ async function doPromoteToKB(name, category) {
     });
     const data = await res.json();
     if (res.ok) { refreshKnowledgeBase(); }
-    else { showToast('cmd-toast', 'Promote failed: ' + (data.error || 'unknown'), false); refresh(); }
-  } catch (e) { showToast('cmd-toast', 'Promote error: ' + e.message, false); refresh(); }
+    else { clearDeleted('inbox:' + name); showToast('cmd-toast', 'Promote failed: ' + (data.error || 'unknown'), false); refresh(); }
+  } catch (e) { clearDeleted('inbox:' + name); showToast('cmd-toast', 'Promote error: ' + e.message, false); refresh(); }
 }
 
 window.MinionsInbox = { renderInbox, promoteToKB, renderNotes, openNotesModal, modalToggleEdit, modalSaveEdit, modalCancelEdit, deleteInboxItem, openInboxInExplorer, openQuickNoteModal, submitQuickNote, doPromoteToKB };

package/dashboard/js/render-meetings.js

@@ -337,17 +337,17 @@ async function _endMeeting(id, btn) {
 }
 
 async function _archiveMeeting(id) {
+  showToast('cmd-toast', 'Meeting archived', true);
   markDeleted('mtg:' + id);
   try { closeModal(); } catch { /* may not be open */ }
   document.querySelectorAll('[onclick*="openMeetingDetail(\'' + id + '\')"]').forEach(function(el) { el.remove(); });
-  showToast('cmd-toast', 'Meeting archived', true);
   try {
     const res = await fetch('/api/meetings/archive', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id })
     });
-    if (!res.ok) { _deletedIds.delete('mtg:' + id); const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'Failed: ' + (d.error || 'unknown'), false); refresh(); }
-  } catch (e) { _deletedIds.delete('mtg:' + id); showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
+    if (!res.ok) { clearDeleted('mtg:' + id); const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'Failed: ' + (d.error || 'unknown'), false); refresh(); }
+  } catch (e) { clearDeleted('mtg:' + id); showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
 async function _unarchiveMeeting(id) {
@@ -480,17 +480,17 @@ async function _createPlanFromMeeting(id, btn) {
 
 async function _deleteMeeting(id) {
   if (!confirm('Delete this meeting? This cannot be undone.')) return;
+  showToast('cmd-toast', 'Meeting deleted', true);
   markDeleted('mtg:' + id);
   try { closeModal(); } catch { /* may not be open */ }
   document.querySelectorAll('[onclick*="openMeetingDetail(\'' + id + '\')"]').forEach(function(el) { el.remove(); });
-  showToast('cmd-toast', 'Meeting deleted', true);
   try {
     const res = await fetch('/api/meetings/delete', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id })
     });
-    if (!res.ok) { const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'Failed: ' + (d.error || 'unknown'), false); refresh(); }
-  } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
+    if (!res.ok) { clearDeleted('mtg:' + id); const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'Failed: ' + (d.error || 'unknown'), false); refresh(); }
+  } catch (e) { clearDeleted('mtg:' + id); showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
 window.MinionsMeetings = { renderMeetings, openMeetingDetail, openCreateMeetingModal };

package/dashboard/js/render-other.js

@@ -23,12 +23,11 @@ function renderProjects(projects) {
 
 async function projectChipRemove(name) {
   if (!confirm('Remove project "' + name + '"? Pending work cancels, active agents are killed, data dir is archived to projects/.archived/.')) return;
+  showToast('cmd-toast', 'Removing project "' + name + '"...', true);
   markDeleted('project:' + name);
-  // Re-render immediately from cached state so the chip disappears without waiting on the API
   if (window._lastStatus && Array.isArray(window._lastStatus.projects)) {
     renderProjects(window._lastStatus.projects);
   }
-  showToast('cmd-toast', 'Removing project "' + name + '"...', true);
   try {
     const res = await fetch('/api/projects/remove', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -37,6 +36,8 @@ async function projectChipRemove(name) {
     const d = await res.json().catch(() => ({}));
     if (!res.ok || !d.ok) {
       // Revert: refresh restores the chip from server state
+      clearDeleted('project:' + name);
+      if (window._lastStatus && Array.isArray(window._lastStatus.projects)) renderProjects(window._lastStatus.projects);
       showToast('cmd-toast', 'Remove failed: ' + (d.error || 'unknown'), false);
       if (typeof refresh === 'function') refresh();
       return;
@@ -50,6 +51,8 @@ async function projectChipRemove(name) {
     showToast('cmd-toast', parts.join(' — '), true);
     if (typeof refresh === 'function') refresh();
   } catch (e) {
+    clearDeleted('project:' + name);
+    if (window._lastStatus && Array.isArray(window._lastStatus.projects)) renderProjects(window._lastStatus.projects);
     showToast('cmd-toast', 'Error: ' + e.message, false);
     if (typeof refresh === 'function') refresh();
   }

package/dashboard/js/render-pinned.js

@@ -77,13 +77,13 @@ async function submitPinnedNote(e) {
 
 async function removePinnedNote(title) {
   if (!confirm('Unpin "' + title + '"?')) return;
+  showToast('cmd-toast', 'Note unpinned', true);
   markDeleted('pin:' + title);
   const btn = (window.event)?.target; if (btn) { const card = btn.closest('.pinned-card') || btn.parentElement?.parentElement; if (card) card.remove(); }
-  showToast('cmd-toast', 'Note unpinned', true);
   try {
     const res = await fetch('/api/pinned/remove', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ title }) });
-    if (!res.ok) { showToast('cmd-toast', 'Unpin failed', false); refresh(); }
-  } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
+    if (!res.ok) { clearDeleted('pin:' + title); showToast('cmd-toast', 'Unpin failed', false); refresh(); }
+  } catch (e) { clearDeleted('pin:' + title); showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
 function openPinnedView(idx) {

package/dashboard/js/render-pipelines.js

@@ -248,7 +248,8 @@ function _buildNodeChain(stages, run, options) {
 }
 
 function renderPipelines(pipelines) {
-  _pipelinesData = pipelines || [];
+  pipelines = (pipelines || []).filter(function(p) { return !isDeleted('pipeline:' + p.id); });
+  _pipelinesData = pipelines;
   const el = document.getElementById('pipelines-content');
   const countEl = document.getElementById('pipelines-count');
   if (!el) return;
@@ -515,14 +516,15 @@ async function _continuePipeline(id, stageId, btn) {
 
 async function _deletePipelineConfirm(id) {
   if (!confirm('Delete pipeline "' + id + '"?')) return;
+  showToast('cmd-toast', 'Pipeline deleted', true);
   markDeleted('pipeline:' + id);
   try { closeModal(); } catch {}
-  showToast('cmd-toast', 'Pipeline deleted', true);
+  renderPipelines(_pipelinesData);
   try {
     var res = await fetch('/api/pipelines/delete', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ id: id }) });
-    if (!res.ok) { showToast('cmd-toast', 'Delete failed', false); }
+    if (!res.ok) { clearDeleted('pipeline:' + id); showToast('cmd-toast', 'Delete failed', false); }
     refresh();
-  } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
+  } catch (e) { clearDeleted('pipeline:' + id); showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
 function openCreatePipelineModal() {

package/dashboard/js/render-plans.js

@@ -575,9 +575,11 @@ async function planApprove(file, btn) {
 async function planDelete(file) {
   _stopPlanPoll();
   if (!confirm('Delete plan "' + file + '"? This cannot be undone.')) return;
-  markDeleted('plan:' + file);
-  closeModal();
   showToast('cmd-toast', 'Plan deleted', true);
+  markDeleted('plan:' + file);
+  try { closeModal(); } catch { /* may not be open */ }
+  if (window._lastPlans) renderPlans(window._lastPlans);
+  if (typeof rerenderPrdFromCache === 'function') rerenderPrdFromCache();
   try {
     const res = await fetch('/api/plans/delete', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -587,10 +589,11 @@ async function planDelete(file) {
       refresh();
     } else {
       const d = await res.json().catch(() => ({}));
-      alert('Delete failed: ' + (d.error || 'unknown'));
+      clearDeleted('plan:' + file);
+      showToast('cmd-toast', 'Delete failed: ' + (d.error || 'unknown'), false);
       refresh(); // revert optimistic
     }
-  } catch (e) { alert('Error: ' + e.message); refresh(); }
+  } catch (e) { clearDeleted('plan:' + file); showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
 async function planArchive(file, btn) {
@@ -600,6 +603,7 @@ async function planArchive(file, btn) {
     : 'Archive this plan?';
   if (!confirm(confirmMsg)) return;
   _stopPlanPoll();
+  showToast('cmd-toast', isPrd ? 'Archiving PRD and linked source plan...' : 'Archiving plan...', true);
   markDeleted('plan:' + file);
   if (isPrd) {
     var prdRecord = (window._lastPlans || []).find(function(p) { return p.file === file && p.sourcePlan; });
@@ -608,7 +612,6 @@ async function planArchive(file, btn) {
   if (window._lastPlans) renderPlans(window._lastPlans);
   if (typeof rerenderPrdFromCache === 'function') rerenderPrdFromCache();
   try { closeModal(); } catch { /* may not be open */ }
-  showToast('cmd-toast', isPrd ? 'Archiving PRD and linked source plan...' : 'Archiving plan...', true);
   try {
     const res = await fetch('/api/plans/archive', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -626,10 +629,12 @@ async function planArchive(file, btn) {
       }
       refresh();
     } else {
+      clearDeleted('plan:' + file);
+      if (prdRecord) clearDeleted('plan:' + prdRecord.sourcePlan);
       showToast('cmd-toast', 'Archive failed: ' + (d.error || 'unknown'), false);
       refresh();
     }
-  } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
+  } catch (e) { clearDeleted('plan:' + file); if (prdRecord) clearDeleted('plan:' + prdRecord.sourcePlan); showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
 async function planPause(file, btn) {

package/dashboard/js/render-prd.js

@@ -1,5 +1,9 @@
 // render-prd.js — PRD-related rendering functions extracted from dashboard.html
 
+function _prdItemDeleteKey(source, itemId) {
+  return 'prd-item:' + (source || '') + ':' + (itemId || '');
+}
+
 function renderPrd(prd, prog) {
   const section = document.getElementById('prd-content');
   const badge = document.getElementById('prd-badge');
@@ -19,7 +23,7 @@ function renderPrd(prd, prog) {
   // Show per-PRD status summary in header when multiple PRDs exist
   const existing = (prd.existing || []).filter(function(p) { return !isDeleted('plan:' + (p.file || '')); });
   const allWi = window._lastWorkItems || [];
-  const prdItems = (prog?.items || []).filter(i => !i._archived && !isDeleted('plan:' + (i.source || '')));
+  const prdItems = (prog?.items || []).filter(i => !i._archived && !isDeleted('plan:' + (i.source || '')) && !isDeleted(_prdItemDeleteKey(i.source, i.id)));
 
   if (existing.length === 0 && prdItems.length === 0) {
     section.innerHTML = '<p class="prd-pending" style="margin-bottom:0">No PRD found.</p>';
@@ -84,7 +88,7 @@ function renderPrdProgress(prog) {
   const el = document.getElementById('prd-progress-content');
   const countEl = document.getElementById('prd-progress-count');
   if (!prog) { el.innerHTML = ''; countEl.textContent = '—'; return; }
-  const visibleItems = (prog.items || []).filter(i => !isDeleted('plan:' + (i.source || '')));
+  const visibleItems = (prog.items || []).filter(i => !isDeleted('plan:' + (i.source || '')) && !isDeleted(_prdItemDeleteKey(i.source, i.id)));
 
   // Compute overall progress from active (non-archived) items
   const activeItems = visibleItems.filter(i => !i._archived);
@@ -748,16 +752,18 @@ async function prdItemSave(source, itemId) {
 
 async function prdItemRemove(source, itemId) {
   if (!confirm('Remove item ' + itemId + '? This also cancels any pending work item.')) return;
-  closeModal();
   showToast('cmd-toast', 'Item removed', true);
+  markDeleted(_prdItemDeleteKey(source, itemId));
+  closeModal();
+  if (typeof rerenderPrdFromCache === 'function') rerenderPrdFromCache();
   try {
     const res = await fetch('/api/prd-items/remove', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ source, itemId })
     });
     if (res.ok) { refresh(); }
-    else { const d = await res.json().catch(() => ({})); alert('Remove failed: ' + (d.error || 'unknown')); refresh(); }
-  } catch (e) { alert('Error: ' + e.message); refresh(); }
+    else { const d = await res.json().catch(() => ({})); clearDeleted(_prdItemDeleteKey(source, itemId)); showToast('cmd-toast', 'Remove failed: ' + (d.error || 'unknown'), false); refresh(); }
+  } catch (e) { clearDeleted(_prdItemDeleteKey(source, itemId)); showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
 async function prdItemRequeue(workItemId, source, prdFile) {

package/dashboard/js/render-prs.js

@@ -154,18 +154,17 @@ async function _submitLinkPr() {
 
 async function unlinkPr(id) {
   if (!confirm('Remove ' + id + ' from tracking?')) return;
-  // Optimistic: remove row immediately
+  showToast('cmd-toast', id + ' removed', true);
   markDeleted('pr:' + id);
   const row = document.querySelector('[data-pr-id="' + id + '"]')?.closest('tr');
   if (row) row.remove();
-  showToast('cmd-toast', id + ' removed', true);
   try {
     const res = await fetch('/api/pull-requests/delete', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id })
     });
-    if (!res.ok) { _deletedIds.delete('pr:' + id); const d = await res.json().catch(() => ({})); alert('Failed: ' + (d.error || 'unknown')); refresh(); return; }
-  } catch (e) { _deletedIds.delete('pr:' + id); alert('Error: ' + e.message); refresh(); }
+    if (!res.ok) { clearDeleted('pr:' + id); const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'Failed: ' + (d.error || 'unknown'), false); refresh(); return; }
+  } catch (e) { clearDeleted('pr:' + id); showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
 window.MinionsPrs = { prRow, prTableHtml, renderPrs, prPrev, prNext, openAllPrs, openModal, openAddPrModal, unlinkPr };

package/dashboard/js/render-schedules.js

@@ -540,16 +540,16 @@ async function toggleScheduleEnabled(id, enabled) {
 
 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(); });
-  showToast('cmd-toast', 'Schedule deleted', true);
   try {
     const res = await fetch('/api/schedules/delete', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id })
     });
-    if (!res.ok) { const d = await res.json().catch(() => ({})); _showScheduleError('Delete failed: ' + (d.error || 'unknown')); refresh(); }
-  } catch (e) { _showScheduleError('Delete error: ' + e.message); refresh(); }
+    if (!res.ok) { const d = await res.json().catch(() => ({})); clearDeleted('sched:' + id); _showScheduleError('Delete failed: ' + (d.error || 'unknown')); refresh(); }
+  } catch (e) { clearDeleted('sched:' + id); _showScheduleError('Delete error: ' + e.message); refresh(); }
 }
 
 // Expose _generateScheduleId globally for the inline oninput handler

package/dashboard/js/render-watches.js

@@ -193,17 +193,26 @@ function toggleWatchPause(id, pause) {
 
 function deleteWatch(id) {
   if (!confirm('Delete this watch?')) return;
-  markDeleted('watch:' + id);
   showToast('cmd-toast', 'Deleting watch...', true);
+  markDeleted('watch:' + id);
+  renderWatches(window._lastWatches || []);
   fetch('/api/watches/delete', {
     method: 'POST',
     headers: { 'Content-Type': 'application/json' },
     body: JSON.stringify({ id: id })
-  }).then(function(res) { return res.json(); }).then(function(data) {
-    if (data.error) showToast('cmd-toast', 'Error: ' + data.error, false);
-    else renderWatches(window._lastWatches || []);
+  }).then(async function(res) {
+    const data = await res.json().catch(() => ({}));
+    if (!res.ok || data.error) {
+      clearDeleted('watch:' + id);
+      showToast('cmd-toast', 'Delete failed: ' + (data.error || 'unknown'), false);
+      if (typeof refresh === 'function') refresh();
+      return;
+    }
+    renderWatches(window._lastWatches || []);
   }).catch(function(err) {
-    showToast('cmd-toast', 'Error: ' + err.message, false);
+    clearDeleted('watch:' + id);
+    showToast('cmd-toast', 'Delete error: ' + err.message, false);
+    if (typeof refresh === 'function') refresh();
   });
 }
 

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

@@ -197,46 +197,46 @@ async function submitWorkItemEdit(id, source, e) {
 
 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(); });
-  showToast('cmd-toast', 'Work item deleted', true);
   try {
     const res = await fetch('/api/work-items/delete', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id, source: source || undefined })
     });
-    if (!res.ok) { const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'Delete failed: ' + (d.error || 'unknown'), false); refresh(); }
-  } catch (e) { showToast('cmd-toast', 'Delete error: ' + e.message, false); refresh(); }
+    if (!res.ok) { const d = await res.json().catch(() => ({})); clearDeleted('wi:' + id); showToast('cmd-toast', 'Delete failed: ' + (d.error || 'unknown'), false); refresh(); }
+  } catch (e) { clearDeleted('wi:' + id); showToast('cmd-toast', 'Delete error: ' + e.message, false); refresh(); }
 }
 
 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(); });
-  showToast('cmd-toast', 'Work item cancelled', true);
   try {
     const res = await fetch('/api/work-items/cancel', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id, source: source || undefined })
     });
-    if (!res.ok) { const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'Cancel failed: ' + (d.error || 'unknown'), false); refresh(); }
-  } catch (e) { showToast('cmd-toast', 'Cancel error: ' + e.message, false); refresh(); }
+    if (!res.ok) { const d = await res.json().catch(() => ({})); clearDeleted('wi:' + id); showToast('cmd-toast', 'Cancel failed: ' + (d.error || 'unknown'), false); refresh(); }
+  } catch (e) { clearDeleted('wi:' + id); showToast('cmd-toast', 'Cancel error: ' + e.message, false); refresh(); }
 }
 
 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(); });
-  showToast('cmd-toast', 'Archived ' + id, true);
   try {
     const res = await fetch('/api/work-items/archive', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ id, source: source || undefined })
     });
-    if (!res.ok) { const d = await res.json().catch(() => ({})); alert('Archive failed: ' + (d.error || 'unknown')); refresh(); return; }
-  } catch (e) { alert('Archive error: ' + e.message); refresh(); }
+    if (!res.ok) { const d = await res.json().catch(() => ({})); clearDeleted('wi:' + id); showToast('cmd-toast', 'Archive failed: ' + (d.error || 'unknown'), false); refresh(); return; }
+  } catch (e) { clearDeleted('wi:' + id); showToast('cmd-toast', 'Archive error: ' + e.message, false); refresh(); }
 }
 
 let wiArchiveVisible = false;

package/dashboard/js/settings.js

@@ -86,7 +86,7 @@ async function openSettings() {
     '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Projects</h3>' +
     '<div style="display:flex;flex-direction:column;gap:12px;margin-bottom:16px">' +
     (data.projects || []).map(function(p) {
-      return '<div style="border:1px solid var(--border);border-radius:6px;padding:10px 12px">' +
+      return '<div data-settings-project="' + escHtml(p.name) + '" style="border:1px solid var(--border);border-radius:6px;padding:10px 12px">' +
         '<div style="display:flex;align-items:center;justify-content:space-between;margin-bottom:8px">' +
           '<div style="font-size:12px;font-weight:600">' + escHtml(p.name) + '</div>' +
           '<button onclick="MinionsSettings.removeProject(\'' + escHtml(p.name) + '\')" style="font-size:9px;padding:2px 8px;background:transparent;color:var(--red);border:1px solid var(--red);border-radius:3px;cursor:pointer">Remove</button>' +
@@ -409,7 +409,17 @@ async function resetSettingsToDefaults() {
 
 async function removeProject(name) {
   if (!confirm('Remove project "' + name + '"? Pending work cancels, active agents are killed, data dir is archived to projects/.archived/.')) return;
+  const prevProjects = (_settingsData && Array.isArray(_settingsData.projects)) ? _settingsData.projects.slice() : null;
   showToast('cmd-toast', 'Removing project "' + name + '"...', true);
+  markDeleted('project:' + name);
+  if (_settingsData && Array.isArray(_settingsData.projects)) {
+    _settingsData.projects = _settingsData.projects.filter(function(p) { return p.name !== name; });
+  }
+  const card = document.querySelector('[data-settings-project="' + CSS.escape(name) + '"]');
+  if (card) card.remove();
+  if (window._lastStatus && Array.isArray(window._lastStatus.projects) && typeof renderProjects === 'function') {
+    renderProjects(window._lastStatus.projects);
+  }
   try {
     const res = await fetch('/api/projects/remove', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -417,7 +427,13 @@ async function removeProject(name) {
     });
     const d = await res.json().catch(() => ({}));
     if (!res.ok || !d.ok) {
+      if (prevProjects) _settingsData.projects = prevProjects;
+      clearDeleted('project:' + name);
+      if (window._lastStatus && Array.isArray(window._lastStatus.projects) && typeof renderProjects === 'function') {
+        renderProjects(window._lastStatus.projects);
+      }
       showToast('cmd-toast', 'Remove failed: ' + (d.error || 'unknown'), false);
+      openSettings();
       return;
     }
     var parts = ['Removed "' + name + '"'];
@@ -430,7 +446,13 @@ async function removeProject(name) {
     showToast('cmd-toast', parts.join(' — '), true);
     setTimeout(() => openSettings(), 600);
   } catch (e) {
+    if (prevProjects) _settingsData.projects = prevProjects;
+    clearDeleted('project:' + name);
+    if (window._lastStatus && Array.isArray(window._lastStatus.projects) && typeof renderProjects === 'function') {
+      renderProjects(window._lastStatus.projects);
+    }
     showToast('cmd-toast', 'Error: ' + e.message, false);
+    openSettings();
   }
 }
 

package/dashboard/js/utils.js

@@ -3,9 +3,10 @@
 // Signal the engine to tick immediately (pick up new work without waiting 60s)
 function wakeEngine() { fetch('/api/engine/wakeup', { method: 'POST' }).catch(() => {}); }
 
-// Optimistic delete suppression — prevent auto-refresh from re-showing deleted items
-const _deletedIds = new Map(); // key → expiry timestamp
-function markDeleted(key) { _deletedIds.set(key, Date.now() + 10000); } // suppress for 10s
+// Optimistic delete suppression
+const _deletedIds = new Map();
+function markDeleted(key) { _deletedIds.set(key, Date.now() + 10000); }
+function clearDeleted(key) { _deletedIds.delete(key); }
 function isDeleted(key) { const exp = _deletedIds.get(key); if (!exp) return false; if (Date.now() > exp) { _deletedIds.delete(key); return false; } return true; }
 
 // Pin-to-top — persisted server-side so CC and agents can also pin items
@@ -507,4 +508,4 @@ async function submitBugReport() {
   }
 }
 
-window.MinionsUtils = { wakeEngine, escapeHtml, escHtml, renderMd, normalizePlanFile, timeAgo, statusColor, shouldIgnoreSelectionClick, llmCopyBtn, copyLlmText, openBugReport, submitBugReport };
+window.MinionsUtils = { wakeEngine, markDeleted, clearDeleted, escapeHtml, escHtml, renderMd, normalizePlanFile, timeAgo, statusColor, shouldIgnoreSelectionClick, llmCopyBtn, copyLlmText, openBugReport, submitBugReport };

package/docs/self-improvement.md

@@ -257,14 +257,14 @@ Metrics are currently informational — displayed in status and dashboard. Plann
 
 When an agent discovers a repeatable multi-step procedure, it can save it as a **skill** — a structured, reusable workflow compatible with Claude Code's skill system. Skills are stored in two locations:
 
-- **Minions-wide:** `~/.minions/skills/<name>.md` — shared across all agents, no PR required
+- **Minions-wide:** `~/.claude/skills/<name>/SKILL.md` — user-level Claude skill, shared across Minions and normal Claude windows, no PR required
 - **Project-specific:** `<project>/.claude/skills/<name>/SKILL.md` — scoped to one repo, requires a PR
 
 ### Flow
 
 ```
 Agent discovers repeatable pattern during task
-  → writes skills/<name>.md with frontmatter (name, description, trigger, allowed-tools)
+  → writes ~/.claude/skills/<name>/SKILL.md with frontmatter (name, description, trigger, allowed-tools)
   → engine detects new skill files on next tick
   → builds skill index (name + trigger + file path)
   → index injected into every agent's system prompt
@@ -314,10 +314,11 @@ When a git merge or rebase produces conflicts in yarn.lock.
 
 ### When agents should create skills
 
-- Multi-step procedures they had to figure out (build setup, deployment, migration)
-- Error recovery patterns (how to fix a specific class of failure)
-- Project-specific workflows that aren't documented elsewhere
-- Cross-repo coordination steps
+- Only when the workflow is durable, multi-step, and likely to be reused on future tasks
+- Only when the workflow is not already documented in repo docs, notes, playbooks, or existing skills
+- Good candidates: build setup, deployment, migrations, recurring error-recovery patterns, cross-repo coordination steps
+- Prefer zero skills over a weak skill; one-off findings and task-specific notes belong in inbox notes or `notes.md`
+- Do not create skills for isolated fixes, obvious repository facts, or instructions already covered elsewhere
 
 ## Configuration
 

package/engine/playbook.js

@@ -362,14 +362,15 @@ function renderPlaybook(type, vars) {
   content += `- Conventions to follow\n`;
   content += `- **SOURCE REFERENCES for every finding** — file paths with line numbers, PR URLs, API endpoints, config keys. Format: \`(source: path/to/file.ts:42)\` or \`(source: PR-12345)\`. Without references, findings cannot be verified.\n\n`;
   content += `### Skill Extraction (IMPORTANT)\n\n`;
-  content += `If during this task you discovered a **repeatable workflow** — a multi-step procedure, workaround, build process, or pattern that other agents should follow in similar situations — output it as a fenced skill block. The engine will automatically extract it.\n\n`;
+  content += `If during this task you discovered a **repeatable workflow** — a multi-step procedure, workaround, build process, or pattern that other agents should follow in similar situations — only output it as a fenced skill block when **all** of these are true: (1) you had to discover it during this task, (2) it is not already captured in team memory, repo docs, existing playbooks, or existing skills, and (3) another agent is likely to reuse it on future tasks. **Zero skills is the default.** Prefer the inbox findings for one-off notes, repo facts, and task-specific observations.\n\n`;
   content += `Format your skill as a fenced code block with the \`skill\` language tag:\n\n`;
   content += '````\n```skill\n';
   content += `---\nname: short-descriptive-name\ndescription: One-line description of what this skill does\nallowed-tools: Bash, Read, Edit\ntrigger: when should an agent use this\nscope: minions\nproject: any\n---\n\n# Skill Title\n\n## Steps\n1. ...\n2. ...\n\n## Notes\n...\n`;
   content += '```\n````\n\n';
-  content += `- Set \`scope: minions\` for cross-project skills (engine writes to ~/.claude/skills/ automatically)\n`;
-  content += `- Set \`scope: project\` + \`project: <name>\` for repo-specific skills (engine queues a PR to <project>/.claude/skills/)\n`;
-  content += `- Only output a skill block if you genuinely discovered something reusable — don't force it\n`;
+  content += `- Set \`scope: minions\` for cross-project or Minions-wide skills; the engine writes them to ~/.claude/skills/ so they are available in normal Claude windows too\n`;
+  content += `- Set \`scope: project\` + \`project: <name>\` only for repo-specific skills; the engine queues a PR to <project>/.claude/skills/\n`;
+  content += `- Emit at most one skill block per task unless you uncovered two clearly distinct reusable workflows\n`;
+  content += `- Do NOT create a skill for one-off bug fixes, isolated command output, obvious repo facts, or anything already covered by existing docs/playbooks/skills\n`;
 
   // Inject project-level variables from config
   const config = getConfig();
@@ -471,7 +472,7 @@ function buildSystemPrompt(agentId, config, project) {
   prompt += `3. Follow the project conventions in CLAUDE.md if present\n`;
   prompt += `4. Write learnings to the path specified in the task prompt (format: \`notes/inbox/{agent}-{work-item-id}-{date}-{time}.md\`)\n`;
   prompt += `5. Agent status is managed by the engine via dispatch.json — agents do not need to track their own status\n`;
-  prompt += `6. If you discover a repeatable workflow, output it as a \\\`\\\`\\\`skill fenced block — the engine auto-extracts it to ~/.claude/skills/\n\n`;
+  prompt += `6. If you discover a repeatable workflow, output it as a \\\`\\\`\\\`skill fenced block — minions-scoped skills are auto-extracted to ~/.claude/skills/ so they are available in normal Claude windows too\n\n`;
 
   return prompt;
 }
@@ -504,7 +505,7 @@ function buildAgentContext(agentId, config, project) {
 
   // KB and skills: NOT injected — agents can Glob/Read when needed
   // This saves ~27KB per dispatch. Reference note so agents know they exist:
-  context += `## Reference Files\n\nKnowledge base entries are in \`knowledge/{category}/*.md\`. Skills are in \`skills/*.md\` and \`.claude/skills/\`. Use Glob/Read to browse when relevant.\n\n`;
+  context += `## Reference Files\n\nKnowledge base entries are in \`knowledge/{category}/*.md\`. User-level Minions skills live in \`~/.claude/skills/\`, and project-specific skills live in \`<project>/.claude/skills/\`. Use Glob/Read when relevant.\n\n`;
 
   // Minions awareness: what's in flight, who's doing what
   const dispatch = getDispatch();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1562",
+  "version": "0.1.1564",
   "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/explore.md

@@ -60,7 +60,7 @@ Use subagents only for genuinely parallel, independent tasks. For reading files,
 - Use the appropriate MCP tools for PR creation — check available tools before starting.
 - Do NOT checkout branches in the main working tree — use worktrees.
 - Read `notes.md` for all team rules before starting.
-- If you discover a repeatable workflow, output it as a ```skill block (the engine auto-extracts it to ~/.claude/skills/)
+- Only emit a ```skill block if you uncovered a durable reusable workflow that is not already documented and is likely to help future tasks; zero skills is the default, and one-off findings belong in the inbox notes instead.
 
 ## When to Stop
 

package/playbooks/shared-rules.md

@@ -33,7 +33,7 @@ Your context window may be compacted or summarized mid-task by Claude's automati
   5. Previous agent output in `agents/*/live-output.log` for related tasks
   6. Work item descriptions and `resultSummary` for prior completed work on the same topic
   Only after exhausting team memory should you look outside (web search, codebase exploration, external docs). This avoids duplicating research another agent already completed and ensures team decisions are respected.
-- If you discover a repeatable workflow, output it as a fenced skill block. The engine auto-extracts it to `~/.claude/skills/<name>/SKILL.md`. Required format:
+- Only output a fenced skill block when **all** of these are true: (1) you discovered a durable multi-step workflow that was not already documented in team memory, repo docs, existing playbooks, or existing skills, (2) another agent is likely to need it on future tasks, and (3) the workflow is specific enough to be actionable but general enough to stand alone. **Zero skills is the default.** Prefer writing one-off findings, repo facts, or task-specific notes to the inbox findings instead of creating a skill. Emit **at most one skill block per task** unless the task clearly uncovered two unrelated reusable workflows. The engine auto-extracts valid skill blocks to `~/.claude/skills/<name>/SKILL.md`, so `scope: minions` skills become user-level Claude skills available in normal Claude windows too. Required format:
   ````
   ```skill
   ---
@@ -45,7 +45,8 @@ Your context window may be compacted or summarized mid-task by Claude's automati
   Instructions for the skill go here.
   ```
   ````
-  The `name` and `description` fields are required. `scope` defaults to `minions` (global). Use `scope: project` + `project: ProjectName` for project-specific skills.
+  The `name` and `description` fields are required. `scope` defaults to `minions` (global). Use `scope: minions` for user-level reusable skills; use `scope: project` + `project: ProjectName` only for repo-specific skills that should land in that project via PR.
+  Do **not** create a skill for one-off bug fixes, isolated command outputs, obvious repo facts, or anything already covered by existing docs/playbooks/skills.
 - Do TDD where it makes sense — write failing tests first, then implement, then verify tests pass. Especially for bug fixes (write a test that reproduces the bug) and new utility functions.
 
 ## Checking PR and Build Status