Fix: catch IllegalArgumentException during highlighting to prevent scanner crash

Agent-Logs-Url: https://github.com/1c-syntax/sonar-bsl-plugin-community/sessions/dbfba67b-b20e-4a21-bc5f-4f57190271e2

Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>

Author: Copilot

src/main/java/com/github/_1c_syntax/bsl/sonar/BSLHighlighter.java

@@ -28,6 +28,7 @@
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
 import org.antlr.v4.runtime.Token;
 import org.antlr.v4.runtime.Tokenizer;
 import org.eclipse.lsp4j.Range;
@@ -45,6 +46,7 @@
 import java.util.Set;
 import java.util.stream.Collectors;
 
+@Slf4j
 @RequiredArgsConstructor
 public class BSLHighlighter {
 
@@ -158,15 +160,19 @@ public void saveHighlighting(InputFile inputFile, DocumentContext documentContex
 
     highlightingData.stream()
       .filter(HighlightingData::isActive)
-      .forEach(data ->
-        highlighting.highlight(
-          data.getRange().getStart().getLine(),
-          data.getRange().getStart().getCharacter(),
-          data.getRange().getEnd().getLine(),
-          data.getRange().getEnd().getCharacter(),
-          data.getType()
-        )
-      );
+      .forEach(data -> {
+        try {
+          highlighting.highlight(
+            data.getRange().getStart().getLine(),
+            data.getRange().getStart().getCharacter(),
+            data.getRange().getEnd().getLine(),
+            data.getRange().getEnd().getCharacter(),
+            data.getType()
+          );
+        } catch (IllegalArgumentException e) {
+          LOGGER.error("Unable to highlight file {}", inputFile, e);
+        }
+      });
 
     highlighting.save();
   }

src/test/java/com/github/_1c_syntax/bsl/sonar/BSLHighlighterTest.java

@@ -45,6 +45,7 @@
 import java.util.stream.IntStream;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatNoException;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
@@ -130,6 +131,29 @@ void testMergeHighlightingTokens() {
 
   }
 
+  @Test
+  void testSaveHighlightingWithInvalidTokenPosition() {
+    // given
+    context = SensorContextTester.create(Path.of("."));
+    highlighter = new BSLHighlighter(context);
+    documentContext = mock(DocumentContext.class);
+
+    // Create a token with position exceeding line length
+    var token = new CommonToken(BSLLexer.IF_KEYWORD, "Если");
+    token.setLine(1);
+    token.setCharPositionInLine(20);
+
+    when(documentContext.getTokens()).thenReturn(List.of(token));
+
+    // Create InputFile with short content (line has less than 20 characters)
+    inputFile = Tools.inputFileBSL(FILE_NAME, BASE_DIR, "А = 1;");
+
+    // when/then - should not throw
+    assertThatNoException().isThrownBy(() ->
+      highlighter.saveHighlighting(inputFile, documentContext)
+    );
+  }
+
   private void testHighlighting(Vocabulary vocabulary, Map<String, TypeOfText> highlightingMap) {
     // given
     initContext(vocabulary);