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

Co-authored-by: Moderne <team@moderne.io>

…exhaustive (#769)

* Only add null case when all values covered

* Use Switch expression/statement syntax accordingly

* Update src/main/java/org/openrewrite/java/migrate/lang/NullCheckAsSwitchCase.java

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Add Precondition and test to run them

* Add Precondition and test to run them

* Comment

* Hold off on inclusion until we've validated at scale

* Remove unnecessary else

* Return early if we discover a default case

* Name the two branches of case rules vs case statements

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Pierre Delagrave <pierre@moderne.io>
Co-authored-by: Tim te Beek <tim@moderne.io>

Fixes #785

… using a JavaTemplate (#793)

* Let all change-to-var recipes use a created var-Identifier instead of using a JavaTemplate

* Polish

* Polish

* Polish

* Polish

…er (#796)

The recipe failed with a NullPointerException when encountering code like
`iterator().next()` where the iterator() method is called without an
explicit receiver (using implicit `this`). This fix adds a null check
for `iteratorSelect` before attempting to check its type.

Added test case to verify the recipe correctly handles implicit receiver
cases without throwing NPE.

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

Co-authored-by: Claude <noreply@anthropic.com>

* try to rewrite method references

* Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Alter implementation and polish tests

* Stop ignoring type validation issues

* Also update method type

* Light polish

* Minimize test

* Fix annotation order

* Pull up suppressions

---------

Co-authored-by: Tim te Beek <tim@moderne.io>
Co-authored-by: Tim te Beek <timtebeek@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

`ChangeTagAttribute` is now idempotent openrewrite/rewrite#5769

Kick the can down the road on changing how tests do assertions for now

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.staticanalysis.UnwrapElseAfterReturn?organizationId=ODQ2MGExMTUtNDg0My00N2EwLTgzMGMtNGE1NGExMTBmZDkw

Co-authored-by: Moderne <team@moderne.io>

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.staticanalysis.InlineVariable?organizationId=ODQ2MGExMTUtNDg0My00N2EwLTgzMGMtNGE1NGExMTBmZDkw

Co-authored-by: Moderne <team@moderne.io>