Replace getDisplayName() and getDescription() methods with fields

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

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

Author: timtebeek

src/main/java/org/openrewrite/staticanalysis/AddSerialVersionUidToSerializable.java

@@ -47,17 +47,11 @@ public class AddSerialVersionUidToSerializable extends Recipe {
     @Nullable
     String uid;
 
-    @Override
-    public String getDisplayName() {
-        return "Add `serialVersionUID` to a `Serializable` class when missing";
-    }
+    String displayName = "Add `serialVersionUID` to a `Serializable` class when missing";
 
-    @Override
-    public String getDescription() {
-        return "A `serialVersionUID` field is strongly recommended in all `Serializable` classes. If this is not " +
+    String description = "A `serialVersionUID` field is strongly recommended in all `Serializable` classes. If this is not " +
                 "defined on a `Serializable` class, the compiler will generate this value. If a change is later made " +
                 "to the class, the generated value will change and attempts to deserialize the class will fail.";
-    }
 
     @Override
     public Set<String> getTags() {

src/main/java/org/openrewrite/staticanalysis/AnnotateNullableMethods.java

@@ -46,21 +46,15 @@ public class AnnotateNullableMethods extends Recipe {
 
     private static final String DEFAULT_NULLABLE_ANN_CLASS = "org.jspecify.annotations.Nullable";
 
-    @Override
-    public String getDisplayName() {
-        return "Annotate methods which may return `null` with `@Nullable`";
-    }
+    String displayName = "Annotate methods which may return `null` with `@Nullable`";
 
-    @Override
-    public String getDescription() {
-        return "Add `@Nullable` to non-private methods that may return `null`. " +
+    String description = "Add `@Nullable` to non-private methods that may return `null`. " +
                 "By default `org.jspecify.annotations.Nullable` is used, but through the `nullableAnnotationClass` option a custom annotation can be provided. " +
                 "When providing a custom `nullableAnnotationClass` that annotation should be meta annotated with `@Target(TYPE_USE)`. " +
                 "This recipe scans for methods that do not already have a `@Nullable` annotation and checks their return " +
                 "statements for potential null values. It also identifies known methods from standard libraries that may " +
                 "return null, such as methods from `Map`, `Queue`, `Deque`, `NavigableSet`, and `Spliterator`. " +
                 "The return of streams, or lambdas are not taken into account.";
-    }
 
     @Override
     public Validated<Object> validate() {

src/main/java/org/openrewrite/staticanalysis/AnnotateNullableParameters.java

@@ -58,19 +58,13 @@ public class AnnotateNullableParameters extends Recipe {
     @Nullable
     List<String> additionalNullCheckingMethods;
 
-    @Override
-    public String getDisplayName() {
-        return "Annotate null-checked method parameters with `@Nullable`";
-    }
+    String displayName = "Annotate null-checked method parameters with `@Nullable`";
 
-    @Override
-    public String getDescription() {
-        return "Add `@Nullable` to parameters of public methods that are explicitly checked for `null`. " +
+    String description = "Add `@Nullable` to parameters of public methods that are explicitly checked for `null`. " +
                 "By default `org.jspecify.annotations.Nullable` is used, but through the `nullableAnnotationClass` option a custom annotation can be provided. " +
                 "When providing a custom `nullableAnnotationClass` that annotation should be meta annotated with `@Target(TYPE_USE)`. " +
                 "This recipe scans for methods that do not already have parameters annotated with `@Nullable` annotation and checks their usages " +
                 "for potential null checks. Additional null-checking methods can be specified via the `additionalNullCheckingMethods` option.";
-    }
 
     @Override
     public Validated<Object> validate() {

src/main/java/org/openrewrite/staticanalysis/AnnotateRequiredParameters.java

@@ -44,19 +44,13 @@ public class AnnotateRequiredParameters extends Recipe {
     @Nullable
     String nonNullAnnotationClass;
 
-    @Override
-    public String getDisplayName() {
-        return "Annotate required method parameters with `@NonNull`";
-    }
+    String displayName = "Annotate required method parameters with `@NonNull`";
 
-    @Override
-    public String getDescription() {
-        return "Add `@NonNull` to parameters of public methods that are explicitly checked for `null` and throw an exception if null. " +
+    String description = "Add `@NonNull` to parameters of public methods that are explicitly checked for `null` and throw an exception if null. " +
                 "By default `org.jspecify.annotations.NonNull` is used, but through the `nonNullAnnotationClass` option a custom annotation can be provided. " +
                 "When providing a custom `nonNullAnnotationClass` that annotation should be meta annotated with `@Target(TYPE_USE)`. " +
                 "This recipe scans for methods that do not already have parameters annotated with `@NonNull` annotation and checks for " +
                 "null validation patterns that throw exceptions, such as `if (param == null) throw new IllegalArgumentException()`.";
-    }
 
     @Override
     public Validated<Object> validate() {

src/main/java/org/openrewrite/staticanalysis/DeclarationSiteTypeVariance.java

@@ -55,16 +55,10 @@ public class DeclarationSiteTypeVariance extends Recipe {
     @Nullable
     Boolean excludeFinalClasses;
 
-    @Override
-    public String getDisplayName() {
-        return "Properly use declaration-site type variance";
-    }
+    String displayName = "Properly use declaration-site type variance";
 
-    @Override
-    public String getDescription() {
-        return "Currently, Java requires use-site type variance, so if someone has `Function<IN, OUT>` method parameter, it should rather be `Function<? super IN, ? extends OUT>`. " +
+    String description = "Currently, Java requires use-site type variance, so if someone has `Function<IN, OUT>` method parameter, it should rather be `Function<? super IN, ? extends OUT>`. " +
                "Unfortunately, it is not easy to notice that `? super` and `? extends` is missing, so this recipe adds it where that would improve the situation.";
-    }
 
     @Override
     public Validated<Object> validate() {

src/main/java/org/openrewrite/staticanalysis/EqualsAvoidsNull.java

@@ -47,17 +47,11 @@ public class EqualsAvoidsNull extends Recipe {
     private static final MethodMatcher EQUALS_IGNORE_CASE = new MethodMatcher(JAVA_LANG_STRING + " equalsIgnoreCase(" + JAVA_LANG_STRING + ")");
     private static final MethodMatcher CONTENT_EQUALS = new MethodMatcher(JAVA_LANG_STRING + " contentEquals(java.lang.CharSequence)");
 
-    @Override
-    public String getDisplayName() {
-        return "Equals avoids null";
-    }
+    String displayName = "Equals avoids null";
 
-    @Override
-    public String getDescription() {
-        return "Checks that any combination of String literals is on the left side of an `equals()` comparison. " +
+    String description = "Checks that any combination of String literals is on the left side of an `equals()` comparison. " +
                 "Also checks for String literals assigned to some field (such as `someString.equals(anotherString = \"text\"))`. " +
                 "And removes redundant null checks in conjunction with equals comparisons.";
-    }
 
     @Override
     public Set<String> getTags() {

src/main/java/org/openrewrite/staticanalysis/ExplicitCharsetOnStringGetBytes.java

@@ -37,16 +37,10 @@ public class ExplicitCharsetOnStringGetBytes extends Recipe {
     @Nullable
     String encoding;
 
-    @Override
-    public String getDisplayName() {
-        return "Set charset encoding explicitly when calling `String#getBytes`";
-    }
+    String displayName = "Set charset encoding explicitly when calling `String#getBytes`";
 
-    @Override
-    public String getDescription() {
-        return "This makes the behavior of the code platform neutral. It will not override any " +
+    String description = "This makes the behavior of the code platform neutral. It will not override any " +
                "existing explicit encodings, even if they don't match the default encoding option.";
-    }
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

src/main/java/org/openrewrite/staticanalysis/InstanceOfPatternMatch.java

@@ -47,16 +47,10 @@
 @Value
 public class InstanceOfPatternMatch extends Recipe {
 
-    @Override
-    public String getDisplayName() {
-        return "Changes code to use Java 17's `instanceof` pattern matching";
-    }
+    String displayName = "Changes code to use Java 17's `instanceof` pattern matching";
 
-    @Override
-    public String getDescription() {
-        return "Adds pattern variables to `instanceof` expressions wherever the same (side effect free) expression is referenced in a corresponding type cast expression within the flow scope of the `instanceof`. " +
+    String description = "Adds pattern variables to `instanceof` expressions wherever the same (side effect free) expression is referenced in a corresponding type cast expression within the flow scope of the `instanceof`. " +
                "Currently, this recipe supports `if` statements and ternary operator expressions.";
-    }
 
     @Override
     public Duration getEstimatedEffortPerOccurrence() {

src/main/java/org/openrewrite/staticanalysis/MaskCreditCardNumbers.java

@@ -30,16 +30,10 @@
 @Value
 public class MaskCreditCardNumbers extends Recipe {
 
-    @Override
-    public String getDisplayName() {
-        return "Mask credit card numbers";
-    }
+    String displayName = "Mask credit card numbers";
 
-    @Override
-    public String getDescription() {
-        return "When encountering string literals which appear to be credit card numbers, " +
+    String description = "When encountering string literals which appear to be credit card numbers, " +
                 "mask the last eight digits with the letter 'X'.";
-    }
 
     private static final Pattern CC_PATTERN = Pattern.compile("([0-9]{4} ?[0-9]{4} ?)([0-9]{4} ?[0-9]{4} ?)");
     private static final Pattern DIGIT_PATTERN = Pattern.compile("\\d");

src/main/java/org/openrewrite/staticanalysis/MethodNameCasing.java

@@ -51,17 +51,11 @@ public class MethodNameCasing extends ScanningRecipe<List<MethodNameCasing.Metho
     @Nullable
     Boolean renamePublicMethods;
 
-    @Override
-    public String getDisplayName() {
-        return "Standardize method name casing";
-    }
+    String displayName = "Standardize method name casing";
 
-    @Override
-    public String getDescription() {
-        return "Fixes method names that do not follow standard naming conventions. " +
+    String description = "Fixes method names that do not follow standard naming conventions. " +
                "For example, `String getFoo_bar()` would be adjusted to `String getFooBar()` " +
                "and `int DoSomething()` would be adjusted to `int doSomething()`.";
-    }
 
     @Override
     public Set<String> getTags() {