Skip to content

Fix fat jar build and update dependencies#146

Open
TheMeinerLP wants to merge 1 commit into
mainfrom
claude/titan-minestom-deployment-XhoNO
Open

Fix fat jar build and update dependencies#146
TheMeinerLP wants to merge 1 commit into
mainfrom
claude/titan-minestom-deployment-XhoNO

Conversation

@TheMeinerLP
Copy link
Copy Markdown
Contributor

Summary

This PR addresses issues with the fat jar build process and updates project dependencies to their latest versions.

Key Changes

  • Fat jar build fixes: Added exclusions to the jar task to handle LuckPerms' signed and multi-release jars that were breaking the relocation-free application fat jar:
    • Exclude signature files (META-INF/*.SF, META-INF/*.DSA, META-INF/*.RSA)
    • Exclude module-info classes (module-info.class, META-INF/versions/**/module-info.class)
    • Set duplicatesStrategy = DuplicatesStrategy.EXCLUDE to handle duplicate entries
  • Dependency updates:
    • Butterfly: 1.0.211.0.23
    • Minestom: Changed from unversioned to explicit version 2026.05.17-1.21.11
  • Workflow updates: Updated GitHub Actions workflow references to v2.1.0 for both build and publish workflows

Implementation Details

The fat jar exclusions specifically target artifacts pulled by LuckPerms' common module that contain signed JARs and multi-release JAR metadata. These were causing conflicts in the final application JAR. The duplicatesStrategy setting ensures consistent handling of any remaining duplicate entries during the merge process.

https://claude.ai/code/session_01S76fk9ma5Szkf9r1W44RVP

…n CI

- Pin net.minestom:minestom to 2026.05.17-1.21.11 (current Maven Central
  release) instead of the BOM-resolved version so the deployable jar runs
  on the latest Minestom regardless of aonyx/mycelium BOM lag.
- Bump butterfly 1.0.21 -> 1.0.23 to match the current Butterfly release.
- Harden the application shadowJar: strip jar signatures and module-info
  and exclude duplicates so the relocation-free fat jar built from the
  shaded LuckPerms fork stays loadable.
- Pin the reusable PR/publish workflows to the v2.1.0 release tag instead
  of a temporary branch ref.

https://claude.ai/code/session_01S76fk9ma5Szkf9r1W44RVP
@TheMeinerLP TheMeinerLP requested a review from a team as a code owner May 18, 2026 12:25
@github-actions
Copy link
Copy Markdown

Job Summary for Gradle

Build PR :: build
Gradle Root Project Requested Tasks Gradle Version Build Outcome Build Scan®
titan build test 9.5.0 Build Scan not published

2 similar comments
@github-actions
Copy link
Copy Markdown

Job Summary for Gradle

Build PR :: build
Gradle Root Project Requested Tasks Gradle Version Build Outcome Build Scan®
titan build test 9.5.0 Build Scan not published

@github-actions
Copy link
Copy Markdown

Job Summary for Gradle

Build PR :: build
Gradle Root Project Requested Tasks Gradle Version Build Outcome Build Scan®
titan build test 9.5.0 Build Scan not published

@github-actions
Copy link
Copy Markdown

Test results

12 files  12 suites   7s ⏱️
25 tests 25 ✅ 0 💤 0 ❌
50 runs  50 ✅ 0 💤 0 ❌

Results for commit d58cdd0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants