Split openapi support in two modules, to avoid adding dependencies needed only for code-generation to project runtime classpath

Author: koperagen

core/src/main/kotlin/org/jetbrains/kotlinx/dataframe/jupyter/Integration.kt

@@ -181,6 +181,7 @@ internal class Integration(private val notebook: Notebook, private val options:
                 "org.jetbrains.kotlinx:dataframe-jdbc:$version",
                 "org.jetbrains.kotlinx:dataframe-arrow:$version",
                 "org.jetbrains.kotlinx:dataframe-openapi:$version",
+                "org.jetbrains.kotlinx:dataframe-openapi-generator:$version"
             )
         }
 

dataframe-openapi-generator/build.gradle.kts

@@ -0,0 +1,53 @@
+plugins {
+    with(libs.plugins) {
+        alias(kotlin.jvm)
+        alias(publisher)
+        alias(serialization)
+        alias(kover)
+        alias(ktlint)
+        alias(jupyter.api)
+    }
+}
+
+group = "org.jetbrains.kotlinx"
+
+val jupyterApiTCRepo: String by project
+
+repositories {
+    mavenLocal()
+    mavenCentral()
+    maven(jupyterApiTCRepo)
+}
+
+dependencies {
+    api(project(":core"))
+    api(project(":dataframe-openapi"))
+
+    implementation(libs.sl4j)
+    implementation(libs.kotlinLogging)
+    implementation(libs.kotlin.reflect)
+    implementation(libs.kotlinpoet)
+    api(libs.swagger) {
+        // Fix for Android
+        exclude("jakarta.validation")
+    }
+
+    testApi(project(":core"))
+    testImplementation(libs.junit)
+    testImplementation(libs.kotestAssertions) {
+        exclude("org.jetbrains.kotlin", "kotlin-stdlib-jdk8")
+    }
+}
+
+kotlinPublications {
+    publication {
+        publicationName = "dataframeOpenApi"
+        artifactId = project.name
+        description = "OpenAPI code generation support for Kotlin Dataframe"
+        packageName = artifactId
+    }
+}
+
+kotlin {
+    explicitApi()
+}

dataframe-openapi/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/DefaultReadOpenApiMethod.kt renamed to dataframe-openapi-generator/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/DefaultReadOpenApiMethod.kt

@@ -0,0 +1,30 @@
+package org.jetbrains.kotlinx.dataframe.io
+
+import org.jetbrains.kotlinx.dataframe.codeGen.FieldType
+import org.jetbrains.kotlinx.dataframe.codeGen.Marker
+import org.jetbrains.kotlinx.dataframe.codeGen.MarkerVisibility
+import org.jetbrains.kotlinx.dataframe.codeGen.ValidFieldName
+
+internal fun AdditionalProperty.Companion.getMarker(typeArguments: List<String>) =
+    Marker(
+        name = AdditionalProperty::class.qualifiedName!!,
+        isOpen = false,
+        fields = listOf(
+            generatedFieldOf(
+                fieldName = ValidFieldName.of(AdditionalProperty<*>::key.name),
+                columnName = AdditionalProperty<*>::key.name,
+                overrides = false,
+                fieldType = FieldType.ValueFieldType(String::class.qualifiedName!!),
+            ),
+            generatedFieldOf(
+                fieldName = ValidFieldName.of(AdditionalProperty<*>::`value`.name),
+                columnName = AdditionalProperty<*>::`value`.name,
+                overrides = false,
+                fieldType = FieldType.ValueFieldType(Any::class.qualifiedName!! + "?"),
+            ),
+        ),
+        superMarkers = emptyList(),
+        visibility = MarkerVisibility.EXPLICIT_PUBLIC,
+        typeParameters = emptyList(),
+        typeArguments = typeArguments,
+    )

dataframe-openapi/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/OpenApi.kt renamed to dataframe-openapi-generator/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/OpenApi.kt

