feat: ignore certain messages when logging backend messages (#1742)

packages/extension-core/src/handlers/index.ts

@@ -30,6 +30,13 @@ const OBFUSCATE_LOG_MESSAGES: MessageTypes[] = [
 ]
 const OBFUSCATED_PAYLOAD = "#OBFUSCATED#"
 
+// ignore the ones that generate too much spam, making it hard to debug other things
+const IGNORED_LOG_MESSAGES: MessageTypes[] = [
+  "pri(keepalive)",
+  "pri(keepunlocked)",
+  "pri(app.analyticsCapture)",
+]
+
 const formatFrom = (source: string) => {
   if (["extension", "<unknown>"].includes(source)) return source
   if (!source) return source
@@ -60,8 +67,8 @@ const talismanHandler = <TMessageType extends MessageTypes>(
   }`
   const shouldLog = !OBFUSCATE_LOG_MESSAGES.includes(message)
 
-  // eslint-disable-next-line no-console
-  log.debug(`[${port.name} REQ] ${source}`, { request: shouldLog ? request : OBFUSCATED_PAYLOAD })
+  if (!IGNORED_LOG_MESSAGES.includes(message))
+    log.debug(`[${port.name} REQ] ${source}`, { request: shouldLog ? request : OBFUSCATED_PAYLOAD })
 
   const safePostMessage = (port: chrome.runtime.Port | undefined, message: unknown): void => {
     // only send message back to port if it's still connected, unfortunately this check is not reliable in all browsers
@@ -87,10 +94,11 @@ const talismanHandler = <TMessageType extends MessageTypes>(
   // resolve the promise and send back the response
   promise
     .then((response): void => {
-      log.debug(`[${port.name} RES] ${source}`, {
-        request: shouldLog ? request : OBFUSCATED_PAYLOAD,
-        response: shouldLog ? response : OBFUSCATED_PAYLOAD,
-      })
+      if (!IGNORED_LOG_MESSAGES.includes(message))
+        log.debug(`[${port.name} RES] ${source}`, {
+          request: shouldLog ? request : OBFUSCATED_PAYLOAD,
+          response: shouldLog ? response : OBFUSCATED_PAYLOAD,
+        })
 
       // between the start and the end of the promise, the user may have closed
       // the tab, in which case port will be undefined