fix(ext/console): Error Cause Not Inspect-Formatted when printed (#24526)

This pull request addresses an issue where the Error.cause property was
not formatted correctly when printed using console.log, leading to
confusion.

solution:
Implemented a fix to ensure that Error.cause is formatted properly when
printed by console.log, and the fix done by using JSON.stringify

This PR fixes #23416

---------

Signed-off-by: MujahedSafaa <[email protected]>

Author: MujahedSafaa

ext/console/01_console.js

@@ -1484,12 +1484,18 @@ function inspectError(value, ctx) {
       finalMessage += `[${stack || ErrorPrototypeToString(value)}]`;
     }
   }
+  const doubleQuoteRegExp = new SafeRegExp('"', "g");
   finalMessage += ArrayPrototypeJoin(
     ArrayPrototypeMap(
       causes,
       (cause) =>
         "\nCaused by " + (MapPrototypeGet(refMap, cause) ?? "") +
-        (cause?.stack ?? cause),
+        (cause?.stack ??
+          StringPrototypeReplace(
+            inspect(cause),
+            doubleQuoteRegExp,
+            "",
+          )),
     ),
     "",
   );

tests/unit/console_test.ts

@@ -2206,6 +2206,22 @@ Deno.test(function inspectErrorCircular() {
   );
 });
 
+Deno.test(function inspectErrorWithCauseFormat() {
+  const error = new Error("This is an error", {
+    cause: {
+      code: 100500,
+    },
+  });
+  assertStringIncludes(
+    stripColor(Deno.inspect(error)),
+    "Error: This is an error",
+  );
+  assertStringIncludes(
+    stripColor(Deno.inspect(error)),
+    "Caused by { code: 100500 }",
+  );
+});
+
 Deno.test(function inspectColors() {
   assertEquals(Deno.inspect(1), "1");
   assertStringIncludes(Deno.inspect(1, { colors: true }), "\x1b[");