Small trim warning cleanup

[baronfel]

@JeremyKuhne Here's a couple cleanups like we were talking about

[Copilot]

Pull request overview

This PR removes now-unnecessary AOT/trimming suppression attributes and simplifies CLI help/error handling so it’s consistently invoked from CommandBase (including the NativeAOT-bridged build).

Changes:

• Removed UnconditionalSuppressMessage attributes that referenced Newtonsoft.Json trim/AOT warnings in Windows installer-related types.
• Made ParseResult.ShowHelpOrErrorIfAppropriate() available in CLI_AOT builds and invoked it unconditionally from CommandBase.
• Updated a command constructor to rely on the base CommandBase behavior instead of calling help/error handling directly.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/Cli/dotnet/Installer/Windows/MsiPayload.cs	Removes trim/AOT suppression attributes from the manifest property.
src/Cli/dotnet/Installer/Windows/MsiPackageCache.cs	Removes trim/AOT suppression attributes from JSON-manifest related methods.
src/Cli/dotnet/Installer/Windows/InstallRequestMessage.cs	Removes trim/AOT suppression attributes from the JSON deserialization entry point.
src/Cli/dotnet/Extensions/ParseResultExtensions.cs	Adjusts CLI_AOT conditional compilation so help/error logic is available in AOT, while keeping debug-switch code excluded.
src/Cli/dotnet/Commands/Reference/List/ReferenceListCommand.cs	Drops redundant help/error invocation now handled by CommandBase.
src/Cli/dotnet/CommandBase.cs	Centralizes help/error handling by calling the extension method unconditionally from the base constructor.
src/Cli/dotnet-aot/dotnet-aot.csproj	Adds a commented toggle for AOT reference compatibility verification.

Comments suppressed due to low confidence (2)

src/Cli/dotnet/Installer/Windows/InstallRequestMessage.cs:158

• After removing the UnconditionalSuppressMessage attributes, this file no longer uses anything from System.Diagnostics.CodeAnalysis. With TreatWarningsAsErrors enabled, the now-unused using System.Diagnostics.CodeAnalysis; will trigger IDE0005 and break the build; please remove it from the file header.

    /// <summary>
    /// Converts a deserialized array of bytes into an <see cref="InstallRequestMessage"/>.
    /// </summary>
    /// <param name="bytes">The array of bytes to convert.</param>
    /// <returns>An <see cref="InstallRequestMessage"/>.</returns>
    public static InstallRequestMessage Create(byte[] bytes)
    {
        string json = Encoding.UTF8.GetString(bytes);
        return JsonSerializer.Deserialize(json, InstallerJsonSerializerContext.Default.InstallRequestMessage)

src/Cli/dotnet/Extensions/ParseResultExtensions.cs:238

• With CLI_AOT defined, the HandleDebugSwitch method is excluded, which makes using System.Diagnostics; at the top of this file unused in the AOT build. Since the repo sets TreatWarningsAsErrors=true, this will surface as IDE0005 and break the dotnet-aot build; please wrap that using in #if !CLI_AOT (or otherwise ensure it remains used under CLI_AOT).

#if !CLI_AOT
    [Conditional("DEBUG")]
    public static void HandleDebugSwitch(this ParseResult parseResult)
    {
        if (parseResult.HasOption(CommonOptions.DebugOption))

[baronfel]

/ba-g flaky test: Microsoft.DotNet.Cli.Run.Tests.RunFileTests_BuildCommands.Build_Library_MultiTarget