Should pydantic-graph End node store `output` instead of `data` to match the change to output everywhere else?

[mwildehahn]

Initial Checks

• I confirm that I'm using the latest version of Pydantic AI
• I confirm that I searched for my issue in https://github.com/pydantic/pydantic-ai/issues before opening this issue

Description

pydantic-ai/pydantic_graph/pydantic_graph/nodes.py

Line 149 in 7487ab4

data: RunEndT

Example Code

Python, Pydantic AI & LLM client version

n/a

[DouweM]

I think this makes sense as End.data ends up on GraphRunResult.output.

@samuelcolvin Was there a specific reason you left this out of #1248?

[DouweM]

@mwildehahn I think this makes sense, we'd accept a pull request, although we'd want to handle the breaking change gracefully with a deprecation, until we drop the old field completely in v0.2.0 or v1.0.0.