@@ -1,41 +1,18 @@
 plugins {
-    with(libs.plugins) {
-        alias(kotlin.jvm)
-        alias(publisher)
-        alias(serialization)
-        alias(kover)
-        alias(ktlint)
-        alias(jupyter.api)
-    }
+    alias(libs.plugins.kotlin.jvm)
+    alias(libs.plugins.ktlint)
+    alias(libs.plugins.publisher)
+    alias(libs.plugins.jupyter.api)
 }
 
 group = "org.jetbrains.kotlinx"
 
-val jupyterApiTCRepo: String by project
-
 repositories {
-    mavenLocal()
     mavenCentral()
-    maven(jupyterApiTCRepo)
 }
 
 dependencies {
     api(project(":core"))
-
-    implementation(libs.sl4j)
-    implementation(libs.kotlinLogging)
-    implementation(libs.kotlin.reflect)
-    implementation(libs.kotlinpoet)
-    api(libs.swagger) {
-        // Fix for Android
-        exclude("jakarta.validation")
-    }
-
-    testApi(project(":core"))
-    testImplementation(libs.junit)
-    testImplementation(libs.kotestAssertions) {
-        exclude("org.jetbrains.kotlin", "kotlin-stdlib-jdk8")
-    }
 }
 
 kotlinPublications {

dataframe-openapi/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/OpenApiMarker.kt renamed to dataframe-openapi-generator/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/OpenApiMarker.kt

@@ -2,10 +2,6 @@ package org.jetbrains.kotlinx.dataframe.io
 
 import org.jetbrains.kotlinx.dataframe.annotations.DataSchema
 import org.jetbrains.kotlinx.dataframe.api.KeyValueProperty
-import org.jetbrains.kotlinx.dataframe.codeGen.FieldType
-import org.jetbrains.kotlinx.dataframe.codeGen.Marker
-import org.jetbrains.kotlinx.dataframe.codeGen.MarkerVisibility
-import org.jetbrains.kotlinx.dataframe.codeGen.ValidFieldName
 
 /**
  * A [DataSchema] interface can implement this if it represents a map-like data schema (so key: value).
@@ -19,29 +15,5 @@ public interface AdditionalProperty<T> : KeyValueProperty<T> {
     /** Value of the property. */
     override val value: T
 
-    public companion object {
-        internal fun getMarker(typeArguments: List<String>) =
-            Marker(
-                name = AdditionalProperty::class.qualifiedName!!,
-                isOpen = false,
-                fields = listOf(
-                    generatedFieldOf(
-                        fieldName = ValidFieldName.of(AdditionalProperty<*>::key.name),
-                        columnName = AdditionalProperty<*>::key.name,
-                        overrides = false,
-                        fieldType = FieldType.ValueFieldType(String::class.qualifiedName!!),
-                    ),
-                    generatedFieldOf(
-                        fieldName = ValidFieldName.of(AdditionalProperty<*>::`value`.name),
-                        columnName = AdditionalProperty<*>::`value`.name,
-                        overrides = false,
-                        fieldType = FieldType.ValueFieldType(Any::class.qualifiedName!! + "?"),
-                    ),
-                ),
-                superMarkers = emptyList(),
-                visibility = MarkerVisibility.EXPLICIT_PUBLIC,
-                typeParameters = emptyList(),
-                typeArguments = typeArguments,
-            )
-    }
+    public companion object
 }

dataframe-openapi/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/OpenApiType.kt renamed to dataframe-openapi-generator/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/OpenApiType.kt

@@ -127,7 +127,7 @@ dependencies {
     implementation("org.jetbrains.kotlinx:dataframe-excel:%dataFrameVersion%")
     implementation("org.jetbrains.kotlinx:dataframe-jdbc:%dataFrameVersion%")
     implementation("org.jetbrains.kotlinx:dataframe-arrow:%dataFrameVersion%")
-    implementation("org.jetbrains.kotlinx:dataframe-openapi:%dataFrameVersion%")
+    implementation("org.jetbrains.kotlinx:dataframe-openapi-generator:%dataFrameVersion%")
 }
 ```
 
@@ -144,6 +144,7 @@ dependencies {
     implementation 'org.jetbrains.kotlinx:dataframe-jdbc:%dataFrameVersion%'
     implementation 'org.jetbrains.kotlinx:dataframe-arrow:%dataFrameVersion%'
     implementation 'org.jetbrains.kotlinx:dataframe-openapi:%dataFrameVersion%'
+    implementation 'org.jetbrains.kotlinx:dataframe-openapi-generator:%dataFrameVersion%'
 }
 ```
 

