Release v3.1

application/config.json.template

@@ -8,7 +8,6 @@
     "heavyModerationRolePattern": "Moderator",
     "softModerationRolePattern": "Moderator|Staff Assistant",
     "tagManageRolePattern": "Moderator|Staff Assistant|Top Helpers .+",
-   "helpChannelPattern": "([a-zA-Z_]+_)?help(_\\d+)?",
    "suggestions": {
        "channelPattern": "tj_suggestions",
        "upVoteEmoteName": "peepo_yes",

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

@@ -17,6 +17,7 @@
 import org.togetherjava.tjbot.commands.reminder.RemindCommand;
 import org.togetherjava.tjbot.commands.reminder.RemindRoutine;
 import org.togetherjava.tjbot.commands.system.BotCore;
+import org.togetherjava.tjbot.commands.system.LogLevelCommand;
 import org.togetherjava.tjbot.commands.tags.TagCommand;
 import org.togetherjava.tjbot.commands.tags.TagManageCommand;
 import org.togetherjava.tjbot.commands.tags.TagSystem;
@@ -85,6 +86,7 @@ public enum Features {
         features.add(new RejoinModerationRoleListener(actionsStore, config));
 
         // Slash commands
+        features.add(new LogLevelCommand());
         features.add(new PingCommand());
         features.add(new TeXCommand());
         features.add(new TagCommand(tagSystem));

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

@@ -109,7 +109,7 @@ private boolean handleIsValidTitle(@NotNull CharSequence title, @NotNull IReplyC
         }
 
         event.reply(
-                "Sorry, but the titel length (after removal of special characters) has to be between %d and %d."
+                "Sorry, but the title 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();

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

@@ -108,11 +108,14 @@ private static boolean shouldMessageBeCleanedUp(@NotNull Message message) {
 
     private static @NotNull RestAction<Void> cleanupBotMessages(
             @NotNull GuildMessageChannel channel, @NotNull Collection<? extends Message> messages) {
+        logger.debug("Cleaning up old bot messages in the staging channel");
         List<String> messageIdsToDelete = messages.stream()
             .filter(BotMessageCleanup::shouldMessageBeCleanedUp)
             .map(Message::getId)
             .toList();
 
+        logger.debug("Found {} messages to delete", messageIdsToDelete.size());
+
         if (messageIdsToDelete.isEmpty()) {
             return new CompletedRestAction<>(channel.getJDA(), null, null);
         }

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

@@ -85,7 +85,7 @@ RestAction<Message> sendExplanationMessage(@NotNull MessageChannel threadChannel
                                 If nobody is calling back, that usually means that your question was **not well asked** and \
                                     hence nobody feels confident enough answering. Try to use your time to elaborate, \
                                     **provide details**, context, more code, examples and maybe some screenshots. \
-                                    With enough info, someone knows the answer for sure ."""));
+                                    With enough info, someone knows the answer for sure."""));
 
         MessageAction action = threadChannel.sendMessage(message);
         if (useCodeSyntaxExampleImage) {

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

@@ -32,6 +32,7 @@ public final class HelpThreadOverviewUpdater extends MessageReceiverAdapter impl
     private static final Logger logger = LoggerFactory.getLogger(HelpThreadOverviewUpdater.class);
 
     private static final String STATUS_TITLE = "Active questions";
+    private static final int OVERVIEW_QUESTION_LIMIT = 150;
 
     private final HelpSystemHelper helper;
     private final List<String> allCategories;
@@ -121,17 +122,22 @@ private void updateOverviewForGuild(@NotNull Guild guild) {
 
     private void updateOverview(@NotNull IThreadContainer stagingChannel,
             @NotNull MessageChannel overviewChannel) {
+        logger.debug("Updating overview of active questions");
+
         List<ThreadChannel> activeThreads = stagingChannel.getThreadChannels()
             .stream()
             .filter(Predicate.not(ThreadChannel::isArchived))
             .toList();
 
+        logger.debug("Found {} active questions", activeThreads.size());
+
         MessageEmbed embed = new EmbedBuilder().setTitle(STATUS_TITLE)
             .setDescription(createDescription(activeThreads))
             .setColor(HelpSystemHelper.AMBIENT_COLOR)
             .build();
 
         getStatusMessage(overviewChannel).flatMap(maybeStatusMessage -> {
+            logger.debug("Sending the updated question overview");
             if (maybeStatusMessage.isEmpty()) {
                 return overviewChannel.sendMessageEmbeds(embed);
             }
@@ -148,6 +154,7 @@ private void updateOverview(@NotNull IThreadContainer stagingChannel,
 
         return activeThreads.stream()
             .sorted(Comparator.comparing(ThreadChannel::getTimeCreated).reversed())
+            .limit(OVERVIEW_QUESTION_LIMIT)
             .collect(Collectors
                 .groupingBy(thread -> helper.getCategoryOfChannel(thread).orElse("Uncategorized")))
             .entrySet()

application/src/main/java/org/togetherjava/tjbot/commands/system/BotCore.java

@@ -92,14 +92,19 @@ public BotCore(@NotNull JDA jda, @NotNull Database database, @NotNull Config con
             .filter(Routine.class::isInstance)
             .map(Routine.class::cast)
             .forEach(routine -> {
+                Runnable command = () -> {
+                    String routineName = routine.getClass().getSimpleName();
+                    logger.debug("Running routine %s...".formatted(routineName));
+                    routine.runRoutine(jda);
+                    logger.debug("Finished routine %s.".formatted(routineName));
+                };
+
                 Routine.Schedule schedule = routine.createSchedule();
                 switch (schedule.mode()) {
-                    case FIXED_RATE -> ROUTINE_SERVICE.scheduleAtFixedRate(
-                            () -> routine.runRoutine(jda), schedule.initialDuration(),
-                            schedule.duration(), schedule.unit());
-                    case FIXED_DELAY -> ROUTINE_SERVICE.scheduleWithFixedDelay(
-                            () -> routine.runRoutine(jda), schedule.initialDuration(),
-                            schedule.duration(), schedule.unit());
+                    case FIXED_RATE -> ROUTINE_SERVICE.scheduleAtFixedRate(command,
+                            schedule.initialDuration(), schedule.duration(), schedule.unit());
+                    case FIXED_DELAY -> ROUTINE_SERVICE.scheduleWithFixedDelay(command,
+                            schedule.initialDuration(), schedule.duration(), schedule.unit());
                     default -> throw new AssertionError("Unsupported schedule mode");
                 }
             });

application/src/main/java/org/togetherjava/tjbot/commands/system/LogLevelCommand.java

@@ -0,0 +1,61 @@
+package org.togetherjava.tjbot.commands.system;
+
+import net.dv8tion.jda.api.events.interaction.command.SlashCommandInteractionEvent;
+import net.dv8tion.jda.api.interactions.commands.OptionType;
+import net.dv8tion.jda.api.interactions.commands.build.OptionData;
+import org.apache.logging.log4j.Level;
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.core.config.Configurator;
+import org.jetbrains.annotations.NotNull;
+import org.togetherjava.tjbot.commands.SlashCommandAdapter;
+import org.togetherjava.tjbot.commands.SlashCommandVisibility;
+
+import java.util.stream.Stream;
+
+/**
+ * Implements the '/set-log-level' command which can be used to change the log level used by the
+ * bot, while it is running.
+ * <p>
+ * Example usage:
+ *
+ * <pre>
+ * {@code
+ * /set-log-level level: INFO
+ * }
+ * </pre>
+ */
+public final class LogLevelCommand extends SlashCommandAdapter {
+    private static final String LOG_LEVEL_OPTION = "level";
+
+    /**
+     * Creates a new instance.
+     */
+    public LogLevelCommand() {
+        super("set-log-level", "Changes the log level of the bot while it is running.",
+                SlashCommandVisibility.GUILD);
+
+        OptionData option =
+                new OptionData(OptionType.STRING, LOG_LEVEL_OPTION, "the log level to set", true);
+        Stream.of(Level.values()).map(Level::name).forEach(level -> option.addChoice(level, level));
+
+        getData().addOptions(option);
+    }
+
+    // Security warning about changing log configs. We only change the level, that is safe.
+    @SuppressWarnings("squid:S4792")
+    @Override
+    public void onSlashCommand(@NotNull SlashCommandInteractionEvent event) {
+        String levelText = event.getOption(LOG_LEVEL_OPTION).getAsString();
+        Level level = Level.getLevel(levelText);
+
+        if (level == null) {
+            event.reply("The selected log level '%s' is unknown.".formatted(levelText))
+                .setEphemeral(true)
+                .queue();
+            return;
+        }
+
+        Configurator.setAllLevels(LogManager.getRootLogger().getName(), level);
+        event.reply("Set the log level to '%s'.".formatted(levelText)).queue();
+    }
+}

application/src/main/java/org/togetherjava/tjbot/config/Config.java

@@ -22,7 +22,6 @@ public final class Config {
     private final String heavyModerationRolePattern;
     private final String softModerationRolePattern;
     private final String tagManageRolePattern;
-    private final String helpChannelPattern;
     private final SuggestionsConfig suggestions;
     private final String quarantinedRolePattern;
     private final ScamBlockerConfig scamBlocker;
@@ -40,7 +39,6 @@ private Config(@JsonProperty("token") String token,
             @JsonProperty("heavyModerationRolePattern") String heavyModerationRolePattern,
             @JsonProperty("softModerationRolePattern") String softModerationRolePattern,
             @JsonProperty("tagManageRolePattern") String tagManageRolePattern,
-            @JsonProperty("helpChannelPattern") String helpChannelPattern,
             @JsonProperty("suggestions") SuggestionsConfig suggestions,
             @JsonProperty("quarantinedRolePattern") String quarantinedRolePattern,
             @JsonProperty("scamBlocker") ScamBlockerConfig scamBlocker,
@@ -55,7 +53,6 @@ private Config(@JsonProperty("token") String token,
         this.heavyModerationRolePattern = heavyModerationRolePattern;
         this.softModerationRolePattern = softModerationRolePattern;
         this.tagManageRolePattern = tagManageRolePattern;
-        this.helpChannelPattern = helpChannelPattern;
         this.suggestions = suggestions;
         this.quarantinedRolePattern = quarantinedRolePattern;
         this.scamBlocker = scamBlocker;
@@ -160,16 +157,6 @@ public String getTagManageRolePattern() {
         return tagManageRolePattern;
     }
 
-    /**
-     * Gets the REGEX pattern used to identify channels that are used for helping people with their
-     * questions.
-     *
-     * @return the channel name pattern
-     */
-    public @NotNull String getHelpChannelPattern() {
-        return helpChannelPattern;
-    }
-
     /**
      * Gets the config for the suggestion system.
      *

settings.gradle

@@ -3,4 +3,6 @@ rootProject.name = 'TJ-Bot'
 include 'application'
 include 'database'
 include 'formatter'
-include 'logviewer'
+// NOTE The logviewer does not properly work as of today.
+//  But it is causing major build slowdowns, so we exclude it for the time being.
+// include 'logviewer'