Skip to content

Conversation

@lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Sep 4, 2025

COMPASS-9805

This switches all the configs to the production url.

If for some reason you want to develop against staging:

COMPASS_ASSISTANT_BASE_URL_OVERRIDE=https://knowledge.staging.corp.mongodb.com/api/v1 npm start

For easier review maybe just read the file in context: https://github.com/mongodb-js/compass/blob/switch-chatbot-urls/packages/atlas-service/src/util.ts#L125-L225

@Copilot Copilot AI review requested due to automatic review settings September 4, 2025 14:23
@lerouxb lerouxb requested a review from a team as a code owner September 4, 2025 14:23
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 switches from staging to production URLs for the compass assistant chatbot API across QA and production environments, while keeping local/dev environments on staging. The change updates both CSP policies and configuration URLs to point to the production knowledge.mongodb.com domain.

  • Updates Content Security Policy to allow the production chatbot URL
  • Switches assistant API base URLs from staging to production for QA and production configurations
  • Maintains staging URLs for local development environments

Reviewed Changes

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

File Description
packages/compass/src/app/utils/csp.ts Updates CSP to allow production chatbot domain
packages/atlas-service/src/util.ts Changes assistant API URLs from staging to production for QA/prod configs

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

@lerouxb lerouxb merged commit cf32285 into main Sep 4, 2025
55 of 58 checks passed
@lerouxb lerouxb deleted the switch-chatbot-urls branch September 4, 2025 16:55
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