Update module 4 demo 4

[hansmbakker]

Fixes #33

⚠️ Warning the old approach of having a shared ChatHistory object does not exist in Agent Framework
AgentSession is agent-specific and cannot be shared cross-agents.

Options are to

• use the workflow-approach
• use https://learn.microsoft.com/en-us/agent-framework/agents/conversations/storage?pivots=programming-language-csharp#simple-chathistoryprovider-implementation
• write a custom AIContextProvider?
• simple workaround using extracted chatmessages passed to another session (see db6dc75)

@roycornelissen this needs some discussion

[hansmbakker]

Note: in Agent Framework, the workflow functionality is meant to deal with multiple agents working together:

When to use agents vs workflows

[hansmbakker]

I created an issue at microsoft/agent-framework#4341 to ask for documentation on Human-in-the-Loop, RequestInfoEvent / RequestPort and their compatibility with AgentWorkflowBuilder orchestrations

[roycornelissen]

@hansmbakker should we keep this one open until you get the requested documentation?

[hansmbakker]

@hansmbakker should we keep this one open until you get the requested documentation?

Yes, let's do that. However if we decide we need to use workflows, then it would make sense to create a separate module out of lab 4 and lab 5 that is dedicated to workflows.

Optionally, we could keep the simplified version I created in db6dc75 but that is really a workaround. I think that's not the way how it should be done in Agent Framework.

HOLAgentFramework/demos/module-04/demo04/ChatWithAgent.cs

Lines 17 to 32 in db6dc75

	Console.WriteLine("******** Create the hotel agent ***********");
	var hotelAgent = HotelBookingAgent.CreateChatCompletionAgent(chatClient);
	Console.WriteLine("******** Create the ride agent ***********");
	var rideAgent = CreateTransportationAgent();
	var hotelSession = await hotelAgent.CreateSessionAsync();
	await RunUntilGoalReached(hotelAgent, [userMessage], hotelSession);
	var provider = hotelAgent.GetService<InMemoryChatHistoryProvider>();
	List<ChatMessage>? messages = provider?.GetMessages(hotelSession);
	Console.WriteLine("******** hotel agent done ***********");
	var rideSession = await rideAgent.CreateSessionAsync();
	await RunUntilGoalReached(rideAgent, messages, rideSession);

[hansmbakker]

https://devblogs.microsoft.com/agent-framework/a-tour-of-handoff-orchestration-pattern/ is a nice article that goes into this. It explains that the Sequential Workflow is not a great option because it has no way to ask for clarification.