Feature: Implement /rewrite command for message improvement using Cha… #1378

firasrg · 2026-01-13T21:56:16Z

New Feature: Rewrite Command

The /rewrite command allows users to have their messages rewritten in a clearer, more professional, or better structured form using ChatGPT AI (GPT 5 mini). The rewritten message is displayed as an ephemeral message (visible only to the user), making it perfect for getting quick writing improvements without cluttering the channel where they are.

Usage

/rewrite message:<text> [tone:<style>]

Parameters

message (required): The message to rewrite (3-500 characters)
tone (optional): The tone/style enum value for the rewrite. Defaults to CLEAR if not provided

Available Tones

CLEAR (default): Make the message clear and easy to understand
PRO: Use a professional and polished tone
DETAILED: Expand with more detail and explanation
TECHNICAL: Use technical and specialized language where appropriate

How It Works

You invoke the command with your message and optional tone
ChatGPT analyzes and rewrites the message according to the selected tone
A formatted embed displays both the original and rewritten versions
The response is ephemeral—only visible to you for 15 minutes
You can copy the rewritten text and use it.

Example

UPDATE

The command name has been changed into /rewrite-msg for consistency with the class in source code.
The following is a demo that shows all possible states :

…tGPT

Zabuzard · 2026-01-14T08:02:52Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+ * <p>
+ * Users can optionally specify a tone/style for the rewrite. If not provided, defaults to CLEAR.
+ */
+public final class RewriteMsgCommand extends SlashCommandAdapter {


dont abbreviate, RewriteMsgCommand. but to avoid confusion i would try to stick to the actual slash-command name, so just RewriteCommand would be better imo

I thought about doing that in the beginning, but I found that it may be confusing, since it's possible to be interpreted "rewrite something else, other than a message", so I found it better to write "Message" as abbreviation to avoid verbosity

but that confusion then also applies to the slashcommand. if u think its confusing then the slash command has to be renamed as well, to /rewrite-msg for example. then the class name is okay again

/rewrite is perfectly fine

@tj-wazei is this a requirement or an optional suggestion?

Zabuzard · 2026-01-14T08:03:16Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+
+

double new line

Zabuzard · 2026-01-14T08:03:53Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+
+
+    public RewriteMsgCommand(RewriteMsgService rewriteMsgService) {
+        super(COMMAND_NAME, "Rewrite your message in a clearer, more professional form",


I would reword it to "Let AI rephrase and improve your message"

Zabuzard · 2026-01-14T08:05:00Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+ * Users can optionally specify a tone/style for the rewrite. If not provided, defaults to CLEAR.
+ */
+public final class RewriteMsgCommand extends SlashCommandAdapter {
+    private static final Logger logger = LoggerFactory.getLogger(RewriteMsgCommand.class);


your logging in this entire class is mostly all over. almost all your "debug" logs are trace logs instead.
and most of your info logs are debug logs.

this has to be adjusted. ill try to leave comments on a per-log base

Zabuzard · 2026-01-14T08:05:18Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+
+        this.rewriteMsgService = rewriteMsgService;
+
+        logger.debug("Initializing RewriteMsgCommand with ChatGptService and HelpSystemHelper");


obsolete, BotCore already logs this stuff on trace level. remove

Zabuzard · 2026-01-14T08:28:20Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgService.java

+        final int previewLength = Math.min(50, message.length());
+        final String preview = message.substring(0, previewLength);
+
+        logger.debug("Message content preview: {}", preview);


that log is missing context and not that helpful without

Zabuzard · 2026-01-14T08:30:00Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgTone.java

+ * <p>
+ * Each tone provides a specific instruction to ChatGPT for how to approach the rewrite.
+ */
+public enum RewriteMsgTone {


put it as private helper inside RewriteCommand, then remove all the javadoc. all javadoc in this class is obsolete and just documenting what the code already says, its not needed and just bloat

Zabuzard · 2026-01-14T08:30:28Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgTone.java

+    public String getPromptInstruction() {
+        return switch (this) {
+            case CLEAR -> "Make it clear and easy to understand.";
+            case PRO -> "Use a professional and polished tone.";
+            case DETAILED -> "Expand with more detail and explanation.";
+            case TECHNICAL -> "Use technical and specialized language where appropriate.";
+        };
+    }


make that a field instead, like your displayName

Zabuzard · 2026-01-14T08:32:29Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgTone.java

+    public String getDisplayName() {
+        return displayName;
+    }
+
+    /**
+     * Gets the prompt instruction for this tone.
+     *
+     * @return the prompt instruction to include in ChatGPT prompt
+     */
+    public String getPromptInstruction() {


once you moved that into RewriteCommand as private enum Tone {...}, the getters are obsolete as RewriteCommand can access the fields directly. so go for just:

private enum Tone { CLEAR("Clear", "Make it clear and easy to understand."), PRO("Pro", "Use a professional and polished tone."), ... final String displayName; final String promptInstructions; Tone(String displayName, String promptInstructions) { ... } }

Zabuzard · 2026-01-14T08:32:46Z

application/src/main/java/org/togetherjava/tjbot/features/Features.java

        TopHelpersService topHelpersService = new TopHelpersService(database);
        TopHelpersAssignmentRoutine topHelpersAssignmentRoutine =
                new TopHelpersAssignmentRoutine(config, topHelpersService);
+        RewriteMsgService rewriteService = new RewriteMsgService(chatGptService, helpSystemHelper);


(as commented on earlier, this service shouldnt exist)

…ash-command-rewrite-msg

…view;

sonarqubecloud · 2026-01-16T17:52:47Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

tj-wazei

Please see comments. There's a lot of final abuse that does nothing but add clutter to the code. It's best to use final when you need it.

tj-wazei · 2026-01-16T18:26:52Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+
+/**
+ * The implemented command is {@code /rewrite-msg}, which allows users to have their message
+ * rewritten in a clearer, more professional, or better structured form using ChatGPT AI.


Please remove ChatGPT from the docs as you leak too much implementation details. "using AI" is fine.

tj-wazei · 2026-01-16T18:27:26Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+ * rewritten in a clearer, more professional, or better structured form using ChatGPT AI.
+ * <p>
+ * The rewritten message is shown as an ephemeral message visible only to the user who triggered the
+ * command, making it perfect for getting quick writing improvements without cluttering the channel.


making it perfect for getting quick writing improvements without cluttering the channel. is unnecessary for code documentation.

tj-wazei · 2026-01-16T18:30:02Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+ * The rewritten message is shown as an ephemeral message visible only to the user who triggered the
+ * command, making it perfect for getting quick writing improvements without cluttering the channel.
+ * <p>
+ * Users can optionally specify a tone/style for the rewrite. If not provided, defaults to CLEAR.


defaults to CLEAR does not tell a developer what CLEAR is so please can you change this so it makes sense without coupling it to the enum (that isn't even linked).

tj-wazei · 2026-01-16T18:33:27Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+ * <p>
+ * Users can optionally specify a tone/style for the rewrite. If not provided, defaults to CLEAR.
+ */
+public final class RewriteMsgCommand extends SlashCommandAdapter {


/rewrite is perfectly fine

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

tj-wazei · 2026-01-16T18:44:15Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+        final Optional<String> rewrittenMessage = this.rewrite(userMessage, tone);
+
+        if (rewrittenMessage.isEmpty()) {
+            logger.debug("Failed to obtain a response for /rewrite-msg, original message: '{}'",


do not hard code the name here, instead used the constant you defined COMMAND_NAME

tj-wazei · 2026-01-16T18:45:40Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+            logger.debug("Failed to obtain a response for /rewrite-msg, original message: '{}'",
+                    userMessage);
+            event.getHook()
+                .editOriginal(


This isn't good UX. If the user spent a fair bit of time trying to craft their message and an error occurs, they lose it. Instead, return back the original message - an embed might be useful here to add the fact there was an "error" but the user can still copy their original message and send that, should they wish.

tj-wazei · 2026-01-16T18:48:45Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+            return;
+        }
+
+        final String response = buildResponse(userMessage, rewrittenMessage.orElseThrow(), tone);


Unnecessary use of orElseThrow since it will never throw. You are doing an isEmpty check above.

Ideally, this entire block is a good candidate for rewrittenMessage.ifPresentOrElse() if you want that fluency.

tj-wazei · 2026-01-16T18:49:18Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+    }
+
+    private Optional<String> rewrite(String userMessage, MsgTone tone) {
+        final String rewritePrompt = buildChatGptPrompt(userMessage, tone);


You can inline this String

tj-wazei · 2026-01-16T18:51:54Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+    private Optional<String> rewrite(String userMessage, MsgTone tone) {
+        final String rewritePrompt = buildChatGptPrompt(userMessage, tone);
+
+        return chatGptService.ask(rewritePrompt, tone.displayName, CHAT_GPT_MODEL);


You'll need to update the ask method. The original implementor of the ChatGptService did not make this abstract enough so your request will be mixed with the following prefixed:

For code supplied for review, refer to the old code supplied rather than rewriting the code. DON'T supply a corrected version of the code. KEEP IT CONCISE, NOT MORE THAN 280 WORDS %s Question: %s

Taz03 · 2026-01-17T03:22:40Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+                %s
+
+                **Rewritten:**
+                %s""".formatted(toneLabel, userMessage, rewrittenMessage);


dont forget to String#stripIndent when using multi-line strings

Taz03 · 2026-01-17T03:23:17Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+                **Original:**
+                %s
+
+                **Rewritten:**


better to use discord embed fields

Taz03 · 2026-01-17T03:23:34Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+                If the message is already well-written, provide minor improvements.
+
+                Original message:
+                %s""".formatted(tone.description, userMessage);


dont forget to String#stripIndent when using multi-line strings

Taz03 · 2026-01-17T03:33:51Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+    @Override
+    public void onSlashCommand(SlashCommandInteractionEvent event) {
+        final String userMessage =
+                Objects.requireNonNull(event.getOption(MESSAGE_OPTION)).getAsString();


no need for requireNonNull

Taz03 · 2026-01-17T03:35:31Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+        final OptionData toneOption = new OptionData(OptionType.STRING, TONE_OPTION,
+                "The tone/style for the rewritten message (default: " + MsgTone.CLEAR.displayName
+                        + ")",
+                false);


y not also make it auto-completeable?

Taz03 · 2026-01-17T03:37:03Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+        final String toneLabel = tone.displayName;
+
+        return """
+                **Rewritten message (%s)**


use embed title

Taz03 · 2026-01-17T03:37:29Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+
+                If the message is already well-written, provide minor improvements.
+
+                Original message:


use embed field

Taz03 · 2026-01-17T03:37:52Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+                and better structured. Maintain the original meaning while improving the quality \
+                of the writing. Do NOT use em-dashes (—). %s
+
+                If the message is already well-written, provide minor improvements.


put this in embed description

Taz03 · 2026-01-17T03:40:49Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+        if (toneOption == null) {
+            logger.debug("Tone option not provided for user: {}, using default CLEAR", userId);
+            return MsgTone.CLEAR;
+        }


cannot be null

Taz03 · 2026-01-17T03:43:10Z

application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java

+        event.getHook().editOriginal(response).queue();
+    }
+
+    private MsgTone parseTone(@Nullable OptionMapping toneOption, String userId)


are you taking in the userId just for the logs? seems unnecessary

Feature: Implement /rewrite command for message improvement using Cha…

e4f888c

…tGPT

firasrg requested a review from a team as a code owner January 13, 2026 21:56

Zabuzard requested changes Jan 14, 2026

View reviewed changes

firasrg added 2 commits January 16, 2026 18:47

Merge remote-tracking branch 'origin/develop' into firasrg/feature-sl…

ef2626f

…ash-command-rewrite-msg

feature "rewrite-msg command": applies changes due to Zabu's first re…

5707f50

…view;

tj-wazei requested changes Jan 16, 2026

View reviewed changes

Taz03 requested changes Jan 17, 2026

View reviewed changes



		public RewriteMsgCommand(RewriteMsgService rewriteMsgService) {
		super(COMMAND_NAME, "Rewrite your message in a clearer, more professional form",


		this.rewriteMsgService = rewriteMsgService;

		logger.debug("Initializing RewriteMsgCommand with ChatGptService and HelpSystemHelper");


		If the message is already well-written, provide minor improvements.

		Original message:

Uh oh!

Feature: Implement /rewrite command for message improvement using Cha… #1378

Are you sure you want to change the base?

Feature: Implement /rewrite command for message improvement using Cha… #1378

Conversation

firasrg commented Jan 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

New Feature: Rewrite Command

Usage

Parameters

Available Tones

How It Works

Example

UPDATE

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Jan 16, 2026

Quality Gate passed

Uh oh!

tj-wazei left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

firasrg commented Jan 13, 2026 •

edited

Loading