David Newell
David Newell
@hpouillot adding you as a reviewer here because you've done a ton of work on the Node SDK recently too and might have some thoughts about things worth including in...
Looks good to me but going to tag the Product Analytics team for a review because they will be much more familiar with the HogQL code
More feedback in https://posthoghelp.zendesk.com/agent/tickets/18398 that the resizer when the inspector is vertically stacked is not obvious to people  Chrome makes this obvious by dedicating a whole row to it
Closing in favour of https://github.com/PostHog/posthog/pull/24711
@pauldambra have a fix ready for this https://github.com/PostHog/posthog-js/pull/1129
I believe the size gains were over reported... things in fact did not work @pauldambra this might need another 👀 because I had do do some weird build fangling to...
> `dist/recorder-v2.js` 122 kB +11.5 kB (+10.45%) ⚠️ > `dist/recorder.js` 122 kB +11.5 kB (+10.45%) ⚠️ Not good :/ Seems to be pulling in more of rrweb. @pauldambra any ideas...
@pauldambra I recalled something about `Array.from` and thought rrweb had a custom implementation. It's actually just a redeclaration that very much relies on it existing already
Tested and confirmed that `alpha.16` was running: Recordings seemed to work and specifically checked canvas elements 
@pauldambra after many rounds of the Yalc dance I think I've managed to fix this (tldr patches are **haard**, one small mistake and the whole things blows up). Might be...