Deduplicate SignalR connection bar per hub in plugin

Ensure the SignalR connection bar is rendered only once per hub, even when multiple tags map to the same hub path. Track the primary tag for each hub and render the connection bar only for that tag. Add integration test to verify deduplication logic in the plugin JavaScript.

Author: icnocop

src/SignalR.OpenApi.SwaggerUi/Resources/signalr-openapi-plugin.js

@@ -441,7 +441,10 @@ var SignalROpenApiPlugin = function (system) {
   // operations in the spec. Each operation's tags are associated with
   // the hub path from its x-signalr extension. This handles custom
   // [Tags] attributes where the tag name differs from the hub name.
+  // Also tracks the primary (first-seen) tag for each hub path so the
+  // connection bar is rendered only once per hub.
   var _cachedTagHubMap = null;
+  var _cachedHubPrimaryTag = null;
   var _cachedTagHubMapVersion = null;
 
   var _getTagHubMap = function () {
@@ -451,6 +454,7 @@ var SignalROpenApiPlugin = function (system) {
     }
 
     var tagMap = {};
+    var hubPrimaryTag = {};
     var pathsIm = specIm.get("paths");
     if (pathsIm) {
       pathsIm.keySeq().forEach(function (path) {
@@ -479,6 +483,9 @@ var SignalROpenApiPlugin = function (system) {
               var tagStr = typeof tag === "string" ? tag : (tag.get ? tag.get("name") || tag : tag);
               if (typeof tagStr === "string" && !tagMap[tagStr]) {
                 tagMap[tagStr] = hubPath;
+                if (!hubPrimaryTag[hubPath]) {
+                  hubPrimaryTag[hubPath] = tagStr;
+                }
               }
             });
           }
@@ -488,6 +495,7 @@ var SignalROpenApiPlugin = function (system) {
 
     _cachedTagHubMapVersion = specIm;
     _cachedTagHubMap = tagMap;
+    _cachedHubPrimaryTag = hubPrimaryTag;
     return tagMap;
   };
 
@@ -499,6 +507,14 @@ var SignalROpenApiPlugin = function (system) {
     return tagMap[tagName] || null;
   };
 
+  // Check if a tag is the primary (first-seen) tag for its hub.
+  // Used to render the connection bar only once per hub when
+  // multiple tags map to the same hub path.
+  var _isPrimaryTagForHub = function (tagName, hubPath) {
+    _getTagHubMap();
+    return _cachedHubPrimaryTag && _cachedHubPrimaryTag[hubPath] === tagName;
+  };
+
    // Parse request body from the SwaggerUI OAS3 state.
   // The executeRequest wrapper intercepts before the original action reads
   // the request body, so we must read it from the OAS3 selectors directly.
@@ -991,6 +1007,12 @@ var SignalROpenApiPlugin = function (system) {
             return result;
           }
 
+          // Only render one connection bar per hub. When multiple tags
+          // map to the same hub, show the bar on the primary tag only.
+          if (!_isPrimaryTagForHub(tagName, hubPath)) {
+            return result;
+          }
+
           return React.createElement("div", null,
             React.createElement(SignalRHubConnectionBar, { hubPath: hubPath }),
             result

test/SignalR.OpenApi.Tests/SwaggerUiIntegrationTests.cs

@@ -548,6 +548,24 @@ public async Task PluginJs_ContainsTagHubMapFunction()
         Assert.IsTrue(content.Contains("_getTagHubMap"), "Plugin JS should contain _getTagHubMap function for custom [Tags] support");
     }
 
+    /// <summary>
+    /// Verifies that the plugin JS deduplicates connection bars per hub
+    /// by only rendering on the primary tag via _isPrimaryTagForHub.
+    /// </summary>
+    /// <returns>A <see cref="Task"/> representing the asynchronous test.</returns>
+    [TestMethod]
+    public async Task PluginJs_ContainsPrimaryTagDeduplication()
+    {
+        using var host = await CreateTestHost();
+        using var client = host.GetTestClient();
+
+        using var response = await client.GetAsync("/signalr-swagger/_resources/signalr-openapi-plugin.js");
+        var content = await response.Content.ReadAsStringAsync();
+
+        Assert.IsTrue(content.Contains("_isPrimaryTagForHub"), "Plugin JS should contain _isPrimaryTagForHub for connection bar deduplication");
+        Assert.IsTrue(content.Contains("_cachedHubPrimaryTag"), "Plugin JS should cache the primary tag per hub path");
+    }
+
     /// <summary>
     /// Verifies that the CSS contains connection bar styles.
     /// </summary>