@salesforce/sfdx-agent-sdk 0.32.0 → 0.33.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/CHANGELOG.md CHANGED
@@ -3,6 +3,21 @@
3
3
  All notable changes to `@salesforce/sfdx-agent-sdk` are documented in this file.
4
4
  Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
5
5
 
6
+ ## [0.33.0] - 2026-07-14
7
+
8
+ ### Features
9
+ - **agent-sdk,harness-mastra,harness-claude**: cross-harness conformance test suite @W-23372341@ ([#663](https://github.com/forcedotcom/agentic-dx/pull/663))
10
+ - **sdk,harness-mastra,harness-claude**: delete deprecated tool-approval surfaces @W-23090034 ([#637](https://github.com/forcedotcom/agentic-dx/pull/637))
11
+
12
+ ### Fixes
13
+ - **harness-mastra**: thread model maxOutputTokens instead of flooring max_tokens at 4096 @W-23413250@ ([#656](https://github.com/forcedotcom/agentic-dx/pull/656))
14
+
15
+ ### Chores
16
+ - **deps-dev**: bump typescript-eslint from 8.62.1 to 8.63.0 in the dev-dependencies group across 1 directory ([#665](https://github.com/forcedotcom/agentic-dx/pull/665))
17
+ - **deps-dev**: bump the eslint group across 1 directory with 2 updates ([#659](https://github.com/forcedotcom/agentic-dx/pull/659))
18
+ - **deps-dev**: bump @vitest/eslint-plugin from 1.6.22 to 1.6.23 in the vitest group ([#660](https://github.com/forcedotcom/agentic-dx/pull/660))
19
+ - **deps-dev**: bump the vitest group across 1 directory with 3 updates ([#655](https://github.com/forcedotcom/agentic-dx/pull/655))
20
+
6
21
  ## [0.32.0] - 2026-07-08
7
22
 
8
23
  ### Fixes
package/README.md CHANGED
@@ -232,33 +232,27 @@ function onApprovalRequest(event: ToolApprovalRequestEvent): Promise<boolean> {
232
232
 
233
233
  #### `AgentConfig`
234
234
 
235
- | Field | Type | Description |
236
- | ---------------------- | -------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
237
- | `orgAlias?` | `string` | Salesforce org alias or username. Falls back to project/default org. |
238
- | `modelId?` | `ModelName \| Model` | LLM model selector. Pass a `ModelName` enum value for an in-tree model (e.g. `'llmgateway__OpenAIGPT5'`), or a pre-built `Model` instance to opt into a Bedrock-Anthropic Claude variant the SDK has not yet released — see `createClaudeModel(gatewayId, overrides)` exported from this package. |
239
- | `name?` | `string` | Human-readable agent name. |
240
- | `description?` | `string` | Agent purpose description. |
241
- | `instructions?` | `string` | System instructions for the agent. |
242
- | `tools?` | `ToolDefinition[]` | Consumer-executed tool schemas. |
243
- | `mcpServers?` | `MCPConfiguration` | MCP server connections. |
244
- | `skills?` | `string[]` | Each entry is either an individual skill folder (containing `SKILL.md`) or a parent folder containing skill subfolders. Relative and absolute paths supported; forms can be mixed in the same array. |
245
- | `rules?` | `string[]` | Each entry is either an individual `.md` rule file or a directory of `.md` rule files (scanned one level deep, alphabetical, non-`.md` skipped). Bodies are composed verbatim into the agent's effective system prompt; YAML frontmatter is optional and stripped if present. Matches Claude Code's `.claude/rules/*.md` convention. |
246
- | `toolPolicies?` | `ToolPolicyRule[]` | Ordered per-tool approval rules resolved by `resolveToolApprovalPolicy` (cross-tier deny-wins / within-tier last-wins). Author directly or via `definePolicy(...)`. See "Tool Approval Policy" below. Has no effect until a harness wires the resolver (Phase 2); until then gating uses the deprecated `StreamOptions.requireToolApproval`. |
247
- | `defaultToolDecision?` | `Decision` | Fallback decision when no rule matches. Defaults to `'allow'` (no policy ⇒ no gating). Set to `'require-approval'` for a fail-closed posture (recommended for catalogs with un-annotated MCP servers). |
235
+ | Field | Type | Description |
236
+ | ---------------------- | -------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
237
+ | `orgAlias?` | `string` | Salesforce org alias or username. Falls back to project/default org. |
238
+ | `modelId?` | `ModelName \| Model` | LLM model selector. Pass a `ModelName` enum value for an in-tree model (e.g. `'llmgateway__OpenAIGPT5'`), or a pre-built `Model` instance to opt into a Bedrock-Anthropic Claude variant the SDK has not yet released — see `createClaudeModel(gatewayId, overrides)` exported from this package. |
239
+ | `name?` | `string` | Human-readable agent name. |
240
+ | `description?` | `string` | Agent purpose description. |
241
+ | `instructions?` | `string` | System instructions for the agent. |
242
+ | `tools?` | `ToolDefinition[]` | Consumer-executed tool schemas. |
243
+ | `mcpServers?` | `MCPConfiguration` | MCP server connections. |
244
+ | `skills?` | `string[]` | Each entry is either an individual skill folder (containing `SKILL.md`) or a parent folder containing skill subfolders. Relative and absolute paths supported; forms can be mixed in the same array. |
245
+ | `rules?` | `string[]` | Each entry is either an individual `.md` rule file or a directory of `.md` rule files (scanned one level deep, alphabetical, non-`.md` skipped). Bodies are composed verbatim into the agent's effective system prompt; YAML frontmatter is optional and stripped if present. Matches Claude Code's `.claude/rules/*.md` convention. |
246
+ | `toolPolicies?` | `ToolPolicyRule[]` | Ordered per-tool approval rules resolved by `resolveToolApprovalPolicy` (cross-tier deny-wins / within-tier last-wins). Author directly or via `definePolicy(...)`. See "Tool Approval Policy" below. |
247
+ | `defaultToolDecision?` | `Decision` | Fallback decision when no rule matches. Defaults to `'allow'` (no policy ⇒ no gating). Set to `'require-approval'` for a fail-closed posture (recommended for catalogs with un-annotated MCP servers). |
248
248
 
249
249
  #### `StreamOptions`
250
250
 
251
- | Field | Type | Description |
252
- | ---------------------- | ----------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
253
- | `abortSignal?` | `AbortSignal` | Abort the streaming operation. |
254
- | `requireToolApproval?` | `boolean \| ToolApprovalMode` | **Deprecated.** Per-call all-or-nothing gating. Superseded by per-tool policy on `AgentConfig.toolPolicies`; the serial-vs-batch UX axis moved to `batchApprovals`. Kept functional for one release so harness PRs migrate independently. `true` / `'serial'` emits one approval per stream; `'batch'` surfaces parallel approvals on one stream (requires Pattern A iterators). See "Tool Approval Flow" below. |
255
- | `batchApprovals?` | `boolean` | When `true`, parallel approval-requests within a turn surface on the same stream so the consumer can render a batch approval card (requires Pattern A iterators collect-all-then-settle). Defaults to `false` (serial). The self-documenting replacement for the UX half of the deprecated `requireToolApproval` enum; the gating half moves to `AgentConfig.toolPolicies`. No effect when no tool in the turn resolves to `'require-approval'`. |
256
- | `maxSteps?` | `number` | Maximum number of LLM call steps the agent may take per `stream()` invocation. Each step is one LLM call (which may produce text, tool calls, or both). Must be `>= 1`. Defaults to `DEFAULT_MAX_STEPS` (1024) — high enough to be effectively unlimited for real tasks; the practical ceiling is the context window and cost. The constant is exported so consumers and harness authors share one source of truth. |
257
-
258
- `ToolApprovalMode` is the exported (deprecated) type alias `'serial' | 'batch'`. Pair with the (deprecated)
259
- `resolveToolApprovalMode(boolean | ToolApprovalMode | undefined)` to normalize consumer input the same way the SDK does
260
- internally (`undefined` / `false` → `undefined`, `true` → `'serial'`, strings pass through, unknown strings throw). New
261
- code should configure `AgentConfig.toolPolicies` and set `batchApprovals` for the UX axis instead.
251
+ | Field | Type | Description |
252
+ | ----------------- | ------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
253
+ | `abortSignal?` | `AbortSignal` | Abort the streaming operation. |
254
+ | `batchApprovals?` | `boolean` | When `true`, parallel approval-requests within a turn surface on the same stream so the consumer can render a batch approval card (requires Pattern A iterators — collect-all-then-settle). Defaults to `false` (serial). Controls only the serial-vs-batch emission UX; the gating on/off decision lives on `AgentConfig.toolPolicies` / `AgentConfig.defaultToolDecision`. No effect when no tool in the turn resolves to `'require-approval'`. |
255
+ | `maxSteps?` | `number` | Maximum number of LLM call steps the agent may take per `stream()` invocation. Each step is one LLM call (which may produce text, tool calls, or both). Must be `>= 1`. Defaults to `DEFAULT_MAX_STEPS` (1024) high enough to be effectively unlimited for real tasks; the practical ceiling is the context window and cost. The constant is exported so consumers and harness authors share one source of truth. |
262
256
 
263
257
  #### Tool Approval Policy
264
258
 
@@ -722,21 +716,16 @@ consumers can branch on tool hints (e.g. auto-approve `readOnlyHint`) or group b
722
716
  reparsing namespaced tool names. `annotations` is `undefined` when the source did not declare them (per the MCP spec);
723
717
  `serverName` is `undefined` when the tool is not from an MCP server.
724
718
 
725
- `requireToolApproval` accepts a `boolean` (`true` is shorthand for `'serial'`) or one of the mode strings exposed via
726
- the `ToolApprovalMode` type alias (`'serial' | 'batch'`):
727
-
728
- - **`'serial'`** (the safe default) — `tool-approval-request` events surface one at a time. The next request (if any)
729
- arrives later on the same `eventStream` after the consumer settles the current one.
730
- - **`'batch'`** — when the model emits parallel `tool_use` blocks, all approval-requests for that batch surface on the
731
- same `eventStream` together, so the consumer can render a batch-approval card. The consumer must gather all approvals
732
- before settling — a `break`-on-first-approval loop will hang the turn because Mastra/Claude won't continue emitting
733
- tool-results until the consumer has settled the entire batch.
719
+ Gating is configured on the agent via `AgentConfig.toolPolicies` / `AgentConfig.defaultToolDecision` (see "Tool Approval
720
+ Policy" above). The per-call `StreamOptions.batchApprovals` flag controls only how the resulting `tool-approval-request`
721
+ events are emitted:
734
722
 
735
- The SDK also exports `resolveToolApprovalMode(value)`the canonical
736
- `boolean | ToolApprovalMode | undefined ToolApprovalMode | undefined` normalizer harness implementations should use to
737
- dispatch (`undefined` / `false` `undefined`, `true` `'serial'`, strings pass through). Reject unknown strings via
738
- `throw` instead of reimplementing the boolean-vs-string branch the helper's defensive throw catches `as any` consumers
739
- passing invalid values that would otherwise silently degrade to "approval gating on but no coordinator allocated."
723
+ - **`batchApprovals: false`** (the safe default) — `tool-approval-request` events surface one at a time. The next
724
+ request (if any) arrives later on the same `eventStream` after the consumer settles the current one.
725
+ - **`batchApprovals: true`** when the model emits parallel `tool_use` blocks, all approval-requests for that batch
726
+ surface on the same `eventStream` together, so the consumer can render a batch-approval card. The consumer must gather
727
+ all approvals before settling a `break`-on-first-approval loop will hang the turn because Mastra/Claude won't
728
+ continue emitting tool-results until the consumer has settled the entire batch.
740
729
 
741
730
  #### Single stream per turn
742
731
 
@@ -749,12 +738,12 @@ in-line as approval-requests / consumer-tool-calls arrive.
749
738
 
750
739
  #### Pattern: settle in-line, continue iterating
751
740
 
752
- The same shape works for `'serial'` and `'batch'` mode:
741
+ The same shape works for serial and batch emission. Configure gating on the agent
742
+ (`defaultToolDecision: 'require-approval'`, or per-tool `toolPolicies`); the `chat()` call needs no approval flag:
753
743
 
754
744
  ```typescript
755
- const { eventStream } = await session.chat('Run the deployment', {
756
- requireToolApproval: true, // or 'serial' / 'batch'
757
- });
745
+ // Agent configured with e.g. { defaultToolDecision: 'require-approval' }
746
+ const { eventStream } = await session.chat('Run the deployment');
758
747
 
759
748
  for await (const event of eventStream) {
760
749
  if (event.type === 'tool-approval-request') {
@@ -789,12 +778,13 @@ for await (const event of eventStream) {
789
778
 
790
779
  #### Pattern variant: batch-collect approvals before settling
791
780
 
792
- Same single-stream loop, just gather the batch before deciding (useful for "approve these N tools?" UI cards under
793
- `requireToolApproval: 'batch'`):
781
+ Same single-stream loop, just gather the batch before deciding (useful for "approve these N tools?" UI cards). Set
782
+ `batchApprovals: true` on the `chat()` call; gating itself stays on the agent's `toolPolicies` / `defaultToolDecision`:
794
783
 
795
784
  ```typescript
785
+ // Agent configured with e.g. { defaultToolDecision: 'require-approval' }
796
786
  const { eventStream } = await session.chat('Run the deployment', {
797
- requireToolApproval: 'batch',
787
+ batchApprovals: true,
798
788
  });
799
789
 
800
790
  const pendingBatch: ToolApprovalRequestEvent[] = [];
@@ -873,8 +863,8 @@ for await (const event of eventStream) {
873
863
  }
874
864
  ```
875
865
 
876
- When `requireToolApproval: true` is also set, consumer-executed tools bypass the approval gate by construction (per
877
- `StreamOptions.requireToolApproval` JSDoc). They surface as a normal `tool-call` event without a preceding
866
+ When approval gating is active (via `AgentConfig.toolPolicies` / `defaultToolDecision`), consumer-executed tools bypass
867
+ the approval gate by construction. They surface as a normal `tool-call` event without a preceding
878
868
  `tool-approval-request`. Built-in / MCP tools still gate normally.
879
869
 
880
870
  ### Tool-Result Redaction
@@ -75,17 +75,12 @@ export type AgentConfig = {
75
75
  * the common cases. Per-MCP-server policy is expressed here via
76
76
  * `{ matcher: { type: 'mcp', serverName } }` rules — `MCPServerConfig`
77
77
  * carries no policy fields.
78
- *
79
- * Has no effect until a harness wires the resolver into its gate site
80
- * (Phase 2). Until then, harnesses gate via the deprecated
81
- * {@link StreamOptions.requireToolApproval}.
82
78
  */
83
79
  toolPolicies?: ToolPolicyRule[];
84
80
  /**
85
81
  * Fallback decision for tool invocations that match no rule in
86
82
  * {@link toolPolicies} (or any built-in / harness tier). Defaults to
87
- * `'allow'` for back-compat ("no policy ⇒ no gating," matching today's
88
- * behavior when `requireToolApproval` is unset).
83
+ * `'allow'` ("no policy ⇒ no gating").
89
84
  *
90
85
  * Set to `'require-approval'` for a fail-closed posture — recommended for
91
86
  * tenants whose MCP catalog includes un-annotated servers, since
@@ -126,84 +121,12 @@ export type HarnessAgentConfig = Omit<AgentConfig, 'orgAlias'> & {
126
121
  * `test/harness/harness-config.test.ts` that asserts unknown fields survive.
127
122
  */
128
123
  export declare function toHarnessConfig(config: AgentConfig, orgJwt?: JSONWebToken): HarnessAgentConfig;
129
- /**
130
- * Approval-mode selector for `StreamOptions.requireToolApproval`.
131
- *
132
- * Distinguishes the legacy "serial" UX (one approval per stream;
133
- * consumer settles before the next is surfaced) from the parallel
134
- * "batch" UX (all approval-requests for a parallel `tool_use` batch
135
- * surface on the same stream so the consumer can render them as a
136
- * batch approval card). See `requireToolApproval` for the safety
137
- * note on choosing `batch`.
138
- *
139
- * @deprecated Tool-approval gating is moving to per-tool policy on
140
- * {@link AgentConfig.toolPolicies}, resolved by `resolveToolApprovalPolicy`.
141
- * The serial-vs-batch UX axis survives as {@link StreamOptions.batchApprovals}.
142
- * Removed in a future major once harnesses and consumers have migrated.
143
- */
144
- export type ToolApprovalMode = 'serial' | 'batch';
145
- /**
146
- * Resolves `StreamOptions.requireToolApproval` to its canonical mode:
147
- * `undefined` (gating off), `'serial'`, or `'batch'`. Centralizes the
148
- * boolean-vs-string normalization so harnesses don't duplicate the
149
- * resolution logic.
150
- *
151
- * Semantics:
152
- * - `undefined` / `false` → `undefined` (no gating).
153
- * - `true` → `'serial'` (back-compat shorthand for the original `boolean` shape).
154
- * - `'serial'` → `'serial'` (explicit, equivalent to `true`).
155
- * - `'batch'` → `'batch'`.
156
- *
157
- * @deprecated Superseded by per-tool policy on {@link AgentConfig.toolPolicies}
158
- * (resolved by `resolveToolApprovalPolicy`) plus {@link StreamOptions.batchApprovals}
159
- * for the UX axis. Kept functional for one release so harness PRs can migrate
160
- * independently; removed in a future major.
161
- */
162
- export declare function resolveToolApprovalMode(requireToolApproval: boolean | ToolApprovalMode | undefined): ToolApprovalMode | undefined;
163
124
  /**
164
125
  * Per-call options controlling streaming behavior.
165
126
  */
166
127
  export type StreamOptions = {
167
128
  /** Signal to abort the streaming operation. */
168
129
  abortSignal?: AbortSignal;
169
- /**
170
- * When set, the harness requires human approval before executing any
171
- * native tool (e.g., MCP tools). The stream emits a `tool-approval-request`
172
- * event and suspends until the consumer calls `approveToolCall()` or
173
- * `declineToolCall()`.
174
- *
175
- * Accepts a `boolean` (back-compatible shorthand) or one of the
176
- * approval-mode strings:
177
- *
178
- * - **`true` or `'serial'`** (the safe default): each chat-stream
179
- * surfaces ONE `tool-approval-request` at a time. The consumer
180
- * settles the approval; the next `tool-approval-request` (if any)
181
- * appears on the continuation stream. Identical to the SDK's
182
- * behavior before parallel-approval UX (#447) — safe for consumers
183
- * whose iterator returns on the first approval-request and
184
- * re-iterates the continuation (Pattern B).
185
- *
186
- * - **`'batch'`**: when the model emits parallel `tool_use` blocks, the
187
- * broker surfaces ALL approval-requests on the same stream so the
188
- * consumer can render a batch approval UI ("Approve these N tools?").
189
- * Consumers MUST iterate to natural park collecting approvals
190
- * (Pattern A); a `break`-on-first-approval loop will miss the
191
- * subsequent approvals on the same stream and the chat will hang.
192
- * Only opt into `'batch'` after the consumer's iterator collects all
193
- * approvals before settling.
194
- *
195
- * Does not affect consumer-executed tools (those defined via
196
- * `AgentConfig.tools` without an execute handler) — the consumer
197
- * already controls execution for those via `submitToolResult()`.
198
- *
199
- * @deprecated Per-call all-or-nothing gating is superseded by per-tool
200
- * policy on {@link AgentConfig.toolPolicies}. The serial-vs-batch UX axis
201
- * moved to {@link batchApprovals}. Kept functional for one release so
202
- * harness PRs can migrate independently; removed in a future major. New
203
- * code should configure `AgentConfig.toolPolicies` and, if a batch UX is
204
- * wanted, set `batchApprovals: true`.
205
- */
206
- requireToolApproval?: boolean | ToolApprovalMode;
207
130
  /**
208
131
  * When `true`, parallel approval-requests within a turn (the model emits
209
132
  * multiple `tool_use` blocks in one batch, more than one of which resolves
@@ -212,14 +135,12 @@ export type StreamOptions = {
212
135
  * natural park before settling.
213
136
  *
214
137
  * Defaults to `false` (serial: one approval-request at a time; the consumer
215
- * settles before the next surfaces). Identical to pre-#447 behavior under
216
- * `requireToolApproval: true`.
138
+ * settles before the next surfaces).
217
139
  *
218
140
  * Has no effect when no tool in the turn resolves to `'require-approval'` —
219
- * there is nothing to batch. This is the self-documenting replacement for
220
- * the UX half of the deprecated {@link requireToolApproval} enum; the
221
- * "gating on/off" half moves to per-tool policy on
222
- * {@link AgentConfig.toolPolicies}.
141
+ * there is nothing to batch. The "gating on/off" decision lives on per-tool
142
+ * policy in {@link AgentConfig.toolPolicies} / {@link AgentConfig.defaultToolDecision};
143
+ * this flag controls only the serial-vs-batch emission UX.
223
144
  */
224
145
  batchApprovals?: boolean;
225
146
  /**
@@ -24,36 +24,6 @@ export function toHarnessConfig(config, orgJwt) {
24
24
  const { orgAlias: _, ...rest } = config;
25
25
  return { ...rest, orgJwt };
26
26
  }
27
- /**
28
- * Resolves `StreamOptions.requireToolApproval` to its canonical mode:
29
- * `undefined` (gating off), `'serial'`, or `'batch'`. Centralizes the
30
- * boolean-vs-string normalization so harnesses don't duplicate the
31
- * resolution logic.
32
- *
33
- * Semantics:
34
- * - `undefined` / `false` → `undefined` (no gating).
35
- * - `true` → `'serial'` (back-compat shorthand for the original `boolean` shape).
36
- * - `'serial'` → `'serial'` (explicit, equivalent to `true`).
37
- * - `'batch'` → `'batch'`.
38
- *
39
- * @deprecated Superseded by per-tool policy on {@link AgentConfig.toolPolicies}
40
- * (resolved by `resolveToolApprovalPolicy`) plus {@link StreamOptions.batchApprovals}
41
- * for the UX axis. Kept functional for one release so harness PRs can migrate
42
- * independently; removed in a future major.
43
- */
44
- export function resolveToolApprovalMode(requireToolApproval) {
45
- if (requireToolApproval === undefined || requireToolApproval === false)
46
- return undefined;
47
- if (requireToolApproval === true)
48
- return 'serial';
49
- if (requireToolApproval === 'serial' || requireToolApproval === 'batch')
50
- return requireToolApproval;
51
- // Defensive: an `as any` consumer could pass an unknown string. Without
52
- // this guard the value flows through to harness checks, which then
53
- // silently degrade to "approval gating on but no broker allocated" and
54
- // the chat hangs without an error a consumer can debug.
55
- throw new Error(`Invalid requireToolApproval value: ${JSON.stringify(requireToolApproval)}. Expected boolean, 'serial', or 'batch'.`);
56
- }
57
27
  /**
58
28
  * Default maximum steps for a single agent stream invocation.
59
29
  *
@@ -37,10 +37,9 @@
37
37
  * counterparts live only here.
38
38
  *
39
39
  * Symbols that consumers and harnesses both need (`AgentConfig`,
40
- * `StreamOptions`, `DEFAULT_MAX_STEPS`, `resolveToolApprovalMode`,
41
- * `ChatEvent` types, `Message` types, `MCPConfiguration`, etc.) stay on
42
- * the bare specifier. The split is "harness-only" vs. "consumer-AND-harness",
43
- * not "harness vs. consumer."
40
+ * `StreamOptions`, `DEFAULT_MAX_STEPS`, `ChatEvent` types, `Message`
41
+ * types, `MCPConfiguration`, etc.) stay on the bare specifier. The split
42
+ * is "harness-only" vs. "consumer-AND-harness", not "harness vs. consumer."
44
43
  */
45
44
  export type { AgentHarness, HarnessFactory, WithAgentConfig, ConfigOf } from './index.js';
46
45
  export type { AgentHooks } from '../types/redaction.js';
package/dist/index.d.ts CHANGED
@@ -5,8 +5,8 @@ export { BUILT_IN_TOOL_POLICIES, SKILL_BRIDGE_SERVER_ID, definePolicy, matcherMa
5
5
  export type { ResolverResult, ResolverTiers, ToolInvocation } from './policy-resolver.js';
6
6
  export type { ContextUsage, FinishReason, UsageMetadata } from './types/usage.js';
7
7
  export type { AgentHooks, HooksForAgent, ToolResultRedactor, ToolResultRedactionInput, ToolResultRedactionResult, } from './types/redaction.js';
8
- export type { AgentConfig, HarnessAgentConfig, StreamOptions, ToolApprovalMode } from './harness/harness-config.js';
9
- export { DEFAULT_MAX_STEPS, resolveToolApprovalMode } from './harness/harness-config.js';
8
+ export type { AgentConfig, HarnessAgentConfig, StreamOptions } from './harness/harness-config.js';
9
+ export { DEFAULT_MAX_STEPS } from './harness/harness-config.js';
10
10
  export type { MCPConfiguration, MCPServerConfig, MCPStdioServerConfig, MCPRemoteServerConfig, McpServerInfo, McpServerErrorCategory, McpServerErrorDetail, McpToolInfo, McpToolAnnotations, } from './mcp-config.js';
11
11
  export { McpServerStatus, mcpServerConfigEqual } from './mcp-config.js';
12
12
  export { Model, ModelName, createClaudeModel, Models, validateMultimodalFiles } from './models/index.js';
package/dist/index.js CHANGED
@@ -8,7 +8,7 @@
8
8
  // `resolveToolApprovalPolicy` into their gate sites (Phase 2); consumers author
9
9
  // `AgentConfig.toolPolicies` (directly or via `definePolicy`).
10
10
  export { BUILT_IN_TOOL_POLICIES, SKILL_BRIDGE_SERVER_ID, definePolicy, matcherMatches, resolveToolApprovalPolicy, } from './policy-resolver.js';
11
- export { DEFAULT_MAX_STEPS, resolveToolApprovalMode } from './harness/harness-config.js';
11
+ export { DEFAULT_MAX_STEPS } from './harness/harness-config.js';
12
12
  export { McpServerStatus, mcpServerConfigEqual } from './mcp-config.js';
13
13
  export { Model, ModelName, createClaudeModel, Models, validateMultimodalFiles } from './models/index.js';
14
14
  export { MimeType } from './models/index.js';
@@ -36,8 +36,10 @@ export type ResolverTiers = {
36
36
  /** Harness-shipped built-in rules (e.g. `MASTRA_BUILT_IN_TOOL_POLICIES`). */
37
37
  harness?: ReadonlyArray<ToolPolicyRule>;
38
38
  /**
39
- * Synthetic rules from harness factory inputs during a deprecation window.
40
- * Today: Claude's `bypassApprovalTools` translation. Empty after Phase 4.
39
+ * Reserved slot for synthetic rules a harness factory injects between the
40
+ * harness built-ins and the consumer's `toolPolicies`. No harness populates
41
+ * it today; it is kept so a future factory-injected tier can slot in
42
+ * without reshaping the resolver signature or the concatenation order.
41
43
  */
42
44
  factory?: ReadonlyArray<ToolPolicyRule>;
43
45
  };
@@ -21,8 +21,8 @@ import type { McpToolAnnotations } from '../mcp-config.js';
21
21
  * For harness-executed (native) tools such as MCP tools, use
22
22
  * `AgentConfig.mcpServers` — those run in the harness and never need
23
23
  * `submitToolResult`. To gate harness-executed tool calls with human approval,
24
- * use `StreamOptions.requireToolApproval` plus `approveToolCall` /
25
- * `declineToolCall`.
24
+ * configure `AgentConfig.toolPolicies` / `AgentConfig.defaultToolDecision`
25
+ * plus `approveToolCall` / `declineToolCall`.
26
26
  *
27
27
  * Structurally compatible with AI SDK `LanguageModelV4FunctionTool`.
28
28
  */
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@salesforce/sfdx-agent-sdk",
3
- "version": "0.32.0",
3
+ "version": "0.33.0",
4
4
  "description": "Harness-agnostic agentic infrastructure for Salesforce developer experience tooling",
5
5
  "type": "module",
6
6
  "main": "dist/index.js",
@@ -46,23 +46,23 @@
46
46
  },
47
47
  "devDependencies": {
48
48
  "@eslint/js": "^10.0.1",
49
- "@salesforce/sfdx-agent-harness-claude": "0.28.0",
50
- "@salesforce/sfdx-agent-harness-mastra": "0.31.0",
49
+ "@salesforce/sfdx-agent-harness-claude": "0.29.0",
50
+ "@salesforce/sfdx-agent-harness-mastra": "0.32.0",
51
51
  "@types/node": "^22.20.0",
52
- "@vitest/coverage-istanbul": "^4.1.8",
53
- "@vitest/eslint-plugin": "^1.6.20",
54
- "eslint": "^10.6.0",
52
+ "@vitest/coverage-istanbul": "^4.1.10",
53
+ "@vitest/eslint-plugin": "^1.6.23",
54
+ "eslint": "^10.7.0",
55
55
  "eslint-config-prettier": "^10.1.8",
56
56
  "eslint-import-resolver-typescript": "^4.4.5",
57
57
  "eslint-plugin-import": "^2.32.0",
58
- "eslint-plugin-n": "^18.2.1",
58
+ "eslint-plugin-n": "^18.2.2",
59
59
  "globals": "^17.6.0",
60
60
  "lint-staged": "^17.0.7",
61
61
  "prettier": "^3.9.4",
62
62
  "rimraf": "^6.1.3",
63
63
  "tsx": "^4.23.0",
64
64
  "typescript": "^6.0.3",
65
- "typescript-eslint": "^8.62.1",
65
+ "typescript-eslint": "^8.63.0",
66
66
  "vitest": "^4.1.8"
67
67
  },
68
68
  "engines": {