Skip to content

Fix compatibility with Minecraft 26.1.2#2

Open
GW228g wants to merge 1 commit into
JuliGame:masterfrom
GW228g:fix/mc-26-1-2-compatibility
Open

Fix compatibility with Minecraft 26.1.2#2
GW228g wants to merge 1 commit into
JuliGame:masterfrom
GW228g:fix/mc-26-1-2-compatibility

Conversation

@GW228g
Copy link
Copy Markdown

@GW228g GW228g commented May 11, 2026

Summary

  • Recipe.removeRecipe(): The old code compared full ItemStack objects (including custom display name, lore, and persistent data) against plain vanilla recipe results — they never matched, causing 30+ "Item to remove not found" log spam on every startup. Fixed by comparing Material type only, which is the correct intent.
  • pom.xml: Updated Paper API dependency to 26.1.2.build.63-stable, bumped Java source/target to 21 (required by Paper 26.1.2), upgraded maven-shade-plugin from 3.2.4 to 3.6.0 (3.2.4 cannot process Java 21 class files), updated ProtocolLib to 5.4.0-SNAPSHOT (5.0.0 is no longer in any public repo), removed hardcoded Windows output path.
  • plugin.yml: Updated api-version from 1.21 to 26.1 to match the new Minecraft versioning scheme.

Test plan

  • Plugin loads without errors on Paper 26.1.2 build 63
  • No "Item to remove not found" errors on startup
  • Custom foods load and apply effects correctly
  • Crafting recipes for custom foods work in-game (needs player testing)

Tested on macOS with Java 25.0.3 and Paper 26.1.2 build 63.

- Recipe.removeRecipe(): compare by Material type instead of full ItemStack equality — old comparison never matched because recipe results are plain ItemStacks while ours carry custom meta/lore, causing 30+ startup errors
- pom.xml: update Paper API to 26.1.2.build.63-stable, Java to 21, maven-shade-plugin to 3.6.0, ProtocolLib to 5.4.0-SNAPSHOT, remove hardcoded Windows output path
- plugin.yml: update api-version from 1.21 to 26.1

Tested on Paper 26.1.2 build 63 with Java 25.
@GW228g
Copy link
Copy Markdown
Author

GW228g commented May 11, 2026

@coderabbitai review

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.

1 participant