Clean up VerticalTabsSummaryMode feature flag - #15814
Conversation
|
@warp-for-oss[bot] 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 |
There was a problem hiding this comment.
Overview
This PR removes the VerticalTabsSummaryMode feature flag and makes vertical tabs Summary mode unconditional by deleting the Cargo feature, removing the runtime flag bridge and enum variant, and simplifying the affected vertical-tabs UI branches.
Concerns
- No blocking correctness, security, comment-guideline, test-guideline, or spec-drift concerns found in the attached diff.
- Comment audit: the diff does not add or change comments; it only removes the doc comment attached to the removed feature-flag variant.
- Test audit: the diff does not add or change tests. Given this is a cleanup of a default-enabled feature flag with no new independent code path, I did not find a missing-test issue.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
Automated cleanup of the
VerticalTabsSummaryModefeature flag.This PR was generated by the feature flag cleanup workflow.