@yemi33/minions 0.1.28 → 0.1.30

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.30 (2026-03-28)
+
+### Dashboard
+- dashboard/js/render-work-items.js
+
+## 0.1.29 (2026-03-28)
+
+### Dashboard
+- dashboard.js
+
 ## 0.1.28 (2026-03-28)
 
 ### Engine

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

@@ -226,20 +226,42 @@ async function retryWorkItem(id, source) {
 function wiPrev() { if (wiPage > 0) { wiPage--; renderWorkItems(allWorkItems); } }
 function wiNext() { const tp = Math.ceil(allWorkItems.length / WI_PER_PAGE); if (wiPage < tp-1) { wiPage++; renderWorkItems(allWorkItems); } }
 
+let _feedbackRating = null;
 function feedbackWorkItem(id, source) {
+  _feedbackRating = null;
   document.getElementById('modal-title').textContent = 'Feedback on ' + id;
   document.getElementById('modal-body').innerHTML =
-    '<div style="display:flex;flex-direction:column;gap:16px;align-items:center">' +
-      '<div style="display:flex;gap:24px">' +
-        '<button onclick="submitFeedback(\'' + escHtml(id) + '\',\'' + escHtml(source) + '\',\'up\')" style="font-size:40px;background:none;border:2px solid var(--border);border-radius:12px;padding:16px 24px;cursor:pointer;transition:all 0.2s" onmouseover="this.style.borderColor=\'var(--green)\'" onmouseout="this.style.borderColor=\'var(--border)\'">&#x1F44D;</button>' +
-        '<button onclick="submitFeedback(\'' + escHtml(id) + '\',\'' + escHtml(source) + '\',\'down\')" style="font-size:40px;background:none;border:2px solid var(--border);border-radius:12px;padding:16px 24px;cursor:pointer;transition:all 0.2s" onmouseover="this.style.borderColor=\'var(--red)\'" onmouseout="this.style.borderColor=\'var(--border)\'">&#x1F44E;</button>' +
+    '<div style="display:flex;flex-direction:column;gap:16px">' +
+      '<div style="display:flex;gap:16px;justify-content:center">' +
+        '<button id="fb-up" onclick="_selectRating(\'up\')" style="font-size:36px;background:none;border:2px solid var(--border);border-radius:12px;padding:12px 20px;cursor:pointer;transition:all 0.2s">&#x1F44D;</button>' +
+        '<button id="fb-down" onclick="_selectRating(\'down\')" style="font-size:36px;background:none;border:2px solid var(--border);border-radius:12px;padding:12px 20px;cursor:pointer;transition:all 0.2s">&#x1F44E;</button>' +
+      '</div>' +
+      '<textarea id="feedback-comment" rows="3" style="width:100%;padding:8px;background:var(--bg);border:1px solid var(--border);border-radius:var(--radius-sm);color:var(--text);font-family:inherit;resize:vertical" placeholder="What was good or needs improvement?"></textarea>' +
+      '<div style="display:flex;justify-content:flex-end;gap:8px">' +
+        '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
+        '<button id="fb-send" onclick="submitFeedback(\'' + escHtml(id) + '\',\'' + escHtml(source) + '\')" style="padding:6px 16px;background:var(--surface2);color:var(--muted);border:1px solid var(--border);border-radius:var(--radius-sm);cursor:not-allowed" disabled>Select rating first</button>' +
       '</div>' +
-      '<textarea id="feedback-comment" rows="3" style="width:100%;padding:8px;background:var(--bg);border:1px solid var(--border);border-radius:var(--radius-sm);color:var(--text);font-family:inherit;resize:vertical" placeholder="Optional: what was good or needs improvement?"></textarea>' +
     '</div>';
   document.getElementById('modal').classList.add('open');
 }
+function _selectRating(r) {
+  _feedbackRating = r;
+  document.getElementById('fb-up').style.borderColor = r === 'up' ? 'var(--green)' : 'var(--border)';
+  document.getElementById('fb-up').style.background = r === 'up' ? 'rgba(63,185,80,0.1)' : 'none';
+  document.getElementById('fb-down').style.borderColor = r === 'down' ? 'var(--red)' : 'var(--border)';
+  document.getElementById('fb-down').style.background = r === 'down' ? 'rgba(248,81,73,0.1)' : 'none';
+  const btn = document.getElementById('fb-send');
+  btn.disabled = false;
+  btn.style.background = 'var(--blue)';
+  btn.style.color = '#fff';
+  btn.style.cursor = 'pointer';
+  btn.style.borderColor = 'var(--blue)';
+  btn.textContent = 'Send Feedback';
+}
 
-async function submitFeedback(id, source, rating) {
+async function submitFeedback(id, source) {
+  const rating = _feedbackRating;
+  if (!rating) { alert('Please select a rating first'); return; }
   const comment = document.getElementById('feedback-comment')?.value || '';
   try {
     const res = await fetch('/api/work-items/feedback', {

package/dashboard.js

@@ -596,6 +596,7 @@ function jsonReply(res, code, data, req) {
  */
 function cleanDispatchEntries(matchFn) {
   const dispatchPath = path.join(MINIONS_DIR, 'engine', 'dispatch.json');
+  const engineDir = path.join(MINIONS_DIR, 'engine');
   try {
     let removed = 0;
     mutateJsonFileLocked(dispatchPath, (dispatch) => {
@@ -606,16 +607,18 @@ function cleanDispatchEntries(matchFn) {
         const before = dispatch[queue].length;
         if (queue === 'active') {
           for (const d of dispatch[queue]) {
-            if (matchFn(d) && d.agent) {
-              const statusPath = path.join(MINIONS_DIR, 'agents', d.agent, 'status.json');
-              try {
-                const status = JSON.parse(safeRead(statusPath) || '{}');
-                if (status.pid) try { process.kill(status.pid, 'SIGTERM'); } catch {}
-                status.status = 'idle';
-                delete status.currentTask;
-                safeWrite(statusPath, status);
-              } catch {}
-            }
+            if (!matchFn(d)) continue;
+            // Kill the running agent process via PID file
+            const pidFile = path.join(engineDir, `pid-${d.id}.pid`);
+            try {
+              const pid = parseInt(fs.readFileSync(pidFile, 'utf8').trim());
+              if (pid) process.kill(pid, 'SIGTERM');
+            } catch {}
+            try { fs.unlinkSync(pidFile); } catch {}
+            // Clean up temp prompt files
+            try { fs.unlinkSync(path.join(engineDir, 'tmp', `prompt-${d.id}.md`)); } catch {}
+            try { fs.unlinkSync(path.join(engineDir, 'tmp', `sysprompt-${d.id}.md`)); } catch {}
+            try { fs.unlinkSync(path.join(engineDir, 'tmp', `sysprompt-${d.id}.md.tmp`)); } catch {}
           }
         }
         dispatch[queue] = dispatch[queue].filter(d => !matchFn(d));
@@ -814,15 +817,24 @@ const server = http.createServer(async (req, res) => {
       safeWrite(wiPath, items);
 
       // Clean dispatch entries + kill running agent
-      const sourcePrefix = (!source || source === 'central') ? 'central-work-' : `work-${source}-`;
-      const dispatchKey = sourcePrefix + id;
-      cleanDispatchEntries(d =>
-        d.meta?.dispatchKey === dispatchKey ||
-        (d.meta?.parentKey && d.meta.parentKey === dispatchKey) ||
-        d.meta?.item?.id === id
+      const dispatchRemoved = cleanDispatchEntries(d =>
+        d.meta?.item?.id === id ||
+        d.meta?.dispatchKey?.endsWith(id)
       );
 
-      return jsonReply(res, 200, { ok: true, id });
+      // Clean cooldown entries so item can be re-created immediately
+      try {
+        const cooldownPath = path.join(MINIONS_DIR, 'engine', 'cooldowns.json');
+        const cooldowns = JSON.parse(safeRead(cooldownPath) || '{}');
+        let cleaned = false;
+        for (const key of Object.keys(cooldowns)) {
+          if (key.includes(id)) { delete cooldowns[key]; cleaned = true; }
+        }
+        if (cleaned) safeWrite(cooldownPath, cooldowns);
+      } catch {}
+
+      invalidateStatusCache();
+      return jsonReply(res, 200, { ok: true, id, dispatchRemoved });
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 

package/package.json

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