fix(paper): stop the command category logging on every tree build

Three findings from review of #296.

Brigadier evaluates a `requires` predicate when it builds the command tree it
sends to a player - on every join and every updateCommands() - not when someone
is refused. The denial line sitting there put one entry per gated node per
player in the log, burying the command results the category exists to show. It
is dropped rather than moved: a hidden node is never reached by an execution, so
there is no refusal path to log from.

`all` was honoured by `debug: all` and `/lc debug all on` but not by
`categories: [all]`, which resolved to an unknown name - leaving `enabled: true`
with nothing switched on, reported as off by /lc status, explained only by a
warning in the log. The word now lives in one place that all three read.

Two dumps in the same second left one file, because the name is only accurate to
the second and the write replaces. Reproducing a bug means dumping either side of
the step that triggers it, so the second one now takes a suffix.

Co-Authored-By: Claude <noreply@anthropic.com>

2 files changed, 15 insertions, 1 deletions

diff --git a/engine/src/main/kotlin/dev/m1sk9/lunaticChat/engine/debug/DebugCategories.kt b/engine/src/main/kotlin/dev/m1sk9/lunaticChat/engine/debug/DebugCategories.kt
index 3231295..c3d7d95 100644
--- a/engine/src/main/kotlin/dev/m1sk9/lunaticChat/engine/debug/DebugCategories.kt
+++ b/engine/src/main/kotlin/dev/m1sk9/lunaticChat/engine/debug/DebugCategories.kt
@@ -19,7 +19,10 @@ object DebugCategories {
         val active: Set<DebugCategory> get() = if (enabled) categories else emptySet()
     }
 
-    private val allWords = setOf("true", "yes", "on", "y", "all")
+    /** The word that stands for every category at once, wherever a category may be named. */
+    const val ALL = "all"
+
+    private val allWords = setOf("true", "yes", "on", "y", ALL)
     private val noneWords = setOf("false", "no", "off", "n", "none")
 
     /**
@@ -48,6 +51,14 @@ object DebugCategories {
             .map { it.trim() }
             .filter { it.isNotEmpty() }
             .forEach { name ->
+                // [ALL] is honoured here as well as in [parse] and /lc debug: an operator who has
+                // read `debug: all` or `/lc debug all on` writes `categories: [all]` next, and
+                // reporting that as an unknown name would leave them with a switch that logs
+                // nothing while saying it is on.
+                if (name.equals(ALL, ignoreCase = true)) {
+                    categories += DebugCategory.entries
+                    return@forEach
+                }
                 val category = DebugCategory.fromKey(name)
                 if (category == null) unknown += name else categories += category
             }
diff --git a/engine/src/main/kotlin/dev/m1sk9/lunaticChat/engine/debug/DebugCategory.kt b/engine/src/main/kotlin/dev/m1sk9/lunaticChat/engine/debug/DebugCategory.kt
index 9c12a92..40cc0e8 100644
--- a/engine/src/main/kotlin/dev/m1sk9/lunaticChat/engine/debug/DebugCategory.kt
+++ b/engine/src/main/kotlin/dev/m1sk9/lunaticChat/engine/debug/DebugCategory.kt
@@ -23,6 +23,9 @@ enum class DebugCategory(
     companion object {
         private val byKey = entries.associateBy { it.key }
 
+        /** Every [key], for the "known categories are ..." half of a message about a typo. */
+        val keyList: String = entries.joinToString(", ") { it.key }
+
         /** The category [key] names, or null when no category is spelled that way. */
         fun fromKey(key: String): DebugCategory? = byKey[key.trim().lowercase()]
     }