refactor(sentry): Migrate from fxa-shared/sentry to @fxa/sentry-* packages

Migrated all Sentry imports from the legacy fxa-shared/sentry module to the new
scoped @fxa/sentry-* packages in libs/shared/:
- Browser code: @fxa/shared/sentry/browser
- Node.js code: @fxa/sentry-node
- Shared utilities: @fxa/sentry-utils

Changes:
- 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
- Ports fxa-shared/monitoring to libs
- Updates references to fxa-shared/monitoring

This completes the migration to the modular sentry package structure.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>

Author: dschom

libs/shared/monitoring/.eslintrc.json

@@ -0,0 +1,18 @@
+{
+  "extends": ["../../../.eslintrc.json"],
+  "ignorePatterns": ["!**/*"],
+  "overrides": [
+    {
+      "files": ["*.ts", "*.tsx", "*.js", "*.jsx"],
+      "rules": {}
+    },
+    {
+      "files": ["*.ts", "*.tsx"],
+      "rules": {}
+    },
+    {
+      "files": ["*.js", "*.jsx"],
+      "rules": {}
+    }
+  ]
+}

libs/shared/monitoring/.swcrc

@@ -0,0 +1,14 @@
+{
+  "jsc": {
+    "target": "es2017",
+    "parser": {
+      "syntax": "typescript",
+      "decorators": true,
+      "dynamicImport": true
+    },
+    "transform": {
+      "decoratorMetadata": true,
+      "legacyDecorator": true
+    }
+  }
+}

libs/shared/monitoring/README.md

@@ -0,0 +1,34 @@
+# shared-monitoring
+
+This library provides monitoring functionality including error tracking with Sentry and distributed tracing.
+
+It exports the `initMonitoring` function which initializes both error monitoring (Sentry) and performance monitoring (tracing) for server applications.
+
+## Installation
+
+```bash
+npm install @fxa/shared/monitoring
+```
+
+## Usage
+
+```typescript
+import { initMonitoring } from '@fxa/shared/monitoring';
+
+initMonitoring({
+  log: logger,
+  config: {
+    tracing: {
+      /* tracing config */
+    },
+    sentry: {
+      /* sentry config */
+    },
+  },
+});
+```
+
+## Exported APIs
+
+- `initMonitoring(opts: MonitoringConfig)` - Initialize monitoring components
+- `MonitoringConfig` - Type definition for monitoring configuration

libs/shared/monitoring/jest.config.ts

@@ -0,0 +1,42 @@
+import { Config } from 'jest';
+/* eslint-disable */
+import { readFileSync } from 'fs';
+
+// Reading the SWC compilation config and remove the "exclude"
+// for the test files to be compiled by SWC
+const { exclude: _, ...swcJestConfig } = JSON.parse(
+  readFileSync(`${__dirname}/.swcrc`, 'utf-8')
+);
+
+// disable .swcrc look-up by SWC core because we're passing in swcJestConfig ourselves.
+// If we do not disable this, SWC Core will read .swcrc and won't transform our test files due to "exclude"
+if (swcJestConfig.swcrc === undefined) {
+  swcJestConfig.swcrc = false;
+}
+
+// Uncomment if using global setup/teardown files being transformed via swc
+// https://nx.dev/packages/jest/documents/overview#global-setup/teardown-with-nx-libraries
+// jest needs EsModule Interop to find the default exported setup/teardown functions
+// swcJestConfig.module.noInterop = false;
+
+const config: Config = {
+  displayName: 'shared-monitoring',
+  preset: '../../../jest.preset.js',
+  transform: {
+    '^.+\\.[tj]s$': ['@swc/jest', swcJestConfig],
+  },
+  moduleFileExtensions: ['ts', 'js', 'html'],
+  testEnvironment: 'node',
+  coverageDirectory: '../../../coverage/libs/shared/monitoring',
+  reporters: [
+    'default',
+    [
+      'jest-junit',
+      {
+        outputDirectory: 'artifacts/tests/shared-monitoring',
+        outputName: 'shared-monitoring-jest-unit-results.xml',
+      },
+    ],
+  ],
+};
+export default config;

libs/shared/monitoring/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "@fxa/shared/monitoring",
+  "version": "0.0.0"
+}

libs/shared/monitoring/project.json

