Skip to content

Conversation

@hartungstenio
Copy link
Contributor

@hartungstenio hartungstenio commented Jul 25, 2025

This should fix #2201

@DouweM
Copy link
Collaborator

DouweM commented Jul 25, 2025

@hartungstenio Thanks Christian, looks good! Should we do this in Agent.__aenter__ as well?

@hartungstenio
Copy link
Contributor Author

hartungstenio commented Jul 25, 2025

@hartungstenio Thanks Christian, looks good! Should we do this in Agent.__aenter__ as well?

@DouweM As far as I can see, the only resource in Agent.__aenter__ is the toolset, so this issue shouldn't happen there.

I could change there just to follow the same pattern. Or leave it ready for when someone add another resource there.

@hartungstenio
Copy link
Contributor Author

@hartungstenio Thanks Christian, looks good! Should we do this in Agent.__aenter__ as well?

@DouweM Done!

@DouweM DouweM merged commit 5cf372a into pydantic:main Jul 25, 2025
18 checks passed
@DouweM
Copy link
Collaborator

DouweM commented Jul 25, 2025

@hartungstenio Thanks Christian!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Multiples MCPs: AnyIO cancel scope stack corruption

2 participants