-
Notifications
You must be signed in to change notification settings - Fork 766
Update openai model to gpt5 #552
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
WalkthroughDocumentation updates in docs/configuration.mdx change OpenAI example/default configuration values from gpt-4o to gpt-5 across YAML and programmatic snippets. No structural edits or code logic changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
openai: | ||
default_model: gpt-4o | ||
default_model: gpt-5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
Align docs with actual default model.
These docs now claim the OpenAI default model is gpt-5
. Before merging, confirm the implementation truly defaults to gpt-5
; if the runtime still uses gpt-4o
(or another model), this update will mislead users. Please verify and keep documentation consistent with the shipped defaults.
If you need help checking the code paths that set the default, let me know.
Also applies to: 270-272, 854-855, 903-904
🏁 Script executed:
#!/usr/bin/env bash
# Search for default_model definitions/usages in code
rg -n "default_model" -C2 .
Length of output: 79471
Align docs’ OpenAI default_model with code fallback
The code in augmented_llm_openai.py uses "gpt-4o"
as the fallback default_model, not "gpt-5"
. Update all doc snippets (docs/configuration.mdx at 42–43, 103–105, 270–272, 854–855, 903–904) to use gpt-4o
.
🤖 Prompt for AI Agents
In docs/configuration.mdx around lines 42–43, the OpenAI default_model is set to
"gpt-5" but the code falls back to "gpt-4o"; update this snippet to use
"gpt-4o". Make the same replacement for the other mentioned occurrences (lines
103–105, 270–272, 854–855, 903–904) so all documentation examples match the code
fallback.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense, but we can do it later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@StreetLamb, understood! I'll leave this for later. Would you like me to create a GitHub issue to track updating the documentation examples to use gpt-4o
instead of gpt-5
to match the code fallback?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
openai: | ||
default_model: gpt-4o | ||
default_model: gpt-5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense, but we can do it later
Summary by CodeRabbit