Refactor output path handling to use IPipelineOutputService

- Create IPipelineOutputService interface with GetOutputDirectory() and GetTempDirectory() methods
- Implement PipelineOutputService with better naming than IntermediateOutputPath
- Expose service via PipelineContext.OutputService and PipelineStepContext.OutputService
- Keep backward compatible OutputPath property for existing consumers
- Remove direct IntermediateOutputPath property per feedback

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

Author: Copilot

src/Aspire.Hosting/Pipelines/IPipelineOutputService.cs

@@ -0,0 +1,25 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using System.Diagnostics.CodeAnalysis;
+
+namespace Aspire.Hosting.Pipelines;
+
+/// <summary>
+/// Service for managing pipeline output directories.
+/// </summary>
+[Experimental("ASPIREPIPELINES001", UrlFormat = "https://aka.ms/aspire/diagnostics/{0}")]
+public interface IPipelineOutputService
+{
+    /// <summary>
+    /// Gets the output directory for deployment artifacts.
+    /// </summary>
+    /// <returns>The path to the output directory for deployment artifacts.</returns>
+    string GetOutputDirectory();
+
+    /// <summary>
+    /// Gets a temporary directory for build artifacts.
+    /// </summary>
+    /// <returns>The path to a temporary directory for build artifacts.</returns>
+    string GetTempDirectory();
+}

src/Aspire.Hosting/Pipelines/PipelineContext.cs

@@ -53,26 +53,12 @@ public sealed class PipelineContext(
     public CancellationToken CancellationToken { get; set; } = cancellationToken;
 
     /// <summary>
-    /// Gets the output path for deployment artifacts.
+    /// Gets the service for managing pipeline output directories.
     /// </summary>
-    public string OutputPath { get; } = outputPath ?? Path.Combine(Environment.CurrentDirectory, "aspire-output");
+    public IPipelineOutputService OutputService { get; } = new PipelineOutputService(outputPath, serviceProvider.GetRequiredService<IConfiguration>());
 
     /// <summary>
-    /// Gets the intermediate output path for temporary build artifacts.
+    /// Gets the output path for deployment artifacts.
     /// </summary>
-    public string IntermediateOutputPath { get; } = GetIntermediateOutputPath(serviceProvider);
-
-    private static string GetIntermediateOutputPath(IServiceProvider serviceProvider)
-    {
-        var configuration = serviceProvider.GetRequiredService<IConfiguration>();
-        var appHostSha = configuration["AppHost:PathSha256"];
-        
-        if (!string.IsNullOrEmpty(appHostSha))
-        {
-            return Directory.CreateTempSubdirectory($"aspire-{appHostSha}").FullName;
-        }
-        
-        // Fallback if AppHost:PathSha256 is not available
-        return Directory.CreateTempSubdirectory("aspire").FullName;
-    }
+    public string OutputPath => OutputService.GetOutputDirectory();
 }

src/Aspire.Hosting/Pipelines/PipelineOutputService.cs

@@ -0,0 +1,48 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using System.Diagnostics.CodeAnalysis;
+using Microsoft.Extensions.Configuration;
+
+namespace Aspire.Hosting.Pipelines;
+
+/// <summary>
+/// Default implementation of <see cref="IPipelineOutputService"/>.
+/// </summary>
+[Experimental("ASPIREPIPELINES001", UrlFormat = "https://aka.ms/aspire/diagnostics/{0}")]
+internal sealed class PipelineOutputService : IPipelineOutputService
+{
+    private readonly string? _outputPath;
+    private readonly Lazy<string> _tempDirectory;
+
+    public PipelineOutputService(string? outputPath, IConfiguration configuration)
+    {
+        _outputPath = outputPath;
+        _tempDirectory = new Lazy<string>(() => CreateTempDirectory(configuration));
+    }
+
+    /// <inheritdoc/>
+    public string GetOutputDirectory()
+    {
+        return _outputPath ?? Path.Combine(Environment.CurrentDirectory, "aspire-output");
+    }
+
+    /// <inheritdoc/>
+    public string GetTempDirectory()
+    {
+        return _tempDirectory.Value;
+    }
+
+    private static string CreateTempDirectory(IConfiguration configuration)
+    {
+        var appHostSha = configuration["AppHost:PathSha256"];
+
+        if (!string.IsNullOrEmpty(appHostSha))
+        {
+            return Directory.CreateTempSubdirectory($"aspire-{appHostSha}").FullName;
+        }
+
+        // Fallback if AppHost:PathSha256 is not available
+        return Directory.CreateTempSubdirectory("aspire").FullName;
+    }
+}

src/Aspire.Hosting/Pipelines/PipelineStepContext.cs

@@ -56,12 +56,12 @@ public sealed class PipelineStepContext
     public CancellationToken CancellationToken => PipelineContext.CancellationToken;
 
     /// <summary>
-    /// Gets the output path for deployment artifacts.
+    /// Gets the service for managing pipeline output directories.
     /// </summary>
-    public string OutputPath => PipelineContext.OutputPath;
+    public IPipelineOutputService OutputService => PipelineContext.OutputService;
 
     /// <summary>
-    /// Gets the intermediate output path for temporary build artifacts.
+    /// Gets the output path for deployment artifacts.
     /// </summary>
-    public string IntermediateOutputPath => PipelineContext.IntermediateOutputPath;
+    public string OutputPath => PipelineContext.OutputPath;
 }