fix: migrate pin feature + e2e fixtures to BanManager v8 placeholder syntax

BanManager v8 dropped legacy [token] placeholders in favour of MiniMessage
<token> tags, and rejects messages containing legacy & colour codes outright.
Without this migration the pin feature silently no-ops and every e2e ban /
tempban kick test sees the unsubstituted "<pin>" template (or, after rejection,
the bundled defaults that have no pin placeholder at all).

- CommonPlayerDeniedListener: detect <pin> via Message.getRawTemplate() (the
  resolved string is a Component-backed legacy render and never contains the
  raw template token after v8). Bail when the template has no <pin>.
- Bundled common/src/main/resources/messages.yml: rewrite pin.notify /
  pin.pin / pin.rateLimited in MiniMessage format with <pin>, <expires>,
  <seconds>.
- e2e fixtures (bukkit / bungee / fabric / sponge / velocity):
    - banmanager-webenhancer/messages.yml: same MiniMessage rewrite.
    - banmanager/messages.yml: shrink from a full clone of the legacy
      messages.yml to a minimal MiniMessage override of just ban / tempban /
      banip / tempbanip kick + disallowed templates so the WebEnhancer pin
      feature has a <pin> token to substitute. Everything else falls through
      to BanManager v8's bundled defaults.
- Update CommonPlayerDeniedListenerTest to mock getRawTemplate() and add a
  null-template case.

Author: confuser

common/src/main/java/me/confuser/banmanager/webenhancer/common/listeners/CommonPlayerDeniedListener.java

@@ -13,11 +13,11 @@ public CommonPlayerDeniedListener(WebEnhancerPlugin plugin) {
   }
 
   public void handlePin(PlayerData player, Message message) {
-    if (!message.toString().contains("[pin]")) return;
+    String template = message.getRawTemplate();
+    if (template == null || !template.contains("<pin>")) return;
 
     PlayerPinData pin = plugin.getPlayerPinStorage().getValidPin(player);
 
-
     if (pin != null) {
       message.set("pin", String.valueOf(pin.getGeneratedPin()));
     }

common/src/main/resources/messages.yml

@@ -1,5 +1,8 @@
+# Message templates use MiniMessage formatting: https://docs.advntr.dev/minimessage/format.html
+# Placeholders use angle brackets, e.g. <pin>, <expires>, <seconds>
+
 messages:
   pin:
-    notify: '&6Your pin expires in [expires]'
-    pin: '[pin]'
-    rateLimited: '&cPlease wait [seconds] seconds before generating a new pin'
+    notify: '<gold>Your pin expires in <expires>'
+    pin: '<pin>'
+    rateLimited: '<red>Please wait <seconds> seconds before generating a new pin'

common/src/test/java/me/confuser/banmanager/webenhancer/common/listeners/CommonPlayerDeniedListenerTest.java

@@ -39,13 +39,11 @@ public void setUp() {
 
     @Test
     public void handlePin_replacesPlaceholderWithPin() {
-        // Setup
         when(playerPinStorage.getValidPin(player)).thenReturn(pinData);
         when(pinData.getGeneratedPin()).thenReturn(123456);
 
-        // Create a mocked message with [pin] placeholder
         Message message = mock(Message.class);
-        when(message.toString()).thenReturn("Your login pin is: [pin]");
+        when(message.getRawTemplate()).thenReturn("Your login pin is: <pin>");
 
         listener.handlePin(player, message);
 
@@ -54,9 +52,19 @@ public void handlePin_replacesPlaceholderWithPin() {
 
     @Test
     public void handlePin_ignoresMessagesWithoutPlaceholder() {
-        // Create a mocked message without [pin] placeholder
         Message message = mock(Message.class);
-        when(message.toString()).thenReturn("You have been banned!");
+        when(message.getRawTemplate()).thenReturn("You have been banned!");
+
+        listener.handlePin(player, message);
+
+        verify(playerPinStorage, never()).getValidPin(any());
+        verify(message, never()).set(anyString(), anyString());
+    }
+
+    @Test
+    public void handlePin_ignoresMessagesWithNullTemplate() {
+        Message message = mock(Message.class);
+        when(message.getRawTemplate()).thenReturn(null);
 
         listener.handlePin(player, message);
 
@@ -66,12 +74,10 @@ public void handlePin_ignoresMessagesWithoutPlaceholder() {
 
     @Test
     public void handlePin_handlesNullPinGracefully() {
-        // Setup - getValidPin returns null
         when(playerPinStorage.getValidPin(player)).thenReturn(null);
 
-        // Create a mocked message with [pin] placeholder
         Message message = mock(Message.class);
-        when(message.toString()).thenReturn("Your login pin is: [pin]");
+        when(message.getRawTemplate()).thenReturn("Your login pin is: <pin>");
 
         listener.handlePin(player, message);
 

e2e/platforms/bukkit/configs/banmanager-webenhancer/messages.yml

@@ -1,5 +1,5 @@
 messages:
   pin:
-    notify: '&6Your pin expires in [expires]'
-    pin: '[pin]'
-
+    notify: '<gold>Your pin expires in <expires>'
+    pin: '<pin>'
+    rateLimited: '<red>Please wait <seconds> seconds before generating a new pin'