Make it so the artifacts folders register files for cleanup (#49595)

Author: baronfel

src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.DefaultOutputPaths.targets

@@ -51,6 +51,11 @@ Copyright (c) .NET Foundation. All rights reserved.
     <ArtifactsBinOutputName Condition="'$(ArtifactsBinOutputName)' == ''">bin</ArtifactsBinOutputName>
     <ArtifactsPublishOutputName Condition="'$(ArtifactsPublishOutputName)' == ''">publish</ArtifactsPublishOutputName>
     <ArtifactsPackageOutputName Condition="'$(ArtifactsPackageOutputName)' == ''">package</ArtifactsPackageOutputName>
+    <!-- By default MSBuild won't allow tracking/automatic clean-up of files that are outside of a Project's child directory structure.
+         This flag opts us into a mode where the Common targets will track and clean such files, which is good because
+         in Artifacts layout virtually all transitive and package references are such files. 
+         See https://github.com/dotnet/msbuild/pull/12096 for full details. -->
+    <TrackFileWritesShareableOutsideOfProjectDirectory>true</TrackFileWritesShareableOutsideOfProjectDirectory>
   </PropertyGroup>
 
   <PropertyGroup Condition="'$(UseArtifactsOutput)' == 'true' And '$(ArtifactsPivots)' == ''">
@@ -90,9 +95,9 @@ Copyright (c) .NET Foundation. All rights reserved.
 
     <!-- The package output path does not include the project name, and only includes the Configuration as a pivot -->
     <PackageOutputPath Condition="'$(PackageOutputPath)' == ''">$(ArtifactsPath)\$(ArtifactsPackageOutputName)\$(Configuration.ToLowerInvariant())\</PackageOutputPath>
-    
+
   </PropertyGroup>
-  
+
   <PropertyGroup Condition="'$(UseArtifactsOutput)' != 'true'">
     <BaseOutputPath Condition="'$(BaseOutputPath)' == ''">bin\</BaseOutputPath>
     <BaseOutputPath Condition="!HasTrailingSlash('$(BaseOutputPath)')">$(BaseOutputPath)\</BaseOutputPath>
@@ -155,7 +160,7 @@ Copyright (c) .NET Foundation. All rights reserved.
 
     <NetSdkError Condition="'$(UseArtifactsOutput)' == 'true' and '$(_ArtifactsPathSetEarly)' != 'true'"
                  ResourceName="ArtifactsPathCannotBeSetInProject" />
-    
+
     <NetSdkError Condition="'$(_ArtifactsPathLocationType)' == 'ProjectFolder'"
                  ResourceName="UseArtifactsOutputRequiresDirectoryBuildProps" />
 

test/Microsoft.NET.Build.Tests/ArtifactsOutputPathTests.cs

@@ -531,6 +531,79 @@ public void ItCanBuildWithMicrosoftBuildArtifactsSdk()
             new FileInfo(Path.Combine(testAsset.Path, "MSBuildSdk", "obj", "Debug", ToolsetInfo.CurrentTargetFramework, "MSBuildSdk.dll")).Should().Exist();
 
         }
+
+        [Fact]
+        public void PublishingRegistersWrittenFilesForProperCleanup()
+        {
+            var testProject = new TestProject()
+            {
+                IsExe = true,
+                UseArtifactsOutput = true,
+            };
+
+            var hostfxrName =
+                RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? "hostfxr.dll" :
+                RuntimeInformation.IsOSPlatform(OSPlatform.Linux) ? "libhostfxr.so" :
+                "libhostfxr.dylib";
+
+            var testAsset = _testAssetsManager.CreateTestProject(testProject);
+
+            //  Now add a Directory.Build.props file setting UseArtifactsOutput to true
+            File.WriteAllText(Path.Combine(testAsset.Path, "Directory.Build.props"), """
+                <Project>
+                  <PropertyGroup>
+                    <UseArtifactsOutput>true</UseArtifactsOutput>
+                  </PropertyGroup>
+                </Project>
+                """);
+
+            var projectDir = Path.Combine(testAsset.Path, testAsset.TestProject.Name);
+
+            // publish the app
+            // we publish self-contained so that we include hostfxr.dll.
+            // if we don't clean up this file, when we build in Release,
+            // the generated exe will pick up the hostfxr and fail to run.
+            // so the only way to successfully run the exe is to clean up
+            // the hostfxr.dll and other self-contained files.
+            new DotnetPublishCommand(Log)
+                .WithWorkingDirectory(projectDir)
+                .Execute("--self-contained")
+                .Should()
+                .Pass();
+
+            var outputDir = new DirectoryInfo(OutputPathCalculator.FromProject(testAsset.Path, testProject).GetOutputDirectory(configuration: "release"));
+            outputDir.Should().Exist().And.HaveFile(hostfxrName);
+            LocateAndRunApp(outputDir);
+
+            var publishDir = new DirectoryInfo(OutputPathCalculator.FromProject(testAsset.Path, testProject).GetPublishDirectory(configuration: "release"));
+            publishDir.Should().Exist().And.HaveFile(hostfxrName);
+            LocateAndRunApp(publishDir);
+
+            // now build the app in Release configuration.
+            // now self-contained, so that we are forced to clean up the runtime
+            // files that were published.
+            new DotnetBuildCommand(Log)
+                .WithWorkingDirectory(projectDir)
+                .Execute("-c", "Release")
+                .Should()
+                .Pass();
+            outputDir.Should().Exist();
+            outputDir.Should().NotHaveFiles([hostfxrName]);
+            LocateAndRunApp(outputDir);
+
+            void LocateAndRunApp(DirectoryInfo root)
+            {
+                var appBinaryName = RuntimeInformation.IsOSPlatform(OSPlatform.Windows)
+                ? $"{testProject.Name}.exe"
+                : testProject.Name;
+                root.Should().HaveFiles([appBinaryName]);
+                var binary = root.GetFiles(appBinaryName).First();
+                new RunExeCommand(Log, binary.FullName)
+                    .Execute()
+                    .Should()
+                    .Pass();
+            }
+        }
     }
 
     namespace ArtifactsTestExtensions