Skip to content

Add codex-auto-review to OpenAI default models#2951

Merged
Wei-Shaw merged 1 commit into
Wei-Shaw:mainfrom
EricLi404:00_lyy/add-codex-auto-review-model
Jun 6, 2026
Merged

Add codex-auto-review to OpenAI default models#2951
Wei-Shaw merged 1 commit into
Wei-Shaw:mainfrom
EricLi404:00_lyy/add-codex-auto-review-model

Conversation

@EricLi404

Copy link
Copy Markdown
Contributor

Summary

  • add codex-auto-review to the OpenAI default /v1/models fallback catalog
  • add a focused regression test so the default model list keeps exposing it

Validation

  • go test ./internal/pkg/openai ./internal/handler -run 'TestDefaultModelsContainsCodexAutoReview|TestGatewayModels'\n- make test-unit\n- git diff --check\n\n## Notes\n- golangci-lint run ./... was started locally with golangci-lint 2.11.4, but it ran for several minutes without output and was stopped; CI uses v2.9 and should provide the authoritative lint result.

Copilot AI review requested due to automatic review settings June 1, 2026 10:22
@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA. ✅
Posted by the CLA Assistant Lite bot.

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds the codex-auto-review model to the OpenAI default model list and introduces a regression test to ensure it remains included.

Changes:

  • Added codex-auto-review entry to DefaultModels
  • Added a unit test to verify DefaultModels contains codex-auto-review

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
backend/internal/pkg/openai/constants.go Adds codex-auto-review to the DefaultModels slice.
backend/internal/pkg/openai/constants_test.go Adds a test ensuring DefaultModels includes codex-auto-review.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@EricLi404

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@EricLi404 EricLi404 force-pushed the 00_lyy/add-codex-auto-review-model branch from 2b54f4f to fd7758d Compare June 1, 2026 10:34
github-actions Bot added a commit that referenced this pull request Jun 1, 2026
@EricLi404 EricLi404 force-pushed the 00_lyy/add-codex-auto-review-model branch from fd7758d to 2400613 Compare June 1, 2026 10:38
Include codex-auto-review in the OpenAI fallback models list so /v1/models exposes it when no account mapping is configured. Keep the entry aligned with the existing default model catalog.
@EricLi404 EricLi404 force-pushed the 00_lyy/add-codex-auto-review-model branch from 2400613 to bd0b1ff Compare June 1, 2026 10:46
@Wei-Shaw Wei-Shaw merged commit c06ad75 into Wei-Shaw:main Jun 6, 2026
2 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants