fix(logger): better handle of errors instances

Author: lowlighter

logger/_logger.ts

@@ -32,6 +32,8 @@
  * @author Simon Lecoq (lowlighter)
  * @license MIT
  */
+import { red } from "@std/fmt/colors"
+
 // deno-lint-ignore ban-types
 export class Logger<T extends Record<PropertyKey, unknown> = {}> {
   /** Constructor. */
@@ -337,9 +339,16 @@ export class Logger<T extends Record<PropertyKey, unknown> = {}> {
    * Inspect a value.
    * When running on Deno, this method will use the `Deno.inspect` method.
    */
-  #inspect(value: unknown) {
+  #inspect(value: unknown, { tag = false } = {}) {
     value = this.#censor(value)
-    return typeof value === "string" ? value : globalThis.Deno?.inspect(value, { colors: true, depth: Infinity, strAbbreviateSize: Infinity }) ?? value
+    switch (true) {
+      case !tag && typeof value === "string":
+        return value
+      case value && value instanceof Error:
+        return red(value.stack || value.message)
+      default:
+        return globalThis.Deno?.inspect(value, { colors: true, depth: Infinity, strAbbreviateSize: Infinity }) ?? value
+    }
   }
 
   /** Logger formatter. */
@@ -406,7 +415,7 @@ export class Logger<T extends Record<PropertyKey, unknown> = {}> {
       }
       const tags = []
       for (const [key, value] of Object.entries(logger.tags)) {
-        tags.push(`${key}:${logger.#inspect(value)}`)
+        tags.push(`${key}:${logger.#inspect(value, { tag: true })}`)
       }
       text.push(`%c ${tags.join(" ").trim()} %c`)
       styles.push(`background-color: black`, "")

logger/_logger_test.ts

@@ -222,7 +222,7 @@ if (runtime === "deno") {
       .censor({ values: [/(api_)\w+/], replace: "$1****" })
       .censor({ keys: ["password", /^secret_/], replace: "****" })
       .log("my key is <api_foobar>", { user: "foo", password: "bar", secret_a: "a", secret_b: "b", friend: "bar", meta: { created: 123, deleted: null, secret_c: "c", api: "<api_foobar>", fn() {} } })
-    expect(text(output.log, { call: 0 }).header).toContain("token:api_****")
+    expect(text(output.log, { call: 0 }).header).toContain('token:"api_****"')
     expect(text(output.log, { call: 0 }).content[0]).toContain("my key is <api_****>")
     expect(text(output.log, { call: 0 }).content[1]).toContain('user: "foo"')
     expect(text(output.log, { call: 0 }).content[1]).toContain('password: "****"')