v1.3.8 — 정답 단계 채팅 broadcast 차단 해제

사용자 요청: 정답 입력 단계에서도 친 채팅이 다른 플레이어한테 보이게.
정답 보호는 데이터팩이 아니라 룸 운영자의 신뢰 기반 운영으로 처리한다는
방침. 다른 단계는 원래도 차단하지 않았으니 변경 없음.

- ChatAnswerCore.handleChat 가 항상 true 반환. 정답 단계(state 5) 일
  때만 부가적으로 mq:answer/submit 호출하고 broadcast 는 그대로 허용.
- Fabric ALLOW_CHAT_MESSAGE / NeoForge ServerChatEvent 핸들러는 그대로
  반환값을 전달 — 항상 true 라 cancel 안 됨.
- README.md: 차단 동작 설명을 v1.3.8 부터 broadcast 허용으로 정정.

호환: 데이터팩 mq:answer/submit 함수는 동일하게 호출되므로 음악퀴즈
v1.0.26 + chat_answer v1.3.8 조합으로 그대로 동작. 정답 보이는 게
싫으면 v1.3.7 으로 다운그레이드.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

README.md

@@ -9,8 +9,9 @@
 execute as <플레이어 UUID> run function mq:answer/submit {text:'<채팅 내용>'}
 ```
 
-채팅은 다른 플레이어에게 broadcast 되지 않으므로 정답이 화면에 노출되지 않는다.
+v1.3.8 부터는 정답 단계여도 채팅이 평소대로 broadcast 된다 (다른 플레이어
-`init` 점수가 5 가 아닐 때는 채팅이 평소처럼 모두에게 보인다.
+화면에 그대로 노출됨). 정답 보호는 데이터팩이 아니라 룸 운영자의 신뢰 기반
+운영으로 처리한다. v1.3.7 까지는 정답 단계에서 채팅 broadcast 가 차단됐었다.
 
 ## 빌드
 

common/src/main/java/kr/tkrmagid/chatanswer/core/ChatAnswerCore.java

@@ -1,5 +1,9 @@
 package kr.tkrmagid.chatanswer.core;
 
+import java.util.Iterator;
+import java.util.Map;
+import java.util.UUID;
+import java.util.concurrent.ConcurrentHashMap;
 import net.minecraft.commands.CommandSourceStack;
 import net.minecraft.server.MinecraftServer;
 import net.minecraft.server.level.ServerPlayer;
@@ -17,8 +21,11 @@ import org.slf4j.LoggerFactory;
  *   execute as <player UUID> run function mq:answer/submit {text:'<채팅>'}
  * 을 OP 레벨로 실행한다.
  *
- * 각 로더 진입점(Fabric / NeoForge) 에서 chat 이벤트 받자마자 {@link #handleChat}
+ * v1.3.8 부터 채팅은 어떤 단계에서도 broadcast 차단하지 않는다 — 정답 입력
- * 호출 → 반환값이 false 면 그 채팅은 broadcast 차단해야 함.
+ * 단계에서도 친 채팅이 평소처럼 채팅창에 보인다. (사용자 요청: 정답 화면
+ * 노출을 데이터팩이 관리하지 않고 룸 운영자가 신뢰 기반으로 처리.)
+ * 따라서 {@link #handleChat} 는 항상 true 를 반환하며, 정답 단계일 때만
+ * 부가적으로 정답 제출 함수를 호출한다.
  */
 public final class ChatAnswerCore {
     public static final String MOD_ID = "chat_answer";
@@ -29,49 +36,121 @@ public final class ChatAnswerCore {
     private static final String SCOREBOARD_HOLDER = "init";
     private static final int ACCEPTING_ANSWER_STATE = 5;
 
+    /** 음악퀴즈 데이터팩이 선언한 "모드 존재 확인" 점수 이름.
+     *  본 모드는 서버 측에서 채팅을 가로채는 server-only 모드 — 클라이언트는
+     *  설치할 필요가 없고 server 한 곳에 있으면 모든 플레이어에게 적용된다.
+     *  따라서 per-player 검증은 무의미하고, fake player {@link #PRESENCE_HOLDER}
+     *  점수만 1 로 set 한다. 데이터팩의 start 가드는
+     *  `score <PRESENCE_HOLDER> <OBJECTIVE> matches 1` 로 검사.
+     *
+     *  presence pulse 는 여러 이벤트에서 중복 호출한다 — banner/mohist 같은
+     *  fabric-bukkit 하이브리드 호스트에서 일부 Fabric 이벤트(특히
+     *  ServerTickEvents.END_SERVER_TICK) 가 안 들어오는 케이스가 보고됨.
+     *  SERVER_STARTED / PlayerJoin / TickEnd 셋 중 하나라도 firing 되면
+     *  데이터팩 가드가 통과하도록 모든 진입점에서 markModPresence 호출. */
+    private static final String MOD_PRESENCE_OBJECTIVE = "mq_chat_mod";
+    private static final String PRESENCE_HOLDER = "#server";
+
+    /** JOIN 이벤트 시점엔 클라이언트가 chat HUD 를 받을 준비가 안 됐을 수 있어
+     *  tellraw 패킷이 사라지는 경우가 있다. 그래서 N 틱 늦춰서 호출한다. */
+    private static final int NOTICE_DELAY_TICKS = 20;
+    private static final Map<UUID, Integer> PENDING_NOTICES = new ConcurrentHashMap<>();
+
     private ChatAnswerCore() {}
 
     /**
-     * 플레이어 로그인 직후 호출. 음악퀴즈 데이터팩의
+     * 플레이어 로그인 시점에 호출. 음악퀴즈 데이터팩의
      *   mq:players/mod_active_notice
-     * 함수를 해당 플레이어 컨텍스트로 직접 호출한다. 데이터팩이 메세지를 정의하고,
+     * 함수를 해당 플레이어 컨텍스트로 호출한다. 단, JOIN 이벤트가 너무 일러서
-     * 모드는 "지금 막 들어온 이 플레이어에게 보여라" 만 트리거한다.
+     * 즉시 호출 시 tellraw 가 클라이언트에 도달하지 못하는 race 가 있어
-     *
+     * {@link #NOTICE_DELAY_TICKS} 만큼 늦춘다 ({@link #onServerTick} 가 처리).
-     * 이전엔 storage chat_answer:status active=1b 플래그를 썼는데, 통합 서버에서
-     * mq:load 의 0b 초기화가 player join 이후에 도는 케이스 때문에 race 가 발생했다.
-     * 함수를 모드가 직접 호출하면 race 가 사라지고, 데이터팩이 없을 땐 함수가
-     * 존재하지 않아 커맨드가 (suppressed source 라 채팅엔 안 뜨고 log warn 으로)
-     * 무시된다.
      */
     public static void onPlayerJoin(ServerPlayer player) {
         String name = player.getName().getString();
-        LOG.info("[{}] onPlayerJoin fired for {}", MOD_ID, name);
+        LOG.info("[{}] onPlayerJoin fired for {}, scheduling notice in {} ticks",
+                MOD_ID, name, NOTICE_DELAY_TICKS);
+        PENDING_NOTICES.put(player.getUUID(), NOTICE_DELAY_TICKS);
+        // tick 이벤트가 안 들어오는 호스트 대비 — join 시점에도 presence 한 번 찍는다.
         MinecraftServer server = player.level().getServer();
-        if (server == null) {
+        if (server != null) markModPresence(server);
-            LOG.warn("[{}] onPlayerJoin: server is null, skipping notice for {}", MOD_ID, name);
+    }
-            return;
+
-        }
+    /** 각 로더 entrypoint 가 서버 부팅 완료 시점에 호출. tick 이벤트가
-        CommandSourceStack source = server.createCommandSourceStack().withSuppressedOutput();
+     *  발화되지 않는 환경(banner/mohist) 에서 최소 한 번은 presence 가 찍히도록.
-        String command = "execute as " + player.getStringUUID()
+     *  데이터팩 load 가 SERVER_STARTED 보다 먼저 끝나므로 objective 도 이미 존재. */
-                + " at @s run function mq:players/mod_active_notice";
+    public static void onServerStarted(MinecraftServer server) {
-        try {
+        LOG.info("[{}] onServerStarted fired, marking presence", MOD_ID);
-            server.getCommands().performPrefixedCommand(source, command);
+        markModPresence(server);
-            LOG.info("[{}] mod_active_notice invoked for {}", MOD_ID, name);
+    }
-        } catch (Exception e) {
+
-            LOG.warn("[{}] failed to invoke mod_active_notice for {}: {}", MOD_ID, name, e.toString(), e);
+    /** /reload 직후 호출. load.mcfunction 이 mq_chat_mod objective 를 remove/add
+     *  하고 `#server` 점수를 0 으로 재설정하므로, reload 끝난 직후 즉시
+     *  다시 1 로 찍어야 함. tick 이벤트가 죽은 호스트 + 이미 접속 중인
+     *  플레이어 조합에서 SERVER_STARTED/JOIN 둘 다 발화 안 되는 케이스 커버. */
+    public static void onDataPackReload(MinecraftServer server) {
+        LOG.info("[{}] onDataPackReload fired, re-marking presence", MOD_ID);
+        markModPresence(server);
+    }
+
+    /** 각 로더 entrypoint 가 매 server tick 마다 호출해야 한다. */
+    public static void onServerTick(MinecraftServer server) {
+        markModPresence(server);
+        if (PENDING_NOTICES.isEmpty()) return;
+        Iterator<Map.Entry<UUID, Integer>> it = PENDING_NOTICES.entrySet().iterator();
+        while (it.hasNext()) {
+            Map.Entry<UUID, Integer> e = it.next();
+            int remaining = e.getValue() - 1;
+            if (remaining > 0) {
+                e.setValue(remaining);
+                continue;
+            }
+            UUID uuid = e.getKey();
+            it.remove();
+            ServerPlayer player = server.getPlayerList().getPlayer(uuid);
+            if (player == null) continue;
+            deliverNotice(server, player);
         }
     }
 
     /**
-     * @return true = 채팅을 평소처럼 broadcast / false = 채팅 차단 (이미 정답 제출 처리됨)
+     * 데이터팩의 mq_chat_mod 점수(fake player #server 키) 를 1 로 set.
+     * 데이터팩이 아직 load 되지 않아 objective 가 없으면 조용히 skip.
+     * 점수 값이 이미 1 이면 Minecraft 가 packet 전송을 생략하므로
+     * 매 tick 호출해도 트래픽은 늘지 않는다.
+     */
+    private static void markModPresence(MinecraftServer server) {
+        Scoreboard scoreboard = server.getScoreboard();
+        Objective objective = scoreboard.getObjective(MOD_PRESENCE_OBJECTIVE);
+        if (objective == null) return;
+        scoreboard.getOrCreatePlayerScore(ScoreHolder.forNameOnly(PRESENCE_HOLDER), objective).set(1);
+    }
+
+    private static void deliverNotice(MinecraftServer server, ServerPlayer player) {
+        String name = player.getName().getString();
+        // 플레이어 자체를 source 로 써서 함수 안의 @s 가 그대로 player.
+        CommandSourceStack source = player.createCommandSourceStack().withSuppressedOutput();
+        try {
+            server.getCommands().performPrefixedCommand(source, "function mq:players/mod_active_notice");
+            LOG.info("[{}] mod_active_notice delivered for {}", MOD_ID, name);
+        } catch (Exception e) {
+            LOG.warn("[{}] failed to deliver mod_active_notice for {}: {}", MOD_ID, name, e.toString(), e);
+        }
+    }
+
+    /**
+     * 항상 true 반환 — 어떤 단계에서도 채팅을 차단하지 않는다.
+     * 정답 단계(state 5) 일 때만 부가적으로 정답 제출 함수를 호출한다.
+     *
+     * @return 항상 true (broadcast 허용). 로더 진입점은 반환값을 그대로 이벤트
+     *         allow/cancel 결정에 전달하면 된다.
      */
     public static boolean handleChat(ServerPlayer sender, String rawText) {
         MinecraftServer server = sender.level().getServer();
         if (server == null) return true;
-        if (!isAcceptingAnswer(server)) return true;
+        if (isAcceptingAnswer(server)) {
-
+            submitAnswer(server, sender, rawText);
-        submitAnswer(server, sender, rawText);
+        }
-        return false;
+        return true;
     }
 
     private static boolean isAcceptingAnswer(MinecraftServer server) {

fabric-1216/src/main/java/kr/tkrmagid/chatanswer/fabric/ChatAnswerFabric.java

@@ -2,6 +2,8 @@ package kr.tkrmagid.chatanswer.fabric;
 
 import kr.tkrmagid.chatanswer.core.ChatAnswerCore;
 import net.fabricmc.api.ModInitializer;
+import net.fabricmc.fabric.api.event.lifecycle.v1.ServerLifecycleEvents;
+import net.fabricmc.fabric.api.event.lifecycle.v1.ServerTickEvents;
 import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
 import net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents;
 import org.slf4j.Logger;
@@ -17,10 +19,15 @@ public final class ChatAnswerFabric implements ModInitializer {
             ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
                 ChatAnswerCore.handleChat(sender, message.signedContent())
             );
+            ServerLifecycleEvents.SERVER_STARTED.register(ChatAnswerCore::onServerStarted);
+            ServerLifecycleEvents.END_DATA_PACK_RELOAD.register((server, resourceManager, success) -> {
+                if (success) ChatAnswerCore.onDataPackReload(server);
+            });
             ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
                 ChatAnswerCore.onPlayerJoin(handler.player)
             );
