@yemi33/minions 0.1.417 → 0.1.419

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.1.419 (2026-04-06)
+
+### Features
+- Fix 6 medium bugs: dispatch pruning, null guards, skill regex, meeting advancement, CLI PID check, pipeline retry
+- Convert remaining lifecycle.js safeWrite calls to mutateJsonFileLocked
+
+### Fixes
+-  address review feedback — pipeline.js socket leak and magic numbers
+
 ## 0.1.417 (2026-04-06)
 
 ### Features

package/engine/dispatch.js

@@ -87,7 +87,7 @@ function completeDispatch(id, result = DISPATCH_RESULT.SUCCESS, reason = '', res
     if (reason) item.reason = reason;
     if (resultSummary) item.resultSummary = resultSummary;
     delete item.prompt;
-    if (dispatch.completed.length > 100) {
+    if (dispatch.completed.length >= 100) {
       dispatch.completed = dispatch.completed.slice(-100);
     }
     dispatch.completed.push(item);

package/engine/lifecycle.js

@@ -901,6 +901,7 @@ function extractSkillsFromOutput(output, agentId, dispatchItem, config) {
   if (!fullText) fullText = output;
   const skillBlocks = [];
   const skillRegex = /```skill\s*\n([\s\S]*?)```/g;
+  skillRegex.lastIndex = 0;
   let match;
   while ((match = skillRegex.exec(fullText)) !== null) {
     skillBlocks.push(match[1].trim());

package/engine/pipeline.js

@@ -7,7 +7,8 @@
 const fs = require('fs');
 const path = require('path');
 const shared = require('./shared');
-const { safeJson, safeWrite, safeRead, safeReadDir, uid, log, ts, dateStamp, mutateJsonFileLocked, WI_STATUS, WORK_TYPE, PLAN_STATUS, PR_STATUS, PIPELINE_STATUS, STAGE_TYPE, MEETING_STATUS } = shared;
+const { safeJson, safeWrite, safeRead, safeReadDir, uid, log, ts, dateStamp, mutateJsonFileLocked, WI_STATUS, WORK_TYPE, PLAN_STATUS, PR_STATUS, PIPELINE_STATUS, STAGE_TYPE, MEETING_STATUS, ENGINE_DEFAULTS } = shared;
+const http = require('http');
 const { parseCronExpr, shouldRunNow } = require('./scheduler');
 
 const PIPELINES_DIR = path.join(__dirname, '..', 'pipelines');
@@ -313,18 +314,34 @@ function executeApiStage(stage, stageState, run) {
   for (const call of calls) {
     const url = `http://localhost:${process.env.MINIONS_PORT || 7331}${call.endpoint}`;
     const body = typeof call.body === 'string' ? call.body : JSON.stringify(call.body || {});
-    // Fire and forget — use Node's http module
-    try {
-      const http = require('http');
-      const parsed = new URL(url);
-      const req = http.request({
-        hostname: parsed.hostname, port: parsed.port, path: parsed.pathname,
-        method: call.method || 'POST',
-        headers: { 'Content-Type': 'application/json' },
-      });
-      req.write(body);
-      req.end();
-    } catch (e) { log('warn', `Pipeline API call failed: ${e.message}`); }
+    const maxAttempts = ENGINE_DEFAULTS.pipelineApiRetries;
+    const retryDelay = ENGINE_DEFAULTS.pipelineApiRetryDelay;
+    const makeRequest = (attempt) => {
+      try {
+        const parsed = new URL(url);
+        const req = http.request({
+          hostname: parsed.hostname, port: parsed.port, path: parsed.pathname,
+          method: call.method || 'POST',
+          headers: { 'Content-Type': 'application/json' },
+        }, (res) => {
+          res.resume(); // drain body to free socket
+          if (res.statusCode >= 400) {
+            log('warn', `Pipeline API call to ${call.endpoint} returned ${res.statusCode} (attempt ${attempt})`);
+            if (attempt < maxAttempts) setTimeout(() => makeRequest(attempt + 1), retryDelay);
+          }
+        });
+        req.on('error', (err) => {
+          log('warn', `Pipeline API call to ${call.endpoint} failed: ${err.message} (attempt ${attempt})`);
+          if (attempt < maxAttempts) setTimeout(() => makeRequest(attempt + 1), retryDelay);
+        });
+        req.write(body);
+        req.end();
+      } catch (e) {
+        log('warn', `Pipeline API call to ${call.endpoint} threw: ${e.message} (attempt ${attempt})`);
+        if (attempt < maxAttempts) setTimeout(() => makeRequest(attempt + 1), retryDelay);
+      }
+    };
+    makeRequest(1);
   }
   return { status: PIPELINE_STATUS.COMPLETED, completedAt: ts() };
 }

package/engine/shared.js

@@ -438,6 +438,8 @@ const ENGINE_DEFAULTS = {
   evalMaxIterations: 3, // max review→fix cycles before escalating to human
   evalMaxCost: null, // USD ceiling per work item across all eval iterations; null = no limit (gather baseline data first)
   maxRetries: 3, // max dispatch retries before marking work item as failed
+  pipelineApiRetries: 2, // max attempts for pipeline API calls
+  pipelineApiRetryDelay: 2000, // ms delay between pipeline API retries
 };
 
 // ─── Status & Type Constants ─────────────────────────────────────────────────

package/package.json

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