Promote ConfigurableContextWindow to Stable#10386
Merged
Merged
Conversation
Contributor
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
Contributor
There was a problem hiding this comment.
Overview
This PR promotes the existing ConfigurableContextWindow gate by adding the Cargo feature to the default feature set and removing the now-redundant dogfood runtime flag entry.
Concerns
- No blocking correctness, security, or validation concerns found in the changed hunks.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
Add configurable_context_window to the default Cargo feature list and remove it from DOGFOOD_FLAGS, enabling the context window slider for all build channels. Co-Authored-By: Oz <oz-agent@warp.dev>
a7cfe88 to
5f7e5c9
Compare
szgupta
approved these changes
May 7, 2026
trungtai1805
pushed a commit
to trungtai1805/warp
that referenced
this pull request
May 9, 2026
zhangyu1818-bot
pushed a commit
to zhangyu1818/warply
that referenced
this pull request
May 10, 2026
(cherry picked from commit 7ec0ec3)
tungd
pushed a commit
to tungd/warp
that referenced
this pull request
May 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Promotes the
ConfigurableContextWindowfeature flag from Dogfood to Stable, enabling the user-configurable context window slider in AI settings and the execution profile editor for all users.Changes
configurable_context_windowto thedefaultCargo feature list inapp/Cargo.tomlFeatureFlag::ConfigurableContextWindowfromDOGFOOD_FLAGSincrates/warp_features/src/lib.rs(no longer needed since the Cargo default feature enables it in all builds via the existing bridge inapp/src/lib.rs)Linked Issue
N/A
Testing
Feature flag promotion only — no behavioral changes. The flag was previously validated in Dogfood builds.
Agent Mode
Co-Authored-By: Oz oz-agent@warp.dev