Find All References fails for CommonJS named exports in checkJs Mode

[golopot]

🔎 Search Terms

"export", "references", "commonjs", "checkJs", "allowJs",

🕗 Version & Regression Information

• This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
  This bugs reproduces in typescript@3,4,5

⏯ Playground Link

No response

💻 Code

Request find all references for the symbol f in f.js

// f.js:
function f() {}
module.exports = {
  f,
};

// main.js
const { f } = require("./f");
f();

Reproduction repo: https://github.com/golopot/testcommonjs

🙁 Actual behavior

Doesn't find the reference from the dependent ./main.js.

🙂 Expected behavior

Do find the reference from the dependent ./main.js.

Additional information about the issue

{
  "compilerOptions": {
    "target": "es2016",                                 
    "module": "commonjs",                                
    "allowJs": true,                                 
    "checkJs": true,                                  
    "noEmit": true,                                  
    "esModuleInterop": true,                            
    "forceConsistentCasingInFileNames": true,            
    "strict": false,                                    
    "skipLibCheck": true                          
  }
}

[heysujal]

So we try to find all the occurences where the require syntax is being and it is not able to reference from where it is exported. I want to know what do we intend to do after that? should we convert all of them to import syntax?