-
Notifications
You must be signed in to change notification settings - Fork 2.6k
[Condense] Move condense settings out of experimental and defualt enable #4088
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
| "condensingApiConfiguration": { | ||
| "label": "API Configuration for Context Condensing", | ||
| "description": "Select which API configuration to use for context condensing operations. Leave unselected to use the current active configuration.", | ||
| "useCurrentConfig": "Default" | ||
| }, |
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.
Here too
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.
Great catch, will audit these
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.
Oh interesting, looks like the issue predates the PR. Will update here anyway
mrubens
left a 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.
Some of the translations look wrong, but LGTM otherwise
* protobuf executeQuickWin * whitespaces --------- Co-authored-by: Andrei Eternal <[email protected]>
Description
Moves the context condensing settings out of experimental and to the Context settings tab.
Also enables
autoCondenseContextby default.Test Procedure
Type of Change
srcor test files.Pre-Submission Checklist
npm run lint).console.log) has been removed.npm test).mainbranch.npm run changesetif this PR includes user-facing changes or dependency updates.Screenshots / Videos
Documentation Updates
Documentation for context condensing settings should be moved out of experimental and to the appropriate section.
Important
Moves
autoCondenseContextfrom experimental to main context settings, enabling it by default, and updates related UI components and tests.autoCondenseContextfrom experimental to context settings and enables it by default.ClineProviderandTaskto handleautoCondenseContextas a main setting.ContextManagementSettings.tsxto includeautoCondenseContextsettings.autoCondenseContextfromExperimentalSettings.tsx.autoCondenseContextdefault behavior inClineProvider.test.ts.ContextManagementSettings.test.tsxto test new settings behavior.autoCondenseContextfrom experimental settings inexperiment.tsandexperiments.test.ts.This description was created by
for bbd1d9a. You can customize this summary. It will automatically update as commits are pushed.