refactor(modal): add early guard for MutationObserver

Author: ShaneK

core/src/components/modal/modal.tsx

@@ -1166,6 +1166,10 @@ export class Modal implements ComponentInterface, OverlayInterface {
   }
 
   private initParentRemovalObserver() {
+    if (typeof MutationObserver === 'undefined') {
+      return;
+    }
+
     // Only observe if we have a cached parent and are in browser environment
     if (typeof window === 'undefined' || !this.cachedOriginalParent) {
       return;
@@ -1184,35 +1188,33 @@ export class Modal implements ComponentInterface, OverlayInterface {
       return;
     }
 
-    if (typeof MutationObserver !== 'undefined') {
-      this.parentRemovalObserver = new MutationObserver((mutations) => {
-        mutations.forEach((mutation) => {
-          if (mutation.type === 'childList' && mutation.removedNodes.length > 0) {
-            // Check if our cached original parent was removed
-            const cachedParentWasRemoved = Array.from(mutation.removedNodes).some((node) => {
-              const isDirectMatch = node === this.cachedOriginalParent;
-              const isContainedMatch = this.cachedOriginalParent
-                ? (node as HTMLElement).contains?.(this.cachedOriginalParent)
-                : false;
-              return isDirectMatch || isContainedMatch;
-            });
-
-            // Also check if parent is no longer connected to DOM
-            const cachedParentDisconnected = this.cachedOriginalParent && !this.cachedOriginalParent.isConnected;
-
-            if (cachedParentWasRemoved || cachedParentDisconnected) {
-              this.dismiss(undefined, 'parent-removed');
-            }
+    this.parentRemovalObserver = new MutationObserver((mutations) => {
+      mutations.forEach((mutation) => {
+        if (mutation.type === 'childList' && mutation.removedNodes.length > 0) {
+          // Check if our cached original parent was removed
+          const cachedParentWasRemoved = Array.from(mutation.removedNodes).some((node) => {
+            const isDirectMatch = node === this.cachedOriginalParent;
+            const isContainedMatch = this.cachedOriginalParent
+              ? (node as HTMLElement).contains?.(this.cachedOriginalParent)
+              : false;
+            return isDirectMatch || isContainedMatch;
+          });
+
+          // Also check if parent is no longer connected to DOM
+          const cachedParentDisconnected = this.cachedOriginalParent && !this.cachedOriginalParent.isConnected;
+
+          if (cachedParentWasRemoved || cachedParentDisconnected) {
+            this.dismiss(undefined, 'parent-removed');
           }
-        });
+        }
       });
+    });
 
-      // Observe with subtree to catch nested removals
-      this.parentRemovalObserver.observe(grandParent, {
-        childList: true,
-        subtree: true,
-      });
-    }
+    // Observe with subtree to catch nested removals
+    this.parentRemovalObserver.observe(grandParent, {
+      childList: true,
+      subtree: true,
+    });
   }
 
   private cleanupParentRemovalObserver() {