Files
9router/tests/translator/bugs-kiro.test.js
T
Edison42 cef5dd4d61 fix(kiro): map GPT-5.6 reasoning effort fields
Route GPT-5.6 reasoning effort through Kiro's native reasoning.effort field instead of the legacy Claude output_config.effort path. GPT-5.6 models now emit reasoning.effort for low/medium/high/xhigh, with max mapped to the xhigh wire value.

Preserve the Responses API reasoning.effort through the OpenAI intermediate by copying it to reasoning_effort before the field is dropped. Skip legacy thinking_mode prompt tags when a supported native GPT effort is emitted, while keeping the legacy fallback for unsupported values (auto/minimal/ultra) and explicit disable semantics (none/off/disabled). Claude adaptive effort continues to use thinking plus output_config.effort.
2026-07-20 11:11:37 +07:00

71 lines
2.4 KiB
JavaScript

// OpenAI → Kiro (AWS CodeWhisperer) request translation.
import { describe, it, expect } from "vitest";
import "./registerAll.js";
import { translateRequest } from "../../open-sse/translator/index.js";
import { FORMATS } from "../../open-sse/translator/formats.js";
const O2K = (body) => translateRequest(FORMATS.OPENAI, FORMATS.KIRO, "m", body, true, null, "kiro");
const R2K = (model, body) => translateRequest(
FORMATS.OPENAI_RESPONSES,
FORMATS.KIRO,
model,
body,
true,
null,
"kiro"
);
describe("OpenAI → Kiro", () => {
it.each([
["high", "gpt-5.6-sol"],
["medium", "gpt-5.6-terra"],
["low", "gpt-5.6-luna"],
])("preserves Responses reasoning.effort %s through the full Kiro route", (effort, model) => {
const out = R2K(model, {
input: "Use the requested effort",
reasoning: { effort },
});
expect(out.additionalModelRequestFields).toEqual({
reasoning: { effort },
});
expect(out.systemPrompt || "").not.toContain("<thinking_mode>");
expect(out.systemPrompt || "").not.toContain("<max_thinking_length>");
});
// openai-to-kiro.js — safeJSONParse guards bad tool-call JSON (fixed in PR #1582)
it("malformed tool arguments do not throw the whole request", () => {
expect(() =>
O2K({
messages: [
{ role: "user", content: "go" },
{ role: "assistant", content: "", tool_calls: [
{ id: "c1", type: "function", function: { name: "f", arguments: "{not json" } },
] },
{ role: "tool", tool_call_id: "c1", content: "r" },
],
})
).not.toThrow();
});
// openai-to-kiro.js:309 — maxTokens hardcoded to 32000, ignores body.max_tokens
// KNOWN BUG
it.fails("respects client max_tokens", () => {
const out = O2K({ max_tokens: 100, messages: [{ role: "user", content: "hi" }] });
expect(out.inferenceConfig?.maxTokens, "client max_tokens ignored").toBe(100);
});
// openai-to-kiro.js:132-134 — remote http image becomes "[Image: url]" text (lost)
// KNOWN BUG
it.fails("remote image url is preserved as an image, not text", () => {
const out = O2K({
messages: [{ role: "user", content: [
{ type: "text", text: "see" },
{ type: "image_url", image_url: { url: "https://x.com/p.png" } },
] }],
});
const content = out.conversationState?.currentMessage?.userInputMessage?.content || "";
expect(content, "remote image flattened to text").not.toContain("[Image:");
});
});