5.4 omits certain `import * as ...` statements from .d.ts

[ammut]

5.4 omits certain `import * as ...` statements from .d.ts

[jakebailey]

This isn't a super minimal repro; can you try https://www.npmjs.com/package/every-ts to bisect this down between the two releases?

[ammut]

I tried it, but every-ts kept crashing on my machine. Will try again.

[ammut]

every-ts found:

1ed8ed6c2704f0d7e1af05273226119bc5b83705 is the first bad commit
commit 1ed8ed6c2704f0d7e1af05273226119bc5b83705
Author: Jake Bailey <[email protected]>
Date:   Mon Nov 6 15:45:16 2023 -0800

    Preserve original type parameter names more often when shadowed (#55820)
    
    Co-authored-by: Wesley Wigham <[email protected]>

[ammut]

I made a smaller example here.

the '.D.TS' output is clearly missing an import statement - that's the gist of the Bug I'm reporting here. The missing import statement immediately appears, when either E is renamed.

However, I'm not able to create a Playground project where the '.D.TS' output is correct despite the name conflict. I've only been able to achieve that when working locally, using TS v5.3.3 (or any commit before the abovementioned First Bad Commit, really).