-            LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + JOIN", ChatAnswerCore.MOD_ID);
+            ServerTickEvents.END_SERVER_TICK.register(ChatAnswerCore::onServerTick);
+            LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + SERVER_STARTED + END_DATA_PACK_RELOAD + JOIN + TICK", ChatAnswerCore.MOD_ID);
         } catch (Throwable t) {
             LOG.error("[{}] Fabric entrypoint event registration failed", ChatAnswerCore.MOD_ID, t);
             throw t;

fabric-2612/src/main/java/kr/tkrmagid/chatanswer/fabric/ChatAnswerFabric.java

@@ -2,6 +2,8 @@ package kr.tkrmagid.chatanswer.fabric;
 
 import kr.tkrmagid.chatanswer.core.ChatAnswerCore;
 import net.fabricmc.api.ModInitializer;
+import net.fabricmc.fabric.api.event.lifecycle.v1.ServerLifecycleEvents;
+import net.fabricmc.fabric.api.event.lifecycle.v1.ServerTickEvents;
 import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
 import net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents;
 import org.slf4j.Logger;
@@ -17,10 +19,15 @@ public final class ChatAnswerFabric implements ModInitializer {
             ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
                 ChatAnswerCore.handleChat(sender, message.signedContent())
             );
+            ServerLifecycleEvents.SERVER_STARTED.register(ChatAnswerCore::onServerStarted);
+            ServerLifecycleEvents.END_DATA_PACK_RELOAD.register((server, resourceManager, success) -> {
+                if (success) ChatAnswerCore.onDataPackReload(server);
+            });
             ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
                 ChatAnswerCore.onPlayerJoin(handler.player)
             );
