[WIP] Show time alongside health check status

.github/copilot-instructions.md

@@ -6,6 +6,11 @@
 * Make only high confidence suggestions when reviewing code changes.
 * Always use the latest version C#, currently C# 13 features.
 * Never change global.json unless explicitly asked to.
+* Avoid breaking changes on public APIs. When adding new members to existing public types:
+  * Add new properties rather than positional parameters to records
+  * Add new optional parameters at the end of method signatures
+  * Use overloads instead of changing existing method signatures
+  * Consider backward compatibility and existing consumers
 
 ## Formatting
 

src/Aspire.Dashboard/Components/Controls/ResourceDetails.razor

@@ -228,7 +228,7 @@
                                    EnableHighlighting="@(!string.IsNullOrEmpty(_filter))"
                                    HighlightText="@_filter" />
                     </AspireTemplateColumn>
-                    <AspireTemplateColumn Title="@ControlStringsLoc[nameof(ControlsStrings.StateColumnHeader)]" Class="stateColumn">
+                    <AspireTemplateColumn Title="@ControlStringsLoc[nameof(ControlsStrings.StateColumnHeader)]" Class="stateColumn" Tooltip="true" TooltipText="@(c => GetHealthCheckTooltip(c))">
                         <GridValue ValueDescription="@ControlStringsLoc[nameof(ControlsStrings.StateColumnHeader)]"
                                    Value="@(context.HealthStatus?.Humanize() ?? Loc[nameof(Resources.WaitingHealthDataStatusMessage)])"
                                    EnableHighlighting="@(!string.IsNullOrEmpty(_filter))"
@@ -329,4 +329,15 @@
             return @<a href="@vm.Url" title="@vm.Url" target="_blank">@vm.DisplayName</a>;
         }
     }
+
+    private string? GetHealthCheckTooltip(HealthReportViewModel healthReport)
+    {
+        if (healthReport.LastRun.HasValue)
+        {
+            // Convert UTC to local time zone
+            var lastRunLocal = TimeZoneInfo.ConvertTimeFromUtc(healthReport.LastRun.Value, TimeZoneInfo.Local);
+            return $"Last check: {lastRunLocal:yyyy-MM-dd HH:mm:ss}";
+        }
+        return null;
+    }
 }

src/Aspire.Dashboard/Model/ResourceViewModel.cs

@@ -388,7 +388,7 @@ public bool MatchesFilter(string filter) =>
         Target?.Contains(filter, StringComparison.CurrentCultureIgnoreCase) == true;
 }
 
