De-prioritize "./index" when picking auto-import path from within the same folder

[mhegazy]

Consider a file structure as such:

|   tsconfig.json
|
\---src
        a.ts
        foo.ts
        index.ts

// src/index.ts

export * from "./foo";

// src/foo.ts

export const foo = 0;

in a.ts, completion for imported foo adds:

import { foo } from ".";

foo;

It seems that . or ./index is the wrong choice here. most likely users want to get to the local implementation in the same folder.

on the other hand, an external library, you want the index instead.. e.g.:

import { Subject} from ""rxjs"";

and not

import { Subject} from ""rxjs/Subject"";

Related: #23410

[j-oliveras]

@mhegazy In the last two code blocks, the imports has double quotes.

[mhegazy]

@Andy-MS can we also verify that rxjs case works as expected. i.e. we are getting the top most export and not digging inside the module.