feat: update tool choice handling and enhance response interfaces

Author: caozhiyuan

src/routes/messages/responses-translation.ts

@@ -19,6 +19,8 @@ import {
   type ResponseFunctionToolCallItem,
   type ResponseFunctionCallOutputItem,
   type Tool,
+  type ToolChoiceFunction,
+  type ToolChoiceOptions,
 } from "~/services/copilot/create-responses"
 
 import {
@@ -315,9 +317,9 @@ const convertAnthropicTools = (
 
 const convertAnthropicToolChoice = (
   choice: AnthropicMessagesPayload["tool_choice"],
-): unknown => {
+): ToolChoiceOptions | ToolChoiceFunction => {
   if (!choice) {
-    return undefined
+    return "auto"
   }
 
   switch (choice.type) {
@@ -328,13 +330,13 @@ const convertAnthropicToolChoice = (
       return "required"
     }
     case "tool": {
-      return choice.name ? { type: "function", name: choice.name } : undefined
+      return choice.name ? { type: "function", name: choice.name } : "auto"
     }
     case "none": {
       return "none"
     }
     default: {
-      return undefined
+      return "auto"
     }
   }
 }
@@ -541,9 +543,6 @@ const mapResponsesStopReason = (
     if (incompleteDetails?.reason === "content_filter") {
       return "end_turn"
     }
-    if (incompleteDetails?.reason === "tool_use") {
-      return "tool_use"
-    }
   }
 
   return null

src/services/copilot/create-responses.ts

@@ -10,11 +10,11 @@ export interface ResponsesPayload {
   instructions?: string | null
   input?: string | Array<ResponseInputItem>
   tools?: Array<Tool> | null
-  tool_choice?: unknown
+  tool_choice?: ToolChoiceOptions | ToolChoiceFunction
   temperature?: number | null
   top_p?: number | null
   max_output_tokens?: number | null
-  metadata?: Record<string, unknown> | null
+  metadata?: Metadata | null
   stream?: boolean | null
   safety_identifier?: string | null
   prompt_cache_key?: string | null
@@ -25,6 +25,13 @@ export interface ResponsesPayload {
   [key: string]: unknown
 }
 
+export type ToolChoiceOptions = "none" | "auto" | "required"
+
+export interface ToolChoiceFunction {
+  name: string
+  type: "function"
+}
+
 export type Tool = FunctionTool
 
 export interface FunctionTool {
@@ -112,16 +119,22 @@ export interface ResponsesResult {
   status: string
   usage?: ResponseUsage | null
   error: ResponseError | null
-  incomplete_details: Record<string, unknown> | null
+  incomplete_details: IncompleteDetails | null
   instructions: string | null
-  metadata: Record<string, unknown> | null
+  metadata: Metadata | null
   parallel_tool_calls: boolean
   temperature: number | null
   tool_choice: unknown
-  tools: Array<Record<string, unknown>>
+  tools: Array<Tool>
   top_p: number | null
 }
 
+export type Metadata = { [key: string]: string }
+
+export interface IncompleteDetails {
+  reason?: "max_output_tokens" | "content_filter"
+}
+
 export interface ResponseError {
   message: string
 }

tests/translation.test.ts renamed to tests/responses-translation.test.ts

@@ -50,7 +50,6 @@ describe("translateAnthropicMessagesToResponsesPayload", () => {
   it("converts anthropic text blocks into response input messages", () => {
     const result = translateAnthropicMessagesToResponsesPayload(samplePayload)
 
-    console.log("result:", JSON.stringify(result, null, 2))
     expect(Array.isArray(result.input)).toBe(true)
     const input = result.input as Array<ResponseInputMessage>
     expect(input).toHaveLength(1)
@@ -81,7 +80,7 @@ describe("translateResponsesResultToAnthropic", () => {
         {
           id: "reason_1",
           type: "reasoning",
-          summary: [{ type: "text", text: "Thinking about the task." }],
+          summary: [{ type: "summary_text", text: "Thinking about the task." }],
           status: "completed",
           encrypted_content: "encrypted_reasoning_content",
         },
@@ -116,7 +115,7 @@ describe("translateResponsesResultToAnthropic", () => {
         total_tokens: 156,
       },
       error: null,
-      incomplete_details: { reason: "tool_use" },
+      incomplete_details: { reason: "content_filter" },
       instructions: null,
       metadata: null,
       parallel_tool_calls: false,
@@ -129,7 +128,7 @@ describe("translateResponsesResultToAnthropic", () => {
     const anthropicResponse =
       translateResponsesResultToAnthropic(responsesResult)
 
-    expect(anthropicResponse.stop_reason).toBe("tool_use")
+    expect(anthropicResponse.stop_reason).toBe("end_turn")
     expect(anthropicResponse.content).toHaveLength(3)
 
     const [thinkingBlock, toolUseBlock, textBlock] = anthropicResponse.content