fix: canvas quality value

[pauldambra]

when clamping canvas quality to range we aren't first checking it's a number 😓

this improves that and adds a matrix test to validate the config parsing

[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 11, 2025 1:57pm

[greptile-apps]

PR Summary

This PR fixes a bug in canvas recording quality handling by properly converting string values to numbers before applying range validation.

• Added type safety and parseFloat conversion for canvas quality values in src/extensions/replay/sessionrecording.ts
• Added comprehensive test matrix in sessionrecording.test.ts to validate quality/fps parsing edge cases
• Fixed default value handling (0.4 for quality, 4 for fps) when values are null/undefined
• Improved clampToRange function usage by providing explicit default values

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

[github-actions]

Size Change: +650 B (+0.02%)

Total Size: 3.3 MB

Filename	Size	Change
dist/array.full.es5.js	268 kB	+65 B (+0.02%)
dist/array.full.js	370 kB	+65 B (+0.02%)
dist/array.full.no-external.js	369 kB	+65 B (+0.02%)
dist/array.js	183 kB	+65 B (+0.04%)
dist/array.no-external.js	181 kB	+65 B (+0.04%)
dist/main.js	184 kB	+65 B (+0.04%)
dist/module.full.js	370 kB	+65 B (+0.02%)
dist/module.full.no-external.js	369 kB	+65 B (+0.02%)
dist/module.js	183 kB	+65 B (+0.04%)
dist/module.no-external.js	181 kB	+65 B (+0.04%)

ℹ️ 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}