Some comments:

• getReferencesForStringLiteral, why does it use type

• getReferencesInContainer, consider passing in the containig sourceFile

• searchForImportsOfExport, consider changing the switch in the IIFE to just set a variable

• sourceFileHasName, consider inlining this at use site

• getSymbolScope, consider making the comment more detailed for export as namespace

• getDirectImportsMap, I would change the map to use a Map object instead

• export assignment handling, export = foo, export default foo

• consider renaming isProperImportEquals to isExternalModuleImportEqual

• forEachImport, consider storing the require calls on the sourceFile like we do with the imports
  this makes forEachPossibleImportOrExportStatement jsut a walk over ambient modules

• getImportOrExportSymbol, consider changing the return expression to return comingFromExport ? getExport() : getExport() || getImport();

• getExport, make sure to check comingFromExport before return an import

• isNodeImport, in case for ImportEqualsDeclaration, check for is it an import .. = require or just an import .. = X

• nodeIsEligibleForRename, check the parent of the node to make sure it is an import or export specifier, and worth adding a test of x.default vs a case of import * as ns from "mod"; ns.default;

@Andy-MS can you also validate that this change addresses #14346 and

@mhegazy Should all be done except for #14880, which I assigned to myself.

@Andy-MS i still do not see #14346 with this change..

Should be in 28a3604