-
Notifications
You must be signed in to change notification settings - Fork 2.4k
feat: Add GPT-5 Pro with background mode auto-resume and polling #8608
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?
Changes from all commits
c55a71c
dcc8791
373fbc9
957b8d9
6bb0bc0
41dadd5
9c2a830
d93aeef
4d40225
ac17911
760a233
85ddaeb
f8be63e
3a0add7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -37,6 +37,21 @@ export const openAiNativeModels = { | |
| { name: "priority", contextWindow: 400000, inputPrice: 2.5, outputPrice: 20.0, cacheReadsPrice: 0.25 }, | ||
| ], | ||
| }, | ||
| "gpt-5-pro-2025-10-06": { | ||
| maxTokens: 128000, | ||
| contextWindow: 400000, | ||
| supportsImages: true, | ||
| supportsPromptCache: false, | ||
| supportsReasoningEffort: false, // This is set to false to prevent the ui from displaying the reasoning effort selector | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Reasoning effort config is contradictory here: supportsReasoningEffort is false but reasoningEffort is set to "high". With this combination the UI hides the selector while the backend still injects a reasoning parameter (see src/shared/api.ts and src/api/providers/openai-native.ts). This can be confusing for users and maintainers. Consider either enabling supportsReasoningEffort to reflect configurability or removing the default reasoningEffort and documenting that the model runs with provider defaults. Fix it with Roo Code or mention @roomote and request a fix. |
||
| reasoningEffort: "high", // Pro model uses high reasoning effort by default and must be specified | ||
| inputPrice: 15.0, | ||
| outputPrice: 120.0, | ||
hannesrudolph marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| description: | ||
| "GPT-5 Pro: A slow, reasoning-focused model for complex problems. Uses background mode with resilient streaming — requests may take some time and will automatically reconnect if they time out.", | ||
hannesrudolph marked this conversation as resolved.
Show resolved
Hide resolved
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Style nit: The description uses an em dash (—). Project text typically avoids em dashes for consistency with UI strings. Consider replacing with a spaced hyphen form to match style elsewhere, e.g.: "GPT-5 Pro: A slow, reasoning-focused model for complex problems. Uses background mode with resilient streaming - requests may take some time and will automatically reconnect if they time out."Fix it with Roo Code or mention @roomote and request a fix. |
||
| supportsVerbosity: true, | ||
| supportsTemperature: false, | ||
| backgroundMode: true, | ||
| }, | ||
| "gpt-5-mini-2025-08-07": { | ||
| maxTokens: 128000, | ||
| contextWindow: 400000, | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.