Refuse to merge a `close()` into try-with-resources when non-close
statements precede it in the finally block. Try-with-resources runs the
auto-close before the finally, so those statements would otherwise move
from before to after the close, changing semantics.

…ClassConstructor` (#867)

Enables meta-annotation matching so classes annotated with `@TestConfiguration`,
`@SpringBootApplication`, and any user-defined stereotype layered on top of
`@Configuration` are also skipped.

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

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

* OpenRewrite recipe best practices

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

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

* Use rewrite-java-25 test runtime, revert preview-feature unnamed lambda parameters

`_` unnamed variables are still a preview feature when compiled
without `--enable-preview`, so the recipe-best-practices run that
introduced `_ ->` broke `compileTestJava`. Revert those two lambda
parameters back to `cu ->` and bump the test runtime parser to
rewrite-java-25.

---------

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

Without the cast, calls like `<T extends SourceFile> T withSourcePath(Path)`
inside a lambda lose `T` to its bound during outer inference (e.g.
`Optional.map(...).ifPresent(generated::add)`), causing compilation errors.

Fixes #871

* Preserve `(Object)` bridge cast for unchecked generic casts

When a cast is wrapped by another cast and neither the outer cast type
nor the inner expression type are assignable to one another, the inner
cast is acting as a bridge for an unchecked generic conversion (e.g.
`(List<String>) (Object) values`). Removing it leaves a direct cast
that does not compile.

* Always preserve a cast nested inside another cast

The earlier "neither type assignable to the other" check leaves room for
edge cases. If a developer wrote two casts they almost always had a
reason — bridge cast, primitive widen-then-narrow, controlling overload
resolution — so just preserve. Also skip both `J.Parentheses` and
`J.ControlParentheses` when walking up so parenthesized variants are
covered.

…#875)

The recipe previously transformed `() -> field != null` into
`Objects::nonNull` even when the lambda had no parameter or the null
check referred to a captured field rather than the lambda parameter,
producing code that no longer compiled.

…` crashes (#879)

Two minimized repros for the `ControlFlowIllegalStateException`s observed in
flagship runs against `moderneinc/moderne-intellij-plugin` (Kotlin),
`moderneinc/rewrite-sql`, and `openrewrite/rewrite-templating`:

- Anonymous class body containing multiple methods whose bodies returned
  (Java) — `No current node!`.
- Kotlin expression-body function whose `||` operands are Boolean-typed
  (`kotlin.Boolean`) — `Condition Node is not a guard!`.

The underlying analyzer fix lives in `openrewrite/rewrite-analysis`. The
Kotlin test ends up replacing `Stack` with `Deque` because the function only
returns the boolean, not the stack — so we just assert that the recipe runs
to completion.

…erence (#882)

When the first statement is a variable declaration and the following
compound assignment's right-hand side references the variable being
declared, folding produced an initializer that read the variable before
it was assigned, e.g. `int result = state ^ result >>> 1;`.