In #32684, I added the ability for auto-imports to write a require variable declaration in JS files. However, it didn’t work for named or default exports, and under default project settings, would never be triggered from completions; it was only reachable by code fix. This is particularly problematic for JS, because unless the user has explicitly enabled type checking JS through VS Code settings, a tsconfig or jsconfig file, or // @ts-check, you can’t get code fixes because you don’t get errors.

This PR removes those limitations, and also recognizes members of object literals assigned to module.exports as importable symbols.

How do we choose between import and require for a JS file?

Rules are evaluated in order (highest priority first).

1. If the imported identifier could be added to an existing import, add it there.
2. If the imported identifier could be added to an existing require, add it there.
3. If the importing file contains any ES2015+ import/export syntax, use import.
4. If the importing file contains any CommonJS syntax, use require.
5. If the compiler options indicate a module target of ES2015+, use import.
6. Use require.

How are require variable declarations written for different kinds of exports?

• const { foo } = require('./foo') maps to named ES exports and CommonJS exported object literal members (module.exports = { foo }).
• const { default: foo } = require('./foo') maps to default ES exports (hopefully people aren’t actually doing this).
• const foo = require('./foo') maps to everything else.

Fixes #20292
Fixes #30549

Related:

• #36611
• #36134