jooby-apt: filter out open-api/swagger annotations as route attributes

- fix #3525
- fix #3527

Author: jknack

modules/jooby-apt/src/main/java/io/jooby/internal/apt/RouteAttributesGenerator.java

@@ -7,7 +7,6 @@
 
 import static io.jooby.apt.JoobyProcessor.Options.SKIP_ATTRIBUTE_ANNOTATIONS;
 import static io.jooby.internal.apt.CodeBlock.indent;
-import static io.jooby.internal.apt.CodeBlock.type;
 
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
@@ -16,7 +15,6 @@
 
 import javax.lang.model.element.*;
 import javax.lang.model.type.TypeMirror;
-import javax.lang.model.util.Elements;
 import javax.lang.model.util.SimpleAnnotationValueVisitor14;
 import javax.lang.model.util.Types;
 
@@ -32,6 +30,8 @@ private record EnumValue(String type, String value) {}
               || it.startsWith("jakarta.ws.rs")
               || it.startsWith("javax.ws.rs");
 
+  private static final Predicate<String> OPEN_API = it -> it.startsWith("io.swagger");
+
   private static final Predicate<String> NULL_ANNOTATION =
       it ->
           it.endsWith("NonNull")
@@ -42,15 +42,13 @@ private record EnumValue(String type, String value) {}
   private static final Predicate<String> KOTLIN_ANNOTATION = it -> it.equals("kotlin.Metadata");
 
   private static final Predicate<String> ATTR_FILTER =
-      HTTP_ANNOTATION.or(NULL_ANNOTATION).or(KOTLIN_ANNOTATION);
+      HTTP_ANNOTATION.or(NULL_ANNOTATION).or(KOTLIN_ANNOTATION).or(OPEN_API);
 
   private final List<String> skip;
-  private final Elements elements;
   private final Types types;
 
   public RouteAttributesGenerator(MvcContext context) {
     var environment = context.getProcessingEnvironment();
-    this.elements = environment.getElementUtils();
     this.types = environment.getTypeUtils();
     this.skip = Options.stringListOpt(environment, SKIP_ATTRIBUTE_ANNOTATIONS);
   }

modules/jooby-apt/src/test/java/tests/i3455/C3455.java

@@ -9,7 +9,6 @@
 import io.jooby.annotation.GET;
 import io.jooby.annotation.Path;
 import io.jooby.annotation.QueryParam;
-import io.swagger.v3.oas.annotations.media.Schema;
 
 @Path("/\"path")
 public class C3455 {

modules/jooby-apt/src/test/java/tests/i3455/Schema.java

@@ -0,0 +1,17 @@
+/*
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
+package tests.i3455;
+
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
+
+@Retention(RetentionPolicy.RUNTIME)
+@Target(ElementType.METHOD)
+public @interface Schema {
+  String description() default "";
+}

modules/jooby-apt/src/test/java/tests/i3525/Issue3525.java

@@ -5,7 +5,7 @@
  */
 package tests.i3525;
 
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 
 import org.junit.jupiter.api.Test;
 
@@ -18,8 +18,8 @@ public void shouldNotRunOnInfiniteLoop() throws Exception {
     new ProcessorRunner(new C3525())
         .withSourceCode(
             source -> {
-              assertTrue(source.contains("\"Parameter.description\", \"paramA\","));
-              assertTrue(source.contains("\"Parameter.description\", \"paramB\","));
+              assertFalse(source.contains("\"Parameter.description\", \"paramA\","));
+              assertFalse(source.contains("\"Parameter.description\", \"paramB\","));
             });
   }
 }