`prepare` function for ToolOutput

[yf-yang]

Description

Can we support prepare function for ToolOutput just like Tool?

References

No response

[DouweM]

@yf-yang Yep, feel free to submit a PR. Note that we already have Agent(prepare_output_tools=...).

[yf-yang]

Yes, I know we have that globally, while I am reading the doc I learned there is tool level prepare, which should be more handy in most cases

[yf-yang]

Now we need to add AgentDepsT to ToolOutput class. Do you think it is breaking?

[DouweM]

@yf-yang Hmm, since AgentDepsT has a default, it wouldn't be breaking as long as we put it after OutputDataT, but it's annoying that it would be in the opposite order to all of the other classes that take both generic params :/ Users will not typically have to specific it explicitly though, as it'll be inferred from the arguments. So I'm OK with adding it.

[yf-yang]

Therefore I'll follow other classes order, which is AgentDepsT, OutputDataT

[DouweM]

@yf-yang We can't because people could have written ToolOutput[...](...) in their code, even if that should be uncommon/unnecessary. But we can't be sure that they haven't, so we can't break that. So I'd add a second param with a comment to change this in v2.

[yf-yang]

So, I'll write OutputDataT, AgentDepsT?

[DouweM]

@yf-yang Yep