@@ -0,0 +1,51 @@
+{
+  "name": "shared-monitoring",
+  "$schema": "../../../node_modules/nx/schemas/project-schema.json",
+  "sourceRoot": "libs/shared/monitoring/src",
+  "projectType": "library",
+  "tags": ["scope:shared:lib"],
+  "targets": {
+    "build": {
+      "executor": "@nx/esbuild:esbuild",
+      "outputs": ["{options.outputPath}"],
+      "defaultConfiguration": "production",
+      "options": {
+        "main": "libs/shared/monitoring/src/index.ts",
+        "outputPath": "dist/libs/shared/monitoring",
+        "outputFileName": "main.js",
+        "tsConfig": "libs/shared/monitoring/tsconfig.lib.json",
+        "declaration": true,
+        "assets": [
+          {
+            "glob": "libs/shared/monitoring/README.md",
+            "input": ".",
+            "output": "."
+          }
+        ],
+        "platform": "node"
+      },
+      "configurations": {
+        "development": {
+          "minify": false
+        },
+        "production": {
+          "minify": true
+        }
+      }
+    },
+    "lint": {
+      "executor": "@nx/eslint:lint",
+      "outputs": ["{options.outputFile}"],
+      "options": {
+        "lintFilePatterns": ["libs/shared/monitoring/**/*.ts"]
+      }
+    },
+    "test-unit": {
+      "executor": "@nx/jest:jest",
+      "outputs": ["{workspaceRoot}/coverage/{projectRoot}"],
+      "options": {
+        "jestConfig": "libs/shared/monitoring/jest.config.ts"
+      }
+    }
+  }
+}

libs/shared/monitoring/src/index.spec.ts

@@ -0,0 +1,63 @@
+import { initTracing } from '@fxa/shared/otel';
+import { initSentry } from '@fxa/sentry-node';
+import { initMonitoring } from '@fxa/shared/monitoring';
+
+jest.mock('@fxa/shared/otel', () => ({ initTracing: jest.fn() }));
+jest.mock('@fxa/sentry-node', () => ({ initSentry: jest.fn() }));
+
+describe('shared-monitoring', () => {
+  beforeEach(() => {
+    jest.resetAllMocks();
+    jest.resetModules();
+  });
+
+  it('initializes tracing and sentry when config contains them', () => {
+    const log = {
+      info: jest.fn(),
+      warn: jest.fn(),
+      error: jest.fn(),
+      debug: jest.fn(),
+    };
+    const opts = {
+      log,
+      config: {
+        sentry: { dsn: 'https://example.com' },
+        tracing: { enabled: true, serviceName: 'test' },
+      },
+    };
+
+    initMonitoring(opts);
+
+    expect(initTracing).toHaveBeenCalledWith(opts.config.tracing, log);
+    expect(initSentry).toHaveBeenCalledWith(opts.config, log);
+  });
+
+  it('does not call tracing or sentry when not configured', () => {
+    const log = { warn: jest.fn() };
+    const opts = { log, config: {} };
+
+    initMonitoring(opts);
+
+    expect(initTracing).not.toHaveBeenCalled();
+    expect(initSentry).not.toHaveBeenCalled();
+  });
+
+  it('warns and skips when initialized more than once', () => {
+    const log = { warn: jest.fn() };
+    const opts = {
+      log,
+      config: {
+        sentry: { dsn: 'https://example.com' },
+        tracing: { enabled: true, serviceName: 'test' },
+      },
+    };
+
+    initMonitoring(opts);
+    initMonitoring(opts);
+
+    expect(log.warn).toHaveBeenCalledWith(
+      'monitoring',
+      'Monitoring can only be initialized once'
+    );
+  });
+});

packages/fxa-shared/monitoring/index.ts libs/shared/monitoring/src/index.tspackages/fxa-shared/monitoring/index.ts renamed to libs/shared/monitoring/src/index.ts

@@ -2,10 +2,10 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
-import { initTracing } from '../tracing/node-tracing';
-import { InitSentryOpts, initSentry } from '../sentry/node';
-import { TracingOpts } from '../tracing/config';
-import { ILogger } from '../log';
+import { initTracing } from '@fxa/shared/otel';
+import { InitSentryOpts, initSentry } from '@fxa/sentry-node';
+import { TracingOpts } from '@fxa/shared/otel';
+import { ILogger } from '@fxa/shared/log';
 
 export type MonitoringConfig = {
   log?: ILogger;
@@ -14,7 +14,7 @@ export type MonitoringConfig = {
 
 let initialized = false;
 
-// IMPORTANT! This initialization function must be called first thing when a server starts.If it's called after server
+// IMPORTANT! This initialization function must be called first thing when a server starts. If it's called after server
 // frameworks initialized instrumentation might not work properly.
 /**
  * Initializes modules related to error monitoring, performance monitoring, and tracing.

libs/shared/monitoring/tsconfig.json

@@ -0,0 +1,16 @@
+{
+  "extends": "../../../tsconfig.base.json",
+  "compilerOptions": {
+    "module": "commonjs"
+  },
+  "files": [],
+  "include": [],
+  "references": [
+    {
+      "path": "./tsconfig.lib.json"
+    },
+    {
+      "path": "./tsconfig.spec.json"
+    }
+  ]
+}

libs/shared/monitoring/tsconfig.lib.json

@@ -0,0 +1,11 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "module": "commonjs",
+    "outDir": "../../../dist/out-tsc",
+    "declaration": true,
+    "types": ["node"]
+  },
+  "exclude": ["jest.config.ts", "src/**/*.spec.ts", "src/**/*.test.ts"],
+  "include": ["src/**/*.ts"]
+}