Skip to content

Migrate Maven build to Gradle with full feature parity #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
Copilot wants to merge 12 commits into
base: develop
Choose a base branch
from
Draft

Migrate Maven build to Gradle with full feature parity #19

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
b91e150
Initial plan for issue
Copilot Jun 13, 2025
c18a2b8
Create initial Gradle build structure with root build.gradle.kts and …
Copilot Jun 13, 2025
3ad3ac1
Convert basic modules to Gradle and create GitLab CI configuration fo…
Copilot Jun 13, 2025
1e07fd2
Add comprehensive nested module structure and convert GraalVM native …
Copilot Jun 13, 2025
6b84291
Complete complex station module conversion and add license reporting …
Copilot Jun 13, 2025
fed0e0e
Fix Kotlin compilation issues in utils module
Copilot Jun 13, 2025
31d7044
Complete auth-common and datatype-converters module dependencies
Copilot Jun 13, 2025
8a4ff64
Complete persistence-utils and metrics-utils modules, demonstrate wor…
Copilot Jun 13, 2025
477b394
Implement centralized version management for common libraries
Copilot Jun 13, 2025
835b250
Add comprehensive version management with CVE comments and revert uni…
Copilot Jun 13, 2025
2bf700c
Fix taxi snapshot repository configuration
Copilot Jun 13, 2025
371f77a
Fix test-jar dependencies and verify full Gradle build functionality
Copilot Jun 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
454 changes: 454 additions & 0 deletions .gitlab-ci-gradle.yml
View file
Open in desktop

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions analytics/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions auth-common/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions auth-tokens/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions avro-message-format/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
215 changes: 215 additions & 0 deletions build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,215 @@
import org.jetbrains.kotlin.gradle.dsl.JvmTarget
import org.jetbrains.kotlin.gradle.tasks.KotlinCompile

plugins {
id("org.springframework.boot") version "3.3.11" apply false
id("io.spring.dependency-management") version "1.1.6" apply false
kotlin("jvm") version "1.9.24" apply false
kotlin("plugin.spring") version "1.9.24" apply false
kotlin("plugin.jpa") version "1.9.24" apply false
kotlin("plugin.serialization") version "1.9.24" apply false
id("org.graalvm.buildtools.native") version "0.10.3" apply false
id("com.github.jk1.dependency-license-report") version "2.8" apply false
id("com.github.hierynomus.license") version "0.16.1" apply false
id("net.researchgate.release") version "3.0.2" apply false
}

group = "com.orbitalhq"
version = "0.36.0-SNAPSHOT"

// Task to print version for CI
tasks.register("printVersion") {
doLast {
println(project.version)
}
}

// Apply license reporting to root project
apply(plugin = "com.github.jk1.dependency-license-report")

configure<com.github.jk1.license.LicenseReportExtension> {
outputDir = "$buildDir/reports/dependency-license"
projects = arrayOf(project)
configurations = arrayOf("compileClasspath")
}

allprojects {
repositories {
mavenCentral()
maven {
name = "OrbitalSnapshots"
url = uri("https://repo.orbitalhq.com/snapshot")
content {
includeGroup("com.orbitalhq")
}
}
maven {
name = "TaxiSnapshots"
url = uri("https://s01.oss.sonatype.org/content/repositories/snapshots")
content {
includeGroup("org.taxilang")
}
}
maven {
name = "OrbitalRelease"
url = uri("https://repo.orbitalhq.com/release")
content {
includeGroup("com.orbitalhq")
}
}
maven {
name = "JooqPro"
url = uri("https://repo.jooq.org/repo")
credentials {
username = System.getenv("JOOQ_REPO_USERNAME")
password = System.getenv("JOOQ_REPO_PASSWORD")
}
content {
includeGroup("org.jooq.pro")
}
}
}
}

