@genesislcap/ai-assistant 15.21.0 → 15.21.1

package/dist/esm/utils/history-transform.js

@@ -22,15 +22,64 @@ function maskToolPayload(msg) {
     return msg;
 }
 /**
- * Prepares history for the LLM only: masks tool call args and results from other
- * agents so the active specialist is not confused by tools it does not have.
- * Canonical history in `ChatDriver` stays unmasked for UI and logging.
+ * Render another agent's tool call as prose: the narration it came with, if any, then the tool
+ * names. Keeps the fact that work happened without pretending the active agent can act on it.
+ */
+function foreignToolCallText(msg) {
+    var _a;
+    const names = ((_a = msg.toolCalls) !== null && _a !== void 0 ? _a : []).map((tc) => tc.name).join(', ');
+    const trace = names ? `[Earlier tool call: ${names}]` : '';
+    return msg.content && trace ? `${msg.content}\n${trace}` : msg.content || trace;
+}
+/**
+ * Prepares history for the LLM only: another agent's tool traffic becomes prose, so the active
+ * specialist sees THAT work happened without being handed calls it cannot make or results it
+ * cannot use. Canonical history in `ChatDriver` stays untouched for UI and logging.
+ *
+ * Why prose rather than masked-in-place blocks (GENC-1573). Masking used to clear a foreign
+ * call's `args` and stub its result's content, but leave both as real `tool_use`/`tool_result`
+ * blocks on the wire. That put a large, wholly uninformative structure in front of every routed
+ * agent — in the measured session the fallback's requests carried 389 `tool_use` and 313
+ * `tool_result` blocks, every one of them another agent's, the args emptied to `{}` and the
+ * results reduced to the same placeholder sentence repeated 313 times. It cost tokens on every
+ * turn, told the model nothing, and asked a tool-less agent to make sense of a tool transcript.
+ *
+ * A foreign result is dropped rather than rendered: its content was already replaced by a
+ * placeholder, so the only thing left to say is "a result came back", which the call line
+ * already implies. It is dropped via `excludeFromProvider` rather than by filtering the array,
+ * so index-aligned callers and the token estimator see the same list they always did.
+ *
+ * Pairs are flattened together, keyed on tool-call id rather than per message, because a
+ * half-flattened pair is a rejected request: an orphaned `tool_result` has no `tool_use` to
+ * answer. Both halves of a pair normally carry the same `agentName` — `appendToHistory` stamps
+ * the active agent on the call and its result alike — but a session restored from before that
+ * stamping, or appended with no agent active, can hold one half unattributed, and per-message
+ * flattening would then split it.
  */
 export function transformHistoryForAgent(history, agentName) {
+    var _a;
+    const isForeign = (msg) => !!msg.agentName && msg.agentName !== agentName;
+    // Pass 1: every tool-call id on either side of a foreign pair.
+    const foreignCallIds = new Set();
+    for (const msg of history) {
+        if (!isForeign(msg))
+            continue;
+        for (const tc of (_a = msg.toolCalls) !== null && _a !== void 0 ? _a : [])
+            foreignCallIds.add(tc.id);
+        if (msg.toolResult)
+            foreignCallIds.add(msg.toolResult.toolCallId);
+    }
+    // Pass 2: flatten both halves of anything the first pass touched, whatever its own attribution.
     return history.map((msg) => {
-        if (!msg.agentName || msg.agentName === agentName)
+        var _a;
+        const pairIsForeign = ((_a = msg.toolCalls) !== null && _a !== void 0 ? _a : []).some((tc) => foreignCallIds.has(tc.id)) ||
+            (msg.toolResult ? foreignCallIds.has(msg.toolResult.toolCallId) : false);
+        if (!pairIsForeign)
             return msg;
-        return maskToolPayload(msg);
+        if (msg.toolResult) {
+            return Object.assign(Object.assign({}, maskToolPayload(msg)), { excludeFromProvider: true });
+        }
+        return Object.assign(Object.assign({}, msg), { content: foreignToolCallText(msg), toolCalls: undefined });
     });
 }
 /**

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@genesislcap/ai-assistant",
   "description": "Genesis AI Assistant micro-frontend",
-  "version": "15.21.0",
+  "version": "15.21.1",
   "license": "SEE LICENSE IN license.txt",
   "main": "dist/esm/index.js",
   "types": "dist/ai-assistant.d.ts",
@@ -74,26 +74,26 @@
     }
   },
   "devDependencies": {
-    "@genesislcap/foundation-testing": "15.21.0",
-    "@genesislcap/genx": "15.21.0",
-    "@genesislcap/rollup-builder": "15.21.0",
-    "@genesislcap/ts-builder": "15.21.0",
-    "@genesislcap/uvu-playwright-builder": "15.21.0",
-    "@genesislcap/vite-builder": "15.21.0",
-    "@genesislcap/webpack-builder": "15.21.0",
+    "@genesislcap/foundation-testing": "15.21.1",
+    "@genesislcap/genx": "15.21.1",
+    "@genesislcap/rollup-builder": "15.21.1",
+    "@genesislcap/ts-builder": "15.21.1",
+    "@genesislcap/uvu-playwright-builder": "15.21.1",
+    "@genesislcap/vite-builder": "15.21.1",
+    "@genesislcap/webpack-builder": "15.21.1",
     "@types/dompurify": "^3.0.5",
     "@types/marked": "^5.0.2",
     "esbuild": "0.25.12"
   },
   "dependencies": {
-    "@genesislcap/foundation-ai": "15.21.0",
-    "@genesislcap/foundation-logger": "15.21.0",
-    "@genesislcap/foundation-notifications": "15.21.0",
-    "@genesislcap/foundation-redux": "15.21.0",
-    "@genesislcap/foundation-ui": "15.21.0",
-    "@genesislcap/foundation-utils": "15.21.0",
-    "@genesislcap/rapid-design-system": "15.21.0",
-    "@genesislcap/web-core": "15.21.0",
+    "@genesislcap/foundation-ai": "15.21.1",
+    "@genesislcap/foundation-logger": "15.21.1",
+    "@genesislcap/foundation-notifications": "15.21.1",
+    "@genesislcap/foundation-redux": "15.21.1",
+    "@genesislcap/foundation-ui": "15.21.1",
+    "@genesislcap/foundation-utils": "15.21.1",
+    "@genesislcap/rapid-design-system": "15.21.1",
+    "@genesislcap/web-core": "15.21.1",
     "dompurify": "^3.3.1",
     "marked": "^17.0.3"
   },