`const x = require("m")` should create an alias symbol

TypeScript Version: 3.0.0-dev.20180707

Code

/// <reference path='fourslash.ts'/>

// @allowJs: true

// @Filename: /a.js
////function [|{| "isWriteAccess": true, "isDefinition": true |}x|]() {};
////module.exports = [|x|];

// @Filename: /b.js
////const [|x|] = require("./a");
////[|x|];

const [r0, r1, r2, r3] = test.ranges();
verify.referenceGroups(r2, [/*todo*/]);

Expected behavior:

x in const x is an alias of the x in function x() {}, so all references are found.

Actual behavior:

x is a variable that happens to be the same type as the other x. Only references in b.js are found.

[sandersn]

The in-file type of commonjs modules is usually wrong, so I'll have to fix bugs related to that before aliasing will really make sense. I will look at #25621 first.

The test from the original post still isn't returning results in a.js.

[sandersn]

Turns out github didn't recognise that the "fix #xxxx" syntax occurred in a conditional clause and unconditionally closed this issue. :) Thanks for re-opening.