@yemi33/minions 0.1.1032 → 0.1.1034

package/CHANGELOG.md

@@ -1,5 +1,82 @@
 # Changelog
 
+## 0.1.1034 (2026-04-16)
+
+### Features
+-  Doc-chat performance — debounced persistence, session TTL, smart disk reads (#1164)
+- split getStatus() into fast/slow state tiers with separate TTLs (#1170)
+-  Cache _countWorktrees() with 30s TTL (#1166)
+- Pre-cache gzipped status buffer alongside JSON cache (#1171)
+- parallelize ADO and GitHub PR polling with Promise.allSettled (#1172)
+- route implement items to dedicated implement playbook (#1115)
+
+### Fixes
+-  use locked write for failed-with-PR reconciliation in cleanup.js (#1169)
+-  prevent test from corrupting live meeting state
+- scheduler enabled field falsy check treats undefined as disabled (#1160)
+-  update branch-dedup tests to use canonical PR IDs after merge with master (#1146)
+-  auto-review not firing for manually-linked PRs with autoObserve=true
+-  mark PR abandoned on 404 instead of silently retrying each tick
+- replace undefined PROJECTS with config.projects in checkWatches (#1108)
+- write permission for publish workflow
+-  run tests inline and post check runs for publish PRs
+- add maxBuffer to all GitHub CLI spawns + paginate PR list (#1130)
+-  add required CI checks for PRs + update publish for auto-merge
+-  revert to PR merge now that stale status check is removed
+- guard live review check against undefined vote/state values (#1132)
+-  push version bump directly to master instead of via PR
+-  add push-triggered CI for chore/publish branches
+-  publish workflow chore PRs failing to merge
+-  harden KB ordering
+-  harden audited state transitions
+
+### Other
+- [E2E] Dashboard & engine perf: gzip cache, tiered status, lock backoff, polling parallelization (#1174)
+- Fix doc chat session isolation
+- test(pipeline): add unit tests for CRUD, stage execution, run lifecycle (#1162)
+- chore: test publish after removing stale status check
+- chore: trigger publish test
+- chore: test publish workflow fix
+- chore: trigger publish workflow test
+
+## 0.1.1033 (2026-04-16)
+
+### Features
+- split getStatus() into fast/slow state tiers with separate TTLs (#1170)
+-  Cache _countWorktrees() with 30s TTL (#1166)
+- Pre-cache gzipped status buffer alongside JSON cache (#1171)
+- parallelize ADO and GitHub PR polling with Promise.allSettled (#1172)
+- route implement items to dedicated implement playbook (#1115)
+
+### Fixes
+-  use locked write for failed-with-PR reconciliation in cleanup.js (#1169)
+-  prevent test from corrupting live meeting state
+- scheduler enabled field falsy check treats undefined as disabled (#1160)
+-  update branch-dedup tests to use canonical PR IDs after merge with master (#1146)
+-  auto-review not firing for manually-linked PRs with autoObserve=true
+-  mark PR abandoned on 404 instead of silently retrying each tick
+- replace undefined PROJECTS with config.projects in checkWatches (#1108)
+- write permission for publish workflow
+-  run tests inline and post check runs for publish PRs
+- add maxBuffer to all GitHub CLI spawns + paginate PR list (#1130)
+-  add required CI checks for PRs + update publish for auto-merge
+-  revert to PR merge now that stale status check is removed
+- guard live review check against undefined vote/state values (#1132)
+-  push version bump directly to master instead of via PR
+-  add push-triggered CI for chore/publish branches
+-  publish workflow chore PRs failing to merge
+-  harden KB ordering
+-  harden audited state transitions
+
+### Other
+- [E2E] Dashboard & engine perf: gzip cache, tiered status, lock backoff, polling parallelization (#1174)
+- Fix doc chat session isolation
+- test(pipeline): add unit tests for CRUD, stage execution, run lifecycle (#1162)
+- chore: test publish after removing stale status check
+- chore: trigger publish test
+- chore: test publish workflow fix
+- chore: trigger publish workflow test
+
 ## 0.1.1032 (2026-04-16)
 
 ### Features

package/dashboard/js/modal-qa.js

@@ -428,6 +428,7 @@ async function _processQaMessage(message, selection, opts) {
         selection: selection,
         filePath: capturedFilePath || null,
         model: window._lastStatus?.autoMode?.ccModel || undefined,
+        contentHash: capturedDocContext.content ? capturedDocContext.content.length + ':' + capturedDocContext.content.charCodeAt(0) + ':' + capturedDocContext.content.charCodeAt(capturedDocContext.content.length - 1) : undefined,
       }),
     });
     const data = await res.json();

package/dashboard.js

@@ -852,16 +852,18 @@ async function executeCCActions(actions) {
 // Session store for doc modals — keyed by filePath or title, persisted to disk
 const CC_SESSIONS_PATH = path.join(ENGINE_DIR, 'cc-sessions.json');
 const DOC_SESSIONS_PATH = path.join(ENGINE_DIR, 'doc-sessions.json');
+const DOC_SESSION_TTL_MS = 7 * 24 * 60 * 60 * 1000; // 604800000 — 7 days
 const docSessions = new Map(); // key → { sessionId, lastActiveAt, turnCount }
 
 // Load persisted doc sessions on startup
 try {
   const saved = safeJson(DOC_SESSIONS_PATH);
   if (saved && typeof saved === 'object') {
+    const now = Date.now();
     for (const [key, s] of Object.entries(saved)) {
-      if (s.turnCount < CC_SESSION_MAX_TURNS) {
-        docSessions.set(key, s);
-      }
+      if (s.turnCount >= CC_SESSION_MAX_TURNS) continue;
+      if (s.lastActiveAt && now - new Date(s.lastActiveAt).getTime() > DOC_SESSION_TTL_MS) continue;
+      docSessions.set(key, s);
     }
   }
 } catch { /* optional */ }
@@ -872,6 +874,25 @@ function persistDocSessions() {
   safeWrite(DOC_SESSIONS_PATH, obj);
 }
 
+// Debounced variant — coalesces rapid writes (e.g. back-to-back doc-chat turns)
+let _persistDocSessionsTimer = null;
+function schedulePersistDocSessions() {
+  if (_persistDocSessionsTimer) clearTimeout(_persistDocSessionsTimer);
+  _persistDocSessionsTimer = setTimeout(() => {
+    _persistDocSessionsTimer = null;
+    persistDocSessions();
+  }, 5000); // 5s debounce — rapid turns produce one write per burst
+}
+
+/** Flush any pending debounced write immediately (call on shutdown). */
+function flushPendingDocSessions() {
+  if (_persistDocSessionsTimer) {
+    clearTimeout(_persistDocSessionsTimer);
+    _persistDocSessionsTimer = null;
+    persistDocSessions();
+  }
+}
+
 // Resolve session from any store (CC global or doc-specific)
 function resolveSession(store, key) {
   if (store === 'cc') {
@@ -885,6 +906,11 @@ function resolveSession(store, key) {
     persistDocSessions();
     return null;
   }
+  if (s.lastActiveAt && Date.now() - new Date(s.lastActiveAt).getTime() > DOC_SESSION_TTL_MS) {
+    docSessions.delete(key);
+    persistDocSessions();
+    return null;
+  }
   return s;
 }
 
@@ -909,7 +935,7 @@ function updateSession(store, key, sessionId, existing) {
       turnCount: (existing && prev ? prev.turnCount : 0) + 1,
       _docHash: prev?._docHash || null,
     });
-    persistDocSessions();
+    schedulePersistDocSessions();
   }
 }
 
@@ -972,7 +998,7 @@ async function ccCall(message, { store = 'cc', sessionKey, extraContext, label =
       safeWrite(path.join(ENGINE_DIR, 'cc-session.json'), ccSession);
     } else if (sessionKey) {
       docSessions.delete(sessionKey);
-      persistDocSessions();
+      schedulePersistDocSessions();
     }
   }
 
@@ -1007,6 +1033,12 @@ async function ccCall(message, { store = 'cc', sessionKey, extraContext, label =
   return result;
 }
 
+// Lightweight content fingerprint — same algorithm used browser-side (no crypto needed)
+function contentFingerprint(str) {
+  if (!str) return '';
+  return str.length + ':' + str.charCodeAt(0) + ':' + str.charCodeAt(str.length - 1);
+}
+
 // Doc-specific wrapper — adds document context, parses ---DOCUMENT---
 async function ccDocCall({ message, document, title, filePath, selection, canEdit, isJson, model, freshSession, onAbortReady }) {
   const sessionKey = filePath || title;
@@ -1047,7 +1079,7 @@ async function ccDocCall({ message, document, title, filePath, selection, canEdi
     // One-shot call — discard the session ccCall just stored so it cannot
     // bleed into future interactions under the same key.
     docSessions.delete(sessionKey);
-    persistDocSessions();
+    schedulePersistDocSessions();
   } else if (result.code === 0 && result.sessionId) {
     // Store doc hash for next call's unchanged check
     const session = resolveSession('doc', sessionKey);
@@ -3221,7 +3253,14 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         try { shared.sanitizePath(body.filePath, MINIONS_DIR); } catch { return jsonReply(res, 400, { error: 'path must be under minions directory' }); }
         fullPath = path.resolve(MINIONS_DIR, body.filePath);
         const diskContent = safeRead(fullPath);
-        if (diskContent !== null) currentContent = diskContent;
+        if (diskContent !== null) {
+          // If client sent a contentHash and it matches disk, skip replacement — client copy is fresh
+          if (body.contentHash && contentFingerprint(diskContent) === body.contentHash) {
+            // body.document is already current — no override needed
+          } else {
+            currentContent = diskContent;
+          }
+        }
       }
 
       const { answer, content, actions } = await ccDocCall({
@@ -4741,7 +4780,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     { method: 'GET', path: /^\/api\/knowledge\/([^/]+)\/([^?]+)/, desc: 'Read a specific knowledge base entry', handler: handleKnowledgeRead },
 
     // Doc chat
-    { method: 'POST', path: '/api/doc-chat', desc: 'Minions-aware doc Q&A + editing via CC session', params: 'message, document, title?, filePath?, selection?', handler: handleDocChat },
+    { method: 'POST', path: '/api/doc-chat', desc: 'Minions-aware doc Q&A + editing via CC session', params: 'message, document, title?, filePath?, selection?, contentHash?', handler: handleDocChat },
 
     // Inbox
     { method: 'POST', path: '/api/inbox/persist', desc: 'Promote an inbox item to team notes', params: 'name', handler: handleInboxPersist },
@@ -5102,3 +5141,8 @@ server.on('error', e => {
   }
   process.exit(1);
 });
+
+// ── Graceful shutdown: flush debounced writes ──────────────────────────────
+server.on('close', () => flushPendingDocSessions());
+process.on('SIGTERM', () => { flushPendingDocSessions(); process.exit(0); });
+process.on('SIGINT', () => { flushPendingDocSessions(); process.exit(0); });

package/package.json

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