typescript-language-server 5.1.1 → 5.1.2

package/CHANGELOG.md

@@ -1,6 +1,13 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
+## [5.1.2](https://github.com/typescript-language-server/typescript-language-server/compare/v5.1.1...v5.1.2) (2025-11-18)
+
+
+### Refactors
+
+* cache file formatting configuration requests ([#1048](https://github.com/typescript-language-server/typescript-language-server/issues/1048)) ([9a366df](https://github.com/typescript-language-server/typescript-language-server/commit/9a366dfc7a989829f4b9e53017c298b565a307fd))
+
 ## [5.1.1](https://github.com/typescript-language-server/typescript-language-server/compare/v5.1.0...v5.1.1) (2025-11-05)
 
 

package/README.md

@@ -305,7 +305,7 @@ The `$/typescriptVersion` notification params include two properties:
 
 ### Workspace Configuration request for formatting settings
 
-Server asks the client for file-specific configuration options (`tabSize` and `insertSpaces`) that are required by `tsserver` to properly format file edits when for example using "Organize imports" or performing other file modifications. Those options have to be dynamically provided by the client/editor since the values can differ for each file. For this reason server sends a `workspace/configuration` request with `scopeUri` equal to file's URI and `section` equal to `formattingOptions`. The client is expected to return a configuration that includes the following properties:
+Server asks the client (provided client supports `workspace/configuration` capability) for file-specific configuration options (`tabSize` and `insertSpaces`) that are required by `tsserver` to properly format file edits when for example using "Organize imports" or performing other file modifications. Those options have to be dynamically provided by the client/editor since the values can differ for each file. For this reason server sends a `workspace/configuration` request with `scopeUri` equal to file's URI and `section` equal to `formattingOptions`. The client is expected to return a configuration that includes the following properties:
 
 ```js
 {

package/lib/cli.mjs

@@ -21442,17 +21442,32 @@ function areFileConfigurationsEqual(a, b) {
 }
 
 class FileConfigurationManager {
-  constructor(client, lspClient, onCaseInsensitiveFileSystem) {
+  constructor(client, lspClient, features, onCaseInsensitiveFileSystem) {
     this.client = client;
     this.lspClient = lspClient;
+    this.features = features;
     this.tsPreferences = deepmerge({}, DEFAULT_TSSERVER_PREFERENCES);
     this.workspaceConfiguration = deepmerge({}, DEFAULT_WORKSPACE_CONFIGURATION);
-    this.formatOptions = new ResourceMap(undefined, {
+    this.fileOptionsCache = new ResourceMap(undefined, {
+      onCaseInsensitiveFileSystem: onCaseInsensitiveFileSystem
+    });
+    this.formatOptionsCache = new ResourceMap(undefined, {
+      onCaseInsensitiveFileSystem: onCaseInsensitiveFileSystem
+    });
+    this.initialConfigurationRequestsMap = new ResourceMap(undefined, {
       onCaseInsensitiveFileSystem: onCaseInsensitiveFileSystem
     });
   }
+  onDidOpenTextDocument(document) {
+    const cancellation = new lsp$1.CancellationTokenSource;
+    this.initialConfigurationRequestsMap.set(document.uri, cancellation);
+    this.ensureConfigurationForDocument(document, cancellation.token).then(() => {
+      this.initialConfigurationRequestsMap.delete(document.uri);
+    }).catch(() => {});
+  }
   onDidCloseTextDocument(documentUri) {
-    this.formatOptions.delete(documentUri);
+    this.fileOptionsCache.delete(documentUri);
+    this.initialConfigurationRequestsMap.get(documentUri)?.cancel();
   }
   mergeTsPreferences(preferences) {
     this.tsPreferences = deepmerge(this.tsPreferences, preferences);
@@ -21493,7 +21508,11 @@ class FileConfigurationManager {
     return this.ensureConfigurationOptions(document, formattingOptions, token);
   }
   async getFormattingOptions(document) {
-    const formatConfiguration = await this.lspClient.getWorkspaceConfiguration(document.uri.toString(), 'formattingOptions') || {};
+    const formatOptionsCached = this.formatOptionsCache.get(document.uri);
+    if (formatOptionsCached) {
+      return formatOptionsCached;
+    }
+    const formatConfiguration = this.features.workspaceConfigurationSuppport ? await this.lspClient.getWorkspaceConfiguration(document.uri.toString(), 'formattingOptions') || {} : {};
     const options = {};
     if (typeof formatConfiguration.tabSize === 'number') {
       options.tabSize = formatConfiguration.tabSize;
@@ -21501,11 +21520,12 @@ class FileConfigurationManager {
     if (typeof formatConfiguration.insertSpaces === 'boolean') {
       options.insertSpaces = formatConfiguration.insertSpaces;
     }
+    this.formatOptionsCache.set(document.uri, options);
     return options;
   }
   async ensureConfigurationOptions(document, options, token) {
     const currentOptions = this.getFileOptions(document, options);
-    const cachedOptions = this.formatOptions.get(document.uri);
+    const cachedOptions = this.fileOptionsCache.get(document.uri);
     if (cachedOptions) {
       const cachedOptionsValue = await cachedOptions;
       if (token?.isCancellationRequested) {
@@ -21526,7 +21546,7 @@ class FileConfigurationManager {
         return undefined;
       }
     })();
-    this.formatOptions.set(document.uri, task);
+    this.fileOptionsCache.set(document.uri, task);
     await task;
   }
   async setGlobalConfigurationFromDocument(document, token) {
@@ -21537,7 +21557,9 @@ class FileConfigurationManager {
     await this.client.execute(CommandTypes.Configure, args, token);
   }
   reset() {
-    this.formatOptions.clear();
+    this.fileOptionsCache.clear();
+    this.formatOptionsCache.clear();
+    this.initialConfigurationRequestsMap.clear();
   }
   getFileOptions(document, options) {
     return {
@@ -23173,7 +23195,7 @@ class LspServer {
     this.referencesCodeLensProvider = null;
     this.logger = new PrefixingLogger(options.logger, '[lspserver]');
     this.tsClient = new TsClient(onCaseInsensitiveFileSystem(), this.logger, options.lspClient);
-    this.fileConfigurationManager = new FileConfigurationManager(this.tsClient, this.options.lspClient, onCaseInsensitiveFileSystem());
+    this.fileConfigurationManager = new FileConfigurationManager(this.tsClient, this.options.lspClient, this.features, onCaseInsensitiveFileSystem());
     this.commandManager = new CommandManager;
     this.diagnosticsManager = new DiagnosticsManager(diagnostics => this.options.lspClient.publishDiagnostics(diagnostics), this.tsClient, this.features, this.logger);
     this.codeActionsManager = new CodeActionManager(this.tsClient, this.fileConfigurationManager, this.commandManager, this.diagnosticsManager, this.features);
@@ -23211,7 +23233,7 @@ class LspServer {
     this.fileConfigurationManager.mergeTsPreferences(userInitializationOptions.preferences || {});
     this.features.completionDisableFilterText = userInitializationOptions.completionDisableFilterText ?? false;
     this.features.moveToFileCodeActionSupport = userInitializationOptions.supportsMoveToFileCodeAction && typescriptVersion.version?.gte(API.v520);
-    const {textDocument: textDocument} = clientCapabilities;
+    const {textDocument: textDocument, workspace: workspace} = clientCapabilities;
     if (textDocument) {
       const {codeAction: codeAction, completion: completion, definition: definition, publishDiagnostics: publishDiagnostics} = textDocument;
       if (codeAction) {
@@ -23234,6 +23256,9 @@ class LspServer {
       this.features.diagnosticsSupport = Boolean(publishDiagnostics);
       this.features.diagnosticsTagSupport = Boolean(publishDiagnostics?.tagSupport);
     }
+    if (workspace?.configuration) {
+      this.features.workspaceConfigurationSuppport = true;
+    }
     this.fileConfigurationManager.mergeTsPreferences({
       useLabelDetailsInCompletionEntries: this.features.completionLabelDetails
     });
@@ -23409,13 +23434,18 @@ class LspServer {
     this.tsClient.interruptGetErr(() => this.diagnosticsManager.updateIgnoredDiagnosticCodes(ignoredDiagnosticCodes));
   }
   didOpenTextDocument(params) {
-    if (this.tsClient.toOpenDocument(params.textDocument.uri, {
+    const {uri: uri, languageId: languageId} = params.textDocument;
+    if (this.tsClient.toOpenDocument(uri, {
       suppressAlertOnFailure: true
     })) {
-      throw new Error(`Can't open already open document: ${params.textDocument.uri}`);
+      throw new Error(`Can't open already open document: ${uri}`);
     }
     if (!this.tsClient.openTextDocument(params.textDocument)) {
-      throw new Error(`Cannot open document '${params.textDocument.uri}' (languageId: ${params.textDocument.languageId}).`);
+      throw new Error(`Cannot open document '${uri}' (languageId: ${languageId}).`);
+    }
+    const document = this.tsClient.toOpenDocument(uri);
+    if (document) {
+      this.fileConfigurationManager.onDidOpenTextDocument(document);
     }
   }
   didCloseTextDocument(params) {