Add ContainerFilesDestinationAnnotation support to ProjectResource (#12355)

* Initial plan

* Add ContainerFilesDestinationAnnotation support to ProjectResource

Implement pipeline step factory for ProjectResource to support copying static files from other container resources before building docker images.

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

* Address PR feedback: use IContainerRuntime, fix PublishDir, generic source build steps

- Added CopyContainerFilesAsync method to IContainerRuntime interface with implementations in ContainerRuntimeBase
- Changed CopyContainerFilesToProjectAsync to use IContainerRuntime instead of calling docker directly
- Fixed GetPublishDirectoryAsync to use 'dotnet msbuild -c Release -getProperty:PublishDir' instead of hardcoded path
- Changed source resource build step to use generic CreateBuildImageStep instead of CreateProjectBuildImageStep

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

* Make it work in the playground

* Refactor to build project image first, then layer container files via Dockerfile

- Build ProjectResource image with normal flow first
- Tag built image with temporary GUID tag
- Generate Dockerfile that FROMs temp image and COPY --from source containers
- Build final image from generated Dockerfile with real tag
- Removed CopyContainerFilesAsync from IContainerRuntime (no longer needed)
- Removed GetPublishDirectoryAsync helper (no longer needed)

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

* Revert CopyContainerFilesAsync from IContainerRuntime - no longer needed

The new design uses DockerfileBuilder with COPY --from: instead of extracting files from containers to the host filesystem, so the CopyContainerFilesAsync method is no longer needed.

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

* Make it work by using the correct image name.

* Update for latest code

* Add RemoveImageAsync to IContainerRuntime for cleanup of temporary images

- Added RemoveImageAsync method to IContainerRuntime interface
- Implemented in ContainerRuntimeBase using 'rmi' command
- Uncommented cleanup code in ProjectResource to remove temporary tagged images
- Ensures temporary images are cleaned up after layering container files

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

* move test

* Clean up code.

* Clean up code

Add more tests

* Fix the playground app

* Address PR feedback

* Address PR feedback: Get ContainerWorkingDirectory from project and improve debugging

- Get $(ContainerWorkingDirectory) from project using dotnet msbuild instead of hardcoding /app
- Add GetContainerWorkingDirectoryAsync helper method with fallback to /app
- Keep temporary Dockerfile on build failure for debugging purposes
- Log debug message with path to temporary Dockerfile when build fails

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

* Apply suggestion from @eerhardt

* Refactor copilots code.

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: eerhardt <[email protected]>
Co-authored-by: Eric Erhardt <[email protected]>

Author: Copilot

playground/AspireWithJavaScript/AspireJavaScript.Angular/default.conf.template

@@ -15,6 +15,5 @@ server {
         proxy_http_version 1.1;
         proxy_ssl_server_name on;
         proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
-        rewrite ^/api(/.*)$ $1 break;
     }
-}
+}

playground/AspireWithJavaScript/AspireJavaScript.Angular/proxy.conf.js

@@ -2,9 +2,6 @@ module.exports = {
   "/api": {
     target:
       process.env["WEATHERAPI_HTTPS"] || process.env["WEATHERAPI_HTTP"],
-    secure: process.env["NODE_ENV"] !== "development",
-    pathRewrite: {
-      "^/api": "",
-    },
+    secure: process.env["NODE_ENV"] !== "development"
   },
 };

playground/AspireWithJavaScript/AspireJavaScript.AppHost/AppHost.cs

@@ -28,10 +28,12 @@
     .WithExternalHttpEndpoints()
     .PublishAsDockerFile();
 
-builder.AddViteApp("reactvite", "../AspireJavaScript.Vite")
+var reactvite = builder.AddViteApp("reactvite", "../AspireJavaScript.Vite")
     .WithNpm(install: true)
     .WithReference(weatherApi)
     .WithEnvironment("BROWSER", "none")
     .WithExternalHttpEndpoints();
 
+weatherApi.PublishWithContainerFiles(reactvite, "./wwwroot");
+
 builder.Build().Run();

playground/AspireWithJavaScript/AspireJavaScript.MinimalApi/AppHost.cs

@@ -30,7 +30,7 @@
     "Freezing", "Bracing", "Chilly", "Cool", "Mild", "Warm", "Balmy", "Hot", "Sweltering", "Scorching"
 };
 
-app.MapGet("/weatherforecast", () =>
+app.MapGet("/api/weatherforecast", () =>
 {
     var forecast =  Enumerable.Range(1, 5).Select(index =>
         new WeatherForecast
@@ -45,6 +45,8 @@
 .WithName("GetWeatherForecast")
 .WithOpenApi();
 
+app.UseFileServer();
+
 app.Run();
 
 sealed record WeatherForecast(DateOnly Date, int TemperatureC, string? Summary)

playground/AspireWithJavaScript/AspireJavaScript.React/default.conf.template

@@ -15,6 +15,5 @@ server {
         proxy_http_version 1.1;
         proxy_ssl_server_name on;
         proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
-        rewrite ^/api(/.*)$ $1 break;
     }
-}
+}

playground/AspireWithJavaScript/AspireJavaScript.React/webpack.config.js

@@ -11,7 +11,6 @@ module.exports = (env) => {
           context: ["/api"],
           target:
             process.env.WEATHERAPI_HTTPS || process.env.WEATHERAPI_HTTP,
-          pathRewrite: { "^/api": "" },
           secure: false,
         },
       ],

playground/AspireWithJavaScript/AspireJavaScript.Vite/default.conf.template

@@ -15,6 +15,5 @@ server {
         proxy_http_version 1.1;
         proxy_ssl_server_name on;
         proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
-        rewrite ^/api(/.*)$ $1 break;
     }
 }

playground/AspireWithJavaScript/AspireJavaScript.Vite/vite.config.ts

@@ -13,7 +13,6 @@ export default defineConfig(({ mode }) => {
                 '/api': {
                     target: process.env.WEATHERAPI_HTTPS || process.env.WEATHERAPI_HTTP,
                     changeOrigin: true,
-                    rewrite: (path) => path.replace(/^\/api/, ''),
                     secure: false,
                 }
             }

playground/AspireWithJavaScript/AspireJavaScript.Vue/default.conf.template

@@ -15,6 +15,5 @@ server {
         proxy_http_version 1.1;
         proxy_ssl_server_name on;
         proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
-        rewrite ^/api(/.*)$ $1 break;
     }
-}
+}

playground/AspireWithJavaScript/AspireJavaScript.Vue/vite.config.ts

@@ -20,7 +20,6 @@ export default defineConfig({
       '/api': {
         target: process.env.WEATHERAPI_HTTPS || process.env.WEATHERAPI_HTTP,
         changeOrigin: true,
-        rewrite: path => path.replace(/^\/api/, ''),
         secure: false
       }
     }