Skip to content

feat: add flow failed event #4121

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.

Sign up for GitHub

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

Jump to bottom
Open
heitorado wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat: add flow failed event #4121

heitorado wants to merge 4 commits into from

Conversation

@heitorado
Copy link
Contributor

@heitorado heitorado commented Dec 18, 2025 •
edited
Loading

related to #4120, #4119

This PR:

  • Adds a new event, FlowFailedEvent
  • Emits the FlowFailedEvent on an except block which will be executed if any exceptions bubble up to the kickoff flow method
  • Add listener to FlowFailedEvent that will wrap up and finalise the trace batch
  • Remove redundant type parameters from emit call since those are already defined in the model
  • Move batch finalisation on flow finished to the FlowFinishedEvent listener so we rely on the event bus to finalise the batch rather than the kickoff method body.

towards ENG-718
@linear
Copy link

linear bot commented Dec 18, 2025

ENG-718 Improve crew/flows debugging UI by adding execution errors into traces
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@heitorado