Michel Weststrate

With additional changes LGTM, thanks for adding this!

Interesting! I think to get a proper read on the performance impact, we should actually update the perf suite to use ESM builds without downcompiled classes; generally speaking _not_ declaring...

Thanks for the detailed follow up @taj-p! tagging @peterm-canva who is likely interested in the results here! Happy to create a beta build to make validation easier.

Ok happy to give this is shot, will likely unroll if noteworthy regressions are reported. Let's release #3901 first however in a separate version, to better be able to pinpoint...

Sure, no problem! On Thu, 18 Jul 2024, 01:01 Taj Pereira, ***@***.***> wrote: > @mweststrate - do you mind holding off > on merge for about a week? There are...

@taj-p what where your further findings?

Thanks for the detailed analysis! will put up a fix

If you want to know a bit more about the reason why it works the way it does, would you mind rephrasing the question in such a way that it...

Let's stick to the concrete problems then. Telling me what my convictions are is at best very presumptions. Fwiw I'd agree in general it'd be great if these would be...

I just realize (mostly as a stop-gap) that toJS itself probably should be updated, e.g. instead of the apiOwnKeys we should probalby just lookup the observable adminstration and map over...