Fixed some bugs revolving around bad titles

* /ask command with too long title
* implicit ask with bad title (file upload or invisible spaces, ...)

Author: Zabuzard

application/src/main/java/org/togetherjava/tjbot/commands/help/AskCommand.java

@@ -16,18 +16,21 @@
 import org.togetherjava.tjbot.commands.SlashCommandVisibility;
 import org.togetherjava.tjbot.config.Config;
 
+import static org.togetherjava.tjbot.commands.help.HelpSystemHelper.TITLE_COMPACT_LENGTH_MAX;
+import static org.togetherjava.tjbot.commands.help.HelpSystemHelper.TITLE_COMPACT_LENGTH_MIN;
+
 /**
  * Implements the {@code /ask} command, which is the main way of asking questions. The command can
  * only be used in the staging channel.
- *
+ * <p>
  * Upon use, it will create a new thread for the question and invite all helpers interested in the
  * given category to it. It will also introduce the user to the system and give a quick explanation
  * message.
- *
+ * <p>
  * The other way to ask questions is by {@link ImplicitAskListener}.
- *
+ * <p>
  * Example usage:
- * 
+ *
  * <pre>
  * {@code
  * /ask title: How to send emails? category: Frameworks
@@ -76,6 +79,10 @@ public void onSlashCommand(@NotNull SlashCommandInteractionEvent event) {
             return;
         }
 
+        if (!handleIsValidTitle(title, event)) {
+            return;
+        }
+
         TextChannel helpStagingChannel = event.getTextChannel();
         helpStagingChannel.createThreadChannel("[%s] %s".formatted(category, title))
             .flatMap(threadChannel -> handleEvent(event, threadChannel, event.getMember(), title,
@@ -96,6 +103,20 @@ private boolean handleIsStagingChannel(@NotNull IReplyCallback event) {
         return false;
     }
 
+    private boolean handleIsValidTitle(@NotNull CharSequence title, @NotNull IReplyCallback event) {
+        if (HelpSystemHelper.isTitleValid(title)) {
+            return true;
+        }
+
+        event.reply(
+                "Sorry, but the titel length (after removal of special characters) has to be between %d and %d."
+                    .formatted(TITLE_COMPACT_LENGTH_MIN, TITLE_COMPACT_LENGTH_MAX))
+            .setEphemeral(true)
+            .queue();
+
+        return false;
+    }
+
     private @NotNull RestAction<Message> handleEvent(@NotNull IReplyCallback event,
             @NotNull ThreadChannel threadChannel, @NotNull Member author, @NotNull String title,
             @NotNull String category) {

application/src/main/java/org/togetherjava/tjbot/commands/help/HelpSystemHelper.java

@@ -34,6 +34,10 @@ public final class HelpSystemHelper {
     private static final Pattern EXTRACT_CATEGORY_TITLE_PATTERN =
             Pattern.compile("(?:\\[(?<%s>.+)] )?(?<%s>.+)".formatted(CATEGORY_GROUP, TITLE_GROUP));
 
+    private static final Pattern TITLE_COMPACT_REMOVAL_PATTERN = Pattern.compile("\\W");
+    static final int TITLE_COMPACT_LENGTH_MIN = 2;
+    static final int TITLE_COMPACT_LENGTH_MAX = 80;
+
     private final Predicate<String> isOverviewChannelName;
     private final String overviewChannelPattern;
     private final Predicate<String> isStagingChannelName;
@@ -175,4 +179,11 @@ boolean isStagingChannelName(@NotNull String channelName) {
     String getStagingChannelPattern() {
         return stagingChannelPattern;
     }
+
+    static boolean isTitleValid(@NotNull CharSequence title) {
+        String titleCompact = TITLE_COMPACT_REMOVAL_PATTERN.matcher(title).replaceAll("");
+
+        return titleCompact.length() >= TITLE_COMPACT_LENGTH_MIN
+                && titleCompact.length() <= TITLE_COMPACT_LENGTH_MAX;
+    }
 }

application/src/main/java/org/togetherjava/tjbot/commands/help/ImplicitAskListener.java

@@ -27,11 +27,11 @@
  * <p>
  * Listens to plain messages in the staging channel, picks them up and transfers them into a proper
  * question thread.
- *
+ * <p>
  * The system can handle spam appropriately and will not create multiple threads for each message.
- *
+ * <p>
  * For example:
- * 
+ *
  * <pre>
  * {@code
  * John sends: How to send emails?
@@ -128,17 +128,22 @@ private Optional<HelpThread> getLastHelpThreadIfOnCooldown(long userId) {
     }
 
     private static @NotNull String createTitle(@NotNull String message) {
+        String titleCandidate;
         if (message.length() < TITLE_MAX_LENGTH) {
-            return message;
-        }
-        // Attempt to end at the last word before hitting the limit
-        // e.g. "[foo bar] baz" for a limit somewhere in between "baz"
-        int lastWordEnd = message.lastIndexOf(' ', TITLE_MAX_LENGTH);
-        if (lastWordEnd == -1) {
-            lastWordEnd = TITLE_MAX_LENGTH;
+            titleCandidate = message;
+        } else {
+            // Attempt to end at the last word before hitting the limit
+            // e.g. "[foo bar] baz" for a limit somewhere in between "baz"
+            int lastWordEnd = message.lastIndexOf(' ', TITLE_MAX_LENGTH);
+            if (lastWordEnd == -1) {
+                lastWordEnd = TITLE_MAX_LENGTH;
+            }
+
+            titleCandidate = message.substring(0, lastWordEnd);
         }
 
-        return message.substring(0, lastWordEnd);
+        return HelpSystemHelper.isTitleValid(titleCandidate) ? titleCandidate : "Untitled";
+
     }
 
     private @NotNull RestAction<?> handleEvent(@NotNull ThreadChannel threadChannel,