Skip to content

Imple no-query-in-collection rule - #6056

Draft
Mary Gao (MaryGao) wants to merge 1 commit into
Azure:mainfrom
MaryGao:fix/no-query-in-collection
Draft

Mary Gao (MaryGao) wants to merge 1 commit into
Azure:mainfrom
MaryGao:fix/no-query-in-collection

Conversation

@MaryGao

Copy link
Copy Markdown
Member

Swagger rule: documentation | implementation

Fixes part of #5940: check the union of TypeSpec list operations and collection-shaped GET paths. Preserve the existing query exemptions and diagnostic deduplication.

List operations may also trigger the independently configured no-query-in-post or no-query-in-point-op rules; their distinct policies remain unchanged.

Focused rule tests, dependency-closure build, repository formatting and lint passed. Real-spec comparison and the migration report are intentionally deferred until manual review and approval.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added int:azure-specs Run integration tests against azure-rest-api-specs lib:azure-resource-manager Issues for @azure-tools/typespec-azure-core library linter Issues related to linter rules labels Nov 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

All changed packages have been documented.

  • ✅ @azure-tools/typespec-azure-resource-manager
Show changes

@azure-tools/typespec-azure-resource-manager - fix ✏️

Check query parameters on list operations regardless of their route or HTTP method.

@pkg-pr-new

pkg-pr-new Bot commented Nov 11, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@azure-tools/typespec-azure-resource-manager@6056

commit: 35bba17

@github-actions

Copy link
Copy Markdown
Contributor

📦 Package size report

✅ No notable package size changes compared to the base branch.

14 package(s) with no notable change
Package Packed (base → head) Δ Packed Unpacked (base → head) Δ Unpacked
@azure-tools/typespec-azure-resource-manager 210.18 KB → 210.23 KB +57 B (+0.0%) 1.26 MB → 1.26 MB +159 B (+0.0%)
@azure-tools/azure-http-specs 155.02 KB → 155.02 KB — 1.27 MB → 1.27 MB —
@azure-tools/typespec-autorest 88.65 KB → 88.65 KB — 427.94 KB → 427.94 KB —
@azure-tools/typespec-autorest-canonical 7.42 KB → 7.42 KB — 26.00 KB → 26.00 KB —
@azure-tools/typespec-azure-core 144.66 KB → 144.66 KB — 771.39 KB → 771.39 KB —
@azure-tools/typespec-azure-examples 96.99 KB → 96.99 KB — 487.04 KB → 487.04 KB —
@azure-tools/typespec-azure-portal-core 36.33 KB → 36.33 KB — 243.09 KB → 243.09 KB —
@azure-tools/typespec-azure-rulesets 6.21 KB → 6.21 KB — 38.47 KB → 38.47 KB —
@azure-tools/typespec-client-generator-core 264.53 KB → 264.53 KB — 1.40 MB → 1.40 MB —
@azure-tools/typespec-go 303.39 KB → 303.39 KB — 1.55 MB → 1.55 MB —
@azure-tools/typespec-java 20.23 MB → 20.23 MB — 22.53 MB → 22.53 MB —
@azure-tools/typespec-metadata 15.88 KB → 15.88 KB — 62.08 KB → 62.08 KB —
@azure-tools/typespec-python 42.66 KB → 42.66 KB — 166.15 KB → 166.15 KB —
@azure-tools/typespec-ts 551.24 KB → 551.24 KB — 2.76 MB → 2.76 MB —

Packed = gzipped .tgz published to npm. Unpacked = total extracted size. 🆕 added, 🗑️ removed. Packages from the core/ submodule are not included.
🔴 grew · 🟢 shrank — only changes of at least 512 B and 0.5% are marked.

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

You can try these changes here

🛝 Playground 🌐 Website

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

int:azure-specs Run integration tests against azure-rest-api-specs lib:azure-resource-manager Issues for @azure-tools/typespec-azure-core library linter Issues related to linter rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant