chore(text-logger-slf4j): Refactor module to use modern language features

Also adds a missing `PlatformAPI` annotation and removes an unused method in CallerClassFinder.

Author: kezz

text-logger-slf4j/src/main/java/net/kyori/adventure/text/logger/slf4j/CallerClassFinder.java

@@ -28,13 +28,9 @@ private CallerClassFinder() {
   }
 
   static String callingClassName() {
-    return callingClassName(2); // this, plus the calling method
-  }
-
-  static String callingClassName(final int elementsToSkip) { // elementsToSkip not counting this method
     return StackWalker.getInstance().walk(stream -> stream.map(StackWalker.StackFrame::getClassName)
-        .skip(elementsToSkip + 1)
+        .skip(2) // Skip this method, plus the calling method.
         .findFirst())
-      .orElseThrow(() -> new IllegalArgumentException("Not enough stack elements to skip " + elementsToSkip + " elements"));
+      .orElseThrow(() -> new IllegalArgumentException("Not enough stack elements to skip 2 elements"));
   }
 }

text-logger-slf4j/src/main/java/net/kyori/adventure/text/logger/slf4j/ComponentLoggerProvider.java

@@ -25,6 +25,7 @@
 
 import java.util.function.Function;
 import net.kyori.adventure.text.Component;
+import net.kyori.adventure.util.PlatformAPI;
 import org.jetbrains.annotations.ApiStatus;
 import org.jetbrains.annotations.NotNull;
 import org.slf4j.Logger;
@@ -34,7 +35,8 @@
  *
  * @since 4.11.0
  */
-@ApiStatus.Internal // SPI for platform use only
+@ApiStatus.Internal
+@PlatformAPI
 public interface ComponentLoggerProvider {
   /**
    * Create a component logger for the provided logger name.
@@ -52,10 +54,11 @@ public interface ComponentLoggerProvider {
    * @since 4.11.0
    */
   @ApiStatus.NonExtendable
+  @PlatformAPI
   interface LoggerHelper {
 
     /**
-     * Create a serializer function that will translate logged output into the system default locale, and then serialize it to plain text.
+     * Create a serializer function that will translate logged output into the system default locale and then serialize it to plain text.
      *
      * @return a plain serializer
      * @since 4.11.0

text-logger-slf4j/src/main/java/net/kyori/adventure/text/logger/slf4j/UnpackedComponentThrowable.java

@@ -23,6 +23,7 @@
  */
 package net.kyori.adventure.text.logger.slf4j;
 
+import java.io.Serial;
 import java.util.function.Function;
 import net.kyori.adventure.text.Component;
 import net.kyori.adventure.util.ComponentMessageThrowable;
@@ -33,7 +34,7 @@
  */
 @SuppressWarnings("OverrideThrowableToString")
 final class UnpackedComponentThrowable extends Throwable {
-  private static final long serialVersionUID = -1L;
+  @Serial private static final long serialVersionUID = -1L;
 
   private final Class<? extends Throwable> backingType;
 
@@ -46,10 +47,8 @@ static Throwable unpack(final Throwable maybeRich, final Function<Component, Str
 
     final UnpackedComponentThrowable ret = new UnpackedComponentThrowable(maybeRich.getClass(), serializer.apply(message), cause);
     ret.setStackTrace(maybeRich.getStackTrace());
-    if (suppressed.length > 0) {
-      for (int i = 0; i < suppressed.length; i++) {
-        ret.addSuppressed(unpack(suppressed[i], serializer));
-      }
+    for (final Throwable throwable : suppressed) {
+      ret.addSuppressed(unpack(throwable, serializer));
     }
 
     return ret;