feat: Enhance DashboardController to fetch game server data concurrently and enrich telemetry information

Author: frasermolyneux

src/XtremeIdiots.Portal.Web/Controllers/DashboardController.cs

@@ -2,6 +2,7 @@
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
 
+using XtremeIdiots.Portal.Repository.Abstractions.Constants.V1;
 using XtremeIdiots.Portal.Repository.Api.Client.V1;
 using XtremeIdiots.Portal.Web.Auth.Constants;
 using XtremeIdiots.Portal.Web.Extensions;
@@ -70,7 +71,43 @@ public async Task<IActionResult> Index(CancellationToken cancellationToken = def
             // Agent telemetry (non-critical — dashboard renders without it)
             try
             {
-                viewModel.AgentStatuses = await agentTelemetryService.GetAllServersStatusAsync(cancellationToken).ConfigureAwait(false);
+                var telemetryTask = agentTelemetryService.GetAllServersStatusAsync(cancellationToken);
+                var gameServersTask = repositoryApiClient.GameServers.V1.GetGameServers(
+                    null, null, GameServerFilter.AgentEnabled, 0, 100,
+                    GameServerOrder.BannerServerListPosition, cancellationToken);
+
+                await Task.WhenAll(telemetryTask, gameServersTask).ConfigureAwait(false);
+
+                var telemetry = await telemetryTask.ConfigureAwait(false);
+                var gameServersResponse = await gameServersTask.ConfigureAwait(false);
+
+                var serverLookup = gameServersResponse.IsSuccess && gameServersResponse.Result?.Data?.Items is not null
+                    ? gameServersResponse.Result.Data.Items
+                        .GroupBy(gs => gs.GameServerId)
+                        .ToDictionary(g => g.Key, g => g.First())
+                    : [];
+
+                // Enrich telemetry with server names from the repository API
+                var telemetryByServer = telemetry
+                    .GroupBy(t => t.ServerId)
+                    .ToDictionary(g => g.Key, g => g.First());
+                viewModel.AgentStatuses = serverLookup.Values.Select(gs =>
+                {
+                    telemetryByServer.TryGetValue(gs.GameServerId, out var summary);
+
+                    return new AgentServerSummary
+                    {
+                        ServerId = gs.GameServerId,
+                        ServerTitle = string.IsNullOrWhiteSpace(gs.LiveTitle) ? gs.Title : gs.LiveTitle,
+                        GameType = gs.GameType.ToString(),
+                        LastEventReceived = summary?.LastEventReceived,
+                        EventsLastHour = summary?.EventsLastHour ?? 0,
+                        PlayerCount = summary?.PlayerCount ?? 0,
+                        CurrentMap = summary?.CurrentMap ?? gs.LiveMap,
+                        IsAgentActive = summary?.IsAgentActive ?? false,
+                        ActivityStatus = summary?.ActivityStatus ?? AgentActivityStatus.Offline
+                    };
+                }).ToList();
             }
             catch (Exception ex)
             {

src/XtremeIdiots.Portal.Web/Program.cs

@@ -135,7 +135,10 @@
 
 builder.Services.AddCors(options =>
 {
-    var corsOrigin = GetConfigValue(builder.Configuration, "XtremeIdiots:Forums:BaseUrl", "XtremeIdiots:Forums:BaseUrl configuration is required");
+    var corsBaseUrl = GetConfigValue(builder.Configuration, "XtremeIdiots:Forums:BaseUrl", "XtremeIdiots:Forums:BaseUrl configuration is required");
+    if (!Uri.TryCreate(corsBaseUrl, UriKind.Absolute, out var corsUri))
+        throw new InvalidOperationException($"XtremeIdiots:Forums:BaseUrl value '{corsBaseUrl}' is not a valid absolute URI for CORS origin configuration");
+    var corsOrigin = corsUri.GetLeftPart(UriPartial.Authority);
     options.AddPolicy("CorsPolicy",
         policy => policy
             .WithOrigins(corsOrigin)

src/XtremeIdiots.Portal.Web/wwwroot/js/forum-widget.js

@@ -41,7 +41,12 @@
 
     function ago(dateStr) {
         if (!dateStr) return '';
-        return portalDate.formatRelativeTime(dateStr);
+        var diff = (Date.now() - new Date(dateStr).getTime()) / 1000;
+        if (diff < 60) return 'just now';
+        if (diff < 3600) return Math.floor(diff / 60) + 'm ago';
+        if (diff < 86400) return Math.floor(diff / 3600) + 'h ago';
+        if (diff < 604800) return Math.floor(diff / 86400) + 'd ago';
+        return new Date(dateStr).toLocaleDateString();
     }
 
     function getActionIcon(actionType) {
@@ -279,7 +284,12 @@
                 container.innerHTML = '<div class="xi-pw-error">Unable to load notifications</div>';
                 return;
             }
-            renderWidget(container, data);
+            try {
+                renderWidget(container, data);
+            } catch (e) {
+                console.error('Forum widget render error:', e);
+                container.innerHTML = '<div class="xi-pw-error">Unable to display notifications</div>';
+            }
         });
     }