improve: Enhance pretty cause

.changeset/sharp-masks-jam.md

@@ -0,0 +1,5 @@
+---
+"effect": patch
+---
+
+improve: Pretty cause printing

packages/effect/src/Inspectable.ts

@@ -113,6 +113,8 @@ export const stringifyCircular = (obj: unknown, whitespace?: number | string | u
           : cache.push(value) && (redactableState.fiberRefs !== undefined && isRedactable(value)
             ? value[symbolRedactable](redactableState.fiberRefs)
             : value)
+        : typeof value === "bigint"
+        ? value.toString()
         : value,
     whitespace
   )

packages/effect/src/internal/cause.ts

@@ -977,10 +977,16 @@ export const pretty = <E>(cause: Cause.Cause<E>, options?: {
     return "All fibers interrupted without errors."
   }
   return prettyErrors<E>(cause).map(function(e) {
-    if (options?.renderErrorCause !== true || e.cause === undefined) {
+    if (options?.renderErrorCause !== true) {
       return e.stack
     }
-    return `${e.stack} {\n${renderErrorCause(e.cause as PrettyError, "  ")}\n}`
+    const { cause, message: _, name: __, stack, ...rest } = e
+    const json = stringifyCircular(toJSON(rest), 2)
+    return !cause && !!Object.keys(rest).length ?
+      stack :
+      `${stack} {${json.substring(1, json.length - 1).split("\n").join("\n").trimEnd()}${
+        cause ? ",\n" + renderErrorCause(cause as PrettyError, "  ") : ""
+      }\n}`
   }).join("\n")
 }
 

packages/effect/test/Cause.test.ts

@@ -8,6 +8,7 @@ import * as Predicate from "effect/Predicate"
 import { causes, equalCauses, errorCauseFunctions, errors } from "effect/test/utils/cause"
 import * as fc from "fast-check"
 import { assert, describe, expect, it } from "vitest"
+import { Data } from "../src/index.js"
 
 describe("Cause", () => {
   it("[internal] prettyErrorMessage", () => {
@@ -81,6 +82,19 @@ describe("Cause", () => {
         }
       }
       expect(Cause.pretty(Cause.fail(new Error5()))).toEqual(`Error: my string`)
+
+      class Error6 extends Data.TaggedError("Error6")<{ message: string; code: number; cause: Error }> {
+      }
+      expect(
+        Cause.pretty(Cause.fail(new Error6({ message: "test", code: 10, cause: new Error("cause") })), {
+          renderErrorCause: true
+        })
+      ).includes(
+        `{
+  "code": 10,
+  "_tag": "Error6",
+  [cause]: Error: cause`
+      )
     })
 
     it("Interrupt", () => {
@@ -350,7 +364,7 @@ describe("Cause", () => {
       if (typeof window === "undefined") {
         // eslint-disable-next-line @typescript-eslint/no-var-requires
         const { inspect } = require("node:util")
-        expect(inspect(ex)).include("Cause.test.ts:348")
+        expect(inspect(ex)).include("Cause.test.ts:362")
       }
     })
   })