Java 1.21.3 Support

[GroobleDierne]

Fixes #929, #928

TODO:

• Protocol
  • packet_recipe_book_add
  • packet_craft_recipe_response should be similar to previous one
  • Fix Slot & SlotComponent types
  • packet_declare_recipes
• entities.json (waiting on minecraft-data-generator)
• tints.json is likely incorrect

[rom1504]

Can you point missing files to 1.21.1 for now

[rom1504]

Let's open an nmp and mineflayer PRs using this

[rom1504]

nmp and mineflayer PRs

PrismarineJS/node-minecraft-protocol#1347
PrismarineJS/mineflayer#3489

fail due to wrong protocol as expected

[rom1504]

see https://github.com/extremeheat/extracted_minecraft_data/pulls as a source of info for the protocol

[rom1504]

https://github.com/extremeheat/extracted_minecraft_data/pull/13.patch

[GroobleDierne]

see https://github.com/extremeheat/extracted_minecraft_data/pulls as a source of info for the protocol

I'm using yarn because some files failed to decompile in extremheat's repository.
Anyway, I'll stop here for tonight.

[rom1504]

https://github.com/PrismarineJS/node-minecraft-protocol/actions/runs/11543601911/job/32356083688 now passing but still some packet errors

[extremeheat]

Please make sure to add up to date # MC: ClassName anchors to the top of all packet defs, this makes it easy to jump between the vanilla implementation and the protodef defs

[extremeheat]

I did some work on the protocol, still some work to do:

• Recipe changes (recipe_book_add/remove/etc.) not yet handled
• SlotComponent type needs fixes, likely back to 1.20.5
• Backport some fixes to 1.20.5
• Added new registryEntryHolder and registryEntryHolderSet types to correspond to the Holder and HolderSet types inside minecraft (ref) to replace usage of the arrayWithLengthOffset type
  • holderSet work by writing a varint to the stream, n, and 0 means "read one [string] entry" and non-zero n, means read an array of 1+n entries of data from the stream
  • holder work by writing a varint to the stream, n, and 0 means "read one [varint] entry" set to n+1 and non-zero means read an array of n entries of data from the stream
  • usage example (not yet implemented) to write a varint, n, and then either a varint or EntityMetadataWolfVariant:

 if wolf_variant: ["registryEntryHolder", {
      "base": { name: "variantId", type: "varint" },
      "otherwise": { name: "variantData", type: "EntityMetadataWolfVariant" }
 }]

• using the bitflags type implemented in bedrock-protocol to make the flags easier than tracking individual bits

["bitflags", {
      "type": "u32",
      "flags": ["x", "y", "z", "yaw", "pitch", "dx", "dy", "dz", "yawDelta"]
   }]

[GroobleDierne]

This name doesn't make sense anymore since this packet is sent even if the player isn't in a vehicle

[MrBruz]

@GroobleDierne For 'registryDataIsMandatory', 'version' needs to be changed to 'versions'

[GroobleDierne]

@GroobleDierne For 'registryDataIsMandatory', 'version' needs to be changed to 'versions'

Thx

[GroobleDierne]

Now only recipes related packets seem to be missing for protocol support

[GroobleDierne]

I'll try debugging them later today