@yemi33/minions 0.1.858 → 0.1.859

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.858 (2026-04-11)
+## 0.1.859 (2026-04-11)
 
 ### Fixes
+-  pipeline race condition, delete refresh, abort error logging
 -  _continuePipeline now calls refresh() to update card list
 -  pipeline modal refresh was silently broken — API returns array not object
 -  pipeline modal now auto-updates on all action buttons

package/dashboard/js/render-pipelines.js

@@ -521,7 +521,8 @@ async function _deletePipelineConfirm(id) {
   showToast('cmd-toast', 'Pipeline deleted', true);
   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); refresh(); }
+    if (!res.ok) { showToast('cmd-toast', 'Delete failed', false); }
+    refresh();
   } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); refresh(); }
 }
 

package/dashboard.js

@@ -4446,8 +4446,9 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (!pipeline) return jsonReply(res, 404, { error: 'Pipeline not found' });
       if (getActiveRun(body.id)) return jsonReply(res, 409, { error: 'Pipeline already has an active run' });
       const run = startRun(body.id, pipeline);
+      if (!run) return jsonReply(res, 409, { error: 'Pipeline already has an active run' });
       invalidateStatusCache();
-      return jsonReply(res, 200, { ok: true, runId: run.runId });
+      return jsonReply(res, 200, { ok: true, runId: run?.runId });
     }},
     { method: 'POST', path: '/api/pipelines/continue', desc: 'Continue a pipeline past a wait stage', params: 'id, stageId', handler: async (req, res) => {
       const body = await readBody(req);
@@ -4482,7 +4483,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
               }
             }
           });
-        } catch {}
+        } catch (e) { console.error(`Pipeline abort: WI cancel error: ${e.message}`); }
       }
       const dispatchCleaned = cleanDispatchEntries(d => d.meta?.item?._pipelineRun === run.runId);
       invalidateStatusCache();

package/engine/pipeline.js

@@ -75,14 +75,24 @@ function startRun(pipelineId, pipeline) {
   }
   const run = { runId, pipelineId, startedAt: ts(), status: PIPELINE_STATUS.RUNNING, stages };
 
+  let alreadyActive = false;
   mutateJsonFileLocked(PIPELINE_RUNS_PATH, (data) => {
     if (!data[pipelineId]) data[pipelineId] = [];
+    // Guard: skip if there's already an active run (prevents race between ticks)
+    if (data[pipelineId].some(r => r.status === PIPELINE_STATUS.RUNNING || r.status === PIPELINE_STATUS.PAUSED)) {
+      alreadyActive = true;
+      return data;
+    }
     // Keep last 10 runs per pipeline
     if (data[pipelineId].length >= 10) data[pipelineId] = data[pipelineId].slice(-9);
     data[pipelineId].push(run);
     return data;
   }, { defaultValue: {} });
 
+  if (alreadyActive) {
+    log('info', `Pipeline ${pipelineId}: skipped startRun — active run already exists`);
+    return null;
+  }
   log('info', `Pipeline ${pipelineId}: started run ${runId}`);
   return run;
 }

package/package.json

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