-public sealed record class HealthReportViewModel(string Name, HealthStatus? HealthStatus, string? Description, string? ExceptionText)
+public sealed record class HealthReportViewModel(string Name, HealthStatus? HealthStatus, string? Description, string? ExceptionText, DateTime? LastRun)
 {
     private readonly string? _humanizedHealthStatus = HealthStatus?.Humanize();
 

src/Aspire.Dashboard/ResourceService/Partials.cs

@@ -50,7 +50,8 @@ public ResourceViewModel ToViewModel(IKnownPropertyLookup knownPropertyLookup, I
 
         HealthReportViewModel ToHealthReportViewModel(HealthReport healthReport)
         {
-            return new HealthReportViewModel(healthReport.Key, healthReport.HasStatus ? MapHealthStatus(healthReport.Status) : null, healthReport.Description, healthReport.Exception);
+            DateTime? lastRun = healthReport.LastRun?.ToDateTime();
+            return new HealthReportViewModel(healthReport.Key, healthReport.HasStatus ? MapHealthStatus(healthReport.Status) : null, healthReport.Description, healthReport.Exception, lastRun);
         }
 
         Microsoft.Extensions.Diagnostics.HealthChecks.HealthStatus MapHealthStatus(HealthStatus healthStatus)

src/Aspire.Hosting/ApplicationModel/CustomResourceSnapshot.cs

@@ -292,7 +292,13 @@ public sealed record ResourceCommandSnapshot(string Name, ResourceCommandState S
 /// <param name="Description">An optional description of the report, for display.</param>
 /// <param name="ExceptionText">An optional string containing exception details.</param>
 [DebuggerDisplay("{Status}", Name = "{Name}")]
-public sealed record HealthReportSnapshot(string Name, HealthStatus? Status, string? Description, string? ExceptionText);
+public sealed record HealthReportSnapshot(string Name, HealthStatus? Status, string? Description, string? ExceptionText)
+{
+    /// <summary>
+    /// The timestamp when this health check was last executed, or <see langword="null"/> if it has never run.
+    /// </summary>
+    public DateTime? LastRun { get; init; }
+}
 
 /// <summary>
 /// The state of a resource command.

src/Aspire.Hosting/Dashboard/proto/Partials.cs

@@ -99,6 +99,11 @@ public static Resource FromSnapshot(ResourceSnapshot snapshot)
                 healthReport.Status = MapHealthStatus(report.Status.Value);
             }
 
+            if (report.LastRun.HasValue)
+            {
+                healthReport.LastRun = Timestamp.FromDateTime(report.LastRun.Value.ToUniversalTime());
+            }
+
             resource.HealthReports.Add(healthReport);
         }
 

src/Aspire.Hosting/Dashboard/proto/resource_service.proto

@@ -166,6 +166,8 @@ message HealthReport {
     string description = 3;
     // Any exception details.
     string exception = 4;
+    // The timestamp when this health check was last executed.
+    optional google.protobuf.Timestamp last_run = 5;
 }
 
 enum HealthStatus {

src/Aspire.Hosting/Health/ResourceHealthCheckService.cs

@@ -272,10 +272,11 @@ await resourceNotificationService.PublishUpdateAsync(resource, s => s with
     private static ImmutableArray<HealthReportSnapshot> MergeHealthReports(ImmutableArray<HealthReportSnapshot> healthReports, HealthReport report)
     {
         var builder = healthReports.ToBuilder();
+        var now = DateTime.UtcNow;
 
         foreach (var (key, entry) in report.Entries)
         {
-            var snapshot = new HealthReportSnapshot(key, entry.Status, entry.Description, entry.Exception?.ToString());
+            var snapshot = new HealthReportSnapshot(key, entry.Status, entry.Description, entry.Exception?.ToString()) { LastRun = now };
 
             var found = false;
             for (var i = 0; i < builder.Count; i++)

src/Aspire.Hosting/api/Aspire.Hosting.cs

@@ -1185,6 +1185,7 @@ public HealthCheckAnnotation(string key) { }
     [System.Diagnostics.DebuggerDisplay("{Status}", Name = "{Name}")]
     public sealed partial record HealthReportSnapshot(string Name, Microsoft.Extensions.Diagnostics.HealthChecks.HealthStatus? Status, string? Description, string? ExceptionText)
     {
+        public System.DateTime? LastRun { get; init; }
     }
 
     public partial record HostUrl(string Url) : IValueProvider, IManifestExpressionProvider

tests/Aspire.Dashboard.Components.Tests/Pages/ResourcesTests.cs

@@ -31,7 +31,7 @@ public void UpdateResources_FiltersUpdated()
                 "Resource1",
                 "Type1",
                 "Running",
-                ImmutableArray.Create(new HealthReportViewModel("Null", null, "Description1", null))),
+                ImmutableArray.Create(new HealthReportViewModel("Null", null, "Description1", null, null))),
         };
         var channel = Channel.CreateUnbounded<IReadOnlyList<ResourceViewModelChange>>();
         var dashboardClient = new TestDashboardClient(isEnabled: true, initialResources: initialResources, resourceChannelProvider: () => channel);
@@ -73,7 +73,7 @@ public void UpdateResources_FiltersUpdated()
                     "Resource2",
                     "Type2",
                     "Running",
-                    ImmutableArray.Create(new HealthReportViewModel("Healthy", HealthStatus.Healthy, "Description2", null))))
+                    ImmutableArray.Create(new HealthReportViewModel("Healthy", HealthStatus.Healthy, "Description2", null, null))))
             ]);
 
         cut.WaitForState(() => cut.Instance.GetFilteredResources().Count() == 2);
