Skip to content

fix: use gpt-5.4 for openai oauth probes#1700

Open
CCLCK wants to merge 1 commit intoWei-Shaw:mainfrom
CCLCK:codex/openai-oauth-probe-model-fix
Open

fix: use gpt-5.4 for openai oauth probes#1700
CCLCK wants to merge 1 commit intoWei-Shaw:mainfrom
CCLCK:codex/openai-oauth-probe-model-fix

Conversation

@CCLCK
Copy link
Copy Markdown

@CCLCK CCLCK commented Apr 16, 2026

Summary

  • add a shared helper to choose a probe-safe OpenAI test model by account type
  • use gpt-5.4 for ChatGPT OAuth probe/test flows while keeping the package default for other account types
  • add unit coverage for the helper and for empty-model OpenAI OAuth account tests

Test Plan

  • cd backend && go test ./internal/service -run 'Test(DefaultOpenAITestModelForAccount|AccountTestService_OpenAIOAuthEmptyModelUsesSafeDefault|AccountTestService_OpenAISuccessPersistsSnapshotFromHeaders|AccountTestService_OpenAI429PersistsSnapshotAndRateLimit)' -tags unit

@Wei-Shaw Wei-Shaw force-pushed the main branch 4 times, most recently from 1e0d466 to 4d0483f Compare April 22, 2026 10:12
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