Prefer Top Level Export For Missing Import Quick Fix

[mjbvz]

From microsoft/vscode#30001

TypeScript Version: 2.4.1

Steps
In a new workspace:

1. npm install @angular/core. Angular core ships with its own d.ts files

2. In a JS file:

@Component({}) 
class Foo{}

3. Trigger Fix Missing import quick fix on Component

Expected behavior:
A quick fix that adds an @angular/core import is shown.

Actual behavior:
Quick fixes for importing from folders under @angular/core are shown

In this case, it appears that the implicit javascript project is loading all d.ts files from @angular/core:

Open files: 
/Users/matb/projects/san/test.js
Info 97   Project '/dev/null/inferredProject1*' (Inferred) 0
/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/lib.es6.d.ts
/Users/matb/projects/san/test.js
/Users/matb/Library/Caches/typescript/2.4/node_modules/@types/reflect-metadata/index.d.ts
/Users/matb/projects/san/node_modules/@angular/core/src/di/injection_token.d.ts
/Users/matb/projects/san/node_modules/@angular/core/src/type.d.ts
/Users/matb/projects/san/node_modules/@angular/core/src/metadata/di.d.ts
/Users/matb/projects/san/node_modules/@angular/core/src/change_detection/constants.d.ts
/Users/matb/projects/san/node_modules/@angular/core/src/di/metadata.d.ts
/Users/matb/projects/san/node_modules/@angular/core/src/di/forward_ref.d.ts
...

Shouldn't this be limited to loading the d.ts from just the package.json typings and any files it references?

[mhegazy]

looks like a duplicate of #15223