fix: circular refs shouldn't explode capture

[pauldambra]

This stops estimation of size of an rrweb event containing an object with a circular reference from exploding

It doesn't edit the actual emitted data in case that's necessary for playback

see https://posthog.com/questions/uncaught-error

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
posthog-js	✅ Ready (Inspect)	Visit Preview	Jun 7, 2024 9:49pm

[github-actions]

Size Change: +673 B (+0.07%)

Total Size: 998 kB

Filename	Size	Change
dist/array.full.js	237 kB	+169 B (+0.07%)
dist/array.js	134 kB	+168 B (+0.13%)
dist/es.js	134 kB	+168 B (+0.13%)
dist/module.js	135 kB	+168 B (+0.12%)

ℹ️ View Unchanged

Filename	Size
dist/exception-autocapture.js	12.2 kB
dist/recorder-v2.js	108 kB
dist/recorder.js	108 kB
dist/surveys-module-previews.js	59.3 kB
dist/surveys.js	60.9 kB
dist/tracing-headers.js	8.26 kB

_{compressed-size-action}

[rahulkgupta]

fixes #1229

[rahulkgupta]

Thanks for the change. I unfortunately dont think it fixed the issue. Here's a PR with more info: 3DStreet/3dstreet#601