Partial support Lombok for java 8 (mostly done)

rewrite-java-11/src/main/java/org/openrewrite/java/isolated/ReloadableJava11ParserVisitor.java

@@ -18,7 +18,6 @@
 
 import com.sun.source.tree.*;
 import com.sun.source.util.TreePathScanner;
-import com.sun.tools.javac.code.Attribute;
 import com.sun.tools.javac.code.Flags;
 import com.sun.tools.javac.code.Symbol;
 import com.sun.tools.javac.tree.DCTree;

rewrite-java-17/src/main/java/org/openrewrite/java/isolated/ReloadableJava17ParserVisitor.java

@@ -19,7 +19,6 @@
 import com.sun.source.doctree.DocCommentTree;
 import com.sun.source.tree.*;
 import com.sun.source.util.TreePathScanner;
-import com.sun.tools.javac.code.Attribute;
 import com.sun.tools.javac.code.Flags;
 import com.sun.tools.javac.code.Symbol;
 import com.sun.tools.javac.tree.DocCommentTable;

rewrite-java-21/src/main/java/org/openrewrite/java/isolated/ReloadableJava21ParserVisitor.java

@@ -19,7 +19,6 @@
 import com.sun.source.doctree.DocCommentTree;
 import com.sun.source.tree.*;
 import com.sun.source.util.TreePathScanner;
-import com.sun.tools.javac.code.Attribute;
 import com.sun.tools.javac.code.Flags;
 import com.sun.tools.javac.code.Symbol;
 import com.sun.tools.javac.tree.DocCommentTable;

rewrite-java-8/src/main/java/org/openrewrite/java/ReloadableJava8ParserVisitor.java

@@ -15,10 +15,8 @@
  */
 package org.openrewrite.java;
 
-import com.sun.org.apache.xalan.internal.xsltc.compiler.sym;
 import com.sun.source.tree.*;
 import com.sun.source.util.TreePathScanner;
-import com.sun.tools.javac.code.Attribute;
 import com.sun.tools.javac.code.Flags;
 import com.sun.tools.javac.code.Symbol;
 import com.sun.tools.javac.tree.DCTree;
@@ -55,7 +53,8 @@
 import java.util.stream.Collectors;
 
 import static java.lang.Math.max;
-import static java.util.Collections.*;
+import static java.util.Collections.emptyList;
+import static java.util.Collections.singletonList;
 import static java.util.stream.Collectors.toList;
 import static java.util.stream.StreamSupport.stream;
 import static org.openrewrite.Tree.randomId;

rewrite-java-tck/src/main/java/org/openrewrite/java/tree/LombokTest.java

@@ -20,7 +20,7 @@
 import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
 import org.openrewrite.java.JavaParser;
-import org.openrewrite.java.search.FindMissingTypes;
+import org.openrewrite.java.MinimumJava11;
 import org.openrewrite.test.RecipeSpec;
 import org.openrewrite.test.RewriteTest;
 import org.openrewrite.test.TypeValidation;
@@ -245,6 +245,12 @@ public class ConstructorExample<T> {
                 public static class NoArgsExample {
                   @NonNull private String field;
                 }
+
+                public void test() {
+                  ConstructorExample<?> x = ConstructorExample.of("desc");
+                  ConstructorExample<?> y = new ConstructorExample<>("1L");
+                  ConstructorExample.NoArgsExample z = new ConstructorExample.NoArgsExample();
+                }
               }
               """
           )
@@ -710,6 +716,7 @@ public void both(String s) {
     }
 
     @Test
+    @MinimumJava11
     void jacksonized() {
         rewriteRun(
           spec -> spec.parser(JavaParser.fromJavaVersion().classpath("jackson-annotations", "lombok")),
@@ -746,18 +753,15 @@ public class ExampleException extends Exception {
     }
 
     @Test
+    @MinimumJava11
     void onConstructor() {
         rewriteRun(
           java(
             """
               public @interface Inject {}
               public @interface Id {}
