Skip to content

Fix OpenRouter model ID mismatches for GPT-5 family#5570

Merged
H2Shami merged 5 commits intomainfrom
fix-openrouter-model-ids
Feb 20, 2026
Merged

Fix OpenRouter model ID mismatches for GPT-5 family#5570
H2Shami merged 5 commits intomainfrom
fix-openrouter-model-ids

Conversation

@replicas-connector
Copy link
Contributor

Summary

  • Fix gpt-5.1-chat-latest:openrouter providerModelId from openai/gpt-5.1-chat-latest to openai/gpt-5.1-chat
  • Fix gpt-5.2-chat-latest:openrouter providerModelId from openai/gpt-5.2-chat-latest to openai/gpt-5.2-chat
  • Remove misplaced codex-mini-latest:openrouter endpoint from gpt-5.1 config (the codex-mini-latest:openai endpoint is unaffected)

Note: gpt-5/endpoints.ts already had the correct value (openai/gpt-5-chat), and chatgpt-4o-latest was not touched as it is correct.

Test plan

  • Verify OpenRouter cost calculations for gpt-5.1-chat-latest and gpt-5.2-chat-latest resolve correctly
  • Confirm no regression in other GPT-5 family endpoint lookups

🤖 Generated with Claude Code

…ints

The OpenRouter providerModelId values for gpt-5.1-chat-latest and
gpt-5.2-chat-latest incorrectly included "-latest" suffix, which doesn't
match the actual OpenRouter model IDs. Also removes the misplaced
codex-mini-latest:openrouter endpoint from the gpt-5.1 config.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Feb 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
helicone Ready Ready Preview, Comment Feb 20, 2026 11:00pm
helicone-bifrost Ready Ready Preview, Comment Feb 20, 2026 11:00pm
helicone-eu Ready Ready Preview, Comment Feb 20, 2026 11:00pm

Request Review

@supabase
Copy link

supabase bot commented Feb 16, 2026

This pull request has been ignored for the connected project bolqqmqbrciybnypvklh because there are no changes detected in supbase/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@supabase
Copy link

supabase bot commented Feb 16, 2026

This pull request has been ignored for the connected project lmahfbbnchpworytrrqk because there are no changes detected in .supabase/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@claude
Copy link
Contributor

claude bot commented Feb 16, 2026

Claude finished @replicas-connector[bot]'s task —— View job


Code Review Complete ✅

Reviewing PR for OpenRouter model ID fixes and providing constructive feedback.

Score: 8/10 - Good to merge with minor considerations

Summary of Changes

✅ Primary Fixes Are Correct

  • Fixed gpt-5.1-chat-latest:openrouter providerModelId: openai/gpt-5.1-chat-latestopenai/gpt-5.1-chat
  • Fixed gpt-5.2-chat-latest:openrouter providerModelId: openai/gpt-5.2-chat-latestopenai/gpt-5.2-chat
  • Removed misplaced codex-mini-latest:openrouter endpoint from GPT-5.1 config

✅ Critical Bug Fix: Duplicate Import Resolution
The PR resolves a critical duplicate import issue that was causing build failures:

✅ Test Snapshots Updated Correctly
The 187-line snapshot removal properly reflects the deleted gemini-3.1-pro endpoints.

Suggestions Summary:

Minor Improvements:

  1. Consider adding integration test: Add a test case that verifies OpenRouter model ID resolution for the fixed endpoints
  2. Documentation: Consider updating any documentation that references the old model IDs

No Critical Issues Found:

  • No security vulnerabilities
  • No breaking API changes
  • Model pricing and parameters remain consistent
  • No performance regressions

Recommendation

Ready to merge - This PR fixes both the intended OpenRouter model ID mismatches and resolves the unintended duplicate import issue causing CI failures. The changes are minimal, targeted, and preserve all existing functionality while fixing real bugs.


Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@replicas-connector
Copy link
Contributor Author

CI/CD Failure

Workflow Packages Test failed on commit c81f169. Investigating.

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.

1 participant