ref(utils): Remove optional chaining (#4289)

Co-authored-by: Armin Ronacher <[email protected]>

Author: AbhiPrasad

packages/utils/src/browser.ts

@@ -77,11 +77,12 @@ function _htmlElementAsString(el: unknown, keyAttrs?: string[]): string {
   out.push(elem.tagName.toLowerCase());
 
   // Pairs of attribute keys defined in `serializeAttribute` and their values on element.
-  const keyAttrPairs = keyAttrs?.length
-    ? keyAttrs.filter(keyAttr => elem.getAttribute(keyAttr)).map(keyAttr => [keyAttr, elem.getAttribute(keyAttr)])
-    : null;
+  const keyAttrPairs =
+    keyAttrs && keyAttrs.length
+      ? keyAttrs.filter(keyAttr => elem.getAttribute(keyAttr)).map(keyAttr => [keyAttr, elem.getAttribute(keyAttr)])
+      : null;
 
-  if (keyAttrPairs?.length) {
+  if (keyAttrPairs && keyAttrPairs.length) {
     keyAttrPairs.forEach(keyAttrPair => {
       out.push(`[${keyAttrPair[0]}="${keyAttrPair[1]}"]`);
     });

packages/utils/src/misc.ts

@@ -142,7 +142,7 @@ export function addExceptionMechanism(event: Event, newMechanism?: Partial<Mecha
   exceptionValue0.mechanism = { ...defaultMechanism, ...currentMechanism, ...newMechanism };
 
   if (newMechanism && 'data' in newMechanism) {
-    const mergedData = { ...currentMechanism?.data, ...newMechanism.data };
+    const mergedData = { ...(currentMechanism && currentMechanism.data), ...newMechanism.data };
     exceptionValue0.mechanism.data = mergedData;
   }
 }
@@ -261,7 +261,7 @@ export function stripUrlQueryAndFragment(urlPath: string): string {
  */
 export function checkOrSetAlreadyCaught(exception: unknown): boolean {
   // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-  if ((exception as any)?.__sentry_captured__) {
+  if (exception && (exception as any).__sentry_captured__) {
     return true;
   }