-
-
Notifications
You must be signed in to change notification settings - Fork 105
Feature: Implement /rewrite command for message improvement using Cha… #1378
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgService.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgTone.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgTone.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgTone.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/Features.java
Outdated
Show resolved
Hide resolved
tj-wazei
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteMsgCommand.java
Outdated
Show resolved
Hide resolved
Zabuzard
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great stuff. Apart from what the others mentioned, im happy now 👍
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
tj-wazei
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there, I still really hate the usage of final, my eyes kept having to "jump" as I was reading your code.
application/src/main/java/org/togetherjava/tjbot/features/chatgpt/ChatGptService.java
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Show resolved
Hide resolved
89285c6 to
0604f9b
Compare
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
application/src/main/java/org/togetherjava/tjbot/features/messages/RewriteCommand.java
Outdated
Show resolved
Hide resolved
|
tj-wazei
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work.



New Feature: Rewrite Command
The
/rewritecommand 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
Parameters
CLEARif not providedAvailable Tones
How It Works
Example
UPDATE 16 Jan 2026
/rewrite-msgfor consistency with the class in source code.UPDATE 20 Jan 2026
/rewrite.Prohas been renamed toProfessionnalChatGptServiceAPI,askRawto avoid touching existing method and for future reusabilities