Skip to content

Improve dashboard performance on large diffs - #162

Merged
devarshishimpi merged 4 commits into
mainfrom
bug/improve-perf-on-heavy-diff
Nov 7, 2026
Merged

devarshishimpi merged 4 commits into
mainfrom
bug/improve-perf-on-heavy-diff

Conversation

@devarshishimpi

Copy link
Copy Markdown
Owner

Summary

  • Reduce render and navigation costs in large job diff and log reviews.
  • Reuse fresh diff cache entries and skip oversized cache payloads.
  • Make polling and route/sign-in error handling more resilient.
  • Add regression coverage for performance-sensitive behavior.

Closes #161

Validation

The commit includes regression tests for the new performance behavior and router error handling. They have not been rerun as part of opening this PR.

@codra-app-personal codra-app-personal Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codra Review

✅ Nothing to flag. Reviewed 12 files (846 changed lines) and found no issues worth raising.

Reviewed commit: 7e80c85f16

ℹ️ About Codra in GitHub

Your team has set up Codra to review pull requests in this repo. Reviews are triggered when you:

  • Open a pull request for review
  • Mark a draft as ready

Every review posts a summary here. A clean pass also gets a 👍 on the pull request itself.

@codra-app-personal codra-app-personal Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codra Review

✅ Nothing to flag. Reviewed 12 files (904 changed lines) and found no issues worth raising.

Reviewed commit: 71448b7e6e

ℹ️ About Codra in GitHub

Your team has set up Codra to review pull requests in this repo. Reviews are triggered when you:

  • Open a pull request for review
  • Mark a draft as ready

Every review posts a summary here. A clean pass also gets a 👍 on the pull request itself.

@codra-app-personal codra-app-personal Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codra Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9bd7df4921

ℹ️ About Codra in GitHub

Your team has set up Codra to review pull requests in this repo. Reviews are triggered when you:

  • Open a pull request for review
  • Mark a draft as ready

If Codra has suggestions, it will comment; otherwise it will react with 👍.

Comment thread apps/dashboard/src/lib/json-size.ts Resolved
- Track only active ancestors so repeated references are handled without rejecting valid JSON
- Remove objects from the ancestor set after traversal

@codra-app-personal codra-app-personal Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codra Review

✅ Nothing to flag. Reviewed 12 files (912 changed lines) and found no issues worth raising.

Note

1 candidate did not survive the evidence and claim gates — see the dashboard for what was dropped and why.

Reviewed commit: bb3589d266

ℹ️ About Codra in GitHub

Your team has set up Codra to review pull requests in this repo. Reviews are triggered when you:

  • Open a pull request for review
  • Mark a draft as ready

Every review posts a summary here. A clean pass also gets a 👍 on the pull request itself.

@devarshishimpi
devarshishimpi merged commit 1dd74a7 into main Nov 7, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve dashboard performance for large job diffs

1 participant