Skip to content

fix: update ChatGPT Codex model hints#317

Merged
Sun-sunshine06 merged 1 commit intomainfrom
fix/chatgpt-codex-model-hints
May 8, 2026
Merged

fix: update ChatGPT Codex model hints#317
Sun-sunshine06 merged 1 commit intomainfrom
fix/chatgpt-codex-model-hints

Conversation

@Sun-sunshine06
Copy link
Copy Markdown
Collaborator

Summary

  • Update the ChatGPT subscription Codex model hint list to start with gpt-5.5.
  • Remove stale gpt-5.1* and gpt-5.2-codex options from the static picker list.
  • Refresh stale chatgpt-codex provider entries on boot, resetting obsolete active models to gpt-5.5.
  • Update localized ChatGPT subscription copy to reference gpt-5.5.

Verification

  • pnpm --filter @open-codesign/desktop test -- codex-oauth-ipc.test.ts
  • pnpm --filter @open-codesign/desktop test -- ModelSwitcher.test.ts
  • pnpm --filter @open-codesign/desktop typecheck
  • pnpm --filter @open-codesign/i18n test
  • pnpm exec biome check apps/desktop/src/main/codex-oauth-ipc.ts apps/desktop/src/main/codex-oauth-ipc.test.ts packages/i18n/src/locales/en.json packages/i18n/src/locales/zh-CN.json packages/i18n/src/locales/es.json packages/i18n/src/locales/pt-BR.json

Note: full local pnpm lint is currently blocked by an unrelated untracked issues_summary.json formatting issue in this checkout; the touched files pass Biome.

@github-actions github-actions Bot added the area:desktop apps/desktop (Electron shell, renderer) label May 8, 2026
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Review mode: initial

Findings

  • [Minor] Missing changeset for user-visible changes — The PR updates the Codex model hint list (user-facing model picker order and default model) and modifies i18n descriptions across four locales. Project conventions require a changeset for user-visible changes. Without one, the changelog will not reflect this update and version bump automation may miss it.
    Suggested fix: Run pnpm changeset and commit the resulting markdown file with a message like chore: add changeset for Codex model hint update.

Summary

The PR correctly updates the ChatGPT Codex model hints to promote gpt-5.5 as the default, removes stale gpt-5.1* and gpt-5.2-codex entries, and updates the migration logic to refresh the provider entry and active model if needed. All four locale files are updated consistently. Test coverage is expanded appropriately to verify the new model list and migration behavior. No security, regression, or license issues found. The only missing piece is a changeset to track the user-visible change.

Open-CoDesign Bot

@Sun-sunshine06 Sun-sunshine06 merged commit 5d5c863 into main May 8, 2026
6 checks passed
@Sun-sunshine06 Sun-sunshine06 deleted the fix/chatgpt-codex-model-hints branch May 8, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:desktop apps/desktop (Electron shell, renderer)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant