Examples multi response

[timotheeguerin]

No description provided.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http@10180

npm i https://pkg.pr.new/@typespec/openapi3@10180

commit: 038d130

[github-actions]

All changed packages have been documented.

• ✅ @typespec/http
• ✅ @typespec/openapi3

Show changes

@typespec/http - feature ✏️

[API] Operation returning a union of types without status code or content type will be treated as a single response

@typespec/openapi3 - fix ✏️

Fix examples when operation return type have union of response mapping to same status code

[azure-sdk]

You can try these changes here

🛝 Playground	🌐 Website	🛝 VSCode Extension

[markcowl]

It feels like we should have a discriminated union test, and a test over the union variant flattening logic.

Also wonder about the potential for breaking changes in emitters given the changes in the way responses are processed

[timotheeguerin]

@markcowl good point, will add test in http as well.
For the breaking change, it did hit one rule test in azure core because the test was returning : Lro | {erorr: string} combing them both now as 200 response. It used to work because we'd be happy the first 200 returned a LRO but this feels like to me it caught a bug ( error was also encoded as 200 but didn't have lro annotations)