#673 Add ChatReasoningEffortLevel.Minimal value

[kurnakovv]

#673

[christothes]

Thank you for your contribution @kurnakovv!

This PR modifies generated code. Instead, it should modify the TypeSpec spec that defines the ReasoningEffort type.

That said, there is an updated spec in the works that already includes this value, so this change will show up as soon as we incorporate the new spec and regenerate.

[jsquire]

Hi @kurnakovv. As Chris mentioned here and I detailed in the associated issue, the value must be added to the REST API specification in order for it to appear in the client. While we maintain a local copy of the spec in the repository, the official version is authored upstream and then flows to us. Anything that is changed in the local spec or the C# code generated from it will be overwritten when the official spec is next released.

Since we're unable to move forward with this contribution, I'm going to close out the PR. Thank you, again, for your contribution and interested in improving the OpenAI developer experience.

[kurnakovv]

@christothes @jsquire

That said, there is an updated spec in the works that already includes this value, so this change will show up as soon as we incorporate the new spec and regenerate.

Thank you very much for your answer, and could you tell me where I can look at this? I just don't quite understand where it changes (I didn't find a branch/PR with changes)
Or is it that it changes in a different way?

If anything, then I am ready to change the code that you indicated in the link (but most likely not, because you said that the code has already changed somewhere)

[christothes]

The place I linked is correct, however we are in the process updating a private version of the spec which should be published soon. In the meantime, if you are curious about the process for updating the spec, you could add the new minimal value to the type I linked and then run the Invoke-CodeGen script.