Fix formatting of code longer than Discord's message limit by Neil-Tomar · Pull Request #549 · Java-Discord/JavaBot

Neil-Tomar · 2026-06-20T15:17:16Z

Description of the Changes

The "Format Code" / "Format and Indent Code" message commands and the /format-code slash command failed when the target message was longer than Discord's 2000-character limit: the whole snippet was wrapped in a single code block and sent as one message, which Discord rejects — so the command silently did nothing.

This PR splits the formatted output into chunks that each stay under the limit:

New Code class splits content into ≤1980-character chunks (breaking on newlines where possible) and renders each as a code block.
The commands acknowledge the interaction by replying with the full, unsplit code as an attached file, then post the chunks in order.
New Language enum + Language.fromString(...) maps the slash command's format option to a code-fence tag.

A follow-up PR will add the language-selection dropdown.

danthe1st

Congratulations on writing your first pull request. I commented on a few things I noticed. Please don't unnecessarily duplicate code.

danthe1st · 2026-06-20T16:19:55Z

+		}
+		channel.sendMessage(messages.get(index))
 				.setAllowedMentions(List.of())
-				.setComponents(FormatCodeCommand.buildActionRow(event.getTarget(), event.getUser().getIdLong()))


The action row including the buttons for deleting and the URL for jumping back are no longer present because you removed them here. Please add them to all messages.

There is a reason for that. The buttons only affect the message they are attached to, so adding them to each message would only allow deletion of that specific message.

Adding buttons to every message would also introduce a visual break inside the code block, which could hurt readability, as shown in the screenshots.

If you'd prefer the buttons to appear only on the last message while still affecting the entire code block, I can look into implementing that. I'm not familiar with that approach yet, but I'm happy to investigate it.

Here is with buttons:

as you can see it's hard to read through indentation.

If it is a single message, both buttons should definitely be there.
If it consists of multiple messages, at least the View Original button should be present in the last message. There could also be a delete button that deletes all codeblock messages on the last message (and the file upload message should also have both buttons).

danthe1st · 2026-06-20T16:27:54Z

+		this.content = content;
+	}
+
+	public String getContent() {


I think getContent() and setLanguage are unused. If so, please remove them.

getContent() is still present.

getContent() in now being used in FormatCodeDispatcher at line 34 and 44.

danthe1st · 2026-06-20T16:40:06Z

Fixes #537

…nto fix/format-code-long-messages # Conflicts: # src/main/java/net/discordjug/javabot/systems/user_commands/format_code/Code.java # src/main/java/net/discordjug/javabot/systems/user_commands/format_code/FormatCodeDispatcher.java # src/main/java/net/discordjug/javabot/systems/user_commands/format_code/Language.java

danthe1st

These are mostly minor comments. Is there any good reason why users would want a file response in addition to the codeblocks?

danthe1st · 2026-06-23T17:34:50Z

+	 */
+	private static final int MAX_SIZE = 1980;
+
+	private Language language;


Do you plan on needing to mutate Language in the follow-up PR you planned? If not, you could make this final or make Code a record.

danthe1st · 2026-06-23T17:39:40Z

+ * downloadable file, then posts it as one or more ordered code-block messages that each respect
+ * Discord's 2000-character limit.
+ */
+public class FormatCodeDispatcher {


This class is only used internally and could be package private.

danthe1st · 2026-06-23T17:45:28Z

+	 * @return an action row containing the delete and "View Original" buttons
+	 */
 	@Contract("_ -> new")
 	static @NotNull ActionRow buildActionRow(@NotNull Message target, long requesterId) {


I guess this could be moved to FormatCodeDispatcher as that's the place for common code but it's fine as-is.

danthe1st · 2026-06-23T17:47:26Z

+
+		MessageChannel channel = target.getChannel();
+
+		event.replyFiles(file)


What is the reason to reply with a file vs just using event.reply... on the first message and skipping the file?

At least if it's only a single chunk, there shouldn't be any file (but I think it's also better without a file with multiple messages).

Make sure to update the Javadoc accordingly.

I tried dropping the file and replying with the first chunk as suggested, but it hits two problems:

Sending the chunks as plain messages with no reply leaves the interaction unacknowledged → "This application did not respond."
Replying with the first chunk does acknowledge it, but the reply renders as its own attributed block, separate from the follow-up messages, so the code gets split visually mid-block.

Replying with the file avoids both: it acknowledges the interaction, and since the reply is a file (not a chunk) the code-block messages stay uniform with no break — plus it's a clean, copyable full version that reads nicely on desktop. The file seemed like the most useful thing to put there.

So, my preference would actually be to keep the file for both the single- and multi-chunk cases — it gives a consistent experience either way. But if you'd rather it was gone, I'm happy to remove it for the single-chunk case (just reply with the one block directly), or if you have a different approach in mind, I'll gladly go with that.

danthe1st · 2026-06-23T17:50:45Z

+			Responses.error(event.getHook(), "There is no code to format in that message.").queue();
+			return;
+		}
+		// Currently we always format as Java. A language dropdown will be added in the future.


That comment is incorrect. The Code can have a language with /format-code.

danthe1st · 2026-06-23T17:53:51Z

+				.setAllowedMentions(List.of());
+
+		if (index == messages.size() - 1) {
+			action.setComponents(buildActionRow(target, event.getUser().getIdLong()));


Please also add the standard delete button if there is only a single message.

danthe1st · 2026-06-23T17:59:52Z

+		if (index >= messages.size()) {
+			return;
+		}
+		var action = channel.sendMessage(messages.get(index))


If you remove the file upload, you can do something like this:

String content = messages.get(index); MessageRequest<?> action = index == 0 ? event.reply(content) : channel.sendMessage(content); action.setAllowedMentions(List.of());

danthe1st · 2026-06-23T18:04:59Z

+ * syntax highlighting inside code blocks.
+ */
+public enum Language {
+	/**


I think you don't need Javadoc comments for the enum constants here (but please check using mvn verify).

danthe1st · 2026-06-23T18:06:24Z

+	/**
+	 * Fallback used when the language is unrecognised; renders as plain text.
+	 */
+	UNKNOWN("Unknown", "txt");


If you are removing the replyFiles, I think you can change the "txt" to "".

fix formatting of code longer than Discord's message limit

7679c4b

Neil-Tomar requested a review from a team as a code owner June 20, 2026 15:17

danthe1st requested changes Jun 20, 2026

View reviewed changes

danthe1st linked an issue Jun 20, 2026 that may be closed by this pull request

Exception formatting code >2k characters #537

Open

add javadocs, and fixed requested changes.

d2f25e6

Neil-Tomar requested a review from danthe1st June 23, 2026 15:30

Neil-Tomar added 2 commits June 23, 2026 22:45

added javadocs, and fixed requested changes, fixed build issue.

1cdeef7

danthe1st requested changes Jun 23, 2026

View reviewed changes


		MessageChannel channel = target.getChannel();

		event.replyFiles(file)

Conversation

Neil-Tomar commented Jun 20, 2026

Description of the Changes

Uh oh!

danthe1st left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

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!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Neil-Tomar Jun 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

danthe1st commented Jun 20, 2026

Uh oh!

danthe1st 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Neil-Tomar Jun 24, 2026 •

edited

Loading