Skip to content

Provide an operation id resolver #8632

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 11 commits into from
Oct 2, 2025
Merged

Provide an operation id resolver #8632

merged 11 commits into from
Oct 2, 2025

Conversation

timotheeguerin
Copy link
Member

@timotheeguerin timotheeguerin commented Oct 1, 2025 •
edited
Loading

fix #8511 (Resolve duplicate operationId)
fix #3080 (Provide a way to configure Operation id)

@microsoft-github-policy-service microsoft-github-policy-service bot added the emitter:openapi3 Issues for @typespec/openapi3 emitter label Oct 1, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 1, 2025 •
edited
Loading

All changed packages have been documented.

  • ✅ @typespec/compiler
  • ✅ @typespec/openapi3
Show changes

@typespec/openapi3 - feature ✏️

Add a new operation-id-strategy option.,> ,> - parent-container (default and previous behavior) Join operation name with its parent if applicable with an underscore,> - fqn Join the path from the service root to the operation with .,> - none Do not generate operation ids, only include explicit ones set with @operationId

@typespec/openapi3 - fix ✏️

Deduplicate operation ids that would resolve to the same one

@typespec/compiler - feature ✏️

[API] Allow using union in emitter schemas

timotheeguerin
timotheeguerin commented Oct 1, 2025
View reviewed changes
timotheeguerin
timotheeguerin commented Oct 1, 2025
View reviewed changes
@azure-sdk
Copy link
Collaborator

azure-sdk commented Oct 1, 2025 •
edited
Loading

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

witemple-msft
witemple-msft reviewed Oct 1, 2025
View reviewed changes
@timotheeguerin timotheeguerin marked this pull request as ready for review October 1, 2025 20:06
witemple-msft
witemple-msft approved these changes Oct 2, 2025
View reviewed changes
@timotheeguerin timotheeguerin added this pull request to the merge queue Oct 2, 2025
Merged via the queue into microsoft:main with commit 5e9dba1 Oct 2, 2025
26 checks passed
@timotheeguerin timotheeguerin deleted the operation-id-strategy branch October 2, 2025 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@witemple-msft witemple-msft witemple-msft approved these changes

@bterlson bterlson Awaiting requested review from bterlson bterlson is a code owner

@markcowl markcowl Awaiting requested review from markcowl markcowl is a code owner

@chrisradek chrisradek Awaiting requested review from chrisradek chrisradek is a code owner

@AlitzelMendez AlitzelMendez Awaiting requested review from AlitzelMendez AlitzelMendez is a code owner

Assignees
No one assigned
Labels
emitter:openapi3 Issues for @typespec/openapi3 emitter
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Bug]: openapi same name inner namespaces results in duplicate operationId OpenAPI3 Emitter could provide a way to configure operationId
3 participants
@timotheeguerin @azure-sdk @witemple-msft