Fix OpenRouter reasoning instruction role

This commit is contained in:
PriNova
2026-05-30 11:15:30 +02:00
parent dbb9911a54
commit b55fe1bbd4
2 changed files with 47 additions and 2 deletions

View File

@@ -1075,6 +1075,7 @@ function detectCompat(model: Model<"openai-completions">): ResolvedOpenAIComplet
const isTogether =
provider === "together" || baseUrl.includes("api.together.ai") || baseUrl.includes("api.together.xyz");
const isMoonshot = provider === "moonshotai" || provider === "moonshotai-cn" || baseUrl.includes("api.moonshot.");
const isOpenRouter = provider === "openrouter" || baseUrl.includes("openrouter.ai");
const isCloudflareWorkersAI = provider === "cloudflare-workers-ai" || baseUrl.includes("api.cloudflare.com");
const isCloudflareAiGateway = provider === "cloudflare-ai-gateway" || baseUrl.includes("gateway.ai.cloudflare.com");
@@ -1101,7 +1102,7 @@ function detectCompat(model: Model<"openai-completions">): ResolvedOpenAIComplet
return {
supportsStore: !isNonStandard,
supportsDeveloperRole: !isNonStandard,
supportsDeveloperRole: !isNonStandard && !isOpenRouter,
supportsReasoningEffort: !isGrok && !isZai && !isMoonshot && !isTogether && !isCloudflareAiGateway,
supportsUsageInStreaming: true,
maxTokensField: useMaxTokens ? "max_tokens" : "max_completion_tokens",
@@ -1115,7 +1116,7 @@ function detectCompat(model: Model<"openai-completions">): ResolvedOpenAIComplet
? "zai"
: isTogether
? "together"
: provider === "openrouter" || baseUrl.includes("openrouter.ai")
: isOpenRouter
? "openrouter"
: "openai",
openRouterRouting: {},

View File

@@ -817,6 +817,50 @@ describe("openai-completions tool_choice", () => {
expect(writeCall).not.toHaveProperty("partialArgs");
});
it("uses system messages for OpenRouter reasoning model instructions", async () => {
const model = getModel("openrouter", "deepseek/deepseek-v4-pro")!;
let payload: unknown;
await streamSimple(
model,
{
systemPrompt: "Follow instructions.",
messages: [{ role: "user", content: "Hi", timestamp: Date.now() }],
},
{
apiKey: "test",
onPayload: (params: unknown) => {
payload = params;
},
},
).result();
const params = payload as { messages?: Array<{ role?: string }> };
expect(params.messages?.[0]?.role).toBe("system");
});
it("keeps developer messages for OpenAI reasoning model instructions", async () => {
const model = getModel("openai", "gpt-5.5")!;
let payload: unknown;
await streamSimple(
model,
{
systemPrompt: "Follow instructions.",
messages: [{ role: "user", content: "Hi", timestamp: Date.now() }],
},
{
apiKey: "test",
onPayload: (params: unknown) => {
payload = params;
},
},
).result();
const params = payload as { messages?: Array<{ role?: string }> };
expect(params.messages?.[0]?.role).toBe("developer");
});
it("stores Xiaomi MiMo reasoning replay compat in built-in metadata", () => {
const providers = ["xiaomi", "xiaomi-token-plan-cn", "xiaomi-token-plan-ams", "xiaomi-token-plan-sgp"] as const;