Skip to content

replace packet class names with packet type registries#6492

Merged
MineGame159 merged 3 commits into
MeteorDevelopment:masterfrom
crosby-moe:feat/refactor-packet-mappings
Jun 27, 2026
Merged

replace packet class names with packet type registries#6492
MineGame159 merged 3 commits into
MeteorDevelopment:masterfrom
crosby-moe:feat/refactor-packet-mappings

Conversation

@crosby-moe

Copy link
Copy Markdown
Collaborator

Type of change

  • Bug fix
  • New feature
  • Needed internal rewrite

Description

  • replace usages of Class<? extends Packet<?>> with PacketType<? extends Packet<?>>
  • use modern Clientbound & Serverbound naming conventions in api rather than the old C2S & S2C
  • fully remove PacketUtilsUtil

addons using PacketUtils or PacketListSetting will break, existing configs will not

migration is simple:

  • Class<? extends Packet<?>> -> PacketType<? extends Packet<?>>
  • PacketUtils#getName(packet) -> packet.type().toString()
  • PacketUtils#PACKETS -> PacketUtils#getPackets
  • PacketUtils#getS2CPackets -> PacketUtils#getClientboundPackets
  • PacketUtils#getC2SPackets -> PacketUtils#getServerboundPackets

Related issues

supersedes #4748

How Has This Been Tested?

the J

Checklist:

  • My code follows the style guidelines of this project.
  • I have added comments to my code in more complex areas.
  • I have tested the code in both development and production environments.

@MineGame159 MineGame159 merged commit ef3dc1e into MeteorDevelopment:master Jun 27, 2026
1 check passed
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