@yemi33/minions 0.1.403 → 0.1.405

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.1.405 (2026-04-06)
+
+### Fixes
+-  PR delete is optimistic — row removed immediately, reverted on failure
+-  PR delete searches all project files, not just the first project
+
 ## 0.1.403 (2026-04-06)
 
 ### Fixes

package/dashboard/js/render-prs.js

@@ -39,6 +39,7 @@ function prTableHtml(rows) {
 }
 
 function renderPrs(prs) {
+  prs = prs.filter(p => !isDeleted('pr:' + p.id));
   allPrs = prs;
   const el = document.getElementById('pr-content');
   const count = document.getElementById('pr-count');
@@ -153,15 +154,18 @@ async function _submitLinkPr() {
 
 async function unlinkPr(id) {
   if (!confirm('Remove ' + id + ' from tracking?')) return;
+  // Optimistic: remove row immediately
+  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) { const d = await res.json().catch(() => ({})); alert('Failed: ' + (d.error || 'unknown')); return; }
-    showToast('cmd-toast', id + ' removed', true);
-    refresh();
-  } catch (e) { alert('Error: ' + e.message); }
+    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(); }
 }
 
 window.MinionsPrs = { prRow, prTableHtml, renderPrs, prPrev, prNext, openAllPrs, openModal, openAddPrModal, unlinkPr };

package/dashboard.js

@@ -3637,19 +3637,24 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
     { method: 'POST', path: '/api/pull-requests/delete', desc: 'Remove a PR from tracking', params: 'id, project?', handler: async (req, res) => {
       const body = await readBody(req);
-      const { id, project: projectName } = body;
+      const { id } = body;
       if (!id) return jsonReply(res, 400, { error: 'id required' });
       reloadConfig();
-      const projects = shared.getProjects(CONFIG);
-      const targetProject = projectName ? projects.find(p => p.name?.toLowerCase() === projectName.toLowerCase()) : projects[0];
-      const prPath = targetProject ? shared.projectPrPath(targetProject) : path.join(MINIONS_DIR, 'pull-requests.json');
+      // Search all project PR files and central file
+      const prPaths = [
+        ...shared.getProjects(CONFIG).map(p => shared.projectPrPath(p)),
+        path.join(MINIONS_DIR, 'pull-requests.json'),
+      ];
       let found = false;
-      mutateJsonFileLocked(prPath, (prs) => {
-        if (!Array.isArray(prs)) return prs;
-        const idx = prs.findIndex(p => p.id === id);
-        if (idx >= 0) { prs.splice(idx, 1); found = true; }
-        return prs;
-      }, { defaultValue: [] });
+      for (const prPath of prPaths) {
+        if (found) break;
+        mutateJsonFileLocked(prPath, (prs) => {
+          if (!Array.isArray(prs)) return prs;
+          const idx = prs.findIndex(p => p.id === id);
+          if (idx >= 0) { prs.splice(idx, 1); found = true; }
+          return prs;
+        }, { defaultValue: [] });
+      }
       if (!found) return jsonReply(res, 404, { error: 'PR not found' });
       invalidateStatusCache();
       return jsonReply(res, 200, { ok: true });

package/package.json

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