Add Kotlin build script by CelDaemon · Pull Request #156 · FabricMC/fabricmc.net

CelDaemon · 2026-02-21T22:42:59Z

Unlike the other PR, this one tries to stay as similar as possible to the Groovy build script.

The only big difference is the loom version being set in the settings.gradle.kts file.

CelDaemon · 2026-02-22T03:57:34Z

Is missing config from fabric cli, I would convert to a draft but there seems to be no option to do that from the GitHub mobile app.

modmuss50

Also update the CLI and add it to the test script here: https://github.com/FabricMC/fabricmc.net/blob/main/cli/test.ts and run the tests to make sure this works on all versions.

scripts/src/lib/template/templates/gradle/kotlin/settings.gradle.kts.eta

scripts/src/lib/template/templates/gradle/kotlin/gradle.properties.eta

scripts/src/lib/template/templates/gradle/kotlin/build.gradle.kts.eta

scripts/src/lib/template/templates/gradle/groovy/build.gradle.eta

scripts/src/lib/Template.svelte

This reverts commit 4bac270. Will create a separate PR for this patch after the Kotlin DSL has been merged.

CelDaemon · 2026-02-22T17:13:31Z

I have successfully ran the tests (158 passed), though with the Kotlin language disabled and an extra check to skip yarn on unobfuscated versions.

