@salesforce/sfdx-agent-harness-mastra 0.83.0 → 0.84.0

package/CHANGELOG.md

@@ -3,6 +3,11 @@
 All notable changes to `@salesforce/sfdx-agent-harness-mastra` are documented in this file.
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [0.84.0] - 2026-09-16
+
+### Fixes
+- **harness-mastra**: carry toolName on tool-call-delta so AG-UI TOOL_CALL_START isn't blank @W-23954558@ ([#826](https://github.com/forcedotcom/agentic-dx/pull/826))
+
 ## [0.83.0] - 2026-09-16
 
 _No changes — released alongside dependent packages._

package/dist/mastra-event-adapter.d.ts

@@ -56,6 +56,7 @@ import type { ChatEvent } from '@salesforce/sfdx-agent-sdk';
 export declare class MastraEventAdapter {
     private readonly logBus?;
     private stepIndex;
+    private readonly toolNamesByCallId;
     /**
      * @param logBus - Optional structured-log sink. When supplied, the adapter emits a `debug`
      *   log for every chunk it intentionally drops — informational lifecycle events, known-but-

package/dist/mastra-event-adapter.js

@@ -60,6 +60,12 @@ import { isClassifiedStreamError } from './stream-errors.js';
 export class MastraEventAdapter {
     logBus;
     stepIndex = 0;
+    // `toolCallId → toolName`, populated from each `tool-call-input-streaming-start`
+    // chunk. Mastra omits `toolName` from the `tool-call-delta` chunks that follow
+    // (it rides only on the streaming-start), so this lets `tool-call-delta` recover
+    // it — see the `tool-call-delta` case. Per-stream instance, so it never grows
+    // unbounded or leaks across turns.
+    toolNamesByCallId = new Map();
     /**
      * @param logBus - Optional structured-log sink. When supplied, the adapter emits a `debug`
      *   log for every chunk it intentionally drops — informational lifecycle events, known-but-
@@ -208,26 +214,46 @@ export class MastraEventAdapter {
                     toolName: chunk.payload.toolName ?? 'unknown',
                     output: chunk.payload.output,
                 };
-            case 'tool-call-delta':
+            case 'tool-call-delta': {
+                // Mastra drops `toolName` from this chunk — it carries it only on the
+                // preceding `tool-call-input-streaming-start` — so recover it from the
+                // per-stream map. Downstream AG-UI opens its TOOL_CALL_START off the
+                // first delta and can't correct a blank name from the terminal
+                // `tool-call`, so a missing name here surfaces as a blank tool name.
+                // Strict `!== undefined` (not truthy) so an empty-string toolName is
+                // forwarded honestly instead of silently swallowed.
+                const toolName = chunk.payload.toolName ?? this.toolNamesByCallId.get(chunk.payload.toolCallId);
                 return {
                     type: 'tool-call-delta',
                     toolCallId: chunk.payload.toolCallId,
-                    // Strict `!== undefined` (not truthy) so an empty-string
-                    // toolName from upstream — unlikely but possible — is
-                    // forwarded honestly instead of silently swallowed.
-                    ...(chunk.payload.toolName !== undefined ? { toolName: chunk.payload.toolName } : {}),
+                    ...(toolName !== undefined ? { toolName } : {}),
                     argsTextDelta: chunk.payload.argsTextDelta,
                 };
+            }
             // Known harness chunk types not yet supported in the SDK's ChatEvent
             // model. Silently dropped to avoid leaking harness internals onto the
             // public stream. Candidates for future ChatEvent variants when the
             // corresponding features (citations, file generation, tool input
             // lifecycle bookends, tool suspension, structured output) are
             // supported.
-            case 'source':
-            case 'file':
             case 'tool-call-input-streaming-start':
+                // Not surfaced on the public stream, but its `toolName` is the only
+                // place Mastra carries it for the streamed args — stash it so the
+                // following `tool-call-delta` chunks can recover it (see that case).
+                if (chunk.payload.toolName !== undefined) {
+                    this.toolNamesByCallId.set(chunk.payload.toolCallId, chunk.payload.toolName);
+                }
+                this.logBus?.debug('Skipping unsupported Mastra chunk type', { chunkType: chunk.type });
+                return undefined;
             case 'tool-call-input-streaming-end':
+                // End of the streamed-args span: no more deltas for this id will
+                // arrive, and the terminal `tool-call` carries its own name, so drop
+                // the tracked entry to keep the map bounded within a long stream.
+                this.toolNamesByCallId.delete(chunk.payload.toolCallId);
+                this.logBus?.debug('Skipping unsupported Mastra chunk type', { chunkType: chunk.type });
+                return undefined;
+            case 'source':
+            case 'file':
             case 'tool-call-suspended':
             case 'object':
                 this.logBus?.debug('Skipping unsupported Mastra chunk type', { chunkType: chunk.type });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/sfdx-agent-harness-mastra",
-  "version": "0.83.0",
+  "version": "0.84.0",
   "description": "Mastra-backed AgentHarness implementation for @salesforce/sfdx-agent-sdk",
   "type": "module",
   "main": "dist/index.js",
@@ -42,12 +42,12 @@
     "openai": "^7.0.0"
   },
   "peerDependencies": {
-    "@salesforce/sfdx-agent-sdk": "0.84.0"
+    "@salesforce/sfdx-agent-sdk": "0.85.0"
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
     "@salesforce/harness-conformance": "0.0.1",
-    "@salesforce/sfdx-agent-sdk": "0.84.0",
+    "@salesforce/sfdx-agent-sdk": "0.85.0",
     "@types/node": "^22.20.1",
     "@vitest/coverage-istanbul": "^4.1.11",
     "@vitest/eslint-plugin": "^1.6.27",