Conversation
✅ Circular References ReportGenerated at: 2026-04-08T08:52:30.056Z Summary
Click to view all circular references in PR (62)Click to view all circular references in base branch (62)Analysis✅ No Change: This PR does not introduce or remove any circular references. This report was generated automatically by comparing against the |
c230ce4 to
d805e81
Compare
There was a problem hiding this comment.
Pull request overview
This PR updates the avatar image caching hook to avoid Sentry reports caused by Promises being thrown outside of the React Suspense render flow (notably from cache invalidation / image error paths).
Changes:
- Refactors
ImageCache.read()to centralize subscriber notifications and simplify cache entry updates. - Updates image load failure handling to avoid throwing Promises from
img.onerror. - Simplifies
useImageCache()to rely solely onuseSyncExternalStore(removing the extrauseEffect/useStatepath).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
resolve this Sentry error (
Error: object Promise)https://konghq.sentry.io/issues/7310989160/events/7f7157eacd76418b9975fe7f45e42f0e/?project=6311804&query=is%3Aunresolved&referrer=previous-event