Merge latest master and resolve MessageListener conflict without regressing whisper refactor - #5174
Open
jamesrudolph with Copilot wants to merge 6 commits into
Open
Merge latest master and resolve MessageListener conflict without regressing whisper refactor#5174jamesrudolph with Copilot wants to merge 6 commits into
jamesrudolph with Copilot wants to merge 6 commits into
Conversation
Co-authored-by: jamesrudolph <39609802+jamesrudolph@users.noreply.github.com>
Co-authored-by: jamesrudolph <39609802+jamesrudolph@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
jamesrudolph
June 1, 2026 02:06
View session
🔬 Test Results✅ JSON ValidationNo JSON files changed. ✅ Formatting Checks PassedAll files pass Spotless formatting. ✅ JUnit Tests PassedAll tests passed successfully. |
jamesrudolph
marked this pull request as ready for review
June 16, 2026 20:05
Co-authored-by: jamesrudolph <39609802+jamesrudolph@users.noreply.github.com>
Copilot
AI
changed the title
Refactor whisper handling into WhisperService and add
Merge latest master and resolve MessageListener conflict without regressing whisper refactor
Jun 27, 2026
/whisper, /w, and /remind_me commands
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces a significant refactor and enhancement of the "whisper" and "remind me" command handling in the Discord bot, moving legacy message-based whisper handling into explicit slash commands and centralizing logic. It also adds new commands for whispering and reminders, improves option auto-completion, and cleans up unused code.
New Features and Slash Commands:
/whisper,/w(alias for whisper), and/remind_mefor sending private or future-dated messages between players, or reminders to oneself. (WhisperCommand.java,WCommand.java,RemindMeCommand.java,SlashCommandManager.java) [1] [2] [3] [4] [5]WhisperCommandBaseto encapsulate shared logic for whisper commands, supporting options for target player, message, and future delivery.Refactoring and Centralization:
MessageListenerintoWhisperService, removing a large block of parsing and processing code from the listener. Now,MessageListenerdelegates whisper handling toWhisperService.handleWhispers. [1] [2] [3]MessageListener.java. [1] [2] [3]Improvements to Option Handling and Auto-complete:
TOand updated auto-complete logic to support the/whispercommand’s recipient field. [1] [2]These changes modernize whisper interactions, make them more robust and user-friendly, and lay groundwork for future extensibility.