Make Azure.ResourceManager AOT-compatible

[agocke]

There were only two places which were not AOT compatible:

1. Use of non-source generated JSON.
2. One method which uses non-generated configuration bindings, which I've simply marked incompatible for now.

Note: almost all of this code was generated by VS Code CoPilot w/ Agent Mode. If you're interested in the prompts I used and the chat session, contact me privately and I'd be happy to share the whole session.

[github-actions]

Thank you for your contribution @agocke! We will review the pull request and get back to you soon.

[Copilot]

Pull Request Overview

This PR updates Azure.ResourceManager to be AOT-compatible by replacing non-source generated JSON with source-generated contexts and marking incompatible configuration bindings.

• Added attributes in ArmClientBuilderExtensions to flag reflection and dynamic code usage.
• Introduced new JSON context classes and updated serialization/deserialization calls to use them.
• Updated the project file and ArmClientOptions to support AOT compatibility.

Reviewed Changes

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

Show a summary per file

File	Description
sdk/resourcemanager/Azure.ResourceManager/src/Extensions/ArmClientBuilderExtensions.cs	Added AOT-related attributes to the AddArmClient method.
sdk/resourcemanager/Azure.ResourceManager/src/Common/Custom/Models/ManagedServiceIdentityJsonContext.cs	Added a new JSON source generation context.
sdk/resourcemanager/Azure.ResourceManager/src/Common/Custom/Models/ManagedServiceIdentity.Serialization.cs	Updated JSON serialization/deserialization to use the new context.
sdk/resourcemanager/Azure.ResourceManager/src/Azure.ResourceManager.csproj	Added AOT compatibility property.
sdk/resourcemanager/Azure.ResourceManager/src/ArmEnvironmentJsonContext.cs	Introduced a context for ArmEnvironment JSON serialization.
sdk/resourcemanager/Azure.ResourceManager/src/ArmEnvironment.cs	Updated ToString to utilize the source-generated context.
sdk/resourcemanager/Azure.ResourceManager/src/ArmClientOptionsJsonContext.cs	Added a JSON context for ArmClientOptions.
sdk/resourcemanager/Azure.ResourceManager/src/ArmClientOptions.cs	Migrated deserialization to use the new JSON context.

[jsquire]

@m-nash: Could you give this another pair of eyes from the management + MRW perspective?
@m-redding: Could you give this an AOT look?

[agocke]

FYI, still trying to repro those code generation failures locally. For some reason restore is taking forever

[jsquire]

LGTM, though I lack in-depth knowledge of the management packages. I've pulled a couple of others in for additional perspectives.

[jsquire]

@agocke: The failures are:

• The new attributes constitute a change in the API shape, so you'll need to export via: eng\scripts\Export-API.ps1 core
• The serialization format for managed identity seems to have changed enough that the recording no longer matches. That may or may not be a concern.

[agocke]

I've fixed up all the formatting and added the attribute, so I think this is ready to go.

[m-redding]

This LGTM from my perspective although I also don't have much knowledge on the mgmt packages

[jsquire]

@ArthurMa1978, @ArcturusZhang, @live1206, @m-nash: Folks, can you please give this a quick look from the management ownership perspective? Since everything is green, I'm going to move forward with the merge tomorrow EOD if nobody registers any objections or questions.

[m-nash]

I think we want to switch this to SCM SG to stay in line with the other core libraries. Spoke with @agocke today offline and we will come up with a overarching plan to fix the cross library JsonSerializer calls

[m-nash]

Changes look good to me