@@ -120,17 +120,17 @@ public void FilterResources()
                 "Resource1",
                 "Type1",
                 "Running",
-                ImmutableArray.Create(new HealthReportViewModel("Null", null, "Description1", null))),
+                ImmutableArray.Create(new HealthReportViewModel("Null", null, "Description1", null, null))),
             CreateResource(
                 "Resource2",
                 "Type2",
                 "Running",
-                ImmutableArray.Create(new HealthReportViewModel("Healthy", HealthStatus.Healthy, "Description2", null))),
+                ImmutableArray.Create(new HealthReportViewModel("Healthy", HealthStatus.Healthy, "Description2", null, null))),
             CreateResource(
                 "Resource3",
                 "Type3",
                 "Stopping",
-                ImmutableArray.Create(new HealthReportViewModel("Degraded", HealthStatus.Degraded, "Description3", null))),
+                ImmutableArray.Create(new HealthReportViewModel("Degraded", HealthStatus.Degraded, "Description3", null, null))),
         };
         var dashboardClient = new TestDashboardClient(isEnabled: true, initialResources: initialResources, resourceChannelProvider: Channel.CreateUnbounded<IReadOnlyList<ResourceViewModelChange>>);
         ResourceSetupHelpers.SetupResourcesPage(
@@ -182,7 +182,7 @@ public void ResourceGraph_MultipleRenders_InitializeOnce()
                 "Resource1",
                 "Type1",
                 "Running",
-                ImmutableArray.Create(new HealthReportViewModel("Null", null, "Description1", null))),
+                ImmutableArray.Create(new HealthReportViewModel("Null", null, "Description1", null, null))),
         };
         var dashboardClient = new TestDashboardClient(isEnabled: true, initialResources: initialResources, resourceChannelProvider: Channel.CreateUnbounded<IReadOnlyList<ResourceViewModelChange>>);
         ResourceSetupHelpers.SetupResourcesPage(

tests/Aspire.Dashboard.Tests/Model/ResourceViewModelTests.cs

@@ -26,7 +26,7 @@ public sealed class ResourceViewModelTests
     [InlineData(KnownResourceState.Running, DiagnosticsHealthStatus.Degraded, new string?[] {"Healthy", "Degraded"})]
     public void Resource_WithHealthReportAndState_ReturnsCorrectHealthStatus(KnownResourceState? state, DiagnosticsHealthStatus? expectedStatus, string?[]? healthStatusStrings)
     {
-        var reports = healthStatusStrings?.Select<string?, HealthReportViewModel>((h, i) => new HealthReportViewModel(i.ToString(), h is null ? null : System.Enum.Parse<DiagnosticsHealthStatus>(h), null, null)).ToImmutableArray() ?? [];
+        var reports = healthStatusStrings?.Select<string?, HealthReportViewModel>((h, i) => new HealthReportViewModel(i.ToString(), h is null ? null : System.Enum.Parse<DiagnosticsHealthStatus>(h), null, null, null)).ToImmutableArray() ?? [];
         var actualStatus = ResourceViewModel.ComputeHealthStatus(reports, state);
         Assert.Equal(expectedStatus, actualStatus);
     }

tests/Shared/DashboardModel/ModelTestHelpers.cs

@@ -40,7 +40,7 @@ public static ResourceViewModel CreateResource(
             State = state?.ToString(),
             KnownState = state,
             StateStyle = stateStyle,
-            HealthReports = reportHealthStatus is null && !createNullHealthReport ? [] : [new HealthReportViewModel("healthcheck", reportHealthStatus, null, null)],
+            HealthReports = reportHealthStatus is null && !createNullHealthReport ? [] : [new HealthReportViewModel("healthcheck", reportHealthStatus, null, null, null)],
             Commands = commands ?? [],
             Relationships = relationships ?? [],
             IsHidden = hidden