Skip to content

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.

Sign up for GitHub

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update openai model to gpt5 #552

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions docs/configuration.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ mcp-agent uses two configuration files:
args: ["-y", "@modelcontextprotocol/server-filesystem", "."]

openai:
default_model: gpt-4o
default_model: gpt-5
Comment on lines 42 to +43
Copy link

@coderabbitai coderabbitai bot Oct 10, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🧩 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.

Copy link
Collaborator

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

Copy link

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?

```
</Step>

Expand Down Expand Up @@ -102,7 +102,7 @@ mcp-agent uses two configuration files:
),
openai=OpenAISettings(
api_key="your-openai-api-key",
default_model="gpt-4o",
default_model="gpt-5",
),
)
```
Expand Down Expand Up @@ -268,7 +268,7 @@ mcp:
<CodeGroup>
```yaml Config File
openai:
default_model: gpt-4o
default_model: gpt-5
max_tokens: 4096
temperature: 0.7
```
Expand Down Expand Up @@ -851,7 +851,7 @@ mcp:
args: ["mcp-server-fetch"]

openai:
default_model: gpt-4o
default_model: gpt-5
reasoning_effort: "medium" # For o-series models: low, medium, high
api_key: "${OPENAI_API_KEY}"
```
Expand Down Expand Up @@ -900,7 +900,7 @@ mcp:

# Configure multiple providers
openai:
default_model: gpt-4o
default_model: gpt-5
api_key: "${OPENAI_API_KEY}"

anthropic:
Expand Down
Loading