Fix duplicate @Getter annotations on multiple variable declarations

[timtebeek]

Summary

• Fixed issue where @Getter annotation was being added twice to field declarations containing multiple variables
• Added test case multipleVariableDeclarations to verify the fix

Problem

When a class had multiple variables declared on one line (e.g., int foo, bar = 9;) with corresponding getter methods for each variable, the UseLombokGetter recipe was incorrectly adding the @Getter annotation twice to the same field declaration.

This happened because:

1. Each getter method (getFoo() and getBar()) triggered a separate FieldAnnotator visitor
2. Both visitors encountered the same VariableDeclarations node (since both variables share the same declaration)
3. Each visitor attempted to add the annotation without checking if it already existed

Solution

Updated FieldAnnotator.visitVariableDeclarations() to check if the annotation already exists before applying it. The fix:

• Checks if the annotation simple name already exists in the variable declaration's leading annotations
• Only applies the annotation if it doesn't already exist
• Returns the multiVariable node appropriately in both cases

Test plan

• Test multipleVariableDeclarations passes

• All tests in UseLombokGetterTest pass

• All tests in UseLombokSetterTest pass

• All Lombok-related tests pass

• Fixes Support lombok getter/setter generation for multiple variables declared on a single line #876

🤖 Generated with Claude Code