Skip to content

Conversation

@Anemy
Copy link
Member

@Anemy Anemy commented Jul 23, 2025

Copilot AI review requested due to automatic review settings July 23, 2025 19:37
@Anemy Anemy requested a review from a team as a code owner July 23, 2025 19:37
@Anemy Anemy added the no release notes Fix or feature not for release notes label Jul 23, 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 adds error code cause chain tracking to connection failure telemetry events. The change enhances error reporting by capturing error codes from the entire cause chain of connection failures, which is particularly useful for debugging issues with the MongoDB driver and OIDC library that utilize error cause chains.

  • Adds error_code_cause_chain field to the ConnectionFailedEvent telemetry type
  • Implements getErrorCodeCauseChain utility function to extract error codes from cause chains
  • Integrates the new error code cause chain data into connection failure tracking

Reviewed Changes

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

File Description
packages/compass-telemetry/src/telemetry-events.ts Adds error_code_cause_chain field to ConnectionFailedEvent type definition
packages/compass-connections/src/utils/telemetry.tsx Implements utility function to extract error codes from error cause chains
packages/compass-connections/src/utils/telemetry.spec.tsx Adds comprehensive test coverage for the new getErrorCodeCauseChain function
packages/compass-connections/src/stores/connections-store-redux.ts Integrates error code cause chain data into connection failure telemetry tracking

@Anemy Anemy merged commit b3f8c5f into main Jul 28, 2025
61 checks passed
@Anemy Anemy deleted the COMPASS-9527-add-server-cause-chain-error-codes-to-connection-failed-telemetry branch July 28, 2025 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no release notes Fix or feature not for release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants