Skip to content

Conversation

@kuhe
Copy link
Contributor

@kuhe kuhe commented Oct 8, 2025

Issue

internal JS-6266

Description

  • replaces the browser SDK-UA detection with navigator userAgentData when possible, or else a basic UA string parser
  • bowser based legacy parsing behavior is available by an override explained in the package readme

This saves about 20kb pre-compression that was taken up by bowser.

Testing

Manual testing with local web app using both the new and the legacy UA providers.

Checklist

  • If the PR is a feature, add integration tests (*.integ.spec.ts).
  • If you wrote E2E tests, are they resilient to concurrent I/O?
  • If adding new public functions, did you add the @public tag and enable doc generation on the package?

@kuhe kuhe requested a review from a team as a code owner October 8, 2025 18:28
@kuhe kuhe changed the title chore(util-user-agent-bbrowser): remove bowser from default UA provider chore(util-user-agent-browser): remove bowser from default UA provider Oct 8, 2025
@kuhe kuhe merged commit a94d95f into main Oct 8, 2025
7 checks passed
@kuhe kuhe deleted the chore/bowser branch October 8, 2025 22:36
@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants