fix(ai-sdk): Add model mapping to azure#892
Conversation
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed everything up to 4a1e238 in 10 seconds. Click for details.
- Reviewed
39lines of code in2files - Skipped
0files when reviewing. - Skipped posting
0draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
Workflow ID: wflow_EmMUbYeyfJISrniR
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis change adds Azure vendor prefix recognition to the AI SDK tracing transformations and extends test coverage to validate Azure-specific provider string transformations and normalization. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Comment |
Fixes: TLP-1863
Important
Add Azure to AI SDK vendor mapping and test its transformation to Azure system.
azuretoVENDOR_MAPPINGinai-sdk-transformations.tsto map toAzure.ai-sdk-transformations.test.tsto verify transformation ofazureprovider toAzuresystem.azureprovider transformation inai-sdk-transformations.test.ts.azure.chat,azure.completions, andazureare correctly transformed.This description was created by
for 4a1e238. You can customize this summary. It will automatically update as commits are pushed.
Summary by CodeRabbit