Skip to content

fix: report execute hook failures - #897

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

fix: report execute hook failures#897
Eilen6316 wants to merge 1 commit into
apache:mainfrom
Eilen6316:fix/execute-hooks-report-failures

Conversation

@Eilen6316

Copy link
Copy Markdown

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

Testing

  • uv run --with pytest --with pytest-asyncio --with aiosqlite --with 'pydantic[email]>=2.0' python -m pytest tests/core --ignore=tests/core/test_graphviz_display.py -q\n- uv run --with 'opentelemetry-api' --with 'opentelemetry-sdk' --with 'langfuse>=3.0.0' --with pytest --with pytest-asyncio --with 'pydantic[email]>=2.0' python -m pytest tests/integrations/test_opentelemetry.py tests/integrations/test_burr_opentelemetry.py tests/integrations/test_burr_langfuse.py -q\n- black --check and flake8 on modified Python files

@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
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