Skip to content

Remove aiChatSuggestion feature flag#8351

Merged
YoussefKeyrouz merged 1 commit into
developfrom
feature/youssef/remove_aiChatSuggestions_ff
Apr 27, 2026
Merged

Remove aiChatSuggestion feature flag#8351
YoussefKeyrouz merged 1 commit into
developfrom
feature/youssef/remove_aiChatSuggestions_ff

Conversation

@YoussefKeyrouz
Copy link
Copy Markdown
Collaborator

@YoussefKeyrouz YoussefKeyrouz commented Apr 26, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/project/1212810093780571/task/1213381906205784?focus=true

Description

  • Remove the aiChatSuggestions feature flag and treat the AI chat suggestions feature (pinned and recent chats) as always enabled.
  • Remove DuckChat.isChatSuggestionsFeatureAvailable() API and inline the always-true branches at every call site (InputScreenFragment, ChatTabFragment, InputScreenViewModel, NativeInputManager, GeneralSettingsViewModel).
  • Remove dead helpers (hideAutoCompleteIfOnChatTab, enableViewPagerInputIfNoFavorites) and feature flag related tests.

Steps to test this PR

  • Open the input screen, type in chat mode, confirm pinned/recent chats and chat URL suggestions still render
  • General Settings shows the chat suggestions toggle when Duck.ai + input screen are enabled. Setting should work as expected when turned on/off.
  • Switching between Search and Chat tabs hides/shows overlays as expected
  • Enable native input and confirm pinned/recent chat suggestions render under the input

Note

Medium Risk
Removes a remote-config gate around chat suggestions, so pinned/recent chat suggestions will now activate wherever the input screen/native input supports them; this is user-visible and could affect overlay/focus behavior and suggestion fetching frequency.

Overview
Chat suggestions (pinned/recent chats and related overlays) are now treated as always available by removing the aiChatSuggestions feature flag and the DuckChat.isChatSuggestionsFeatureAvailable() API.

Call sites in the input screen and native input (InputScreenFragment, ChatTabFragment, InputScreenViewModel, NativeInputManager, GeneralSettingsViewModel) were simplified to always run the chat-suggestions code paths, and related dead helper methods/branches were deleted.

Tests were updated accordingly by removing feature-flag expectations and dropping coverage for the “feature not available” scenario.

Reviewed by Cursor Bugbot for commit 5a3e99b. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Copy Markdown
Contributor

@joshliebe joshliebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🙂

@YoussefKeyrouz YoussefKeyrouz merged commit 6fc892c into develop Apr 27, 2026
18 checks passed
@YoussefKeyrouz YoussefKeyrouz deleted the feature/youssef/remove_aiChatSuggestions_ff branch April 27, 2026 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants