Replace getEstimatedEffortPerOccurrence() method with field

timtebeek · TeamModerne · timtebeek · commit c7f8d21111f2 · 2026-01-17T11:52:29.000Z
Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.java.recipes.UseEstimatedEffortPerOccurrenceField?organizationId=QUxML01vZGVybmUvTW9kZXJuZSArIE9wZW5SZXdyaXRl Co-authored-by: Moderne <team@moderne.io>
diff --git a/src/main/java/org/openrewrite/java/migrate/UpgradeJavaVersion.java b/src/main/java/org/openrewrite/java/migrate/UpgradeJavaVersion.java
@@ -63,10 +63,7 @@ public List<Recipe> getRecipeList() {
      *
      * @return Zero estimated time.
      */
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(0);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 0 );
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
diff --git a/src/main/java/org/openrewrite/java/migrate/guava/AbstractNoGuavaImmutableOf.java b/src/main/java/org/openrewrite/java/migrate/guava/AbstractNoGuavaImmutableOf.java
@@ -15,6 +15,7 @@
  */
 package org.openrewrite.java.migrate.guava;
 
+import lombok.Getter;
 import org.jspecify.annotations.Nullable;
 import org.openrewrite.*;
 import org.openrewrite.internal.ListUtils;
@@ -70,10 +71,8 @@ public String getDescription() {
         return "Replaces `" + getShortType(guavaType) + ".of(..)` if the returned type is immediately down-cast.";
     }
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(10);
-    }
+    @Getter
+    final Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 10 );
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
diff --git a/src/main/java/org/openrewrite/java/migrate/javax/AddJaxbRuntime.java b/src/main/java/org/openrewrite/java/migrate/javax/AddJaxbRuntime.java
@@ -69,10 +69,7 @@ public class AddJaxbRuntime extends ScanningRecipe<AtomicBoolean> {
                "dependency on the JAXB API. **The resulting dependencies still use the `javax` namespace, despite " +
                "the move to the Jakarta artifact**.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(30);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 30 );
 
     Set<String> tags = new HashSet<>( Arrays.asList( "javax", "jakarta", "javaee", "jaxb", "glassfish", "java11" ) );
 
diff --git a/src/main/java/org/openrewrite/java/migrate/javax/AddJaxwsRuntime.java b/src/main/java/org/openrewrite/java/migrate/javax/AddJaxwsRuntime.java
@@ -55,10 +55,7 @@ public class AddJaxwsRuntime extends Recipe {
                 "dependency on the JAX-WS API. **The resulting dependencies still use the `javax` namespace, despite " +
                 "the move to the Jakarta artifact**.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(30);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 30 );
 
     Set<String> tags = new HashSet<>( Arrays.asList( "javax", "jakarta", "javaee", "jaxws", "java11" ) );
 
diff --git a/src/main/java/org/openrewrite/java/migrate/lang/NullCheckAsSwitchCase.java b/src/main/java/org/openrewrite/java/migrate/lang/NullCheckAsSwitchCase.java
@@ -50,10 +50,7 @@ public class NullCheckAsSwitchCase extends Recipe {
                 "This recipe will only add null checks to existing switch cases if there are no other statements in between them " +
                 "or if the block in the if statement is not impacting the flow of the switch.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(3);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 3 );
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
diff --git a/src/main/java/org/openrewrite/java/migrate/lang/StringFormatted.java b/src/main/java/org/openrewrite/java/migrate/lang/StringFormatted.java
@@ -52,10 +52,7 @@ public class StringFormatted extends Recipe {
 
     String description = "Prefer `String.formatted(Object...)` over `String.format(String, Object...)` in Java 17 or higher.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(1);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 1 );
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
diff --git a/src/main/java/org/openrewrite/java/migrate/lang/UseTextBlocks.java b/src/main/java/org/openrewrite/java/migrate/lang/UseTextBlocks.java
@@ -65,10 +65,7 @@ public UseTextBlocks(boolean convertStringsWithoutNewlines) {
 
     String description = "Text blocks are easier to read than concatenated strings.";
 
-    @Override
-    public @Nullable Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(3);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 3 );
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
diff --git a/src/main/java/org/openrewrite/java/migrate/lombok/LombokValToFinalVar.java b/src/main/java/org/openrewrite/java/migrate/lombok/LombokValToFinalVar.java
@@ -46,10 +46,8 @@ public class LombokValToFinalVar extends Recipe {
     @Getter
     final Set<String> tags = singleton( "lombok" );
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(1);
-    }
+    @Getter
+    final Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 1 );
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
diff --git a/src/main/java/org/openrewrite/java/migrate/util/OptionalNotEmptyToIsPresent.java b/src/main/java/org/openrewrite/java/migrate/util/OptionalNotEmptyToIsPresent.java
@@ -38,10 +38,8 @@ public class OptionalNotEmptyToIsPresent extends Recipe {
     @Getter
     final String description = "Prefer `Optional.isPresent()` instead of using `!Optional.isEmpty()` in Java 11 or higher.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(1);
-    }
+    @Getter
+    final Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 1 );
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
diff --git a/src/main/java/org/openrewrite/java/migrate/util/OptionalNotPresentToIsEmpty.java b/src/main/java/org/openrewrite/java/migrate/util/OptionalNotPresentToIsEmpty.java
@@ -38,10 +38,8 @@ public class OptionalNotPresentToIsEmpty extends Recipe {
     @Getter
     final String description = "Prefer `Optional.isEmpty()` instead of using `!Optional.isPresent()` in Java 11 or higher.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(1);
-    }
+    @Getter
+    final Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 1 );
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
diff --git a/src/main/java/org/openrewrite/java/migrate/util/ReplaceStreamCollectWithToList.java b/src/main/java/org/openrewrite/java/migrate/util/ReplaceStreamCollectWithToList.java
@@ -56,10 +56,7 @@ public class ReplaceStreamCollectWithToList extends Recipe {
     String description = "Replace `Stream.collect(Collectors.toUnmodifiableList())` with Java 16+ `Stream.toList()`. " +
                "Also replaces `Stream.collect(Collectors.toList())` if `convertToList` is set to `true`.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(1);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 1 );
 
     Set<String> tags = new HashSet<>( singletonList( "RSPEC-S6204" ) );
 
diff --git a/src/main/java/org/openrewrite/java/migrate/util/UseEnumSetOf.java b/src/main/java/org/openrewrite/java/migrate/util/UseEnumSetOf.java
@@ -52,10 +52,7 @@ public class UseEnumSetOf extends Recipe {
 
     String description = "Prefer `EnumSet of(..)` instead of using `Set of(..)` when the arguments are enums in Java 9 or higher.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(2);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes( 2 );
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
