Skip to content

Conversation

@gagik
Copy link
Contributor

@gagik gagik commented Dec 19, 2025

The prompt is actually unchanged so this just removes the feature flag.

Screenshot 2025-12-19 at 2 41 46 PM

Copilot AI review requested due to automatic review settings December 19, 2025 13:41
@gagik gagik requested a review from a team as a code owner December 19, 2025 13:41
@gagik gagik requested a review from mabaasit December 19, 2025 13:41
@github-actions github-actions bot added the feat label Dec 19, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the enablePerformanceInsightsEntrypoints feature flag, effectively promoting the performance insights AI Assistant entrypoints to be always available when the AI Assistant is enabled.

Key Changes:

  • Removed the feature flag definition from the preferences model
  • Simplified the useAssistantActions hook to always return tellMoreAboutInsight when AI features are enabled
  • Updated test descriptions and removed the feature flag from test setup

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
packages/compass-preferences-model/src/feature-flags.ts Removed the enablePerformanceInsightsEntrypoints feature flag definition
packages/compass-assistant/src/compass-assistant-provider.tsx Removed conditional logic that gated tellMoreAboutInsight behind the feature flag
packages/compass-assistant/src/compass-assistant-provider.spec.tsx Updated test case to reflect the removal of the feature flag

@gagik gagik changed the title feat(compass-assistant): remove the feature flag for performance insights COMPASS-10146 chore(compass-assistant): remove the feature flag for performance insights COMPASS-10146 Dec 19, 2025
@gagik gagik removed the feat label Dec 19, 2025
@gagik gagik force-pushed the gagik/assistant-toggle branch from dc7c4f8 to 00b558c Compare December 19, 2025 14:04
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.

3 participants