Ensure updates to openFileVersion data is not delayed

microsoft · Oct 18, 2024 · c062fa5 · c062fa5
1 parent 9df3a17
commit c062fa5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/Extension/src/LanguageServer/extension.ts b/Extension/src/LanguageServer/extension.ts
@@ -170,6 +170,7 @@ export async function activate(): Promise<void> {
  });
 
  disposables.push(vscode.workspace.onDidChangeConfiguration(onDidChangeSettings));
+ disposables.push(vscode.workspace.onDidChangeTextDocument(onDidChangeTextDocument));
  disposables.push(vscode.window.onDidChangeTextEditorVisibleRanges((e) => clients.ActiveClient.enqueue(async () => onDidChangeTextEditorVisibleRanges(e))));
  disposables.push(vscode.window.onDidChangeActiveTextEditor((e) => clients.ActiveClient.enqueue(async () => onDidChangeActiveTextEditor(e))));
  ui.didChangeActiveEditor(); // Handle already active documents (for non-cpp files that we don't register didOpen).
@@ -288,6 +289,11 @@ async function onDidChangeSettings(event: vscode.ConfigurationChangeEvent): Prom
  }
 }
 
+async function onDidChangeTextDocument(event: vscode.TextDocumentChangeEvent): Promise<void> {
+ const me: Client = clients.getClientFor(event.document.uri);
+ me.onDidChangeTextDocument(event);
+}
+
 let noActiveEditorTimeout: NodeJS.Timeout | undefined;
 
 async function onDidChangeTextEditorVisibleRanges(event: vscode.TextEditorVisibleRangesChangeEvent): Promise<void> {

diff --git a/Extension/src/LanguageServer/protocolFilter.ts b/Extension/src/LanguageServer/protocolFilter.ts
@@ -59,11 +59,7 @@ export function createProtocolFilter(): Middleware {
  }
  }
  }),
- didChange: async (textDocumentChangeEvent, sendMessage) => clients.ActiveClient.enqueue(async () => {
- const me: Client = clients.getClientFor(textDocumentChangeEvent.document.uri);
- me.onDidChangeTextDocument(textDocumentChangeEvent);
- await sendMessage(textDocumentChangeEvent);
- }),
+ didChange: invoke1,
  willSave: invoke1,
  willSaveWaitUntil: async (event, sendMessage) => {
  // await clients.ActiveClient.ready;