fix: heatmaps unload listener

[pauldambra]

fixes #1735

we shouldn't be setting up listeners in the constructor anyway since we can init heatmaps and not be enabled

so, moves into the setup method, and binds this

[vercel]

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

Name	Status	Preview	Updated (UTC)
posthog-js	✅ Ready (Inspect)	Visit Preview	Feb 14, 2025 10:27am

[greptile-apps]

PR Summary

This PR fixes an issue with the heatmaps unload listener where the flush method was being called with an incorrect this context, causing "undefined buffer" errors during page unload.

• Moved beforeunload event listener from constructor to _setupListeners() in src/heatmaps.ts
• Added proper bind(this) to flush method in event listener to ensure correct context
• Improved initialization flow by only setting up listeners when heatmaps are enabled
• Prevents potential undefined buffer access by moving listener setup out of constructor

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[github-actions]

Size Change: +110 B (0%)

Total Size: 3.3 MB

Filename	Size	Change
dist/array.full.es5.js	267 kB	+11 B (0%)
dist/array.full.js	370 kB	+11 B (0%)
dist/array.full.no-external.js	369 kB	+11 B (0%)
dist/array.js	183 kB	+11 B (+0.01%)
dist/array.no-external.js	181 kB	+11 B (+0.01%)
dist/main.js	183 kB	+11 B (+0.01%)
dist/module.full.js	370 kB	+11 B (0%)
dist/module.full.no-external.js	369 kB	+11 B (0%)
dist/module.js	183 kB	+11 B (+0.01%)
dist/module.no-external.js	181 kB	+11 B (+0.01%)

ℹ️ View Unchanged

Filename	Size
dist/all-external-dependencies.js	216 kB
dist/customizations.full.js	13.8 kB
dist/dead-clicks-autocapture.js	14.5 kB
dist/exception-autocapture.js	9.51 kB
dist/external-scripts-loader.js	2.64 kB
dist/recorder-v2.js	115 kB
dist/recorder.js	115 kB
dist/surveys-preview.js	69.4 kB
dist/surveys.js	72.4 kB
dist/tracing-headers.js	1.76 kB
dist/web-vitals.js	10.4 kB

_{compressed-size-action}