Skip to content

fix(ai-gateway): normalize model ids for variants - #5048

Merged
chrarnoldus merged 2 commits into
mainfrom
fix/openrouter-variant-normalization
Aug 5, 2026
Merged

fix(ai-gateway): normalize model ids for variants#5048
chrarnoldus merged 2 commits into
mainfrom
fix/openrouter-variant-normalization

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • fall back to the normalized OpenRouter model ID when deriving reasoning variants
  • preserve exact model metadata lookup precedence

Validation

  • pnpm --filter web run typecheck passed
  • pnpm exec oxfmt --list-different apps/web/src/lib/ai-gateway/providers/model-settings.ts passed
  • git diff --check passed
  • targeted Jest suites were attempted but could not run because the local test PostgreSQL and Redis services are unavailable

Untracked .kilo/design/ tooling files were intentionally excluded.

@chrarnoldus chrarnoldus self-assigned this Aug 5, 2026
@kilo-code-bot

kilo-code-bot Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Single-line bug fix that adds a normalized-model-ID fallback to OpenRouter reasoning-variant lookup while preserving exact-match precedence; verified against normalizeModelId semantics and existing codebase conventions with high confidence.

Files Reviewed (1 files)
  • apps/web/src/lib/ai-gateway/providers/model-settings.ts

Reviewed by kimi-k3 · Input: 50.7K · Output: 6.2K · Cached: 362.5K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus merged commit 7470318 into main Aug 5, 2026
15 checks passed
@chrarnoldus
chrarnoldus deleted the fix/openrouter-variant-normalization branch August 5, 2026 17:34
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