- c195a4f#r168134435

…already on left side (#756)

* Fix EqualsAvoidsNull to remove redundant null checks when literal is already on left side

The recipe was failing to remove redundant null checks in patterns like:
  if (s != null && "test".equals(s)) {}

The issue was that visitMethodInvocation would return early before calling
maybeHandleParentBinary when the select was already a literal. This prevented
the null check removal logic from running.

Additionally, maybeHandleParentBinary only checked if the null check matched
the select of the method invocation, but not the argument. For cases where the
literal is already on the left side (select), we need to check if the null
check matches the argument.

Changes:
- Moved maybeHandleParentBinary call before early returns to ensure null check
  removal is always attempted for string comparison methods
- Extended null check matching to check both select and argument expressions
- Added comment clarifying that null check removal runs independently of
  argument swapping logic

Fixes: org.openrewrite.staticanalysis.EqualsAvoidsNullTest#removeUnnecessaryNullCheckByItself

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

Co-Authored-By: Claude <[email protected]>

* Update description

* Expand unit test

* Replace local methods with existing utilities to expand coverage

* Add parentheses and format to see repetition

---------

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

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.recipes.rewrite.OpenRewriteRecipeBestPractices?organizationId=QUxML09wZW4gU291cmNlL09wZW5SZXdyaXRl

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

… modifiers (#759)

* Fix RemoveUnusedPrivateFields to detect annotations after visibility modifiers

The recipe was incorrectly removing Lombok-annotated fields when the annotation
was placed after the visibility modifier (e.g., `private @Getter String foo;`)
instead of before it (e.g., `@Getter private String foo;`).

The fix adds a helper method `hasAnyAnnotations()` that checks for both:
- Leading annotations (before modifiers)
- Type annotations (after modifiers, stored as J.AnnotatedType)

Fixes #757

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

Co-Authored-By: Claude <[email protected]>

* Simplify approach

---------

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