Clean up WebSearchUI feature flag - #15273
Conversation
|
@oz-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 |
![oz-for-oss[bot]](https://avatars.githubusercontent.com/in/3450139?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview
This PR cleans up the WebSearchUI feature flag by removing the Cargo feature and enum variant, then making the existing WebSearch UI handling and execution-profile controls always available.
Concerns
- The PR ungates user-facing WebSearch UI surfaces, including the execution-profile web search toggle, permissions display, and inline WebSearch rendering, but the PR description does not include screenshots or a screen recording. For this user-facing change, please include screenshots or a screen recording demonstrating it working end to end.
app/src/ai/blocklist/block.rs:2013changes a comment that only narrates the call below it; repo guidance requires comments to explain non-obvious why, not restate what the code does.
Verdict
Found: 0 critical, 2 important, 0 suggestions
Request changes
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
| // Handle WebSearch messages | ||
| self.handle_web_search_messages(&output.messages, ctx); | ||
| } | ||
| // Handle WebSearch messages |
There was a problem hiding this comment.
handle_web_search_messages call below; remove it per the repo's why-only comment guidance.
1 participant
Automated cleanup of the
WebSearchUIfeature flag.This PR was generated by the feature flag cleanup workflow.