Recipe to migrate JSR305 annotations

Author: jonesbusy

plugin-modernizer-core/pom.xml

@@ -311,6 +311,17 @@
                   <artifactId>hamcrest</artifactId>
                   <version>3.0</version>
                 </dependency>
+                <!-- JSR305 annotations -->
+                <dependency>
+                  <groupId>javax.annotation</groupId>
+                  <artifactId>javax.annotation-api</artifactId>
+                  <version>1.3.2</version>
+                </dependency>
+                <dependency>
+                  <groupId>com.google.code.findbugs</groupId>
+                  <artifactId>jsr305</artifactId>
+                  <version>3.0.2</version>
+                </dependency>
               </artifactItems>
             </configuration>
           </execution>

plugin-modernizer-core/src/main/resources/META-INF/rewrite/recipes.yml

@@ -734,11 +734,17 @@ tags: ['chore']
 recipeList:
   - io.jenkins.tools.pluginmodernizer.core.recipes.MigrateTomakehurstToWiremock
 ---
----
 type: specs.openrewrite.org/v1beta/recipe
 name: io.jenkins.tools.pluginmodernizer.JavaxAnnotationsToSpotbugs
 displayName: Migrate `javax.annotations` to SpotBugs annotations.
 description: SpotBugs is the [preferred replacement](https://www.jenkins.io/doc/developer/tutorial-improve/replace-jsr-305-annotations/) of JSR-305 annotations for Jenkins plugins.
 tags: ['chore']
 recipeList:
-  - org.openrewrite.jenkins.JavaxAnnotationsToSpotbugs
+  - org.openrewrite.java.ChangeType:
+      oldFullyQualifiedTypeName: javax.annotation.Nonnull
+      newFullyQualifiedTypeName: edu.umd.cs.findbugs.annotations.NonNull
+      ignoreDefinition: true
+  - org.openrewrite.java.ChangePackage:
+      oldPackageName: javax.annotation
+      newPackageName: edu.umd.cs.findbugs.annotations
+      recursive: false

plugin-modernizer-core/src/test/java/io/jenkins/tools/pluginmodernizer/core/recipes/DeclarativeRecipesTest.java

@@ -783,8 +783,21 @@ void upgradeToRecommendCoreVersionTest() {
                                 import hudson.util.IOException2;
                                 import java.io.File;
                                 import java.io.IOException;
+                                import javax.annotation.CheckForNull;
+                                import javax.annotation.Nonnull;
 
                                 public class Foo {
+
+                                    @CheckForNull
+                                    public String getSomething() {
+                                       return "something";
+                                    }
+
+                                    @Nonnull
+                                    public String getOther() {
+                                       return "something";
+                                    }
+
                                     public static void main(String[] args) {
                                         try {
                                             parseFile(new File("invalid.xml"));
@@ -802,10 +815,23 @@ private static void parseFile(File file) throws IOException2 {
                                     }
                                 }
                                 """, """
+                                import edu.umd.cs.findbugs.annotations.NonNull;
                                 import java.io.File;
                                 import java.io.IOException;
+                                import edu.umd.cs.findbugs.annotations.CheckForNull;
 
                                 public class Foo {
+
+                                    @CheckForNull
+                                    public String getSomething() {
+                                       return "something";
+                                    }
+
+                                    @NonNull
+                                    public String getOther() {
+                                       return "something";
+                                    }
+
                                     public static void main(String[] args) {
                                         try {
                                             parseFile(new File("invalid.xml"));

scripts/validate_metadata.py

@@ -95,7 +95,8 @@
     "io.jenkins.tools.pluginmodernizer.MigrateTomakehurstToWiremock",
     "io.jenkins.tools.pluginmodernizer.MigrateCommonsLang2ToLang3AndCommonText",
     "io.jenkins.tools.pluginmodernizer.RemoveOldJavaVersionForModernJenkins",
-    "io.jenkins.tools.pluginmodernizer.SwitchToRenovate"
+    "io.jenkins.tools.pluginmodernizer.SwitchToRenovate",
+    "io.jenkins.tools.pluginmodernizer.JavaxAnnotationsToSpotbugs"
 ]
 
 def validate_metadata(file_path):