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

@@ -67,11 +67,14 @@ export interface ExpoRouterErrorBoundaryProps {
 export function wrapExpoRouterErrorBoundary<P extends ExpoRouterErrorBoundaryProps>(
   OriginalErrorBoundary: React.ComponentType<P>,
 ): React.ComponentType<P> {
-  const Wrapped: React.FC<P> = props => {
-    React.useEffect(() => {
-      registerFeatureMarker(EXPO_ROUTER_ERROR_BOUNDARY_INTEGRATION_NAME);
-    }, []);
+  // 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
     // render can be discarded — we only want to report errors that React

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

@@ -89,9 +89,8 @@ describe('wrapExpoRouterErrorBoundary', () => {
     expect(getByTestId('fallback').props.children).toBe('boom');
   });
 
-  it('registers the ExpoRouterErrorBoundary marker on mount', () => {
-    const Wrapped = wrapExpoRouterErrorBoundary(OriginalErrorBoundary);
-    render(<Wrapped error={new Error('boom')} retry={jest.fn()} />);
+  it('registers the ExpoRouterErrorBoundary marker at wrap-call time (before any mount)', () => {
+    wrapExpoRouterErrorBoundary(OriginalErrorBoundary);
     expect(mockAddIntegration).toHaveBeenCalledWith({ name: EXPO_ROUTER_ERROR_BOUNDARY_INTEGRATION_NAME });
   });