Merge pull request #3767 from 1c-syntax/copilot/fix-nullpointerexception-node-null

Fix NullPointerException in VariableSymbolComputer when parsing malformed lValue

Author: nixel2007

src/main/java/com/github/_1c_syntax/bsl/languageserver/context/computer/VariableSymbolComputer.java

@@ -165,6 +165,7 @@ public ParseTree visitParam(BSLParser.ParamContext ctx) {
   public ParseTree visitLValue(BSLParser.LValueContext ctx) {
     if (
       ctx.getChildCount() > 1
+      || ctx.IDENTIFIER() == null
       || currentMethodVariables.containsKey(ctx.getText())
       || moduleVariables.containsKey(ctx.getText())
     ) {

src/test/java/com/github/_1c_syntax/bsl/languageserver/context/computer/VariableSymbolTest.java

@@ -36,6 +36,7 @@
 import java.util.stream.Collectors;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatCode;
 
 @SpringBootTest
 class VariableSymbolTest {
@@ -148,4 +149,24 @@ void testVariableKind() {
 
   }
 
+  @Test
+  void testNoNPEOnMalformedLValue() {
+    // Test for issue BSL-LANGUAGE-SERVER-G5
+    // Malformed code with syntax errors should not cause NPE
+    String code = """
+      Процедура Тест()
+        Переменная =;
+        =;
+        Док.Записать;
+      КонецПроцедуры
+      """;
+    
+    // Symbol tree construction happens during getDocumentContext
+    // and should not throw NullPointerException
+    assertThatCode(() -> {
+      var ctx = TestUtils.getDocumentContext(code);
+      ctx.getSymbolTree().getVariables();
+    }).doesNotThrowAnyException();
+  }
+
 }