fix(responses): type fixes + error handling + route tests (#2, #7)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: HXYerror

src/routes/responses/handler.ts

@@ -5,11 +5,14 @@ import consola from "consola"
 import type { ResponsesPayload } from "./types"
 
 export async function handleResponses(c: Context): Promise<Response> {
-  const payload = await c.req.json<ResponsesPayload>()
-  consola.debug("Responses API request payload:", JSON.stringify(payload))
+  try {
+    const payload = await c.req.json<ResponsesPayload>()
+    consola.debug("Responses API request payload:", JSON.stringify(payload))
+  } catch {
+    consola.debug("Responses API request received (could not parse body)")
+  }
 
   // TODO(#4): wire up createResponses() service client
-  // For now return a structured 501 so the route is exercisable
   consola.warn(
     "POST /v1/responses is not yet implemented — service client pending (#4)",
   )

src/routes/responses/types.ts

@@ -13,7 +13,7 @@ export interface ResponsesPayload {
   top_p?: number | null
   max_output_tokens?: number | null
   reasoning?: {
-    effort?: "minimal" | "low" | "medium" | "high"
+    effort?: "low" | "medium" | "high"
     summary?: "auto" | "concise" | "detailed"
   } | null
   previous_response_id?: string | null
@@ -30,6 +30,7 @@ export interface ResponsesPayload {
 // Input item types (union)
 export type ResponsesInputItem =
   | ResponsesInputMessage
+  | ResponsesInputFunctionCall
   | ResponsesFunctionCallOutput
   | ResponsesReasoningItem
 
@@ -41,6 +42,15 @@ export interface ResponsesInputMessage {
   status?: "completed" | "incomplete" | null
 }
 
+export interface ResponsesInputFunctionCall {
+  type: "function_call"
+  id?: string
+  call_id: string
+  name: string
+  arguments: string
+  status?: "completed" | "in_progress" | "incomplete" | null
+}
+
 export interface ResponsesFunctionCallOutput {
   type: "function_call_output"
   call_id: string
@@ -57,7 +67,12 @@ export interface ResponsesReasoningItem {
 
 export type ResponsesContentPart =
   | { type: "input_text"; text: string }
-  | { type: "input_image"; image_url: string; detail?: "low" | "high" | "auto" }
+  | {
+      type: "input_image"
+      image_url?: string | null
+      file_id?: string | null
+      detail?: "low" | "high" | "auto"
+    }
 
 // Tool types
 export interface ResponsesTool {

src/services/copilot/create-chat-completions.ts

@@ -150,7 +150,7 @@ export interface ChatCompletionsPayload {
     | { type: "function"; function: { name: string } }
     | null
   user?: string | null
-  reasoning_effort?: "minimal" | "low" | "medium" | "high" | null
+  reasoning_effort?: "low" | "medium" | "high" | null
 }
 
 export interface Tool {

tests/responses-route.test.ts

@@ -0,0 +1,39 @@
+import { describe, test, expect } from "bun:test"
+
+import { server } from "../src/server"
+
+describe("POST /v1/responses stub", () => {
+  test("returns 501 with structured error body", async () => {
+    const res = await server.request("/v1/responses", {
+      method: "POST",
+      headers: { "Content-Type": "application/json" },
+      body: JSON.stringify({ model: "gpt-4o", input: [] }),
+    })
+    expect(res.status).toBe(501)
+    const body = (await res.json()) as {
+      error: { type: string; code: string; message: string }
+    }
+    expect(body.error.type).toBe("not_implemented")
+    expect(body.error.code).toBe("responses_not_implemented")
+    expect(typeof body.error.message).toBe("string")
+  })
+
+  test("bare /responses path also returns 501", async () => {
+    const res = await server.request("/responses", {
+      method: "POST",
+      headers: { "Content-Type": "application/json" },
+      body: JSON.stringify({ model: "gpt-4o", input: [] }),
+    })
+    expect(res.status).toBe(501)
+    const body = (await res.json()) as { error: { code: string } }
+    expect(body.error.code).toBe("responses_not_implemented")
+  })
+
+  test("empty body returns 501 (not 500)", async () => {
+    const res = await server.request("/v1/responses", {
+      method: "POST",
+      // no body
+    })
+    expect(res.status).toBe(501)
+  })
+})