From a67ec47f89fba6af2e0bcb58be12a1ae0d233eb3 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 14 May 2026 03:15:04 +0900 Subject: [PATCH] =?UTF-8?q?v1.3.3=20=E2=80=94=20replace=20storage=20flag?= =?UTF-8?q?=20with=20direct=20function=20call=20(race-free)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 이전 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 at @s run function mq:players/mod_active_notice 를 호출. 데이터팩이 메세지 텍스트를 정의하고, 모드는 "지금 들어온 이 플레이어에게 보여라" 만 트리거한다. 데이터팩 자체가 없으면 함수가 없어 suppressed source 의 command 실패로 silent → 안전. 데이터팩(music_quiz) 도 동일 커밋으로 함수 추가 및 flag 제거됨. --- .../chatanswer/core/ChatAnswerCore.java | 27 +++++++++++-------- gradle.properties | 2 +- 2 files changed, 17 insertions(+), 12 deletions(-) diff --git a/common/src/main/java/kr/tkrmagid/chatanswer/core/ChatAnswerCore.java b/common/src/main/java/kr/tkrmagid/chatanswer/core/ChatAnswerCore.java index 611be67..c314c37 100644 --- a/common/src/main/java/kr/tkrmagid/chatanswer/core/ChatAnswerCore.java +++ b/common/src/main/java/kr/tkrmagid/chatanswer/core/ChatAnswerCore.java @@ -32,28 +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) { - LOG.warn("[{}] onPlayerJoin: server is null, skipping active-flag set for {}", MOD_ID, name); + 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" - ); - LOG.info("[{}] active=1b set after {} joined", MOD_ID, name); + server.getCommands().performPrefixedCommand(source, command); + LOG.info("[{}] mod_active_notice invoked for {}", MOD_ID, name); } catch (Exception e) { - LOG.warn("[{}] failed to set active flag for {}: {}", MOD_ID, name, e.toString(), e); + LOG.warn("[{}] failed to invoke mod_active_notice for {}: {}", MOD_ID, name, e.toString(), e); } } diff --git a/gradle.properties b/gradle.properties index e92d083..985d773 100644 --- a/gradle.properties +++ b/gradle.properties @@ -3,7 +3,7 @@ org.gradle.parallel=true # ───── mod metadata ───────────────────────────────────────────────────────── mod_id=chat_answer -mod_version=1.3.2 +mod_version=1.3.3 mod_group=kr.tkrmagid.chatanswer mod_name=채팅정답