Make map constructor generic

[osdiab]

Fixes #29786

@DanielRosenwasser 's suggested change to MapConstructor. I shot myself in the foot because of a sneaky implicit any from this!

[typescript-bot]

It looks like you've sent a pull request to update our 'lib' files. These files aren't meant to be edited by hand, as they consist of last-known good states of the compiler and are generated from 'src'. Unless this is necessary, consider closing the pull request and sending a separate PR to update 'src'.

[osdiab]

hmm looks like this is actually undoing what #28052 did before. looking into that to understand the context why its this way...

[osdiab]

huh. So the goal was to have Map<string> be an error since applying defaults lets you skip the generics, which is also not desirable.

Guess that this is probably a no-go then. Any alternatives @DanielRosenwasser ? for now i guess i'll just deal with it on my project with a lint rule or something.

[andrewbranch]

Good find @osdiab; yeah, we decided it’s best to leave it as is. Thanks!