Polish Java recipe display names and descriptions (K7 of #1330) (#1359)

* Fix typos in recipe display names and descriptions

Covers `@Slf4` for `@Slf4j`, a stray quote after `@Inject`, embedded
newlines in `AddColumnAnnotation`, `EnumSet of(..)`, `List.Of(..)`, an
unclosed backtick and made-up `Optional#or` signature, garbled
`RemoveFinalizerFromZip` text and doubled spaces.

* Use sentence case and verb-first recipe display names

Display names now start with the action the recipe takes, rather than
stating a rule such as "`@Entity` objects with constructors must also
have a default constructor", and only capitalize proper nouns and
product names such as SDKMAN!.

* Wrap code identifiers in recipe texts in backticks

Method calls, types, file names, Maven plugins and Docker images in
display names and descriptions now render as code, as elsewhere.

* Correct recipe descriptions that misstate what the recipe does

`AddJaxbRuntime` adds the runtime in `runtimeOnly`/`runtime` scope rather
than `compileOnly`/`provided`, `ApplicationPathWildcardNoLongerAccepted`
strips `/*` rather than a slash, `NoMapsAndSetsWithExpectedSize` emits
constructors, `OptionalStreamRecipe` is gated on Java 9, and
`ThreadStopUnsupported` also comments on older Java versions.

* Give recipe options distinct, sentence-case display names

The seven `ReplaceComSunAWTUtilitiesMethods` and two
`ReplaceLocalizedStreamMethods` options each shared one display name and
description, and `AboutJavaVersion.whenUsesType` had no display name.
Option display names that were phrased as questions or carried their
default value now read as short labels, and descriptions end in a
period.

* Make `UpdateAnnotationAttributeJavaxToJakarta` `signature` option required

The option was declared optional, but a missing signature failed with a
`NullPointerException` when building the `AnnotationMatcher`. Every
declarative use passes a signature, so require it instead.

* Regenerate `recipes.csv`

Author: timtebeek

src/main/java/org/openrewrite/java/migrate/AddJDeprScanPlugin.java

@@ -30,11 +30,11 @@
 @RequiredArgsConstructor
 public class AddJDeprScanPlugin extends Recipe {
 
-    @Option(displayName = "release", description = "Specifies the Java SE release that provides the set of deprecated APIs for scanning.", required = false, example = "11")
+    @Option(displayName = "Release", description = "Specifies the Java SE release that provides the set of deprecated APIs for scanning.", required = false, example = "11")
     private final String release;
 
     @Getter
-    final String displayName = "Add `JDeprScan` Maven Plug-in";
+    final String displayName = "Add `JDeprScan` Maven plugin";
 
     @Getter
     final String description = "Add the `JDeprScan` Maven plugin to scan class files for uses of deprecated APIs.";

src/main/java/org/openrewrite/java/migrate/AddMissingMethodImplementation.java

@@ -36,23 +36,23 @@
 public class AddMissingMethodImplementation extends Recipe {
 
     @Option(displayName = "Fully qualified class name",
-            description = "A fully qualified class being implemented with missing method.",
+            description = "The fully qualified name of the interface whose method may be missing from its implementations.",
             example = "com.yourorg.FooBar")
     String fullyQualifiedClassName;
 
     @Option(displayName = "Method pattern",
-            description = "A method pattern for matching required method definition.",
+            description = "A method pattern matching the required method declaration.",
             example = "*..* hello(..)")
     String methodPattern;
 
     @Option(displayName = "Method template",
-            description = "Template of method to add",
+            description = "Template of the method to add.",
             example = "public String hello() { return \\\"Hello from #{}!\\\"; }")
     String methodTemplateString;
 
-    String displayName = "Adds missing method implementations";
+    String displayName = "Add missing method implementations";
 
-    String description = "Check for missing methods required by interfaces and adds them.";
+    String description = "Add a method to classes that implement the given interface but are missing the method.";
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

src/main/java/org/openrewrite/java/migrate/AddMockitoJavaAgentToMavenSurefirePlugin.java

@@ -61,7 +61,7 @@ public class AddMockitoJavaAgentToMavenSurefirePlugin extends Recipe {
     private static final String CONFIGURATION_TAG_TEMPLATE = "<configuration>\n<!--suppress MavenModelInspection --><argLine>%s</argLine></configuration>";
 
     @Getter
-    final String displayName = "Add Mockito Java Agent to Maven Surefire Plugin";
+    final String displayName = "Add Mockito Java agent to Maven Surefire plugin";
 
     @Getter
     final String description = "Mockito attaches its Byte Buddy agent to the running JVM at test time, which the JDK has " +

src/main/java/org/openrewrite/java/migrate/AddStaticVariableOnProducerSessionBean.java

@@ -38,7 +38,7 @@ public class AddStaticVariableOnProducerSessionBean extends ScanningRecipe<Set<S
     private static final XPathMatcher EJB_PATH = new XPathMatcher("ejb-jar/enterprise-beans/session");
 
     @Getter
-    final String displayName = "Adds `static` modifier to `@Produces` fields that are in session beans";
+    final String displayName = "Add `static` modifier to `@Produces` fields in session beans";
 
     @Getter
     final String description = "Ensures that the fields annotated with `@Produces` which is inside the session bean (`@Stateless`, `@Stateful`, or `@Singleton`) are declared `static`.";

src/main/java/org/openrewrite/java/migrate/AddSuppressionForIllegalReflectionWarningsPlugin.java

@@ -41,10 +41,10 @@ public class AddSuppressionForIllegalReflectionWarningsPlugin extends Recipe {
     private final String version;
 
     @Getter
-    final String displayName = "Add maven jar plugin to suppress illegal reflection warnings";
+    final String displayName = "Add `maven-jar-plugin` to suppress illegal reflection warnings";
 
     @Getter
-    final String description = "Adds a maven jar plugin that's configured to suppress Illegal Reflection Warnings.";
+    final String description = "Adds a `maven-jar-plugin` with `Add-Opens` manifest entries to `war` and `ear` modules to suppress illegal reflection warnings.";
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

src/main/java/org/openrewrite/java/migrate/AddSurefireFailsafeArgLine.java

@@ -36,15 +36,15 @@
 public class AddSurefireFailsafeArgLine extends Recipe {
 
     @Option(displayName = "Arg line",
-            description = "The arguments to add to the surefire and failsafe plugin `argLine` configuration. " +
-                          "Individual arguments are space-separated. Arguments already present in the existing argLine are not duplicated.",
+            description = "The arguments to add to the Maven Surefire and Failsafe plugin `argLine` configuration. " +
+                          "Individual arguments are space-separated. Arguments already present in the existing `argLine` are not duplicated.",
             example = "--add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.util=ALL-UNNAMED")
     String argLine;
 
-    String displayName = "Add `argLine` to surefire and failsafe plugins";
+    String displayName = "Add `argLine` to Maven Surefire and Failsafe plugins";
 
     String description = "Adds the specified arguments to the `argLine` configuration of the Maven Surefire and Failsafe plugins, " +
-            "merging with any existing argLine value without duplicating arguments. " +
+            "merging with any existing `argLine` value without duplicating arguments. " +
             "The `@{argLine}` [late property reference](https://maven.apache.org/surefire/maven-surefire-plugin/faq.html) is prepended " +
             "so that an agent injected by another plugin during the build, such as the JaCoCo coverage agent from " +
             "`jacoco-maven-plugin:prepare-agent`, is preserved rather than overwritten. It is not added when the existing " +

src/main/java/org/openrewrite/java/migrate/BeanDiscovery.java

@@ -35,9 +35,9 @@ public class BeanDiscovery extends Recipe {
     private static final XPathMatcher BEANS_MATCHER = new XPathMatcher("/beans");
     private static final Pattern VERSION_PATTERN = Pattern.compile("_([^\\/\\.]+)\\.xsd");
 
-    String displayName = "Behavior change to bean discovery in modules with `beans.xml` file with no version specified";
+    String displayName = "Add `version` and `bean-discovery-mode` to `beans.xml` files without a version";
 
-    String description = "Alters beans with missing version attribute to include this attribute as well as the bean-discovery-mode=\"all\" attribute to maintain an explicit bean archive.";
+    String description = "Alters `beans.xml` files with a missing `version` attribute to include this attribute as well as `bean-discovery-mode=\"all\"`, to maintain an explicit bean archive.";
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

src/main/java/org/openrewrite/java/migrate/ChangeDefaultKeyStore.java

@@ -35,11 +35,11 @@ public class ChangeDefaultKeyStore extends Recipe {
     private static final MethodMatcher KEYSTORE_METHOD_REF = new MethodMatcher("java.security.KeyStore getDefaultType()");
 
     @Getter
-    final String displayName = "Return String `jks` when  `KeyStore.getDefaultType()` is called";
+    final String displayName = "Replace `KeyStore.getDefaultType()` with `\"jks\"`";
 
     @Getter
     final String description = "In Java 11 the default keystore was updated from JKS to PKCS12. " +
-            "As a result, applications relying on KeyStore.getDefaultType() may encounter issues after migrating, " +
+            "As a result, applications relying on `KeyStore.getDefaultType()` may encounter issues after migrating, " +
             "unless their JKS keystore has been converted to PKCS12. " +
             "This recipe returns default key store of `jks` when `KeyStore.getDefaultType()` method is called to " +
             "use the pre Java 11 default keystore.";

src/main/java/org/openrewrite/java/migrate/IllegalArgumentExceptionToAlreadyConnectedException.java

@@ -38,7 +38,7 @@ public class IllegalArgumentExceptionToAlreadyConnectedException extends Recipe
     final String displayName = "Replace `IllegalArgumentException` with `AlreadyConnectedException` in `DatagramChannel.send()` method";
 
     @Getter
-    final String description = "Replace `IllegalArgumentException` with `AlreadyConnectedException` for DatagramChannel.send() to ensure compatibility with Java 11+.";
+    final String description = "Replace `IllegalArgumentException` with `AlreadyConnectedException` for `DatagramChannel.send()` to ensure compatibility with Java 11+.";
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

src/main/java/org/openrewrite/java/migrate/JREThrowableFinalMethods.java

@@ -63,9 +63,9 @@ public JREThrowableFinalMethods(
     final String displayName = "Rename final method declarations `getSuppressed()` and `addSuppressed(Throwable exception)` in classes that extend `Throwable`";
 
     @Getter
-    final String description = "The recipe renames  `getSuppressed()` and `addSuppressed(Throwable exception)` methods  in classes " +
+    final String description = "The recipe renames `getSuppressed()` and `addSuppressed(Throwable exception)` methods in classes " +
             "that extend `java.lang.Throwable` to `myGetSuppressed` and `myAddSuppressed(Throwable)`. " +
-            "These methods were added to Throwable in Java 7 and are marked final which cannot be overridden.";
+            "These methods were added to `Throwable` in Java 7 and are marked final which cannot be overridden.";
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {