@yemi33/minions 0.1.475 → 0.1.477

package/CHANGELOG.md

@@ -1,8 +1,10 @@
 # Changelog
 
-## 0.1.475 (2026-04-07)
+## 0.1.477 (2026-04-07)
 
 ### Features
+- Add mtime-based caching to getPrdInfo()
+- Optimize getAgentStatus() to read only head+tail of live-output.log
 - 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

package/engine/queries.js

@@ -13,6 +13,35 @@ const { safeRead, safeReadDir, safeJson, safeWrite, getProjects,
   projectWorkItemsPath, projectPrPath, parseSkillFrontmatter, KB_CATEGORIES,
   WI_STATUS } = shared;
 
+/**
+ * Read the first `bytes` and last `bytes` of a file efficiently using byte offsets.
+ * For files <= 2*bytes, reads the whole file. Returns { head, tail } strings.
+ * Returns { head: '', tail: '' } on any error.
+ */
+function readHeadTail(filePath, bytes = 1024) {
+  try {
+    const stat = fs.statSync(filePath);
+    const size = stat.size;
+    if (size === 0) return { head: '', tail: '' };
+    if (size <= bytes * 2) {
+      const full = fs.readFileSync(filePath, 'utf8');
+      return { head: full, tail: full };
+    }
+    const fd = fs.openSync(filePath, 'r');
+    try {
+      const headBuf = Buffer.alloc(bytes);
+      fs.readSync(fd, headBuf, 0, bytes, 0);
+      const tailBuf = Buffer.alloc(bytes);
+      fs.readSync(fd, tailBuf, 0, bytes, size - bytes);
+      return { head: headBuf.toString('utf8'), tail: tailBuf.toString('utf8') };
+    } finally {
+      fs.closeSync(fd);
+    }
+  } catch {
+    return { head: '', tail: '' };
+  }
+}
+
 // ── Paths ───────────────────────────────────────────────────────────────────
 
 const MINIONS_DIR = shared.MINIONS_DIR;
@@ -130,19 +159,20 @@ function getAgentStatus(agentId) {
       branch: active.meta?.branch || '',
       started_at: active.started_at || active.created_at || null,
     };
