Preserve alias symbols on references to type aliases via imports - #51152
Conversation
|
TypeScript Bot (@typescript-bot) perf test this |
|
Heya Wesley Wigham (@weswigham), I've started to run the perf test suite on this PR at aa26036. You can monitor the build here. Update: The results are in! |
|
Wesley Wigham (@weswigham) Here they are:
CompilerComparison Report - main..51152
System
Hosts
Scenarios
TSServerComparison Report - main..51152
System
Hosts
Scenarios
Developer Information: |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Perf looks good (sometimes unreasonably so - looking at you TFS), so this looks like a pretty good approach to the problem. TypeScript Bot (@typescript-bot) test top100 |
|
Heya Wesley Wigham (@weswigham), I've started to run the diff-based top-repos suite on this PR at bcdac1a. You can monitor the build here. Update: The results are in! |
|
Heya Wesley Wigham (@weswigham), I've started to run the extended test suite on this PR at bcdac1a. You can monitor the build here. |
|
Heya Wesley Wigham (@weswigham), I've started to run the diff-based user code test suite on this PR at bcdac1a. You can monitor the build here. Update: The results are in! |
|
Wesley Wigham (@weswigham) Here are the results of running the user test suite comparing Everything looks good! |
|
Wesley Wigham (@weswigham) Here are the results of running the top-repos suite comparing Something interesting changed - please have a look. Details
|
|
Heya Nathan Shively-Sanders (@sandersn), I've started to run the parallelized Definitely Typed test suite on this PR at bcdac1a. You can monitor the build here. |
|
Looks good; I started a DT run because something that increases the number of type identities sometimes fails there. |
|
Heya Wesley Wigham (@weswigham), I've started to run the parallelized Definitely Typed test suite on this PR at ce91909. You can monitor the build here. |
|
Breaks 2 DT packages (lambda-tester, mparticle__web-sdk), in a way I can't understand yet. I'll open a bug. |
Fixes #51106
Technically, we could always use the alias we're referring to as the alias symbol saved for an alias lookup, but that could be a lot of new type identities, and by limiting it to only cases where the type reference refers to an import alias, we capture the cases where referring to exactly the symbol exported is particularly important for reusing the imports/names the user has made available.