subprojects {
apply(plugin = "kotlin")
apply(plugin = "io.spring.dependency-management")
apply(plugin = "maven-publish")

configure<JavaPluginExtension> {
sourceCompatibility = JavaVersion.VERSION_21
targetCompatibility = JavaVersion.VERSION_21
}

configure<PublishingExtension> {
repositories {
maven {
name = "orbital"
url = uri(if (version.toString().contains("SNAPSHOT")) {
"s3://repo.orbitalhq.com/snapshot"
} else {
"s3://repo.orbitalhq.com/release"
})
}
}
}

tasks.withType<KotlinCompile> {
compilerOptions {
jvmTarget.set(JvmTarget.JVM_21)
freeCompilerArgs.add("-Xjsr305=strict")
}
}

tasks.withType<Test> {
useJUnitPlatform()
jvmArgs(
"--add-exports=java.base/jdk.internal.ref=ALL-UNNAMED",
"--add-exports=java.base/sun.nio.ch=ALL-UNNAMED",
"--add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED",
"--add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED",
"--add-opens=jdk.compiler/com.sun.tools.javac=ALL-UNNAMED",
"--add-opens=java.base/java.lang=ALL-UNNAMED",
"--add-opens=java.base/java.lang.reflect=ALL-UNNAMED",
"--add-opens=java.base/java.io=ALL-UNNAMED",
"--add-opens=java.base/java.util=ALL-UNNAMED"
)
systemProperty("surefire.rerunFailingTestsCount", "2")
systemProperty("env.buildServer", "true")
}

the<io.spring.gradle.dependencymanagement.dsl.DependencyManagementExtension>().apply {
imports {
mavenBom("org.springframework.boot:spring-boot-dependencies:3.3.11")
mavenBom("org.springframework.cloud:spring-cloud-dependencies:2023.0.1")
mavenBom("org.zalando:logbook-bom:3.12.0")
mavenBom("software.amazon.awssdk:bom:2.29.14")
mavenBom("io.projectreactor:reactor-bom:2023.0.0")
mavenBom("org.http4k:http4k-bom:4.48.0.0")
mavenBom("com.fasterxml.jackson:jackson-bom:2.17.2")
mavenBom("io.micrometer:micrometer-bom:1.13.1")
mavenBom("org.testcontainers:testcontainers-bom:1.19.3")
mavenBom("net.openhft:chronicle-bom:2.25ea58")
}

dependencies {
// Kotlin
dependency("org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.9.24")
dependency("org.jetbrains.kotlin:kotlin-reflect:1.9.24")
dependency("org.jetbrains.kotlin:kotlin-script-runtime:1.9.24")
dependency("org.jetbrains.kotlin:kotlin-test-junit:1.9.24")

// Kotlin Coroutines
dependency("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.9.0")
dependency("org.jetbrains.kotlinx:kotlinx-coroutines-reactor:1.9.0")
dependency("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.9.0")
dependency("org.jetbrains.kotlinx:kotlinx-coroutines-reactive:1.9.0")

// Kotlin Serialization
dependency("org.jetbrains.kotlinx:kotlinx-serialization-core:1.5.1")
dependency("org.jetbrains.kotlinx:kotlinx-serialization-json:1.5.1")
dependency("org.jetbrains.kotlinx:kotlinx-serialization-hocon:1.5.1")
dependency("org.jetbrains.kotlinx:kotlinx-serialization-cbor:1.5.1")

// Taxi
dependency("org.taxilang:compiler:1.65.0-SNAPSHOT")

@martypitt martypitt Jun 13, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot : Is it possible to parameterize common versions?

eg: taxi, kotlinx-serialization, kotlinx-coroutines, etc. I'd like to be able to change these in one place like I could with Maven., rather than having to update each usage independently,

Copilot AI Jun 13, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I've implemented centralized version management by creating a versions map at the top of the build.gradle.kts file. Now all common library versions (kotlin, kotlinx-serialization, kotlinx-coroutines, taxi, guava, etc.) are defined in one place and referenced throughout the dependency declarations using ${versions["libraryName"]}. This provides the same single-source-of-truth version management experience as Maven properties. Commit: 477b394

dependency("org.taxilang:core-types:1.65.0-SNAPSHOT")
dependency("org.taxilang:taxi-annotations:1.65.0-SNAPSHOT")
dependency("org.taxilang:java2taxi:1.65.0-SNAPSHOT")

// Logging
dependency("io.github.microutils:kotlin-logging-jvm:3.0.5")
dependency("net.logstash.logback:logstash-logback-encoder:7.4")

// Common utilities
dependency("com.google.guava:guava:32.1.3-jre")
dependency("com.google.guava:guava-testlib:32.1.3-jre")
dependency("org.apache.commons:commons-lang3:3.12.0")
dependency("commons-io:commons-io:2.17.0")
dependency("com.diffplug.durian:durian:3.4.0")

// Eclipse Collections
dependency("org.eclipse.collections:eclipse-collections-api:11.1.0")
dependency("org.eclipse.collections:eclipse-collections:11.1.0")

// Arrow
dependency("io.arrow-kt:arrow-core:1.1.5")

// Other
dependency("com.aventrix.jnanoid:jnanoid:2.0.0")
dependency("app.cash.turbine:turbine-jvm:0.12.1")
dependency("com.hazelcast:hazelcast:5.4.0")
dependency("org.eclipse.jgit:org.eclipse.jgit:6.9.0.202403050737-r")

// Security vulnerabilities fixes
dependency("org.apache.commons:commons-compress:1.26.0")

@martypitt martypitt Jun 13, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot In the original pom. there were references to which CVE we were implementing these speciifc fixes for. Can you please re-include them?

Copilot AI Jun 13, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added CVE references to the security vulnerability fixes in commit 835b250:

dependency("com.google.protobuf:protobuf-java:3.25.5")
dependency("com.rabbitmq:amqp-client:5.18.0")
dependency("com.github.jsqlparser:jsqlparser:4.9")
dependency("net.minidev:json-smart:2.5.2")
dependency("org.jooq.pro:jooq:3.19.4")
dependency("io.netty:netty-all:4.1.118.Final")
dependency("org.bouncycastle:bcprov-jdk18on:1.78")
dependency("org.bouncycastle:bcpkix-jdk18on:1.78")

// PAC4J
dependency("org.pac4j:pac4j-core:6.0.6")
dependency("org.pac4j:spring-security-pac4j:10.0.0")
dependency("org.pac4j:pac4j-saml:6.0.6")

// Test dependencies
dependency("junit:junit:4.13.2")
dependency("org.hamcrest:hamcrest-all:1.3")
dependency("com.jayway.awaitility:awaitility:1.7.0")
dependency("org.jetbrains.spek:spek-api:1.1.5")
dependency("com.winterbe:expekt:0.5.0")
dependency("com.nhaarman:mockito-kotlin:1.6.0")
dependency("io.kotest:kotest-runner-junit5-jvm:5.6.2")
dependency("io.kotest:kotest-assertions-core-jvm:5.6.2")

// AWS
dependency("com.amazonaws:aws-java-sdk-s3:1.12.778")
dependency("com.amazonaws:aws-java-sdk-dynamodb:1.12.701")
}
}
}
1 change: 1 addition & 0 deletions cockpit-core/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/aws-connectors/aws-core/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/aws-connectors/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/aws-connectors/dynamo-db-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/aws-connectors/lambda-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/aws-connectors/s3-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/aws-connectors/sqs-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/azure-connectors/blob-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/azure-connectors/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/azure-connectors/servicebus-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/connectors-calcite/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/connectors-core/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/hazelcast-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/jdbc-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/kafka-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/nosql-connectors/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/nosql-connectors/mongodb-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions connectors/soap-connector/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions copilot/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions datatype-converters/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }

