Feat/add addable sponge config serializers

[ammodev]

Warten bis @TheBjoRedCraft updated hat, bevor merge.

https://i.imgur.com/ZeMLBRs.png

[Copilot]

Pull request overview

Introduces a platform-extensible way to register Sponge Configurate serializers by moving SpongeConfigSerializers to a ServiceLoader-provided service and adding platform-specific providers (Paper/Velocity).

Changes:

• Reworked SpongeConfigSerializers (core-api) into an abstract, registry-based serializer provider loaded via requiredService().
• Updated SpongeConfigManager to use the service-provided serializer module when building Configurate loaders.
• Added Paper/Velocity @AutoService implementations (including a Bukkit ItemStack serializer).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
surf-api-velocity/.../VelocitySpongeConfigSerializers.kt	Adds Velocity ServiceLoader provider for SpongeConfigSerializers.
surf-api-core/.../SpongeConfigSerializers.kt	Refactors serializers into a service-loaded abstract registry with registration APIs.
surf-api-core/.../SpongeConfigManager.kt	Switches loader options to use service-provided serializer module.
surf-api-core-api/api/surf-api-core-api.api	Updates API surface to reflect new SpongeConfigSerializers shape and new accessor.
surf-api-bukkit/.../PaperSpongeConfigSerializers.kt	Adds Paper provider and ItemStack Base64 serializer.
surf-api-bukkit/surf-api-bukkit-server/build.gradle.kts	Moves generatePaperPluginDescription block (no functional change expected).
surf-api-bukkit-api/api/surf-api-bukkit-api.api	API dump reflects removal of legacy inventory GUI APIs (large, unrelated surface change).

[Copilot]

@AutoService providers are instantiated via ServiceLoader, but Kotlin object singletons have a private constructor. This will cause ServiceLoader to fail to instantiate the provider at runtime (ServiceConfigurationError). Convert this to a regular class with a public no-arg constructor (or provide a @JvmStatic provider factory pattern) so the service can be loaded.

Suggested change

	object VelocitySpongeConfigSerializers : SpongeConfigSerializers()
	class VelocitySpongeConfigSerializers : SpongeConfigSerializers()

[Copilot]

SpongeConfigManager now pulls serializers from surfSpongeConfigSerializers (a required ServiceLoader service). There is currently no SpongeConfigSerializers provider in the standalone artifact (only Bukkit/Velocity providers exist), so standalone usage of SpongeConfigManager will throw ServiceConfigurationError. Add a @AutoService(SpongeConfigSerializers::class) provider in surf-api-standalone (or provide a safe default/fallback implementation) to avoid breaking standalone configs.

[Copilot]

New public API val surfSpongeConfigSerializers = requiredService<SpongeConfigSerializers>() is missing KDoc. Other top-level surf* service accessors in core-api include KDoc (e.g. surfCoreApi, surfConfigApi), and this should too to keep public API documentation consistent.

[Copilot]

SpongeConfigSerializers exposes several new public API members (typeTokenSerializers, classSerializers, register*/unregister*) without KDoc. Given this is in core-api and will be consumed externally, add KDoc for these members to clarify intended usage (especially around when/how it is safe to mutate the registries).

[Copilot]

ItemStackSerializer.deserialize can throw IllegalArgumentException (invalid Base64) and whatever ItemStack.deserializeBytes throws (invalid payload). Those exceptions will currently bubble up without node context. Catch and wrap them in a SerializationException (ideally including the node/path) so configuration errors are reported as proper Configurate serialization failures.

Suggested change

	?: throw SerializationException("Expected a Base64 string for ItemStack deserialization")
	val decoded = Base64.getDecoder().decode(itemStackBase64)
	return ItemStack.deserializeBytes(decoded)
	?: throw SerializationException(
	node,
	type,
	"Expected a Base64 string for ItemStack deserialization"
	)
	try {
	val decoded = Base64.getDecoder().decode(itemStackBase64)
	return ItemStack.deserializeBytes(decoded)
	} catch (e: SerializationException) {
	throw e
	} catch (e: Exception) {
	throw SerializationException(
	node,
	type,
	"Failed to deserialize ItemStack from Base64",
	e
	)
	}

[Copilot]

This API surface change removes multiple Bukkit inventory GUI-related public types (SinglePlayerGui, SurfGui, etc.), which looks unrelated to this PR’s stated purpose (Sponge config serializers). If intentional, it should be called out explicitly in the PR description/changelog (breaking change); if not intentional, the API dump likely needs to be regenerated/fixed so it matches the intended public API.

[Copilot]

unregisterSerializer(clazz) is ambiguous now that there are two registries (class-based and TypeToken-based). For symmetry with registerClassSerializer(...) and clarity for API consumers, consider renaming this to unregisterClassSerializer(...) (and keeping unregisterTypeTokenSerializer(...) as-is).

[Copilot]

Changing SpongeConfigSerializers from a Kotlin object with mutable SERIALIZERS to an abstract class loaded via requiredService() is a breaking public API change (source + binary). Make sure this is intentional and that downstream consumers have a clear migration path (e.g., release notes and version bump consistent with your compatibility policy).