fix(47821): skip nodes with export modifiers

Author: a-tarasyuk

src/services/importTracker.ts

@@ -392,14 +392,14 @@ namespace ts.FindAllReferences {
             if (cancellationToken) cancellationToken.throwIfCancellationRequested();
             forEachImport(sourceFile, (importDecl, moduleSpecifier) => {
                 const moduleSymbol = checker.getSymbolAtLocation(moduleSpecifier);
-                if (moduleSymbol) {
-                    const id = getSymbolId(moduleSymbol).toString();
+                forEach(moduleSymbol?.declarations, declaration => {
+                    const id = getSymbolId(declaration.symbol).toString();
                     let imports = map.get(id);
                     if (!imports) {
                         map.set(id, imports = []);
                     }
                     imports.push(importDecl);
-                }
+                });
             });
         }
 

src/services/refactors/convertExport.ts

@@ -66,7 +66,6 @@ namespace ts.refactor {
         }
 
         const exportingModuleSymbol = isSourceFile(exportNode.parent) ? exportNode.parent.symbol : exportNode.parent.parent.symbol;
-
         const flags = getSyntacticModifierFlags(exportNode) || ((isExportAssignment(exportNode) && !exportNode.isExportEquals) ? ModifierFlags.ExportDefault : ModifierFlags.None);
 
         const wasDefault = !!(flags & ModifierFlags.Default);
@@ -165,6 +164,7 @@ namespace ts.refactor {
         const checker = program.getTypeChecker();
         const exportSymbol = Debug.checkDefined(checker.getSymbolAtLocation(exportName), "Export name should resolve to a symbol");
         FindAllReferences.Core.eachExportReference(program.getSourceFiles(), checker, cancellationToken, exportSymbol, exportingModuleSymbol, exportName.text, wasDefault, ref => {
+            if (exportName === ref) return;
             const importingSourceFile = ref.getSourceFile();
             if (wasDefault) {
                 changeDefaultToNamedImport(importingSourceFile, ref, changes, exportName.text);

tests/cases/fourslash/refactorConvertExport_namedToDefaultInModuleAugmentation1.ts

@@ -0,0 +1,26 @@
+/// <reference path='fourslash.ts' />
+
+// @Filename: /node_modules/@types/foo/index.d.ts
+////export {};
+////declare module "foo" {
+////    /*a*/export function foo(): void;/*b*/
+////}
+
+// @Filename: /b.ts
+////import { foo } from "foo";
+
+goTo.select("a", "b");
+edit.applyRefactor({
+    refactorName: "Convert export",
+    actionName: "Convert named export to default export",
+    actionDescription: "Convert named export to default export",
+    newContent: {
+        "/node_modules/@types/foo/index.d.ts":
+`export {};
+declare module "foo" {
+    export default function foo(): void;
+}`,
+        "/b.ts":
+`import foo from "foo";`
+    }
+});

tests/cases/fourslash/refactorConvertExport_namedToDefaultInModuleAugmentation2.ts

@@ -0,0 +1,18 @@
+/// <reference path='fourslash.ts' />
+
+////export {};
+////declare module "foo" {
+////    /*a*/export function func(): void;/*b*/
+////}
+
+goTo.select("a", "b");
+edit.applyRefactor({
+    refactorName: "Convert export",
+    actionName: "Convert named export to default export",
+    actionDescription: "Convert named export to default export",
+    newContent:
+`export {};
+declare module "foo" {
+    export default function func(): void;
+}`
+});