Fix subscription ID not being disabled on Azure provisioning dialog

[JamesNK]

Description

PR #12172 added a new tenant ID option. However, if there is no subscription ID then the choice input is left enabled.

PR changes:

• Subscription ID is disabled until a tenant is selected
• Removes unnessessary dynamic load options whe subscription ID is a disabled textbox

Fixes #12882

Checklist

• Is this feature complete?
  • Yes. Ready to ship.
  • No. Follow-up changes expected.
• Are you including unit tests for the changes and scenario tests if relevant?
  • Yes
  • No
• Did you add public API?
  • Yes
    • If yes, did you have an API Review for it?
      • Yes
      • No
    • Did you add <remarks /> and <code /> elements on your triple slash comments?
      • Yes
      • No
  • No
• Does the change make any security assumptions or guarantees?
  • Yes
    • If yes, have you done a threat model and had a security review?
      • Yes
      • No
  • No
• Does the change require an update in our Aspire docs?
  • Yes
    • Link to aspire-docs issue (consider using one of the following templates):
      • New (or update) doc-idea template
      • New breaking-change template
      • New diagnostic template
  • No

[github-actions]

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 12888

Or

• Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 12888"

[Copilot]

Pull Request Overview

This PR fixes a bug where the subscription ID input field was not properly disabled when there's no pre-configured subscription ID. The issue was introduced in PR #12172 which added tenant ID selection.

Key changes:

• Conditional creation of InputLoadOptions for subscription ID (only when not pre-configured)
• Always initialize subscription ID input as disabled
• Removed redundant null check inside the LoadCallback

[davidfowl]

Does this preserve when the subscription is already set from config? (Which means the tenant isn’t required)

[JamesNK]

I think so. The changes are minor (listed in PR body).

I would test, but I don't know how to get just a SubscriptionId preset but still have the dialog pop up.

[davidfowl]

Set Azure:SubscriptionId in your config

[JamesNK]

It's already broken on main.

Location input is disabled and is never enabled when there is an Azure:SubscriptionId in the config.

[JamesNK]

I believe I've fixed both scenarios. Please double check.

[JamesNK]

Unrelated. Label width not being the same as input has bugged me

[JamesNK]

/backport to release/13.0

[github-actions]

Started backporting to release/13.0: https://github.com/dotnet/aspire/actions/runs/19281772994