fix(deps): bump @microlink/react-json-view to fix invalid element type crash - #1355
Merged
Merged
Conversation
…e crash
Vite's CJS interop for the old 1.23.x bundle resolves the default import
to `{ default: Component }` instead of the component itself, throwing
"Element type is invalid" (React error getsentry#130) wherever JsonViewer renders.
Fixed upstream in 1.31.9 via a proper ESM entry point and exports map.
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
BYK
approved these changes
Sep 15, 2026
Member
|
Thanks a lot! |
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
2 participants
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
JsonViewer(used by trace/event context views, span DB query descriptions, envelope details, logs) can crash the whole app with React error meta(changelog): Update package versions #130 ("Element type is invalid... but got: object").@microlink/react-json-view@1.23.x's CJS/UMD bundle ends up double-wrapped as{ default: Component }under Vite's dependency interop (bothvite buildand the dev server's esbuild pre-bundling), so the default import resolves to an object instead of the component.1.31.9by adding a properindex.mjsESM entry point and a conditionalexportsmap (getsentry/spotlight is unaffected by this class of interop bug once resolving to that entry). This PR just bumps the dependency to pick up their fix; no application code changes needed.Test plan
vite --config vite.dev.config.tsand the productionvite build --config vite.ui.config.tsbundle.JsonViewer.tsc --noEmitclean for the affected files; existingAnsiTextunit tests in the same directory still pass.