-            LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + JOIN", ChatAnswerCore.MOD_ID);
+            ServerTickEvents.END_SERVER_TICK.register(ChatAnswerCore::onServerTick);
+            LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + SERVER_STARTED + END_DATA_PACK_RELOAD + JOIN + TICK", ChatAnswerCore.MOD_ID);
         } catch (Throwable t) {
             LOG.error("[{}] Fabric entrypoint event registration failed", ChatAnswerCore.MOD_ID, t);
             throw t;

gradle.properties

@@ -3,7 +3,7 @@ org.gradle.parallel=true
 
 # ───── mod metadata ─────────────────────────────────────────────────────────
 mod_id=chat_answer
-mod_version=1.3.3
+mod_version=1.3.8
 mod_group=kr.tkrmagid.chatanswer
 mod_name=채팅정답
 

neoforge-1216/src/main/java/kr/tkrmagid/chatanswer/neoforge/ChatAnswerNeoForge.java

@@ -6,14 +6,20 @@ import net.neoforged.bus.api.IEventBus;
 import net.neoforged.bus.api.SubscribeEvent;
 import net.neoforged.fml.common.Mod;
 import net.neoforged.neoforge.common.NeoForge;
+import net.neoforged.neoforge.event.OnDatapackSyncEvent;
 import net.neoforged.neoforge.event.ServerChatEvent;
 import net.neoforged.neoforge.event.entity.player.PlayerEvent;
+import net.neoforged.neoforge.event.server.ServerStartedEvent;
+import net.neoforged.neoforge.event.tick.ServerTickEvent;
 
 @Mod(ChatAnswerCore.MOD_ID)
 public final class ChatAnswerNeoForge {
     public ChatAnswerNeoForge(IEventBus modBus) {
         NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onServerChat);
+        NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onServerStarted);
+        NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onDatapackSync);
         NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onPlayerLogin);
