@yemi33/minions 0.1.838 → 0.1.840

package/CHANGELOG.md

@@ -1,12 +1,14 @@
 # Changelog
 
-## 0.1.838 (2026-04-11)
+## 0.1.840 (2026-04-11)
 
 ### Features
+-  optimistic UI updates for all dashboard actions
 -  per-item Re-open button on done PRD items (deterministic fallback)
 -  stale PRD shows Regenerate PRD + Resume as-is buttons
 
 ### Fixes
+-  restore text selection → doc-chat flow + regression tests
 -  use structured marker for diff-aware PRD update detection
 -  match playbook trigger phrase for diff-aware PRD updates
 -  engine only flags stale on plan revision, never auto-regenerates

package/dashboard/js/modal-qa.js

@@ -414,4 +414,30 @@ function _showThreadWrap() {
   });
 })();
 
+// ── Text selection → "Ask about this" button ─────────────────────────────────
+document.addEventListener('mouseup', function() {
+  var askBtn = document.getElementById('ask-selection-btn');
+  if (!askBtn) return;
+  // Only act inside modal body
+  var sel = window.getSelection();
+  if (!sel || sel.isCollapsed || !sel.toString().trim()) {
+    askBtn.style.display = 'none';
+    return;
+  }
+  var modalBody = document.getElementById('modal-body');
+  if (!modalBody || !modalBody.contains(sel.anchorNode)) {
+    askBtn.style.display = 'none';
+    return;
+  }
+  var text = sel.toString().trim();
+  if (!text) { askBtn.style.display = 'none'; return; }
+  _modalDocContext.selection = text;
+  // Position near the selection
+  var range = sel.getRangeAt(0);
+  var rect = range.getBoundingClientRect();
+  askBtn.style.top = (rect.bottom + 4) + 'px';
+  askBtn.style.left = rect.left + 'px';
+  askBtn.style.display = 'block';
+});
+
 window.MinionsQA = { showModalQa, modalAskAboutSelection, clearQaSelection, clearQaConversation, modalSend, qaAbort, toggleDocChat, _showThreadWrap };

package/dashboard/js/render-plans.js

@@ -558,16 +558,16 @@ async function planView(file) {
 
 async function planApprove(file, btn) {
   if (btn) { btn.dataset.origText = btn.textContent; btn.textContent = 'Approving...'; btn.style.pointerEvents = 'none'; btn.style.opacity = '0.6'; }
+  showToast('cmd-toast', 'Plan approved — work will begin on next engine tick', true);
   try {
     const res = await fetch('/api/plans/approve', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ file }) });
     if (res.ok) {
-      showToast('cmd-toast', 'Plan approved — work will begin on next engine tick', true);
       refreshPlans();
       refresh();
     } else {
       if (btn) { btn.textContent = btn.dataset.origText || 'Approve'; btn.style.pointerEvents = ''; btn.style.opacity = ''; }
       const d = await res.json().catch(() => ({}));
-      alert('Approve failed: ' + (d.error || 'unknown'));
+      showToast('cmd-toast', 'Approve failed: ' + (d.error || 'unknown'), false);
     }
   } catch (e) { if (btn) { btn.textContent = btn.dataset.origText || 'Approve'; btn.style.pointerEvents = ''; btn.style.opacity = ''; } showToast('cmd-toast', 'Error: ' + e.message, false); }
 }
@@ -626,16 +626,16 @@ async function planArchive(file, btn) {
 
 async function planPause(file, btn) {
   if (btn) { btn.dataset.origText = btn.textContent; btn.textContent = 'Pausing...'; btn.style.pointerEvents = 'none'; btn.style.opacity = '0.6'; }
+  showToast('cmd-toast', 'Plan paused — no new items will be dispatched', true);
   try {
     const res = await fetch('/api/plans/pause', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ file }) });
     if (res.ok) {
-      showToast('cmd-toast', 'Plan paused — no new items will be dispatched', true);
       refreshPlans();
       refresh();
     } else {
       if (btn) { btn.textContent = btn.dataset.origText || 'Pause'; btn.style.pointerEvents = ''; btn.style.opacity = ''; }
       const d = await res.json().catch(() => ({}));
-      alert('Pause failed: ' + (d.error || 'unknown'));
+      showToast('cmd-toast', 'Pause failed: ' + (d.error || 'unknown'), false);
     }
   } catch (e) { if (btn) { btn.textContent = btn.dataset.origText || 'Pause'; btn.style.pointerEvents = ''; btn.style.opacity = ''; } showToast('cmd-toast', 'Error: ' + e.message, false); }
 }
@@ -643,14 +643,14 @@ async function planPause(file, btn) {
 async function planReject(file) {
   if (!confirm('Reject this plan? It will not be executed.')) return;
   const reason = prompt('Reason for rejection (optional):') || '';
+  showToast('cmd-toast', 'Plan rejected', true);
   try {
     const res = await fetch('/api/plans/reject', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ file, reason }) });
     if (res.ok) {
-      showToast('cmd-toast', 'Plan rejected', true);
       refreshPlans();
     } else {
       const d = await res.json().catch(() => ({}));
-      alert('Reject failed: ' + (d.error || 'unknown'));
+      showToast('cmd-toast', 'Reject failed: ' + (d.error || 'unknown'), false);
     }
   } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); }
 }
