Добавлено отключение проверки для кейса когда производится соединение переменной с текстом запроса.

Author: huxuxuya

src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/IncorrectLineBreakDiagnostic.java

@@ -34,6 +34,7 @@
 import org.antlr.v4.runtime.Token;
 import org.antlr.v4.runtime.tree.ParseTree;
 
+import java.util.HashSet;
 import java.util.Optional;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
@@ -57,12 +58,23 @@ public class IncorrectLineBreakDiagnostic extends AbstractDiagnostic {
     "\\s+(:?ИЛИ|И|OR|AND|\\+|-|\\/|%|\\*)\\s*(?:\\/\\/.*)?$"
   );
 
+  private HashSet<Integer> setFirstQueryLines = new HashSet<Integer>();
+
   @Override
   protected void check() {
 
+    findFirstQueryLines();
+
     checkContent(INCORRECT_START_LINE_PATTERN);
     checkContent(INCORRECT_END_LINE_PATTERN);
+  }
 
+  private void findFirstQueryLines() {
+
+    documentContext.getQueries().forEach(query -> {
+        setFirstQueryLines.add(query.getAst().start.getLine());
+      }
+    );
   }
 
   private void checkContent(Pattern pattern) {
@@ -78,8 +90,8 @@ private void checkContent(Pattern pattern) {
 
       Matcher matcher = pattern.matcher(checkText);
 
-      if (matcher.find()) {
-        diagnosticStorage.addDiagnostic(i + 1, matcher.start(1) , i + 1, matcher.end(1));
+      if (matcher.find() && !setFirstQueryLines.contains(i + 2)) {
+        diagnosticStorage.addDiagnostic(i + 1, matcher.start(1), i + 1, matcher.end(1));
       }
     }
   }

src/test/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/IncorrectLineBreakDiagnosticTest.java

@@ -38,14 +38,13 @@ void test() {
 
     List<Diagnostic> diagnostics = getDiagnostics();
 
-    assertThat(diagnostics).hasSize(12);
+    assertThat(diagnostics).hasSize(11);
     assertThat(diagnostics, true)
       .hasRange(7, 32, 33)
       .hasRange(8, 35, 36)
       .hasRange(16, 32, 33)
       .hasRange(17, 22, 23)
       .hasRange(21, 49, 50)
-      .hasRange(29, 28, 29)
       .hasRange(70, 80, 83)
       .hasRange(83, 89, 92)
       .hasRange(45, 25, 26)

src/test/resources/diagnostics/IncorrectLineBreakDiagnostic.bsl

@@ -18,14 +18,14 @@
     СуммаАвтоматическойСкидки;
 
 // Error
-ПоляОтбора = "Номенклатура,Характеристика,Склад" +
+ПоляОтбора = "Номенклатура,Характеристика,Склад" + // Дополнительный комментарий
    ДополнительныеПоляОтбора;
 
 // Pass
 ПоляОтбора = "Номенклатура,Характеристика,Склад"
    + ДополнительныеПоляОтбора;
 
-// Error
+// Pass
 ТекстЗапроса = ТекстЗапроса +
 "ВЫБРАТЬ
 | Номенклатура.Ссылка КАК Ссылка