Skip to content

chore: Add toggle for context engine enable/disable#110645

Merged
shruthilayaj merged 3 commits into
masterfrom
shruthi/chore/add-frontend-toggle-for-seer-explorer-chat
Mar 13, 2026
Merged

chore: Add toggle for context engine enable/disable#110645
shruthilayaj merged 3 commits into
masterfrom
shruthi/chore/add-frontend-toggle-for-seer-explorer-chat

Conversation

@shruthilayaj
Copy link
Copy Markdown
Member

@shruthilayaj shruthilayaj commented Mar 13, 2026

Adds a toggle in the chat toolbar to allow AI/ML team to enable or
disable context engine
Screenshot 2026-03-13 at 12 36 09 PM

@shruthilayaj shruthilayaj requested a review from a team as a code owner March 13, 2026 16:30
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 13, 2026
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

query,
insert_index: calculatedInsertIndex,
on_page_context: screenshot,
override_ce_enable: overrideCtxEngEnable,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Override field always sent regardless of feature flag

Medium Severity

The override_ce_enable field is unconditionally included in every sendMessage POST request, regardless of whether the seer-explorer-context-engine-fe-override-ui-flag feature flag is active. Before this PR, the field was never sent. Now all users — even those without the feature flag — send override_ce_enable: true by default. If the backend distinguishes between the field being absent and it being explicitly true, this changes behavior for all users, not just those with the toggle UI.

Additional Locations (1)
Fix in Cursor Fix in Web

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

that's fine, it's only used in the backend if the backend flag is enabled

: t('Context engine disabled (click to enable)')
}
>
<ContextEngineToggle>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This can probably just use <Flex align="center" gap="xs" padding="xs sm" height="100%">. The remaining styles look unnecessary.

@shruthilayaj shruthilayaj merged commit 2b1ff87 into master Mar 13, 2026
63 checks passed
@shruthilayaj shruthilayaj deleted the shruthi/chore/add-frontend-toggle-for-seer-explorer-chat branch March 13, 2026 18:44
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants