@yemi33/minions 0.1.2318 → 0.1.2320

package/dashboard/js/command-center.js

@@ -2051,15 +2051,6 @@ async function ccExecuteAction(action, targetTabId, opts) {
         wakeEngine();
         break;
       }
-      case 'link-pr': {
-        var prLinkRes = await _ccFetch('/api/pull-requests/link', { url: action.url, title: action.title || '', project: action.project || '', contextOnly: action.autoObserve === false });
-        var prLinkData = await prLinkRes.json().catch(function() { return {}; });
-        // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; PR URL and server message wrapped in escHtml() (fields: action.url, prLinkData.message)
-        status.innerHTML = '&#10003; PR linked: <strong>' + escHtml(action.url) + '</strong>' +
-          (prLinkData.message ? '<div style="font-size:var(--text-base);color:var(--muted);margin-top:4px">' + escHtml(prLinkData.message) + '</div>' : '');
-        status.style.color = 'var(--green)';
-        break;
-      }
       case 'archive-meeting': {
         await _ccFetch('/api/meetings/archive', { id: action.id });
         // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; meeting id wrapped in escHtml() (fields: action.id)

package/engine/runtimes/copilot.js

@@ -1098,6 +1098,28 @@ function createStreamConsumer(ctx) {
   // form `...prev.next...`. First push is NEVER prefixed (would leave every
   // reply with a leading blank line). Reset by reset() on stream restart.
   let _hadPriorAssistantMessage = false;
+  // `terminalText` accumulates EVERY non-tool assistant.message across the
+  // whole turn, including ones separated by intervening tool calls. That
+  // whole-turn scope is wrong for `ctx.setText(...)` on the `result` event:
+  // the dashboard's `ccSegmentsFinalize`/`_ccSegMergeText` (render-utils.js)
+  // expect the authoritative final text to describe ONLY the *trailing* text
+  // segment (correct for Claude's single last-message `result.result`), and
+  // merge it into just the last segment. Sending the whole-turn text there
+  // duplicated earlier paragraphs (W-mr3rdn9j000u6d5d) because the trailing
+  // segment only ever received the post-tool delta via ctx.pushText, not the
+  // full accumulated turn.
+  //
+  // `trailingText`/`_hadPriorTrailingMessage` mirror `terminalText`/
+  // `_hadPriorAssistantMessage` but are reset to '' / false on every tool
+  // boundary (a real tool call — not `task_complete`), so their scope always
+  // matches the trailing segment the dashboard will merge into.
+  let trailingText = '';
+  let _hadPriorTrailingMessage = false;
+
+  function _resetTrailingSegment() {
+    trailingText = '';
+    _hadPriorTrailingMessage = false;
+  }
 
   function _captureTaskComplete(summary, success = true, { clearBuffer = false } = {}) {
     if (typeof summary !== 'string' || !summary) return;
@@ -1117,7 +1139,7 @@ function createStreamConsumer(ctx) {
       // The result event is the first Copilot event that contains the resumable
       // sessionId. Do not mark the earlier assistant.message as terminal or
       // Minions can resolve before session persistence data is available.
-      const finalText = terminalText || copilotMessageBuffer || taskCompleteSummary;
+      const finalText = trailingText || copilotMessageBuffer || taskCompleteSummary;
       if (finalText) ctx.setText(finalText);
     }
 
@@ -1149,21 +1171,30 @@ function createStreamConsumer(ctx) {
         // the terminal answer. A non-tool assistant.message overrides any
         // streamed deltas (Copilot's authoritative final text for the turn).
         if (content && !hasTools) {
-          // Second-and-later terminating message in a single turn: prefix the
-          // pushed text with `\n\n` so the dashboard merge sees a clean
-          // paragraph boundary instead of welding `<prev>.<next>` together
-          // (W-mq1jwwqo000f20d9 — CC chunk welding fix). The first push must
-          // NOT be prefixed or every reply starts with a blank line. Tool-use
-          // boundaries are skipped — those get rendered as visible chips
-          // client-side and already act as natural separators.
-          const pushed = _hadPriorAssistantMessage ? '\n\n' + content : content;
+          // Second-and-later terminating message within the SAME trailing
+          // segment (no tool boundary since): prefix the pushed text with
+          // `\n\n` so the dashboard merge sees a clean paragraph boundary
+          // instead of welding `<prev>.<next>` together (W-mq1jwwqo000f20d9
+          // — CC chunk welding fix). The first push in a segment must NOT be
+          // prefixed (would leave every reply/segment with a leading blank
+          // line). This is now gated on `_hadPriorTrailingMessage` (reset on
+          // every real tool boundary below), not the whole-turn flag — a
+          // tool boundary already opens a brand-new segment on the dashboard
+          // side (ccSegmentsApplyChunk), so the first post-tool push needs no
+          // extra separator, and `trailingText` must track exactly what was
+          // pushed so ctx.setText(...) on `result` reconciles cleanly
+          // (W-mr3rdn9j000u6d5d).
+          const pushed = _hadPriorTrailingMessage ? '\n\n' + content : content;
           terminalText = _hadPriorAssistantMessage ? terminalText + '\n\n' + content : content;
+          trailingText = _hadPriorTrailingMessage ? trailingText + '\n\n' + content : content;
           ctx.pushText(pushed);
           _hadPriorAssistantMessage = true;
+          _hadPriorTrailingMessage = true;
         }
         copilotMessageBuffer = '';
       }
       if (Array.isArray(data.toolRequests)) {
+        let sawRealTool = false;
         for (const tr of data.toolRequests) {
           if (!tr || !tr.name) continue;
           if (tr.name === 'task_complete') {
@@ -1171,7 +1202,12 @@ function createStreamConsumer(ctx) {
             continue;
           }
           ctx.pushToolUse(tr.name, tr.arguments || {});
+          sawRealTool = true;
         }
+        // A real (non-task_complete) tool call closes the trailing segment —
+        // the NEXT non-tool assistant.message starts a fresh segment on the
+        // dashboard side, so its finalize-time scope must reset here too.
+        if (sawRealTool) _resetTrailingSegment();
       }
       return;
     }
@@ -1188,6 +1224,11 @@ function createStreamConsumer(ctx) {
       if (!ctx.toolUseAlreadySeen(name, input)) {
         ctx.pushToolUse(name, input);
       }
+      // Same trailing-segment boundary reset as the toolRequests branch above
+      // — `tool.execution_start` is the standalone event shape observed in
+      // the primary repro (assistant.message → tool.execution_start →
+      // assistant.message → result).
+      _resetTrailingSegment();
     }
   }
 
@@ -1196,6 +1237,7 @@ function createStreamConsumer(ctx) {
     terminalText = '';
     taskCompleteSummary = '';
     _hadPriorAssistantMessage = false;
+    _resetTrailingSegment();
   }
 
   return { consume, reset };

package/package.json

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