From 3c0adac8f1252ad380ac3aaf8e59d0beae717343 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Eckhart=20W=C3=B6rner?= <ewoerner@users.noreply.github.com>
Date: Tue, 17 Sep 2024 21:04:16 +0200
Subject: [PATCH 1/2] Handle errors when serializing logged objects

---
 plugins/debug-console.js | 27 +++++++++++++++------------
 1 file changed, 15 insertions(+), 12 deletions(-)

diff --git a/plugins/debug-console.js b/plugins/debug-console.js
index 1f73e877c..9f2bc1cab 100644
--- a/plugins/debug-console.js
+++ b/plugins/debug-console.js
@@ -51,19 +51,22 @@ debugTab.renderLine = function (errorType, args) {
   var text = [];
   args.forEach(function (v) {
     if (typeof v !== 'string' && typeof v !== 'number') {
-      var cache = [];
-      v = JSON.stringify(v, function (key, value) {
-        if (typeof value === 'object' && value !== null) {
-          if (cache.indexOf(value) !== -1) {
-            // Circular reference found, discard key
-            return;
+      try {
+        var cache = [];
+        v = JSON.stringify(v, function (key, value) {
+          if (typeof value === 'object' && value !== null) {
+            if (cache.indexOf(value) !== -1) {
+              // Circular reference found, discard key
+              return;
+            }
+            // Store value in our collection
+            cache.push(value);
           }
-          // Store value in our collection
-          cache.push(value);
-        }
-        return value;
-      });
-      cache = null;
+          return value;
+        });
+      } finally {
+        cache = null;
+      }
     }
     text.push(v);
   });

From a3e5e90ab85eba641fdd27855666bcf85d534bcc Mon Sep 17 00:00:00 2001
From: Alexander Danilov <git@modos189.ru>
Date: Wed, 2 Oct 2024 17:03:00 +0500
Subject: [PATCH 2/2] Fix for correct work with recursive objects

---
 plugins/debug-console.js | 56 ++++++++++++++++++++++++----------------
 1 file changed, 34 insertions(+), 22 deletions(-)

diff --git a/plugins/debug-console.js b/plugins/debug-console.js
index 9f2bc1cab..04aa4c3a6 100644
--- a/plugins/debug-console.js
+++ b/plugins/debug-console.js
@@ -1,7 +1,7 @@
 // @author         jaiperdu
 // @name           Debug console tab
 // @category       Debug
-// @version        0.1.1
+// @version        0.2.0
 // @description    Add a debug console tab
 
 /* exported setup, changelog --eslint */
@@ -10,7 +10,7 @@
 var changelog = [
   {
     version: '0.2.0',
-    changes: ['Use channel new API', 'Handle multiline messages'],
+    changes: ['Use channel new API', 'Handle multiline messages', 'Handle errors when serializing logged objects'],
   },
   {
     version: '0.1.1',
@@ -47,59 +47,71 @@ debugTab.create = function () {
 };
 
 debugTab.renderLine = function (errorType, args) {
+  // Convert arguments to an array
   args = Array.prototype.slice.call(args);
   var text = [];
+
+  // Function to safely stringify objects with depth limitation
+  function safeStringify(obj, depth = 5) {
+    let cache = [];
+    return JSON.stringify(obj, function (key, value) {
+      if (typeof value === 'object' && value !== null) {
+        // Detect circular references or if the depth exceeds the limit
+        if (cache.indexOf(value) !== -1 || cache.length > depth) {
+          return '[Circular]'; // Return a placeholder for circular references
+        }
+        // Store object in cache for future reference
+        cache.push(value);
+      }
+      return value;
+    });
+  }
+
   args.forEach(function (v) {
+    // If v is not a string or number, attempt to stringify
     if (typeof v !== 'string' && typeof v !== 'number') {
       try {
-        var cache = [];
-        v = JSON.stringify(v, function (key, value) {
-          if (typeof value === 'object' && value !== null) {
-            if (cache.indexOf(value) !== -1) {
-              // Circular reference found, discard key
-              return;
-            }
-            // Store value in our collection
-            cache.push(value);
-          }
-          return value;
-        });
-      } finally {
-        cache = null;
+        v = safeStringify(v);
+      } catch {
+        // In case of error, return error message with the object's string representation
+        v = 'error rendering: ' + String(v);
       }
     }
+    // Add the value to the text array
     text.push(v);
   });
+
+  // Join text array into a single string with spaces between values
   text = text.join(' ');
 
-  // Time
+  // Time element creation
   var time = document.createElement('time');
   var d = new Date();
   time.textContent = d.toLocaleTimeString();
   time.title = d.toLocaleString();
   time.dataset.timestamp = d.getTime();
 
-  // Type
+  // Type element creation (for log type)
   var type = document.createElement('mark');
   type.textContent = errorType;
   type.className = errorType;
 
-  // Text
+  // Text element creation (for the log message)
   var pre = document.createElement('pre');
   pre.textContent = text;
 
-  // Check if the last message is visible
+  // Check if the last message is visible (scroll position)
   var debugContainer = document.getElementById('chatdebug');
   var isAtBottom = debugContainer.scrollTop >= debugContainer.scrollTopMax;
 
-  // Insert Row
+  // Insert a new row in the debug table
   var table = document.querySelector('#chatdebug table');
   var row = table.insertRow();
   row.insertCell().append(time);
   row.insertCell().append(type);
   row.insertCell().append(pre);
 
-  // Auto-scroll to bottom
+  // Auto-scroll to the bottom if the user was at the bottom
   if (isAtBottom) debugContainer.scrollTo(0, debugContainer.scrollTopMax);
 };