@yemi33/minions 0.1.1598 → 0.1.1599

package/CHANGELOG.md

@@ -1,12 +1,13 @@
 # Changelog
 
-## 0.1.1598 (2026-04-28)
+## 0.1.1599 (2026-04-28)
 
 ### Features
 - match runtime tags to actual logos (pixel-crab Claude, mascot Copilot)
 - replace runtime text tag with inline SVG logos
 
 ### Fixes
+-  keep cc stream final text complete
 - switch Copilot icon to outline style for cleaner inline read
 - un-invert Copilot — purple silhouette + white cutouts
 - redraw Copilot icon to match actual mascot — vertical eye pills, not grill bars

package/engine/llm.js

@@ -270,11 +270,20 @@ function _createStreamAccumulator({
   const toolUses = [];
 
   // Copilot streams `assistant.message_delta` with `data.deltaContent` chunks
-  // before emitting the final `assistant.message`. We accumulate the deltas
-  // for the live onChunk feed; the final `assistant.message.data.content`
-  // value is the authoritative text.
+  // before emitting `assistant.message`. Tool-request messages can include
+  // narration ("I'll inspect...") that is only progress text, so terminal text
+  // comes from non-tool assistant messages or trailing deltas.
   let copilotMessageBuffer = '';
 
+  function _streamText(value) {
+    return maxTextLength ? value.slice(-maxTextLength) : value;
+  }
+
+  function _copilotAssistantMessageHasTools(obj) {
+    const requests = obj?.data?.toolRequests;
+    return Array.isArray(requests) && requests.length > 0;
+  }
+
   function captureEvent(obj) {
     if (!obj || typeof obj !== 'object') return;
 
@@ -322,12 +331,12 @@ function _createStreamAccumulator({
       }
     }
     if (obj.type === 'assistant.message' && typeof obj.data?.content === 'string') {
-      // Authoritative final assistant text for this turn.
+      // Tool-request narration ("I'll look into this...") is progress text, not
+      // the final answer. Keep streaming it live, but don't let it become the
+      // terminal result if the process exits before a final answer message.
       const content = obj.data.content;
-      if (content) {
-        text = maxTextLength ? content.slice(-maxTextLength) : content;
-        copilotMessageBuffer = '';
-      }
+      if (content && !_copilotAssistantMessageHasTools(obj)) text = _streamText(content);
+      copilotMessageBuffer = '';
       if (Array.isArray(obj.data.toolRequests)) {
         for (const tr of obj.data.toolRequests) {
           if (tr && tr.name) {
@@ -372,6 +381,9 @@ function _createStreamAccumulator({
       const ev = runtime.parseStreamChunk(trimmed);
       if (ev) captureEvent(ev);
     }
+    if (copilotMessageBuffer) {
+      text = _streamText(copilotMessageBuffer);
+    }
     // Reconciliation: if any field is still missing, ask the runtime adapter
     // to re-parse the whole stdout. parseOutput() may catch a result event
     // that was malformed when streamed in chunks.

package/engine/runtimes/copilot.js

@@ -300,8 +300,10 @@ const KNOWN_EVENT_TYPES = new Set([
  * Returns { text, usage, sessionId, model } — same shape as the Claude adapter
  * so engine/lifecycle.js can consume both transparently.
  *
- *   - text:      concatenation of every `assistant.message.data.content` value
- *                across turns (multi-turn autopilot loops emit one per turn)
+ *   - text:      concatenation of answer `assistant.message.data.content`
+ *                values, plus trailing deltas if the CLI exits before a final
+ *                `assistant.message`. Narration attached to tool requests is
+ *                progress text and is excluded from the final answer.
  *   - usage:     mapped from the terminal `result` event. Copilot doesn't
  *                report cost/tokens — those fields are NULL, not 0, so the
  *                dashboard can distinguish "Copilot didn't tell us" from
@@ -322,6 +324,7 @@ function parseOutput(raw, { maxTextLength = 0 } = {}) {
   let model = null;
   let outputTokensTotal = 0;
   let turnEndCount = 0;
+  let pendingDeltaContent = '';
 
   for (const rawLine of safeRaw.split('\n')) {
     const line = rawLine.trim();
@@ -331,9 +334,17 @@ function parseOutput(raw, { maxTextLength = 0 } = {}) {
     if (!obj || typeof obj !== 'object') continue;
 
     const type = obj.type;
-    if (type === 'assistant.message') {
+    if (type === 'assistant.message_delta') {
+      const delta = obj.data?.deltaContent;
+      if (typeof delta === 'string') pendingDeltaContent += delta;
+    } else if (type === 'assistant.message') {
       const content = obj.data?.content;
-      if (typeof content === 'string' && content) messageContents.push(content);
+      const toolRequests = obj.data?.toolRequests;
+      const hasToolRequests = Array.isArray(toolRequests) && toolRequests.length > 0;
+      if (typeof content === 'string') {
+        if (content && !hasToolRequests) messageContents.push(content);
+        pendingDeltaContent = '';
+      }
       const ot = obj.data?.outputTokens;
       if (typeof ot === 'number') outputTokensTotal += ot;
     } else if (type === 'assistant.turn_end') {
@@ -366,7 +377,7 @@ function parseOutput(raw, { maxTextLength = 0 } = {}) {
     }
   }
 
-  let text = messageContents.join('');
+  let text = messageContents.join('') + pendingDeltaContent;
   if (maxTextLength && text.length > maxTextLength) {
     text = text.slice(-maxTextLength);
   }

package/package.json

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