@yemi33/minions 0.1.1802 → 0.1.1803

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.1803 (2026-05-08)
+
+### Other
+- Guarantee Copilot steering delivery (#2225)
+
 ## 0.1.1800 (2026-05-08)
 
 ### Other

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-08T17:22:37.672Z"
+  "cachedAt": "2026-05-08T17:37:37.501Z"
 }

package/engine.js

@@ -407,6 +407,50 @@ function mergePendingSteeringEntries(...groups) {
   return merged;
 }
 
+function promoteCheckpointSteeringForClose(agentId, procInfo, runtime, liveOutputPath) {
+  if (!procInfo || procInfo._steeringMessage) return { status: 'none', entries: [] };
+  if (runtime?.capabilities?.midRunSessionId !== false) return { status: 'none', entries: [] };
+
+  const startedAtMs = Date.parse(procInfo.startedAt);
+  const runStartMs = Number.isFinite(startedAtMs) ? startedAtMs : 0;
+  const pendingPaths = new Set((procInfo._pendingSteeringFiles || []).map(entry => entry?.path || entry).filter(Boolean));
+  const deferredPaths = new Set((procInfo._deferredSteeringFiles || []).filter(Boolean));
+  const unread = steering.listUnreadSteeringMessages(agentId).filter(entry => entry.message.trim());
+  const entriesByPath = new Map(unread.map(entry => [entry.path, entry]));
+  const pendingDeferred = Array.from(deferredPaths)
+    .map(filePath => entriesByPath.get(filePath))
+    .filter(Boolean);
+  const lateCheckpoint = unread.filter(entry =>
+    entry.createdAtMs >= runStartMs
+    && !pendingPaths.has(entry.path)
+    && !deferredPaths.has(entry.path)
+  );
+  const checkpointEntries = mergePendingSteeringEntries(pendingDeferred, lateCheckpoint);
+  if (checkpointEntries.length === 0) {
+    delete procInfo._deferredSteeringFiles;
+    return { status: 'none', entries: [] };
+  }
+
+  if (!procInfo.sessionId) {
+    log('warn', `Steering: ${agentId} exited before a resumable sessionId was available - ${checkpointEntries.length} message(s) remain pending`);
+    try { fs.appendFileSync(liveOutputPath, `\n[steering-pending] Agent exited before a resumable session was available. Your message remains unread and will be retried on the next dispatch.\n`); } catch {}
+    return { status: 'pending', entries: checkpointEntries };
+  }
+
+  if (pendingDeferred.length > 0) {
+    log('info', `Steering: delivering ${pendingDeferred.length} deferred message(s) for ${agentId} at resumable checkpoint`);
+  }
+  if (lateCheckpoint.length > 0) {
+    log('info', `Steering: delivering ${lateCheckpoint.length} late checkpoint message(s) for ${agentId} at resumable checkpoint`);
+  }
+  procInfo._steeringMessage = checkpointEntries.map(entry => entry.message.trim()).join('\n\n');
+  procInfo._steeringSessionId = procInfo.sessionId;
+  procInfo._steeringEntry = checkpointEntries;
+  procInfo._steeringDeferredCheckpoint = true;
+  delete procInfo._deferredSteeringFiles;
+  return { status: 'promoted', entries: checkpointEntries };
+}
+
 // Resolve dependency plan item IDs to their PR branches
 function resolveDependencyBranches(depIds, sourcePlan, project, config) {
   const results = []; // [{ branch, prId }]
@@ -1193,6 +1237,7 @@ async function spawnAgent(dispatchItem, config) {
   const MAX_OUTPUT = 1024 * 1024; // 1MB
   let stdout = '';
   let stderr = '';
+  let steeringAckStdout = '';
   const sessionCaptureState = { sessionLineBuffer: '' };
   let _trustCheckDone = false;
   const _spawnTime = Date.now();
@@ -1207,6 +1252,7 @@ async function spawnAgent(dispatchItem, config) {
     const chunk = data.toString();
     realActivityMap.set(id, Date.now());
     if (stdout.length < MAX_OUTPUT) stdout += chunk.slice(0, MAX_OUTPUT - stdout.length);
+    if (steeringAckStdout.length < MAX_OUTPUT) steeringAckStdout += chunk.slice(0, MAX_OUTPUT - steeringAckStdout.length);
     try { fs.appendFileSync(liveOutputPath, chunk); } catch { /* optional */ }
 
     // Trust gate detection: check first 30s of output for trust/permission prompts
@@ -1254,26 +1300,8 @@ async function spawnAgent(dispatchItem, config) {
     }
 
     const procInfo = activeProcesses.get(id);
-    ackPendingSteeringFiles(agentId, procInfo, stdout);
-
-    if (procInfo?._deferredSteeringFiles?.length && procInfo.sessionId) {
-      const deferredPaths = new Set(procInfo._deferredSteeringFiles);
-      const pendingDeferred = steering.listUnreadSteeringMessages(agentId)
-        .filter(entry => deferredPaths.has(entry.path) && entry.message.trim());
-      if (pendingDeferred.length > 0) {
-        log('info', `Steering: delivering ${pendingDeferred.length} deferred message(s) for ${agentId} at resumable checkpoint`);
-        procInfo._steeringMessage = pendingDeferred.map(entry => entry.message.trim()).join('\n\n');
-        procInfo._steeringSessionId = procInfo.sessionId;
-        procInfo._steeringEntry = pendingDeferred;
-        procInfo._steeringDeferredCheckpoint = true;
-        delete procInfo._deferredSteeringFiles;
-      } else {
-        delete procInfo._deferredSteeringFiles;
-      }
-    } else if (procInfo?._deferredSteeringFiles?.length) {
-      log('warn', `Steering: ${agentId} exited before a resumable sessionId was available — message remains pending`);
-      try { fs.appendFileSync(liveOutputPath, `\n[steering-pending] Agent exited before a resumable session was available. Your message remains unread and will be retried on the next dispatch.\n`); } catch {}
-    }
+    ackPendingSteeringFiles(agentId, procInfo, steeringAckStdout);
+    promoteCheckpointSteeringForClose(agentId, procInfo, runtime, liveOutputPath);
 
     // Check if this was a steering kill — re-spawn with resume
     if (procInfo?._steeringMessage) {
@@ -1401,6 +1429,7 @@ async function spawnAgent(dispatchItem, config) {
         ),
       });
 
+      steeringAckStdout = '';
       // Live steering kills discard partial old output. Deferred checkpoint
       // steering keeps the completed turn output so completion parsing still
       // sees the original work if the follow-up only acknowledges steering.
@@ -1414,6 +1443,7 @@ async function spawnAgent(dispatchItem, config) {
         const chunk = data.toString();
         realActivityMap.set(id, Date.now());
         if (stdout.length < MAX_OUTPUT) stdout += chunk.slice(0, MAX_OUTPUT - stdout.length);
+        if (steeringAckStdout.length < MAX_OUTPUT) steeringAckStdout += chunk.slice(0, MAX_OUTPUT - steeringAckStdout.length);
         try { fs.appendFileSync(liveOutputPath, chunk); } catch { /* optional */ }
         const resumeInfo = activeProcesses.get(id);
         markRuntimeResumeOutputSeen(resumeInfo);
@@ -4719,6 +4749,7 @@ module.exports = {
   parseConflictFiles, pruneAncestorDeps, preflightMergeSimulation, // exported for testing
   isWorktreeRetryableError, removeStaleIndexLock, syncReusedWorktree, // exported for testing
   _maxTurnsForType, buildProjectContext, normalizeAc, _buildAgentSpawnFlags, _classifyAgentFailure, // exported for testing
+  promoteCheckpointSteeringForClose, // exported for testing
   normalizePrBranch, resolvePrBranch, prCausePart, getPrCauseHead, getPrCauseBase, getPrAutomationCauseKey, getPrAutomationDispatchKey, // exported for testing
 
   // Playbooks

package/package.json

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