fix: minecraft velocity name with multiple layers of color encoding (#…

…595)

* fix: minecraft velocity recursive name

* chore: update CHANGELOG.md

* feat: remove obvious (?) condition

* feat: simplify using javascript code moment

CHANGELOG.md

@@ -1,7 +1,7 @@
 
 ## To Be Released...
-## 5.0.0
-* To be made...
+## 5.X.Y
+* Fix the `name` field on Minecraft servers running Velocity with multiple layers of color encoding (#595)
 
 ## 5.1.0
 * FOUNDRY - Added support (#585)

protocols/minecraft.js

@@ -65,7 +65,19 @@ export default class minecraft extends Core {
           name = description.text
         }
         if (!name && typeof description === 'object' && description.extra) {
-          name = description.extra.map(part => part.text).join('')
+          let stack = [description];
+
+          while (stack.length) {
+            let current = stack.pop();
+
+            if (current.text) {
+              name += current.text;
+            }
+
+            if (Array.isArray(current.extra)) {
+              stack.push(...current.extra.reverse());
+            }
+          }
         }
         state.name = name
       } catch (e) {}