+        NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onServerTick);
     }
 
     @SubscribeEvent
@@ -24,10 +30,28 @@ public final class ChatAnswerNeoForge {
         }
     }
 
+    @SubscribeEvent
+    public static void onServerStarted(ServerStartedEvent event) {
+        ChatAnswerCore.onServerStarted(event.getServer());
+    }
+
+    /** OnDatapackSyncEvent: /reload 끝나면 player=null 로 한 번 broadcast,
+     *  로그인 때마다 해당 player 로 한 번 더 fire. 어느 쪽이든 reload 직후
+     *  presence 가 다시 찍히는 것이 목적이라 둘 다 OK. */
+    @SubscribeEvent
+    public static void onDatapackSync(OnDatapackSyncEvent event) {
+        ChatAnswerCore.onDataPackReload(event.getPlayerList().getServer());
+    }
+
     @SubscribeEvent
     public static void onPlayerLogin(PlayerEvent.PlayerLoggedInEvent event) {
         if (event.getEntity() instanceof ServerPlayer player) {
             ChatAnswerCore.onPlayerJoin(player);
         }
     }
+
+    @SubscribeEvent
+    public static void onServerTick(ServerTickEvent.Post event) {
+        ChatAnswerCore.onServerTick(event.getServer());
+    }
 }