This reverts commit 27256832 which turns out did not break the import of vs/nls

This reverts commit 7de696a2.

Seems to have broken loading vs/nls in at least one case in insiders

Don't use this import form in places where it is not needed. This prevents us from correctly detecting duplicate imports

There are many places in the code that do `import Event, { ...} from '.../event'`.

Fixes #37501

Add a mechanism to be a customer of the extension host (share lifecycle) and begin reducing usage of IThreadService

This was to be used by tasks as an extenion which ended up
not happening.

Part of #23177

This may turn into an enum or change behavior soon pending on the result of
task-terminal integration. Removing from the API until more thought has been
put into it.

Part of #15583
Part of #18661

Fixes #15583

Fixes #12434

Fixes #12434

* a declared module allows to have a sibling module containing experimental API
* this change also cleans up some bogous type pollution that happened because vscode used to be both, a declare module and a global namespace

Part of #12770

Fixes #11919