From 177c7d682e64adbd278ef517af82f744ad079ac2 Mon Sep 17 00:00:00 2001
From: Odin Thomas Rochmann <odin.rochmann@gmail.com>
Date: Tue, 27 Aug 2024 12:57:42 +0200
Subject: [PATCH] fix(utils/log): Refactor ConsoleLogger to improve title
 formatting

---
 packages/utils/log/src/ConsoleLogger.ts        |  9 +++++++--
 packages/utils/log/tests/ConsoleLogger.test.ts | 11 ++++++++---
 2 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/packages/utils/log/src/ConsoleLogger.ts b/packages/utils/log/src/ConsoleLogger.ts
index e3ed13440..0ad6afd43 100644
--- a/packages/utils/log/src/ConsoleLogger.ts
+++ b/packages/utils/log/src/ConsoleLogger.ts
@@ -66,8 +66,13 @@ export class ConsoleLogger extends Logger {
     }
 
     protected _formatTitle(_lvl: LogLevel): string {
-        const title = [this.title, this.subtitle].filter((x) => !!x).join(' - ');
-        return chalk.magenta(title);
+        const title = chalk.magenta([this.title, this.subtitle].filter((x) => !!x).join(' - '));
+        switch (_lvl) {
+            case LogLevel.Warning:
+            case LogLevel.Error:
+                return chalk.bold(title);
+        }
+        return title;
     }
 
     /**
diff --git a/packages/utils/log/tests/ConsoleLogger.test.ts b/packages/utils/log/tests/ConsoleLogger.test.ts
index d44ef76c9..dd9b7e2fb 100644
--- a/packages/utils/log/tests/ConsoleLogger.test.ts
+++ b/packages/utils/log/tests/ConsoleLogger.test.ts
@@ -11,7 +11,8 @@ describe('Console logger', () => {
         logger.level = LogLevel.Debug;
         logger.debug('This is a debug message');
         expect(spy).toHaveBeenCalledWith(
-            chalk.dim(chalk.magenta('MainLogger'), 'This is a debug message'),
+            chalk(chalk.magenta('MainLogger')),
+            'This is a debug message',
         );
     });
     it('should log info messages', () => {
@@ -28,7 +29,8 @@ describe('Console logger', () => {
         logger.level = LogLevel.Warning;
         logger.warn('This is a warning message');
         expect(spy).toHaveBeenCalledWith(
-            chalk.bold(chalk.magenta('MainLogger'), 'This is a warning message'),
+            chalk.bold.magenta('MainLogger'),
+            'This is a warning message',
         );
     });
 
@@ -37,7 +39,10 @@ describe('Console logger', () => {
         const logger = new ConsoleLogger('MainLogger');
         logger.level = LogLevel.Error;
         logger.error('This is an error message');
-        expect(spy).toHaveBeenCalledWith(chalk.magenta('MainLogger'), 'This is an error message');
+        expect(spy).toHaveBeenCalledWith(
+            chalk.bold.magenta('MainLogger'),
+            'This is an error message',
+        );
     });
 
     it('should log multiple messages', () => {