.NET: fix service-stored history provider resolution#6209
Conversation
1135e79 to
db3fc3f
Compare
|
Rebased on current Validated: dotnet build .\tests\Microsoft.Agents.AI.UnitTests\Microsoft.Agents.AI.UnitTests.csproj -f net10.0 -c Debug --tl:off
dotnet test --project .\tests\Microsoft.Agents.AI.UnitTests\Microsoft.Agents.AI.UnitTests.csproj -f net10.0 -c Debug --no-build -v Normal --report-xunit-trx --ignore-exit-code 8 -- --filter-class Microsoft.Agents.AI.UnitTests.ChatClient.ChatClientAgent_ChatHistoryManagementTests
git diff --check upstream/main...HEADAll passed after the rebase. |
|
Correction: my first test command used the wrong class namespace and selected 0 tests. I reran with the actual class name. Validated: dotnet test --project .\tests\Microsoft.Agents.AI.UnitTests\Microsoft.Agents.AI.UnitTests.csproj -f net10.0 -c Debug --no-build -v Normal --report-xunit-trx --ignore-exit-code 8 -- --filter-class Microsoft.Agents.AI.UnitTests.ChatClientAgent_ChatHistoryManagementTests
git diff --check upstream/main...HEADResult: 19/19 tests passed. |
db3fc3f to
5503e3f
Compare
|
Rebased this PR onto the latest upstream Validation:
|
5503e3f to
756b594
Compare
|
Rebased this PR onto the latest upstream main and resolved the ChatClientAgent.cs conflict by preserving the new AG-UI provider exception while keeping this PR's service-stored-history behavior for non-AG-UI providers. Validation after the rebase: git diff --check, dotnet build .\dotnet\tests\Microsoft.Agents.AI.UnitTests\Microsoft.Agents.AI.UnitTests.csproj -f net10.0 -c Debug --tl:off, and dotnet .\dotnet\tests\Microsoft.Agents.AI.UnitTests\bin\Debug\net10.0\Microsoft.Agents.AI.UnitTests.dll --filter-class Microsoft.Agents.AI.UnitTests.ChatClientAgent_ChatHistoryManagementTests --no-progress (19 passed). |
756b594 to
7511346
Compare
|
Rebased this branch onto current Validation run locally from
I used |
|
Rebased this branch onto current main and pushed the updated head $head. Validation run locally from dotnet/:
|
9b4f539 to
290fea3
Compare
290fea3 to
c0c7ba4
Compare
Summary
Fixes #6120.
To verify