Fix BungeeCord hex colour format not recognised in legacyToMiniMessage

translateColorCodes serialises &#RRGGBB to §x§R§R§G§G§B§B (BungeeCord
format) via LegacyComponentSerializer.  When that string was subsequently
passed to parseMiniMessageOrLegacy → legacyToMiniMessage, the &x prefix
was silently ignored and each following &R digit was misread as a named
colour code (&2 = dark_green, &3 = dark_aqua, …), producing completely
wrong colours.

Fix: after normalising § to &, detect the &x&R&R&G&G&B&B pattern in
both legacyToMiniMessage and replaceLegacyCodesInline and convert it to
the &#RRGGBB form that HEX_PATTERN already handles.

Fixes #2943

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: tastybento

src/main/java/world/bentobox/bentobox/util/Util.java

@@ -98,6 +98,13 @@ public class Util {
      */
     private static final Pattern LEGACY_HEX_CODE_PATTERN = Pattern.compile("&#[0-9a-fA-F]{3,6}|\u00A7x(\u00A7[0-9a-fA-F]){6}");
 
+    /**
+     * Pattern to match the BungeeCord/Spigot {@code &x&R&R&G&G&B&B} hex format
+     * (after {@code §} has been normalised to {@code &}).
+     * Produced by {@link LegacyComponentSerializer} with {@code useUnusualXRepeatedCharacterHexFormat()}.
+     */
+    private static final Pattern BUNGEE_HEX_PATTERN = Pattern.compile("&x(&[0-9a-fA-F]){6}");
+
     /**
      * MiniMessage instance for parsing MiniMessage-formatted strings.
      */
@@ -1047,6 +1054,23 @@ public static String legacyToMiniMessage(@NonNull String legacy) {
         // First, normalize § to & for uniform processing
         String text = legacy.replace('\u00A7', '&');
 
+        // Convert BungeeCord/Spigot §x§R§R§G§G§B§B hex format (now &x&R&R...) to &#RRGGBB
+        // so the HEX_PATTERN step below handles all hex input uniformly.
+        // This format is emitted by LegacyComponentSerializer.useUnusualXRepeatedCharacterHexFormat()
+        // and must be re-parsed here because legacyToMiniMessage does not recognise &x sequences.
+        Matcher bungeeMatcher = BUNGEE_HEX_PATTERN.matcher(text);
+        if (bungeeMatcher.find()) {
+            StringBuilder bungeeBuffer = new StringBuilder();
+            bungeeMatcher.reset();
+            while (bungeeMatcher.find()) {
+                // "&x&2&3&8&a&f&0" → strip "&x" prefix and remaining "&" chars → "238af0"
+                String digits = bungeeMatcher.group(0).substring(2).replace("&", "");
+                bungeeMatcher.appendReplacement(bungeeBuffer, "&#" + digits);
+            }
+            bungeeMatcher.appendTail(bungeeBuffer);
+            text = bungeeBuffer.toString();
+        }
+
         // Convert hex codes &#RRGGBB → <color:#RRGGBB>
         Matcher hexMatcher = HEX_PATTERN.matcher(text);
         StringBuilder sb = new StringBuilder();
@@ -1211,6 +1235,18 @@ public static String legacyToMiniMessage(@NonNull String legacy) {
     public static String replaceLegacyCodesInline(@NonNull String text) {
         // Normalize § to &
         text = text.replace('\u00A7', '&');
+        // Convert BungeeCord/Spigot &x&R&R&G&G&B&B hex format to &#RRGGBB (see legacyToMiniMessage)
+        Matcher bungeeMatcher = BUNGEE_HEX_PATTERN.matcher(text);
+        if (bungeeMatcher.find()) {
+            StringBuilder bungeeBuffer = new StringBuilder();
+            bungeeMatcher.reset();
+            while (bungeeMatcher.find()) {
+                String digits = bungeeMatcher.group(0).substring(2).replace("&", "");
+                bungeeMatcher.appendReplacement(bungeeBuffer, "&#" + digits);
+            }
+            bungeeMatcher.appendTail(bungeeBuffer);
+            text = bungeeBuffer.toString();
+        }
         // Replace hex codes &#RRGGBB → <color:#RRGGBB>
         Matcher hexMatcher = HEX_PATTERN.matcher(text);
         StringBuilder sb = new StringBuilder();

src/test/java/world/bentobox/bentobox/util/LegacyToMiniMessageTest.java

@@ -2,6 +2,7 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import net.kyori.adventure.text.Component;
@@ -285,4 +286,45 @@ void testStripSpaceAfterColorCodesRespectsBoundary() {
         // Reset must NOT strip
         assertEquals("\u00A7r world", Util.stripSpaceAfterColorCodes("\u00A7r world"));
     }
+
+    /**
+     * Regression for <a href="https://github.com/BentoBoxWorld/BentoBox/issues/2943">BentoBox#2943</a>:
+     * hex colors using the {@code &#RRGGBB} format were broken because
+     * {@code translateColorCodes} serialises them to the BungeeCord
+     * {@code §x§R§R§G§G§B§B} format, which {@code legacyToMiniMessage} (and
+     * {@code replaceLegacyCodesInline}) did not recognise.  The colour was then
+     * corrupted to a sequence of named colours (&amp;2, &amp;3, …) instead of
+     * the intended hex value.
+     */
+    @Test
+    void testBungeeCordHexFormatRoundTrip() {
+        // Simulate the full path: user writes &#238af0&l in a locale file.
+        // convertToLegacy (pure-legacy path) calls translateColorCodes which
+        // serialises to §x§2§3§8§a§f§0§l.  sendRawMessage then calls
+        // parseMiniMessageOrLegacy which must reconstruct the original colour.
+        String bungeeHex = "\u00A7x\u00A72\u00A73\u00A78\u00A7a\u00A7f\u00A70\u00A7l";
+        Component component = Util.parseMiniMessageOrLegacy(bungeeHex + "test");
+        // The component must carry the original hex colour, not a named-colour approximation.
+        net.kyori.adventure.text.format.TextColor color = component.children().isEmpty()
+                ? component.color()
+                : component.children().get(0).color();
+        assertNotNull(color, "Expected a colour on the component");
+        assertEquals(0x238af0, color.value(), "Hex colour #238af0 must survive the BungeeCord round-trip");
+    }
+
+    /**
+     * {@code &#RRGGBB&l} (the raw user-written format) must also round-trip correctly
+     * through {@code legacyToMiniMessage}.
+     */
+    @Test
+    void testRawHexWithBoldRoundTrip() {
+        String mm = Util.legacyToMiniMessage("&#238af0&lBold text");
+        Component component = Util.parseMiniMessage(mm);
+        // Must have the correct hex colour
+        net.kyori.adventure.text.format.TextColor color = component.color() != null
+                ? component.color()
+                : component.children().isEmpty() ? null : component.children().get(0).color();
+        assertNotNull(color, "Expected a colour");
+        assertEquals(0x238af0, color.value());
+    }
 }