[.NET] Add happy path test for in-memory agent && Simplify HelloAgent example && some clean-up in extension APIs - Githubissues

microsoft / autogen

A programming framework for agentic AI 🤖

https://microsoft.github.io/autogen/

Creative Commons Attribution 4.0 International

34.67k stars 5.01k forks source link

[.NET] Add happy path test for in-memory agent && Simplify HelloAgent example && some clean-up in extension APIs #4227

Closed LittleLittleCloud closed 3 days ago

LittleLittleCloud commented 6 days ago

Why are these changes needed?

This PR does the following work

Add happy-path test for in-memory use case
Improve the existing extension APIs (essentially replacing WebApplicationBuilder -> IHostApplicationBuilder)

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

LittleLittleCloud commented 5 days ago

Did you try this with grpc as well to make sure it still works?

What this PR does shouldn't affect the behavior of w/ grpc so I think it should still work. I can verify the case If you point me to a grpc example to try.

the changes to HelloAgents I am less thrilled about mainly because I liked how the previous version did everything in one shot. maybe we could have both?

It indeed have both. The AgentApp.StartAsync API should still work.