Update workflow configuration

Upgrade Java version from 17 to 21 in workflow configuration

.github/workflows/build.yml

@@ -12,7 +12,7 @@ jobs:
       matrix:
         # Use these Java versions
         java: [
-          17,    # Current Java LTS & minimum supported by Minecraft
+          21,    # Current Java LTS & minimum supported by Minecraft
         ]
         # and run on both Linux and Windows
         os: [ubuntu-22.04, windows-2022]
@@ -33,7 +33,7 @@ jobs:
       - name: build
         run: ./gradlew build
       - name: capture build artifacts
-        if: ${{ runner.os == 'Linux' && matrix.java == '17' }} # Only upload artifacts built from latest java on one OS
+        if: ${{ runner.os == 'Linux' && matrix.java == '21' }} # Only upload artifacts built from latest java on one OS
         uses: actions/upload-artifact@v3
         with:
           name: Artifacts

README.md

@@ -2,8 +2,8 @@
  A Simple mod add bukkit style with an "&" and "[item]" display for Fabric Server.
  
  It's a **server-side** mod and doesn't need to be installed on the client.
- 
+
- Current Minecraft version: 1.20.1
+Currently supported Minecraft versions: 1.20.1, 1.20.4
 
 ## Screenshot
 ![image](https://github.com/CPTProgrammer/ChatPlus/assets/46586216/dcd2ca4b-79e9-4692-a8de-02fddfe4a392)

build.gradle

@@ -1,5 +1,5 @@
 plugins {
-	id 'fabric-loom' version '1.5-SNAPSHOT'
+	id 'fabric-loom' version '1.9-SNAPSHOT'
 	id 'maven-publish'
 }
 

gradle.properties

@@ -4,14 +4,14 @@ org.gradle.parallel=true
 
 # Fabric Properties
 # check these on https://fabricmc.net/develop
-minecraft_version=1.20.4
+minecraft_version=1.21.4
-yarn_mappings=1.20.4+build.3
+yarn_mappings=1.21.4+build.8
-loader_version=0.15.6
+loader_version=0.16.9
 
 # Mod Properties
-mod_version=0.20.2
+mod_version=0.21.0
 maven_group=cn.revaria.chatplus
 archives_base_name=chat-plus
 
 # Dependencies
-fabric_version=0.95.1+1.20.4
+fabric_version=0.115.0+1.21.4

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.5-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.12-bin.zip
 networkTimeout=10000
 validateDistributionUrl=true
 zipStoreBase=GRADLE_USER_HOME

src/main/java/cn/revaria/chatplus/ChatPlus.java

@@ -1,11 +1,6 @@
 package cn.revaria.chatplus;
 
 import net.fabricmc.api.ModInitializer;
-
-import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
-import net.minecraft.network.message.MessageType;
-import net.minecraft.network.message.SignedMessage;
-import net.minecraft.server.network.ServerPlayerEntity;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -13,7 +8,7 @@ public class ChatPlus implements ModInitializer {
 	// This logger is used to write text to the console and the log file.
 	// It is considered best practice to use your mod id as the logger's name.
 	// That way, it's clear which mod wrote info, warnings, and errors.
-    public static final Logger LOGGER = LoggerFactory.getLogger("chat-plus");
+    public static final Logger LOGGER = LoggerFactory.getLogger("Chat Plus");
 
 	@Override
 	public void onInitialize() {

src/main/java/cn/revaria/chatplus/mixin/MixinChat.java

@@ -1,22 +1,19 @@
 package cn.revaria.chatplus.mixin;
 
-import net.minecraft.SharedConstants;
+import net.minecraft.MinecraftVersion;
 import net.minecraft.item.ItemStack;
 import net.minecraft.network.ClientConnection;
-import net.minecraft.network.message.LastSeenMessageList;
+import net.minecraft.network.message.*;
-import net.minecraft.network.message.MessageChain;
-import net.minecraft.network.message.MessageType;
-import net.minecraft.network.message.SignedMessage;
-import net.minecraft.network.message.MessageChainTaskQueue;
-import net.minecraft.server.MinecraftServer;
 import net.minecraft.network.packet.c2s.play.ChatMessageC2SPacket;
+import net.minecraft.server.MinecraftServer;
 import net.minecraft.server.filter.FilteredMessage;
 import net.minecraft.server.network.ConnectedClientData;
 import net.minecraft.server.network.ServerCommonNetworkHandler;
-import net.minecraft.server.network.ServerPlayerEntity;
 import net.minecraft.server.network.ServerPlayNetworkHandler;
+import net.minecraft.server.network.ServerPlayerEntity;
 import net.minecraft.text.MutableText;
 import net.minecraft.text.Text;
+import net.minecraft.util.StringHelper;
 import org.spongepowered.asm.mixin.Final;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Shadow;
@@ -24,6 +21,7 @@ import org.spongepowered.asm.mixin.injection.At;
 import org.spongepowered.asm.mixin.injection.Inject;
 import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
 
+import java.lang.reflect.Field;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 import java.util.ArrayDeque;
@@ -46,7 +44,9 @@ public abstract class MixinChat extends ServerCommonNetworkHandler {
 	@Shadow
 	public ServerPlayerEntity player;
 
-	@Shadow protected abstract Optional<LastSeenMessageList> validateMessage(LastSeenMessageList.Acknowledgment acknowledgment);
+//	@Shadow protected abstract Optional<LastSeenMessageList> validateMessage(LastSeenMessageList.Acknowledgment acknowledgment);
+
+	@Shadow protected abstract Optional<LastSeenMessageList> validateAcknowledgment(LastSeenMessageList.Acknowledgment acknowledgment);
 
 	@Shadow protected abstract SignedMessage getSignedMessage(ChatMessageC2SPacket packet, LastSeenMessageList lastSeenMessages) throws MessageChain.MessageChainException;
 
@@ -63,7 +63,7 @@ public abstract class MixinChat extends ServerCommonNetworkHandler {
 		if (hasIllegalCharacter(packet.chatMessage())) {
 			disconnect(Text.translatable("multiplayer.disconnect.illegal_characters"));
 		} else {
-			Optional<LastSeenMessageList> optional = this.validateMessage(packet.acknowledgment());
+			Optional<LastSeenMessageList> optional = this.validateAcknowledgment(packet.acknowledgment());
 			if (optional.isPresent()) {
 				if (!packet.chatMessage().startsWith("/")){
 
@@ -100,15 +100,42 @@ public abstract class MixinChat extends ServerCommonNetworkHandler {
 					try {
 						SignedMessage signedMessage = getSignedMessage(packet, (LastSeenMessageList) optional.get());
 						server.getPlayerManager().broadcast(signedMessage.withUnsignedContent(
-								changedText
+							changedText
 						), player, MessageType.params(MessageType.CHAT, player));
-						// handleDecoratedMessage(signedMessage.withUnsignedContent(changedText));
+
+						// Compatible with mod "Discord Integration"
 						try {
-							Class<?> DiscordIntegrationMod = Class.forName("de.erdbeerbaerlp.dcintegration.fabric.DiscordIntegrationMod");
+							Class<?> DiscordIntegrationMod = Class.forName("de.erdbeerbaerlp.dcintegration.architectury.DiscordIntegrationMod");
 							Method handleChatMessage = DiscordIntegrationMod.getMethod("handleChatMessage", SignedMessage.class, ServerPlayerEntity.class);
 							handleChatMessage.invoke(null, signedMessage.withUnsignedContent(changedText), player);
 						} catch (ClassNotFoundException | NoSuchMethodException | InvocationTargetException |
 								 IllegalAccessException ignored) { }
+
+						// Compatible with mod "Dynmap"
+						String[] minecraftVersion = MinecraftVersion.CURRENT.getName().split("\\.");
+						int minecraftPatchVersion = minecraftVersion.length == 3 ? Integer.parseInt(minecraftVersion[2]) : 0;
+						for (int i = minecraftPatchVersion; i >= 0; i--){
+							String version = minecraftVersion[0] + "." + minecraftVersion[1] + (i == 0 ? "" : ("." + i));
+							try {
+								/*
+								  Warning: Using reflection can make the code harder to maintain, debug, and understand.
+								  Statement: DynmapMod.plugin.chathandler.handleChat()
+								 */
+								Class<?> DynmapMod = Class.forName("org.dynmap.fabric_" + version.replaceAll("\\.", "_") + ".DynmapMod");
+								Field pluginField = DynmapMod.getField("plugin");
+								Object plugin = pluginField.get(null);
+								Class<?> pluginClass = plugin.getClass();
+								Field chatHandlerField = pluginClass.getDeclaredField("chathandler");
+								chatHandlerField.setAccessible(true);
+								Object chatHandler = chatHandlerField.get(plugin);
+								Class<?> chatHandlerClass = chatHandler.getClass();
+								Method handleChat = chatHandlerClass.getMethod("handleChat", ServerPlayerEntity.class, String.class);
+								handleChat.invoke(chatHandler, player, signedMessage.withUnsignedContent(changedText).getContent().getString());
+								break;
+							} catch (ClassNotFoundException | NoSuchMethodException | NoSuchFieldException | NullPointerException |
+									 IllegalAccessException | InvocationTargetException ignored) { }
+						}
+
 					} catch (MessageChain.MessageChainException e) {
 						handleMessageChainException(e);
 					}
@@ -116,7 +143,7 @@ public abstract class MixinChat extends ServerCommonNetworkHandler {
 					this.server.submit(() -> {
 						SignedMessage signedMessage;
 						try {
-							signedMessage = this.getSignedMessage(packet, (LastSeenMessageList) optional.get());
+							signedMessage = this.getSignedMessage(packet, optional.get());
 						} catch (MessageChain.MessageChainException var6) {
 							handleMessageChainException(var6);
 							return;
@@ -139,7 +166,7 @@ public abstract class MixinChat extends ServerCommonNetworkHandler {
 
 	private static boolean hasIllegalCharacter(String message) {
 		for(int i = 0; i < message.length(); ++i) {
-			if (!SharedConstants.isValidChar(message.charAt(i))) {
+			if (!StringHelper.isValidChar(message.charAt(i))) {
 				return true;
 			}
 		}

src/main/resources/fabric.mod.json

@@ -8,8 +8,9 @@
 		"Rev_Aria"
 	],
 	"contact": {
-		"homepage": "https://fabricmc.net/",
+		"homepage": "https://modrinth.com/mod/chatplus",
-		"sources": "https://github.com/FabricMC/fabric-example-mod"
+		"sources": "https://github.com/CPTProgrammer/ChatPlus/",
+		"issues": "https://github.com/CPTProgrammer/ChatPlus/issues"
 	},
 	"license": "CC0-1.0",
 	"icon": "assets/chat-plus/icon.png",
@@ -27,8 +28,8 @@
 		}
 	],
 	"depends": {
-		"fabricloader": ">=0.14.21",
+		"fabricloader": ">=0.16.9",
-		"minecraft": "~1.20.1",
+		"minecraft": "~1.21.4",
 		"java": ">=17",
 		"fabric-api": "*"
 	},