fix: Disable the string reduction code until we can battle test it more.

[benjackwhite]

Changes

Disables the may size checker for now. This should be less of a big deal with our new ingestion pipeline, but more importantly it doesn't at all work as expected. It replaces big data values in some cases but often it actually just modifies a big snapshots, where there happen to be a bunch of data urls.

We should definitely add it back in and figure out how to do this more cleverly but for now lets remove it and watch to see what happens.

Checklist

• Tests for new code (see advice on the tests we use)
• Accounted for the impact of any changes across different browsers

[github-actions]

Size Change: -2.7 kB (0%)

Total Size: 728 kB

Filename	Size	Change
dist/array.full.js	178 kB	-676 B (0%)
dist/array.js	119 kB	-676 B (-1%)
dist/es.js	119 kB	-676 B (-1%)
dist/module.js	119 kB	-676 B (-1%)

ℹ️ View Unchanged

Filename	Size
dist/recorder-v2.js	95.1 kB
dist/recorder.js	58.4 kB
dist/surveys.js	39.7 kB

_{compressed-size-action}

[pauldambra]

🚢

could comment on ensureMaxMessageSize to mark it as purposefully unused but 🤷