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?

it is his opinion as code-reviewer. and now we come to a consensus. i am also fine with rewrite but im still proposing to call the class RewriteCommand in that case

Alright, here's the plan:

Rename the class to RewriteCommand

Restore the original name of the command (/rewrite)

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

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

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

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

…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

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

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

+ */
+public final class RewriteMsgCommand extends SlashCommandAdapter {
+    private static final Logger logger = LoggerFactory.getLogger(RewriteMsgCommand.class);
+    public static final String COMMAND_NAME = "rewrite-msg";


COMMAND_NAME does not need to be public

@SquidXTV it sounds like you accidently marked this as resolved

Damn, my bad

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.

FYI discord shows the original prompt on slash commands if u click on the "foo used /bar" that didcord shows in small blue text above the message

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.

to be fair, if (foo.isEmpty()) ... foo.orElseThrow() is the normal way ur supposed to deal with this in case you are not able to utilize the other, better, methods of the API. (but in this case, try to put the extraction of the content right after the if-check, not 20 lines later)

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

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

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

the getOption() return is Nullable based on its docs

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?

indeed, it works with autocomplete already without the need to do anything else in code!

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

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

I think you're confused here, the call of the method is like this this.parseTone(event.getOption(TONE_OPTION), event.getUser().getId());, where the event.getOption(TONE_OPTION) refers to toneOption argument, and the return of getOption() is marked @Nullable based on docs and source

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

Zabuzard

Great stuff. Apart from what the others mentioned, im happy now 👍

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

Zabuzard reviewed Jan 17, 2026

View reviewed changes

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!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

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!

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!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

firasrg commented Jan 13, 2026 •

edited

Loading

firasrg Jan 17, 2026 •

edited

Loading