.Net: Ensure tool results follow corresponding tool calls in chat history

[bjornhandersson]

Motivation and Context

Fixes #13243.

Description

Ensure each "tool_use" is directly followed by it's corresponding "tool_result" in the ChatHistory, this is a soft convention but Anthropic Sonnet is strict to it and fails.

Contribution Checklist

• The code builds clean without any errors or warnings
• The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
• All unit tests pass, and I have added new tests where possible
• I didn't break anyone 😄

[bjornhandersson]

@microsoft-github-policy-service agree

[bjornhandersson]

@microsoft-github-policy-service agree company="Microsoft"

[bjornhandersson]

This is not a good solution where I mutate the history after yielding the result.
I have no personal interest in solving this anymore since I've picked another model for the project which works with unordered tool_use and tool_result.
But if someone can point me to the right direction I'm still willing to contribute.

[westey-m]

Thanks for filing a bug and providing this PR @bjornhandersson.
I recently changed this code to yield FunctionCallContent and FunctionResultContent as soon as possible instead of deferring them until the end. This was causing issues upstream where callers were expecting these to arrive as they happened (before the final result) instead of only after the final result. This is obviously conflicting with the problem you have.
I'm thinking there may be a (slightly more complicated) middle ground where we yield both FCC & FRC as soon as the FRC is available but not until it is available, but I suppose we'll have to test to ensure that that solves the issue you found.

[bjornhandersson]

Closing this since this is not a good solution