[Discussion] Revisit the LanguageServerDownloader class and redesign for DI and more general use cases.

[d3r3kk]

Our current class for downloading the Language Server is one that was made-for-purpose and as such has a bit of an anti-pattern design for what we would like for the extension overall.

Discuss improving upon the current implementation to make it easier to re-use for future cases where we may need to download extra libs/interpreters/etc.

Discussion should also include cost/benefit of redesigning the current implementation to incorporate DI, and to improve test-ability.

[MikhailArkhipov]

IMO we can redesign it when it comes to downloading something else.

[DonJayamanne]

IMO we can redesign it when it comes to downloading something else.

Agreed.

[brettcannon]

We can rewrite it when there's a need to (e.g. testing needs or another use).