Skip to content

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

Open
ardaerzin wants to merge 4 commits intomainfrom
frontend-fix/age-3698-evaluators-keep-loading-when-llm-output-fails
Open

[Frontend/Fix] Cascade failure and cancellation states to downstream evaluators#3960
ardaerzin wants to merge 4 commits intomainfrom
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

Preview URL https://gateway-production-791c.up.railway.app/w
Project agenta-oss-pr-3960
Image tag pr-3960-ab0ea69
Status Deployed
Railway logs Open logs
Workflow logs View workflow run
Updated at 2026-03-12T09:54:33.419Z

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

2 participants