microsoft / microsoft/agent-framework
Python: Investigate not implementing `AgentProtocol` for `DurableAIAgent`
- Dominant language
- Python
- Stars
- 13.6k
- Forks
- 2.3k
- Avg merge
- 2d 45m
- Merged PRs (30d)
- 358
Description
We currently implement AgentProtocol for `DurableAIAgent` that the customer accesses. But this `TaskT` violates `AgentProtocol` contract. Any reason to use `TaskT` here instead of `AgentResponse`?
_Originally posted by @dmytrostruk in https://github.com/microsoft/agent-framework/pull/3420#discussion_r2728849807_
Opening this issue to track if we need to modify the contract to reduce confusion for the customer.
Contributor guide
Research direction
Start by reviewing the AgentProtocol contract and DurableAIAgent implementation, then read the linked discussion in PR #3420. Determine whether TaskT or AgentResponse matches the customer-facing contract, and consider whether the contract itself needs clarification. Done means the contract and implementation are aligned without confusing customers.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- ai, backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100