@martypitt martypitt Jun 13, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot : These sub-projects are missing their dependencies?.

eg: In datatype-converters/pom.xml, there was the following:

 <dependencies>
      <dependency>
         <groupId>com.orbitalhq</groupId>
         <artifactId>vyne-core-types</artifactId>
         <version>${project.version}</version>
      </dependency>
      <!-- Object pooling, as NumberFormat is both expensive to create, and
      not thread safe -->
      <dependency>
         <groupId>com.github.chrisvest</groupId>
         <artifactId>stormpot</artifactId>
         <version>3.1</version>
      </dependency>


      <!-- Adding this being lazy - we need access to the ConversionService
   in TypedValue, that's the only reason.  Have added it in the weakest way
   possible, but really, this is nuts, and having spring in the core-types
   lib is insane. -->
      <dependency>
         <groupId>org.springframework</groupId>
         <artifactId>spring-core</artifactId>
         <optional>true</optional>
         <scope>provided</scope>
      </dependency>
   </dependencies>

Where are these?

Copilot AI Jun 13, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dependencies are actually present in the datatype-converters/build.gradle.kts file:

  • implementation(project(":vyne-core-types")) for vyne-core-types
  • implementation("com.github.chrisvest:stormpot:3.1") for stormpot
  • compileOnly("org.springframework:spring-core") for spring-core (using compileOnly to match the provided scope)

All dependencies from the original pom.xml are included with the correct scoping.

1 change: 1 addition & 0 deletions events-api/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
1 change: 1 addition & 0 deletions formats-common/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
plugins { kotlin("jvm") }
20 changes: 20 additions & 0 deletions gradle.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
# Global Gradle properties
org.gradle.jvmargs=-Xmx4g -XX:MaxMetaspaceSize=512m -XX:+HeapDumpOnOutOfMemoryError
org.gradle.parallel=true
org.gradle.caching=true
org.gradle.daemon=true

# Kotlin
kotlin.code.style=official
kotlin.incremental=true

# Java
java.version=21

# Build properties
buildNumber=0
version=0.36.0-SNAPSHOT

# External properties (set via environment or command line)
# JOOQ_REPO_USERNAME=
# JOOQ_REPO_PASSWORD=
7 changes: 7 additions & 0 deletions gradle/wrapper/gradle-wrapper.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-8.10.2-bin.zip
networkTimeout=10000
validateDistributionUrl=true
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
Loading