include Error.cause stack in log output (#3194)

.changeset/chilly-berries-talk.md

@@ -0,0 +1,5 @@
+---
+"effect": patch
+---
+
+include Error.cause stack in log output

.changeset/clever-parents-leave.md

@@ -0,0 +1,5 @@
+---
+"effect": minor
+---
+
+add renderErrorCause option to Cause.pretty

.changeset/fast-dodos-impress.md

@@ -0,0 +1,5 @@
+---
+"effect": patch
+---
+
+set stackTraceLimit to 1 in PrettyError to address performance issues

.changeset/ninety-starfishes-flow.md

@@ -0,0 +1,5 @@
+---
+"effect": minor
+---
+
+add .groupCollapsed to UnsafeConsole

packages/effect/src/Cause.ts

@@ -957,7 +957,9 @@ export const isExceededCapacityException: (u: unknown) => u is ExceededCapacityE
  * @since 2.0.0
  * @category rendering
  */
-export const pretty: <E>(cause: Cause<E>) => string = internal.pretty
+export const pretty: <E>(cause: Cause<E>, options?: {
+  readonly renderErrorCause?: boolean | undefined
+}) => string = internal.pretty
 
 /**
  * @since 3.2.0

packages/effect/src/Console.ts

@@ -64,6 +64,7 @@ export interface UnsafeConsole {
   dirxml(...args: ReadonlyArray<any>): void
   error(...args: ReadonlyArray<any>): void
   group(label?: string | undefined): void
+  groupCollapsed(label?: string | undefined): void
   groupEnd(): void
   info(...args: ReadonlyArray<any>): void
   log(...args: ReadonlyArray<any>): void

packages/effect/src/internal/cause.ts

@@ -970,25 +970,46 @@ export const reduceWithContext = dual<
 // -----------------------------------------------------------------------------
 
 /** @internal */
-export const pretty = <E>(cause: Cause.Cause<E>): string => {
+export const pretty = <E>(cause: Cause.Cause<E>, options?: {
+  readonly renderErrorCause?: boolean | undefined
+}): string => {
   if (isInterruptedOnly(cause)) {
     return "All fibers interrupted without errors."
   }
-  return prettyErrors<E>(cause).map((e) => e.stack).join("\n")
+  return prettyErrors<E>(cause).map(function(e) {
+    if (options?.renderErrorCause !== true || e.cause === undefined) {
+      return e.stack
+    }
+    return `${e.stack} {\n${renderErrorCause(e.cause as PrettyError, "  ")}\n}`
+  }).join("\n")
+}
+
+const renderErrorCause = (cause: PrettyError, prefix: string) => {
+  const lines = cause.stack!.split("\n")
+  let stack = `${prefix}[cause]: ${lines[0]}`
+  for (let i = 1, len = lines.length; i < len; i++) {
+    stack += `\n${prefix}${lines[i]}`
+  }
+  return stack
 }
 
 class PrettyError extends globalThis.Error implements Cause.PrettyError {
   span: undefined | Span = undefined
   constructor(originalError: unknown) {
+    const originalErrorIsObject = typeof originalError === "object" && originalError !== null
     const prevLimit = Error.stackTraceLimit
-    Error.stackTraceLimit = 0
-    super(prettyErrorMessage(originalError))
+    Error.stackTraceLimit = 1
+    super(prettyErrorMessage(originalError), {
+      cause: originalErrorIsObject && "cause" in originalError && typeof originalError.cause !== "undefined"
+        ? new PrettyError(originalError.cause)
+        : undefined
+    })
     if (this.message === "") {
       this.message = "An error has occurred"
     }
     Error.stackTraceLimit = prevLimit
     this.name = originalError instanceof Error ? originalError.name : "Error"
-    if (typeof originalError === "object" && originalError !== null) {
+    if (originalErrorIsObject) {
       if (spanSymbol in originalError) {
         this.span = originalError[spanSymbol] as Span
       }

packages/effect/src/internal/logger.ts

@@ -185,7 +185,7 @@ export const stringLogger: Logger.Logger<unknown, string> = makeLogger(
 
     if (cause != null && cause._tag !== "Empty") {
       output = output + " cause="
-      output = appendQuoted(Cause.pretty(cause), output)
+      output = appendQuoted(Cause.pretty(cause, { renderErrorCause: true }), output)
     }
 
     if (List.isCons(spans)) {
@@ -255,7 +255,7 @@ export const logfmtLogger = makeLogger<unknown, string>(
 
     if (cause != null && cause._tag !== "Empty") {
       output = output + " cause="
-      output = appendQuotedLogfmt(Cause.pretty(cause), output)
+      output = appendQuotedLogfmt(Cause.pretty(cause, { renderErrorCause: true }), output)
     }
 
     if (List.isCons(spans)) {
@@ -324,7 +324,7 @@ export const structuredLogger = makeLogger<unknown, {
       message: messageArr.length === 1 ? structuredMessage(messageArr[0]) : messageArr.map(structuredMessage),
       logLevel: logLevel.label,
       timestamp: date.toISOString(),
-      cause: Cause.isEmpty(cause) ? undefined : Cause.pretty(cause),
+      cause: Cause.isEmpty(cause) ? undefined : Cause.pretty(cause, { renderErrorCause: true }),
       annotations: annotationsObj,
       spans: spansObj,
       fiberId: _fiberId.threadName(fiberId)
@@ -454,43 +454,30 @@ export const prettyLogger = (options?: {
       }
 
       if (isBrowser) {
-        console.group(firstLine)
+        console.groupCollapsed(firstLine)
       } else {
         log(firstLine)
         console.group()
       }
-      let currentMessage = ""
-      const params: Array<unknown> = []
-
       if (!Cause.isEmpty(cause)) {
-        const errors = Cause.prettyErrors(cause)
-        for (let i = 0; i < errors.length; i++) {
-          if (isBrowser) {
-            console.error(errors[i].stack)
-          } else {
-            currentMessage += "\n%s"
-            params.push(errors[i].stack)
-          }
+        if (isBrowser) {
+          console.error(Cause.pretty(cause, { renderErrorCause: true }))
+        } else {
+          log(Cause.pretty(cause, { renderErrorCause: true }))
         }
       }
 
       if (messageIndex < message.length) {
         for (; messageIndex < message.length; messageIndex++) {
-          currentMessage += "\n%O"
-          params.push(message[messageIndex])
+          log(message[messageIndex])
         }
       }
 
       if (HashMap.size(annotations) > 0) {
         for (const [key, value] of annotations) {
-          currentMessage += "\n" + color(`${key}:`, colors.bold, colors.white) + " %O"
-          params.push(value)
+          log(color(`${key}:`, colors.bold, colors.white), value)
         }
       }
-
-      if (currentMessage.length > 0) {
-        log(currentMessage.slice(1), ...params)
-      }
       console.groupEnd()
     }
   )

packages/effect/test/Effect/cause-rendering.test.ts

@@ -154,4 +154,14 @@ error
 message
     at`))
     }))
+
+  it.effect("pretty includes error.cause with renderErrorCause: true", () =>
+    Effect.gen(function*() {
+      const cause = yield* Effect.fail(new Error("parent", { cause: new Error("child") })).pipe(
+        Effect.sandbox,
+        Effect.flip
+      )
+      const pretty = Cause.pretty(cause, { renderErrorCause: true })
+      assert.include(pretty, "[cause]: Error: child")
+    }))
 })