-              public @interface Column {
-                  String name();
-              }
-              public @interface Max {
-                  long value();
-              }
+              public @interface Column { String name(); }
+              public @interface Max { long value(); }
               """
           ),
           java(
@@ -766,10 +770,10 @@ void onConstructor() {
               import lombok.Getter;
               import lombok.Setter;
 
-              @AllArgsConstructor(onConstructor_=@Inject) //JDK8
+              @AllArgsConstructor(onConstructor_=@Inject)
               public class OnXExample {
-                  @Getter(onMethod_={@Id, @Column(name="unique-id")}) //JDK8
-                  @Setter(onParam_=@Max(10000)) //JDK8
+                  @Getter(onMethod_={@Id, @Column(name="unique-id")})
+                  @Setter(onParam_=@Max(10000))
                   private long unid;
 
                   public void test() {
@@ -784,6 +788,7 @@ public void test() {
     }
 
     @Test
+    @MinimumJava11
     void onConstructorNoArgs() {
         rewriteRun(
           java(
@@ -821,6 +826,107 @@ public void test() {
     @SuppressWarnings("MismatchedReadAndWriteOfArray")
     @Nested
     class LessSupported {
+        /*
+         java 8 cannot figure out all type checking:
+         - When the @AllArgsConstructorHandler, @NoArgsConstructorHandler and @NoArgsConstructorHandler annotations are
+           used with the `onConstructor_` param, Lombok does not call the JavacAnnotationHandlers.
+         - The @Jacksonized annotation does somehow turns into `ClassDeclaration->CompilationUni` error
+         */
+
+        @Test
+        // TODO: Find solution and remove this test
+        void jacksonizedForJava8() {
+            rewriteRun(
+              spec -> spec
+                .parser(JavaParser.fromJavaVersion().classpath("jackson-annotations", "lombok"))
+                .typeValidationOptions(TypeValidation.none()),
+              java(
+                """
+                  import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
+                  import lombok.Builder;
+                  import lombok.extern.jackson.Jacksonized;
+
+                  @Jacksonized
+                  @Builder
+                  @JsonIgnoreProperties(ignoreUnknown = true)
+                  public class JacksonExample {
+                      private List<String> strings;
+                  }
+                  """
+              )
+            );
+        }
+
+        @Test
+        // TODO: Find solution and remove this test
+        void onConstructorForJava8() {
+            rewriteRun(
+              spec -> spec.typeValidationOptions(TypeValidation.none()),
+              java(
+                """
+                  public @interface Inject {}
+                  public @interface Id {}
+                  public @interface Column { String name(); }
+                  public @interface Max { long value(); }
+                  """
+              ),
+              java(
+                """
+                  import lombok.AllArgsConstructor;
+                  import lombok.Getter;
+                  import lombok.Setter;
+
+                  @AllArgsConstructor(onConstructor_=@Inject)
+                  public class OnXExample {
+                      @Getter(onMethod_={@Id, @Column(name="unique-id")})
+                      @Setter(onParam_=@Max(10000))
+                      private long unid;
+
+                      public void test() {
+                          OnXExample x = new OnXExample(1L);
+                          x.setUnid(2L);
+                          System.out.println(x.getUnid());
+                      }
+                  }
+                  """
+              )
+            );
+        }
+
+        @Test
+        // TODO: Find solution and remove this test
+        void onConstructorNoArgsForJava8() {
+            rewriteRun(
+              spec -> spec.typeValidationOptions(TypeValidation.none()),
+              java(
+                """
+                  public @interface Inject {}
+                  """
+              ),
+              java(
+                """
+                  import lombok.NoArgsConstructor;
+                  import lombok.NonNull;
+                  import lombok.RequiredArgsConstructor;
+
+                  import javax.inject.Inject;
+
+                  @NoArgsConstructor(onConstructor_ = @Inject)
+                  @RequiredArgsConstructor(onConstructor_ = @Inject)
+                  public class OnXExample {
+                      @NonNull private Long unid;
+
+                      public void test() {
+                          new OnXExample();
+                          new OnXExample(1L);
+                      }
+                  }
+                  """
+              )
+            );
+        }
+
+
         @Test
         void extensionMethod() {
             rewriteRun(