From 1436186c75dd9cd6c85d60ad359e520ee06c630d Mon Sep 17 00:00:00 2001 From: gavrielc Date: Wed, 4 Mar 2026 18:07:13 +0200 Subject: [PATCH] fix: rename _chatJid to chatJid in onMessage callback The underscore prefix convention signals an unused parameter, but it's now actively used by the sender allowlist logic. Co-Authored-By: Claude Opus 4.6 --- src/index.ts | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/index.ts b/src/index.ts index 8abf660..50d790d 100644 --- a/src/index.ts +++ b/src/index.ts @@ -478,21 +478,21 @@ async function main(): Promise { // Channel callbacks (shared by all channels) const channelOpts = { - onMessage: (_chatJid: string, msg: NewMessage) => { + onMessage: (chatJid: string, msg: NewMessage) => { // Sender allowlist drop mode: discard messages from denied senders before storing if ( !msg.is_from_me && !msg.is_bot_message && - registeredGroups[_chatJid] + registeredGroups[chatJid] ) { const cfg = loadSenderAllowlist(); if ( - shouldDropMessage(_chatJid, cfg) && - !isSenderAllowed(_chatJid, msg.sender, cfg) + shouldDropMessage(chatJid, cfg) && + !isSenderAllowed(chatJid, msg.sender, cfg) ) { if (cfg.logDenied) { logger.debug( - { chatJid: _chatJid, sender: msg.sender }, + { chatJid, sender: msg.sender }, 'sender-allowlist: dropping message (drop mode)', ); }