@yemi33/minions 0.1.2327 → 0.1.2329

package/engine/cli.js

@@ -1488,7 +1488,7 @@ const commands = {
     }
 
     console.log('');
-    const metrics = shared.safeJson(path.join(ENGINE_DIR, 'metrics.json')) || {};
+    const metrics = shared.safeJsonObj(path.join(ENGINE_DIR, '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`);
     const pidSummary = summarizeActiveDispatchPids(dispatch.active || []);

package/engine/meeting.js

@@ -881,7 +881,15 @@ function deleteMeeting(id) {
   _killMeetingDispatches(id);
   const filePath = path.join(MEETINGS_DIR, id + '.json');
   if (!fs.existsSync(filePath)) return false;
-  fs.unlinkSync(filePath);
+  try {
+    fs.unlinkSync(filePath);
+  } catch (e) {
+    // TOCTOU: a concurrent delete of the same meeting may win the race
+    // between existsSync and unlinkSync. Treat ENOENT as already-deleted
+    // (idempotent false) instead of letting it propagate as a 500.
+    if (e.code !== 'ENOENT') throw e;
+    return false;
+  }
   // mutateMeeting writes a .backup sidecar; safeJson auto-restores from it
   // when the primary is missing, so deletion must also drop the backup.
   try { fs.unlinkSync(filePath + '.backup'); } catch { /* sidecar may not exist */ }

package/package.json

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