@patricktobias86/node-red-telegram-account 1.2.1 → 1.2.3

package/CHANGELOG.md

@@ -2,10 +2,18 @@
 
 All notable changes to this project will be documented in this file.
 
+## [1.2.2] - 2026-03-07
+### Added
+- Added guarded checks before the Telegram client is called to avoid errors when the client is not available.
+
 ## [1.2.1] - 2026-01-22
 ### Added
 - Receiver node option to disable emitting edited message updates (useful to prevent duplicate outputs when counters/markup change on channel posts).
 
+## [1.2.2] - 2026-01-31
+### Fixed
+- Receiver node chat/sender filters now work (include/exclude chats and senders).
+
 ## [1.2.0] - 2026-01-12
 ### Changed
 - Switched underlying Telegram MTProto client dependency from `telegram` (GramJS) to `teleproto`.

package/nodes/command.js

@@ -46,6 +46,11 @@ module.exports = function (RED) {
         }
     };
 
+    if (!client) {
+      node.error('No Telegram client available. Check account configuration.');
+      return;
+    }
+
     try {
       client.addEventHandler(handler, event);
     } catch (err) {

package/nodes/delete-message.js

@@ -18,6 +18,11 @@ module.exports = function (RED) {
               /** @type {TelegramClient} */
             const client = msg.payload?.client ? msg.payload.client : this.config.client;
 
+            if (!client) {
+                node.error('No Telegram client available. Check account configuration.');
+                return;
+            }
+
             try {
                 const response = await client.deleteMessages(chatId, messageIds, { revoke });
 

package/nodes/get-entity.js

@@ -14,6 +14,11 @@ module.exports = function (RED) {
               /** @type {TelegramClient} */
             const client = msg.payload?.client ? msg.payload.client : this.config.client;
 
+            if (!client) {
+                node.error('No Telegram client available. Check account configuration.');
+                return;
+            }
+
             try {
                 let entity;
 

package/nodes/iter-dialogs.js

@@ -15,6 +15,12 @@ module.exports = function (RED) {
     
             /** @type {TelegramClient} */
             const client = msg.payload?.client ? msg.payload.client : this.config.client;
+
+            if (!client) {
+                node.error('No Telegram client available. Check account configuration.');
+                return;
+            }
+
             const limit = msg.payload.limit || config.limit;
             const offsetDate = msg.payload.offsetDate || config.offsetDate;
             const offsetId = msg.payload.offsetId || config.offsetId;

package/nodes/iter-messages.js

@@ -15,6 +15,12 @@ module.exports = function (RED) {
     
             /** @type {TelegramClient} */
             const client = msg.payload?.client ? msg.payload.client : this.config.client;
+
+            if (!client) {
+                node.error('No Telegram client available. Check account configuration.');
+                return;
+            }
+
             const chatId = msg.payload?.chatId ? msg.payload.chatId : config.chatId;
             let peerId = chatId === "me" ? chatId : utils.parseID(chatId);
 

package/nodes/promote-admin.js

@@ -15,6 +15,12 @@ module.exports = function (RED) {
                 node.log('promote-admin input: ' + JSON.stringify(msg));
             }
             const client = msg.payload?.client || this.config.client;
+
+            if (!client) {
+                node.error('No Telegram client available. Check account configuration.');
+                return;
+            }
+
             const chatId = msg.payload.chatId || config.chatId;
             const userId = msg.payload.userId || config.userId;
             const rank = msg.payload.rank || config.rank || "admin";

package/nodes/receiver.html

@@ -218,7 +218,7 @@
     </dl>
 
     <h3>Details</h3>
-    <p>The <b>receiver</b> node uses the Telegram client to listen for all new messages in real-time. It emits a message to the next connected Node-RED node whenever a new Telegram message is received, provided the sender's user ID is not in the ignore list.</p>
+    <p>The <b>receiver</b> node uses the Telegram client to listen for all new messages in real-time. It emits a message to the next connected Node-RED node whenever a new Telegram message is received, provided it passes the configured filters.</p>
 
     <h3>Example</h3>
     <pre>
@@ -256,7 +256,9 @@
     <h3>Notes</h3>
     <ul>
         <li>Ensure the Telegram bot has sufficient permissions to receive messages in the configured chat or channel.</li>
-        <li>The <b>Ignore List</b> only filters messages based on the sender's user ID.</li>
+        <li>The <b>Ignore List</b> only filters messages where the sender can be identified as a user.</li>
+        <li><b>Include/Exclude chats</b> match against <code>payload.chatId</code>.</li>
+        <li><b>Include/Exclude senders</b> match against <code>payload.senderId</code> (user/chat/channel IDs depending on the update).</li>
         <li>Use <b>Ignore message types</b> to drop updates that contain media you don't want to process, such as large videos or documents.</li>
         <li>For advanced filtering based on message content, consider chaining this node with additional processing nodes in Node-RED.</li>
     </ul>

package/nodes/receiver.js

@@ -377,6 +377,26 @@ module.exports = function (RED) {
     const includeSenders = splitList(config.includeSenders || "");
     const excludeSenders = splitList(config.excludeSenders || "");
 
+    const toNormalizedIdSet = (values) => {
+        const result = new Set();
+        for (const value of values) {
+            if (value == null) {
+                continue;
+            }
+            const normalized = String(value).trim();
+            if (!normalized) {
+                continue;
+            }
+            result.add(normalized);
+        }
+        return result;
+    };
+
+    const includeChatSet = toNormalizedIdSet(includeChats);
+    const excludeChatSet = toNormalizedIdSet(excludeChats);
+    const includeSenderSet = toNormalizedIdSet(includeSenders);
+    const excludeSenderSet = toNormalizedIdSet(excludeSenders);
+
     const extractPhotoSize = (photo) => {
         if (!photo || !Array.isArray(photo.sizes)) {
             return null;
@@ -457,13 +477,9 @@ module.exports = function (RED) {
         node.send([null, { payload }]);
     };
 
-    let rawOptions = {};
-    if (includeChats.length > 0) rawOptions.chats = includeChats;
-    if (excludeChats.length > 0) rawOptions.blacklistChats = excludeChats;
-    if (includeSenders.length > 0) rawOptions.senders = includeSenders;
-    if (excludeSenders.length > 0) rawOptions.blacklistSenders = excludeSenders;
-
-    const event = new Raw(rawOptions);
+    // teleproto's Raw event builder only supports `types` and `func` options.
+    // Chat/sender filtering is implemented below after we normalize the message metadata.
+    const event = new Raw({});
     const handler = (rawUpdate) => {
         const debug = node.debugEnabled;
         if (debug) {
@@ -523,6 +539,44 @@ module.exports = function (RED) {
                 null;
 
             const chatId = peerChatId(peer);
+            const chatIdString = chatId != null ? String(chatId) : null;
+            const senderIdString = senderId != null ? String(senderId) : null;
+
+            if (includeChatSet.size > 0) {
+                const allowed = chatIdString != null && includeChatSet.has(chatIdString);
+                if (!allowed) {
+                    debugLog(`receiver ignoring message due to includeChats; chatId=${chatIdString ?? 'unknown'}`);
+                    debugSend({ event: 'ignored', reason: 'includeChats', chatId });
+                    continue;
+                }
+            }
+
+            if (excludeChatSet.size > 0) {
+                const blocked = chatIdString != null && excludeChatSet.has(chatIdString);
+                if (blocked) {
+                    debugLog(`receiver ignoring message due to excludeChats; chatId=${chatIdString}`);
+                    debugSend({ event: 'ignored', reason: 'excludeChats', chatId });
+                    continue;
+                }
+            }
+
+            if (includeSenderSet.size > 0) {
+                const allowed = senderIdString != null && includeSenderSet.has(senderIdString);
+                if (!allowed) {
+                    debugLog(`receiver ignoring message due to includeSenders; senderId=${senderIdString ?? 'unknown'} senderType=${senderType}`);
+                    debugSend({ event: 'ignored', reason: 'includeSenders', senderId, senderType });
+                    continue;
+                }
+            }
+
+            if (excludeSenderSet.size > 0) {
+                const blocked = senderIdString != null && excludeSenderSet.has(senderIdString);
+                if (blocked) {
+                    debugLog(`receiver ignoring message due to excludeSenders; senderId=${senderIdString} senderType=${senderType}`);
+                    debugSend({ event: 'ignored', reason: 'excludeSenders', senderId, senderType });
+                    continue;
+                }
+            }
 
             if (ignoredMessageTypes.size > 0) {
                 const shouldIgnoreType = Array.from(messageTypes).some((type) => ignoredMessageTypes.has(type));

package/nodes/send-file.js

@@ -36,6 +36,11 @@ module.exports = function (RED) {
              /** @type {TelegramClient} */
             const client = msg.payload?.client ? msg.payload.client : this.config.client;
 
+            if (!client) {
+                node.error('No Telegram client available. Check account configuration.');
+                return;
+            }
+
             try {
                 const params = {
                     file: files,

package/nodes/send-message.js

@@ -34,6 +34,12 @@ module.exports = function (RED) {
 
             /** @type {TelegramClient} */
             const client = msg.payload?.client ? msg.payload.client : this.config.client;
+
+            if (!client) {
+                node.error('No Telegram client available. Check account configuration.');
+                return;
+            }
+
             let peerId = chatId === "me" ? chatId : parseID(chatId);
 
             try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patricktobias86/node-red-telegram-account",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "description": "Node-RED nodes to communicate with TeleProto.",
   "main": "nodes/config.js",
   "keywords": [
@@ -44,7 +44,7 @@
     "registry": "https://registry.npmjs.org"
   },
   "dependencies": {
-    "teleproto": "^1.221.2"
+    "teleproto": "^1.223.1"
   },
   "devDependencies": {
     "mocha": "^11.7.1",

package/test/receiver.test.js

@@ -101,6 +101,87 @@ describe('Receiver node', function() {
     assert.strictEqual(sent.length, 1);
   });
 
+  it('filters by includeChats', function() {
+    const { NodeCtor, addCalls } = load();
+    const sent = [];
+    const node = new NodeCtor({config:'c', ignore:'', ignoreMessageTypes:'', maxFileSizeMb:'', includeChats:'111'});
+    node.send = (msg) => sent.push(msg);
+    const handler = addCalls[0].fn;
+
+    handler({
+      className: 'UpdateNewMessage',
+      message: { fromId: { userId: 123 }, peerId:{ userId: 111 }, message: 'allowed' }
+    });
+    handler({
+      className: 'UpdateNewMessage',
+      message: { fromId: { userId: 123 }, peerId:{ userId: 222 }, message: 'blocked' }
+    });
+
+    assert.strictEqual(sent.length, 1);
+    assert.strictEqual(sent[0].payload.chatId, 111);
+    assert.strictEqual(sent[0].payload.message.message, 'allowed');
+  });
+
+  it('filters by excludeChats', function() {
+    const { NodeCtor, addCalls } = load();
+    const sent = [];
+    const node = new NodeCtor({config:'c', ignore:'', ignoreMessageTypes:'', maxFileSizeMb:'', excludeChats:'222'});
+    node.send = (msg) => sent.push(msg);
+    const handler = addCalls[0].fn;
+
+    handler({
+      className: 'UpdateNewMessage',
+      message: { fromId: { userId: 123 }, peerId:{ userId: 111 }, message: 'allowed' }
+    });
+    handler({
+      className: 'UpdateNewMessage',
+      message: { fromId: { userId: 123 }, peerId:{ userId: 222 }, message: 'blocked' }
+    });
+
+    assert.strictEqual(sent.length, 1);
+    assert.strictEqual(sent[0].payload.chatId, 111);
+  });
+
+  it('filters by includeSenders', function() {
+    const { NodeCtor, addCalls } = load();
+    const sent = [];
+    const node = new NodeCtor({config:'c', ignore:'', ignoreMessageTypes:'', maxFileSizeMb:'', includeSenders:'123'});
+    node.send = (msg) => sent.push(msg);
+    const handler = addCalls[0].fn;
+
+    handler({
+      className: 'UpdateNewMessage',
+      message: { fromId: { userId: 123 }, peerId:{ userId: 111 }, message: 'allowed' }
+    });
+    handler({
+      className: 'UpdateNewMessage',
+      message: { fromId: { userId: 456 }, peerId:{ userId: 111 }, message: 'blocked' }
+    });
+
+    assert.strictEqual(sent.length, 1);
+    assert.strictEqual(sent[0].payload.senderId, 123);
+  });
+
+  it('filters by excludeSenders', function() {
+    const { NodeCtor, addCalls } = load();
+    const sent = [];
+    const node = new NodeCtor({config:'c', ignore:'', ignoreMessageTypes:'', maxFileSizeMb:'', excludeSenders:'456'});
+    node.send = (msg) => sent.push(msg);
+    const handler = addCalls[0].fn;
+
+    handler({
+      className: 'UpdateNewMessage',
+      message: { fromId: { userId: 123 }, peerId:{ userId: 111 }, message: 'allowed' }
+    });
+    handler({
+      className: 'UpdateNewMessage',
+      message: { fromId: { userId: 456 }, peerId:{ userId: 111 }, message: 'blocked' }
+    });
+
+    assert.strictEqual(sent.length, 1);
+    assert.strictEqual(sent[0].payload.senderId, 123);
+  });
+
   it('populates chatId and senderId for string ids', function() {
     const { NodeCtor, addCalls } = load();
     const sent = [];