Merge pull request #37 from avaje/maven-plugin

Generate Provides Maven Plugin

Author: SentryMan

avaje-spi-core/pom.xml

@@ -13,7 +13,7 @@
 		<tag>HEAD</tag>
 	</scm>
   <properties>
-    <avaje.prisms.version>1.34</avaje.prisms.version>
+    <avaje.prisms.version>1.36</avaje.prisms.version>
   </properties>
 
   <dependencies>

avaje-spi-core/src/main/java/io/avaje/spi/internal/PomPluginWriter.java

@@ -0,0 +1,64 @@
+package io.avaje.spi.internal;
+
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.StandardOpenOption;
+
+import javax.lang.model.element.ModuleElement;
+
+final class PomPluginWriter {
+
+  private static final String PLUGIN =
+      "  <!-- generated by avaje spi service -->\n"
+          + "      <plugin>\n"
+          + "        <groupId>io.avaje</groupId>\n"
+          + "        <artifactId>avaje-provides-maven-plugin</artifactId>\n"
+          + "        <version>%s</version>\n"
+          + "        <executions>\n"
+          + "          <execution>\n"
+          + "            <!-- Will transform compiled module-info to add missing spi provides -->"
+          + "            <goals>\n"
+          + "              <goal>disable-apt-validation</goal>\n"
+          + "              <goal>add-module-spi</goal>\n"
+          + "            </goals>\n"
+          + "          </execution>\n"
+          + "        </executions>\n"
+          + "      </plugin>\n"
+          + "    ";
+
+  static void addPlugin2Pom() throws IOException {
+    var module = APContext.getProjectModuleElement();
+    // don't need mvn plugin if not using JPMS
+    if (disabledOrNotUsingModulePath(module)) {
+      return;
+    }
+
+    var pomPath = APContext.getBuildResource("").getParent().resolve("pom.xml");
+    if (!pomPath.toFile().exists()) {
+      return;
+    }
+
+    var pomContent = Files.readString(pomPath);
+    // if not already present in pom.xml
+    if (pomContent.contains("avaje-provides-maven-plugin")) {
+      return;
+    }
+    APContext.logNote("Adding avaje-provides-maven-plugin to pom");
+    var pluginsIndex = pomContent.indexOf("</plugins>");
+    var builder = new StringBuilder(pomContent);
+    if (pluginsIndex != -1) {
+      builder.insert(
+          pluginsIndex,
+          String.format(PLUGIN, "2.0"));
+
+      Files.writeString(
+          pomPath, builder.toString(), StandardOpenOption.CREATE, StandardOpenOption.WRITE);
+    }
+  }
+
+  private static boolean disabledOrNotUsingModulePath(ModuleElement module) {
+    return APContext.jdkVersion() < 24
+        || !APContext.getOption("buildPlugin").map(Boolean::valueOf).orElse(true)
+        || module == null || module.isUnnamed();
+  }
+}

avaje-spi-core/src/main/java/io/avaje/spi/internal/ServiceProcessor.java

@@ -31,6 +31,7 @@
 import javax.annotation.processing.ProcessingEnvironment;
 import javax.annotation.processing.RoundEnvironment;
 import javax.annotation.processing.SupportedAnnotationTypes;
+import javax.annotation.processing.SupportedOptions;
 import javax.lang.model.SourceVersion;
 import javax.lang.model.element.Element;
 import javax.lang.model.element.ElementKind;
@@ -53,6 +54,7 @@
 @GenerateAPContext
 @SuppressWarnings("exports")
 @GenerateModuleInfoReader
+@SupportedOptions("buildPlugin")
 @SupportedAnnotationTypes({
   ServiceProviderPrism.PRISM_TYPE,
   // makes the processor automatically run if any of the other avaje processors are active
@@ -64,6 +66,7 @@
   "io.avaje.recordbuilder.Generated",
   "io.avaje.prism.GenerateAPContext",
   "io.avaje.validation.spi.Generated",
+  "io.ebean.typequery.Generated",
   "javax.annotation.processing.Generated",
   "javax.annotation.processing.SupportedAnnotationTypes",
   "javax.annotation.processing.SupportedOptions",
@@ -143,6 +146,7 @@ public synchronized void init(ProcessingEnvironment env) {
         addition.append("avaje-spi-core");
       }
       Files.writeString(file, addition.toString(), StandardOpenOption.CREATE, StandardOpenOption.WRITE);
+      PomPluginWriter.addPlugin2Pom();
     } catch (IOException e) {
       // not an issue worth failing over
     }