Skip to content

core[patch]: add on_tool_error to _AstreamEventsCallbackHandler #30709

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vbarda
Copy link
Contributor

@vbarda vbarda commented Apr 7, 2025

Fixes #30708

@vbarda vbarda requested a review from eyurtsev April 7, 2025 15:00
Copy link

vercel bot commented Apr 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Apr 7, 2025 3:00pm

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 7, 2025
@deershark
Copy link
Contributor

@vbarda @eyurtsev Because of this problem, some functions of our project cannot be implemented at present :( ..... May I ask if there are any problems with this PR? It has been two weeks.

@ben-gineer
Copy link

@vbarda - I'm also keen to use this to help handling tool errors.

@eyurtsev eyurtsev self-assigned this May 16, 2025
@eyurtsev
Copy link
Collaborator

@ben-gineer and @deershark we'll try to get this in soon sorry the delay.

But you should also be favoring the .stream() or .astream() APIs and orchestrating using langgraph. astream_events API should largely be unnecessary these days for development

@ben-gineer
Copy link

Thanks. The only reason I’m using it is to allow me to stream events within a Streamlit app using LangGraph MCP tools without introducing a FastAPI intermediate service.

If you know of alternative solutions, I’d be happy to switch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug caused by create_react_agent method and runnable asteam_events
4 participants