Azure / typespec-azure

About TypeSpec Azure Libraries
https://azure.github.io/typespec-azure/
MIT License
11 stars 37 forks source link

Spread is spread (parameters) #1043

Closed iscai-msft closed 1 month ago

iscai-msft commented 3 months ago

Clear and concise description of the problem

We no longer want to recover a model from spread by default. The current behavior creates a complicated story for spreading, and leads to confusing emitter logic. The following needs to be done in this order

  1. [x] Decorate brownfield services with @bodyRoot
  2. [x] Update emitter logic
  3. [x] Add overrideClientMethod decorator
  4. [ ] Move vendored BodyParameter to core

The final step is not required for the story to be complete, I'm only linking it in this issue as a reference.

Down the line, we also want to look into clearing up the logic in responses too. We will wait for a traits redesign first.

Checklist

weidongxu-microsoft commented 2 months ago

typespec-java 0.18.0 released with this logic

iscai-msft commented 1 month ago

Closing because all of the tcgc work has been completed with the introduction of the @override decorator