Skip to content

Conversation

@roomote
Copy link

@roomote roomote bot commented Nov 2, 2025

Summary

This PR adds a user-facing warning when the gpt5-codex model is selected, informing users that it performs poorly with Roo's tool call functionality and suggesting they use standard GPT-5 models instead.

Changes

  • Added performanceWarning field to the ModelInfo type schema to support model-specific warnings
  • Added warning message to the gpt-5-codex model definition
  • Updated both ApiOptions and ModelPicker components to display performance warnings
  • Warning is shown using the existing ApiErrorMessage component for consistency

Related Issue

Fixes #8983

Testing

  • ✅ All existing tests pass
  • ✅ Type checking passes
  • ✅ Linting passes
  • ✅ Manual testing: Warning displays correctly when gpt-5-codex is selected
  • ✅ Warning doesn't interfere with deprecated model warnings

Screenshots

The warning message reads: "GPT-5-Codex performs poorly with Roo's tool call functionality. Consider using standard GPT-5 models instead for better performance."

This warning appears in both the main settings view and the model picker component.


Important

Adds a performance warning for the gpt-5-codex model, displayed in ApiOptions and ModelPicker using ApiErrorMessage, with schema updates in model.ts and model definition in openai.ts.

  • Behavior:
    • Adds performanceWarning field to ModelInfo schema in model.ts.
    • Adds warning message to gpt-5-codex model in openai.ts.
    • Displays warning in ApiOptions.tsx and ModelPicker.tsx using ApiErrorMessage.
  • Testing:
    • Manual testing confirms warning displays correctly for gpt-5-codex.
    • Warning does not interfere with deprecated model warnings.

This description was created by Ellipsis for 912803c. You can customize this summary. It will automatically update as commits are pushed.

- Add performanceWarning field to ModelInfo type schema
- Add warning message to gpt-5-codex model definition
- Display warning in ApiOptions and ModelPicker components
- Warning advises users to use standard GPT-5 models for better tool call performance

Fixes #8983
@roomote roomote bot requested review from cte, jr and mrubens as code owners November 2, 2025 23:35
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Nov 2, 2025
@roomote
Copy link
Author

roomote bot commented Nov 2, 2025

See this task on Roo Code Cloud

Review complete. No issues found.

The implementation correctly adds the performanceWarning field to the ModelInfo schema, applies it to the gpt-5-codex model, and displays the warning appropriately in both ApiOptions and ModelPicker components. The warning is only shown when the model is not deprecated, following the existing pattern for model deprecation warnings.

Mention @roomote in a comment to trigger your PR Fixer agent and make changes to this pull request.

@dosubot dosubot bot added enhancement New feature or request UI/UX UI/UX related or focused labels Nov 2, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Nov 2, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Nov 3, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:S This PR changes 10-29 lines, ignoring generated files. UI/UX UI/UX related or focused

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[ENHANCEMENT] Warn users to not use gpt5-codex and instead use a normal gpt5 model

3 participants