@yemi33/minions 0.1.474 → 0.1.475

package/CHANGELOG.md

@@ -1,11 +1,14 @@
 # Changelog
 
-## 0.1.474 (2026-04-07)
+## 0.1.475 (2026-04-07)
 
 ### Features
+- Fix unlocked metrics.json in lifecycle.js post-merge hook
+- Fix unlocked metrics.json read-modify-write in trackEngineUsage
 - Limit SSE initial live-stream payload to last 64KB
 
 ### Fixes
+-  add optional chaining for config.agents in updatePrAfterReview
 -  pass config to updatePrAfterReview to fix test failure
 
 ## 0.1.473 (2026-04-07)

package/engine/lifecycle.js

@@ -717,7 +717,7 @@ function updatePrAfterReview(agentId, pr, project, config) {
   if (!pr?.id) return;
 
   if (!config) config = getConfig();
-  const reviewerName = config.agents[agentId]?.name || agentId;
+  const reviewerName = config.agents?.[agentId]?.name || agentId;
   const dispatch = getDispatch();
   const completedEntry = (dispatch.completed || []).find(d => d.agent === agentId && d.type === 'review');
 
@@ -853,10 +853,11 @@ async function handlePostMerge(pr, project, config, newStatus) {
   const agentId = (pr.agent || '').toLowerCase();
   if (agentId && config.agents?.[agentId]) {
     const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
-    const metrics = safeJson(metricsPath) || {};
-    if (!metrics[agentId]) metrics[agentId] = { tasksCompleted:0, tasksErrored:0, prsCreated:0, prsApproved:0, prsRejected:0, prsMerged:0, reviewsDone:0, lastTask:null, lastCompleted:null };
-    metrics[agentId].prsMerged = (metrics[agentId].prsMerged || 0) + 1;
-    shared.safeWrite(metricsPath, metrics);
+    mutateJsonFileLocked(metricsPath, (metrics) => {
+      if (!metrics[agentId]) metrics[agentId] = { tasksCompleted:0, tasksErrored:0, prsCreated:0, prsApproved:0, prsRejected:0, prsMerged:0, reviewsDone:0, lastTask:null, lastCompleted:null };
+      metrics[agentId].prsMerged = (metrics[agentId].prsMerged || 0) + 1;
+      return metrics;
+    });
   }
 
   const teamsUrl = process.env.TEAMS_PLAN_FLOW_URL;

package/engine/llm.js

@@ -5,7 +5,7 @@
 
 const path = require('path');
 const shared = require('./shared');
-const { safeRead, safeWrite, safeUnlink, uid, runFile, cleanChildEnv, parseStreamJsonOutput } = shared;
+const { safeWrite, safeUnlink, uid, runFile, cleanChildEnv, parseStreamJsonOutput, mutateJsonFileLocked } = shared;
 
 const MINIONS_DIR = path.resolve(__dirname, '..');
 const ENGINE_DIR = __dirname;
@@ -14,31 +14,30 @@ function trackEngineUsage(category, usage) {
   if (!usage) return;
   try {
     const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
-    const raw = safeRead(metricsPath);
-    const metrics = raw ? JSON.parse(raw) : {};
-
-    if (!metrics._engine) metrics._engine = {};
-    if (!metrics._engine[category]) {
-      metrics._engine[category] = { calls: 0, costUsd: 0, inputTokens: 0, outputTokens: 0, cacheRead: 0, cacheCreation: 0 };
-    }
-    const cat = metrics._engine[category];
-    cat.calls++;
-    cat.costUsd += usage.costUsd || 0;
-    cat.inputTokens += usage.inputTokens || 0;
-    cat.outputTokens += usage.outputTokens || 0;
-    cat.cacheRead += usage.cacheRead || 0;
-    cat.cacheCreation = (cat.cacheCreation || 0) + (usage.cacheCreation || 0);
-
-    const today = new Date().toISOString().slice(0, 10);
-    if (!metrics._daily) metrics._daily = {};
-    if (!metrics._daily[today]) metrics._daily[today] = { costUsd: 0, inputTokens: 0, outputTokens: 0, cacheRead: 0, tasks: 0 };
-    const daily = metrics._daily[today];
-    daily.costUsd += usage.costUsd || 0;
-    daily.inputTokens += usage.inputTokens || 0;
-    daily.outputTokens += usage.outputTokens || 0;
-    daily.cacheRead += usage.cacheRead || 0;
-
-    safeWrite(metricsPath, metrics);
+    mutateJsonFileLocked(metricsPath, (metrics) => {
+      if (!metrics._engine) metrics._engine = {};
+      if (!metrics._engine[category]) {
+        metrics._engine[category] = { calls: 0, costUsd: 0, inputTokens: 0, outputTokens: 0, cacheRead: 0, cacheCreation: 0 };
+      }
+      const cat = metrics._engine[category];
+      cat.calls++;
+      cat.costUsd += usage.costUsd || 0;
+      cat.inputTokens += usage.inputTokens || 0;
+      cat.outputTokens += usage.outputTokens || 0;
+      cat.cacheRead += usage.cacheRead || 0;
+      cat.cacheCreation = (cat.cacheCreation || 0) + (usage.cacheCreation || 0);
+
+      const today = new Date().toISOString().slice(0, 10);
+      if (!metrics._daily) metrics._daily = {};
+      if (!metrics._daily[today]) metrics._daily[today] = { costUsd: 0, inputTokens: 0, outputTokens: 0, cacheRead: 0, tasks: 0 };
+      const daily = metrics._daily[today];
+      daily.costUsd += usage.costUsd || 0;
+      daily.inputTokens += usage.inputTokens || 0;
+      daily.outputTokens += usage.outputTokens || 0;
+      daily.cacheRead += usage.cacheRead || 0;
+
+      return metrics;
+    });
   } catch (e) { console.error('metrics update:', e.message); }
 }
 

package/package.json

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