Skip to content

refactor(sentry): Migrate from fxa-shared/sentry to @fxa/sentry-* pac…#20040

Draft
dschom wants to merge 14 commits intomainfrom
use-sentry-from-libs
Draft

refactor(sentry): Migrate from fxa-shared/sentry to @fxa/sentry-* pac…#20040
dschom wants to merge 14 commits intomainfrom
use-sentry-from-libs

Conversation

@dschom
Copy link
Contributor

@dschom dschom commented Feb 11, 2026

Because

  • We want to stop using fxa-shared/sentry code and start using the helper code we migrated to libs

This pull request

  • Added shorter TypeScript path aliases (@fxa/sentry-*) in tsconfig.base.json
  • Updated 25 files across auth-server, content-server, settings, admin-panel, event-broker, and payments-server
  • Removed tagCriticalEvent usage (deprecated)
  • Added tracesSampler support to SentryConfigOpts types
  • Deleted legacy packages/fxa-shared/sentry/ directory and tests
  • Removed sentry exports from fxa-shared package.json

Issue that this pull request solves

Closes: FXA-9920

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

Other information (Optional)

This was done almost entirely by Claude

@dschom dschom requested a review from a team as a code owner February 11, 2026 01:20
vbudhram
vbudhram reviewed Feb 11, 2026
View reviewed changes
Copy link
Contributor

@vbudhram vbudhram left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looksl ike some build errors

@dschom
Copy link
Contributor Author

dschom commented Feb 11, 2026

Putting back in draft...

@dschom dschom marked this pull request as draft February 11, 2026 18:36
@dschom dschom force-pushed the use-sentry-from-libs branch from b01dcb7 to f8eec3f Compare February 12, 2026 00:39
dschom added 14 commits February 12, 2026 11:07
@dschom
polish(sentry): Cleanup browser.ts and tests
fa24292
@dschom
polish(shared): Move fxa-shared/monitoring to libs
3b7fb33
@dschom
polish(shared): Clean up browser.ts
70e2a30
@dschom
polish(shared/sentry): Fix duplicate typedef for SentryConfigOpts
d8805c1
@dschom
polish(shared): Output cjs and esm for otel package
4f68aa1
@dschom
chore(admin): Update refs to use libs instead of fxa-shared
80f998a 
Because:
- Cleaning up code duplication

This Commit:
- References @fxa/shared/sentry
- References @fxa/shared/otel
- References @fxa/shared/monitoring
@dschom
chore(auth): Update refs to use libs instead of fxa-shared
15dc309 
Because:
- Cleaning up code duplication

This Commit:
- References @fxa/shared/sentry
- References @fxa/shared/otel
- References @fxa/shared/monitoring
@dschom
chore(content): Update refs to use libs instead of fxa-shared
419af1b 
Because:
- Cleaning up code duplication

This Commit:
- References @fxa/shared/sentry
- References @fxa/shared/otel
- References @fxa/shared/monitoring
@dschom
chore(customs): Update refs to use libs instead of fxa-shared
63d3e18 
Because:
- Cleaning up code duplication

This Commit:
- References @fxa/shared/sentry
- References @fxa/shared/otel
- References @fxa/shared/monitoring
@dschom
chore(event-broker): Update refs to use libs instead of fxa-shared
94173fb 
Because:
- Cleaning up code duplication

This Commit:
- References @fxa/shared/sentry
- References @fxa/shared/otel
- References @fxa/shared/monitoring
@dschom
chore(payments): Update refs to use libs instead of fxa-shared
0e3c81b 
Because:
- Cleaning up code duplication

This Commit:
- References @fxa/shared/sentry
- References @fxa/shared/otel
- References @fxa/shared/monitoring
@dschom
chore(profile): Update refs to use libs instead of fxa-shared
2f07f9e 
Because:
- Cleaning up code duplication

This Commit:
- References @fxa/shared/sentry
- References @fxa/shared/otel
- References @fxa/shared/monitoring
@dschom
chore(settings): Update refs to use libs instead of fxa-shared
cd7fedc 
Because:
- Cleaning up code duplication

This Commit:
- References @fxa/shared/sentry
- References @fxa/shared/otel
- References @fxa/shared/monitoring
@dschom
chore(admin): Delete unused code from fxa-shared
c214e87 
Because:
- Cleaning up code duplication, in favor of code ported to libs

This Commit:
- Deletes unreferenced code
@dschom dschom force-pushed the use-sentry-from-libs branch from f8eec3f to c214e87 Compare February 14, 2026 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vbudhram vbudhram vbudhram left review comments

At least 1 approving review is required to merge this pull request.

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

@dschom @vbudhram