Skip to content

Switch default OpenAI model to gpt-5.4-mini#673

Closed
timrogers wants to merge 2 commits into
mainfrom
codex/switch-default-openai-model-to-gpt-5.4-mini-aftjuc
Closed

Switch default OpenAI model to gpt-5.4-mini#673
timrogers wants to merge 2 commits into
mainfrom
codex/switch-default-openai-model-to-gpt-5.4-mini-aftjuc

Conversation

@timrogers

Copy link
Copy Markdown
Owner

Motivation

  • Use a newer OpenAI chat-completions model by default, switching the configured model from gpt-4o to gpt-5.4-mini.

Description

  • Update the OPENAI_MODEL constant in src/llm.rs from "gpt-4o" to "gpt-5.4-mini".

Testing

  • Ran git diff --check (clean), attempted pre-commit run --all-files (failed because pre-commit is not installed), and attempted cargo test --locked --all-features and cargo build --release --locked (both failed because the local rustc 1.89.0 is below the repository MSRV 1.94).

Codex Task

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

This PR updates the OpenAI chat-completions default model used by the formanator LLM integration by changing the OPENAI_MODEL constant in src/llm.rs.

Changes:

  • Switch the default OpenAI model from gpt-4o to gpt-5.4-mini.
Show a summary per file
File Description
src/llm.rs Updates the hard-coded default OpenAI model string used for chat-completions requests.

Copilot's findings

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@timrogers

Copy link
Copy Markdown
Owner Author

@copilot Break a test in this PR

Co-authored-by: timrogers <116134+timrogers@users.noreply.github.com>

Copilot AI commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

@copilot Break a test in this PR

Added a targeted unit test in src/llm.rs that asserts the legacy default OpenAI model value (gpt-4o) and confirmed it fails against the current implementation. Commit: 18f15d5.

@timrogers timrogers force-pushed the codex/switch-default-openai-model-to-gpt-5.4-mini-aftjuc branch from 78a0882 to 18f15d5 Compare June 24, 2026 15:27
@timrogers timrogers closed this Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants