We compile using the alwaysStrict flag so these directives are not needed

fix #54070

- avoid public modifier
- use Disposable where applicable
- fix some event handler leaks
- clean up some TODO@ben

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

* dialogs - move dialog handling to main side

* dialogs - adopt windows dialogs methods in main

* dialogs - use async dialog API and queue up dialogs on the main side