Making Move To File Action appear less often

src/services/refactors/extractSymbol.ts

@@ -2221,7 +2221,7 @@ function isExtractableExpression(node: Node): boolean {
     return true;
 }
 
-function isBlockLike(node: Node): node is BlockLike {
+export function isBlockLike(node: Node): node is BlockLike {
     switch (node.kind) {
         case SyntaxKind.Block:
         case SyntaxKind.SourceFile:

src/services/refactors/moveToFile.ts

@@ -60,6 +60,7 @@ import {
     getRelativePathFromFile,
     getSourceFileOfNode,
     getSynthesizedDeepClone,
+    getTokenAtPosition,
     getUniqueName,
     hasJSFileExtension,
     hasSyntacticModifier,
@@ -151,6 +152,9 @@ import {
 import {
     registerRefactor,
 } from "../refactorProvider";
+import {
+    isBlockLike,
+} from "./extractSymbol";
 
 const refactorNameForMoveToFile = "Move to file";
 const description = getLocaleSpecificMessage(Diagnostics.Move_to_file);
@@ -167,6 +171,18 @@ registerRefactor(refactorNameForMoveToFile, {
         if (!interactiveRefactorArguments) {
             return emptyArray;
         }
+        const endPosition = context.endPosition;
+        if (endPosition) {
+            const file = context.file;
+            const startNode = getTokenAtPosition(file, context.startPosition);
+            const endNode = getTokenAtPosition(file, endPosition);
+            if (
+                startNode.kind !== SyntaxKind.SourceFile && isBlockLike(startNode)
+                && endNode.kind !== SyntaxKind.SourceFile && isBlockLike(endNode)
+            ) {
+                return emptyArray;
+            }
+        }
         if (context.preferences.allowTextChangesInNewFiles && statements) {
             return [{ name: refactorNameForMoveToFile, description, actions: [moveToFileAction] }];
         }

tests/baselines/reference/api/typescript.d.ts

@@ -10358,6 +10358,11 @@ declare namespace ts {
             readDirectory(rootDir: string, extensions: readonly string[], excludes: readonly string[] | undefined, includes: readonly string[] | undefined, depth?: number): string[];
         }
     }
+    namespace refactor {
+        namespace extractSymbol {
+            function isBlockLike(node: Node): node is BlockLike;
+        }
+    }
     function getDefaultFormatCodeSettings(newLineCharacter?: string): FormatCodeSettings;
     /**
      * Represents an immutable snapshot of a script at a specified time.Once acquired, the

tests/baselines/reference/tsserver/fourslashServer/moveToFile_emptyTargetFile.js

@@ -225,17 +225,6 @@ Info seq  [hh:mm:ss:mss] response:
             }
           ]
         },
-        {
-          "name": "Move to file",
-          "description": "Move to file",
-          "actions": [
-            {
-              "name": "Move to file",
-              "description": "Move to file",
-              "kind": "refactor.move.file"
-            }
-          ]
-        },
         {
           "name": "Extract Symbol",
           "description": "Extract function",
@@ -257,6 +246,17 @@ Info seq  [hh:mm:ss:mss] response:
               "kind": "refactor.extract.constant"
             }
           ]
+        },
+        {
+          "name": "Move to file",
+          "description": "Move to file",
+          "actions": [
+            {
+              "name": "Move to file",
+              "description": "Move to file",
+              "kind": "refactor.move.file"
+            }
+          ]
         }
       ]
     }