chore: remove remaining Bedrock non-native tool conversion

Author: hannesrudolph

src/api/providers/bedrock.ts

@@ -840,12 +840,10 @@ export class AwsBedrockHandler extends BaseProvider implements SingleCompletionH
 		usePromptCache: boolean = false,
 		modelInfo?: any,
 		conversationId?: string, // Optional conversation ID to track cache points across messages
-		useNativeTools: boolean = false, // Whether native tool calling is being used
+		_useNativeTools: boolean = false, // Deprecated: Bedrock converter is native-only now
 	): { system: SystemContentBlock[]; messages: Message[] } {
 		// First convert messages using shared converter for proper image handling
-		const convertedMessages = sharedConverter(anthropicMessages as Anthropic.Messages.MessageParam[], {
-			useNativeTools,
-		})
+		const convertedMessages = sharedConverter(anthropicMessages as Anthropic.Messages.MessageParam[])
 
 		// If prompt caching is disabled, return the converted messages directly
 		if (!usePromptCache) {

src/api/transform/__tests__/bedrock-converse-format.spec.ts

@@ -104,7 +104,7 @@ describe("convertToBedrockConverseMessages", () => {
 		}
 	})
 
-	it("converts tool use messages correctly when useNativeTools=true (native tools format)", () => {
+	it("converts tool use messages correctly (native tools format)", () => {
 		const messages: Anthropic.Messages.MessageParam[] = [
 			{
 				role: "assistant",
@@ -121,8 +121,7 @@ describe("convertToBedrockConverseMessages", () => {
 			},
 		]
 
-		// With useNativeTools: true, keeps tool_use as native format
-		const result = convertToBedrockConverseMessages(messages, { useNativeTools: true })
+		const result = convertToBedrockConverseMessages(messages)
 
 		if (!result[0] || !result[0].content) {
 			expect.fail("Expected result to have content")
@@ -177,7 +176,7 @@ describe("convertToBedrockConverseMessages", () => {
 		}
 	})
 
-	it("converts tool result messages to native format (useNativeTools: true)", () => {
+	it("converts tool result messages to native format", () => {
 		const messages: Anthropic.Messages.MessageParam[] = [
 			{
 				role: "user",
@@ -191,8 +190,7 @@ describe("convertToBedrockConverseMessages", () => {
 			},
 		]
 
-		// With useNativeTools: true, keeps tool_result as native format
-		const result = convertToBedrockConverseMessages(messages, { useNativeTools: true })
+		const result = convertToBedrockConverseMessages(messages)
 
 		if (!result[0] || !result[0].content) {
 			expect.fail("Expected result to have content")
@@ -247,7 +245,7 @@ describe("convertToBedrockConverseMessages", () => {
 		}
 	})
 
-	it("converts tool result messages with string content to native format (useNativeTools: true)", () => {
+	it("converts tool result messages with string content to native format", () => {
 		const messages: Anthropic.Messages.MessageParam[] = [
 			{
 				role: "user",
@@ -261,7 +259,7 @@ describe("convertToBedrockConverseMessages", () => {
 			},
 		]
 
-		const result = convertToBedrockConverseMessages(messages, { useNativeTools: true })
+		const result = convertToBedrockConverseMessages(messages)
 
 		if (!result[0] || !result[0].content) {
 			expect.fail("Expected result to have content")

src/api/transform/bedrock-converse-format.ts

@@ -25,14 +25,8 @@ interface BedrockMessageContent {
 /**
  * Convert Anthropic messages to Bedrock Converse format
  * @param anthropicMessages Messages in Anthropic format
- * @param options Optional configuration for conversion
- * @param options.useNativeTools When true (default), uses Bedrock's native toolUse/toolResult blocks
  */
-export function convertToBedrockConverseMessages(
-	anthropicMessages: Anthropic.Messages.MessageParam[],
-	options?: { useNativeTools?: boolean },
-): Message[] {
-	const useNativeTools = options?.useNativeTools ?? true
+export function convertToBedrockConverseMessages(anthropicMessages: Anthropic.Messages.MessageParam[]): Message[] {
 	return anthropicMessages.map((anthropicMessage) => {
 		// Map Anthropic roles to Bedrock roles
 		const role: ConversationRole = anthropicMessage.role === "assistant" ? "assistant" : "user"
@@ -104,7 +98,6 @@ export function convertToBedrockConverseMessages(
 			}
 
 			if (messageBlock.type === "tool_result") {
-				void useNativeTools
 				// Handle content field - can be string or array (native tool format)
 				if (messageBlock.content) {
 					// Content is a string