fix(anthropic): restore OAuth guard in service-tier stream wrappers

This commit is contained in:
openperf
2026-04-06 12:06:23 +08:00
committed by Josh Lehman
parent 0c63fccc1e
commit 35b4b04e51
2 changed files with 152 additions and 0 deletions

View File

@@ -3,6 +3,8 @@ import { afterEach, describe, expect, it, vi } from "vitest";
import {
__testing,
createAnthropicBetaHeadersWrapper,
createAnthropicFastModeWrapper,
createAnthropicServiceTierWrapper,
wrapAnthropicProviderStream,
} from "./stream-wrappers.js";
@@ -46,6 +48,33 @@ describe("anthropic stream wrappers", () => {
expect(warn).not.toHaveBeenCalled();
});
it("skips service_tier for OAuth token in composed stream chain", () => {
const captured: { headers?: Record<string, string>; payload?: Record<string, unknown> } = {};
const base: StreamFn = (model, _context, options) => {
captured.headers = options?.headers;
const payload = {} as Record<string, unknown>;
options?.onPayload?.(payload as never, model as never);
captured.payload = payload;
return {} as never;
};
const wrapped = wrapAnthropicProviderStream({
streamFn: base,
modelId: "claude-sonnet-4-6",
extraParams: { context1m: true, serviceTier: "auto" },
} as never);
wrapped?.(
{ provider: "anthropic", api: "anthropic-messages", id: "claude-sonnet-4-6" } as never,
{} as never,
{ apiKey: "sk-ant-oat01-oauth-token" } as never,
);
expect(captured.headers?.["anthropic-beta"]).toContain(OAUTH_BETA);
expect(captured.headers?.["anthropic-beta"]).not.toContain(CONTEXT_1M_BETA);
expect(captured.payload?.service_tier).toBeUndefined();
});
it("composes the anthropic provider stream chain from extra params", () => {
const captured: { headers?: Record<string, string>; payload?: Record<string, unknown> } = {};
const base: StreamFn = (model, _context, options) => {
@@ -72,3 +101,118 @@ describe("anthropic stream wrappers", () => {
expect(captured.payload).toMatchObject({ service_tier: "auto" });
});
});
describe("createAnthropicFastModeWrapper", () => {
function runFastModeWrapper(params: {
apiKey?: string;
provider?: string;
api?: string;
baseUrl?: string;
enabled?: boolean;
}): Record<string, unknown> | undefined {
const captured: { payload?: Record<string, unknown> } = {};
const base: StreamFn = (_model, _context, options) => {
if (options?.onPayload) {
const payload: Record<string, unknown> = {};
options.onPayload(payload, _model);
captured.payload = payload;
}
return {} as never;
};
const wrapper = createAnthropicFastModeWrapper(base, params.enabled ?? true);
wrapper(
{
provider: params.provider ?? "anthropic",
api: params.api ?? "anthropic-messages",
baseUrl: params.baseUrl,
id: "claude-sonnet-4-6",
} as never,
{} as never,
{ apiKey: params.apiKey } as never,
);
return captured.payload;
}
it("does not inject service_tier for OAuth token", () => {
const payload = runFastModeWrapper({ apiKey: "sk-ant-oat01-test-token" });
expect(payload?.service_tier).toBeUndefined();
});
it("injects service_tier for regular API keys", () => {
const payload = runFastModeWrapper({ apiKey: "sk-ant-api03-test-key" });
expect(payload?.service_tier).toBe("auto");
});
it("injects service_tier=standard_only when disabled for API keys", () => {
const payload = runFastModeWrapper({ apiKey: "sk-ant-api03-test-key", enabled: false });
expect(payload?.service_tier).toBe("standard_only");
});
it("does not inject service_tier for non-anthropic provider", () => {
const payload = runFastModeWrapper({
apiKey: "sk-ant-api03-test-key",
provider: "openai",
api: "openai-completions",
});
expect(payload?.service_tier).toBeUndefined();
});
});
describe("createAnthropicServiceTierWrapper", () => {
function runServiceTierWrapper(params: {
apiKey?: string;
provider?: string;
api?: string;
serviceTier?: "auto" | "standard_only";
}): Record<string, unknown> | undefined {
const captured: { payload?: Record<string, unknown> } = {};
const base: StreamFn = (_model, _context, options) => {
if (options?.onPayload) {
const payload: Record<string, unknown> = {};
options.onPayload(payload, _model);
captured.payload = payload;
}
return {} as never;
};
const wrapper = createAnthropicServiceTierWrapper(base, params.serviceTier ?? "auto");
wrapper(
{
provider: params.provider ?? "anthropic",
api: params.api ?? "anthropic-messages",
id: "claude-sonnet-4-6",
} as never,
{} as never,
{ apiKey: params.apiKey } as never,
);
return captured.payload;
}
it("does not inject service_tier for OAuth token", () => {
const payload = runServiceTierWrapper({ apiKey: "sk-ant-oat01-test-token" });
expect(payload?.service_tier).toBeUndefined();
});
it("injects service_tier for regular API keys", () => {
const payload = runServiceTierWrapper({ apiKey: "sk-ant-api03-test-key" });
expect(payload?.service_tier).toBe("auto");
});
it("injects service_tier=standard_only for regular API keys", () => {
const payload = runServiceTierWrapper({
apiKey: "sk-ant-api03-test-key",
serviceTier: "standard_only",
});
expect(payload?.service_tier).toBe("standard_only");
});
it("does not inject service_tier for non-anthropic provider", () => {
const payload = runServiceTierWrapper({
apiKey: "sk-ant-api03-test-key",
provider: "openai",
api: "openai-completions",
});
expect(payload?.service_tier).toBeUndefined();
});
});

View File

@@ -152,6 +152,10 @@ export function createAnthropicFastModeWrapper(
const underlying = baseStreamFn ?? streamSimple;
const serviceTier = resolveAnthropicFastServiceTier(enabled);
return (model, context, options) => {
if (isAnthropicOAuthApiKey(options?.apiKey)) {
return underlying(model, context, options);
}
const payloadPolicy = resolveAnthropicPayloadPolicy({
provider: typeof model.provider === "string" ? model.provider : undefined,
api: typeof model.api === "string" ? model.api : undefined,
@@ -174,6 +178,10 @@ export function createAnthropicServiceTierWrapper(
): StreamFn {
const underlying = baseStreamFn ?? streamSimple;
return (model, context, options) => {
if (isAnthropicOAuthApiKey(options?.apiKey)) {
return underlying(model, context, options);
}
const payloadPolicy = resolveAnthropicPayloadPolicy({
provider: typeof model.provider === "string" ? model.provider : undefined,
api: typeof model.api === "string" ? model.api : undefined,