@salesforce/sfdx-agent-harness-mastra 0.45.0 → 0.46.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.46.0] - 2026-08-10
+
+### Fixes
+- **harness-mastra**: normalize empty tool-call args to "{}" across all passthroughs @W-23789221@ ([#745](https://github.com/forcedotcom/agentic-dx/pull/745))
+
 ## [0.45.0] - 2026-08-10
 
 ### Fixes

package/dist/pass-through-anthropic.js

@@ -4,7 +4,7 @@
  */
 import { AnthropicBedrock } from '@anthropic-ai/bedrock-sdk';
 import Anthropic from '@anthropic-ai/sdk';
-import { TextSpanLifecycle, createGatewayFetch, isEmptyUsage, isParameterExcluded, mapAnthropicFinishReason, mapAnthropicUsage, toolResultOutputToString, } from './v3-stream-mapping.js';
+import { TextSpanLifecycle, createGatewayFetch, isEmptyUsage, isParameterExcluded, mapAnthropicFinishReason, mapAnthropicUsage, normalizeToolArgsString, toolResultOutputToString, } from './v3-stream-mapping.js';
 /**
  * Builds the in-process `LanguageModelV3` impl for the Salesforce LLM Gateway's
  * `/invoke-with-response-stream` (Bedrock-Anthropic pass-through) endpoint.
@@ -266,17 +266,9 @@ async function doStream(ctx, options) {
                                     type: 'tool-call',
                                     toolCallId: tu.id,
                                     toolName: tu.name,
-                                    // A no-arg tool (e.g. startBuilding) streams no
-                                    // input_json_delta events, so argsBuffer stays ''.
-                                    // '' is not a schema-conforming stringified JSON
-                                    // object — a strict consumer's JSON.parse throws and
-                                    // the model retries. Coerce the empty buffer to '{}'
-                                    // (the only valid no-arg value); non-empty buffers,
-                                    // even malformed ones, pass through verbatim so we
-                                    // never mask a genuinely bad payload. W-23789221;
-                                    // matches the non-streaming path + parseToolInput
-                                    // (same `.trim()` empty check as the latter).
-                                    input: tu.argsBuffer.trim() === '' ? '{}' : tu.argsBuffer,
+                                    // No-arg tool → empty argsBuffer; normalize to '{}'.
+                                    // See normalizeToolArgsString (W-23789221).
+                                    input: normalizeToolArgsString(tu.argsBuffer),
                                 });
                                 toolUseByIndex.delete(event.index);
                             }

package/dist/pass-through-openai-chat-completions.js

@@ -3,7 +3,7 @@
  * See LICENSE.txt for license terms.
  */
 import OpenAI from 'openai';
-import { TextSpanLifecycle, createGatewayFetch, isEmptyUsage, isParameterExcluded, mapOpenAIUsage, toolResultOutputToString, } from './v3-stream-mapping.js';
+import { TextSpanLifecycle, createGatewayFetch, isEmptyUsage, isParameterExcluded, mapOpenAIUsage, normalizeToolArgsString, toolResultOutputToString, } from './v3-stream-mapping.js';
 /**
  * Builds the in-process `LanguageModelV3` impl that drives the `openai` SDK's
  * `chat.completions.create({ stream: true })` against an OpenAI-compatible
@@ -94,7 +94,8 @@ async function doGenerate(ctx, options) {
                 type: 'tool-call',
                 toolCallId: tc.id,
                 toolName: tc.function.name,
-                input: tc.function.arguments ?? '',
+                // No-arg tool → empty arguments; normalize to '{}' (W-23789221).
+                input: normalizeToolArgsString(tc.function.arguments),
             });
         }
     }
@@ -212,7 +213,8 @@ async function doStream(ctx, options) {
                                 type: 'tool-call',
                                 toolCallId: acc.id,
                                 toolName: acc.name,
-                                input: acc.argsBuffer,
+                                // No-arg tool → empty argsBuffer; normalize to '{}' (W-23789221).
+                                input: normalizeToolArgsString(acc.argsBuffer),
                             });
                         }
                         toolCallsByIndex.clear();

package/dist/pass-through-openai.js

@@ -3,7 +3,7 @@
  * See LICENSE.txt for license terms.
  */
 import OpenAI from 'openai';
-import { TextSpanLifecycle, createGatewayFetch, isEmptyUsage, isParameterExcluded, mapOpenAIFinishReason, mapOpenAIUsage, toolResultOutputToString, } from './v3-stream-mapping.js';
+import { TextSpanLifecycle, createGatewayFetch, isEmptyUsage, isParameterExcluded, mapOpenAIFinishReason, mapOpenAIUsage, normalizeToolArgsString, toolResultOutputToString, } from './v3-stream-mapping.js';
 /**
  * Builds the in-process `LanguageModelV3` impl that drives the `openai` SDK's
  * `responses.create({ stream: true })` against the Salesforce LLM Gateway's
@@ -89,7 +89,8 @@ async function doGenerate(ctx, options) {
                 type: 'tool-call',
                 toolCallId: item.call_id,
                 toolName: item.name,
-                input: item.arguments ?? '',
+                // No-arg tool → empty arguments; normalize to '{}' (W-23789221).
+                input: normalizeToolArgsString(item.arguments),
             });
         }
         // Other output-item variants (reasoning, web/file/code search, MCP) are
@@ -149,7 +150,8 @@ async function doStream(ctx, options) {
                                     type: 'tool-call',
                                     toolCallId: item.call_id,
                                     toolName: item.name,
-                                    input: finalArgs,
+                                    // No-arg tool → empty finalArgs; normalize to '{}' (W-23789221).
+                                    input: normalizeToolArgsString(finalArgs),
                                 });
                                 if (item.id)
                                     toolCallById.delete(item.id);

package/dist/v3-stream-mapping.d.ts

@@ -265,6 +265,34 @@ type WireCommunicationLikeEvent = {
  * worst place for a copy-paste drift to land.
  */
 export declare function toolResultOutputToString(output: LanguageModelV3ToolResultOutput): string;
+/**
+ * Normalize a streamed/returned tool-call arguments STRING to a
+ * schema-conforming stringified JSON object.
+ *
+ * A no-argument tool (input schema `{ type:'object', properties:{} }`) accrues
+ * no argument fragments on the wire — Anthropic streams no `input_json_delta`,
+ * the OpenAI Responses/Chat-Completions APIs send no arg deltas — so the
+ * accumulated buffer is the empty string `''`. `''` is NOT a schema-conforming
+ * stringified JSON object: a strict consumer (Mastra `secureJsonParse`, any
+ * `JSON.parse`) throws "the provided JSON could not be parsed" and the model
+ * retries, producing the empty-args retry loop (W-23789221; same class as the
+ * closed gateway-processor defect W-22696051). `'{}'` is the only valid no-arg
+ * value.
+ *
+ * Empty (including whitespace-only) → `'{}'`. Every non-empty buffer — even a
+ * malformed partial like `'{"q":'` — is returned VERBATIM: we never re-parse or
+ * re-stringify real content, so a genuinely bad payload still reaches the
+ * consumer and is still rejected. Applied at every provider's tool-call
+ * emission seam so the rule lives in one auditable place.
+ *
+ * Sibling to `sfdx-agent-chat-generations`'s `normalizeToolArguments` (the
+ * gpt5-processor path). This one additionally coerces a whitespace-only buffer
+ * — the pass-through streaming paths accumulate raw wire fragments that could in
+ * principle be whitespace, whereas the chat-generations processor normalizes
+ * earlier. The packages share no dependency, so the rule is duplicated rather
+ * than imported; keep the two in sync if the empty-definition changes.
+ */
+export declare function normalizeToolArgsString(args: string | null | undefined): string;
 /**
  * Predicate the missing-usage faithfulness invariant (W-22692131) enforces at
  * every terminal `finish` site: when every input/output token field on the

package/dist/v3-stream-mapping.js

@@ -595,6 +595,36 @@ export function toolResultOutputToString(output) {
             return output.reason ?? 'tool execution denied';
     }
 }
+/**
+ * Normalize a streamed/returned tool-call arguments STRING to a
+ * schema-conforming stringified JSON object.
+ *
+ * A no-argument tool (input schema `{ type:'object', properties:{} }`) accrues
+ * no argument fragments on the wire — Anthropic streams no `input_json_delta`,
+ * the OpenAI Responses/Chat-Completions APIs send no arg deltas — so the
+ * accumulated buffer is the empty string `''`. `''` is NOT a schema-conforming
+ * stringified JSON object: a strict consumer (Mastra `secureJsonParse`, any
+ * `JSON.parse`) throws "the provided JSON could not be parsed" and the model
+ * retries, producing the empty-args retry loop (W-23789221; same class as the
+ * closed gateway-processor defect W-22696051). `'{}'` is the only valid no-arg
+ * value.
+ *
+ * Empty (including whitespace-only) → `'{}'`. Every non-empty buffer — even a
+ * malformed partial like `'{"q":'` — is returned VERBATIM: we never re-parse or
+ * re-stringify real content, so a genuinely bad payload still reaches the
+ * consumer and is still rejected. Applied at every provider's tool-call
+ * emission seam so the rule lives in one auditable place.
+ *
+ * Sibling to `sfdx-agent-chat-generations`'s `normalizeToolArguments` (the
+ * gpt5-processor path). This one additionally coerces a whitespace-only buffer
+ * — the pass-through streaming paths accumulate raw wire fragments that could in
+ * principle be whitespace, whereas the chat-generations processor normalizes
+ * earlier. The packages share no dependency, so the rule is duplicated rather
+ * than imported; keep the two in sync if the empty-definition changes.
+ */
+export function normalizeToolArgsString(args) {
+    return args && args.trim() !== '' ? args : '{}';
+}
 /**
  * Predicate the missing-usage faithfulness invariant (W-22692131) enforces at
  * every terminal `finish` site: when every input/output token field on the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/sfdx-agent-harness-mastra",
-  "version": "0.45.0",
+  "version": "0.46.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.46.0"
+    "@salesforce/sfdx-agent-sdk": "0.47.0"
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
     "@salesforce/harness-conformance": "0.0.1",
-    "@salesforce/sfdx-agent-sdk": "0.46.0",
+    "@salesforce/sfdx-agent-sdk": "0.47.0",
     "@types/node": "^22.20.1",
     "@vitest/coverage-istanbul": "^4.1.10",
     "@vitest/eslint-plugin": "^1.6.26",