advance MavenPlugin trait to also be limitable on groupd and artifact id

Author: MBoegers

rewrite-maven/src/main/java/org/openrewrite/maven/AddAnnotationProcessor.java

@@ -92,10 +92,7 @@ class AddAnnotationProcessorPath extends MavenIsoVisitor<ExecutionContext> {
                 @Override
                 public Xml.Tag visitTag(Xml.Tag tag, ExecutionContext ctx) {
                     Xml.Tag plugins = super.visitTag(tag, ctx);
-                    plugins = (Xml.Tag) new MavenPlugin.Matcher(addToManaged).asVisitor(plugin -> {
-                        if (!MAVEN_COMPILER_PLUGIN_GROUP_ID.equals(plugin.getGroupId()) || !MAVEN_COMPILER_PLUGIN_ARTIFACT_ID.equals(plugin.getArtifactId())) {
-                            return plugin.getTree();
-                        }
+                    plugins = (Xml.Tag)  new MavenPlugin.Matcher(addToManaged,MAVEN_COMPILER_PLUGIN_GROUP_ID, MAVEN_COMPILER_PLUGIN_ARTIFACT_ID).asVisitor(plugin -> {
 
                         MavenResolutionResult mrr = getResolutionResult();
                         AtomicReference<TreeVisitor<?, ExecutionContext>> afterVisitor = new AtomicReference<>();

rewrite-maven/src/main/java/org/openrewrite/maven/trait/MavenPlugin.java

@@ -42,13 +42,19 @@ public static class Matcher extends MavenTraitMatcher<MavenPlugin> {
 
         @Nullable
         Boolean isManaged;
+        @Nullable
+        String groupId;
+        @Nullable
+        String artifactId;
 
         public Matcher() {
-            this(null);
+            this(null,  null, null);
         }
 
-        public Matcher(@Nullable Boolean isManaged) {
+        public Matcher(@Nullable Boolean isManaged,  @Nullable String groupId, @Nullable String artifactId) {
             this.isManaged = isManaged;
+            this.groupId = groupId;
+            this.artifactId = artifactId;
         }
 
         @Override
@@ -65,11 +71,17 @@ public Matcher(@Nullable Boolean isManaged) {
                     return null;
                 }
 
-                return new MavenPlugin(
-                        cursor,
-                        getProperty(cursor, "groupId").orElse("org.apache.maven.plugins"),
-                        getProperty(cursor, "artifactId").orElse(null)
-                );
+                String currentGroupId = getProperty(cursor, "groupId").orElse("org.apache.maven.plugins");
+                if (groupId != null && !groupId.equals(currentGroupId)) {
+                    return null;
+                }
+
+                String currentArtifactId = getProperty(cursor, "artifactId").orElse(null);
+                if (artifactId != null &&  currentArtifactId != null && !artifactId.equals(currentArtifactId)) {
+                    return null;
+                }
+
+                return new MavenPlugin(cursor, currentGroupId, currentArtifactId);
             }
             return null;
         }