microsoft / microsoft/agent-framework

.NET: Review HostApplicationBuilderAgentExtensions for consistency with other Agent registration methods

Open
#1,591 1 comment 2 reactions 1 assignee Claimed by @SergeyMenshykh View on GitHub
.NET agents hosting
Dominant language
Python
Stars
13.6k
Forks
2.3k
Avg merge
2d 45m
Merged PRs (30d)
358

Description

HostApplicationBuilderAgentExtensions has a number of methods to register ChatClientAgent instances, but the signatures of these do not match other parts of the code base, like creating an agent from an SDK.

In other parts of the code there are two overloads with two standard parameter sets:

With separate params:
string? instructions = null,
string? name = null,
string? description = null,
IList? tools = null,

With options:
ChatClientAgentOptions? options,

We should review these and consider making these consistent

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.