fix IndexOutOfBoundsException that occurs in UseNoArgsConstructor when visiting record compact constructor method (#964)

* fix IndexOutOfBoundsException

* fix

* fix

---------

Co-authored-by: zakaria-shahen <zakaria-shahen@users.noreply.github.com>
Co-authored-by: Tim te Beek <tim@moderne.io>

Author: zakaria-shahen

src/main/java/org/openrewrite/java/migrate/lombok/UseNoArgsConstructor.java

@@ -25,6 +25,7 @@
 import org.openrewrite.java.JavaIsoVisitor;
 import org.openrewrite.java.JavaParser;
 import org.openrewrite.java.JavaTemplate;
+import org.openrewrite.java.marker.CompactConstructor;
 import org.openrewrite.java.tree.J;
 import org.openrewrite.java.tree.TypeUtils;
 
@@ -51,8 +52,9 @@ public TreeVisitor<?, ExecutionContext> getVisitor() {
         return new JavaIsoVisitor<ExecutionContext>() {
             @Override
             public J.@Nullable MethodDeclaration visitMethodDeclaration(J.MethodDeclaration method, ExecutionContext ctx) {
-                if (method.isConstructor() &&
-                        method.getParameters().get(0) instanceof J.Empty &&
+				if (method.isConstructor() &&
+						!method.getMarkers().findFirst(CompactConstructor.class).isPresent() &&
+						method.getParameters().get(0) instanceof J.Empty &&
                         method.getBody() != null && method.getBody().getStatements().isEmpty()) {
                     J.ClassDeclaration enclosing = getCursor().firstEnclosing(J.ClassDeclaration.class);
                     AccessLevel accessLevel = LombokUtils.getAccessLevel(method);

src/test/java/org/openrewrite/java/migrate/lombok/UseNoArgsConstructorTest.java

@@ -69,6 +69,24 @@ public A() {
         );
     }
 
+	@Test
+	void keepCompactConstructor() {
+		rewriteRun(
+		  //language=java
+		  java(
+			"""
+			  public record Foo(String id) {
+			      public Foo {
+			  		if (id == null || id.isBlank()) {
+			  			throw new IllegalArgumentException("ID cannot be null or blank");
+			  		}
+			  	}
+			  }
+			  """
+		  )
+		);
+	}
+
     @Test
     void replaceEmptyProtectedConstructor() {
         rewriteRun(