Bedrock support

.github/workflows/gradle.yml

@@ -14,16 +14,14 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v7
       with:
         fetch-depth: 0
-    - name: Set up JDK 17
-      uses: actions/setup-java@v4
+    - uses: actions/setup-java@v5
       with:
         distribution: 'temurin'
-        java-version: '17'
-    - name: Setup Gradle
-      uses: gradle/actions/setup-gradle@v4
+        java-version: '25'
+    - uses: gradle/actions/setup-gradle@v6
       with:
         cache-read-only: ${{ github.ref != 'refs/heads/master' && github.ref != 'refs/heads/chunky-2.4.x' }}
     - name: Grant execute permission for gradlew
@@ -46,12 +44,12 @@ jobs:
             ;;
         esac
     - name: Upload build
-      uses: actions/upload-artifact@v4
+      uses: actions/upload-artifact@v7
       with:
         name: Chunky Build
         path: build/installer
     - name: Upload build
-      uses: actions/upload-artifact@v4
+      uses: actions/upload-artifact@v7
       with:
         name: Chunky Core
         path: build/chunky-core-*.jar

build.gradle

@@ -1,10 +1,17 @@
-project.version = getVersion()
-println "Building version ${project.version}"
+def projectVersion = getVersion().toString()
+println "Building version ${projectVersion}"
 
 allprojects {
     repositories {
         mavenCentral()
-        mavenLocal()
+        // FIXME @NotStirred: DO NOT RELY ON SNAPSHOT LEVELDB-FFI
+        maven {
+          name = "Maven Central SNAPSHOT"
+          url = "https://central.sonatype.com/repository/maven-snapshots/"
+        }
+    }
+    configurations.configureEach { // FIXME: Remove, this makes gradle never cache snapshot jars (leveldb-ffi snapshots are changing throughout dev)
+      it.resolutionStrategy.cacheChangingModulesFor 0, 'seconds'
     }
 }
 
@@ -15,7 +22,7 @@ buildscript {
         }
     }
     dependencies {
-      classpath 'org.openjfx:javafx-plugin:0.0.13'
+      classpath 'org.openjfx:javafx-plugin:0.1.0'
       classpath 'com.github.ben-manes:gradle-versions-plugin:0.46.0'
     }
 }
