Implement new packets, extend API for logging, add config

[MesterMan03]

This PR implements new packet handlers, massively overhauls the PacketInfoBundle API without breaking compatibility and adds configuration via Cloth Config and Mod Menu (also re-enables the previously disabled packet delay feature that triggered the config TODO).

Packets implemented:

• CommandSuggestionsResponse/Request
• PlayerInfoRemove/Update
• SetCursorItem

Overhauled PacketInfoBundle:

• PacketInfoBundle now accepts a list of PacketInfoRow objects using PacketInfoBundle.ofRows(PacketType type, Component name, List<? extends PacketInfoRow> rows).
• PacketInfoBundle.of(PacketType type, Component name, List<PacketInfoSegment> segments) is kept as-is for backwards compatibility, but it internally uses the ofRows constructor.
• 2 new elements are added:
  • PacketInfoList for list-like data, it accepts a list of PacketInfoListElement objects (implemented by PacketInfoSegment and PacketInfoValue, so no list of lists yet)
  • PacketInfoValue for value-only data (best combined with a list, where you don't always want a key-value pair)
• PacketInfoRow is implemented by PacketInfoSegment, PacketInfoList and PacketInfoValue, meaning they can be freely mixed inside a PacketInfoBundle.

Some previous handlers were updated to reflect the changes in PacketInfoBundle:

• ContainerSetContent
• RemoveEntities
• SetPassengers
• SetPlayerTeam

Added configuration via Cloth Config + Mod Menu:

• Currently only one setting is supported: packetDelayMs, which is now enabled in PacketLogger.
• Configuration file is stored in config/packetninja.json.

Misc:

• Marked mod as explicitely client-only in fabric.mod.json

[MesterMan03]

@ImBit I'm still debugging a disconnect error in PlayerInfoUpdate, but I thought you might be interested already, so I opened a draft PR

lmk if you find anything

[MesterMan03]

I think this PR deserves a 0.1.0 version bump, but of course I'll let you decide that.

[ImBit]

Great, will have a look when i get the chance :)

[MesterMan03]

@ImBit with the move towards build.gradle.kts, is there also a Kotlin-rewrite of the mod in mind? I've also readded the reflections library to the build script as the mod wasn't compiling without it, not sure why it was removed during the switch.

Btw in my opinion the build workflow should be separated into a pure build and a publish workflow for automatic builds in PRs and commits, while keeping the publish as a manually activated workflow or for commits on master.

[ImBit]

@MesterMan03 my bad with removing reflections, I planned on adding a classpath util as a replacement, never got round to it, will fix and push asap! a ClassGraph utility has now been included!

I do not have intentions of a kotlin rewrite at the moment. I believe @BenTechDev was planning on migrating to Kotlin when they get the time, along with adding some SheepLib functionality.

Not too sure what you mean with the second part of the build workflow, feel free to draft up something if you feel it would fit.

[MesterMan03]

@ImBit right, so I basically renamed the old build.yml to publish.yml and created a new build.yml that only builds and uploads the artifacts to GitHub without publishing to Modrinth. It should already be working in this PR.

Anyway, unless you find something problematic, the PR should be ready to be merged.

[ImBit]

Mostly looks good, please work on the above changes, then I can make a release :)

[ImBit]

Happy with this, will bump to 0.1.0 and release :)