separate internal and external public processors (#1167)

graemerocher · web-flow · commit 37be4388f2e7 · 2025-08-25T20:49:11.000-04:00
In #1165 I included a new public processor with a previously only internal one. In retrospect these need to be two different modules. This PR separates the internal only and the processor required by `@SdkImport`.
diff --git a/buildSrc/src/main/groovy/io.micronaut.build.internal.ocisdk-metadata-module.gradle b/buildSrc/src/main/groovy/io.micronaut.build.internal.ocisdk-metadata-module.gradle
@@ -6,7 +6,7 @@ plugins {
 }
 
 dependencies {
-    annotationProcessor(projects.micronautOraclecloudSerdeProcessor)
+    annotationProcessor(projects.micronautOraclecloudSerdeInternalProcessor)
     // The dependency is required for annotations and their metadata
     implementation(mnSerde.micronaut.serde.api)
 }
diff --git a/oraclecloud-httpclient-netty/build.gradle b/oraclecloud-httpclient-netty/build.gradle
@@ -25,13 +25,15 @@ dependencies {
 
     testAnnotationProcessor mn.micronaut.inject.java
     testAnnotationProcessor projects.micronautOraclecloudSerdeProcessor
+    testAnnotationProcessor projects.micronautOraclecloudSerdeInternalProcessor
     testAnnotationProcessor(mnSourcegen.micronaut.sourcegen.generator.java)
     testCompileOnly mn.micronaut.jackson.databind
     testRuntimeOnly mn.micronaut.runtime
 
     testImplementation mn.micronaut.inject.java.test
     testImplementation mnSerde.micronaut.serde.processor
     testImplementation projects.micronautOraclecloudSerdeProcessor
+    testImplementation projects.micronautOraclecloudSerdeInternalProcessor
     testImplementation mn.micronaut.management
     testImplementation mn.micronaut.context
     testImplementation projects.micronautOraclecloudCommon
diff --git a/oraclecloud-serde-internal-processor/build.gradle b/oraclecloud-serde-internal-processor/build.gradle
@@ -0,0 +1,15 @@
+plugins {
+    id 'io.micronaut.build.internal.common'
+    id 'io.micronaut.build.internal.oraclecloud-base'
+}
+
+dependencies {
+    // Transitive dependencies
+    implementation mn.micronaut.inject
+    implementation mn.micronaut.inject.java
+    implementation mnSerde.micronaut.serde.processor
+    implementation mnSerde.micronaut.serde.api
+
+    testImplementation mn.micronaut.inject.java.test
+    testImplementation libs.oci.common
+}
diff --git a/oraclecloud-serde-internal-processor/src/main/java/io/micronaut/oraclecloud/httpclient/netty/visitor/OciSdkModelSerdeVisitor.java b/oraclecloud-serde-internal-processor/src/main/java/io/micronaut/oraclecloud/httpclient/netty/visitor/OciSdkModelSerdeVisitor.java
diff --git a/oraclecloud-serde-internal-processor/src/main/resources/META-INF/services/io.micronaut.inject.visitor.TypeElementVisitor b/oraclecloud-serde-internal-processor/src/main/resources/META-INF/services/io.micronaut.inject.visitor.TypeElementVisitor
@@ -0,0 +1 @@
+io.micronaut.oraclecloud.httpclient.netty.visitor.OciSdkModelSerdeVisitor
diff --git a/oraclecloud-serde-internal-processor/src/test/groovy/io/micronaut/oraclecloud/httpclient/netty/visitor/OCISdkModelSerdeVisitorSpec.groovy b/oraclecloud-serde-internal-processor/src/test/groovy/io/micronaut/oraclecloud/httpclient/netty/visitor/OCISdkModelSerdeVisitorSpec.groovy
diff --git a/oraclecloud-serde-internal-processor/src/test/resources/logback.xml b/oraclecloud-serde-internal-processor/src/test/resources/logback.xml
@@ -0,0 +1,13 @@
+<configuration>
+
+    <appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
+        <encoder>
+            <pattern>%d{HH:mm:ss.SSS} [%thread] %-5level %logger{36} - %msg%n</pattern>
+        </encoder>
+    </appender>
+
+    <root level="info">
+        <appender-ref ref="STDOUT" />
+    </root>
+
+</configuration>
diff --git a/oraclecloud-serde-processor/src/main/java/io/micronaut/oraclecloud/sdk/processor/SdkImportVisitor.java b/oraclecloud-serde-processor/src/main/java/io/micronaut/oraclecloud/sdk/processor/SdkImportVisitor.java
@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package io.micronaut.oraclecloud.httpclient.netty.visitor;
+package io.micronaut.oraclecloud.sdk.processor;
 
 import io.micronaut.context.annotation.Bean;
 import io.micronaut.context.annotation.Factory;
diff --git a/oraclecloud-serde-processor/src/main/resources/META-INF/services/io.micronaut.inject.visitor.TypeElementVisitor b/oraclecloud-serde-processor/src/main/resources/META-INF/services/io.micronaut.inject.visitor.TypeElementVisitor
@@ -1,2 +1 @@
-io.micronaut.oraclecloud.httpclient.netty.visitor.OciSdkModelSerdeVisitor
-io.micronaut.oraclecloud.httpclient.netty.visitor.SdkImportVisitor
+io.micronaut.oraclecloud.sdk.processor.SdkImportVisitor
diff --git a/settings.gradle.kts b/settings.gradle.kts
@@ -30,6 +30,7 @@ include("oraclecloud-function-http-test")
 include("oraclecloud-httpclient-apache-http-core")
 include("oraclecloud-httpclient-netty")
 include("oraclecloud-serde-processor")
+include("oraclecloud-serde-internal-processor")
 include("oraclecloud-logging")
 include("oraclecloud-micrometer")
 include("oraclecloud-oke-kubernetes-client")

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ plugins {`
`6`	`6`	`}`
`7`	`7`
`8`	`8`	`dependencies {`
`9`		`- annotationProcessor(projects.micronautOraclecloudSerdeProcessor)`
	`9`	`+ annotationProcessor(projects.micronautOraclecloudSerdeInternalProcessor)`
`10`	`10`	`// The dependency is required for annotations and their metadata`
`11`	`11`	`implementation(mnSerde.micronaut.serde.api)`
`12`	`12`	`}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+io.micronaut.oraclecloud.httpclient.netty.visitor.OciSdkModelSerdeVisitor`
Original file line number	Diff line number	Diff line change
`@@ -1,2 +1 @@`
`1`		`-io.micronaut.oraclecloud.httpclient.netty.visitor.OciSdkModelSerdeVisitor`
`2`		`-io.micronaut.oraclecloud.httpclient.netty.visitor.SdkImportVisitor`
	`1`	`+io.micronaut.oraclecloud.sdk.processor.SdkImportVisitor`