Skip to content

fix: report execute hook failures - #897

Open
Eilen6316 wants to merge 2 commits into
apache:mainfrom
Eilen6316:fix/execute-hooks-report-failures
Open

Eilen6316 wants to merge 2 commits into
apache:mainfrom
Eilen6316:fix/execute-hooks-report-failures

Conversation

@Eilen6316

@Eilen6316 Eilen6316 commented Aug 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • pass the original exception, including cancellation, to top-level execute post hooks
  • defer execute hook completion for generator and async-generator APIs until iteration ends
  • clear execution hook state when pre hooks or streaming setup fail
  • report the correct astream_result execute method

@github-actions github-actions Bot added area/core Application, State, Graph, Actions area/hooks Lifecycle hooks, interceptors area/integrations External integrations (LLMs, frameworks) labels Aug 20, 2026

This branch has not been deployed

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

Labels

area/core Application, State, Graph, Actions area/hooks Lifecycle hooks, interceptors area/integrations External integrations (LLMs, frameworks)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant