Skip to content

Route OpenAI reasoning and GPT-3.5 model IDs - #511

Open
cupkk wants to merge 1 commit into
google:mainfrom
cupkk:fix-openai-model-routing-492
Open

Route OpenAI reasoning and GPT-3.5 model IDs#511
cupkk wants to merge 1 commit into
google:mainfrom
cupkk:fix-openai-model-routing-492

Conversation

@cupkk

@cupkk cupkk commented Aug 5, 2026

Copy link
Copy Markdown

Description

Add the missing OpenAI routing patterns for GPT-3.5 and the o-series reasoning models. Without these patterns, valid model IDs such as o3-mini fall through provider resolution before the OpenAI provider can be initialized.

The registry test uses the centralized pattern tuple and verifies representative IDs from both families.

Fixes #492

Bug fix

How Has This Been Tested?

$ python3 <stdlib pattern contract>
stdlib pattern contract: PASS

$ python3 -m py_compile langextract/providers/patterns.py tests/registry_test.py

The full project test suite was not run locally because this focused routing change can be checked with the standard library, while importing the complete package would require installing the project's Google SDK, pandas, NumPy, and other base dependencies. GitHub CI is expected to run the repository test matrix.

Checklist

  • I have read and acknowledged Google's Open Source Code of Conduct.
  • I have read the Contributing page.
  • Google CLA status must be confirmed by the repository check.
  • Code-owner agreement is not recorded in issue Bug: OpenAI o-series (o1/o3/o4-mini) and gpt-3.5 model IDs route to no provider #492; the implementation follows the issue's proposed two-pattern change.
  • No documentation change is needed for this routing-table fix.
  • I added a regression test.
  • Full pylint was not run locally; affected files follow the existing 80-column, 2-space style.

AI-assisted tooling was used for repository inspection, test design, and drafting. The code change and validation scope are described above.

@google-cla

google-cla Bot commented Aug 5, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@github-actions github-actions Bot added the size/XS Pull request with less than 50 lines changed label Aug 5, 2026
@github-actions

Copy link
Copy Markdown

⚠️ Branch Update Required

Your branch is 1 commits behind main. Please update your branch to ensure CI checks run with the latest code:

git fetch origin main
git merge origin/main
git push

Note: Enable "Allow edits by maintainers" to allow automatic updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Pull request with less than 50 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: OpenAI o-series (o1/o3/o4-mini) and gpt-3.5 model IDs route to no provider

1 participant