Merge pull request #3788 from 1c-syntax/develop

0.28.1

Author: nixel2007

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

@@ -80,8 +80,12 @@ public CompletableFuture<Either<List<? extends SymbolInformation>,List<? extends
 
   @Override
   public void didChangeConfiguration(DidChangeConfigurationParams params) {
+    var settings = params.getSettings();
+    if (settings == null) {
+      return;
+    }
     try {
-      PropertyUtils.copyProperties(configuration, params.getSettings());
+      PropertyUtils.copyProperties(configuration, settings);
     } catch (IllegalAccessException | InvocationTargetException | NoSuchMethodException e) {
       throw new RuntimeException(e);
     }

src/main/java/com/github/_1c_syntax/bsl/languageserver/providers/SemanticTokensProvider.java

@@ -391,6 +391,10 @@ private static List<SemanticTokensEdit> computeEdits(int[] prev, int[] curr) {
    * кроме первого токена, у которого deltaLine смещён на lineOffset.
    * При вставке текста без перевода строки (lineOffset == 0), первый токен
    * может иметь смещённый deltaStart.
+   * <p>
+   * ВАЖНО: Граничный токен (с изменённым deltaLine при lineOffset != 0) НЕ включается
+   * в suffix match, чтобы клиент получил обновлённое значение deltaLine через edit.
+   * Это критично для случая, когда добавляются только пустые строки без нового кода.
    */
   private static int findSuffixMatchWithOffset(int[] prev,
                                                int[] curr,
@@ -447,8 +451,10 @@ private static int findSuffixMatchWithOffset(int[] prev,
           }
         }
       } else if (!foundBoundary && currDeltaLine - prevDeltaLine == lineOffset) {
-        // Граничный токен — deltaLine отличается ровно на lineOffset
-        suffixMatch++;
+        // Граничный токен при вставке/удалении строк.
+        // НЕ включаем его в suffix match, чтобы он попал в edit и клиент получил
+        // обновлённое значение deltaLine. Это критично для случая, когда добавляются
+        // только пустые строки без нового кода.
         foundBoundary = true;
       } else {
         // Не совпадает

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

@@ -25,6 +25,7 @@
 import com.github._1c_syntax.bsl.languageserver.util.CleanupContextBeforeClassAndAfterEachTestMethod;
 import com.github._1c_syntax.utils.Absolute;
 import org.apache.commons.io.FileUtils;
+import org.eclipse.lsp4j.DidChangeConfigurationParams;
 import org.eclipse.lsp4j.DidChangeWatchedFilesParams;
 import org.eclipse.lsp4j.FileChangeType;
 import org.eclipse.lsp4j.FileEvent;
@@ -42,7 +43,10 @@
 import java.util.List;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatCode;
 import static org.awaitility.Awaitility.await;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
 
 /**
  * Тесты для {@link BSLWorkspaceService}.
@@ -276,6 +280,21 @@ void testDidChangeWatchedFiles_MultipleEvents() throws IOException {
     assertThat(serverContext.getDocument(uri3)).isNull();
   }
 
+  @Test
+  void testDidChangeConfiguration_WithNullSettings() {
+    // given
+    // Мокируем params с getSettings(), возвращающим null
+    // Это соответствует реальному сценарию, когда некоторые LSP клиенты
+    // отправляют workspace/didChangeConfiguration без настроек
+    var params = mock(DidChangeConfigurationParams.class);
+    when(params.getSettings()).thenReturn(null);
+
+    // when/then
+    // Не должно быть исключений при вызове с null settings
+    assertThatCode(() -> workspaceService.didChangeConfiguration(params))
+      .doesNotThrowAnyException();
+  }
+
   /**
    * Создает временный тестовый файл с базовым содержимым.
    */

src/test/java/com/github/_1c_syntax/bsl/languageserver/providers/SemanticTokensProviderTest.java

@@ -1402,6 +1402,63 @@ void deltaWithTextInsertedOnSameLine_shouldReturnOptimalDelta() {
     assertThat(editSize).isLessThan(tokens2.getData().size());
   }
 
+  @Test
+  void deltaWithOnlyEmptyLinesInserted_shouldReturnEdits() {
+    // given - simulate inserting only empty lines (no new code)
+    // This reproduces the bug reported by user: when inserting/deleting empty lines,
+    // semantic highlighting "jumps" - tokens shift by 2+ lines
+    String bsl1 = """
+      Процедура Тест()
+        А = 1;
+      КонецПроцедуры
+      """;
+
+    String bsl2 = """
+      Процедура Тест()
+        А = 1;
+      
+      
+      КонецПроцедуры
+      """;
+
+    DocumentContext context1 = TestUtils.getDocumentContext(bsl1);
+    referenceIndexFiller.fill(context1);
+    TextDocumentIdentifier textDocId1 = TestUtils.getTextDocumentIdentifier(context1.getUri());
+    SemanticTokens tokens1 = provider.getSemanticTokensFull(context1, new SemanticTokensParams(textDocId1));
+
+    // Verify original tokens structure
+    var decoded1 = decode(tokens1.getData());
+    // КонецПроцедуры should be on line 2
+    var endProcToken1 = decoded1.stream()
+      .filter(t -> t.line == 2 && t.length == 14) // КонецПроцедуры has length 14
+      .findFirst();
+    assertThat(endProcToken1).isPresent();
+
+    DocumentContext context2 = TestUtils.getDocumentContext(context1.getUri(), bsl2);
+    referenceIndexFiller.fill(context2);
+    SemanticTokens tokens2 = provider.getSemanticTokensFull(context2, new SemanticTokensParams(textDocId1));
+
+    // Verify modified tokens structure
+    var decoded2 = decode(tokens2.getData());
+    // КонецПроцедуры should now be on line 4 (shifted by 2 empty lines)
+    var endProcToken2 = decoded2.stream()
+      .filter(t -> t.line == 4 && t.length == 14) // КонецПроцедуры has length 14
+      .findFirst();
+    assertThat(endProcToken2).isPresent();
+
+    // when
+    var deltaParams = new SemanticTokensDeltaParams(textDocId1, tokens1.getResultId());
+    var result = provider.getSemanticTokensFullDelta(context2, deltaParams);
+
+    // then - should return delta with edits, NOT empty edits
+    // The bug was that empty edits were returned, causing the client to use stale token positions
+    assertThat(result.isRight()).isTrue();
+    var delta = result.getRight();
+    assertThat(delta.getEdits())
+      .as("Delta should contain edits when empty lines are inserted, because token positions changed")
+      .isNotEmpty();
+  }
+
   // endregion
 
   // region Range tokens tests