-    // Detect permission-waiting: check live output for non-bypass permission mode
+    // Detect permission-waiting: read only head+tail of live-output.log (max 2KB total)
     try {
-      const liveLog = shared.safeRead(path.join(AGENTS_DIR, agentId, 'live-output.log'));
-      if (liveLog) {
-        // Check init message for permission mode
-        const initMatch = liveLog.match(/"permissionMode"\s*:\s*"([^"]+)"/);
+      const liveLogPath = path.join(AGENTS_DIR, agentId, 'live-output.log');
+      const { head, tail } = readHeadTail(liveLogPath, 1024);
+      if (head) {
+        // Check init message (in head) for permission mode
+        const initMatch = head.match(/"permissionMode"\s*:\s*"([^"]+)"/);
         if (initMatch && initMatch[1] !== 'bypassPermissions') {
           result._permissionMode = initMatch[1];
         }
-        // Check if agent has been silent for >60s (possible permission prompt wait)
-        const lastLine = liveLog.trimEnd().split('\n').pop();
+        // Check if agent has been silent for >60s (use tail for recent activity)
+        const lastLine = tail.trimEnd().split('\n').pop();
         if (lastLine && lastLine.includes('"type":"assistant"') && lastLine.includes('"tool_use"')) {
-          const liveStat = fs.statSync(path.join(AGENTS_DIR, agentId, 'live-output.log'));
+          const liveStat = fs.statSync(liveLogPath);
           const silentMs = Date.now() - liveStat.mtimeMs;
           if (silentMs > 60000 && result._permissionMode) {
             result._warning = 'Possibly waiting for permission approval — agent is not in bypass mode';
@@ -597,12 +627,51 @@ function getWorkItems(config) {
 
 // ── PRD Progress ────────────────────────────────────────────────────────────
 
+// Module-level caches for getPrdInfo() — avoids re-reading unchanged PRD files
+const _prdFileCache = new Map();       // filePath → { mtimeMs, plan }
+let _prdDirMtimes = { prd: 0, archive: 0 }; // directory mtimes to detect new/deleted files
+let _prdResultCache = null;            // cached final result
+let _prdResultInputHash = '';          // hash of all input mtimes to detect any change
+
+/**
+ * Collect mtimes of all input files that affect getPrdInfo() output.
+ * Returns a string hash for quick equality check plus the dir mtimes.
+ */
+function _getPrdInputHash(projects) {
+  const mtimes = [];
+  // PRD directory mtimes (detect new/deleted files)
+  let prdDirMtime = 0, archiveDirMtime = 0;
+  try { prdDirMtime = fs.statSync(PRD_DIR).mtimeMs; } catch { /* optional */ }
+  const archiveDir = path.join(PRD_DIR, 'archive');
+  try { archiveDirMtime = fs.statSync(archiveDir).mtimeMs; } catch { /* optional */ }
+  mtimes.push(prdDirMtime, archiveDirMtime);
+  // Work-items file mtimes (affect status display)
+  for (const project of projects) {
+    try { mtimes.push(fs.statSync(projectWorkItemsPath(project)).mtimeMs); } catch { mtimes.push(0); }
+  }
+  try { mtimes.push(fs.statSync(path.join(MINIONS_DIR, 'work-items.json')).mtimeMs); } catch { mtimes.push(0); }
+  // PR file mtimes (affect PR links)
+  for (const project of projects) {
+    try { mtimes.push(fs.statSync(projectPrPath(project)).mtimeMs); } catch { mtimes.push(0); }
+  }
+  return { hash: mtimes.join(','), prdDirMtime, archiveDirMtime };
+}
+
 function getPrdInfo(config) {
   config = config || getConfig();
   const projects = getProjects(config);
+
+  // Quick mtime check — return cached result if nothing changed
+  const { hash, prdDirMtime, archiveDirMtime } = _getPrdInputHash(projects);
+  if (_prdResultCache && hash === _prdResultInputHash) return _prdResultCache;
+
   let allPrdItems = [];
   let latestStat = null;
 
+  // Check if directory listings need refresh
+  const dirsChanged = prdDirMtime !== _prdDirMtimes.prd || archiveDirMtime !== _prdDirMtimes.archive;
+  _prdDirMtimes = { prd: prdDirMtime, archive: archiveDirMtime };
+
   // Scan active PRDs and archived PRDs (completed PRDs still need to show progress)
   const planDirs = [
     { dir: PRD_DIR, archived: false },
@@ -613,10 +682,21 @@ function getPrdInfo(config) {
       const planFiles = fs.readdirSync(dir).filter(f => f.endsWith('.json'));
       for (const pf of planFiles) {
         try {
-          const plan = safeJson(path.join(dir, pf));
-          if (!plan || !plan.missing_features) continue;
-          const stat = fs.statSync(path.join(dir, pf));
+          const filePath = path.join(dir, pf);
+          const stat = fs.statSync(filePath);
           if (!latestStat || stat.mtimeMs > latestStat.mtimeMs) latestStat = stat;
+
+          // Per-file mtime cache: only re-read files that changed
+          const cached = _prdFileCache.get(filePath);
+          let plan;
+          if (cached && cached.mtimeMs === stat.mtimeMs) {
+            plan = cached.plan;
+          } else {
+            plan = safeJson(filePath);
+            _prdFileCache.set(filePath, { mtimeMs: stat.mtimeMs, plan });
+          }
+          if (!plan || !plan.missing_features) continue;
+
           // Staleness: compare source plan mtime to recorded sourcePlanModifiedAt
           let planStale = false;
           if (!archived && plan.source_plan) {
@@ -638,6 +718,12 @@ function getPrdInfo(config) {
           }
         } catch { /* optional */ }
       }
+      // Clean stale entries from file cache when dirs changed
+      if (dirsChanged) {
+        for (const cachedPath of _prdFileCache.keys()) {
+          if (cachedPath.startsWith(dir) && !fs.existsSync(cachedPath)) _prdFileCache.delete(cachedPath);
+        }
+      }
     } catch { /* optional */ }
   }
 
@@ -736,7 +822,18 @@ function getPrdInfo(config) {
     missingList: items.filter(i => i.status === 'missing').map(f => ({ id: f.id, name: f.name || f.title, priority: f.priority, complexity: f.estimated_complexity || f.size })),
   };
 
-  return { progress, status };
+  const result = { progress, status };
+  _prdResultCache = result;
+  _prdResultInputHash = hash;
+  return result;
+}
+
+/** Reset PRD info cache — exported for testing */
+function resetPrdInfoCache() {
+  _prdFileCache.clear();
+  _prdDirMtimes = { prd: 0, archive: 0 };
+  _prdResultCache = null;
+  _prdResultInputHash = '';
 }
 
 // ── Exports ─────────────────────────────────────────────────────────────────
@@ -748,6 +845,8 @@ module.exports = {
 
   // Helpers
   timeSince,
+  readHeadTail, // exported for testing
+  resetPrdInfoCache,
 
   // Core state
   getConfig, getControl, getDispatch, getDispatchQueue,

package/package.json

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