Skip to content

chore: tidy up Antigravity Provider model list - #1627

Closed
backryun wants to merge 0 commit into
cheaper-inference:release/v3.7.1from
backryun:antigravitymodelfix
Closed

backryun wants to merge 0 commit into
cheaper-inference:release/v3.7.1from
backryun:antigravitymodelfix

Conversation

@backryun

Copy link
Copy Markdown
Contributor

Before
samsunginternet_Qa2nJSIjLO

After
image

Since this change doesn't significantly affect other areas, it's a very safe commit. Please merge it!

@backryun
backryun requested a review from diegosouzapw as a code owner April 26, 2026 17:24

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the public model list by replacing preview versions with stable releases and removing several Gemini-routed Claude models. A review comment highlights that removing model aliases from ANTIGRAVITY_MODEL_ALIASES constitutes a breaking change for existing integrations and recommends retaining them for backward compatibility.

I am having trouble creating individual review comments. Click here to see my feedback.

open-sse/config/antigravityModelAliases.ts (26-32)

medium

Removing these model aliases is a breaking change for existing chats or integrations that still reference the older model IDs (such as gemini-3-pro-preview or the gemini-claude-* routes). While it is appropriate to remove them from the ANTIGRAVITY_PUBLIC_MODELS list to clean up the user interface, these aliases should be maintained in ANTIGRAVITY_MODEL_ALIASES to ensure that legacy IDs continue to resolve correctly to their updated counterparts in the backend.

@backryun
backryun marked this pull request as draft April 26, 2026 17:46
@backryun backryun closed this Apr 26, 2026
@diegosouzapw

Copy link
Copy Markdown
Collaborator

Thanks @backryun for tidying up the Antigravity model aliases! 🎉 These changes were already integrated as part of a prior cleanup pass on the release branch, so the diff shows 0 changes — but your contribution kicked off the fix. Merging now so you get proper credit. This will ship in v3.7.1.

@backryun

Copy link
Copy Markdown
Contributor Author

Thanks @backryun for tidying up the Antigravity model aliases! 🎉 These changes were already integrated as part of a prior cleanup pass on the release branch, so the diff shows 0 changes — but your contribution kicked off the fix. Merging now so you get proper credit. This will ship in v3.7.1.

I just noticed a mistake in my work and am redoing it; I’ll upload the revised version shortly.

This was referenced Apr 30, 2026
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