From c58308b70f75af1b44a4995165ade906aba698d0 Mon Sep 17 00:00:00 2001
From: Narasimha-sc <166327228+Narasimha-sc@users.noreply.github.com>
Date: Thu, 16 Apr 2026 08:51:22 +0000
Subject: [PATCH] simplex-chat-nodejs: mute benign group-member teardown errors

The Haskell CLI already mutes SEConnectionNotFound at default log level
(View.hs:2632, "-- mutes delete group error"). The Node SDK had no
equivalent filter, so every apiRemoveMembers / apiLeaveGroup call logged
two noisy but harmless errors:
- ErrorStore connectionNotFound (DB row deleted before async agent event)
- ErrorAgent CONN NOT_FOUND with empty agentConnId (bulk SMP ERR)

Add isMutedChatError() to skip these two exact shapes in the event loop,
matching the CLI's default behavior. All other errors still log.
---
 packages/simplex-chat-nodejs/src/api.ts | 27 +++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/packages/simplex-chat-nodejs/src/api.ts b/packages/simplex-chat-nodejs/src/api.ts
index 8bc56db41c..0504a1a0d5 100644
--- a/packages/simplex-chat-nodejs/src/api.ts
+++ b/packages/simplex-chat-nodejs/src/api.ts
@@ -56,6 +56,32 @@ interface EventSubscriber<K extends CEvt.Tag> {
   once: boolean
 }
 
+// Mirrors src/Simplex/Chat/View.hs:2632 ("-- mutes delete group error"): the
+// Haskell CLI hides SEConnectionNotFound at default log level because it fires
+// benignly during group-member teardown. The Node SDK has no log-level knob,
+// so we mute the two exact error shapes that originate from that teardown path:
+//   1. ChatError.ErrorStore { storeError: { type: "connectionNotFound", ... } }
+//      Emitted via getChatLockEntity (Connections.hs:54) / getConnectionEntity
+//      (Connections.hs:93) when an async agent event references a connection
+//      whose row was just deleted by deleteMembersConnections'.
+//   2. ChatError.ErrorAgent with empty agentConnId + CONN NOT_FOUND.
+//      Emitted at Subscriber.hs:112 (processAgentMessage _ _ "" (ERR e)) when
+//      the SMP agent sends a bulk ERR after teardown.
+function isMutedChatError(err: T.ChatError): boolean {
+  if (err.type === "errorStore" && err.storeError.type === "connectionNotFound") {
+    return true
+  }
+  if (
+    err.type === "errorAgent" &&
+    err.agentConnId === "" &&
+    err.agentError.type === "CONN" &&
+    err.agentError.connErr.type === "NOT_FOUND"
+  ) {
+    return true
+  }
+  return false
+}
+
 /**
  * Main API class for interacting with the chat core library.
  */
@@ -147,6 +173,7 @@ export class ChatApi {
       } catch(err) {
         const e = err as core.ChatAPIError
         if ("chatError" in e) {
+          if (e.chatError && isMutedChatError(e.chatError)) continue
           console.log("Chat error", e.chatError)
         } else {
           console.log("Invalid event", e)