Skip to content
Discussion options

You must be logged in to vote
        // FIGURE OUT HOW TODO THIS
       // Currently this will not include the returned encrypted reasoning items in the next conversation round
       chatHistory.Add(new Microsoft.Extensions.AI.ChatMessage(ChatRole.Assistant, response.Text));

For non-streaming, you should be able to do:

chatHistory.AddMessages(response);

or:

chatHistory.AddRange(response.Messages);

Does that not work? I expect it would.

// Currently this does not include the returned encrypted reasoning items

For streaming, it looks like we currently have a bug in how ReasoningResponseItems are handled. I'll put up a PR to fix it.

Replies: 1 comment

Comment options

You must be logged in to vote
0 replies
Answer selected by alientourist
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
Q&A
Labels
None yet
2 participants