@@ -762,6 +762,7 @@ async function openVerifyGuide(file) {
 
 async function triggerVerify(file, btn) {
   if (btn) { btn.dataset.origText = btn.textContent; btn.textContent = 'Verifying...'; btn.style.pointerEvents = 'none'; btn.style.opacity = '0.6'; }
+  showToast('cmd-toast', 'Verify task queued', true);
   try {
     const res = await fetch('/api/plans/trigger-verify', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
@@ -771,7 +772,6 @@ async function triggerVerify(file, btn) {
     if (res.ok && d.ok) {
       try { closeModal(); } catch { /* may not be open */ }
       refresh();
-      showToast('cmd-toast', d.verifyId ? 'Verify task ' + d.verifyId + ' created' : (d.message || 'Done'), true);
     } else {
       if (btn) { btn.textContent = btn.dataset.origText || 'Verify'; btn.style.pointerEvents = ''; btn.style.opacity = ''; }
       alert('Failed: ' + (d.error || 'unknown'));

package/dashboard/js/render-prd.js

@@ -713,20 +713,22 @@ async function prdItemEdit(source, itemId) {
 
 async function prdItemSave(source, itemId) {
   var btn = event?.target; if (btn) { btn.disabled = true; btn.textContent = 'Saving...'; }
+  const body = {
+    source, itemId,
+    name: document.getElementById('prd-edit-name').value,
+    description: document.getElementById('prd-edit-desc').value,
+    priority: document.getElementById('prd-edit-priority').value,
+    estimated_complexity: document.getElementById('prd-edit-complexity').value,
+  };
+  closeModal(); showToast('cmd-toast', 'Item updated', true);
   try {
     const res = await fetch('/api/prd-items/update', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({
-        source, itemId,
-        name: document.getElementById('prd-edit-name').value,
-        description: document.getElementById('prd-edit-desc').value,
-        priority: document.getElementById('prd-edit-priority').value,
-        estimated_complexity: document.getElementById('prd-edit-complexity').value,
-      })
+      body: JSON.stringify(body)
     });
-    if (res.ok) { closeModal(); refresh(); showToast('cmd-toast', 'Item updated', true); }
-    else { if (btn) { btn.disabled = false; btn.textContent = 'Save'; } const d = await res.json(); alert('Failed: ' + (d.error || 'unknown')); }
-  } catch (e) { if (btn) { btn.disabled = false; btn.textContent = 'Save'; } alert('Error: ' + e.message); }
+    if (res.ok) { refresh(); }
+    else { const d = await res.json().catch(() => ({})); showToast('cmd-toast', 'Save failed: ' + (d.error || 'unknown'), false); }
+  } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); }
 }
 
 async function prdItemRemove(source, itemId) {
@@ -775,19 +777,17 @@ async function prdItemRequeue(workItemId, source, prdFile) {
 
 async function prdItemReopen(source, itemId) {
   if (!confirm('Re-open this item? It will be set to "updated" and the engine will re-dispatch it on the existing branch.')) return;
+  showToast('cmd-toast', 'Item re-opened — will dispatch on next tick', true);
   try {
-    // Set item to "updated" + clear stale flag + approve so materializer picks it up
     const res = await fetch('/api/prd-items/update', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ source, itemId, status: 'updated' })
     });
     if (res.ok) {
-      // Also approve the plan (clears planStale so materializer runs)
       await fetch('/api/plans/approve', {
         method: 'POST', headers: { 'Content-Type': 'application/json' },
         body: JSON.stringify({ file: source, skipRegen: true })
       });
-      showToast('cmd-toast', 'Item re-opened — will dispatch on next tick', true);
       refresh();
     } else {
       const d = await res.json().catch(() => ({}));
@@ -798,19 +798,19 @@ async function prdItemReopen(source, itemId) {
 
 async function _planApproveAction(prdFile, skipRegen, confirmMsg, successMsg) {
   if (!confirm(confirmMsg)) return;
+  showToast('cmd-toast', successMsg || (skipRegen ? 'Plan resumed' : 'PRD regeneration queued'), true);
   try {
     const res = await fetch('/api/plans/approve', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ file: prdFile, skipRegen: skipRegen || undefined })
     });
-    const d = await res.json();
     if (res.ok) {
-      showToast('cmd-toast', successMsg || (d.diffAwareUpdate ? 'Diff-aware PRD update queued' : 'Plan approved'), true);
       refresh();
     } else {
-      alert('Failed: ' + (d.error || 'unknown'));
+      const d = await res.json().catch(() => ({}));
+      showToast('cmd-toast', 'Failed: ' + (d.error || 'unknown'), false);
     }
-  } catch (e) { alert('Error: ' + e.message); }
+  } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); }
 }
 
 function prdRegenerate(prdFile) {

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

@@ -198,13 +198,14 @@ async function deleteWorkItem(id, source) {
   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(() => ({})); alert('Delete failed: ' + (d.error || 'unknown')); refresh(); }
-  } catch (e) { alert('Delete error: ' + e.message); refresh(); }
+    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(); }
 }
 
 async function archiveWorkItem(id, source) {

package/package.json

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