@yemi33/minions 0.1.478 → 0.1.479

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.478 (2026-04-07)
+## 0.1.479 (2026-04-07)
 
 ### Features
+- Buffer log() writes to reduce lock contention
 - Cache getStatus() JSON serialization and add mtime-based invalidation
 - Add mtime-based caching to getPrdInfo()
 - Optimize getAgentStatus() to read only head+tail of live-output.log

package/engine/cli.js

@@ -391,6 +391,7 @@ const commands = {
       if (e.activeProcesses.size === 0) {
         safeWrite(CONTROL_PATH, { state: 'stopped', stopped_at: e.ts() });
         e.log('info', 'Graceful shutdown complete (no active agents)');
+        shared.flushLogs(); // drain buffered log entries before exit
         console.log('No active agents — stopped.');
         process.exit(0);
       }
@@ -404,6 +405,7 @@ const commands = {
           clearInterval(poll);
           safeWrite(CONTROL_PATH, { state: 'stopped', stopped_at: e.ts() });
           e.log('info', 'Graceful shutdown complete (all agents finished)');
+          shared.flushLogs(); // drain buffered log entries before exit
           console.log('All agents finished — stopped.');
           process.exit(0);
         }
@@ -411,6 +413,7 @@ const commands = {
           clearInterval(poll);
           safeWrite(CONTROL_PATH, { state: 'stopped', stopped_at: e.ts() });
           e.log('warn', `Graceful shutdown timed out after ${timeout / 1000}s with ${e.activeProcesses.size} agent(s) still active`);
+          shared.flushLogs(); // drain buffered log entries before exit
           console.log(`Shutdown timeout (${timeout / 1000}s) — force exiting with ${e.activeProcesses.size} agent(s) still running.`);
           process.exit(1);
         }

package/engine/shared.js

@@ -18,20 +18,56 @@ function ts() { return new Date().toISOString(); }
 function logTs() { return new Date().toLocaleTimeString(); }
 function dateStamp() { return new Date().toISOString().slice(0, 10); }
 
+// ── Log Buffering ──────────────────────────────────────────────────────────
+// Buffer log entries in memory and flush to disk periodically to reduce lock
+// contention (~139 calls/tick → 1 lock acquisition per flush).
+const _logBuffer = [];
+let _logFlushTimer = null;
+
 function log(level, msg, meta = {}) {
   const entry = { timestamp: ts(), level, message: msg, ...meta };
+  // Console output remains immediate
   console.log(`[${logTs()}] [${level}] ${msg}`);
 
+  _logBuffer.push(entry);
+
+  // Start the flush timer lazily on first buffered entry
+  if (!_logFlushTimer) {
+    _logFlushTimer = setInterval(() => {
+      _flushLogBuffer();
+    }, ENGINE_DEFAULTS.logFlushInterval);
+    // Unref so the timer doesn't keep the process alive during shutdown
+    if (_logFlushTimer.unref) _logFlushTimer.unref();
+  }
+
+  // Flush immediately when buffer exceeds threshold
+  if (_logBuffer.length >= ENGINE_DEFAULTS.logBufferSize) {
+    _flushLogBuffer();
+  }
+}
+
+function _flushLogBuffer() {
+  if (_logBuffer.length === 0) return;
+  const entries = _logBuffer.splice(0);
   try {
     mutateJsonFileLocked(LOG_PATH, (logData) => {
       if (!Array.isArray(logData)) logData = logData?.entries || [];
-      logData.push(entry);
+      logData.push(...entries);
       if (logData.length >= 2500) logData.splice(0, logData.length - 2000);
       return logData;
     }, { defaultValue: [] });
   } catch { /* logging should never crash the caller */ }
 }
 
+/** Flush buffered log entries to disk. Call during graceful shutdown to drain the buffer. */
+function flushLogs() {
+  _flushLogBuffer();
+  if (_logFlushTimer) {
+    clearInterval(_logFlushTimer);
+    _logFlushTimer = null;
+  }
+}
+
 // ── File I/O ─────────────────────────────────────────────────────────────────
 
 function safeRead(p) {
@@ -441,6 +477,8 @@ const ENGINE_DEFAULTS = {
   pipelineApiRetries: 2, // max attempts for pipeline API calls
   pipelineApiRetryDelay: 2000, // ms delay between pipeline API retries
   versionCheckInterval: 3600000, // 1 hour — how often to check npm for updates (ms)
+  logFlushInterval: 5000, // 5s — how often to flush buffered log entries to disk
+  logBufferSize: 50, // flush immediately when buffer exceeds this many entries
 };
 
 // ─── Status & Type Constants ─────────────────────────────────────────────────
@@ -729,5 +767,7 @@ module.exports = {
   killGracefully,
   killImmediate,
   LOCK_STALE_MS,
+  flushLogs,
+  _logBuffer, // exported for testing
 };
 

package/package.json

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