@yemi33/minions 0.1.519 → 0.1.521

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.521 (2026-04-07)
+
+### Features
+-  test isolation — tests no longer pollute production data
+
+### Other
+- perf: reduce curl timeout to 5s, cache getDispatch() reads
+
 ## 0.1.519 (2026-04-07)
 
 ### Other

package/engine/ado.js

@@ -495,7 +495,7 @@ function checkLiveReviewStatus(pr, project) {
     const orgBase = shared.getAdoOrgBase(project);
     const prNum = (pr.id || '').replace(/^PR-/, '');
     const url = `${orgBase}/${project.adoProject}/_apis/git/repositories/${project.repositoryId}/pullrequests/${prNum}?api-version=7.1`;
-    const result = exec(`curl -s -H "Authorization: Bearer ${token}" "${url}"`, { encoding: 'utf-8', timeout: 15000, windowsHide: true });
+    const result = exec(`curl -s --max-time 4 -H "Authorization: Bearer ${token}" "${url}"`, { encoding: 'utf-8', timeout: 5000, windowsHide: true });
     const prData = JSON.parse(result);
     const votes = (prData.reviewers || []).map(r => r.vote).filter(v => v !== undefined);
     if (votes.length === 0) return 'pending';

package/engine/dispatch.js

@@ -24,12 +24,15 @@ function lifecycle() { if (!_lifecycle) _lifecycle = require('./lifecycle'); ret
 
 function mutateDispatch(mutator) {
   const defaultDispatch = { pending: [], active: [], completed: [] };
-  return mutateJsonFileLocked(DISPATCH_PATH, (dispatch) => {
+  const result = mutateJsonFileLocked(DISPATCH_PATH, (dispatch) => {
     dispatch.pending = Array.isArray(dispatch.pending) ? dispatch.pending : [];
     dispatch.active = Array.isArray(dispatch.active) ? dispatch.active : [];
     dispatch.completed = Array.isArray(dispatch.completed) ? dispatch.completed : [];
     return mutator(dispatch) || dispatch;
   }, { defaultValue: defaultDispatch });
+  // Invalidate the read cache so next getDispatch() sees fresh data
+  try { require('./queries').invalidateDispatchCache(); } catch {}
+  return result;
 }
 
 // ─── Add to Dispatch ─────────────────────────────────────────────────────────

package/engine/lifecycle.js

@@ -1012,6 +1012,7 @@ function createReviewFeedbackForAuthor(reviewerAgentId, pr, config) {
 }
 
 function updateMetrics(agentId, dispatchItem, result, taskUsage, prsCreatedCount, model) {
+  if (!agentId || agentId.startsWith('temp-') || agentId === 'agent1' || agentId === 'reviewer' || agentId.startsWith('_test')) return;
 
   const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
   mutateJsonFileLocked(metricsPath, metrics => {

package/engine/llm.js

@@ -7,11 +7,12 @@ const path = require('path');
 const shared = require('./shared');
 const { safeWrite, safeUnlink, uid, runFile, cleanChildEnv, parseStreamJsonOutput, mutateJsonFileLocked } = shared;
 
-const MINIONS_DIR = path.resolve(__dirname, '..');
-const ENGINE_DIR = __dirname;
+const MINIONS_DIR = shared.MINIONS_DIR;
+const ENGINE_DIR = path.join(MINIONS_DIR, 'engine');
 
 function trackEngineUsage(category, usage) {
   if (!usage) return;
+  if (category && (category.startsWith('_test') || category.startsWith('test-'))) return;
   try {
     const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
     mutateJsonFileLocked(metricsPath, (metrics) => {

package/engine/queries.js

@@ -79,9 +79,17 @@ function getControl() {
   return safeJson(CONTROL_PATH) || { state: 'stopped', pid: null };
 }
 
+let _dispatchCache = null;
+let _dispatchCacheAt = 0;
 function getDispatch() {
-  return safeJson(DISPATCH_PATH) || { pending: [], active: [], completed: [] };
+  // Short-lived cache — dispatch.json is read 10+ times per tick but only changes on mutateDispatch
+  const now = Date.now();
+  if (_dispatchCache && (now - _dispatchCacheAt) < 2000) return _dispatchCache;
+  _dispatchCache = safeJson(DISPATCH_PATH) || { pending: [], active: [], completed: [] };
+  _dispatchCacheAt = now;
+  return _dispatchCache;
 }
+function invalidateDispatchCache() { _dispatchCache = null; _dispatchCacheAt = 0; }
 
 function getDispatchQueue() {
   const d = getDispatch();
@@ -874,7 +882,7 @@ module.exports = {
   resetPrdInfoCache,
 
   // Core state
-  getConfig, getControl, getDispatch, getDispatchQueue,
+  getConfig, getControl, getDispatch, getDispatchQueue, invalidateDispatchCache,
   getNotes, getNotesWithMeta, getEngineLog, getMetrics,
 
   // Inbox

package/engine/shared.js

@@ -6,7 +6,7 @@
 const fs = require('fs');
 const path = require('path');
 
-const MINIONS_DIR = path.resolve(__dirname, '..');
+const MINIONS_DIR = process.env.MINIONS_TEST_DIR || path.resolve(__dirname, '..');
 const PR_LINKS_PATH = path.join(MINIONS_DIR, 'engine', 'pr-links.json');
 const LOG_PATH = path.join(__dirname, 'log.json');
 

package/package.json

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