Repository navigation
Python: clarify WorkflowAgent policy configuration - #8585
Eduard van Valkenburg (eavanvalkenburg) merged 1 commit into
Conversation
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The documentation accurately reflects the existing implementation and introduces no runtime changes.
Review effort: Balanced
Findings: None
What changed in this PR
Clarifies WorkflowAgent policy boundaries without changing runtime behavior.
Changes:
- Documents forwarded context-provider lifecycle and messages.
- Clarifies where middleware, tools, and instructions must be configured.
| File | Description |
|---|---|
_workflow.py |
Clarifies Workflow.as_agent() configuration behavior. |
_agent.py |
Documents WorkflowAgent policy limitations. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
MAF Automated Review — Iteration 1
Result: No findings
Scope: full PR (1 commit(s)): 4309364ff24c
Model: gpt-5.6-sol-fast
Overview
This documentation-only change accurately explains that WorkflowAgent forwards provider-contributed messages while leaving policy configuration to agents and clients inside the workflow. The implementation and tests confirm the main distinctions around context providers, wrapper middleware, and explicit runtime tools. No Critical, High, or Medium issue was established.
Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
No publishable findings remained after source verification for this scope.
Motivation & Context
Workflow.as_agent()accepts base-agent configuration and context providers, but the workflow wrapper supports a narrower policy surface than a chat agent. Clarifying that boundary helps callers configure middleware, tools, and instructions on the agents or clients that execute inside the workflow.Description & Review Guide
Workflow.as_agent()andWorkflowAgentconstructor docstrings to describe which context-provider contributions are forwarded and where middleware should be configured.Related Issue
N/A — documentation-only clarification; no public issue.
Contribution Checklist
breaking changelabel (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.