Fix errorPusher Performance

[aminya]

• [x] All compiled assets are included (atom packages are git tags and hence the built files need to be a part of the source control)

takes out the loop invariant code

This snippet of the code is called tens or hundreds of times. Previously the number was multiplied by the number of diagnostics! If you can take this out from this nested loop, it would be better.

After

now after this PR pushError itself takes this much time, which is ~50ms faster in each call

Before

Previously errorPusher took this much:

[lierdakil]

Something similar done in v13.9.2. Good catch.