Fix Gradle build performance issues and enable configuration cache support

[MattBDev]

Overview

Three small changes were made to the build scripts that remove redundant or legacy Gradle tasks/behavior. One of which unblocks Gradle's configuration cache and removes a dependency on a legacy Gradle plugin.

Description

• Replaced our dependency on Grgit with a providers.exec-based helper for reading the commit hash and date. Previously, Grgit forked git twice at configuration time, which the configuration cache does not support. Grgit will never add support for the configuration cache and is in fact considered to be in "maintenance mode". providers.exec results are recorded as declared build inputs instead, so they can be cached and replayed.
  • Hash length from Grgit/Jgit is preserved.
• Removed the allprojects { gradle.projectsEvaluated { ... } } wrapper around the configurateEach blocks. configureEach is already lazy and doesn't need an outer lifecycle hook. The wrapper was registering 18 identical callbacks (one per subproject) for no effect.
  • There is a behavioral change with this. The root's compiler-arg/test-logging settings now register earlier relative to subproject configuration. A subproject setting the same property would now win where root previously won. There are currently no subprojects currently setting these (only worldedit-core:test.maxHeapSize, untouched by this block).
• Removed tasks.compileJava { dependsOn(":worldedit-libs:build") }. :worldedit-libs:build is a lifecycle task, not a jar — depending on it pulled in the full build/check/sourcesJar/javadocJar of all four worldedit-libs:* projects, including bukkit and cli, which worldedit-core does not use. The existingapi(project(":worldedit-libs:core")) / annotationProcessor(project(":worldedit-libs:core:ap")) dependencies already carry correct task ordering via Gradle's variant-aware resolution (outgoing.artifact(tasks.named("jar")) inbuildlogic.libs.gradle.kts), so the explicit dependsOn was redundant as well as too broad. It also broke configureondemand for any single-project build:

  > ./gradlew :worldedit-core:compileJava
  Task with name 'build' not found in project ':worldedit-libs:bukkit'
  

• processResources: moved the fawe.properties expansion values into a block-local val, since a top-level val would capture the script object, which the configuration cache can't serialize.
• configurations.all { } -> configurations.configureEach { }. Same resolutionStrategy, applied lazily instead of eagerly realizing every configuration in all 18 projects at configuration time.

Submitter Checklist

• Ensure that the pull request title represents the desired changelog entry.
• New public fields and methods are annotated with @since TODO.
• I read and followed the contribution guidelines.

[Copilot]

Pull request overview

This PR updates the Gradle build logic to improve configuration-time performance and make the build compatible with Gradle’s configuration cache by removing legacy/eager configuration patterns and replacing Grgit-based Git metadata reads.

Changes:

• Replaced Grgit usage with providers.exec Git CLI calls for revision/date metadata to support configuration cache.
• Removed an overly broad compileJava dependency on :worldedit-libs:build and adjusted resource expansion inputs to be configuration-cache safe.
• Switched eager configuration hooks (projectsEvaluated, configurations.all) to lazy equivalents (configureEach) to reduce configuration overhead.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File	Description
worldedit-core/build.gradle.kts	Removes redundant task dependency and adjusts processResources expansion to avoid configuration-cache serialization issues.
gradle/libs.versions.toml	Removes the Grgit version-catalog entry to drop the legacy dependency.
build.gradle.kts	Replaces Grgit-based Git metadata with providers.exec to support configuration cache and reduce configuration-time work.
build-logic/src/main/kotlin/buildlogic.common.gradle.kts	Makes resolution strategy configuration lazy by using configurations.configureEach.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 5 changed files in this pull request and generated no new comments.