Fix structured_output propagation in Graph nodes #3
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
When using
structured_output_modelin Graph agents, output from dependency nodes was not propagated to subsequent nodes. The_build_node_inputmethod was callingstr(result)which only extracts message text content, ignoring thestructured_outputfield entirely.Changes:
_build_node_inputto preferstructured_output.model_dump_json()when availablestr(result)for backward compatibilityBefore: Second agent receives empty input
After: Second agent receives structured JSON
Related Issues
strands-agents#1118
Documentation PR
N/A
Type of Change
Bug fix
Testing
Added
test_graph_structured_output_propagation- verifies structured output is propagated as JSONAdded
test_graph_structured_output_fallback_to_text- verifies backward compatibility when no structured outputI ran
hatch run prepareChecklist
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.