Fix a crash in DeprecatedVariable on synthetic record trees

`@Deprecated` annotations on record components are propagated to the getter methods, where they do have an effect.

PiperOrigin-RevId: 891873165

Author: cushon

core/src/main/java/com/google/errorprone/bugpatterns/DeprecatedVariable.java

@@ -28,6 +28,7 @@
 import com.google.errorprone.matchers.Description;
 import com.google.errorprone.util.ASTHelpers;
 import com.sun.source.tree.VariableTree;
+import com.sun.tools.javac.code.Flags;
 import com.sun.tools.javac.code.Symbol;
 import java.util.Optional;
 
@@ -48,6 +49,10 @@ public Description matchVariable(VariableTree tree, VisitorState state) {
         return NO_MATCH;
       }
     }
+    if ((sym.flags() & Flags.MANDATED) == Flags.MANDATED) {
+      // Don't match on synthetic variable declarations for record components
+      return NO_MATCH;
+    }
     Description.Builder description = buildDescription(tree);
     Optional.ofNullable(
             getAnnotationWithSimpleName(tree.getModifiers().getAnnotations(), "Deprecated"))

core/src/test/java/com/google/errorprone/bugpatterns/DeprecatedVariableTest.java

@@ -78,4 +78,22 @@ void f() {
             """)
         .doTest();
   }
+
+  @Test
+  public void recordComponent() {
+    testHelper
+        .addInputLines(
+            "Test.java",
+            """
+            import java.util.Objects;
+
+            public record Test(@Deprecated int x) {
+              public Test {
+                Objects.requireNonNull(x);
+              }
+            }
+            """)
+        .expectUnchanged()
+        .doTest();
+  }
 }