this change makes style changes visible to the mutation observer

This makes it easier to track which objects are disposable, since we can use findAllReferences on them

This makes it easier to track references to these methods

Use `Array<T | U>` when dealing with union types instead of  `(T | U)[]`

Coalesce has the correct return type of `T[]`. Filter may return `(T | undefined)[]`

These typing errors were hidden by us previously using `any`

fixing simple cases of any[]

This code pattern indicates that the type should be nullable

In these cases, the programmer has explicitly indicated that the type should be nullable

Using a custom version of the node typings is causing a lot of pain while trying to upgrade our build to use `@types` instead of the `d.ts` files we have to maintain. I believe we primarily maintain our own version for two reason:

- Custom require function
- setTimeout and other timer functions return NodeJs.timer instead of a number

These can both be delt with, the first by using interface augmentation and the second by switching to use any types for timer. I believe that the pain of maintaining our own node `d.ts` outweighs the benefits here

This change switches us to use the standard @types/node package for our node typings

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

This part removes most `use strict` directives that are right after the file header

* use alt to focus the menu and and navigation

* updating action runner to return focus properly and set state

* return focus in any case of lost focus
except intentionally focusing elsewhere

* convert SetTimeout to RunOnceScheduler to fix builder issue