diff options
| author | Sho Sakuma <me@m1sk9.dev> | 2026-08-30 21:11:46 +0900 |
|---|---|---|
| committer | Sho Sakuma <me@m1sk9.dev> | 2026-08-30 21:11:46 +0900 |
| commit | 1c79bee8f3c8fe20582afa96527cd298c043daa5 (patch) | |
| tree | 1c28055f6296d7866ea3916a65a005be2fb1a6bc /website/src/docs/developers | |
| parent | 68052ce335ae11c4cb10a388848b42571bfc65b9 (diff) | |
| download | LunaticChat-1c79bee8f3c8fe20582afa96527cd298c043daa5.tar.gz LunaticChat-1c79bee8f3c8fe20582afa96527cd298c043daa5.tar.bz2 LunaticChat-1c79bee8f3c8fe20582afa96527cd298c043daa5.zip | |
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>
Diffstat (limited to 'website/src/docs/developers')
0 files changed, 0 insertions, 0 deletions
