From d7a92e0b926f5cc45c5b7b9ad8e95306012fb2da Mon Sep 17 00:00:00 2001
From: Nevo David <me@nevos.io>
Date: Tue, 13 Jan 2026 19:33:25 +0700
Subject: [PATCH] feat: safe stringify

---
 .../src/integrations/social.abstract.ts       | 20 ++++++++++++++++---
 1 file changed, 17 insertions(+), 3 deletions(-)

diff --git a/libraries/nestjs-libraries/src/integrations/social.abstract.ts b/libraries/nestjs-libraries/src/integrations/social.abstract.ts
index 150d642a..4d96e605 100644
--- a/libraries/nestjs-libraries/src/integrations/social.abstract.ts
+++ b/libraries/nestjs-libraries/src/integrations/social.abstract.ts
@@ -30,6 +30,20 @@ export class NotEnoughScopes {
   constructor(public message = 'Not enough scopes') {}
 }
 
+function safeStringify(obj: any) {
+  const seen = new WeakSet();
+
+  return JSON.stringify(obj, (key, value) => {
+    if (typeof value === 'object' && value !== null) {
+      if (seen.has(value)) {
+        return '[Circular]';
+      }
+      seen.add(value);
+    }
+    return value;
+  });
+}
+
 export abstract class SocialAbstract {
   abstract identifier: string;
   maxConcurrentJob = 1;
@@ -62,7 +76,7 @@ export abstract class SocialAbstract {
     try {
       value = await func();
     } catch (err) {
-      const handle = this.handleErrors(JSON.stringify(err));
+      const handle = this.handleErrors(safeStringify(err));
       value = { err: true, value: 'Unknown Error', ...(handle || {}) };
     }
 
@@ -70,12 +84,12 @@ export abstract class SocialAbstract {
       if (value.type === 'refresh-token') {
         throw new RefreshToken(
           '',
-          JSON.stringify({}),
+          safeStringify({}),
           {} as any,
           value.value || ''
         );
       }
-      throw new BadBody('', JSON.stringify({}), {} as any, value.value || '');
+      throw new BadBody('', safeStringify({}), {} as any, value.value || '');
     }
 
     return value;