Workflow Evals: Add streaming to project page#6185
Open
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
ui/app/routes/workflow-evaluations/projects/$project_name/route.tsx
Outdated
Show resolved
Hide resolved
simeonlee
changed the title
Extract data fetching to route.server.ts and add Suspense streaming for the results section. The run selector data (runInfos) is awaited synchronously since it must render outside the Suspense boundary. The results data (stats, episodes, count) streams independently via ResultsSection with its own skeleton and error states.
simeonlee
force-pushed
the
simeonlee/wf-eval-project-streaming
branch
from
d6baf91 to
7134312
Compare
simeonlee
changed the base branch from
simeonlee/wf-eval-project-refactor
to
main
4 tasks
simeonlee
commented
Feb 6, 2026
| const client = getTensorZeroClient(); | ||
| const runInfos = await client | ||
| .getWorkflowEvaluationRuns(runIds, projectName) | ||
| .then((response) => response.runs); |
Member
Author
There was a problem hiding this comment.
We dont stream this selector to avoid suspense boundary resuspension rendering issues
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
route.server.tsand add Suspense streaming to the workflow evaluation project pagerunInfossynchronously — the run selector needs this data and must remain outside the Suspense boundaryResultsSectionTest plan
pnpm test-e2eNote
Medium Risk
Moderate risk because it changes route loader data-flow and introduces deferred rendering/error handling; regressions would primarily affect results visibility and pagination state rather than data integrity.
Overview
Adds Suspense-based streaming to the workflow evaluation project page by deferring the heavy results fetch (run statistics, episodes, and total count) behind a new
ResultsSectionboundary.Refactors the loader to await
runInfossynchronously for the run selector while concurrently kicking offfetchResultsData, and introduces dedicated skeleton + error UI plus pagination handling inside the deferred results section.Written by Cursor Bugbot for commit d95e3f9. This will update automatically on new commits. Configure here.