Address PR feedback: add install parameter and use IsRunMode instead of IsPublishMode

Co-authored-by: tommasodotNET <12819039+tommasodotNET@users.noreply.github.com>

Author: Copilot

src/CommunityToolkit.Aspire.Hosting.Golang/GolangAppHostingExtension.cs

@@ -239,16 +239,18 @@ private static string GetDefaultGoBaseImage(string workingDirectory, IServicePro
     /// Ensures Go module dependencies are tidied before the application starts using <c>go mod tidy</c>.
     /// </summary>
     /// <param name="builder">The Golang app resource builder.</param>
+    /// <param name="install">When true (default), automatically runs go mod tidy before the application starts. When false, this method does nothing.</param>
     /// <param name="configureInstaller">Optional action to configure the installer resource.</param>
     /// <returns>A reference to the <see cref="IResourceBuilder{T}"/>.</returns>
     public static IResourceBuilder<GolangAppExecutableResource> WithGoModTidy(
         this IResourceBuilder<GolangAppExecutableResource> builder,
+        bool install = true,
         Action<IResourceBuilder<GoModInstallerResource>>? configureInstaller = null)
     {
         ArgumentNullException.ThrowIfNull(builder, nameof(builder));
 
-        // Only install packages during development, not in publish mode
-        if (!builder.ApplicationBuilder.ExecutionContext.IsPublishMode)
+        // Only install packages if in run mode and install is true
+        if (builder.ApplicationBuilder.ExecutionContext.IsRunMode && install)
         {
             var installerName = $"{builder.Resource.Name}-go-mod-tidy";
             var installer = new GoModInstallerResource(installerName, builder.Resource.WorkingDirectory);
@@ -271,16 +273,18 @@ public static IResourceBuilder<GolangAppExecutableResource> WithGoModTidy(
     /// Ensures Go module dependencies are downloaded before the application starts using <c>go mod download</c>.
     /// </summary>
     /// <param name="builder">The Golang app resource builder.</param>
+    /// <param name="install">When true (default), automatically runs go mod download before the application starts. When false, this method does nothing.</param>
     /// <param name="configureInstaller">Optional action to configure the installer resource.</param>
     /// <returns>A reference to the <see cref="IResourceBuilder{T}"/>.</returns>
     public static IResourceBuilder<GolangAppExecutableResource> WithGoModDownload(
         this IResourceBuilder<GolangAppExecutableResource> builder,
+        bool install = true,
         Action<IResourceBuilder<GoModInstallerResource>>? configureInstaller = null)
     {
         ArgumentNullException.ThrowIfNull(builder, nameof(builder));
 
-        // Only install packages during development, not in publish mode
-        if (!builder.ApplicationBuilder.ExecutionContext.IsPublishMode)
+        // Only install packages if in run mode and install is true
+        if (builder.ApplicationBuilder.ExecutionContext.IsRunMode && install)
         {
             var installerName = $"{builder.Resource.Name}-go-mod-download";
             var installer = new GoModInstallerResource(installerName, builder.Resource.WorkingDirectory);

src/CommunityToolkit.Aspire.Hosting.Golang/README.md

@@ -43,6 +43,14 @@ var golang = builder.AddGolangApp("golang", "../gin-api")
     .WithHttpEndpoint(env: "PORT");
 ```
 
+By default, `WithGoModTidy()` runs `go mod tidy` before the application starts (equivalent to `install: true`). You can disable this behavior by setting `install: false`:
+
+```csharp
+var golang = builder.AddGolangApp("golang", "../gin-api")
+    .WithGoModTidy(install: false)  // Does not run go mod tidy
+    .WithHttpEndpoint(env: "PORT");
+```
+
 ### Using `go mod download`
 
 To run `go mod download` before your application starts (to download dependencies without verification):
@@ -53,7 +61,15 @@ var golang = builder.AddGolangApp("golang", "../gin-api")
     .WithHttpEndpoint(env: "PORT");
 ```
 
-Both methods create an installer resource that runs before your application starts, ensuring dependencies are available. The installer resource appears as a child resource in the Aspire dashboard.
+Similarly, you can control whether the download runs:
+
+```csharp
+var golang = builder.AddGolangApp("golang", "../gin-api")
+    .WithGoModDownload(install: false)  // Does not run go mod download
+    .WithHttpEndpoint(env: "PORT");
+```
+
+Both methods create an installer resource that runs before your application starts when `install` is `true`, ensuring dependencies are available. The installer resource appears as a child resource in the Aspire dashboard.
 
 You can also customize the installer resource using the optional `configureInstaller` parameter:
 
@@ -66,7 +82,7 @@ var golang = builder.AddGolangApp("golang", "../gin-api")
     .WithHttpEndpoint(env: "PORT");
 ```
 
-> **Note:** The `WithGoModTidy` and `WithGoModDownload` methods only run during development. When publishing, the generated Dockerfile handles dependency management automatically.
+> **Note:** The `WithGoModTidy` and `WithGoModDownload` methods only run in run mode (when the application is started locally). They do not run when publishing, as the generated Dockerfile handles dependency management automatically.
 
 ## Publishing
 

tests/CommunityToolkit.Aspire.Hosting.Golang.Tests/ResourceCreationTests.cs

@@ -182,4 +182,34 @@ public void WithGoModDownloadNullBuilderThrows()
 
         Assert.Throws<ArgumentNullException>(() => builder.WithGoModDownload());
     }
+
+    [Fact]
+    public void GolangAppWithGoModTidyInstallFalseDoesNotCreateInstaller()
+    {
+        var builder = DistributedApplication.CreateBuilder();
+
+        builder.AddGolangApp("golang", "../../examples/golang/gin-api").WithGoModTidy(install: false);
+
+        using var app = builder.Build();
+
+        var appModel = app.Services.GetRequiredService<DistributedApplicationModel>();
+
+        var golangResource = Assert.Single(appModel.Resources.OfType<GolangAppExecutableResource>());
+        Assert.Empty(appModel.Resources.OfType<GoModInstallerResource>());
+    }
+
+    [Fact]
+    public void GolangAppWithGoModDownloadInstallFalseDoesNotCreateInstaller()
+    {
+        var builder = DistributedApplication.CreateBuilder();
+
+        builder.AddGolangApp("golang", "../../examples/golang/gin-api").WithGoModDownload(install: false);
+
+        using var app = builder.Build();
+
+        var appModel = app.Services.GetRequiredService<DistributedApplicationModel>();
+
+        var golangResource = Assert.Single(appModel.Resources.OfType<GolangAppExecutableResource>());
+        Assert.Empty(appModel.Resources.OfType<GoModInstallerResource>());
+    }
 }