Merge pull request #3642 from 1c-syntax/copilot/move-populatecontext-to-initialized

Перенести запуск populateContext из обработки initialize в initialized

Author: nixel2007

src/main/java/com/github/_1c_syntax/bsl/languageserver/BSLLanguageServer.java

@@ -48,6 +48,7 @@
 import org.eclipse.lsp4j.HoverOptions;
 import org.eclipse.lsp4j.InitializeParams;
 import org.eclipse.lsp4j.InitializeResult;
+import org.eclipse.lsp4j.InitializedParams;
 import org.eclipse.lsp4j.InlayHintRegistrationOptions;
 import org.eclipse.lsp4j.ReferenceOptions;
 import org.eclipse.lsp4j.RenameCapabilities;
@@ -108,13 +109,6 @@ public CompletableFuture<InitializeResult> initialize(InitializeParams params) {
 
     setConfigurationRoot(params);
 
-    var factory = new NamedForkJoinWorkerThreadFactory("populate-context-");
-    var executorService = new ForkJoinPool(ForkJoinPool.getCommonPoolParallelism(), factory, null, true);
-    CompletableFuture
-      .runAsync(context::populateContext, executorService)
-      .thenAccept(unused -> executorService.shutdown())
-    ;
-
     var capabilities = new ServerCapabilities();
     capabilities.setTextDocumentSync(getTextDocumentSyncOptions());
     capabilities.setDocumentRangeFormattingProvider(getDocumentRangeFormattingProvider());
@@ -163,6 +157,20 @@ private void setConfigurationRoot(InitializeParams params) {
     context.setConfigurationRoot(configurationRoot);
   }
 
+  @Override
+  public void initialized(InitializedParams params) {
+    var factory = new NamedForkJoinWorkerThreadFactory("populate-context-");
+    var executorService = new ForkJoinPool(ForkJoinPool.getCommonPoolParallelism(), factory, null, true);
+    CompletableFuture
+      .runAsync(context::populateContext, executorService)
+      .whenComplete((unused, throwable) -> {
+        executorService.shutdown();
+        if (throwable != null) {
+          LOGGER.error("Error populating context", throwable);
+        }
+      });
+  }
+
   @Override
   public CompletableFuture<Object> shutdown() {
     shutdownWasCalled = true;

src/test/java/com/github/_1c_syntax/bsl/languageserver/BSLLanguageServerTest.java

@@ -29,6 +29,7 @@
 import org.eclipse.lsp4j.ClientCapabilities;
 import org.eclipse.lsp4j.InitializeParams;
 import org.eclipse.lsp4j.InitializeResult;
+import org.eclipse.lsp4j.InitializedParams;
 import org.eclipse.lsp4j.TextDocumentSyncKind;
 import org.eclipse.lsp4j.RenameCapabilities;
 import org.eclipse.lsp4j.TextDocumentClientCapabilities;
@@ -102,6 +103,23 @@ void initializeRename() throws ExecutionException, InterruptedException {
     assertThat(initialize.getCapabilities().getRenameProvider().isRight()).isTrue();
   }
 
+  @Test
+  void initialized() {
+    // given
+    InitializeParams initParams = new InitializeParams();
+    WorkspaceFolder workspaceFolder = new WorkspaceFolder(Absolute.path(PATH_TO_METADATA).toUri().toString(), "test");
+    List<WorkspaceFolder> workspaceFolders = List.of(workspaceFolder);
+    initParams.setWorkspaceFolders(workspaceFolders);
+
+    // initialize first
+    server.initialize(initParams);
+
+    // when
+    InitializedParams params = new InitializedParams();
+    // then - should not throw
+    server.initialized(params);
+  }
+
   @Test
   void shutdown() throws ExecutionException, InterruptedException {
     CompletableFuture<Object> shutdown = server.shutdown();