[moveToFile] Fix symbols with empty declarations being treated as i…

…mportable (#61084)
microsoft · Jan 30, 2025 · 739d729 · 739d729
1 parent 66e1b24
commit 739d729
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/src/services/refactors/moveToFile.ts b/src/services/refactors/moveToFile.ts
@@ -886,7 +886,7 @@ export function getUsageInfo(oldFile: SourceFile, toMove: readonly Statement[],
     const unusedImportsFromOldFile = new Set<Symbol>();
     for (const statement of toMove) {
         forEachReference(statement, checker, enclosingRange, (symbol, isValidTypeOnlyUseSite) => {
-            if (!symbol.declarations) {
+            if (!some(symbol.declarations)) {
                 return;
             }
             if (existingTargetLocals.has(skipAlias(symbol, checker))) {

diff --git a/tests/cases/fourslash/moveToFile_undefined.ts b/tests/cases/fourslash/moveToFile_undefined.ts
@@ -0,0 +1,15 @@
+/// <reference path="fourslash.ts" />
+
+// @module: esnext
+// @moduleResolution: bundler
+
+// @Filename: /orig.ts
+//// [|export const variable = undefined;|]
+
+verify.moveToFile({
+    newFileContents: {
+      "/orig.ts": "",
+      "/new.ts": "export const variable = undefined;\n"
+    },
+    interactiveRefactorArguments: { targetFile: "/new.ts" },
+});