Skip to content

fix(lib): keep error text in telemetry when the account's region is unknown - #574

Merged
gaurav-singh-9227 merged 2 commits into
browserstack:mainfrom
SavioBS629:fix/unknown-region-error-telemetry
Nov 11, 2026
Merged

fix(lib): keep error text in telemetry when the account's region is unknown#574
gaurav-singh-9227 merged 2 commits into
browserstack:mainfrom
SavioBS629:fix/unknown-region-error-telemetry

Conversation

@SavioBS629

@SavioBS629 SavioBS629 commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Region not confirmed: telemetry now keeps the error text along with the usage fields. Most of these accounts are US accounts, and the error text is needed to tell why their calls failed. Free-text inputs and feedback are still dropped.
  • EU and IN accounts: unchanged, usage fields only.
  • US accounts: unchanged.
  • When no host accepts the account during the region probe, each host's answer (status, or unreachable) is now logged as a warning instead of only at debug level.

Testing

npm run build passes. Unit tests cover telemetry for an unconfirmed region inside and outside a tool call, for thrown errors and returned failures.

🤖 Generated with Claude Code

@SavioBS629 @claude
fix(lib): keep error text in telemetry when the account's region is u…
6f7f90a 
…nknown

Most accounts whose region lookup fails are US accounts, so their error
text is kept to diagnose those failures. Free-text inputs and feedback
are still dropped, and EU and IN accounts are unchanged. When no region
probe accepts the account, each host's answer is logged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •
edited
Loading

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central YAML (base), Organization UI (inherited), Workspace UI (inherited); Enforced overrides: Organization
  • Review profile: ASSERTIVE
  • Plan: Enterprise
  • Run ID: cde57f8d-9a83-4708-83ee-2440f2ff4947

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

@SavioBS629 @claude
refactor(lib): drop comment on unknown-region error text
3d15991 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@gaurav-singh-9227
gaurav-singh-9227 merged commit a5ba6b6 into browserstack:main Nov 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gaurav-singh-9227 gaurav-singh-9227 gaurav-singh-9227 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SavioBS629 @gaurav-singh-9227