feat(frontend): bring back the rivet compute#4503
feat(frontend): bring back the rivet compute#4503jog1t wants to merge 1 commit intographite-base/4503from
Conversation
|
🚅 Deployed to the rivet-pr-4503 environment in rivet-frontend
|
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
PR Review: feat(frontend): bring back the rivet computeOverall this is a solid improvement — switching from local state to React Query infinite cache for log history is the right architectural move, and the hybrid historical+live approach is well-structured. A few issues worth addressing before merge. Bug: Stream exhaustion is silently swallowedFile: After all Add a // after the for loop body
setStreamError("Failed to connect to log stream after multiple attempts.");Bug: Live SSE events silently dropped before initial history fetch completesFile: queryClient.setQueryData(queryKeyRef.current, (prev: RawInfiniteData | undefined) => {
if (!prev) return prev; // events dropped here
...
});If the SSE stream delivers events before Minor: Hardcoded limit tightly coupled to pagination conditionFile: The value const LOGS_PAGE_SIZE = 500;
// ...
limit: LOGS_PAGE_SIZE,
// ...
getPreviousPageParam: (firstPage) =>
firstPage.length >= LOGS_PAGE_SIZE ? firstPage[0].timestamp : undefined,Minor: Stream error banner not cleared on reconnectFile:
|
Merge activity
|

Enable Rivet Cloud deployment option and enhance logs functionality
This change re-enables the Rivet Cloud deployment option in the frontend and significantly improves the deployment logs viewing experience.
Type of change
Changes Made
Rivet Cloud Deployment Option
Enhanced Deployment Logs
Technical Improvements
How Has This Been Tested?
The changes have been tested by:
Checklist: