perf: Suspense streaming for dashboard + async cache warming#153
Merged
perf: Suspense streaming for dashboard + async cache warming#153
Conversation
ESLint only allows console.warn and console.error — console.debug is not in the allowed list.
) Split the dashboard into a shell (tabs, nav, filters) that renders instantly and a Suspense-wrapped async Server Component that streams data in. The shell shows placeholder counts ("·") that update via React context when data arrives. Also converts add-repo cache warming to fire-and-forget so the response returns immediately after the DB write, and adds CDN cache-busting to Octokit fetches.
- Remove ZERO_COUNTS push in DashboardContent catch block so tabs keep the "·" placeholder instead of showing misleading "0" - Remove dead outer try/catch in gatherPulls (inner per-repo catch already handles errors, Promise.all can never reject) - Promote post-retry revalidation failure log from warn to error
- Escalate gatherPulls per-repo failure log from warn to error (data loss) - Rename serverStderrChunks → serverOutputChunks (captures both streams) - Update research doc status to reflect implemented items
Keep our improvements: no ZERO_COUNTS on error, no dead outer try/catch in gatherPulls, error-level PR fetch logging, serverOutputChunks rename, updated research doc status.
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
1 participant
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
DashboardContent) wrapped in<Suspense>. The shell (tabs, filters, nav) renders immediately while GitHub data streams in, replacing the previous all-or-nothing SSR.awaitto fire-and-forget so the Settings page responds instantly._nctimestamp param +cache: "no-store"to bypass CDN/Next.js caching of GitHub API responses.Closes #134, closes #139.
Test plan
pnpm turbo typecheckpassespnpm turbo buildpassespnpm --filter @issuectl/web test:e2e— data-freshness A1/A2/A3 pass