Skip to content

Commit 703a296

Browse files
committed
chore: hide parallel tool calls experiment and disable feature
- Hide MULTIPLE_NATIVE_TOOL_CALLS from experimental settings UI - Force parallelToolCallsEnabled to false in Task.ts - Force isMultipleNativeToolCallsEnabled to false in presentAssistantMessage.ts - Feature is on hold, existing users with setting enabled will have it ignored
1 parent ce22901 commit 703a296

File tree

3 files changed

+8
-12
lines changed

3 files changed

+8
-12
lines changed

src/core/assistant-message/presentAssistantMessage.ts

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -478,13 +478,9 @@ export async function presentAssistantMessage(cline: Task) {
478478
const toolCallId = (block as any).id
479479
const toolProtocol = toolCallId ? TOOL_PROTOCOL.NATIVE : TOOL_PROTOCOL.XML
480480

481-
// Check experimental setting for multiple native tool calls
482-
const provider = cline.providerRef.deref()
483-
const state = await provider?.getState()
484-
const isMultipleNativeToolCallsEnabled = experiments.isEnabled(
485-
state?.experiments ?? {},
486-
EXPERIMENT_IDS.MULTIPLE_NATIVE_TOOL_CALLS,
487-
)
481+
// Multiple native tool calls feature is on hold - always disabled
482+
// Previously resolved from experiments.isEnabled(..., EXPERIMENT_IDS.MULTIPLE_NATIVE_TOOL_CALLS)
483+
const isMultipleNativeToolCallsEnabled = false
488484

489485
const pushToolResult = (content: ToolResponse) => {
490486
if (toolProtocol === TOOL_PROTOCOL.NATIVE) {

src/core/task/Task.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3610,11 +3610,9 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
36103610
})
36113611
}
36123612

3613-
// Resolve parallel tool calls setting from experiment (will move to per-API-profile setting later)
3614-
const parallelToolCallsEnabled = experiments.isEnabled(
3615-
state?.experiments ?? {},
3616-
EXPERIMENT_IDS.MULTIPLE_NATIVE_TOOL_CALLS,
3617-
)
3613+
// Parallel tool calls are disabled - feature is on hold
3614+
// Previously resolved from experiments.isEnabled(..., EXPERIMENT_IDS.MULTIPLE_NATIVE_TOOL_CALLS)
3615+
const parallelToolCallsEnabled = false
36183616

36193617
const metadata: ApiHandlerCreateMessageMetadata = {
36203618
mode: mode,

webview-ui/src/components/settings/ExperimentalSettings.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,8 @@ export const ExperimentalSettings = ({
5555
<Section>
5656
{Object.entries(experimentConfigsMap)
5757
.filter(([key]) => key in EXPERIMENT_IDS)
58+
// Hide MULTIPLE_NATIVE_TOOL_CALLS - feature is on hold
59+
.filter(([key]) => key !== "MULTIPLE_NATIVE_TOOL_CALLS")
5860
.map((config) => {
5961
if (config[0] === "MULTI_FILE_APPLY_DIFF") {
6062
return (

0 commit comments

Comments
 (0)