Skip to content

test(ai-gateway): verify autoFreeModels share the same AI SDK provider#4642

Merged
chrarnoldus merged 2 commits into
mainfrom
test-auto-free-models-ai-sdk-provider
Jul 20, 2026
Merged

test(ai-gateway): verify autoFreeModels share the same AI SDK provider#4642
chrarnoldus merged 2 commits into
mainfrom
test-auto-free-models-ai-sdk-provider

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Adds a test to apps/web/src/lib/ai-gateway/models.test.ts verifying that all autoFreeModels resolve to the same AI SDK provider via getAiSdkProvider in model-settings.ts. The expected shared provider is the openrouter default (getAiSdkProvider returns undefined).

All autoFreeModels must resolve to the same ai_sdk_provider in
model-settings (undefined, i.e. the openrouter default).

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

kilo-code-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The follow-up commit only relaxes the new unit test to check that all autoFreeModels resolve to the same AI SDK provider, without asserting which provider it is; no issues found in the changed code.

Files Reviewed (1 files)
  • apps/web/src/lib/ai-gateway/models.test.ts
Previous Review Summary (commit c5e41e2)

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

Previous review (commit c5e41e2)

Status: No Issues Found | Recommendation: Merge

Executive Summary

The PR adds a single, well-scoped unit test verifying that all autoFreeModels resolve to the same AI SDK provider, with no issues found in the changed code.

Files Reviewed (1 files)
  • apps/web/src/lib/ai-gateway/models.test.ts

Reviewed by claude-sonnet-5 · Input: 16 · Output: 3K · Cached: 357.1K

Review guidance: REVIEW.md from base branch main

Drop the assertion that the shared provider is the openrouter default;
the models only need to resolve to the same provider, whichever it is.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus
chrarnoldus merged commit da921ce into main Jul 20, 2026
15 checks passed
@chrarnoldus
chrarnoldus deleted the test-auto-free-models-ai-sdk-provider branch July 20, 2026 15:20
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