Added /change-help-title command

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

@@ -111,6 +111,7 @@ public enum Features {
         features.add(new AskCommand(config, helpSystemHelper));
         features.add(new CloseCommand(helpSystemHelper));
         features.add(new ChangeHelpCategoryCommand(config, helpSystemHelper));
+        features.add(new ChangeHelpTitleCommand(helpSystemHelper));
 
         // Mixtures
         features.add(new HelpThreadOverviewUpdater(config, helpSystemHelper));

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

@@ -18,6 +18,7 @@
 
 import java.time.Instant;
 import java.time.temporal.ChronoUnit;
+import java.util.Locale;
 import java.util.Optional;
 import java.util.concurrent.TimeUnit;
 
@@ -34,8 +35,8 @@
 public final class ChangeHelpCategoryCommand extends SlashCommandAdapter {
     private static final String CATEGORY_OPTION = "category";
 
-    private static final int COOLDOWN_DURATION_VALUE = 1;
-    private static final ChronoUnit COOLDOWN_DURATION_UNIT = ChronoUnit.HOURS;
+    private static final int COOLDOWN_DURATION_VALUE = 30;
+    private static final ChronoUnit COOLDOWN_DURATION_UNIT = ChronoUnit.MINUTES;
 
     private final HelpSystemHelper helper;
     private final Cache<Long, Instant> helpThreadIdToLastCategoryChange;
@@ -82,8 +83,9 @@ public void onSlashCommand(@NotNull SlashCommandInteractionEvent event) {
 
         if (isHelpThreadOnCooldown(helpThread)) {
             event
-                .reply("Please wait a bit, this command can only be used once per %d %s."
-                    .formatted(COOLDOWN_DURATION_VALUE, COOLDOWN_DURATION_UNIT))
+                .reply("Please wait a bit, this command can only be used once per %d %s.".formatted(
+                        COOLDOWN_DURATION_VALUE,
+                        COOLDOWN_DURATION_UNIT.toString().toLowerCase(Locale.US)))
                 .setEphemeral(true)
                 .queue();
             return;
@@ -92,7 +94,7 @@ public void onSlashCommand(@NotNull SlashCommandInteractionEvent event) {
 
         event.deferReply().queue();
 
-        helper.renameChannelToCategoryTitle(helpThread, category)
+        helper.renameChannelToCategory(helpThread, category)
             .flatMap(any -> sendCategoryChangedMessage(helpThread.getGuild(), event.getHook(),
                     helpThread, category))
             .queue();

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

@@ -0,0 +1,105 @@
+package org.togetherjava.tjbot.commands.help;
+
+import com.github.benmanes.caffeine.cache.Cache;
+import com.github.benmanes.caffeine.cache.Caffeine;
+import net.dv8tion.jda.api.entities.Guild;
+import net.dv8tion.jda.api.entities.Message;
+import net.dv8tion.jda.api.entities.ThreadChannel;
+import net.dv8tion.jda.api.events.interaction.command.SlashCommandInteractionEvent;
+import net.dv8tion.jda.api.interactions.InteractionHook;
+import net.dv8tion.jda.api.interactions.commands.OptionType;
+import net.dv8tion.jda.api.requests.RestAction;
+import org.jetbrains.annotations.NotNull;
+import org.togetherjava.tjbot.commands.SlashCommandAdapter;
+import org.togetherjava.tjbot.commands.SlashCommandVisibility;
+
+import java.time.Instant;
+import java.time.temporal.ChronoUnit;
+import java.util.Locale;
+import java.util.Optional;
+import java.util.concurrent.TimeUnit;
+
+/**
+ * Implements the {@code /change-help-title} command, which is able to change the title of a help
+ * thread.
+ * <p>
+ * This is to adjust a bad title in hindsight, for example if it was automatically created by
+ * {@link ImplicitAskListener}.
+ */
+public final class ChangeHelpTitleCommand extends SlashCommandAdapter {
+    private static final String TITLE_OPTION = "title";
+
+    private static final int COOLDOWN_DURATION_VALUE = 30;
+    private static final ChronoUnit COOLDOWN_DURATION_UNIT = ChronoUnit.MINUTES;
+
+    private final HelpSystemHelper helper;
+    private final Cache<Long, Instant> helpThreadIdToLastTitleChange;
+
+    /**
+     * Creates a new instance.
+     *
+     * @param helper the helper to use
+     */
+    public ChangeHelpTitleCommand(@NotNull HelpSystemHelper helper) {
+        super("change-help-title", "changes the title of a help thread",
+                SlashCommandVisibility.GUILD);
+
+        getData().addOption(OptionType.STRING, TITLE_OPTION, "short and to the point", true);
+
+        helpThreadIdToLastTitleChange = Caffeine.newBuilder()
+            .maximumSize(1_000)
+            .expireAfterAccess(COOLDOWN_DURATION_VALUE, TimeUnit.of(COOLDOWN_DURATION_UNIT))
+            .build();
+
+        this.helper = helper;
+    }
+
+    @Override
+    public void onSlashCommand(@NotNull SlashCommandInteractionEvent event) {
+        String title = event.getOption(TITLE_OPTION).getAsString();
+
+        if (!helper.handleIsHelpThread(event)) {
+            return;
+        }
+
+        ThreadChannel helpThread = event.getThreadChannel();
+        if (helpThread.isArchived()) {
+            event.reply("This thread is already closed.").setEphemeral(true).queue();
+            return;
+        }
+
+        if (isHelpThreadOnCooldown(helpThread)) {
+            event
+                .reply("Please wait a bit, this command can only be used once per %d %s.".formatted(
+                        COOLDOWN_DURATION_VALUE,
+                        COOLDOWN_DURATION_UNIT.toString().toLowerCase(Locale.US)))
+                .setEphemeral(true)
+                .queue();
+            return;
+        }
+        helpThreadIdToLastTitleChange.put(helpThread.getIdLong(), Instant.now());
+
+        event.deferReply().queue();
+
+        helper.renameChannelToTitle(helpThread, title)
+            .flatMap(any -> sendTitleChangedMessage(helpThread.getGuild(), event.getHook(),
+                    helpThread, title))
+            .queue();
+    }
+
+    private @NotNull RestAction<Message> sendTitleChangedMessage(@NotNull Guild guild,
+            @NotNull InteractionHook hook, @NotNull ThreadChannel helpThread,
+            @NotNull String title) {
+        String changedContent = "Changed the title to **%s**.".formatted(title);
+        return hook.editOriginal(changedContent);
+    }
+
+    private boolean isHelpThreadOnCooldown(@NotNull ThreadChannel helpThread) {
+        return Optional
+            .ofNullable(helpThreadIdToLastTitleChange.getIfPresent(helpThread.getIdLong()))
+            .map(lastCategoryChange -> lastCategoryChange.plus(COOLDOWN_DURATION_VALUE,
+                    COOLDOWN_DURATION_UNIT))
+            .filter(Instant.now()::isBefore)
+            .isPresent();
+    }
+}

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

@@ -136,30 +136,24 @@ Optional<Role> handleFindRoleForCategory(@NotNull String category, @NotNull Guil
 
     @NotNull
     Optional<String> getCategoryOfChannel(@NotNull Channel channel) {
-        Matcher matcher = EXTRACT_CATEGORY_TITLE_PATTERN.matcher(channel.getName());
-        if (!matcher.find()) {
-            return Optional.empty();
-        }
-
-        return Optional.ofNullable(matcher.group(CATEGORY_GROUP));
+        return Optional.ofNullable(HelpThreadName.ofChannelName(channel.getName()).category);
     }
 
     @NotNull
-    RestAction<Void> renameChannelToCategoryTitle(@NotNull GuildChannel channel,
+    RestAction<Void> renameChannelToCategory(@NotNull GuildChannel channel,
             @NotNull String category) {
-        String currentTitle = channel.getName();
-        Matcher matcher = EXTRACT_CATEGORY_TITLE_PATTERN.matcher(currentTitle);
+        HelpThreadName currentName = HelpThreadName.ofChannelName(channel.getName());
+        HelpThreadName changedName = new HelpThreadName(category, currentName.title);
 
-        if (!matcher.matches()) {
-            throw new AssertionError("Pattern must match any thread name");
-        }
-        boolean hasCategoryInTitle = matcher.group(CATEGORY_GROUP) != null;
-        String titleWithoutCategory =
-                hasCategoryInTitle ? matcher.group(TITLE_GROUP) : currentTitle;
+        return channel.getManager().setName(changedName.toChannelName());
+    }
 
-        String titleWithCategory = "[%s] %s".formatted(category, titleWithoutCategory);
+    @NotNull
+    RestAction<Void> renameChannelToTitle(@NotNull GuildChannel channel, @NotNull String title) {
+        HelpThreadName currentName = HelpThreadName.ofChannelName(channel.getName());
+        HelpThreadName changedName = new HelpThreadName(currentName.category, title);
 
-        return channel.getManager().setName(titleWithCategory);
+        return channel.getManager().setName(changedName.toChannelName());
     }
 
     boolean isOverviewChannelName(@NotNull String channelName) {
@@ -186,4 +180,20 @@ static boolean isTitleValid(@NotNull CharSequence title) {
         return titleCompact.length() >= TITLE_COMPACT_LENGTH_MIN
                 && titleCompact.length() <= TITLE_COMPACT_LENGTH_MAX;
     }
+
+    private record HelpThreadName(@Nullable String category, @NotNull String title) {
+        static @NotNull HelpThreadName ofChannelName(@NotNull CharSequence channelName) {
+            Matcher matcher = EXTRACT_CATEGORY_TITLE_PATTERN.matcher(channelName);
+
+            if (!matcher.matches()) {
+                throw new AssertionError("Pattern must match any thread name");
+            }
+
+            return new HelpThreadName(matcher.group(CATEGORY_GROUP), matcher.group(TITLE_GROUP));
+        }
+
+        public @NotNull String toChannelName() {
+            return category == null ? title : "[%s] %s".formatted(category, title);
+        }
+    }
 }