Skip to content

feat(ai-gateway): inject extra models for all Vercel providers#4624

Merged
chrarnoldus merged 2 commits into
mainfrom
inject-extra-provider-models
Jul 20, 2026
Merged

feat(ai-gateway): inject extra models for all Vercel providers#4624
chrarnoldus merged 2 commits into
mainfrom
inject-extra-provider-models

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

Renames injectExtraUserByokModels to injectExtraProviderModels in the OpenRouter provider sync and broadens its matching from VercelUserByokInferenceProviderIdSchema to VercelInferenceProviderIdSchema, so all code-defined Vercel inference providers (not just the user BYOK subset) qualify for injecting missing models into the per-provider model lists.

Verification

  • Not manually tested — behavior change is a schema-swap in the sync path; relying on CI (typecheck/lint/tests) and automated review.

Visual Changes

N/A

Reviewer Notes

VercelInferenceProviderIdSchema is the union of VercelUserByokInferenceProviderIdSchema and VercelNonUserByokInferenceProviderIdSchema (see inference-provider-id.ts), so previously-excluded providers like vertex, azure, cerebras, etc. now also get missing models injected.

Rename injectExtraUserByokModels to injectExtraProviderModels and match
endpoints/providers against VercelInferenceProviderIdSchema (the union of
all code-defined Vercel inference providers) instead of only the user
BYOK subset.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus self-assigned this Jul 19, 2026
@kilo-code-bot

kilo-code-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • apps/web/src/lib/ai-gateway/providers/openrouter/sync-providers.ts (incremental: only a formatting change since last review, no logic modified)
Previous Review Summary (commit 2eeb49b)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 2eeb49b)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • apps/web/src/lib/ai-gateway/providers/openrouter/sync-providers.ts

Reviewed by claude-sonnet-5 · Input: 20 · Output: 2.4K · Cached: 352.9K

Review guidance: REVIEW.md from base branch main

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus
chrarnoldus merged commit 8f808d8 into main Jul 20, 2026
15 checks passed
@chrarnoldus
chrarnoldus deleted the inject-extra-provider-models branch July 20, 2026 08:44
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.

2 participants