Remove OutputPath property from PipelineContext and PipelineStepContext

- Remove OutputPath convenience property per @davidfowl feedback
- Consumers now explicitly call OutputService.GetOutputDirectory()
- Update AzureEnvironmentResource to use OutputService.GetOutputDirectory()
- Update PublishingContextUtils to use OutputService.GetOutputDirectory()
- Update test helper to use OutputService.GetOutputDirectory()
- Keep OutputService property for explicit access to output management

Co-authored-by: davidfowl <95136+davidfowl@users.noreply.github.com>

Author: Copilot

src/Aspire.Hosting.Azure/AzureEnvironmentResource.cs

@@ -122,7 +122,7 @@ private Task PublishAsync(PipelineStepContext context)
     {
         var azureProvisioningOptions = context.Services.GetRequiredService<IOptions<AzureProvisioningOptions>>();
         var publishingContext = new AzurePublishingContext(
-            context.OutputPath,
+            context.OutputService.GetOutputDirectory(),
             azureProvisioningOptions.Value,
             context.Services,
             context.Logger,

src/Aspire.Hosting/Pipelines/PipelineContext.cs

@@ -56,9 +56,4 @@ public sealed class PipelineContext(
     /// Gets the service for managing pipeline output directories.
     /// </summary>
     public IPipelineOutputService OutputService { get; } = new PipelineOutputService(outputPath, serviceProvider.GetRequiredService<IConfiguration>());
-
-    /// <summary>
-    /// Gets the output path for deployment artifacts.
-    /// </summary>
-    public string OutputPath => OutputService.GetOutputDirectory();
 }

src/Aspire.Hosting/Pipelines/PipelineStepContext.cs

@@ -59,9 +59,4 @@ public sealed class PipelineStepContext
     /// Gets the service for managing pipeline output directories.
     /// </summary>
     public IPipelineOutputService OutputService => PipelineContext.OutputService;
-
-    /// <summary>
-    /// Gets the output path for deployment artifacts.
-    /// </summary>
-    public string OutputPath => PipelineContext.OutputPath;
 }

src/Shared/PublishingContextUtils.cs

@@ -15,10 +15,10 @@ public static string GetEnvironmentOutputPath(PipelineStepContext context, IComp
         if (context.Model.Resources.OfType<IComputeEnvironmentResource>().Count() > 1)
         {
             // If there are multiple compute environments, append the environment name to the output path
-            return Path.Combine(context.OutputPath, environment.Name);
+            return Path.Combine(context.OutputService.GetOutputDirectory(), environment.Name);
         }
 
         // If there is only one compute environment, use the root output path
-        return context.OutputPath;
+        return context.OutputService.GetOutputDirectory();
     }
 }

tests/Aspire.Hosting.Tests/Helpers/JsonDocumentManifestPublisher.cs

@@ -78,7 +78,7 @@ public static IDistributedApplicationPipeline AddJsonDocumentManifestPublishing(
                 using var stream = new MemoryStream();
                 using var writer = new Utf8JsonWriter(stream, new() { Indented = true });
 
-                var manifestPath = context.OutputPath ?? "aspire-manifest.json";
+                var manifestPath = context.OutputService.GetOutputDirectory();
                 var publishingContext = new ManifestPublishingContext(executionContext, manifestPath, writer, context.CancellationToken);
 
                 await publishingContext.WriteModel(context.Model, context.CancellationToken).ConfigureAwait(false);