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'`.

We should always require an explicit type for this

Fixes #45784

make sure yml extesion is default for yaml files

fixes #44551

updateInlineDecorations when model content changes

Also rename fns and consts to have less ambiguous`InlineValueDecorations` suffix

fixes #45610

fixes #45577

More tweaks for #45406