fix: should use event listener

[pauldambra]

or else the pnpm complains

[vercel]

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

Name	Status	Preview	Updated (UTC)
posthog-js	✅ Ready (Inspect)	Visit Preview	Jan 29, 2025 4:10pm

[greptile-apps]

PR Summary

Modified event listener implementation in surveys component to use a custom addEventListener utility instead of direct window event listeners, addressing pnpm build/lint issues.

• Changed window.addEventListener to custom addEventListener utility in src/extensions/surveys.tsx for URL change events
• Added cleanup of event listeners in component unmount
• Maintains same event handling functionality while resolving pnpm compatibility concerns
• Ensures consistent event listener behavior across different environments

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

[github-actions]

Size Change: -224 B (-0.01%)

Total Size: 3.28 MB

Filename	Size	Change
dist/all-external-dependencies.js	215 kB	-28 B (-0.01%)
dist/array.full.es5.js	266 kB	-28 B (-0.01%)
dist/array.full.js	369 kB	-28 B (-0.01%)
dist/array.full.no-external.js	368 kB	-28 B (-0.01%)
dist/module.full.js	369 kB	-28 B (-0.01%)
dist/module.full.no-external.js	368 kB	-28 B (-0.01%)
dist/surveys-preview.js	68.8 kB	-28 B (-0.04%)
dist/surveys.js	71.8 kB	-28 B (-0.04%)

ℹ️ View Unchanged

Filename	Size
dist/array.js	181 kB
dist/array.no-external.js	180 kB
dist/customizations.full.js	13.8 kB
dist/dead-clicks-autocapture.js	14.5 kB
dist/exception-autocapture.js	9.48 kB
dist/external-scripts-loader.js	2.64 kB
dist/main.js	182 kB
dist/module.js	181 kB
dist/module.no-external.js	180 kB
dist/recorder-v2.js	115 kB
dist/recorder.js	115 kB
dist/tracing-headers.js	1.76 kB
dist/web-vitals.js	10.4 kB

_{compressed-size-action}

[rafaeelaudibert]

Ahn, is this new code? It must've been commited after I started my branch or else my code would've gotten it

[lucasheriques]

learned something new today 🤔

@rafaeelaudibert yup it's my code, merged on a PR yesterday