Open archerzz opened 3 months ago
Per discussion,
Summary
property for InputOperation
in MGC, so we need to add it back.Summary
to Description
<= TCGC description
Description
to Details
<= TCGC details
autorest.csharp
, let's do the conversion in TypeSpecInputOperationConverter
so that we don't need to change the InputOperation
in itDescription
if empty in autorest.csharp
Describe the bug
Summary
property: https://github.com/microsoft/typespec/blob/81f897739eed8b481d2e07de8fe9770ba9f44994/packages/http-client-csharp/emitter/src/type/input-operation.ts#L20autorest.csharp
: https://github.com/Azure/autorest.csharp/blob/0f630c2bf2f7116c3ebf212ce245a75c92d53f16/src/AutoRest.CSharp/Common/Input/InputTypes/InputOperation.cs#L18autorest.csharp
, we'll always set theDescription
if it's not available: https://github.com/Azure/autorest.csharp/blob/27aa096093e4e2efc4aa5c916eafd1c3df4ab48f/src/AutoRest.CSharp/Common/Input/InputTypes/Serialization/TypeSpecInputOperationConverter.cs#L85-L89Reproduction
Check this PR which I tried to adopt TCGC properties regarding operation docs: https://github.com/microsoft/typespec/pull/4158 The root cause is that after adopting TCGC properties, the meaningful doc value is saved in
Summary
, instead ofDescription
. And theDescription
is set to the operation name.Questions
Summary
(for<summary>
) andDescription
(for<remarks>
)? In TCGC, or in emitter, or in our generator?Summary
?Checklist