refactor(models): use catalog capabilities for forced tool support

Author: j15z

apps/sim/providers/anthropic/core.request.test.ts

@@ -296,13 +296,8 @@ describe('executeAnthropicProviderRequest forced tool use', () => {
     expect(payload.tool_choice).toEqual({ type: 'tool', name: 'publish' })
   })
 
-  it.each([
-    'claude-fable-5-1',
-    'claude-fable-5-1-20260901',
-    'azure-anthropic/claude-fable-5-1',
-    'claude-mythos-5-1',
-  ])('drops forced tool_choice on %s because the API rejects it', async (model) => {
-    const { payload, warn } = await runWithForcedTool(model)
+  it('drops forced tool_choice when the catalog model disables Force', async () => {
+    const { payload, warn } = await runWithForcedTool('claude-fable-5-1')
     expect(payload.tools?.map((tool) => tool.name)).toEqual(['publish'])
     expect(payload).not.toHaveProperty('tool_choice')
     expect(warn).toHaveBeenCalledWith(expect.stringContaining('rejects forced tool_choice'))

apps/sim/providers/models.test.ts

@@ -16,62 +16,18 @@ import {
   PROVIDER_DEFINITIONS,
   supportsForcedToolUse,
   updateFireworksModels,
-  updateOpenRouterModels,
 } from '@/providers/models'
 import { supportsPromptCaching } from '@/providers/utils'
 
 describe('forced tool use capability', () => {
-  it('keeps Auto and None support when a model disables Force', () => {
-    expect(getModelCapabilities('claude-fable-5-1')).toMatchObject({
-      toolUsageControl: true,
-      forcedToolUse: false,
-    })
-  })
-
-  it.each([
-    'claude-fable-5-1',
-    'CLAUDE-FABLE-5-1-20260901',
-    'azure-anthropic/claude-fable-5-1',
-    'azure-anthropic/claude-fable-5-1-20260901',
-    'bedrock/anthropic.claude-fable-5-1-v1:0',
-    'claude-mythos-5-1',
-    'claude-mythos-5-1-20260901',
-  ])('disables forced tool use for %s', (model) => {
-    expect(getModelCapabilities(model)).toMatchObject({
-      toolUsageControl: true,
-      forcedToolUse: false,
-    })
-    expect(supportsForcedToolUse(model)).toBe(false)
-  })
-
-  it('adds only known alias capabilities to provider defaults', () => {
-    expect(getModelCapabilities('claude-mythos-5-1')).toEqual({
-      ...PROVIDER_DEFINITIONS.anthropic.capabilities,
-      forcedToolUse: false,
-    })
-  })
-
-  it('inherits alias capabilities for dynamic models and respects explicit overrides', () => {
-    const originalModels = PROVIDER_DEFINITIONS.openrouter.models
-    const modelId = 'anthropic/claude-fable-5-1'
-    try {
-      updateOpenRouterModels([modelId])
-      expect(getModelCapabilities(modelId)?.forcedToolUse).toBe(false)
-      expect(supportsForcedToolUse(modelId)).toBe(false)
-
-      PROVIDER_DEFINITIONS.openrouter.models[0].capabilities.forcedToolUse = true
-      expect(getModelCapabilities(modelId)?.forcedToolUse).toBe(true)
-      expect(supportsForcedToolUse(modelId)).toBe(true)
-    } finally {
-      PROVIDER_DEFINITIONS.openrouter.models = originalModels
-    }
-  })
-
-  it.each(['claude-fable-5-10', 'claude-mythos-5-10', 'claude-not-fable-5-1'])(
-    'does not apply alias restrictions to unrelated model %s',
+  it.each(['claude-fable-5-1', 'CLAUDE-FABLE-5-1'])(
+    'disables Force while keeping Auto and None support for %s',
     (model) => {
-      expect(getModelCapabilities(model)).toEqual(PROVIDER_DEFINITIONS.anthropic.capabilities)
-      expect(supportsForcedToolUse(model)).toBe(true)
+      expect(getModelCapabilities(model)).toMatchObject({
+        toolUsageControl: true,
+        forcedToolUse: false,
+      })
+      expect(supportsForcedToolUse(model)).toBe(false)
     }
   )
 

apps/sim/providers/models.ts

@@ -164,16 +164,6 @@ export function getProviderFileAttachment(providerId: string): ProviderFileAttac
   return PROVIDER_DEFINITIONS[providerId]?.fileAttachment ?? DEFAULT_FILE_ATTACHMENT
 }
 
-const CLAUDE_5_1_TOOL_CAPABILITIES = { forcedToolUse: false } as const satisfies ModelCapabilities
-
-/** Known capabilities for model aliases; explicit catalog capabilities take precedence. */
-const MODEL_CAPABILITY_FALLBACKS = [
-  {
-    pattern: /(?:^|[/.])claude-(?:fable|mythos)-5-1(?:$|[-:])/,
-    capabilities: CLAUDE_5_1_TOOL_CAPABILITIES,
-  },
-] satisfies { pattern: RegExp; capabilities: ModelCapabilities }[]
-
 export const PROVIDER_DEFINITIONS: Record<string, ProviderDefinition> = {
   fireworks: {
     id: 'fireworks',
@@ -846,7 +836,7 @@ export const PROVIDER_DEFINITIONS: Record<string, ProviderDefinition> = {
           updatedAt: '2026-09-01',
         },
         capabilities: {
-          ...CLAUDE_5_1_TOOL_CAPABILITIES,
+          forcedToolUse: false,
           nativeStructuredOutputs: true,
           maxOutputTokens: 128000,
           promptCaching: { minimumCacheableTokens: 512 },
@@ -4307,36 +4297,25 @@ export function getModelPricing(modelId: string): ModelPricing | null {
 }
 
 export function getModelCapabilities(modelId: string): ModelCapabilities | null {
-  const normalizedModel = modelId.toLowerCase()
-  const fallbackCapabilities = MODEL_CAPABILITY_FALLBACKS.find(({ pattern }) =>
-    pattern.test(normalizedModel)
-  )?.capabilities
-
   for (const provider of Object.values(PROVIDER_DEFINITIONS)) {
-    const model = provider.models.find((m) => m.id.toLowerCase() === normalizedModel)
+    const model = provider.models.find((m) => m.id.toLowerCase() === modelId.toLowerCase())
     if (model) {
-      const capabilities: ModelCapabilities = {
-        ...provider.capabilities,
-        ...fallbackCapabilities,
-        ...model.capabilities,
-      }
+      const capabilities: ModelCapabilities = { ...provider.capabilities, ...model.capabilities }
       return capabilities
     }
   }
 
   for (const provider of Object.values(PROVIDER_DEFINITIONS)) {
     if (provider.modelPatterns) {
       for (const pattern of provider.modelPatterns) {
-        if (pattern.test(normalizedModel)) {
-          return fallbackCapabilities
-            ? { ...provider.capabilities, ...fallbackCapabilities }
-            : provider.capabilities || null
+        if (pattern.test(modelId.toLowerCase())) {
+          return provider.capabilities || null
         }
       }
     }
   }
 
-  return fallbackCapabilities || null
+  return null
 }
 
 export function getModelsWithTemperatureSupport(): string[] {