Changes property access to use Gradle property providers instead of project property binding. Using Gradle properties in this way avoids issues when using Isolated Projects (gradle/gradle#32398). An alternative is to use `project.property(...)` instead, which is closer to the Groovy variant but does exhibit these issues.

Shares the gradle.properties.eta file between the Groovy and Kotlin DSL build script types.

Juuxel · 2026-02-26T11:42:48Z

scripts/src/lib/template/templates/gradle/kotlin/build.gradle.kts.eta

+
+	// Fabric API. This is technically optional, but you probably want it anyway.
+	<% if (it.unobfuscated) { %>implementation<% } else { %>modImplementation<% } %>("net.fabricmc.fabric-api:fabric-api:${providers.gradleProperty("fabric_api_version").get()}")
+	<% if (it.kotlin) { %><% if (it.unobfuscated) { %>implementation<% } else { %>modImplementation<% } %>("net.fabricmc:fabric-language-kotlin:${providers.gradleProperty("fabric_kotlin_version").get()}")<% } %>


General thought about these dep strings: I wonder if

modImplementation("net.fabricmc:fabric-language-kotlin:" + providers.gradleProperty("fabric_kotlin_version").get())

would be cleaner/easier to read than

modImplementation("net.fabricmc:fabric-language-kotlin:${providers.gradleProperty("fabric_kotlin_version").get()}")

I'm fine with either in any case.

Yeah it might be, it's closer to the groovy version like this though

kotlin idioms. use string templating not concatenation

also, if you're going to use providers, you may as well use .map {} instead of immediately resolving it. otherwise just use property("...")

property is being avoided to work around some issues relating to Isolated Projects. But yeah, mapping is a good idea.

scripts/src/lib/template/templates/gradle/kotlin/settings.gradle.kts.eta

scripts/src/lib/template/templates/gradle/kotlin/build.gradle.kts.eta

isXander · 2026-02-27T00:16:57Z

scripts/src/lib/template/templates/gradle/kotlin/build.gradle.kts.eta

+tasks.withType<JavaCompile>().configureEach {
+	options.release = <%= it.java.release %>
+}
+<% if (it.kotlin) { %>
+kotlin {
+	compilerOptions {
+		jvmTarget = JvmTarget.JVM_<%= it.java.compatibility %>
+	}
+}
+<% } %>
+java {
+	// Loom will automatically attach sourcesJar to a RemapSourcesJar task and to the "build" task
+	// if it is present.
+	// If you remove this line, sources will not be generated.
+	withSourcesJar()
+
+	sourceCompatibility = JavaVersion.VERSION_<%= it.java.compatibility %>
+	targetCompatibility = JavaVersion.VERSION_<%= it.java.compatibility %>
+}


Suggested change

tasks.withType<JavaCompile>().configureEach {

options.release = <%= it.java.release %>

}

<% if (it.kotlin) { %>

kotlin {

compilerOptions {

jvmTarget = JvmTarget.JVM_<%= it.java.compatibility %>

}

}

<% } %>

java {

// Loom will automatically attach sourcesJar to a RemapSourcesJar task and to the "build" task

// if it is present.

// If you remove this line, sources will not be generated.

withSourcesJar()

sourceCompatibility = JavaVersion.VERSION_<%= it.java.compatibility %>

targetCompatibility = JavaVersion.VERSION_<%= it.java.compatibility %>

}

java {

toolchain {

languageVersion = JavaLanguageVersion.of(<%= it.java.compatibility %>)

}

// Loom will automatically attach sourcesJar to a RemapSourcesJar task and to the "build" task

// if it is present.

// If you remove this line, sources will not be generated.

withSourcesJar()

}

This is a breaking change as it forces gradle to use a specific JDK version instead of configuring the target of the active toolchain. Should probably be a separate PR if you wish to change this.

modmuss50 · 2026-03-07T08:44:28Z

Going to merge this, if there are any future suggestions/improvements they can always be done later.

CelDaemon added 4 commits February 21, 2026 23:00

Add Kotlin DSL template

da7f522

Fix Kotlin target

4bac270

Fix yarn mappings

611cc9b

Fix inconsistent spacing

8d5455d

modmuss50 reviewed Feb 22, 2026

View reviewed changes

CelDaemon added 3 commits February 22, 2026 14:50

Use tab indenting in gradlekotlin.ts

03fbc62

Remove claims about Kotlin DSL

be03f1c

Move properties to a stricter scope

27f051d

CelDaemon force-pushed the kotlin-dsl branch from 4b6ad86 to a3258a3 Compare February 22, 2026 14:32

CelDaemon added 4 commits February 22, 2026 15:32

Use project properties to better match Groovy

f72f488

Revert "Fix Kotlin target"

dece8f7

This reverts commit 4bac270. Will create a separate PR for this patch after the Kotlin DSL has been merged.

Fix Kotlin DSL compatibility setting

52a3838

Add Kotlin DSL option to CLI

a3258a3

CelDaemon force-pushed the kotlin-dsl branch from 036e148 to 4c4ef68 Compare February 22, 2026 14:44

Put archivesBaseName in outer scope

f7f1f86

CelDaemon force-pushed the kotlin-dsl branch from 4c4ef68 to f7f1f86 Compare February 22, 2026 15:27

CelDaemon added 3 commits February 22, 2026 16:27

Default Kotlin DSL to false in CLI

b201474

Add Kotlin DSL option in interactive prompt

68b6774

Add DSL to test matrix

493b449

CelDaemon added 2 commits February 23, 2026 13:36

Share the gradle.properties template

ac40bdf

Shares the gradle.properties.eta file between the Groovy and Kotlin DSL build script types.

CelDaemon requested a review from modmuss50 February 23, 2026 19:26

CelDaemon added 2 commits February 24, 2026 11:23

Merge remote-tracking branch 'upstream/main' into kotlin-dsl

873735b

Fix Kotlin build for Kotlin build script

85dfcc2

modmuss50 approved these changes Feb 26, 2026

View reviewed changes

Juuxel suggested changes Feb 26, 2026

View reviewed changes

Fix whitespace differences

0b3f735

isXander reviewed Feb 27, 2026

View reviewed changes

CelDaemon mentioned this pull request Feb 28, 2026

Set project name instead of artifact name #159

Open

modmuss50 merged commit 46c25aa into FabricMC:main Mar 7, 2026
2 checks passed

modmuss50 mentioned this pull request Mar 7, 2026

Add Kotlin DSL template #23

Closed

Conversation

CelDaemon commented Feb 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

CelDaemon commented Feb 22, 2026

Uh oh!

modmuss50 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

CelDaemon commented Feb 22, 2026

Uh oh!

Juuxel Feb 26, 2026

Choose a reason for hiding this comment

Uh oh!

CelDaemon Feb 26, 2026

Choose a reason for hiding this comment

Uh oh!

isXander Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

isXander Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

CelDaemon Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

isXander Feb 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

CelDaemon Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

modmuss50 commented Mar 7, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

CelDaemon commented Feb 21, 2026 •

edited

Loading

isXander Feb 27, 2026 •

edited

Loading