refactor: OpenRewrite best practices

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.recipes.OpenRewriteBestPractices?organizationId=T3BlblJld3JpdGU%3D

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

rewrite-core/src/main/java/org/openrewrite/TreeVisitor.java

@@ -146,10 +146,10 @@ public final Cursor updateCursor(T currentValue) {
     }
 
     public @Nullable T visit(@Nullable Tree tree, P p, Cursor parent) {
-        assert !(parent.getValue() instanceof Tree)
-               || !((Tree) parent.getValue()).isScope(tree)
-               || !(p instanceof ExecutionContext)
-               || !CursorValidatingExecutionContextView.view((ExecutionContext) p).getValidateCursorAcyclic() :
+        assert !(parent.getValue() instanceof Tree) ||
+               !((Tree) parent.getValue()).isScope(tree) ||
+               !(p instanceof ExecutionContext) ||
+               !CursorValidatingExecutionContextView.view((ExecutionContext) p).getValidateCursorAcyclic() :
                 "The `parent` cursor must not point to the same `tree` as the tree to be visited. " +
                 "This usually indicates that you have used getCursor() where getCursor().getParent() is appropriate. " +
                 "This is a test-only validation which can be opted out of by configuring your test's type validation options with `cursorAcyclic(false)`.";

rewrite-maven/src/main/java/org/openrewrite/maven/ChangeManagedDependencyGroupIdAndArtifactId.java

@@ -22,8 +22,8 @@
 import org.openrewrite.*;
 import org.openrewrite.maven.table.MavenMetadataFailures;
 import org.openrewrite.maven.tree.MavenMetadata;
-import org.openrewrite.maven.tree.ResolvedPom;
 import org.openrewrite.maven.tree.ResolvedManagedDependency;
+import org.openrewrite.maven.tree.ResolvedPom;
 import org.openrewrite.semver.Semver;
 import org.openrewrite.semver.VersionComparator;
 import org.openrewrite.xml.ChangeTagValueVisitor;

rewrite-xml/src/main/java/org/openrewrite/xml/internal/grammar/XMLLexer.java

@@ -15,14 +15,12 @@
  */
 // Generated from java-escape by ANTLR 4.11.1
 package org.openrewrite.xml.internal.grammar;
-import org.antlr.v4.runtime.Lexer;
-import org.antlr.v4.runtime.CharStream;
-import org.antlr.v4.runtime.Token;
-import org.antlr.v4.runtime.TokenStream;
 import org.antlr.v4.runtime.*;
-import org.antlr.v4.runtime.atn.*;
+import org.antlr.v4.runtime.atn.ATN;
+import org.antlr.v4.runtime.atn.ATNDeserializer;
+import org.antlr.v4.runtime.atn.LexerATNSimulator;
+import org.antlr.v4.runtime.atn.PredictionContextCache;
 import org.antlr.v4.runtime.dfa.DFA;
-import org.antlr.v4.runtime.misc.*;
 
 @SuppressWarnings({"all", "warnings", "unchecked", "unused", "cast", "CheckReturnValue"})
 public class XMLLexer extends Lexer {