.NET: [API Proposal] IHostedAgentBuilder.WithAITools with IServiceProvider

[verdie-g]

When registering tools with WithAITool the DI container is not built yet so you can't use it. That's not great because tools usually have dependencies. This problem could be solved with a method like this

IHostedAgentBuilder WithAITools(this IHostedAgentBuilder builder, Func<IServiceProvider, IEnumerable<AITool>> factory);

used like this

builder.Services.AddAIAgent("my_agent", "")
    .WithAITools(sp =>
    {
        var userService = sp.GetRequiredService<UserService>();
        return [AIFunctionFactory.Create(userService.GetUser)];
    });

[markwallace-microsoft]

@westey-m is this something we need to consider now as part of the API stabilization? If not, you can remove the v1.0 label.

[westey-m]

@markwallace-microsoft, I do think this is something we should consider. Note that this is in the hosting layer.
CC @DeagleGross

[DeagleGross]

makes sense, i will do it

[DeagleGross]

Adding

public static IHostedAgentBuilder WithAITool(this IHostedAgentBuilder builder, Func<IServiceProvider, AITool> factory)

API in the linked PR.

Proposed API will not work for registering individual AITools, and will only register the whole IEnumerable<AITool> in the DI, which is not what we want because easy resolve like serviceProvider.GetKeyedServices<AITool>(name) will not work.

public static IHostedAgentBuilder WithAITools(this IHostedAgentBuilder builder, Func<IServiceProvider, IEnumerable<AITool>> factory);

correct me if I am wrong, or propose a new API please, I hope current change helps with the issue

[verdie-g]

Indeed, I don't see how to implement the IEnumerable API with IServiceCollection. That's a bummer because in my use-case I want to do something like that

builder.Services.AddAIAgent("my_agent", "")
    .WithAITools(sp =>
    {
        var userService = sp.GetRequiredService<UserService>();
        return [
            AIFunctionFactory.Create(userService.GetUser),
            AIFunctionFactory.Create(userService.CreateUser),
            AIFunctionFactory.Create(userService.UpdateUser),
        ];
    });

Instead, I'll probably do

services.AddSingleton<AIAgent>(sp => { /* ... */);