fix(app): support anthropic models on azure cognitive services - #9000
fix(app): support anthropic models on azure cognitive services#9000uanandaraja wants to merge 2 commits into
Conversation
Sorry, something went wrong.
|
The following comment was made by an LLM, it may be inaccurate: Based on my search, I found one potentially related PR: Related PR:
The PR #8566 could be worth reviewing to ensure there's no overlap with your current fix. The other results returned were the current PR itself (#9000) or unrelated feature PRs. |
Sorry, something went wrong.
|
/review |
Sorry, something went wrong.
|
lgtm |
Sorry, something went wrong.
|
are we good here? @rekram1-node |
Sorry, something went wrong.
|
I actually have a simpler solution if u wanna test it |
Sorry, something went wrong.
it says RESOURCE NOT FOUND, different env variable? 
|
Sorry, something went wrong.
|
Hm may need a tweak but do you see what I'm trying to do? everything should live in that one module it shouldnt require edits outside of it |
Sorry, something went wrong.
| }, | ||
| ) | ||
|
|
||
| // Azure Cognitive Services with Claude models uses Anthropic SDK |
There was a problem hiding this comment.
this change shouldnt be needed
Sorry, something went wrong.
|
|
||
| export function providerOptions(model: Provider.Model, options: { [x: string]: any }) { | ||
| // Azure Cognitive Services with Claude models uses Anthropic provider options | ||
| const isAzureClaude = model.providerID === "azure-cognitive-services" && model.api.id.includes("claude") |
There was a problem hiding this comment.
this change shouldnt be needed
Sorry, something went wrong.
| // Anthropic rejects messages with empty content - filter out empty string messages | ||
| // and remove empty text/reasoning parts from array content | ||
| if (model.api.npm === "@ai-sdk/anthropic") { | ||
| const isAzureClaude = model.providerID === "azure-cognitive-services" && model.api.id.includes("claude") |
There was a problem hiding this comment.
this shouldnt be needed
Sorry, something went wrong.
| if (id.includes("deepseek") || id.includes("minimax") || id.includes("glm") || id.includes("mistral")) return {} | ||
|
|
||
| // Azure Cognitive Services with Claude models uses Anthropic thinking config | ||
| const isAzureClaude = model.providerID === "azure-cognitive-services" && model.api.id.includes("claude") |
There was a problem hiding this comment.
i dont think this should be needed
Sorry, something went wrong.
| // Special case: google-vertex-anthropic uses a subpath import | ||
| const bundledKey = | ||
| model.providerID === "google-vertex-anthropic" ? "@ai-sdk/google-vertex/anthropic" : model.api.npm | ||
| model.providerID === "google-vertex-anthropic" |
There was a problem hiding this comment.
This shouldnt be needed
Sorry, something went wrong.
| } | ||
|
|
||
| // Special case: Azure Cognitive Services with Anthropic models uses @ai-sdk/anthropic | ||
| const isAzureClaude = model.providerID === "azure-cognitive-services" && model.api.id.includes("claude") |
There was a problem hiding this comment.
you would also need to handle if the resourceName is being passed as a provider option for the provider
Sorry, something went wrong.
When using Anthropic models (like claude-opus-4.5) via Azure Cognitive Services, the code was using @ai-sdk/azure SDK which tries to call sdk.responses(modelID). But @ai-sdk/azure doesn't support Anthropic models - Azure's Anthropic endpoint uses a different API structure that requires @ai-sdk/anthropic.