v1.3.3 — replace storage flag with direct function call (race-free)

이전 v1.3.2: onPlayerJoin 에서 storage chat_answer:status active=1b 를 set.
하지만 통합 서버 (integrated singleplayer) 에서 데이터팩의 mq:load 가
player join 이후에 도는 케이스가 있어 모드가 써놓은 1b 를 데이터팩이
0b 로 덮어쓰는 race 가 있었고, repeat/players 의 첫 tick 체크 시점에
이미 0b 라서 알림 메세지가 안 떴음.

v1.3.3: storage flag 자체를 폐기. onPlayerJoin 에서
  execute as <uuid> at @s run function mq:players/mod_active_notice
를 호출. 데이터팩이 메세지 텍스트를 정의하고, 모드는 "지금 들어온 이
플레이어에게 보여라" 만 트리거한다. 데이터팩 자체가 없으면 함수가 없어
suppressed source 의 command 실패로 silent → 안전.

데이터팩(music_quiz) 도 동일 커밋으로 함수 추가 및 flag 제거됨.

build.gradle

@@ -63,11 +63,17 @@ tasks.register('containerJar', Jar) {
         }
     }
 
-    // 3. Fabric nested jars (Fabric Loader 가 META-INF/jars/ 를 스캔해서
-    //    depends.minecraft 매칭되는 jar 만 활성화).
+    // 3. Fabric nested jars. Fabric Loader 는 META-INF/jars/ 를 자동 스캔하지
+    //    않고 outer fabric.mod.json 의 "jars" 배열에 명시된 파일만 처리하므로,
+    //    container-resources/fabric.mod.json 의 jars 항목과 일치하는 고정 파일명
+    //    (버전 suffix 제거) 으로 넣는다.
     into('META-INF/jars') {
-        from project(':fabric-1216').tasks.named('remapJar').flatMap { it.archiveFile }
-        from project(':fabric-2612').tasks.named('remapJar').flatMap { it.archiveFile }
+        from(project(':fabric-1216').tasks.named('remapJar').flatMap { it.archiveFile }) {
+            rename '.+\\.jar', 'chat_answer-fabric-1216.jar'
+        }
+        from(project(':fabric-2612').tasks.named('remapJar').flatMap { it.archiveFile }) {
+            rename '.+\\.jar', 'chat_answer-fabric-2612.jar'
+        }
     }
 }
 

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

@@ -32,22 +32,33 @@ public final class ChatAnswerCore {
     private ChatAnswerCore() {}
 
     /**
-     * 플레이어 로그인 직후 호출. 데이터팩이 "모드 살아있음" 신호로 쓸 수 있게
-     * storage chat_answer:status 에 active=1b 를 세팅한다. 데이터팩의 mq:load 가
-     * 매 /reload 와 서버 시작 시 이 값을 0b 로 clear 하므로, 모드가 없으면 이
-     * 호출이 일어나지 않아 0b 로 유지되고, 모드가 있으면 첫 로그인 직후 1b 로 갱신.
+     * 플레이어 로그인 직후 호출. 음악퀴즈 데이터팩의
+     *   mq:players/mod_active_notice
+     * 함수를 해당 플레이어 컨텍스트로 직접 호출한다. 데이터팩이 메세지를 정의하고,
+     * 모드는 "지금 막 들어온 이 플레이어에게 보여라" 만 트리거한다.
+     *
+     * 이전엔 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);
         MinecraftServer server = player.level().getServer();
-        if (server == null) return;
+        if (server == null) {
+            LOG.warn("[{}] onPlayerJoin: server is null, skipping notice for {}", MOD_ID, name);
+            return;
+        }
         CommandSourceStack source = server.createCommandSourceStack().withSuppressedOutput();
+        String command = "execute as " + player.getStringUUID()
+                + " at @s run function mq:players/mod_active_notice";
         try {
-            server.getCommands().performPrefixedCommand(
-                source,
-                "data modify storage chat_answer:status active set value 1b"
-            );
+            server.getCommands().performPrefixedCommand(source, command);
+            LOG.info("[{}] mod_active_notice invoked for {}", MOD_ID, name);
         } catch (Exception e) {
-            LOG.debug("[{}] failed to set active flag: {}", MOD_ID, e.toString());
+            LOG.warn("[{}] failed to invoke mod_active_notice for {}: {}", MOD_ID, name, e.toString(), e);
         }
     }
 

container-resources/fabric.mod.json

@@ -8,6 +8,10 @@
   "license": "MIT",
   "icon": "assets/chat_answer/icon.png",
   "environment": "*",
+  "jars": [
+    { "file": "META-INF/jars/chat_answer-fabric-1216.jar" },
+    { "file": "META-INF/jars/chat_answer-fabric-2612.jar" }
+  ],
   "depends": {
     "fabricloader": ">=0.16.0",
     "java": ">=21"

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

@@ -4,15 +4,26 @@ import kr.tkrmagid.chatanswer.core.ChatAnswerCore;
 import net.fabricmc.api.ModInitializer;
 import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
 import net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public final class ChatAnswerFabric implements ModInitializer {
+    private static final Logger LOG = LoggerFactory.getLogger(ChatAnswerCore.MOD_ID);
+
     @Override
     public void onInitialize() {
-        ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
-            ChatAnswerCore.handleChat(sender, message.signedContent())
-        );
-        ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
-            ChatAnswerCore.onPlayerJoin(handler.player)
-        );
+        LOG.info("[{}] Fabric entrypoint onInitialize starting", ChatAnswerCore.MOD_ID);
+        try {
+            ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
+                ChatAnswerCore.handleChat(sender, message.signedContent())
+            );
+            ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
+                ChatAnswerCore.onPlayerJoin(handler.player)
+            );
+            LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + JOIN", 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

@@ -4,15 +4,26 @@ import kr.tkrmagid.chatanswer.core.ChatAnswerCore;
 import net.fabricmc.api.ModInitializer;
 import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
 import net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public final class ChatAnswerFabric implements ModInitializer {
+    private static final Logger LOG = LoggerFactory.getLogger(ChatAnswerCore.MOD_ID);
+
     @Override
     public void onInitialize() {
-        ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
-            ChatAnswerCore.handleChat(sender, message.signedContent())
-        );
-        ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
-            ChatAnswerCore.onPlayerJoin(handler.player)
-        );
+        LOG.info("[{}] Fabric entrypoint onInitialize starting", ChatAnswerCore.MOD_ID);
+        try {
+            ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
+                ChatAnswerCore.handleChat(sender, message.signedContent())
+            );
+            ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
+                ChatAnswerCore.onPlayerJoin(handler.player)
+            );
+            LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + JOIN", 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.0
+mod_version=1.3.3
 mod_group=kr.tkrmagid.chatanswer
 mod_name=채팅정답