Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
63 changes: 13 additions & 50 deletions src/Websites/Websites.Autorest/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,6 @@ input-file:
title: Websites
module-version: 0.1.0
subject-prefix: $(service-name)
identity-correction-for-post: true
resourcegroup-append: true
nested-object-to-string: true

# For new modules, please avoid setting 3.x using the use-extension method and instead, use 4.x as the default option
use-extension:
"@autorest/powershell": "3.x"

directive:
#Modify operationId
Expand Down Expand Up @@ -206,58 +199,20 @@ directive:
verb: Unregister
subject: BuildUserProvidedFunctionApp

# Remove variant
# Remove variant
# Following is two common directive which are normally required in all the RPs
# 1. Remove the unexpanded parameter set
# 2. For New-* cmdlets, ViaIdentity is not required, so CreateViaIdentityExpanded is removed as well
- where:
variant: ^CreateViaIdentityExpanded$|^Create$|^CreateViaIdentity$|^Update$|^UpdateViaIdentity$
# We got to keep the Create variant of CustomDomain because it's special that it doesn't have a
# CreateExpanded variant, because the only parameters are all in URL rather than request body
subject: CustomDomain

remove: true
- where:
verb: Test
variant: ^Validate$|^ValidateViaIdentity$
# We got to keep the Create variant of CustomDomain because it's special that it doesn't have a
# CreateExpanded variant, because the only parameters are all in URL rather than request body
subject: CustomDomain
remove: true

- where:
variant: ^Create$|^CreateViaIdentity$|^Update$|^UpdateViaIdentity$
# We got to keep the Create variant of CustomDomain because it's special that it doesn't have a
# CreateExpanded variant, because the only parameters are all in URL rather than request body
subject: BuildAppSetting
remove: true

- where:
variant: ^Create$|^CreateViaIdentity$|^Update$|^UpdateViaIdentity$
# We got to keep the Create variant of CustomDomain because it's special that it doesn't have a
# CreateExpanded variant, because the only parameters are all in URL rather than request body
subject: FunctionAppSetting
remove: true

- where:
variant: ^Create$|^CreateViaIdentity$|^Update$|^UpdateViaIdentity$
# We got to keep the Create variant of CustomDomain because it's special that it doesn't have a
# CreateExpanded variant, because the only parameters are all in URL rather than request body
subject: Setting
subject: CustomDomain|BuildAppSetting|FunctionAppSetting|Setting|BuildFunctionAppSetting|UserRoleInvitationLink
variant: ^Create(?!.*?(Expanded|JsonFilePath|JsonString))
remove: true
Comment on lines +202 to 209
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment says that for New-* cmdlets the CreateViaIdentityExpanded variant is removed, but the directives only remove CreateViaIdentityExpanded for CustomDomain and the generated help still includes CreateViaIdentityExpanded for other New-AzStaticWebApp* cmdlets (e.g., settings). Please align the comment with the actual directive behavior (or expand the directive if the intent is to remove these variants across the listed subjects).

Copilot uses AI. Check for mistakes.

- where:
variant: ^Create$|^CreateViaIdentity$|^Update$|^UpdateViaIdentity$
variant: ^CreateViaIdentityExpanded$
# We got to keep the Create variant of CustomDomain because it's special that it doesn't have a
# CreateExpanded variant, because the only parameters are all in URL rather than request body
subject: BuildFunctionAppSetting
remove: true

- where:
variant: ^Create$|^CreateViaIdentity$|^Update$|^UpdateViaIdentity$
# We got to keep the Create variant of CustomDomain because it's special that it doesn't have a
# CreateExpanded variant, because the only parameters are all in URL rather than request body
subject: UserRoleInvitationLink
subject: CustomDomain
remove: true
Comment on lines 211 to 216
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment above the variant: ^CreateViaIdentityExpanded$ removal for CustomDomain appears outdated/inaccurate: it claims CustomDomain has no CreateExpanded variant and that parameters are only in the URL, but the generated help now shows CreateExpanded (Default) for New-AzStaticWebAppCustomDomain. Please update/remove this comment (and/or re-evaluate the directive) so future maintainers aren’t misled about why the variant is removed.

Copilot uses AI. Check for mistakes.

- where:
Expand All @@ -272,6 +227,14 @@ directive:
variant: ^Update$|^UpdateViaIdentity$
remove: true

- where:
verb: Test
variant: ^Validate$|^ValidateViaIdentity$
# We got to keep the Create variant of CustomDomain because it's special that it doesn't have a
# CreateExpanded variant, because the only parameters are all in URL rather than request body
subject: CustomDomain
remove: true

- where:
verb: Reset
subject: ApiKey
Expand Down
Loading