@@ -33,20 +40,10 @@ subprojects {
     java {
         toolchain {
             // default java version for the project
-            languageVersion = JavaLanguageVersion.of(17)
-        }
-    }
-    compileTestJava {
-        // compile tests using more recent features
-        javaCompiler = javaToolchains.compilerFor {
-            languageVersion = JavaLanguageVersion.of(21)
+            languageVersion = JavaLanguageVersion.of(25)
         }
     }
     test {
-        // run tests using more recent features
-        javaLauncher = javaToolchains.launcherFor {
-            languageVersion = JavaLanguageVersion.of(21)
-        }
         useJUnitPlatform()
 
         // Always run tests, even when nothing changed.
@@ -55,7 +52,7 @@ subprojects {
         finalizedBy jacocoTestReport // report is always generated after tests run (it's very cheap)
     }
     jacoco {
-        toolVersion = "0.8.11"
+        toolVersion = "0.8.15"
     }
     jacocoTestReport {
         dependsOn test // tests are required to run before generating the report
@@ -113,19 +110,37 @@ defaultTasks 'release'
 
 task releaseVersion {
   doLast {
-    tryCommand([ 'git', 'tag', '-a', "${project.version}", '-m', "Version ${project.version}" ], true)
+    tryCommand([ 'git', 'tag', '-a', "${projectVersion}", '-m', "Version ${projectVersion}" ], true)
   }
 }
 
+task prepareReleaseLibraries {
+    dependsOn ':chunky:copyExternalDependencies'
+    outputs.upToDateWhen { false }
+
+    doLast {
+        def libDir = file('chunky/lib')
+        libDir.mkdirs()
+        copy {
+            from project(':chunky').tasks.named('copyExternalDependencies').get().outputs.files
+            into libDir
+        }
+    }
+}
+
 task versionInfo(type: JavaExec) {
     dependsOn 'copyArtifacts'
+    dependsOn 'prepareReleaseLibraries'
     outputs.upToDateWhen { false }
 
-    outputs.files file("build/chunky-${project.version}.jar")
-    description 'Writes build/chunky-VERSION.jar, latest.json and updates chunky-core-VERSION.jar/version.json'
+    outputs.file(layout.buildDirectory.file("chunky-${projectVersion}.jar"))
+
+    description = 'Writes build/chunky-VERSION.jar, latest.json and updates chunky-core-VERSION.jar/version.json'
+
     classpath = project(':releasetools').sourceSets.main.runtimeClasspath
     mainClass.set('releasetools.ReleaseBuilder')
-    args "${project.version}", "release_notes-${project.version}.txt"
+
+    args projectVersion, "release_notes-${projectVersion}.txt"
 }
 
 task release {
@@ -138,30 +153,34 @@ task release {
         destinationDirectory.mkdirs()
         file("./latest.json").renameTo("${destinationDirectory}/latest.json")
         fileTree(buildDir).matching {
-            include "*-${project.version}.*"
+            include "*-${projectVersion}.*"
         }.each {
             it.renameTo("${destinationDirectory}/${it.name}")
         }
     }
 }
 
+def launcherArchives = project(':launcher').configurations.archives
+
 task buildReleaseJar(type: Jar) {
     dependsOn ':launcher:assembleDist'
+    dependsOn 'prepareReleaseLibraries'
     dependsOn 'versionInfo'
 
-    archiveFileName = "chunky-${project.version}.jar"
+    archiveFileName = "chunky-${projectVersion}.jar"
     destinationDirectory = file('build/installer')
     manifest {
         attributes('Main-Class': 'se.llbit.chunky.launcher.ChunkyLauncher')
+        duplicatesStrategy = DuplicatesStrategy.EXCLUDE
     }
 
     into('lib') {
         from fileTree('chunky/lib').include('*.jar')
-        from file("build/chunky-core-${project.version}.jar")
+        from file("build/chunky-core-${projectVersion}.jar")
     }
 
     from {
-        project(':launcher').configurations.archives.allArtifacts.files.collect {
+        launcherArchives.allArtifacts.files.collect {
             zipTree(it)
         }
     }
@@ -170,10 +189,20 @@ task buildReleaseJar(type: Jar) {
     rename "latest.json", "version.json"
 }
 
-task copyArtifacts(type: Copy) {
+task copyArtifacts {
     dependsOn subprojects.jar
-    from subprojects.jar
-    into buildDir
+    dependsOn ':chunky:jar'
+    outputs.upToDateWhen { false }
+
+    doLast {
+        def sourceJar = project(':chunky').tasks.named('jar').get().archiveFile.get().asFile
+        def rootJar = file("build/chunky-core-${projectVersion}.jar")
+        copy {
+            from sourceJar
+            into rootJar.parentFile
+            rename { rootJar.name }
+        }
+    }
 }
 
 /** Helper function to run a command. Returns the command output if the command succeeded. */

chunky/.gitignore

@@ -1,3 +1,4 @@
 /bin/
 /build/
+/lib/
 /src/gen-res/

chunky/build.gradle

@@ -2,8 +2,13 @@ apply plugin: 'application'
 apply plugin: 'maven-publish'
 apply plugin: 'org.openjfx.javafxplugin'
 
-mainClassName = 'se.llbit.chunky.main.Chunky'
-archivesBaseName = 'chunky-core'
+application {
+  mainClass = 'se.llbit.chunky.main.Chunky'
+}
+
+base {
+  archivesName = 'chunky-core'
+}
 
 configurations {
   implementation.extendsFrom configurations.jsonlib
@@ -25,11 +30,13 @@ dependencies {
   implementation 'com.google.code.gson:gson:2.9.0'
   implementation 'org.lz4:lz4-java:1.8.0'
   implementation 'org.apache.maven:maven-artifact:3.9.9'
+  implementation 'io.github.notstirred:leveldb-ffi:0.1.0-SNAPSHOT'
+  implementation 'org.cloudburstmc:nbt:3.0.0.Final'
   implementation project(':lib')
 }
 
 javafx {
-  version = '17.0.11'
+  version = '25'
   configuration = 'implementation'
   modules = ['javafx.base', 'javafx.controls', 'javafx.fxml']
 }
@@ -52,7 +59,7 @@ jar {
     zipTree(it)
   }
   manifest {
-    attributes('Main-Class': mainClassName)
+    attributes('Main-Class': application.mainClass)
   }
   into('se/llbit/chunky/main') {
     from file("src/gen-res/Version.properties")
@@ -78,25 +85,35 @@ sourceSets {
 
 processResources.dependsOn 'updateVersionString'
 
+def projectVersion = providers.provider {
+    version.toString()
+}
+
 task updateVersionString {
-  description 'Store the current version string in src/gen-res/Version.properties'
+  description = 'Store the current version string in src/gen-res/Version.properties'
+
+  def versionFile = layout.projectDirectory.file('src/gen-res/Version.properties')
 
   outputs.upToDateWhen {
     def props = new Properties()
-    def output = file('src/gen-res/Version.properties')
-    if (output.isFile()) {
-      output.withInputStream { stream -> props.load(stream) }
+    if (versionFile.asFile.isFile()) {
+      versionFile.asFile.withInputStream { stream ->
+        props.load(stream)
+      }
     }
-    props['version'] == project.version
+    props['version'] == projectVersion.get()
   }
 
   doLast {
-    file('src/gen-res').mkdirs()
-    def date = new Date()
-    def versionFile = file('src/gen-res/Version.properties')
-    ant.propertyfile(file: versionFile) {
-      entry(key: 'version', value: project.version)
-      entry(key: 'gitSha', value: tryCommand(['git', 'rev-parse', 'HEAD']).trim())
+    versionFile.asFile.parentFile.mkdirs()
+
+    def gitSha = rootProject.tryCommand(
+      ['git', 'rev-parse', 'HEAD']
+    ).trim()
+
+    ant.propertyfile(file: versionFile.asFile) {
+      entry(key: 'version', value: projectVersion.get())
+      entry(key: 'gitSha', value: gitSha)
     }
   }
 }
@@ -112,12 +129,13 @@ task sourcesJar(type: Jar) {
 }
 
 task copyExternalDependencies(type: Copy) {
-  into "${buildDir}/${libsDirName}"
+  into layout.buildDirectory.dir("libs")
   from configurations.externalDependencies
 }
 
-artifacts {
-  archives javadocJar, sourcesJar
+tasks.named("assemble") {
+    dependsOn(javadocJar)
+    dependsOn(sourcesJar)
 }
 
 publishing {
@@ -129,7 +147,7 @@ publishing {
         packaging = "jar"
         description = "Minecraft mapping and rendering tool"
         url = "http://chunky.llbit.se"
-        artifactId = archivesBaseName
+        artifactId = base.archivesName
         version = "2.5.0-SNAPSHOT"
 
         licenses {
@@ -156,22 +174,20 @@ publishing {
 
     // fix dependency scopes (see https://discuss.gradle.org/t/maven-publish-plugin-generated-pom-making-dependency-scope-runtime/7494/9)
     // and filter dependencies
+    def implementationDependencies = configurations.implementation.allDependencies
     publishing.publications.all {
       pom.withXml {
-        asNode().dependencies.'*'.findAll() {
-          it.scope.text() == 'runtime' && project.configurations.implementation.allDependencies.find { dep ->
+        asNode().dependencies.'*'.findAll {
+          it.scope.text() == 'runtime' &&
+          implementationDependencies.find { dep ->
             dep.name == it.artifactId.text()
           }
-        }.each() {
+        }.each {
           if (it.groupId.text() == "org.openjfx") {
-            // remove javafx dependencies for now (maybe make them optional in the future?)
             it.parent().remove(it)
-            // it.appendNode('optional', 'true')
           } else if (it.groupId.text() == "se.llbit") {
-            // se.llbit dependencies are included in chunky-core
             it.parent().remove(it)
           } else {
-            // everything else is a dependency as usual
             it.scope*.value = 'compile'
           }
         }
@@ -181,8 +197,8 @@ publishing {
 
   repositories {
     maven {
-      name 'Build'
-      url layout.buildDirectory.dir('maven')
+      name = 'Build'
+      url = layout.buildDirectory.dir('maven')
     }
   }
 }

chunky/src/java/se/llbit/chunky/Plugin.java

@@ -18,6 +18,7 @@
 package se.llbit.chunky;
 
 import se.llbit.chunky.main.Chunky;
+import se.llbit.util.concurrent.ChunkyThread;
 
 /**
  * The plugin interface for Chunky plugins.
@@ -33,4 +34,17 @@ public interface Plugin {
    * @param chunky Chunky instance which the plugin should attach to
    */
   void attach(Chunky chunky);
+
+  /**
+   * Called when chunky shuts down, allowing the plugin to close critical resources. Most plugins do not need to do
+   * anything here.
+   *
+   * <p>This function will be called after all {@link ChunkyThread}s have shut down. Chunky's UI thread may still
+   * be running.</p>
+   * <p>This method will not be called if any {@link ChunkyThread} does not shut down within its given time limit</p>
+   * <p>This method may not be called if chunky terminates in a non-normal way, such as through {@link Runtime#halt},
+   * the user killing the process (<code>SIGKILL</code> & <code>TerminateProcess</code>), or an error within a native
+   * function.</p>
+   */
+  default void shutdown(Chunky chunky) { }
 }