Skip to content

feat: 他プラグインが付与する no_see_death_message メタデータを尊重する - #33

Merged
Siroshun09 merged 2 commits into
masterfrom
claude/relaxed-maxwell-epgy1n
Oct 24, 2026
Merged

Siroshun09 merged 2 commits into
masterfrom
claude/relaxed-maxwell-epgy1n

Conversation

@Siroshun09

Copy link
Copy Markdown
Member

背景

MoreFlags は、死亡メッセージを受け取れないプレイヤーにメタデータ no_see_death_message を付け、ToggleDeathMessage(TDM)が導入されている場合は再送信を TDM に任せます。v4 の TDM はこのメタデータを読まないため、併用すると MoreFlags の send-death-message / receive-death-message / isolate-death-message が全部無効になっていました。

変更

  • no_see_death_message メタデータを持つプレイヤーを、/tdm off と同様に「非表示」として扱います。
  • TDM が処理した死亡イベントの終了時に、全オンラインプレイヤーからこのメタデータを必ず削除します。MoreFlags は TDM がある場合に削除しないため、TDM が消費する必要があります。

検証

Paper 26.3 + WorldGuard 7.0.19 + MoreFlags(okocraft/MoreFlags#49 適用版)で、実 Bot が受信した結果を確認しました(14/14 PASS)。

  • send / receive / isolate が期待どおり動作します。
  • 個人設定 /tdm off と MoreFlags のフラグを同時に使うと、両方の除外が効きます。
  • 領域外で死亡した次の死亡では、メタデータが残らず全員に届きます。
  • チームの可視性 4 設定は Vanilla と一致し、死亡画面の理由とコンソール出力も維持されます。

未検証・注意

  • ユニットテストと Folia は未検証です。
  • 死亡イベントが LOW〜HIGHEST の間で他プラグインにキャンセルされると、メタデータが次の死亡まで残る可能性があります。
  • メタデータ API は非推奨とのことなので、暫定対応です。恒久的にはイベント方式などへの置き換えが必要です。
  • この PR は、MoreFlags 側の PR #49(バイパス権限を持つプレイヤーのメタデータ削除)と組み合わせて検証しました。

Generated by Claude Code

MoreFlags marks players who must not receive a death message with this metadata
and defers the delivery to ToggleDeathMessage. v4 ignored it, so the MoreFlags
death message flags had no effect. Treat such players as hiding and consume the
metadata after every handled death event.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01An7Jsqoey6TALGm294uYKf
@Siroshun09
Siroshun09 merged commit 9f06625 into master Oct 24, 2026
2 checks passed
@Siroshun09 Siroshun09 mentioned this pull request Oct 24, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants