From baf644d62b74dcdec823efbb20efa5928a0be620 Mon Sep 17 00:00:00 2001
From: Amit Haridas <amit.wh@gmail.com>
Date: Fri, 22 May 2026 22:08:06 +0530
Subject: [PATCH] fix(modal): prevent duplicate ModalManager declaration

window.ModalManager was set unconditionally, causing "Identifier
'ModalManager' has already been declared" when script tag in HTML
also loaded ModalManager before renderer.js required it.

Now checks !window.ModalManager before setting.

Amit Haridas
---
 src/utils/ModalManager.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/utils/ModalManager.js b/src/utils/ModalManager.js
index 9c1dfd2..6b169be 100644
--- a/src/utils/ModalManager.js
+++ b/src/utils/ModalManager.js
@@ -237,8 +237,8 @@ class ModalManager {
     }
 }
 
-// Export for use in renderer
-if (typeof window !== 'undefined') {
+// Export for use in renderer - avoid duplicate declaration
+if (typeof window !== 'undefined' && !window.ModalManager) {
     window.ModalManager = ModalManager;
 }