From 35206fc0fdc271a4e06324b9925089204e47470c Mon Sep 17 00:00:00 2001
From: JamesPasta <james.pasta@gov.bc.ca>
Date: Wed, 19 Aug 2026 20:05:30 -0700
Subject: [PATCH] feature/AB#34135-InsantMessaging-bundling issue

---
 .../Bundling/NotificationsScriptBundleContributor.cs         | 1 -
 .../src/Unity.Theme.UX2/Themes/UX2/Layouts/Account.cshtml    | 3 +++
 .../Unity.Theme.UX2/Themes/UX2/Layouts/Application.cshtml    | 5 ++++-
 .../src/Unity.Theme.UX2/Themes/UX2/Layouts/Empty.cshtml      | 3 +++
 4 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/applications/Unity.GrantManager/modules/Unity.Notifications/src/Unity.Notifications.Web/Bundling/NotificationsScriptBundleContributor.cs b/applications/Unity.GrantManager/modules/Unity.Notifications/src/Unity.Notifications.Web/Bundling/NotificationsScriptBundleContributor.cs
index 56fd77729b..be08e7c39c 100644
--- a/applications/Unity.GrantManager/modules/Unity.Notifications/src/Unity.Notifications.Web/Bundling/NotificationsScriptBundleContributor.cs
+++ b/applications/Unity.GrantManager/modules/Unity.Notifications/src/Unity.Notifications.Web/Bundling/NotificationsScriptBundleContributor.cs
@@ -7,7 +7,6 @@ public class NotificationsScriptBundleContributor : BundleContributor
 {
     public override void ConfigureBundle(BundleConfigurationContext context)
     {
-        context.Files.AddIfNotContains("/libs/signalr/browser/signalr.min.js");
         context.Files.AddIfNotContains("/js/notifications-realtime-client.js");
     }
 }
diff --git a/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Account.cshtml b/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Account.cshtml
index aa19019208..19a5622d2f 100644
--- a/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Account.cshtml
+++ b/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Account.cshtml
@@ -99,6 +99,9 @@
         </abp-row>
     </div>
 
+    @* Loaded outside the bundle: re-minifying this already-minified library alongside the rest of
+       UX2.Global corrupts its automatic-reconnect closures into unbounded mutual recursion. *@
+    <script src="~/libs/signalr/browser/signalr.min.js"></script>
     <abp-script-bundle name="@UnityThemeUX2Bundles.Scripts.Global" />
 
     <script src="~/Abp/ApplicationLocalizationScript?cultureName=@CultureInfo.CurrentUICulture.Name"></script>
diff --git a/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Application.cshtml b/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Application.cshtml
index 2374b9863c..25f0f92c17 100644
--- a/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Application.cshtml
+++ b/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Application.cshtml
@@ -109,7 +109,10 @@
         @await Component.InvokeLayoutHookAsync(LayoutHooks.PageContent.Last, StandardLayouts.Application)
     </div>
 
-    <abp-script-bundle name="@UnityThemeUX2Bundles.Scripts.Global" />    
+    @* Loaded outside the bundle: re-minifying this already-minified library alongside the rest of
+       UX2.Global corrupts its automatic-reconnect closures into unbounded mutual recursion. *@
+    <script src="~/libs/signalr/browser/signalr.min.js"></script>
+    <abp-script-bundle name="@UnityThemeUX2Bundles.Scripts.Global" />
 
     <script src="~/Abp/ApplicationLocalizationScript?cultureName=@CultureInfo.CurrentUICulture.Name"></script>
     <script src="~/Abp/ApplicationConfigurationScript"></script>
diff --git a/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Empty.cshtml b/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Empty.cshtml
index cb0780c51b..d7d15398a7 100644
--- a/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Empty.cshtml
+++ b/applications/Unity.GrantManager/modules/Unity.Theme.UX2/src/Unity.Theme.UX2/Themes/UX2/Layouts/Empty.cshtml
@@ -66,6 +66,9 @@
 
     <script src="~/js/jquery-polyfills.js"></script>
 
+    @* Loaded outside the bundle: re-minifying this already-minified library alongside the rest of
+       UX2.Global corrupts its automatic-reconnect closures into unbounded mutual recursion. *@
+    <script src="~/libs/signalr/browser/signalr.min.js"></script>
     <abp-script-bundle name="@UnityThemeUX2Bundles.Scripts.Global" />
 
     <script src="~/Abp/ApplicationLocalizationScript?cultureName=@CultureInfo.CurrentUICulture.Name"></script>