chore: upgrade rrweb to alpha.17
Changes
Trying something out...
Checklist
- [ ] Tests for new code (see advice on the tests we use)
- [ ] Accounted for the impact of any changes across different browsers
- [ ] Accounted for backwards compatibility of any changes (no breaking changes in posthog-js!)
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Updated (UTC) |
|---|---|---|---|
| posthog-js | ✅ Ready (Inspect) | Visit Preview | Nov 4, 2024 11:35pm |
Size Change: +76.7 kB (+2.55%)
Total Size: 3.09 MB
| Filename | Size | Change | |
|---|---|---|---|
dist/all-external-dependencies.js |
204 kB | +11 kB (+5.69%) | 🔍 |
dist/array.full.js |
353 kB | +11 kB (+3.2%) | |
dist/array.full.no-external.js |
352 kB | +11 kB (+3.21%) | |
dist/module.full.js |
353 kB | +11 kB (+3.21%) | |
dist/module.full.no-external.js |
352 kB | +11 kB (+3.21%) | |
dist/recorder-v2.js |
113 kB | +10.9 kB (+10.68%) | ⚠️ |
dist/recorder.js |
113 kB | +10.9 kB (+10.68%) | ⚠️ |
ℹ️ View Unchanged
| Filename | Size |
|---|---|
dist/array.full.es5.js |
251 kB |
dist/array.js |
168 kB |
dist/array.no-external.js |
167 kB |
dist/dead-clicks-autocapture.js |
12.8 kB |
dist/exception-autocapture.js |
8.77 kB |
dist/external-scripts-loader.js |
2.19 kB |
dist/main.js |
169 kB |
dist/module.js |
168 kB |
dist/module.no-external.js |
167 kB |
dist/surveys-preview.js |
56.7 kB |
dist/surveys.js |
62.1 kB |
dist/tracing-headers.js |
1.33 kB |
dist/web-vitals.js |
10.3 kB |
we were queueing https://github.com/PostHog/posthog-js/issues/1464 on these upgrades but 🤔
i guess that this isn't working because postcss is already bundled in with rrweb, not being included by our bundler 🤔
tested locally and can still record and playback.
the bundle size increase is pretty extreme but we can loop back and improve that once we have things fixed
holy cow that increase is almost all in the console recorder 🤯
🤣 🤣 🤣 🤣 🤣 🤣 🤣 🤣 🤣 @daibhin search in the console recorder js for postcss 🤣 🤣 🤣 🤣 🤣 🤣 🤣 🤣 🤣
i believe bundle size fix is in https://github.com/rrweb-io/rrweb/pull/1594
🤣 🤣 🤣 🤣 🤣 🤣 🤣 🤣 🤣 @daibhin search in the console recorder js for
postcss🤣 🤣 🤣 🤣 🤣 🤣 🤣 🤣 🤣
Crikey! We might need to contribute our bundle size increase checker back to rrweb 😅 I should really never have been allowed to increase the size of that package so much in the first place
@pauldambra genius!! I removed the postcss code from the console plugin and the package sizes are finally back in a place of reasonable increase. Want to do a little bit of cleanup given all the changes we've made in this PR but ultimately think we're good to ship this upgrade