dataframe-openapi-generator/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/getMarker.kt

@@ -20,7 +20,7 @@ dependencies {
     api(libs.kotlin.reflect)
     implementation(project(":core"))
     implementation(project(":dataframe-arrow"))
-    implementation(project(":dataframe-openapi"))
+    implementation(project(":dataframe-openapi-generator"))
     implementation(project(":dataframe-excel"))
     implementation(project(":dataframe-jdbc"))
 
@@ -116,12 +116,13 @@ val integrationTestTask = task<Test>("integrationTest") {
     dependsOn(":dataframe-arrow:publishToMavenLocal")
     dependsOn(":dataframe-excel:publishToMavenLocal")
     dependsOn(":dataframe-jdbc:publishToMavenLocal")
+    dependsOn(":dataframe-openapi-generator:publishToMavenLocal")
     dependsOn(":dataframe-openapi:publishToMavenLocal")
     dependsOn(":publishApiPublicationToMavenLocal")
     dependsOn(":dataframe-arrow:publishDataframeArrowPublicationToMavenLocal")
     dependsOn(":dataframe-excel:publishDataframeExcelPublicationToMavenLocal")
     dependsOn(":dataframe-jdbc:publishDataframeJDBCPublicationToMavenLocal")
-    dependsOn(":dataframe-openapi:publishDataframeOpenApiPublicationToMavenLocal")
+    dependsOn(":dataframe-openapi-generator:publishDataframeOpenApiPublicationToMavenLocal")
     dependsOn(":plugins:symbol-processor:publishMavenPublicationToMavenLocal")
     dependsOn(":core:publishCorePublicationToMavenLocal")
     description = "Runs integration tests."

dataframe-openapi/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/isOpenApi.kt renamed to dataframe-openapi-generator/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/isOpenApi.kt

@@ -20,7 +20,7 @@ group = "org.jetbrains.kotlinx.dataframe"
 dependencies {
     implementation(project(":core"))
     implementation(project(":dataframe-arrow"))
-    implementation(project(":dataframe-openapi"))
+    implementation(project(":dataframe-openapi-generator"))
     implementation(project(":dataframe-excel"))
     implementation(project(":dataframe-jdbc"))
     implementation(libs.ksp.api)

dataframe-openapi/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/readOpenapi.kt renamed to dataframe-openapi-generator/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/readOpenapi.kt

@@ -1153,7 +1153,7 @@ class DataFrameSymbolProcessorTest {
         }
     }
 
-    private val petstoreYaml = File("../../dataframe-openapi/src/test/resources/petstore.yaml")
+    private val petstoreYaml = File("../../dataframe-openapi-generator/src/test/resources/petstore.yaml")
 
     @Test
     fun `openApi yaml test`(): Unit =
@@ -1238,7 +1238,7 @@ class DataFrameSymbolProcessorTest {
             }
         }
 
-    private val apiGuruMetricsJson = File("../../dataframe-openapi/src/test/resources/apiGuruMetrics.json")
+    private val apiGuruMetricsJson = File("../../dataframe-openapi-generator/src/test/resources/apiGuruMetrics.json")
 
     @Test
     fun `non openApi json test 2`(): Unit =
@@ -1275,7 +1275,7 @@ class DataFrameSymbolProcessorTest {
             }
         }
 
-    private val petstoreJson = File("../../dataframe-openapi/src/test/resources/petstore.json")
+    private val petstoreJson = File("../../dataframe-openapi-generator/src/test/resources/petstore.json")
 
     @Test
     fun `openApi json test`(): Unit =

dataframe-openapi/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/resultTypes.kt renamed to dataframe-openapi-generator/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/resultTypes.kt

@@ -41,3 +41,4 @@ plugins {
 }
 include("dataframe-excel")
 include("core")
+include("dataframe-openapi-generator")