#1703: fixing logger runtime exceptions (#1704)

* #608: fixing logger runtime exceptions

* Fixing linter errors

* add changelog.

---------

Co-authored-by: Daniel Lee <danielylee@google.com>
Co-authored-by: Daniel Lee <taeold@gmail.com>

Author: bitekas

CHANGELOG.md

@@ -1 +1,2 @@
 - Add LLM guidance (#1736)
+- Fix logger runtime exceptions #(1704)

src/logger/index.ts

@@ -52,18 +52,18 @@ export interface LogEntry {
 }
 
 /** @internal */
-function removeCircular(obj: any, refs: any[] = []): any {
+function removeCircular(obj: any, refs: Set<any> = new Set()): any {
   if (typeof obj !== "object" || !obj) {
     return obj;
   }
   // If the object defines its own toJSON, prefer that.
-  if (obj.toJSON) {
+  if (obj.toJSON && typeof obj.toJSON === "function") {
     return obj.toJSON();
   }
-  if (refs.includes(obj)) {
+  if (refs.has(obj)) {
     return "[Circular]";
   } else {
-    refs.push(obj);
+    refs.add(obj);
   }
   let returnObj: any;
   if (Array.isArray(obj)) {
@@ -72,13 +72,21 @@ function removeCircular(obj: any, refs: any[] = []): any {
     returnObj = {};
   }
   for (const k in obj) {
-    if (refs.includes(obj[k])) {
-      returnObj[k] = "[Circular]";
+    if (obj.hasOwnProperty(k)) {
+      try {
+        if (refs.has(obj[k])) {
+          returnObj[k] = "[Circular]";
+        } else {
+          returnObj[k] = removeCircular(obj[k], refs);
+        }
+      } catch {
+        returnObj[k] = "[Error - cannot serialize]";
+      }
     } else {
-      returnObj[k] = removeCircular(obj[k], refs);
+      returnObj[k] = "[Error - defined in the prototype but missing in the object]";
     }
   }
-  refs.pop();
+  refs.delete(obj);
   return returnObj;
 }