Skip to content

Memoize PR review file grouping#438

Merged
BunsDev merged 1 commit into
mainfrom
okcode/greeting-1
Apr 13, 2026
Merged

Memoize PR review file grouping#438
BunsDev merged 1 commit into
mainfrom
okcode/greeting-1

Conversation

@BunsDev

@BunsDev BunsDev commented Apr 13, 2026

Copy link
Copy Markdown
Member

Summary

  • Memoize PR review thread grouping by path to avoid recomputing it on every render.
  • Memoize patch file extraction and single-file visibility filtering for more stable render behavior.
  • Keep the empty PR state handling intact while moving the early return after derived data setup.

Testing

  • Not run

- Cache thread grouping and visible patch files with `useMemo`
- Keep the empty dashboard state rendered after derived data setup
@vercel

vercel Bot commented Apr 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 13, 2026 5:15pm
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 13, 2026 5:15pm

@github-actions github-actions Bot added size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Apr 13, 2026
@BunsDev BunsDev merged commit be226f1 into main Apr 13, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant