From a70eda8d953e3443f013ed8a64c84877070d5089 Mon Sep 17 00:00:00 2001 From: Daniel Rodriguez Date: Wed, 21 Jan 2026 18:38:30 -0600 Subject: [PATCH] fix: include _noop tool in activeTools for LiteLLM proxy compatibility The _noop dummy tool was being added to satisfy LiteLLM's requirement for a tools parameter when message history contains tool calls, but it was filtered out from activeTools. This caused compaction to fail with 'Bedrock doesn't support tool calling without tools= param' error. --- packages/opencode/src/session/llm.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/opencode/src/session/llm.ts b/packages/opencode/src/session/llm.ts index e73f20403f37..55c9c4524737 100644 --- a/packages/opencode/src/session/llm.ts +++ b/packages/opencode/src/session/llm.ts @@ -208,7 +208,7 @@ export namespace LLM { topP: params.topP, topK: params.topK, providerOptions: ProviderTransform.providerOptions(input.model, params.options), - activeTools: Object.keys(tools).filter((x) => x !== "invalid" && x !== "_noop"), + activeTools: Object.keys(tools).filter((x) => x !== "invalid"), tools, maxOutputTokens, abortSignal: input.abort,