Go to definition unnecessary shows export along true definition

VSCode Version: 1.33.1 and 1.34.0-insiders
OS Version: Arch Linux

Steps to Reproduce:

create foo.js
```
function foo() {}
```

module.exports = { foo };

create bar.js

const foo = require('./foo');

foo.foo();


2. In bar.js Ctrl+click on foo() to go to definition. 
It shows both function definition and export from modules.exports
![2019-04-16-201758_592x252_scrot](https://user-images.githubusercontent.com/1936195/56235129-1757eb00-6087-11e9-9384-332307687b0e.png)

<!-- Launch with `code --disable-extensions` to check. -->
Does this issue occur when all extensions are disabled?: Yes

microsoft / TypeScript

Go to definition unnecessary shows export along true definition #31041