Skip to content

[Frontend/Fix] Cascade failure and cancellation states to downstream evaluators#3960

Merged
bekossy merged 4 commits intorelease/v0.94.2from
frontend-fix/age-3698-evaluators-keep-loading-when-llm-output-fails
Mar 12, 2026
Merged

[Frontend/Fix] Cascade failure and cancellation states to downstream evaluators#3960
bekossy merged 4 commits intorelease/v0.94.2from
frontend-fix/age-3698-evaluators-keep-loading-when-llm-output-fails

Conversation

@ardaerzin
Copy link
Contributor

@ardaerzin ardaerzin commented Mar 11, 2026

When root execution fails or is cancelled, propagate the state to all connected downstream evaluator sessions to prevent them from staying in "running" state indefinitely.


Open with Devin

When root execution fails or is cancelled, propagate the state to all connected downstream evaluator sessions to prevent them from staying in "running" state indefinitely.
@linear
Copy link

linear bot commented Mar 11, 2026

@vercel
Copy link

vercel bot commented Mar 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agenta-documentation Ready Ready Preview, Comment Mar 12, 2026 9:46am

Request Review

@ardaerzin ardaerzin marked this pull request as ready for review March 12, 2026 08:51
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. Bug Report Something isn't working Frontend labels Mar 12, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 12, 2026

Railway Preview Environment

Status Destroyed (PR closed)

Updated at 2026-03-12T14:17:02.085Z

devin-ai-integration[bot]

This comment was marked as resolved.

When root execution fails in chat mode, cascade the failure state to all connected downstream evaluator sessions to prevent them from remaining in "running" state indefinitely.
devin-ai-integration[bot]

This comment was marked as resolved.

…ll-chain mode

When targetNodeId is set (targeted execution), downstream evaluators are never started, so they should not be failed or cancelled. Guard the cascade logic with !params.targetNodeId to match the startRunAtom behavior.
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 12, 2026
@bekossy bekossy changed the base branch from main to release/v0.94.2 March 12, 2026 14:16
@bekossy bekossy merged commit d2b016a into release/v0.94.2 Mar 12, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Report Something isn't working Frontend lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants