feat: allow disabling handling updating files on file renames

packages/language-server/src/plugins/typescript/index.ts

@@ -16,6 +16,18 @@ export const create = (ts: typeof import('typescript')): LanguageServicePlugin[]
 					const typeScriptPlugin = plugin.create(context);
 					return {
 						...typeScriptPlugin,
+						async provideFileRenameEdits(oldUri, newUri, token) {
+							const astroConfig = await context.env.getConfiguration?.<{
+								updateImportsOnFileMove: { enabled: boolean };
+							}>('astro');
+
+							// Check for `false` explicitly, as the default value is `true`, but it might not be set explicitly depending on the editor
+							if (astroConfig?.updateImportsOnFileMove.enabled === false) {
+								return null;
+							}
+
+							return typeScriptPlugin.provideFileRenameEdits!(oldUri, newUri, token);
+						},
 						async provideCompletionItems(document, position, completionContext, token) {
 							const originalCompletions = await typeScriptPlugin.provideCompletionItems!(
 								document,

packages/language-server/test/fixture/renameThis.ts

@@ -0,0 +1,3 @@
+export function sayHello() {
+	console.log('Hello');
+}

packages/language-server/test/fixture/renaming.astro

@@ -0,0 +1,3 @@
+---
+import { sayHello } from "./renameThis.js";
+---

packages/language-server/test/server.ts

@@ -46,6 +46,7 @@ export async function getLanguageServer(): Promise<LanguageServer> {
 				workspace: {
 					// Needed for tests that use didChangeWatchedFiles
 					didChangeWatchedFiles: {},
+					configuration: true,
 				},
 			},
 		);

packages/language-server/test/typescript/renames.test.ts

@@ -0,0 +1,67 @@
+import path from 'node:path';
+import { expect } from 'chai';
+import type { RenameFilesParams } from 'vscode-languageserver-protocol';
+import { WillRenameFilesRequest } from 'vscode-languageserver-protocol';
+import { type LanguageServer, getLanguageServer } from '../server.js';
+
+const fixtureDir = path.join(__dirname, '../fixture');
+
+describe('TypeScript - Renaming zzz', async () => {
+	let languageServer: LanguageServer;
+
+	before(async () => (languageServer = await getLanguageServer()));
+
+	it('Does not rename imports for files when setting is disabled', async () => {
+		await languageServer.handle.updateConfiguration({
+			astro: {
+				updateImportsOnFileMove: {
+					enabled: false,
+				},
+			},
+		});
+
+		const documentToBeRenamed = await languageServer.handle.openTextDocument(
+			path.resolve(fixtureDir, 'renameThis.ts'),
+			'typescript',
+		);
+		const newUri = documentToBeRenamed.uri.replace('renameThis.ts', 'renamed.ts');
+
+		const edits = await languageServer.handle.connection.sendRequest(WillRenameFilesRequest.type, {
+			files: [
+				{
+					oldUri: documentToBeRenamed.uri,
+					newUri: newUri,
+				},
+			],
+		} satisfies RenameFilesParams);
+
+		expect(edits).to.be.null;
+	});
+
+	it('Renames imports for files when setting is enabled', async () => {
+		await languageServer.handle.updateConfiguration({
+			astro: {
+				updateImportsOnFileMove: {
+					enabled: true,
+				},
+			},
+		});
+
+		const documentToBeRenamed = await languageServer.handle.openTextDocument(
+			path.resolve(fixtureDir, 'renameThis.ts'),
+			'typescript',
+		);
+		const newUri = documentToBeRenamed.uri.replace('renameThis.ts', 'renamed.ts');
+
+		const edits = await languageServer.handle.connection.sendRequest(WillRenameFilesRequest.type, {
+			files: [
+				{
+					oldUri: documentToBeRenamed.uri,
+					newUri: newUri,
+				},
+			],
+		});
+
+		expect(edits).to.not.be.null;
+	});
+});

packages/vscode/package.json

@@ -154,6 +154,12 @@
           "type": "boolean",
           "default": false,
           "description": "Enable experimental support for content collection intellisense inside Markdown, MDX and Markdoc. Note that this require also enabling the feature in your Astro config (experimental.contentCollectionIntellisense) (Astro 4.14+)"
+        },
+        "astro.updateImportsOnFileMove.enabled": {
+          "scope": "resource",
+          "type": "boolean",
+          "default": false,
+          "description": "Controls whether the extension updates imports when a file is moved to a new location. In most cases, you'll want to keep this disabled as TypeScript and the Astro TypeScript plugin already handles this for you. Having multiple tools updating imports at the same time can lead to corrupted files."
         }
       }
     },