Adding health checks on the npm resource

Author: aaronpowell

src/CommunityToolkit.Aspire.Hosting.McpInspector/McpInspectorResource.cs

@@ -10,6 +10,16 @@ public class McpInspectorResource(string name) : ExecutableResource(name, "npx",
 {
     internal readonly string ConfigPath = Path.GetTempFileName();
 
+    /// <summary>
+    /// The name of the client endpoint.
+    /// </summary>
+    public const string ClientEndpointName = "client";
+
+    /// <summary>
+    /// The name of the server proxy endpoint.
+    /// </summary>
+    public const string ServerProxyEndpointName = "server-proxy";
+
     /// <summary>
     /// Gets the version of the MCP Inspector.
     /// </summary>

src/CommunityToolkit.Aspire.Hosting.McpInspector/McpInspectorResourceBuilderExtensions.cs

@@ -17,9 +17,11 @@ public static IResourceBuilder<McpInspectorResource> AddMcpInspector(this IDistr
         var resource = builder.AddResource(new McpInspectorResource(name))
             .WithArgs(["-y", $"@modelcontextprotocol/inspector@{McpInspectorResource.InspectorVersion}"])
             .ExcludeFromManifest()
-            .WithHttpEndpoint(isProxied: false, port: Random.Shared.Next(3000, 4000), env: "CLIENT_PORT", name: "client")
-            .WithHttpEndpoint(isProxied: false, port: Random.Shared.Next(4000, 5000), env: "SERVER_PORT", name: "server-proxy")
-            .WithEnvironment("DANGEROUSLY_OMIT_AUTH", "true");
+            .WithHttpEndpoint(isProxied: false, port: Random.Shared.Next(3000, 4000), env: "CLIENT_PORT", name: McpInspectorResource.ClientEndpointName)
+            .WithHttpEndpoint(isProxied: false, port: Random.Shared.Next(4000, 5000), env: "SERVER_PORT", name: McpInspectorResource.ServerProxyEndpointName)
+            .WithEnvironment("DANGEROUSLY_OMIT_AUTH", "true")
+            .WithHttpHealthCheck("/", endpointName: McpInspectorResource.ClientEndpointName)
+            .WithHttpHealthCheck("/config", endpointName: McpInspectorResource.ServerProxyEndpointName);
 
         builder.Eventing.Subscribe<BeforeResourceStartedEvent>(resource.Resource, async (@event, ct) =>
         {
@@ -52,8 +54,8 @@ public static IResourceBuilder<McpInspectorResource> AddMcpInspector(this IDistr
         return resource
             .WithEnvironment(ctx =>
             {
-                var clientEndpoint = resource.GetEndpoint("client");
-                var serverProxyEndpoint = resource.GetEndpoint("server-proxy");
+                var clientEndpoint = resource.GetEndpoint(McpInspectorResource.ClientEndpointName);
+                var serverProxyEndpoint = resource.GetEndpoint(McpInspectorResource.ServerProxyEndpointName);
 
                 if (clientEndpoint is null || serverProxyEndpoint is null)
                 {

tests/CommunityToolkit.Aspire.Hosting.McpInspector.Tests/AppHostTests.cs

@@ -5,8 +5,8 @@ namespace CommunityToolkit.Aspire.Hosting.McpInspector.Tests;
 public class AppHostTests(AspireIntegrationTestFixture<Projects.CommunityToolkit_Aspire_Hosting_McpInspector_AppHost> fixture) : IClassFixture<AspireIntegrationTestFixture<Projects.CommunityToolkit_Aspire_Hosting_McpInspector_AppHost>>
 {
     [Theory]
-    [InlineData("client", "/")]
-    [InlineData("server-proxy", "/config")]
+    [InlineData(McpInspectorResource.ClientEndpointName, "/")]
+    [InlineData(McpInspectorResource.ServerProxyEndpointName, "/config")]
     public async Task ResourceStartsAndRespondsOk(string endpointName, string route)
     {
         var resourceName = "mcp-inspector";