Merge branch 'release/10.0.2xx' => 'main'

[dsplaisted]

Fixed version of #51720, where I merged incorrectly.

[dotnet-policy-service]

Thanks for your PR, @@dsplaisted.
To learn about the PR process and branching schedule of this repo, please take a look at the SDK PR Guide.

[Copilot]

Pull Request Overview

This PR merges the release/10.0.2xx branch into main, representing a significant refactoring of the dotnet-watch functionality. The changes extract shared watch/hot-reload functionality into reusable library projects, add support for Aspire-based watching, and update various dependencies.

Key changes:

• Extraction of watch functionality into new Microsoft.DotNet.HotReload.Watch library project
• Addition of Microsoft.DotNet.HotReload.Watch.Aspire tool package for Aspire CLI support
• Creation of shared test utilities project (Microsoft.DotNet.HotReload.Test.Utilities)
• Update to use Microsoft.CodeAnalysis.ExternalAccess.HotReload API (replacing the Watch-specific API)
• Package updates including Microsoft.Build.Locator (1.8.1 → 1.10.2) and MSBuild minimum version
• Removal of VS SDK dependencies

Reviewed Changes

Copilot reviewed 75 out of 120 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
test/dotnet-watch.Tests/dotnet-watch.Tests.csproj	Updated project references and fixed comment typo
test/dotnet-watch.Tests/TestUtilities/WatchableApp.cs	Updated AwaitableProcess API usage
test/dotnet-watch.Tests/TestUtilities/TestOptions.cs	Moved GetWatchTestOutputPath to shared utilities
test/dotnet-watch.Tests/CommandLine/CommandLineOptionsTests.cs	Removed build property parsing tests (moved to BuildUtilitiesTests)
test/dotnet-watch.Tests/Build/BuildUtilitiesTests.cs	New test file for BuildUtilities.ParseBuildProperties
test/TestAssets/TestProjects/*/Program.cs	Added pragma warnings for obsolete API usage
test/Microsoft.WebTools.AspireService.Tests/Microsoft.WebTools.AspireService.Tests.csproj	Updated AssertEx.cs reference to shared utilities
test/Microsoft.NET.Build.Tests/RoslynBuildTaskTests.cs	Updated method reference from AppHostCompilerFileName to DotNetExecCompilerFileName
test/Microsoft.Extensions.DotNetDeltaApplier.Tests/Microsoft.Extensions.DotNetDeltaApplier.Tests.csproj	Updated references to shared test utilities
test/Microsoft.DotNet.HotReload.Client.Tests/Microsoft.DotNet.HotReload.Client.Tests.csproj	Updated AssertEx.cs reference
test/Microsoft.DotNet.HotReload.Test.Utilities/*	New shared test utilities project
test/Microsoft.DotNet.HotReload.Watch.Aspire.Tests/*	New test project for Aspire watch functionality
test/Common/Program.cs	Namespace reorganization
src/RazorSdk/Tool/GenerateCommand.cs	Added missing System.Threading using
src/Cli/dotnet/Commands/xlf/*.xlf	Updated localization files with needs-review-translation state
src/Cli/dotnet/Commands/Run/CSharpCompilerCommand.Generated.cs	Added warning 8002 to nowarn list
src/Cli/dotnet/Commands/CliCommandStrings.resx	Updated ProjectConvertAskForOutputDirectory message
src/BuiltInTools/dotnet-watch/*	Refactored to use new Watch library
src/BuiltInTools/Watch/*	New library project with extracted watch functionality
src/BuiltInTools/Watch.Aspire/*	New Aspire-specific watch tool
src/BuiltInTools/DotNetDeltaApplier/Microsoft.Extensions.DotNetDeltaApplier.csproj	Added DisableTransitiveFrameworkReferenceDownloads property
sdk.slnx, dotnet-watch.slnf	Added new projects to solution files
eng/Versions.props	Updated MSBuild and MSBuild.Locator versions, removed VS SDK
eng/Signing.props	Removed StreamJsonRpc.dll signing entry
eng/Publishing.props	Added workload MSI internal publishing configuration
NuGet.config	Removed vssdk package sources
Directory.Packages.props	Added HotReload external access package, removed VS SDK packages

[dsplaisted]

@DonnaChen888 After talking with other members of my team it sounds like I may have given you some incorrect instructions around how to handle merges between branches and what files to ignore. So take what I said with a grain of salt, and you may not want to change how you were doing things.

I think I may have figured out how to do the branch merge this PR now , we'll see how it goes.

[dsplaisted]

@jjonescz @dotnet/run-file Can someone fix the Microsoft.DotNet.Cli.Run.Tests.RunFileTests.CscArguments test? It looks like it's failing because there's an additional NoWarn for warning 8002.

@dotnet/roslyn @jaredpar Can someonne fix the Microsoft.NET.Build.Tests.RoslynBuildTaskTests.FullMSBuild_SdkStyle_ToolsetPackage test? It looks like it's failing because csc.dll and vbc.dll changed to csc.exe and vbc.exe.

Thanks!

[jjonescz]

Can someone fix the Microsoft.DotNet.Cli.Run.Tests.RunFileTests.CscArguments test? It looks like it's failing because there's an additional NoWarn for warning 8002.

It's actually failing because the NoWarn is missing here; which seems unexpected (the most "up to date" state is that it should be present, that's why it was added in the 10.0.2xx branch in #51612 (comment)).

The reason is that roslyn version in 10.0.2xx is newer than in this flow, that seems wrong. Is this a bad merge?

Can someonne fix the Microsoft.NET.Build.Tests.RoslynBuildTaskTests.FullMSBuild_SdkStyle_ToolsetPackage test? It looks like it's failing because csc.dll and vbc.dll changed to csc.exe and vbc.exe.

I think this is failing for the same reason - the roslyn version in this branch is outdated.

[jjonescz]

Where does this version come from? Even though it's newer than the version in 10.0.2xx (which is 5.3.0-1.25562.107), this one seems to actually contain an older Roslyn version (and hence the tests are still failing). I think that's possible if multiple VMR branches build different versions of Roslyn or something like that. But if we choose only versions from VMR subscription PRs, I think that shouldn't be a problem. But I suspect you chose this version manually?

Suggested change

	<MicrosoftCodeAnalysisPackageVersion>5.3.0-1.25567.101</MicrosoftCodeAnalysisPackageVersion>
	<MicrosoftCodeAnalysisPackageVersion>5.3.0-1.25562.107</MicrosoftCodeAnalysisPackageVersion>

[dsplaisted]

@jjonescz Thanks for looking. I ran darc update-dependencies --channel ".NET 11.0.1xx SDK" --source-repo dotnet/dotnet, which I thought would update to the correct versions from the VMR. Is there something else I should have done? Or have some Roslyn changes not merged to main in the VMR?

[jjonescz]

Oh, right, I guess you did it correctly, but it's true that Roslyn didn't flow to VMR main yet. Although I think that's also blocked by the fact that changes from SDK 10.0.2xx didn't flow to VMR main yet (see dotnet/dotnet#3147 (comment)). So we are kinda in a deadlock here.

But I guess I can revert the test updates so the Roslyn tests will be passing for the old Roslyn version and then update them again for the new version when that's unblocked...

[dsplaisted]

@dotnet/aspnet-blazor-eng @ericstj @rainersigwald A bunch of Blazor tests are failing with MissingMethodExceptions like this in full Framework builds:

"C:\h\w\AA5D0941\t\dotnetSdkTests\ih23pbdq.sng\Publish_Hoste---F07A04C5\classlibrarywithsatelliteassemblies\classlibrarywithsatelliteassemblies.csproj" (default target) (3:6) ->
(LoadStaticWebAssetsBuildManifest target) ->
obj\Debug\net10.0\staticwebassets.build.json : error : MissingMethodException: Method not found: '!!0 System.Text.Json.JsonSerializer.Deserialize(System.ReadOnlySpan1<Byte>, System.Text.Json.Serialization.Metadata.JsonTypeInfo1<!!0>)'. [C:\h\w\AA5D0941\t\dotnetSdkTests\ih23pbdq.sng\Publish_Hoste---F07A04C5\classlibrarywithsatelliteassemblies\classlibrarywithsatelliteassemblies.csproj]
obj\Debug\net10.0\staticwebassets.build.json : error :    at Microsoft.AspNetCore.StaticWebAssets.Tasks.StaticWebAssetsManifest.FromJsonBytes(Byte[] jsonBytes) [C:\h\w\AA5D0941\t\dotnetSdkTests\ih23pbdq.sng\Publish_Hoste---F07A04C5\classlibrarywithsatelliteassemblies\classlibrarywithsatelliteassemblies.csproj]
obj\Debug\net10.0\staticwebassets.build.json : error :    at Microsoft.AspNetCore.StaticWebAssets.Tasks.ReadStaticWebAssetsManifestFile.Execute() in /_/src/StaticWebAssetsSdk/Tasks/ReadStaticWebAssetsManifestFile.cs:line 52 [C:\h\w\AA5D0941\t\dotnetSdkTests\ih23pbdq.sng\Publish_Hoste---F07A04C5\classlibrarywithsatelliteassemblies\classlibrarywithsatelliteassemblies.csproj]
obj\Debug\net10.0\staticwebassets.build.json : error :  [C:\h\w\AA5D0941\t\dotnetSdkTests\ih23pbdq.sng\Publish_Hoste---F07A04C5\classlibrarywithsatelliteassemblies\classlibrarywithsatelliteassemblies.csproj]

The failure doesn't seem to repro locally for me, perhaps I have a newer version of VS with a newer version of System.Text.Json. Did this change recently?