fix(client): preserve MongoDB error labels from responses - #51
medeirosinacio wants to merge 1 commit into
Conversation
Co-authored-by: GPT-5.6 Sol <noreply@openai.com> Co-authored-by: Cursor <cursoragent@cursor.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Summary
errorLabelswhen decoding top-level command errorsUnsentExceptionWhy
Utopia\Mongo\Exceptionalready exposesgetErrorLabels()andisRetryableWrite(), butClient::parseResponse()discarded labels from real wire responses. This makes the existing API work as intended and provides a prerequisite for retryable writes discussed in #45 without implementing retry behavior in this PR.Tests
HostUnreachableresponses remain potentially sent