Skip to content

Conversation

@Cyb3rWard0g
Copy link
Collaborator

Overview

This PR addresses two issues: (1) incorrect warning messages for functions without parameters in the tool system, and (2) improper workflow execution patterns in DurableAgent that prevented proper agent runner integration. The changes ensure DurableAgent workflows can be executed correctly using AgentRunner while eliminating false-positive warnings for parameterless tool functions.

Key Changes

Tool System Fix

  • Fixed infer_func_schema in tool.py: Added condition to only warn about missing type hints when a function actually has parameters, preventing false warnings for parameterless functions.

DurableAgent Workflow Improvements

  • Added @workflow_entry decorator: Applied to agent_workflow method in durable.py to properly register workflows with AgentRunner
  • Fixed workflow execution method: Changed ctx.when_all() to wf.when_all() for correct parallel task execution in DurableAgent workflows

Quickstart Updates

  • Updated 03_durable_agent.py: Refactored to use AgentRunner pattern with proper configuration objects (AgentPubSubConfig, AgentStateConfig, etc.) and graceful shutdown handling
  • Updated workflow documentation: Corrected README to reference wf.when_all() instead of ctx.when_all() for fan-out/fan-in patterns

…schema

Signed-off-by: Roberto Rodriguez <9653181+Cyb3rWard0g@users.noreply.github.com>
…ethod

Signed-off-by: Roberto Rodriguez <9653181+Cyb3rWard0g@users.noreply.github.com>
Signed-off-by: Roberto Rodriguez <9653181+Cyb3rWard0g@users.noreply.github.com>
Signed-off-by: Roberto Rodriguez <9653181+Cyb3rWard0g@users.noreply.github.com>
@Cyb3rWard0g Cyb3rWard0g requested a review from yaron2 as a code owner November 7, 2025 02:32
Signed-off-by: Roberto Rodriguez <9653181+Cyb3rWard0g@users.noreply.github.com>
@yaron2 yaron2 merged commit 398b9e5 into main Nov 7, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants