Fix duplicate @Getter annotations on multiple variable declarations (#877)

When multiple variables are declared on one line (e.g., `int foo, bar = 9;`)
with corresponding getter methods, the recipe was adding @Getter twice to
the same field declaration. This happened because each getter method triggered
a separate FieldAnnotator visitor, and both attempted to annotate the shared
VariableDeclarations node.

The fix adds a check to prevent duplicate annotations by verifying if the
annotation already exists before applying it.

Fixes #876

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <[email protected]>

Author: timtebeek

src/main/java/org/openrewrite/java/migrate/lombok/FieldAnnotator.java

@@ -40,13 +40,19 @@ class FieldAnnotator extends JavaIsoVisitor<ExecutionContext> {
     public J.VariableDeclarations visitVariableDeclarations(J.VariableDeclarations multiVariable, ExecutionContext ctx) {
         for (J.VariableDeclarations.NamedVariable variable : multiVariable.getVariables()) {
             if (variable.getName().getFieldType() == field) {
-                maybeAddImport(annotation.getName());
-                maybeAddImport("lombok.AccessLevel");
-                String suffix = accessLevel == PUBLIC ? "" : String.format("(AccessLevel.%s)", accessLevel.name());
-                return JavaTemplate.builder("@" + annotation.getSimpleName() + suffix)
-                        .imports(annotation.getName(), "lombok.AccessLevel")
-                        .javaParser(JavaParser.fromJavaVersion().classpath("lombok"))
-                        .build().apply(getCursor(), multiVariable.getCoordinates().addAnnotation(comparing(J.Annotation::getSimpleName)));
+                // Check if the annotation already exists (can happen with multiple variable declarations)
+                String annotationName = annotation.getSimpleName();
+                if (multiVariable.getLeadingAnnotations().stream()
+                        .noneMatch(ann -> annotationName.equals(ann.getSimpleName()))) {
+                    maybeAddImport(annotation.getName());
+                    maybeAddImport("lombok.AccessLevel");
+                    String suffix = accessLevel == PUBLIC ? "" : String.format("(AccessLevel.%s)", accessLevel.name());
+                    return JavaTemplate.builder("@" + annotation.getSimpleName() + suffix)
+                            .imports(annotation.getName(), "lombok.AccessLevel")
+                            .javaParser(JavaParser.fromJavaVersion().classpath("lombok"))
+                            .build().apply(getCursor(), multiVariable.getCoordinates().addAnnotation(comparing(J.Annotation::getSimpleName)));
+                }
+                return multiVariable;
             }
         }
         return multiVariable;

src/test/java/org/openrewrite/java/migrate/lombok/UseLombokGetterTest.java

@@ -638,4 +638,36 @@ private Long getNumber() {
           )
         );
     }
+
+    @Issue("https://github.com/openrewrite/rewrite-migrate-java/issues/876")
+    @Test
+    void multipleVariableDeclarations() {
+        rewriteRun(// language=java
+          java(
+            """
+              class A {
+
+                  int foo, bar = 9;
+
+                  public int getFoo() {
+                      return foo;
+                  }
+
+                  public int getBar() {
+                      return bar;
+                  }
+              }
+              """,
+            """
+              import lombok.Getter;
+
+              class A {
+
+                  @Getter
+                  int foo, bar = 9;
+              }
+              """
+          )
+        );
+    }
 }