The major change with the unit tests was no longer requiring
a secondary null field for dispatchparts. As chords can be any length,
it's more forward-thinking to modify the unit tests to accept new behavior.

It was difficult to refactor the ResolvedKeybindings without
also refactoring label creators, as the null checks where handled in the
label generation.

More manual conversion of index based for loops to for-of loops

As a start to support key chords of longer than two parts,
modifying the internal structure of ChordKeybinding to not specifically
call out two parts.

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

Overwrite implies that the setting will be changed rather than superseded.

fixing simple cases of any[]

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

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

fixes #57789