From e668b9b410ac2d2bf5b8d66da64ccfe8b653854b Mon Sep 17 00:00:00 2001 From: Rin Date: Sun, 15 Feb 2026 16:32:47 +0800 Subject: [PATCH] fix(minecraft): nullify EventRegistry context on stop() to prevent stale handler execution When stop() is called before detachFromBot(), mineflayer listeners remain attached. Nullifying context ensures handleMineflayerEvent's guard clause silently ignores any events that fire in this window. --- services/minecraft/src/cognitive/perception/events/index.ts | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/services/minecraft/src/cognitive/perception/events/index.ts b/services/minecraft/src/cognitive/perception/events/index.ts index f1ad37280..8b477c9b5 100644 --- a/services/minecraft/src/cognitive/perception/events/index.ts +++ b/services/minecraft/src/cognitive/perception/events/index.ts @@ -96,6 +96,10 @@ export class EventRegistry { clearInterval(this.timer) this.timer = null } + // NOTICE: Nullify context so that any stale mineflayer listeners that fire + // after stop() (but before detachFromBot()) are silently ignored by the + // guard at the top of handleMineflayerEvent. + this.context = null } public getDebugSnapshot(): SaliencySnapshot {