Fix is form data

[tadelesh]

rollback #215
when we use the spread way to define the body parameter, each time we call getHttpOperation, we will get a new model type for body.

example: https://github.com/bterlson/typespec-todo/blob/ca8a3aab5f3ef52e52cf2b48d2980f808165fada/main.tsp#L293

[azure-sdk]

All changed packages have been documented.

[azure-sdk]

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/364/

Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/364/

[iscai-msft]

@tadelesh this is an http issue right? Shouldn't we wait for the http library to fix it?

[timotheeguerin]

@tadelesh this is an http issue right? Shouldn't we wait for the http library to fix it?

there is no issue in the http library, this is by design today we don't deduplicate types. You should not be calling it twice for the same operation anyway microsoft/typespec#2961

[tadelesh]

@tadelesh this is an http issue right? Shouldn't we wait for the http library to fix it?

as timothee said, it is by design, so i prefer to remove it temporally. i'm doing a refactor for getAllOperations and consolidate all our code to do only one call for getHttpOperation. after all emitter switch to it, many issues (alias, spread, etc.) could be resolved.

[iscai-msft]

@tadelesh can this be closed bc I'm assuming your caching getHttpOperation fixes this issue as well?

#395