Merge pull request #2692 from contentstack/v1-back-merge

V1 back merge

Author: shafeeqd959

.talismanrc

@@ -1,5 +1,9 @@
 fileignoreconfig:
 
 - filename: pnpm-lock.yaml
-  checksum: 3d5920821aca485d93dc90dc2006bb81b077344b5f5fd6487002d8b4222cff09
+  checksum: 98cc0f62b09ea6768f3a20152860a327722a92efaa372acfc7d37bdd2afd014e
+- filename: packages/contentstack/README.md
+  checksum: df0892cbcf3f04972161b30a1a0ef8844a2682ce8a3270f2911909e7789ad8d7
+- filename: packages/contentstack-utilities/src/logger/logger.ts
+  checksum: 8c960b88f59f2efd233e7e00fb69dfb90e6eccf52e7253d144d4e20ff72a6906
   version: '1.0'

packages/contentstack-auth/README.md

@@ -18,7 +18,7 @@ $ npm install -g @contentstack/cli-auth
 $ csdx COMMAND
 running command...
 $ csdx (--version)
-@contentstack/cli-auth/2.0.0-beta.17 darwin-arm64 node-v24.18.0
+@contentstack/cli-auth/2.0.0-beta.17 darwin-arm64 node-v22.13.1
 $ csdx --help [COMMAND]
 USAGE
   $ csdx COMMAND

packages/contentstack-config/README.md

@@ -18,7 +18,7 @@ $ npm install -g @contentstack/cli-config
 $ csdx COMMAND
 running command...
 $ csdx (--version)
-@contentstack/cli-config/2.0.0-beta.15 darwin-arm64 node-v24.18.0
+@contentstack/cli-config/2.0.0-beta.15 darwin-arm64 node-v22.13.1
 $ csdx --help [COMMAND]
 USAGE
   $ csdx COMMAND

packages/contentstack-utilities/src/logger/logger.ts

@@ -63,7 +63,7 @@ export default class Logger {
           winston.format.printf((info) => {
             // Apply minimal redaction for files (debugging info preserved)
             const redactedInfo = this.redact(info, false);
-            return JSON.stringify(redactedInfo);
+            return this.safeStringify(redactedInfo);
           }),
         ),
       }),
@@ -75,7 +75,7 @@ export default class Logger {
       const logConfig = configHandler.get('log') || {};
       const currentModule = logConfig.progressSupportedModule;
       const hasProgressSupport = currentModule && PROGRESS_SUPPORTED_MODULES.includes(currentModule);
-      
+
       if (hasProgressSupport) {
         // Plugin has progress bars - respect user's explicit setting, or default to false (show progress bars)
         showConsoleLogs = logConfig.showConsoleLogs ?? false;
@@ -146,6 +146,26 @@ export default class Logger {
     }
   }
 
+  /**
+   * Stringifies a log entry without throwing on circular references.
+   * `redact()` can fall back to the original (still-circular) object when
+   * cloning fails, so this must never assume the input is cycle-free.
+   */
+  private safeStringify(value: any): string {
+    const seen = new WeakSet();
+    try {
+      return JSON.stringify(value, (_key, val) => {
+        if (typeof val === 'object' && val !== null) {
+          if (seen.has(val)) return '[Circular]';
+          seen.add(val);
+        }
+        return val;
+      });
+    } catch {
+      return JSON.stringify({ level: value?.level, message: value?.message ?? '[Unserializable log entry]' });
+    }
+  }
+
   private shouldLog(level: LogType, target: 'console' | 'file'): boolean {
     // If console logging is disabled, don't log to console
     if (target === 'console' && this.config.consoleLoggingEnabled === false) {
@@ -215,9 +235,12 @@ export default class Logger {
       this.loggers.error.error(logPayload);
     }
 
-    // For console, use debug level if hidden, otherwise error level
+    // For console, use debug level if hidden, otherwise error level. Each level's
+    // winston logger instance bundles both the file and console transports, so only
+    // write again here when the target differs from the one already written above —
+    // otherwise this would duplicate both the console line and the file entry.
     const consoleLevel: LogType = params.hidden ? 'debug' : 'error';
-    if (this.shouldLog(consoleLevel, 'console')) {
+    if (consoleLevel !== 'error' && this.shouldLog(consoleLevel, 'console')) {
       this.loggers[consoleLevel].error(logPayload);
     }
   }