@yemi33/minions 0.1.634 → 0.1.636

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.636 (2026-04-08)
+
+### Fixes
+-  retry items bypass completed-dedup in isAlreadyDispatched
+
+### Other
+- perf: extend caches, skip doc re-send on session resume
+
 ## 0.1.634 (2026-04-08)
 
 ### Fixes

package/dashboard.js

@@ -548,7 +548,7 @@ const _ccPromptHash = require('crypto').createHash('md5').update(CC_STATIC_SYSTE
 
 let _preambleCache = null;
 let _preambleCacheTs = 0;
-const PREAMBLE_TTL = 10000; // 10s — same as status cache
+const PREAMBLE_TTL = 30000; // 30s — longer TTL since preamble is lightweight orientation, not real-time data
 
 function buildCCStatePreamble() {
   const now = Date.now();
@@ -676,6 +676,7 @@ function updateSession(store, key, sessionId, existing) {
       sessionId,
       lastActiveAt: now,
       turnCount: (existing && prev ? prev.turnCount : 0) + 1,
+      _docHash: prev?._docHash || null,
     });
     persistDocSessions();
   }
@@ -772,15 +773,32 @@ async function ccCall(message, { store = 'cc', sessionKey, extraContext, label =
 
 // Doc-specific wrapper — adds document context, parses ---DOCUMENT---
 async function ccDocCall({ message, document, title, filePath, selection, canEdit, isJson, model }) {
-  const docContext = `## Document Context\n**${title || 'Document'}**${filePath ? ' (`' + filePath + '`)' : ''}${isJson ? ' (JSON)' : ''}\n${selection ? '\n**Selected text:**\n> ' + selection.slice(0, 1500) + '\n' : ''}\n\`\`\`\n${document.slice(0, 20000)}\n\`\`\`\n${canEdit ? '\nIf editing: respond with your explanation, then `---DOCUMENT---` on its own line, then the COMPLETE updated file.' : '\n(Read-only — answer questions only.)'}`;
-
-  // Session key: filePath is stable and unique; title is the fallback for read-only Q&A
   const sessionKey = filePath || title;
+  const docSlice = document.slice(0, 20000);
+
+  // Skip re-sending full document on session resume if content unchanged
+  const docHash = require('crypto').createHash('md5').update(docSlice).digest('hex').slice(0, 8);
+  const existing = resolveSession('doc', sessionKey);
+  const docUnchanged = existing?.sessionId && existing._docHash === docHash;
+
+  let docContext;
+  if (docUnchanged) {
+    // Session has the document — only send selection and edit instructions
+    docContext = `## Document: ${title || 'Document'}${filePath ? ' (`' + filePath + '`)' : ''}${selection ? '\n**Selected text:**\n> ' + selection.slice(0, 1500) : ''}${canEdit ? '\nIf editing: respond with your explanation, then `---DOCUMENT---` on its own line, then the COMPLETE updated file.' : ''}`;
+  } else {
+    docContext = `## Document Context\n**${title || 'Document'}**${filePath ? ' (`' + filePath + '`)' : ''}${isJson ? ' (JSON)' : ''}\n${selection ? '\n**Selected text:**\n> ' + selection.slice(0, 1500) + '\n' : ''}\n\`\`\`\n${docSlice}\n\`\`\`\n${canEdit ? '\nIf editing: respond with your explanation, then `---DOCUMENT---` on its own line, then the COMPLETE updated file.' : '\n(Read-only — answer questions only.)'}`;
+  }
+
   const result = await ccCall(message, {
     store: 'doc', sessionKey,
     extraContext: docContext, label: 'doc-chat',
     ...(model ? { model } : {}),
   });
+  // Store doc hash for next call's unchanged check
+  if (result.code === 0 && result.sessionId) {
+    const session = resolveSession('doc', sessionKey);
+    if (session) session._docHash = docHash;
+  }
 
   if (result.code !== 0 || !result.text) {
     console.error(`[doc-chat] Failed: code=${result.code}, empty=${!result.text}, filePath=${filePath}, stderr=${(result.stderr || '').slice(0, 200)}`);

package/engine/llm.js

@@ -50,7 +50,7 @@ function trackEngineUsage(category, usage) {
 
 let _claudeBinCache = null;
 let _claudeBinCacheTs = 0;
-const _CLAUDE_BIN_TTL = 300000; // 5 min — re-validate binary exists periodically
+const _CLAUDE_BIN_TTL = 1800000; // 30 min — binary path rarely changes during a session
 function _resolveClaudeBin() {
   if (_claudeBinCache && Date.now() - _claudeBinCacheTs < _CLAUDE_BIN_TTL) return _claudeBinCache;
   _claudeBinCache = null;

package/engine.js

@@ -1696,15 +1696,28 @@ function discoverFromWorkItems(config, project) {
       delete item._resumedAt;
       needsWrite = true;
     }
+    // Skip dedup for items explicitly marked for retry (_retryCount set by engine)
+    const isRetry = !!item._retryCount;
     if (isAlreadyDispatched(key)) {
-      // Only self-heal to DISPATCHED if actually in dispatch.active (agent spawned) (#480)
-      const existingActive = getDispatch().active?.find(d => d.meta?.dispatchKey === key);
-      if (existingActive) {
-        if (item.status === WI_STATUS.PENDING) { item.status = WI_STATUS.DISPATCHED; needsWrite = true; }
-        if (!item.dispatched_to && existingActive.agent) { item.dispatched_to = existingActive.agent; needsWrite = true; }
+      // Retry items should bypass the completed-dedup but still block if in-flight
+      if (isRetry) {
+        const inFlight = [...(getDispatch().pending || []), ...(getDispatch().active || [])];
+        if (!inFlight.some(d => d.meta?.dispatchKey === key)) {
+          // Not in-flight — allow retry to proceed
+        } else {
+          if (item._pendingReason !== 'already_dispatched') { item._pendingReason = 'already_dispatched'; needsWrite = true; }
+          skipped.gated++; continue;
+        }
+      } else {
+        // Only self-heal to DISPATCHED if actually in dispatch.active (agent spawned) (#480)
+        const existingActive = getDispatch().active?.find(d => d.meta?.dispatchKey === key);
+        if (existingActive) {
+          if (item.status === WI_STATUS.PENDING) { item.status = WI_STATUS.DISPATCHED; needsWrite = true; }
+          if (!item.dispatched_to && existingActive.agent) { item.dispatched_to = existingActive.agent; needsWrite = true; }
+        }
+        if (item._pendingReason !== 'already_dispatched') { item._pendingReason = 'already_dispatched'; needsWrite = true; }
+        skipped.gated++; continue;
       }
-      if (item._pendingReason !== 'already_dispatched') { item._pendingReason = 'already_dispatched'; needsWrite = true; }
-      skipped.gated++; continue;
     }
     if (isOnCooldown(key, cooldownMs)) {
       if (item._pendingReason !== 'cooldown') { item._pendingReason = 'cooldown'; needsWrite = true; }
@@ -2130,7 +2143,8 @@ function discoverCentralWorkItems(config) {
 
     const key = `central-work-${item.id}`;
     // Self-heal: if already dispatched but work item is still pending, fix the status
-    if (isAlreadyDispatched(key)) {
+    // Skip dedup for items explicitly marked for retry (_retryCount set by engine)
+    if (!item._retryCount && isAlreadyDispatched(key)) {
       // Only self-heal to DISPATCHED if actually in dispatch.active (agent spawned) (#480)
       const existingActive = getDispatch().active?.find(d => d.meta?.dispatchKey === key);
       if (existingActive) {
@@ -2141,6 +2155,11 @@ function discoverCentralWorkItems(config) {
       }
       continue;
     }
+    // Still block if actively in flight (pending or active dispatch)
+    if (item._retryCount && isAlreadyDispatched(key)) {
+      const inFlight = [...(getDispatch().pending || []), ...(getDispatch().active || [])];
+      if (inFlight.some(d => d.meta?.dispatchKey === key)) continue;
+    }
     if (isOnCooldown(key, 0)) continue;
 
     const workType = item.type || 'implement';

package/package.json

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