fix(providers): address PR #555 review feedback for Fable 5

Author: edelauna

packages/types/src/providers/bedrock.ts

@@ -606,6 +606,7 @@ export const BEDROCK_1M_CONTEXT_MODEL_IDS = [
 // - Claude Opus 4.5
 // - Claude Opus 4.6
 // - Claude Opus 4.7
+// - Claude Fable 5 (cross-region inference only — can only be used through an inference profile)
 export const BEDROCK_GLOBAL_INFERENCE_MODEL_IDS = [
 	"anthropic.claude-sonnet-4-20250514-v1:0",
 	"anthropic.claude-sonnet-4-5-20250929-v1:0",
@@ -615,6 +616,7 @@ export const BEDROCK_GLOBAL_INFERENCE_MODEL_IDS = [
 	"anthropic.claude-opus-4-6-v1",
 	"anthropic.claude-opus-4-7",
 	"anthropic.claude-opus-4-8",
+	"anthropic.claude-fable-5",
 ] as const
 
 // Amazon Bedrock Service Tier types

packages/types/src/providers/openrouter.ts

@@ -75,6 +75,7 @@ export const OPEN_ROUTER_REASONING_BUDGET_MODELS = new Set([
 	"anthropic/claude-opus-4.1",
 	"anthropic/claude-opus-4.5",
 	"anthropic/claude-opus-4.6",
+	"anthropic/claude-fable-5",
 	"anthropic/claude-sonnet-4",
 	"anthropic/claude-sonnet-4.5",
 	"anthropic/claude-sonnet-4.6",

src/api/providers/__tests__/bedrock.spec.ts

@@ -709,6 +709,19 @@ describe("AwsBedrockHandler", () => {
 			expect(model.info.supportsTemperature).toBe(false)
 			expect(model.maxTokens).toBe(8192)
 		})
+
+		it("should apply global inference prefix for Claude Fable 5 when awsUseGlobalInference is true", () => {
+			const handler = new AwsBedrockHandler({
+				apiModelId: "anthropic.claude-fable-5",
+				awsAccessKey: "test",
+				awsSecretKey: "test",
+				awsRegion: "us-east-1",
+				awsUseGlobalInference: true,
+			})
+
+			const model = handler.getModel()
+			expect(model.id).toBe("global.anthropic.claude-fable-5")
+		})
 	})
 
 	describe("1M context beta feature", () => {

src/api/providers/fetchers/__tests__/openrouter.spec.ts

@@ -314,6 +314,8 @@ describe("OpenRouter API", () => {
 			expect(result.maxTokens).toBe(128000)
 			expect(result.contextWindow).toBe(1000000)
 			expect(result.supportsTemperature).toBe(false)
+			expect(result.supportsReasoningBudget).toBe(true)
+			expect(result.supportsReasoningBinary).toBe(true)
 		})
 
 		it("sets horizon-alpha model to 32k max tokens", () => {

src/api/providers/fetchers/__tests__/requesty.spec.ts

@@ -0,0 +1,67 @@
+// npx vitest run api/providers/fetchers/__tests__/requesty.spec.ts
+
+import axios from "axios"
+
+import { getRequestyModels } from "../requesty"
+
+vi.mock("axios")
+const mockAxiosGet = vi.mocked(axios.get)
+
+function makeRawModel(overrides: Record<string, unknown>) {
+	return {
+		id: "some/model",
+		max_output_tokens: 8192,
+		context_window: 200000,
+		supports_caching: false,
+		supports_vision: false,
+		supports_reasoning: false,
+		input_price: "0.000003",
+		output_price: "0.000015",
+		description: "Test model",
+		caching_price: null,
+		cached_price: null,
+		...overrides,
+	}
+}
+
+describe("getRequestyModels", () => {
+	it("applies Fable 5 overrides when parsing anthropic/claude-fable-5", async () => {
+		const rawFable5 = makeRawModel({
+			id: "anthropic/claude-fable-5",
+			max_output_tokens: 128000,
+			context_window: 1000000,
+			supports_caching: true,
+			supports_vision: true,
+			supports_reasoning: true,
+			input_price: "0.00001",
+			output_price: "0.00005",
+			caching_price: "0.0000125",
+			cached_price: "0.000001",
+		})
+
+		mockAxiosGet.mockResolvedValueOnce({ data: { data: [rawFable5] } })
+
+		const models = await getRequestyModels()
+		const fable5 = models["anthropic/claude-fable-5"]
+
+		expect(fable5).toBeDefined()
+		expect(fable5.supportsReasoningBudget).toBe(true)
+		expect(fable5.supportsReasoningBinary).toBe(true)
+		expect(fable5.supportsTemperature).toBe(false)
+	})
+
+	it("does not apply Fable 5 overrides to other models", async () => {
+		const rawSonnet = makeRawModel({
+			id: "anthropic/claude-sonnet-4.6",
+			supports_reasoning: true,
+		})
+
+		mockAxiosGet.mockResolvedValueOnce({ data: { data: [rawSonnet] } })
+
+		const models = await getRequestyModels()
+		const sonnet = models["anthropic/claude-sonnet-4.6"]
+
+		expect(sonnet.supportsReasoningBinary).toBeUndefined()
+		expect(sonnet.supportsTemperature).toBeUndefined()
+	})
+})

src/api/providers/fetchers/openrouter.ts

@@ -266,6 +266,7 @@ export const parseOpenRouterModel = ({
 	// Set claude-fable-5 model to use the correct Anthropic configuration
 	if (id === "anthropic/claude-fable-5") {
 		modelInfo.maxTokens = anthropicModels["claude-fable-5"].maxTokens
+		modelInfo.supportsReasoningBinary = true
 		modelInfo.supportsTemperature = false
 	}
 

src/api/transform/model-params.ts

@@ -183,8 +183,6 @@ export function getModelParams({
 		// Special case for o1-pro, which doesn't support temperature.
 		// Note that OpenRouter's `supported_parameters` field includes
 		// `temperature`, which is probably a bug.
-		// TODO: Add a `supportsTemperature` field to the model info and populate
-		// it appropriately in the OpenRouter fetcher.
 		if (modelId === "openai/o1-pro") {
 			params.temperature = undefined
 		}