@yemi33/minions 0.1.1798 → 0.1.1800

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.1800 (2026-05-08)
+
+### Other
+- Remove unredacted CC parse warning (#2213)
+
+## 0.1.1799 (2026-05-08)
+
+### Other
+- Clarify active dispatch PID liveness
+
 ## 0.1.1798 (2026-05-08)
 
 ### Other

package/dashboard.js

@@ -1871,10 +1871,9 @@ function parseCCActions(text) {
   const result = { text: displayText, actions };
   if (parseError && actions.length === 0) {
     result._actionParseError = parseError;
-    // Visibility for the engine log — silent failure here previously masked issue #1834.
+    // Visibility for the engine log — shared.log applies SEC-09 redaction before persistence.
     try {
       const snippet = (segment.trim() || '').slice(0, 200);
-      console.warn(`[CC] action JSON parse failed (${parseError}); raw segment: ${snippet}`);
       if (typeof shared !== 'undefined' && shared && typeof shared.log === 'function') {
         shared.log('warn', `CC action JSON parse failed: ${parseError} — segment: ${snippet}`);
       }

package/engine/cli.js

@@ -52,6 +52,44 @@ function isEngineProcessAlive(control) {
   }
 }
 
+function dispatchSafeId(itemId) {
+  return String(itemId || '').replace(/[:\\/*?"<>|]/g, '-');
+}
+
+function readDispatchPid(itemId) {
+  const pidFile = path.join(ENGINE_DIR, 'tmp', `pid-${dispatchSafeId(itemId)}.pid`);
+  let raw;
+  try { raw = fs.readFileSync(pidFile, 'utf8').trim(); }
+  catch { return null; }
+  if (!raw) return null;
+  try { return shared.validatePid(raw); }
+  catch { return null; }
+}
+
+function isPidAlive(pid) {
+  try {
+    process.kill(shared.validatePid(pid), 0);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+function summarizeActiveDispatchPids(activeItems = []) {
+  const summary = { live: 0, dead: 0, missing: 0 };
+  for (const item of activeItems || []) {
+    const pid = readDispatchPid(item.id);
+    if (!pid) {
+      summary.missing++;
+    } else if (isPidAlive(pid)) {
+      summary.live++;
+    } else {
+      summary.dead++;
+    }
+  }
+  return summary;
+}
+
 function createControlOwner(pid = process.pid) {
   return { pid, ownerToken: `${pid}-${shared.uid()}` };
 }
@@ -453,21 +491,13 @@ const commands = {
     e.loadCooldowns();
 
     // Re-attach to surviving agent processes from previous session
-    const { exec } = require('./shared');
     const dispatch = getDispatch();
     const activeOnStart = (dispatch.active || []);
     if (activeOnStart.length > 0) {
       let reattached = 0;
       for (const item of activeOnStart) {
         const agentId = item.agent;
-        let agentPid = null;
-
-        const safeId = item.id.replace(/[:\\/*?"<>|]/g, '-');
-        const pidFile = path.join(ENGINE_DIR, 'tmp', `pid-${safeId}.pid`);
-        try {
-          const pidStr = fs.readFileSync(pidFile, 'utf8').trim();
-          if (pidStr) agentPid = parseInt(pidStr);
-        } catch { /* optional */ }
+        let agentPid = readDispatchPid(item.id);
 
         if (!agentPid) {
           const status = getAgentStatus(agentId);
@@ -484,16 +514,7 @@ const commands = {
         }
 
         const hadPid = agentPid && agentPid > 0; // track before liveness check
-        if (agentPid && agentPid > 0) {
-          try {
-            if (process.platform === 'win32') {
-              const out = exec(`tasklist /FI "PID eq ${agentPid}" /NH`, { encoding: 'utf8', timeout: 3000 }).trim();
-              if (!new RegExp(`\\b${agentPid}\\b`).test(out)) agentPid = null;
-            } else {
-              process.kill(agentPid, 0);
-            }
-          } catch { agentPid = null; }
-        }
+        if (agentPid && agentPid > 0 && !isPidAlive(agentPid)) agentPid = null;
 
         // PID was found but confirmed dead — exempt from restart grace period (#869)
         if (hadPid && !agentPid) {
@@ -1003,7 +1024,8 @@ const commands = {
     const metrics = shared.safeJson(path.join(__dirname, 'metrics.json')) || {};
     const lifetimeCompleted = Object.entries(metrics).filter(([k]) => !k.startsWith('_')).reduce((sum, [, m]) => sum + (m.tasksCompleted || 0) + (m.tasksErrored || 0), 0);
     console.log(`Dispatch: ${dispatch.pending.length} pending | ${(dispatch.active || []).length} active | ${lifetimeCompleted} completed`);
-    console.log(`Active processes: ${e.activeProcesses.size}`);
+    const pidSummary = summarizeActiveDispatchPids(dispatch.active || []);
+    console.log(`Active dispatch PID files: ${pidSummary.live} live | ${pidSummary.dead} dead | ${pidSummary.missing} missing`);
 
     const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
     const metricsData = safeJson(metricsPath);

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-08T16:36:25.273Z"
+  "cachedAt": "2026-05-08T16:56:04.298Z"
 }

package/package.json

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