[AppService] Fix #33732: az webapp show: Restore appServicePlanId in output

[Shreyas Gopalakrishna (shreyas-gopalakrishna)]

Related command

• az webapp show
• az webapp list
• az functionapp list --output table

Description

Fixes #33732
Fixes #33914

Supersedes #33733.

azure-mgmt-web 11.0.0 changed Site models to TypeSpec-generated hybrid models. Adding an undeclared app_service_plan_id attribute no longer affects CLI serialization, so az webapp show/list exposed serverFarmId instead of the established appServicePlanId field. The missing field also caused table formatting to raise KeyError for affected Function App responses.

This change:

• Serializes Site models before normalizing their output.
• Restores the pre-upgrade Web App shape: appServicePlanId is present and serverFarmId is removed from az webapp show/list.
• Adds appServicePlanId while retaining serverFarmId for Function App and Logic App compatibility.
• Makes table formatting tolerate responses with no App Service Plan ID.
• Adds SDK-model unit coverage and Web App list/show scenario assertions.

Testing Guide

az webapp show -g <resource-group> -n <webapp> --query "{appServicePlanId:appServicePlanId,serverFarmId:serverFarmId}"
az webapp list -g <resource-group> --query "[].{name:name,appServicePlanId:appServicePlanId,serverFarmId:serverFarmId}"
az functionapp list -g <resource-group> --output table

Local validation:

• 182 App Service tests passed, plus 2 subtests.
• azdev style appservice
• azdev linter appservice
• git diff --check

History Notes

[AppService] az webapp show/list: Restore appServicePlanId in output
[AppService] az functionapp list --output table: Handle apps without an App Service Plan ID

---

This checklist is used to make sure that common guidelines for a pull request are followed.

• The PR title and description has followed the guideline in Submitting Pull Requests.

• I adhere to the Command Guidelines.

• I adhere to the Error Handling Guidelines.

[azure-pipelines]

Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

[Copilot]

Copilot review overview

🟢 Approval recommended

The implementation addresses both reported regressions with focused compatibility coverage.

Review effort: Balanced
Findings: None

What changed in this PR

Restores established App Service Plan identifiers in AppService command output and prevents table-formatting failures.

Changes:

• Normalizes serialized Site models for Web, Function, and Logic Apps.
• Safely handles missing plan IDs.
• Adds unit and scenario coverage.

File	Description
appservice/​commands.py	Adds output normalization and safe table formatting.
tests/​latest/​test_webapp_commands.py	Verifies Web App list/show output compatibility.
tests/​latest/​test_webapp_commands_thru_mock.py	Tests model serialization and missing-plan behavior.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Shreyas Gopalakrishna (shreyas-gopalakrishna)]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 34149 in repo Azure/azure-cli

[Aditya Pujara (a0x1ab)]

/azp run

[azure-pipelines]

Azure Pipelines:
Successfully started running 3 pipeline(s).

[Shreyas Gopalakrishna (shreyas-gopalakrishna)]

Aditya Pujara (@a0x1ab) Yong Zhang (@yonzhan) could you review this PR
Python312 tests got canceled due to the 10 min timeout, please reschedule it. I do not have access

[Yong Zhang (yonzhan)]

Please fix CI issues

[Shreyas Gopalakrishna (shreyas-gopalakrishna)]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 34149 in repo Azure/azure-cli

[Yong Zhang (yonzhan)]

/azp run

[azure-pipelines]

Azure Pipelines:
Successfully started running 3 pipeline(s).