Skip to content

Retain allowlisted legacy model IDs - #6047

Merged
chrarnoldus merged 2 commits into
mainfrom
whitelist-legacy-id-mappings
Sep 10, 2026
Merged

Retain allowlisted legacy model IDs#6047
chrarnoldus merged 2 commits into
mainfrom
whitelist-legacy-id-mappings

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • partially revert fix: remove legacy OpenRouter alias handling #5943 by restoring legacy OpenRouter aliases that appear in the provided allowlist
  • retain hardcoded OpenRouter-to-Vercel mappings for allowlisted source IDs plus the two Mistral embedding IDs
  • exclude the two OpenAI transcription IDs from the legacy alias set
  • cover retained aliases, removed aliases, and mapping exceptions

Testing

  • Not run locally; CI will handle validation as requested.
  • git diff --check
  • formatted the four changed files with oxfmt

@chrarnoldus chrarnoldus self-assigned this Sep 10, 2026
@kilo-code-bot

kilo-code-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (4 files)
  • apps/web/src/lib/ai-gateway/providers/gateway-models-cache.ts
  • apps/web/src/lib/ai-gateway/providers/gateway-models-cache.test.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.test.ts
Previous Review Summary (commit 20df468)

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

Previous review (commit 20df468)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (4 files)
  • apps/web/src/lib/ai-gateway/providers/gateway-models-cache.ts
  • apps/web/src/lib/ai-gateway/providers/gateway-models-cache.test.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/mapModelIdToVercel.test.ts

Reviewed by grok-4.6 · Input: 80.7K · Output: 5.2K · Cached: 135.3K

Review guidance: REVIEW.md from base branch main

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