Stop Format redistributing MSBuild

[rainersigwald]

Bump MSBuild's baseline version to a patched 17.11 and stop having format drop MSBuild assemblies in its folder--they should never be used from there.

[Copilot]

Pull Request Overview

This PR stops redistributing MSBuild assemblies by preventing the dotnet-format tool from copying unnecessary MSBuild assets and bumps the MSBuild baseline version to a patched 17.11 release.

• Updates the dotnet-format.csproj to add PackageReference elements for various Microsoft.Build packages with ExcludeAssets set to runtime, ensuring transitive dependencies do not bring MSBuild assemblies into the distribution folder.
• Bumps the MicrosoftBuildMinimumVersion in eng/Versions.props from 17.11.4 to 17.11.31 to align with the patched baseline.

Reviewed Changes

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

File	Description
src/BuiltInTools/dotnet-format/dotnet-format.csproj	Added PackageReferences with proper asset exclusions to avoid redistributing MSBuild assemblies.
eng/Versions.props	Updated MicrosoftBuildMinimumVersion to reflect the new patched MSBuild baseline.

Comments suppressed due to low confidence (2)

src/BuiltInTools/dotnet-format/dotnet-format.csproj:39

• Ensure that excluding runtime assets for Microsoft.Build and related packages does not affect any build task behaviors in environments that may require them.

<PackageReference Include="Microsoft.Build" ExcludeAssets="runtime" PrivateAssets="All" />

eng/Versions.props:202

• [nitpick] Document the rationale for updating the MicrosoftBuildMinimumVersion to 17.11.31, ensuring that maintainer instructions and release notes reflect this change.

<MicrosoftBuildMinimumVersion Condition="'$(DotNetBuildSourceOnly)' != 'true'">17.11.31</MicrosoftBuildMinimumVersion>

[jasonmalinowski]

I would expect this to break this:

sdk/src/BuiltInTools/dotnet-format/Workspaces/MSBuildWorkspaceLoader.cs

Lines 32 to 40 in 02ab00a

	Build.Framework.ILogger? binlog = null;
	if (binaryLogPath is not null)
	{
	binlog = new Build.Logging.BinaryLogger()
	{
	Parameters = binaryLogPath,
	Verbosity = Build.Framework.LoggerVerbosity.Diagnostic,
	};
	}

If it's not for some reason, ExcludeAssets="compile;runtime" might be wise to ensure it's not being used by this code's compilation too.

[jasonmalinowski]

@JoeRobich Do we have a tracking bug for your work which might make this unnecessary? Or at least FYI that we can potentially revert this soon enough.