Skip to content

fix(ai-gateway): add versioned gateway models route#4384

Merged
chrarnoldus merged 1 commit into
mainfrom
session/agent_d36ecf64-9472-4e69-887e-acbb0646fbda
Jul 3, 2026
Merged

fix(ai-gateway): add versioned gateway models route#4384
chrarnoldus merged 1 commit into
mainfrom
session/agent_d36ecf64-9472-4e69-887e-acbb0646fbda

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a versioned /api/gateway/v1/models Next.js route that reuses the existing OpenRouter models handler, matching the existing unversioned gateway models alias. Adds a regression assertion that the versioned gateway route exports the shared handler.

Verification

N/A - no manual UI flow; this is an API route alias.

Visual Changes

N/A

Reviewer Notes

Local test execution requires the test Postgres service, which was unavailable in this container because Docker is not installed.

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

kilo-code-bot Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The new /api/gateway/v1/models route is a one-line re-export that exactly matches the existing alias pattern used by other gateway routes, with a straightforward regression test confirming it points at the shared handler.

Files Reviewed (2 files)
  • apps/web/src/app/api/gateway/v1/models/route.ts
  • apps/web/src/tests/openrouter-models.test.ts

Reviewed by claude-sonnet-5-20260630 · Input: 14 · Output: 2.9K · Cached: 268.3K

Review guidance: REVIEW.md from base branch main

@chrarnoldus chrarnoldus merged commit 7d70363 into main Jul 3, 2026
15 checks passed
@chrarnoldus chrarnoldus deleted the session/agent_d36ecf64-9472-4e69-887e-acbb0646fbda branch July 3, 2026 13:02
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