Merge pull request #307 from shrinishLT/DOT-5846

use json-stringify-safe for circular-json errors

Author: parthlambdatest

package.json

@@ -23,14 +23,15 @@
   "author": "LambdaTest <[email protected]>",
   "license": "MIT",
   "dependencies": {
+    "@lambdatest/node-tunnel": "^4.0.9",
     "@playwright/browser-chromium": "^1.47.2",
     "@playwright/browser-firefox": "^1.47.2",
     "@playwright/browser-webkit": "^1.47.2",
     "@playwright/test": "^1.47.2",
     "@types/cross-spawn": "^6.0.4",
+    "@types/json-stringify-safe": "^5.0.3",
     "@types/node": "^20.8.9",
     "@types/which": "^3.0.2",
-    "@lambdatest/node-tunnel": "^4.0.9",
     "ajv": "^8.12.0",
     "ajv-errors": "^3.0.0",
     "axios": "^1.6.0",
@@ -39,6 +40,7 @@
     "cross-spawn": "^7.0.3",
     "fastify": "^4.24.3",
     "form-data": "^4.0.0",
+    "json-stringify-safe": "^5.0.1",
     "listr2": "^7.0.1",
     "node-cache": "^5.1.2",
     "sharp": "^0.33.4",

src/lib/logger.ts

@@ -1,4 +1,5 @@
 import { createLogger, format, transports, log } from 'winston'
+import stringify  from 'json-stringify-safe'
 import constants from './constants.js'
 import { Env } from '../types.js'
 import getEnv from './env.js'
@@ -26,7 +27,7 @@ const logger = createLogger({
 		format.timestamp(),
 		format.printf(info => {
 			let contextString = Object.values(logContext).join(' | ');
-			let message = (typeof info.message === 'object') ? JSON.stringify(info.message).trim() : info.message.trim();
+			let message = (typeof info.message === 'object') ? stringify(info.message) : info.message.trim();
 			switch (info.level) {
 				case 'warn':
 					message = chalk.yellow(message);