The goal is to remove supertest as it does not support typescript well
and there's really no good reason for the dependency. Also no websocket
testing support.

🎅🏼

Now we support pretty much every variable under the sun along with
$NO_PROXY all correctly and with minimal code on our end.

Closes #2197.

git-subtree-dir: lib/vscode
git-subtree-split: e5a624b788d92b8d34d1392e4c4d9789406efe8f

Closes #124

This works by monkey patching the http and https modules's default agent
at runtime to the one given by the proxy-agent package.

I took our website's SVG favicon and plopped it on a round
white rectangle in Affinity Designer. The I exported it as an SVG and
wrote a script that uses imagemagick to convert to the various sizes and
formats we need.

Closes #2307

See previous commit for failure introduced.

Unfortunately we can't use node-mocks-http to test a express.Router
that has async routes. See https://github.com/howardabrams/node-mocks-http/issues/225

router will just return undefined if the executing handler is async and
so the test will have no way to wait for it to complete. Thus, we have
to use supertest which starts an actual HTTP server in the background
and uses a HTTP client to send requests.

Now async routes are handled!

The Express types were throwing errors but regenerating the lockfile
resolved them.

See #1898 and #1905

This reverts commit 6539dd4d.

Breaks linting and wasn't required.

Otherwise the build keeps failing for me as tsc won't compile anymore.

Not sure why things work on CI/for asher but I don't think this will
cause any additional issues.