Switch to Google Java Format

buildSrc/src/main/kotlin/vision.java-conventions.gradle.kts

@@ -23,7 +23,7 @@ java {
 
 spotless {
     java {
-        palantirJavaFormat().style("GOOGLE")
+        googleJavaFormat().formatJavadoc(false)
         formatAnnotations()
     }
 }

examples/paper/src/main/java/me/sparky983/vision/examples/paper/ExampleGuiCommandExecutor.java

@@ -36,8 +36,8 @@ public boolean onCommand(
 
     final AtomicInteger count = new AtomicInteger(0);
 
-    final Button counter = Button.of(ItemType.DIAMOND)
-        .onClick((click) -> sender.sendMessage(click.button().name()));
+    final Button counter =
+        Button.of(ItemType.DIAMOND).onClick((click) -> sender.sendMessage(click.button().name()));
 
     scheduler.runTaskTimer(
         plugin,
@@ -47,11 +47,12 @@ public boolean onCommand(
         20,
         10);
 
-    final Gui gui = Gui.chest()
-        .title(Component.text("Paper Example"))
-        .rows(3)
-        .slot(Slot.of(1, 4), counter)
-        .build();
+    final Gui gui =
+        Gui.chest()
+            .title(Component.text("Paper Example"))
+            .rows(3)
+            .slot(Slot.of(1, 4), counter)
+            .build();
 
     vision.open(player, gui);
 

vision-api/src/main/java/me/sparky983/vision/ButtonImpl.java

@@ -131,12 +131,13 @@ public Button glow(final boolean glow) {
   public Button onClick(final Consumer<? super Click> handler) {
     Objects.requireNonNull(handler, "handler cannot be null");
 
-    subscribe(new Subscriber() {
-      @Override
-      public void click(final Click click) {
-        handler.accept(click);
-      }
-    });
+    subscribe(
+        new Subscriber() {
+          @Override
+          public void click(final Click click) {
+            handler.accept(click);
+          }
+        });
 
     return this;
   }

vision-api/src/main/java/me/sparky983/vision/Chest.java

@@ -30,8 +30,7 @@ public non-sealed interface Chest extends Gui {
    * @since 1.0
    * @vision.experimental because this may be deleted, renamed or moved.
    */
-  @ApiStatus.Experimental
-  int MIN_ROWS = 1;
+  @ApiStatus.Experimental int MIN_ROWS = 1;
 
   /**
    * The maximum number of rows that a {@code Chest} can have.
@@ -40,8 +39,7 @@ public non-sealed interface Chest extends Gui {
    * @since 1.0
    * @vision.experimental because this may be deleted, renamed or moved.
    */
-  @ApiStatus.Experimental
-  int MAX_ROWS = 6;
+  @ApiStatus.Experimental int MAX_ROWS = 6;
 
   /**
    * The number of columns {@code Chest}s have.
@@ -50,8 +48,7 @@ public non-sealed interface Chest extends Gui {
    * @since 1.0
    * @vision.experimental because this may be deleted, renamed or moved.
    */
-  @ApiStatus.Experimental
-  int COLUMNS = 9;
+  @ApiStatus.Experimental int COLUMNS = 9;
 
   /**
    * {@inheritDoc}

vision-api/src/main/java/me/sparky983/vision/ChestImpl.java

@@ -19,8 +19,7 @@ final class ChestImpl implements Chest {
   @VisibleForTesting
   static final Component DEFAULT_TITLE = Component.translatable("container.chest");
 
-  @VisibleForTesting
-  static final int DEFAULT_ROWS = 1;
+  @VisibleForTesting static final int DEFAULT_ROWS = 1;
 
   private final Container container;
 

vision-api/src/main/java/me/sparky983/vision/Container.java

@@ -133,10 +133,11 @@ void slot(final Slot slot, final @Nullable Button button) {
     } else {
       buttons.put(slot, button);
     }
-    subscribers.notify((subscriber) -> {
-      subscriber.button(slot, button); // Backwards compatibility
-      subscriber.slot(slot, button);
-    });
+    subscribers.notify(
+        (subscriber) -> {
+          subscriber.button(slot, button); // Backwards compatibility
+          subscriber.slot(slot, button);
+        });
   }
 
   List<Slot> slots() {
@@ -146,12 +147,13 @@ List<Slot> slots() {
   void onClose(final Consumer<? super Close> handler) {
     Objects.requireNonNull(handler, "handler cannot be null");
 
-    subscribers.subscribe(new Gui.Subscriber() {
-      @Override
-      public void close(final Close close) {
-        handler.accept(close);
-      }
-    });
+    subscribers.subscribe(
+        new Gui.Subscriber() {
+          @Override
+          public void close(final Close close) {
+            handler.accept(close);
+          }
+        });
   }
 
   Gui.Publisher publisher() {

vision-api/src/main/java/me/sparky983/vision/Dropper.java

@@ -29,8 +29,7 @@ public non-sealed interface Dropper extends Gui {
    * @since 1.0
    * @vision.experimental because this may be deleted, renamed or moved.
    */
-  @ApiStatus.Experimental
-  int ROWS = 3;
+  @ApiStatus.Experimental int ROWS = 3;
 
   /**
    * The number of columns {@code Dropper}s have.
@@ -39,8 +38,7 @@ public non-sealed interface Dropper extends Gui {
    * @since 1.0
    * @vision.experimental because this may be deleted, renamed or moved.
    */
-  @ApiStatus.Experimental
-  int COLUMNS = 3;
+  @ApiStatus.Experimental int COLUMNS = 3;
 
   /**
    * {@inheritDoc}

vision-api/src/main/java/me/sparky983/vision/Hopper.java

@@ -29,8 +29,7 @@ public non-sealed interface Hopper extends Gui {
    * @since 1.0
    * @vision.experimental because this may be deleted, renamed or moved.
    */
-  @ApiStatus.Experimental
-  int ROWS = 1;
+  @ApiStatus.Experimental int ROWS = 1;
 
   /**
    * The number of columns {@code Hopper}s have.
@@ -39,8 +38,7 @@ public non-sealed interface Hopper extends Gui {
    * @since 1.0
    * @vision.experimental because this may be deleted, renamed or moved.
    */
-  @ApiStatus.Experimental
-  int COLUMNS = 5;
+  @ApiStatus.Experimental int COLUMNS = 5;
 
   /**
    * {@inheritDoc}

vision-api/src/main/java/me/sparky983/vision/ItemType.java

@@ -8803,6 +8803,7 @@ public sealed interface ItemType extends Keyed, Translatable permits ItemTypeImp
    * @since 0.1
    */
   ItemType ZOMBIFIED_PIGLIN_SPAWN_EGG = item(Key.key("zombified_piglin_spawn_egg"));
+
   // @formatter:on
   // </editor-fold>
 

vision-api/src/main/java/me/sparky983/vision/Subscribers.java

@@ -18,21 +18,22 @@ final class Subscribers<T extends Subscribable.Subscriber> implements Subscribab
   public Subscription subscribe(final T subscriber) {
     Objects.requireNonNull(subscriber, "subscriber cannot be null");
 
-    final Subscription subscription = new Subscription() {
-      @Override
-      public void cancel() {
-        final Map<Subscription, T> copy = new HashMap<>(subscribers);
-        if (copy.remove(this) == null) {
-          throw new IllegalStateException("Subscription has already been cancelled");
-        }
-        subscribers = copy;
-      }
+    final Subscription subscription =
+        new Subscription() {
+          @Override
+          public void cancel() {
+            final Map<Subscription, T> copy = new HashMap<>(subscribers);
+            if (copy.remove(this) == null) {
+              throw new IllegalStateException("Subscription has already been cancelled");
+            }
+            subscribers = copy;
+          }
 
-      @Override
-      public boolean isCanceled() {
-        return !subscribers.containsKey(this);
-      }
-    };
+          @Override
+          public boolean isCanceled() {
+            return !subscribers.containsKey(this);
+          }
+        };
 
     final Map<Subscription, T> copy = new HashMap<>(subscribers);
     copy.put(subscription, subscriber);

vision-api/src/test/java/me/sparky983/vision/ButtonTests.java

@@ -160,8 +160,9 @@ void testVarargsLoreWhenLoreIsNull() {
   void testVarargsLoreWhenLoreContainsNull() {
     final Button button = Button.of(ItemType.STONE);
 
-    final Exception e = assertThrows(
-        NullPointerException.class, () -> button.lore(Arrays.asList(LORE_LINE_1, null)));
+    final Exception e =
+        assertThrows(
+            NullPointerException.class, () -> button.lore(Arrays.asList(LORE_LINE_1, null)));
     assertEquals("lore[1] cannot be null", e.getMessage());
 
     assertNotEquals(Arrays.asList(LORE_LINE_1, null), button.lore());
@@ -191,8 +192,9 @@ void testListLoreWhenLoreIsNull() {
   void testListLoreWhenLoreContainsNull() {
     final Button button = Button.of(ItemType.STONE);
 
-    final Exception e = assertThrows(
-        NullPointerException.class, () -> button.lore(Arrays.asList(LORE_LINE_1, null)));
+    final Exception e =
+        assertThrows(
+            NullPointerException.class, () -> button.lore(Arrays.asList(LORE_LINE_1, null)));
     assertEquals("lore[1] cannot be null", e.getMessage());
   }
 
@@ -367,8 +369,7 @@ void testCancelSubscription() {
 
   @Test
   void testToString() {
-    final Button button =
-        Button.of(ItemType.STONE).name(NAME).lore(LORE_LIST).amount(2).glow(true);
+    final Button button = Button.of(ItemType.STONE).name(NAME).lore(LORE_LIST).amount(2).glow(true);
 
     assertEquals(
         String.format(