Simplify AddViteApp port configuration by using vite.config.ts

Co-authored-by: davidfowl <[email protected]>

examples/nodejs-ext/pnpm-demo/vite.config.ts

@@ -4,4 +4,7 @@ import react from '@vitejs/plugin-react'
 // https://vitejs.dev/config/
 export default defineConfig({
   plugins: [react()],
+  server: {
+    port: parseInt(process.env.PORT ?? '5173')
+  }
 })

examples/nodejs-ext/vite-demo/vite.config.ts

@@ -4,4 +4,7 @@ import react from '@vitejs/plugin-react'
 // https://vitejs.dev/config/
 export default defineConfig({
   plugins: [react()],
+  server: {
+    port: parseInt(process.env.PORT ?? '5173')
+  }
 })

examples/nodejs-ext/yarn-demo/vite.config.ts

@@ -4,4 +4,7 @@ import react from '@vitejs/plugin-react'
 // https://vitejs.dev/config/
 export default defineConfig({
   plugins: [react()],
+  server: {
+    port: parseInt(process.env.PORT ?? '5173')
+  }
 })

src/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions/NodeJSHostingExtensions.cs

@@ -37,34 +37,9 @@ public static IResourceBuilder<NodeAppResource> AddViteApp(this IDistributedAppl
             _ => builder.AddNpmApp(name, wd, "dev")
         };
 
-        var endpointBuilder = useHttps
+        return useHttps
             ? resource.WithHttpsEndpoint(env: "PORT").WithExternalHttpEndpoints()
             : resource.WithHttpEndpoint(env: "PORT").WithExternalHttpEndpoints();
-
-        builder.Eventing.Subscribe<ResourceEndpointsAllocatedEvent>((@event, ct) =>
-        {
-            if (@event.Resource.Name != name)
-            {
-                return Task.CompletedTask;
-            }
-
-            endpointBuilder.WithArgs(ctx =>
-            {
-                if (@event.Resource.TryGetEndpoints(out var endpoints))
-                {
-                    // Set the PORT environment variable to the first endpoint's port
-                    var firstEndpoint = endpoints.FirstOrDefault();
-
-                    ctx.Args.Add("--");
-                    ctx.Args.Add("--port");
-                    ctx.Args.Add(firstEndpoint?.AllocatedEndpoint?.Port.ToString() ?? "5173");
-                }
-            });
-
-            return Task.CompletedTask;
-        });
-
-        return resource;
     }
 
     /// <summary>

tests/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions.Tests/ResourceCreationTests.cs

@@ -217,9 +217,5 @@ public void ViteAppConfiguresPortFromEnvironment()
         var httpEndpoint = endpoints.FirstOrDefault(e => e.UriScheme == "http");
         Assert.NotNull(httpEndpoint);
         Assert.Equal("PORT", httpEndpoint.EnvVar);
-
-        // Verify that command line arguments callback is configured
-        Assert.True(resource.TryGetAnnotationsOfType<CommandLineArgsCallbackAnnotation>(out var argsCallbacks));
-        Assert.NotEmpty(argsCallbacks);
     }
 }