@salesforce/sfdx-agent-harness-mastra 0.97.0 → 0.99.0

package/CHANGELOG.md

@@ -3,6 +3,20 @@
 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.99.0] - 2026-10-06
+
+### Features
+- **agent-sdk,harness-claude,harness-mastra,harness-openai**: add 'application-policy' kind to ToolDeclineReason @W-24368893@ ([#854](https://github.com/forcedotcom/agentic-dx/pull/854))
+
+### Chores
+- **deps-dev**: bump eslint-plugin-n from 18.3.0 to 18.4.0 in the eslint group across 1 directory ([#859](https://github.com/forcedotcom/agentic-dx/pull/859))
+- **deps-dev**: bump the vitest group with 2 updates ([#860](https://github.com/forcedotcom/agentic-dx/pull/860))
+
+## [0.98.0] - 2026-10-05
+
+### Fixes
+- **agent-sdk,harness-claude,harness-mastra,harness-openai**: rewrite MCP clientInfo.name to the running agent's name @W-24101379@ ([#843](https://github.com/forcedotcom/agentic-dx/pull/843))
+
 ## [0.97.0] - 2026-10-05
 
 ### Features

package/README.md

@@ -250,10 +250,10 @@ Consumer-executed tools (`AgentConfig.tools`) always bypass the gate — their e
 via `submitToolResult`. Set `StreamOptions.batchApprovals: true` to surface parallel `require-approval` requests on one
 stream for a batch-approval UI.
 
-With `@mastra/core` 1.62.0, an organization-policy decline is passed through `agent.declineToolCall({ reason })`. Mastra
-emits one native `tool-output-denied` chunk; the adapter maps it to one `tool-result(isError=true)` carrying the same
-exact message the model sees. Automatic policy denies use `Denied by tool policy.`; reasonless consumer declines retain
-Mastra's `Tool call was not approved by the user` default.
+With `@mastra/core` 1.62.0, a typed-reason decline (either `ToolDeclineReason` kind) is passed through
+`agent.declineToolCall({ reason })`. Mastra emits one native `tool-output-denied` chunk; the adapter maps it to one
+`tool-result(isError=true)` carrying the same exact message the model sees. Automatic policy denies use
+`Denied by tool policy.`; reasonless consumer declines retain Mastra's `Tool call was not approved by the user` default.
 
 ### Tool approval timeout
 

package/dist/mastra-harness.js

@@ -623,7 +623,7 @@ export class MastraHarness {
             // object and stores it here so we can attach each to its
             // McpServerState below (W-23231658).
             const httpErrorCaptures = new Map();
-            const enabledServers = mapToMastraMCPServers(config.mcpServers, config.orgJwt, this.innerFetch, httpErrorCaptures, options?.mcpAuthProviders);
+            const enabledServers = mapToMastraMCPServers(config.mcpServers, config.orgJwt, this.innerFetch, httpErrorCaptures, options?.mcpAuthProviders, config.name);
             for (const [serverName, serverDef] of Object.entries(enabledServers)) {
                 const client = new MCPClient({
                     id: `${agentId}:${serverName}`,
@@ -1107,7 +1107,7 @@ export class MastraHarness {
         ]);
         const nextHttpErrorCaptures = new Map();
         const nextEnabledServers = hasEnabledServers(nextConfig)
-            ? mapToMastraMCPServers(nextConfig, config.orgJwt, this.innerFetch, nextHttpErrorCaptures, nextProviders)
+            ? mapToMastraMCPServers(nextConfig, config.orgJwt, this.innerFetch, nextHttpErrorCaptures, nextProviders, config.name)
             : {};
         for (const name of allNames) {
             const prev = prevConfig[name];

package/dist/mcp-config-mapper.d.ts

@@ -57,4 +57,6 @@ innerFetch?: typeof fetch,
  * (W-23231658). Stdio servers contribute no entry. Omitted in call sites
  * that don't need recovery (e.g. unit tests of the mapping shape).
  */
-captures?: Map<string, HttpErrorCapture>, providers?: McpAuthProviders): Record<string, MastraMCPServerDefinition>;
+captures?: Map<string, HttpErrorCapture>, providers?: McpAuthProviders, 
+/** AgentConfig.name; see {@link resolveMcpClientName}. */
+agentName?: string): Record<string, MastraMCPServerDefinition>;

package/dist/mcp-config-mapper.js

@@ -2,7 +2,7 @@
  * Copyright 2026, Salesforce, Inc. All rights reserved.
  * See LICENSE.txt for license terms.
  */
-import { resolveMcpServerHeaders, } from '@salesforce/sfdx-agent-sdk';
+import { resolveMcpClientName, resolveMcpServerHeaders, } from '@salesforce/sfdx-agent-sdk';
 /**
  * Returns `true` if `config` contains at least one server that is not explicitly disabled.
  * A server is considered enabled when its `enabled` property is `true` or absent.
@@ -67,13 +67,17 @@ innerFetch = globalThis.fetch,
  * (W-23231658). Stdio servers contribute no entry. Omitted in call sites
  * that don't need recovery (e.g. unit tests of the mapping shape).
  */
-captures, providers = {}) {
+captures, providers = {}, 
+/** AgentConfig.name; see {@link resolveMcpClientName}. */
+agentName) {
     const result = {};
     for (const [name, serverConfig] of Object.entries(config)) {
         if (serverConfig.enabled === false) {
             continue;
         }
         if (serverConfig.type === 'stdio') {
+            // Stdio has no fetch layer for rewriting `clientInfo`, and `@mastra/mcp`
+            // exposes no constructor option for it. See https://github.com/mastra-ai/mastra/issues/25473.
             const def = {
                 command: serverConfig.command,
                 ...(serverConfig.args !== undefined && { args: serverConfig.args }),
@@ -92,9 +96,11 @@ captures, providers = {}) {
         else if (serverConfig.type === 'remote') {
             const url = serverConfig.url instanceof URL ? serverConfig.url : new URL(serverConfig.url);
             const provider = providers[name];
+            // Wrap the innermost fetch so the client identity reaches every auth path.
+            const clientInfoFetch = wrapFetchWithClientInfoName(innerFetch, agentName, orgJwt);
             const baseFetch = provider === undefined && orgJwt
-                ? createAuthFetch(serverConfig.url, orgJwt, innerFetch, serverConfig.headers)
-                : innerFetch;
+                ? createAuthFetch(serverConfig.url, orgJwt, clientInfoFetch, serverConfig.headers)
+                : clientInfoFetch;
             // Wrap with HTTP-error capture when the harness asked for it, so a
             // non-2xx connect response's status + body survive the error
             // `@mastra/mcp` discards (W-23231658). The mapper owns the capture
@@ -110,7 +116,7 @@ captures, providers = {}) {
                 transportFetch = wrapFetchWithErrorCapture(baseFetch, capture);
             }
             else {
-                transportFetch = orgJwt || provider !== undefined ? baseFetch : undefined;
+                transportFetch = baseFetch;
             }
             const def = {
                 url,
@@ -154,6 +160,54 @@ function createAuthFetch(serverUrl, orgJwt, innerFetch, existingHeaders) {
         return innerFetch(url, { ...init, headers: merged });
     };
 }
+/**
+ * Wraps a transport fetch to rewrite `clientInfo.name` on MCP `initialize`
+ * requests. Other requests pass through unchanged, and extra arguments are forwarded.
+ *
+ * Only the `initialize` handshake is rewritten: a server configured with
+ * `protocolVersion: '2026-07-28'` sends no `initialize`, so the rewrite does nothing there.
+ */
+function wrapFetchWithClientInfoName(baseFetch, agentName, orgJwt) {
+    const forward = baseFetch;
+    const wrapped = async (input, init, ...rest) => {
+        const rewrittenBody = rewriteInitializeClientInfoName(init?.body, () => resolveMcpClientName(agentName, orgJwt));
+        return rewrittenBody === undefined
+            ? forward(input, init, ...rest)
+            : forward(input, { ...init, body: rewrittenBody }, ...rest);
+    };
+    return wrapped;
+}
+/**
+ * If `body` is a JSON-RPC `initialize` request, return a new body string with
+ * `params.clientInfo.name` set to `resolveName()` (creating `clientInfo` when
+ * absent, preserving `version` and any other fields). Returns `undefined` — a
+ * pass-through signal — for any non-`initialize` request or an unparseable /
+ * non-string body. Best-effort: it never throws into the transport.
+ */
+function rewriteInitializeClientInfoName(body, resolveName) {
+    if (typeof body !== 'string')
+        return undefined;
+    let message;
+    try {
+        message = JSON.parse(body);
+    }
+    catch {
+        return undefined;
+    }
+    if (message === null || typeof message !== 'object' || Array.isArray(message))
+        return undefined;
+    const msg = message;
+    if (msg.method !== 'initialize')
+        return undefined;
+    const params = typeof msg.params === 'object' && msg.params !== null && !Array.isArray(msg.params)
+        ? msg.params
+        : {};
+    const nextParams = {
+        ...params,
+        clientInfo: { ...(params.clientInfo ?? {}), name: resolveName() },
+    };
+    return JSON.stringify({ ...msg, params: nextParams });
+}
 /**
  * Wrap a transport fetch so every non-2xx response's status + body is recorded
  * into `capture.latest` before the response is handed back to `@mastra/mcp`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/sfdx-agent-harness-mastra",
-  "version": "0.97.0",
+  "version": "0.99.0",
   "description": "Mastra-backed AgentHarness implementation for @salesforce/sfdx-agent-sdk",
   "type": "module",
   "main": "dist/index.js",
@@ -42,20 +42,20 @@
     "openai": "^7.23.0"
   },
   "peerDependencies": {
-    "@salesforce/sfdx-agent-sdk": "0.98.0"
+    "@salesforce/sfdx-agent-sdk": "0.100.0"
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
     "@salesforce/harness-conformance": "0.0.1",
-    "@salesforce/sfdx-agent-sdk": "0.98.0",
+    "@salesforce/sfdx-agent-sdk": "0.100.0",
     "@types/node": "^22.20.1",
-    "@vitest/coverage-istanbul": "^5.0.1",
+    "@vitest/coverage-istanbul": "^5.0.2",
     "@vitest/eslint-plugin": "^1.6.27",
     "eslint": "^10.11.0",
     "eslint-config-prettier": "^10.1.8",
     "eslint-import-resolver-typescript": "^4.4.5",
     "eslint-plugin-import": "^2.32.0",
-    "eslint-plugin-n": "^18.3.0",
+    "eslint-plugin-n": "^18.4.0",
     "globals": "^17.12.0",
     "lint-staged": "^17.5.1",
     "prettier": "^3.9.7",