feat(tracing): Track wrapExpoRouterErrorBoundary adoption (#6425)

* feat(core): Track GlobalErrorBoundary adoption

Register a no-op `GlobalErrorBoundary` integration when the
component mounts so the name flows through to
`event.sdk.integrations` — the same channel used for feature-
adoption signals like `MobileFeedback` and `AppStart`.

Also introduces a shared `registerFeatureMarker` helper.
Subsequent markers for other opt-in features
(NavigationContainer, ExpoRouter error boundary, AppLoaded, ...)
will use this helper — see #6415.

Refs: #6415

* feat(tracing): Track wrapExpoRouterErrorBoundary adoption

Register a no-op `ExpoRouterErrorBoundary` integration when the
wrapped Expo Router error boundary mounts so the name flows
through to `event.sdk.integrations`.

Refs: #6415

* fix(tracing): Fire ExpoRouterErrorBoundary marker at wrap-call time

The wrapped boundary only mounts when Expo Router actually
renders it (i.e., on an error), so a `useEffect`-based marker
would undercount adoption for apps that ship the wrap but never
hit a route error. Registering at wrap-call time fires as soon
as the user's route file evaluates.

No client is present if the wrap is called before
`Sentry.init()`, but that is very rare in practice — Expo Router
lazy-loads route files during navigation, so init has typically
completed by then.

Author: antonis

packages/core/src/js/tracing/expoRouterErrorBoundary.tsx

@@ -13,8 +13,11 @@ import {
 } from '@sentry/core';
 import * as React from 'react';
 
+import { registerFeatureMarker } from '../utils/featureMarkers';
 import { getCurrentExpoRouterRouteInfo } from './expoRouterStore';
 
+export const EXPO_ROUTER_ERROR_BOUNDARY_INTEGRATION_NAME = 'ExpoRouterErrorBoundary';
+
 /**
  * Errors we have already reported. Module-scoped (rather than a per-instance
  * `useRef`) so that an unmount → remount cycle with the same error instance —
@@ -64,6 +67,13 @@ export interface ExpoRouterErrorBoundaryProps {
 export function wrapExpoRouterErrorBoundary<P extends ExpoRouterErrorBoundaryProps>(
   OriginalErrorBoundary: React.ComponentType<P>,
 ): React.ComponentType<P> {
+  // Register at wrap-call time (module evaluation) so the marker fires as soon
+  // as the user's route file loads, not only when Expo Router actually renders
+  // the boundary on an error. No-op if `Sentry.init()` has not run yet; route
+  // files load lazily during navigation, so init has typically completed by
+  // then.
+  registerFeatureMarker(EXPO_ROUTER_ERROR_BOUNDARY_INTEGRATION_NAME);
+
   const Wrapped: React.FC<P> = props => {
     // Reporting is intentionally done in `useEffect` (commit phase) rather than
     // during render: render must be pure, and in Concurrent Mode an in-progress

packages/core/test/tracing/expoRouterErrorBoundary.test.tsx

@@ -3,11 +3,16 @@ import { render } from '@testing-library/react-native';
 import * as React from 'react';
 import { Text } from 'react-native';
 
-import { wrapExpoRouterErrorBoundary } from '../../src/js/tracing/expoRouterErrorBoundary';
+import {
+  EXPO_ROUTER_ERROR_BOUNDARY_INTEGRATION_NAME,
+  wrapExpoRouterErrorBoundary,
+} from '../../src/js/tracing/expoRouterErrorBoundary';
 
 const mockCaptureException = jest.fn();
 const mockAddBreadcrumb = jest.fn();
 const mockAddExceptionMechanism = jest.fn();
+const mockAddIntegration = jest.fn();
+const mockGetIntegrationByName = jest.fn().mockReturnValue(undefined);
 let mockSendDefaultPii = false;
 let mockActiveSpan: { setStatus: jest.Mock; __origin?: string } | undefined;
 
@@ -18,7 +23,11 @@ jest.mock('@sentry/core', () => {
     captureException: (...args: unknown[]) => mockCaptureException(...args),
     addBreadcrumb: (...args: unknown[]) => mockAddBreadcrumb(...args),
     addExceptionMechanism: (...args: unknown[]) => mockAddExceptionMechanism(...args),
-    getClient: () => ({ getOptions: () => ({ sendDefaultPii: mockSendDefaultPii }) }),
+    getClient: () => ({
+      getOptions: () => ({ sendDefaultPii: mockSendDefaultPii }),
+      getIntegrationByName: mockGetIntegrationByName,
+      addIntegration: mockAddIntegration,
+    }),
     getActiveSpan: () => mockActiveSpan,
     getRootSpan: (span: unknown) => span,
     spanToJSON: (span: { __origin?: string } | undefined) => ({ origin: span?.__origin }),
@@ -80,6 +89,11 @@ describe('wrapExpoRouterErrorBoundary', () => {
     expect(getByTestId('fallback').props.children).toBe('boom');
   });
 
+  it('registers the ExpoRouterErrorBoundary marker at wrap-call time (before any mount)', () => {
+    wrapExpoRouterErrorBoundary(OriginalErrorBoundary);
+    expect(mockAddIntegration).toHaveBeenCalledWith({ name: EXPO_ROUTER_ERROR_BOUNDARY_INTEGRATION_NAME });
+  });
+
   it('captures the error to Sentry once per error instance', () => {
     const Wrapped = wrapExpoRouterErrorBoundary(OriginalErrorBoundary);
     const err = new Error('boom');