.NET: [Bug]: Converstaion Memory doesn't work in DevUI

[viul-sc]

Description

Steps to reproduce

1. Run DevUI sample:

git clone https://github.com/microsoft/agent-framework```
cd agent-framework\dotnet\samples\GettingStarted\DevUI\DevUI_Step01_BasicUsage
dotnet run --framework net10.0 

2. Send 2 messages:

• "My name is X"
• "What is my name?"

Expected result

Agent replies "your name is X"

Actual result

Agent replies "I do not know"

Code Sample

Error Messages / Stack Traces

Package Versions

git a7d924a

.NET Version

.NET 10

Additional Context

No response

[viul-sc]

Related to #3100?

[bryjen]

@viul-sc yeah it looks to be related to #3100. From my understanding, it seems that devui uses a different system, "responses", for interacting with the agent compared to the "agent framework" system shown in most of the samples. This means that agent framework primitives such as ChatHistoryProvider and AgentSession aren't directly compatible with that of the responses system, and an adaptor doesn't currently exist. More specifically, the aforementioned data is ignored because the Responses execution path never creates or resumes an AgentSession and always invokes the agent without a session.

As far as I'm aware, this isn't stated anywhere and can be easy to misunderstand. This means even after manually adding session and chat history management (like in the samples), devui is not capable of utilizing them:

VectorStore vectorStore = new InMemoryVectorStore();
builder.AddAIAgent("assistant", (_, key) =>
    chatClient.AsAIAgent(new ChatClientAgentOptions
{
    Name = key,
    ChatOptions = new ChatOptions
    {
        Instructions = "You are a helpful assistant. Answer questions concisely and accurately.",
        Tools = [
            AIFunctionFactory.Create(GetWeather, name: "get_weather"),
            AIFunctionFactory.Create(GetCurrentTime, name: "get_current_time")
        ]
    },
    ChatHistoryProviderFactory = (ctx, _) => new ValueTask<ChatHistoryProvider>(
            new VectorChatHistoryProvider(vectorStore, ctx.SerializedState, ctx.JsonSerializerOptions))
})).WithInMemorySessionStore();

[ruplisso]

@victordibia @markwallace-microsoft any progress on this issue on devui, thanks in advance.

[victordibia]

Added a fix here, feel free to take a look and review #4294

[viul-sc]

Thank you Victor! The fix sounds totally reasonable