Prevent macros from being added to the chat history

closes #5287

Author: Wide-Cat

src/main/java/meteordevelopment/meteorclient/systems/macros/Macro.java

@@ -60,26 +60,26 @@ public boolean onAction(boolean isKey, int value, int modifiers) {
     }
 
     public boolean onAction() {
-            if (dirty) {
-                scripts.clear();
+        if (dirty) {
+            scripts.clear();
 
-                for (String message : messages.get()) {
-                    Script script = MeteorStarscript.compile(message);
-                    if (script != null) scripts.add(script);
-                }
-
-                dirty = false;
+            for (String message : messages.get()) {
+                Script script = MeteorStarscript.compile(message);
+                if (script != null) scripts.add(script);
             }
 
-            for (Script script : scripts) {
-                String message = MeteorStarscript.run(script);
+            dirty = false;
+        }
+
+        for (Script script : scripts) {
+            String message = MeteorStarscript.run(script);
 
-                if (message != null) {
-                    ChatUtils.sendPlayerMsg(message);
-                }
+            if (message != null) {
+                ChatUtils.sendPlayerMsg(message, false);
             }
+        }
 
-            return true;
+        return true;
     }
 
     @Override

src/main/java/meteordevelopment/meteorclient/utils/player/ChatUtils.java

@@ -49,6 +49,7 @@ public static Text getMeteorPrefix() {
     /**
      * Registers a custom prefix to be used when calling from a class in the specified package. When null is returned from the supplier the default Meteor prefix is used.
      */
+    @SuppressWarnings("unused")
     public static void registerCustomPrefix(String packageName, Supplier<Text> supplier) {
         for (Pair<String, Supplier<Text>> pair : customPrefixes) {
             if (pair.getLeft().equals(packageName)) {
@@ -63,6 +64,7 @@ public static void registerCustomPrefix(String packageName, Supplier<Text> suppl
     /**
      * The package name must match exactly to the one provided through {@link #registerCustomPrefix(String, Supplier)}.
      */
+    @SuppressWarnings("unused")
     public static void unregisterCustomPrefix(String packageName) {
         customPrefixes.removeIf(pair -> pair.getLeft().equals(packageName));
     }
@@ -74,10 +76,17 @@ public static void forceNextPrefixClass(Class<?> klass) {
     // Player
 
     /**
-     * Sends the message as if the user typed it into chat.
+     * Sends the message as if the user typed it into chat and adds it to the chat history.
      */
     public static void sendPlayerMsg(String message) {
-        mc.inGameHud.getChatHud().addToMessageHistory(message);
+        sendPlayerMsg(message, true);
+    }
+
+    /**
+     * Sends the message as if the user typed it into chat.
+     */
+    public static void sendPlayerMsg(String message, boolean addToHistory) {
+        if (addToHistory) mc.inGameHud.getChatHud().addToMessageHistory(message);
 
         if (message.startsWith("/")) mc.player.networkHandler.sendChatCommand(message.substring(1));
         else mc.player.networkHandler.sendChatMessage(message);