improve more wrappers, remove orphans

Author: dmulloy2

src/main/java/net/dmulloy2/protocol/wrappers/configuration/clientbound/WrappedClientboundClearDialogPacket.java

@@ -18,4 +18,10 @@ public WrappedClientboundRegistryDataPacket() {
     public WrappedClientboundRegistryDataPacket(PacketContainer packet) {
         super(packet, TYPE);
     }
+
+    // TODO: missing field 'registry' (NMS type: ResourceKey<? extends Registry<?>>)
+    //   Use getMinecraftKeys().read(0) for the registry key identifier, or getSpecificModifier(ResourceKey.class).read(0).
+    // TODO: missing field 'entries' (NMS type: List<RegistrySynchronization.PackedRegistryEntry>)
+    //   Each entry has an id (Identifier) and an optional NBT tag. Use handle.getModifier().read(1)
+    //   for the raw List, or add a dedicated WrappedPackedRegistryEntry class.
 }

src/main/java/net/dmulloy2/protocol/wrappers/configuration/clientbound/WrappedClientboundFinishConfigurationPacket.java

@@ -19,4 +19,8 @@ public WrappedClientboundSelectKnownPacksPacket() {
     public WrappedClientboundSelectKnownPacksPacket(PacketContainer packet) {
         super(packet, TYPE);
     }
+
+    // TODO: missing field 'knownPacks' (NMS type: List<KnownPack> — record with namespace, id, version Strings)
+    //   Use AutoWrapper or a dedicated WrappedKnownPack class; each KnownPack is a simple 3-String record.
+    //   Alternatively, handle.getModifier().read(0) returns the raw List<KnownPack>.
 }

src/main/java/net/dmulloy2/protocol/wrappers/configuration/clientbound/WrappedClientboundRegistryDataPacket.java

@@ -18,4 +18,8 @@ public WrappedClientboundServerLinksPacket() {
     public WrappedClientboundServerLinksPacket(PacketContainer packet) {
         super(packet, TYPE);
     }
+
+    // TODO: missing field 'links' (NMS type: List<ServerLinks.UntrustedEntry>)
+    //   Same as the game-phase ServerLinks packet. No ProtocolLib accessor exists.
+    //   Use handle.getModifier().read(0) for the raw List.
 }

src/main/java/net/dmulloy2/protocol/wrappers/configuration/clientbound/WrappedClientboundResetChatPacket.java

@@ -19,4 +19,7 @@ public WrappedClientboundShowDialogPacket() {
     public WrappedClientboundShowDialogPacket(PacketContainer packet) {
         super(packet, TYPE);
     }
+
+    // TODO: missing field 'dialog' (NMS type: Holder<Dialog> — registry-backed dialog definition)
+    //   Same as the game-phase ShowDialog packet. Use handle.getModifier().read(0) for the raw Holder<Dialog>.
 }

src/main/java/net/dmulloy2/protocol/wrappers/configuration/clientbound/WrappedClientboundSelectKnownPacksPacket.java

@@ -18,4 +18,8 @@ public WrappedClientboundUpdateTagsPacket() {
     public WrappedClientboundUpdateTagsPacket(PacketContainer packet) {
         super(packet, TYPE);
     }
+
+    // TODO: missing field 'tags' (NMS type: Map<ResourceKey<? extends Registry<?>>, TagNetworkSerialization.NetworkPayload>)
+    //   Same as the game-phase UpdateTags packet. No ProtocolLib accessor exists.
+    //   Use handle.getModifier().read(0) to get the raw Map.
 }

src/main/java/net/dmulloy2/protocol/wrappers/configuration/clientbound/WrappedClientboundServerLinksPacket.java

@@ -18,4 +18,11 @@ public WrappedServerboundClientInformationPacket() {
     public WrappedServerboundClientInformationPacket(PacketContainer packet) {
         super(packet, TYPE);
     }
+
+    // TODO: missing field 'information' (NMS type: ClientInformation — record with locale, viewDistance, chatMode, etc.)
+    //   ClientInformation holds: String locale, int viewDistance, ChatVisiblity chatVisibility,
+    //   boolean chatColors, int modelCustomisation, HumanoidArm mainHand, boolean textFilteringEnabled,
+    //   boolean allowsListing, ParticleStatus particleStatus.
+    //   Use AutoWrapper with a dedicated WrappedClientInformation POJO, or expose individual fields
+    //   via getStrings().read(0), getIntegers().read(0), etc. in declaration order.
 }