Skip to content

Fix tuple resource list handling in mgmt generator#59363

Draft
ArcturusZhang wants to merge 1 commit into
Azure:mainfrom
ArcturusZhang:fix/tuple-resource-list-handling
Draft

Fix tuple resource list handling in mgmt generator#59363
ArcturusZhang wants to merge 1 commit into
Azure:mainfrom
ArcturusZhang:fix/tuple-resource-list-handling

Conversation

@ArcturusZhang
Copy link
Copy Markdown
Member

Summary

  • classify basic array-returning operations as resource list methods
  • support tuple-resource intermediate collection paths by matching operation paths as prefixes of resource instance paths
  • emit all collection GetAll overloads when multiple list operations target the same resource collection

Validation

  • npm exec -- vitest run -c .\emitter\vitest.config.ts .\emitter\test\resource-detection.test.ts
  • npm run prettier
  • npm run lint
  • dotnet test eng\packages\http-client-csharp-mgmt\generator\Azure.Generator.Management\test\Azure.Generator.Mgmt.Tests.csproj --filter ResourceCollectionClientProviderTests
  • RegenSdkLocal.ps1 -Services "Azure.ResourceManager.Compute" -LocalSpecRepoPath "C:\Users\dapzhang\Documents\workspace\spec-compute-wt"
  • dotnet build sdk\compute\Azure.ResourceManager.Compute\tests\Azure.ResourceManager.Compute.Tests.csproj /p:RunApiCompat=false /p:ApiCompatValidateAssemblies=false /p:ApiCompatValidatePackage=false

Classify array-returning tuple resource list operations as resource lists and emit all matching collection GetAll overloads so Compute extension image list operations remain available.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added CodeGen Issues that relate to code generation Mgmt This issue is related to a management package. labels May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CodeGen Issues that relate to code generation Mgmt This issue is related to a management package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant