chore(compass-assistant): warn about non-genuine hosts COMPASS-9675 #7280
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR adds functionality to warn users when connected to non-genuine MongoDB hosts (like DocumentDB or CosmosDB) in the Compass Assistant. The warning message persists even when the chat is cleared to ensure users are consistently informed about limitations when using non-genuine MongoDB servers.
- Adds a preset warning message for non-genuine MongoDB connections
- Implements logic to automatically show/hide warnings based on connection status
- Modifies chat clearing behavior to preserve permanent warning messages
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
preset-messages.ts |
Defines the non-genuine MongoDB warning message with permanent metadata |
compass-assistant-provider.tsx |
Adds isPermanent metadata field and updates clear chat logic to preserve permanent messages |
compass-assistant-provider.spec.tsx |
Updates test structure and adds test for persistent warning messages during chat clearing |
assistant-chat.tsx |
Implements automatic warning message management based on connection genuineness |
assistant-chat.spec.tsx |
Adds comprehensive tests for non-genuine host warning behavior |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Comment on lines
255
to
265
| atlastAiService, | ||
| }: { | ||
| chat: Chat<AssistantMessage>; | ||
| atlastAiService?: Partial<AtlasAiService>; | ||
| }, | ||
| { connections }: { connections?: ConnectionInfo[] } = {} | ||
| ): Promise<ReturnType<typeof render>> { | ||
| const result = render( | ||
| <TestComponent | ||
| chat={mockChat} | ||
| mockAtlasAiService={mockAtlasAiService} | ||
| chat={chat} | ||
| mockAtlasAiService={atlastAiService} |
There was a problem hiding this comment.
Variable name has a typo: 'atlastAiService' should be 'atlasAiService' (missing 's' in 'atlas').
Copilot uses AI. Check for mistakes.
gagik
force-pushed
the
gagik/assistant-non-genuine
branch
from
09c39e2 to
126d5c6
Compare
lerouxb
approved these changes
Sep 4, 2025
addaleax
approved these changes
Sep 4, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Local Testing
If you want to test this locally, you can add
127.0.0.1 docdb-elastic.amazonaws.comto your
/etc/hosts/and connect to
mongodb://docdb-elastic.amazonaws.com:27017