refactor out some optional chaining

Author: edwardgou-sentry

packages/tracing-internal/src/browser/browserTracingIntegration.ts

@@ -543,12 +543,14 @@ function registerInpInteractionListener(
     const client = getClient();
     // We need to get the replay, user, and activeTransaction from the current scope
     // so that we can associate replay id, profile id, and a user display to the span
-    const replay = client?.getIntegrationByName?.('Replay') as
-      | (Integration & { getReplayId: () => string })
-      | undefined;
+    const replay =
+      client !== undefined && client.getIntegrationByName !== undefined
+        ? (client.getIntegrationByName('Replay') as Integration & { getReplayId: () => string })
+        : undefined;
     // eslint-disable-next-line deprecation/deprecation
     const activeTransaction = getActiveTransaction();
-    const user = getCurrentScope()?.getUser();
+    const currentScope = getCurrentScope();
+    const user = currentScope !== undefined ? currentScope.getUser() : undefined;
     for (const entry of entries) {
       if (isPerformanceEventTiming(entry)) {
         const duration = entry.duration;

packages/tracing-internal/src/browser/metrics/index.ts

@@ -224,10 +224,10 @@ function _trackINP(interactionIdtoRouteNameMapping: InteractionRouteNameMapping)
             user: undefined,
             replay: undefined,
           };
+    const userDisplay = user !== undefined ? user.email || user.id || user.ip_address : undefined;
     // eslint-disable-next-line deprecation/deprecation
-    const userDisplay = user?.email || user?.id || user?.ip_address;
-    const profileId = activeTransaction?.getProfileId();
-    const replayId = replay?.getReplayId();
+    const profileId = activeTransaction !== undefined ? activeTransaction.getProfileId() : undefined;
+    const replayId = replay !== undefined ? replay.getReplayId() : undefined;
     const span = new Span({
       startTimestamp: startTime,
       endTimestamp: startTime + duration,