@yemi33/minions 0.1.636 → 0.1.638

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.1.638 (2026-04-08)
+
+### Other
+- refactor: unify retry bypass pattern in discoverCentralWorkItems
+- perf: reduce CC/doc-chat tool eagerness and max-turns
+
 ## 0.1.636 (2026-04-08)
 
 ### Fixes

package/dashboard.js

@@ -538,10 +538,10 @@ Additional: pause-plan, approve-plan, reject-plan, archive-plan, edit-prd-item,
 Terms like schedules, pipelines, agents, inbox, work items, plans, PRD, PRs, dispatch, routing, KB, notes, pinned, meetings have Minions-specific meanings. Always resolve against Minions state first (read files or call APIs). Fall back to generic only if no Minions context exists.
 
 ## Rules
-1. Use tools proactively — read files before answering.
+1. Answer from the state preamble and context first. Only use tools for specific file lookups the user asked about — not to explore or investigate.
 2. Be specific — cite IDs, names, filenames, line numbers.
 3. Never modify engine source. Never push to git without user confirmation.
-4. Delegate exploration to agents. You are the dispatcher, not the worker.`;
+4. Delegate exploration to agents. You are the dispatcher, not the worker. If answering requires reading more than 2-3 files, dispatch an agent instead.`;
 
 // Hash the system prompt so we can detect changes and invalidate stale sessions
 const _ccPromptHash = require('crypto').createHash('md5').update(CC_STATIC_SYSTEM_PROMPT).digest('hex').slice(0, 8);
@@ -694,7 +694,7 @@ function updateSession(store, key, sessionId, existing) {
  * @param {number} opts.maxTurns - Max tool-use turns
  * @param {string} opts.allowedTools - Comma-separated tool list
  */
-async function ccCall(message, { store = 'cc', sessionKey, extraContext, label = 'command-center', timeout = 900000, maxTurns = 50, allowedTools = 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch', skipStatePreamble = false, model } = {}) {
+async function ccCall(message, { store = 'cc', sessionKey, extraContext, label = 'command-center', timeout = 900000, maxTurns = 25, allowedTools = 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch', skipStatePreamble = false, model } = {}) {
   if (!model) model = CONFIG.engine?.ccModel || shared.ENGINE_DEFAULTS.ccModel;
   const ccEffort = CONFIG.engine?.ccEffort || shared.ENGINE_DEFAULTS.ccEffort;
   const existing = resolveSession(store, sessionKey);
@@ -792,6 +792,9 @@ async function ccDocCall({ message, document, title, filePath, selection, canEdi
   const result = await ccCall(message, {
     store: 'doc', sessionKey,
     extraContext: docContext, label: 'doc-chat',
+    allowedTools: canEdit ? 'Read,Write,Edit,Glob,Grep' : 'Read,Glob,Grep',
+    maxTurns: canEdit ? 15 : 10,
+    skipStatePreamble: true,
     ...(model ? { model } : {}),
   });
   // Store doc hash for next call's unchanged check
@@ -3356,7 +3359,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         const streamModel = CONFIG.engine?.ccModel || shared.ENGINE_DEFAULTS.ccModel;
         const streamEffort = CONFIG.engine?.ccEffort || shared.ENGINE_DEFAULTS.ccEffort;
         const llmPromise = callLLMStreaming(prompt, CC_STATIC_SYSTEM_PROMPT, {
-          timeout: 900000, label: 'command-center', model: streamModel, maxTurns: 50,
+          timeout: 900000, label: 'command-center', model: streamModel, maxTurns: 25,
           allowedTools: 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch',
           sessionId, effort: streamEffort, direct: true,
           onChunk: (text) => {

package/engine.js

@@ -2142,23 +2142,25 @@ function discoverCentralWorkItems(config) {
     if (item.status !== WI_STATUS.QUEUED && item.status !== WI_STATUS.PENDING) continue;
 
     const key = `central-work-${item.id}`;
-    // Self-heal: if already dispatched but work item is still pending, fix the status
     // 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) {
-        const m = {};
-        if (item.status === WI_STATUS.PENDING) { m.status = WI_STATUS.DISPATCHED; }
-        if (!item.dispatched_to && existingActive.agent) { m.dispatched_to = existingActive.agent; }
-        if (Object.keys(m).length > 0) mutations.set(item.id, m);
+    const isRetry = !!item._retryCount;
+    if (isAlreadyDispatched(key)) {
+      if (isRetry) {
+        // Retry items bypass completed-dedup but still block if in-flight
+        const inFlight = [...(getDispatch().pending || []), ...(getDispatch().active || [])];
+        if (inFlight.some(d => d.meta?.dispatchKey === key)) continue;
+        // Not in-flight — fall through to dispatch
+      } else {
+        // Self-heal: set DISPATCHED only when in dispatch.active (agent spawned) (#480)
+        const existingActive = getDispatch().active?.find(d => d.meta?.dispatchKey === key);
+        if (existingActive) {
+          const m = {};
+          if (item.status === WI_STATUS.PENDING) { m.status = WI_STATUS.DISPATCHED; }
+          if (!item.dispatched_to && existingActive.agent) { m.dispatched_to = existingActive.agent; }
+          if (Object.keys(m).length > 0) mutations.set(item.id, m);
+        }
+        continue;
       }
-      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;
 

package/package.json

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