From 895a8c42cac5c4f4b7a3a544945bbd4ea34bc7f1 Mon Sep 17 00:00:00 2001
From: Jonathan Bernard <jonathan@jdbernard.com>
Date: Fri, 9 Jan 2026 19:23:38 -0600
Subject: [PATCH] ConsoleLogAppender logs TRACE messages with console.log

console.trace generates and prints a full stacktrace for every log
message. This is not what TRACE means in this library. The TRACE level
is intended for logs that are more verbose than you typically want to
log and should only be turned on when you are trying to trace the
detailed behavior of the logged functionality. Printing the full stack
trace with every message makes an already verbose setting exponentially
worse.
---
 package.json                | 2 +-
 src/console-log-appender.ts | 6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/package.json b/package.json
index 076bfb4..88934e0 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@jdbernard/logging",
-  "version": "2.3.2",
+  "version": "2.3.3",
   "description": "Simple Javascript logging service.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
diff --git a/src/console-log-appender.ts b/src/console-log-appender.ts
index 6fb2819..36b77bf 100644
--- a/src/console-log-appender.ts
+++ b/src/console-log-appender.ts
@@ -39,9 +39,11 @@ export class ConsoleLogAppender implements LogAppender {
     switch (msg.level) {
       case LogLevel.ALL:
       case LogLevel.TRACE:
-        logMethod = console.trace
-        break
       case LogLevel.LOG:
+        // console.trace generates and prints a stack trace attached to the
+        // logged message. This is not really what we want for TRACE level
+        // messages as it just makes things more verbose needlessly, so these
+        // all log to console.log
         logMethod = console.log
         break
       case LogLevel.DEBUG: