Conversation
Adds a new "SILENT" option for the chatguard message-handling config. When enabled, toxic messages detected by AI are cancelled but sent back only to their author, creating the illusion of successful delivery while hiding the message from all other players. The strike system and server logging still apply. Supports both standard Bukkit chat and CarbonChat integration. Locale strings added for all 8 languages. https://claude.ai/code/session_01VxBVHr1ibPXu49CsDL2yd8
Triggers on version tags (v*). Builds shadowJar and publishes the JAR as a GitHub Release with auto-generated release notes. https://claude.ai/code/session_01VxBVHr1ibPXu49CsDL2yd8
ToothyDev
left a comment
ToothyDev
left a comment
There was a problem hiding this comment.
Looks fine otherwise, tested and works. Just change what I said in the comments and resolve the merge conflicts, then we can merge this
There was a problem hiding this comment.
This is nice but we don't really need this at the current stage in the project, doing it manually is trivial as of now
|
|
||
| switch (messageHandling) { | ||
| case "BLOCK" -> event.cancelled(true); | ||
| case "SILENT" -> { |
There was a problem hiding this comment.
I think SILENCE fits better here because it's the verb form like the other actions
|
|
||
| switch (messageHandling) { | ||
| case "BLOCK" -> event.setCancelled(true); | ||
| case "SILENT" -> { |
There was a problem hiding this comment.
SILENT -> SILENCE here as well
| * @param player The player that sent the message | ||
| * @param userMessage The message that the user sent | ||
| * @param classification The classification of the message | ||
| * @param messageHandling The message handling mode: "BLOCK", "CENSOR", or "SILENT" |
There was a problem hiding this comment.
SILENT -> SILENCE here as well
| plugin.getConfigHelperLanguage() | ||
| .getString(blockOrCensor ? LangConstants.ChatGuard.MESSAGE_BLOCKED : LangConstants.ChatGuard.MESSAGE_CENSORED) + | ||
| " " + ChatColor.RED + classification.reason()); | ||
| if (plugin.getConfigHelper().getBoolean(ConfigConstants.ChatGuard.NOTIFY_USER) && !messageHandling.equals("SILENT")) { |
There was a problem hiding this comment.
SILENT -> SILENCE here as well
| .info("Message by " + player.getName() + (blockOrCensor ? " has been blocked: " : " has been censored: ") + userMessage); | ||
| String action = switch (messageHandling) { | ||
| case "BLOCK" -> "blocked"; | ||
| case "SILENT" -> "silently moderated"; |
There was a problem hiding this comment.
SILENT -> SILENCE here as well
| # Options: CENSOR, BLOCK, SILENT | ||
| # SILENT: The message is only visible to the sender, creating the illusion of successful delivery |
There was a problem hiding this comment.
SILENT -> SILENCE here as well
|
Also, please add this PR to the current changelog file |
ToothyDev
changed the title
4 participants
No description provided.