-
Notifications
You must be signed in to change notification settings - Fork 7k
Pashpashpash/gpt 5 release #5413
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
Conversation
|
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.
Pull Request Overview
This PR adds support for GPT-5 models to the Cline codebase, including new model configurations, pricing information, and integration with the next-generation model feature flags.
- Adds GPT-5 model family detection and configuration
- Updates default model to GPT-5 and includes multiple GPT-5 variants with pricing
- Enhances plan mode functionality with exploration control features
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
src/utils/model-utils.ts | Adds GPT-5 model family detection and consolidates next-gen model checking |
src/shared/api.ts | Updates default model and adds GPT-5 model configurations with pricing |
src/core/task/index.ts | Refactors to use centralized next-gen model detection |
src/core/task/ToolExecutor.ts | Refactors to use centralized next-gen model detection and adds exploration control |
src/core/prompts/system.ts | Updates plan mode documentation with exploration parameter |
src/core/prompts/model_prompts/claude4.ts | Updates plan mode documentation and adds markdown formatting guidelines |
src/core/assistant-message/index.ts | Adds needs_more_exploration parameter to tool parameter names |
src/api/providers/openai-native.ts | Adds API handling for GPT-5 models with reasoning effort support |
Coverage ReportExtension CoverageBase branch: 47% PR branch: 48% β Coverage increased or remained the same Webview CoverageBase branch: 17% PR branch: 17% β Coverage increased or remained the same Overall Assessmentβ Test coverage has been maintained or improved Last updated: 2025-08-07T16:50:41.992175 |
const stream = await client.chat.completions.create({ | ||
model: model.id, | ||
temperature: 1, | ||
messages: [{ role: "developer", content: systemPrompt }, ...convertToOpenAiMessages(messages)], |
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.
Related Issue
Issue: #XXXX
Description
Test Procedure
Type of Change
Pre-flight Checklist
npm test
) and code is formatted and linted (npm run format && npm run lint
)npm run changeset
(required for user-facing changes)Screenshots
Additional Notes
Important
Introduce GPT-5 model support, update default model, and enhance plan mode response handling with exploration parameter.
gpt-5-2025-08-07
,gpt-5-mini-2025-08-07
,gpt-5-nano-2025-08-07
, andnectarine-alpha-new-reasoning-effort-2025-07-25
toopenAiNativeModels
inapi.ts
.openAiNativeDefaultModelId
togpt-5-2025-08-07
.isGPT5ModelFamily()
and updateisNextGenModelFamily()
inmodel-utils.ts
to include GPT-5.OpenAiNativeHandler
inopenai-native.ts
to handle new GPT-5 models with streaming and usage options.needs_more_exploration
parameter toplan_mode_respond
insystem.ts
andclaude4.ts
to indicate further exploration is needed.This description was created by
for bb96e6e. You can customize this summary. It will automatically update as commits are pushed.