Skip to content

feat(inference-providers): graduate alibaba to user BYOK#2981

Draft
kilo-code-bot[bot] wants to merge 3 commits intomainfrom
graduate-alibaba-to-user-byok
Draft

feat(inference-providers): graduate alibaba to user BYOK#2981
kilo-code-bot[bot] wants to merge 3 commits intomainfrom
graduate-alibaba-to-user-byok

Conversation

@kilo-code-bot
Copy link
Copy Markdown
Contributor

@kilo-code-bot kilo-code-bot Bot commented May 1, 2026

Summary

  • Moves alibaba from VercelNonUserByokInferenceProviderIdSchema to VercelUserByokInferenceProviderIdSchema, exposing it as a user-BYOK provider via the Vercel AI Gateway.
  • Adds Alibaba to the display-name map in BYOKKeysManager so it appears in the BYOK UI.
  • Adds a UserByokTestModels entry using alibaba/qwen3-coder-plus as the default test model.
  • Updates the qwen model-prefix mapping to reference the new user-BYOK enum location.

Verification

Verified locally that VercelUserByokInferenceProviderIdSchema.enum.alibaba type-checks in getVercelInferenceProviderConfigForUserByok and inferVercelFirstPartyInferenceProviderForModel.

Visual Changes

N/A

Reviewer Notes

The Alibaba default test model was selected from the Vercel gateway catalog; alibaba/qwen3-coder-plus is served by the alibaba upstream provider, making it a natural round-trip check for a user-supplied Alibaba API key.

@kilo-code-bot
Copy link
Copy Markdown
Contributor Author

kilo-code-bot Bot commented May 1, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/components/organizations/byok/BYOKKeysManager.tsx
  • apps/web/src/lib/ai-gateway/providers/openrouter/inference-provider-id.ts

Reviewed by gpt-5.5-20260423 · 296,957 tokens

@chrarnoldus chrarnoldus marked this pull request as draft May 1, 2026 10:05
@chrarnoldus
Copy link
Copy Markdown
Contributor

Difference between Alibaba and Vercel handling of reasoning is a problem here (reasoning_content vs reasoning_details).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant