Skip to content

Conversation

@gagik
Copy link
Contributor

@gagik gagik commented Sep 5, 2025

We had a PR for this which ended up only applying to entry points, so this adds this to the sidebar itself.

We had a PR for this which ended up only applying to entry points, so this adds this to the sidebar itself.
@Copilot Copilot AI review requested due to automatic review settings September 5, 2025 14:39
@gagik gagik requested a review from a team as a code owner September 5, 2025 14:39
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 enhances the AI feature disabling functionality in the Compass Assistant by ensuring the drawer is hidden when AI features are disabled through any configuration option, not just the specific AI assistant toggle.

  • Added isAiFeatureEnabled check to the drawer component's rendering logic
  • Restructured tests to comprehensively cover all AI feature disable scenarios
  • Imported useIsAIFeatureEnabled hook to check broader AI feature availability

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
packages/compass-assistant/src/compass-assistant-drawer.tsx Added isAiFeatureEnabled check to hide drawer when AI features are disabled
packages/compass-assistant/src/compass-assistant-provider.spec.tsx Restructured tests into a describe block and added comprehensive test coverage for all AI disable scenarios

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@gagik gagik added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Sep 5, 2025
@gagik gagik merged commit 5454786 into main Sep 8, 2025
60 of 61 checks passed
@gagik gagik deleted the gagik/assistant-sources branch September 8, 2025 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants