fix: Correct debug log color and align logger styles with CLI conventions (#406)

* fix: correct debug log color and align logger styles with CLI conventions

* fix(tests): Add tests for getSymbols()

Author: chiragchadha1

lib/__tests__/logger.test.ts

@@ -9,6 +9,7 @@ import {
   logger,
   shouldLog,
   getLabels,
+  getSymbols,
 } from '../logger.js';
 import { isUnicodeSupported } from '../isUnicodeSupported.js';
 
@@ -72,6 +73,7 @@ describe('lib/logger', () => {
       expect(labels.warning).toBe('⚠ WARNING');
       expect(labels.error).toBe('✖ ERROR');
       expect(labels.info).toBe('ℹ INFO');
+      expect(labels.debug).toBe('⚙ DEBUG');
     });
 
     it('returns ASCII labels when unicode is not supported', () => {
@@ -82,6 +84,41 @@ describe('lib/logger', () => {
       expect(labels.warning).toBe('[WARNING]');
       expect(labels.error).toBe('[ERROR]');
       expect(labels.info).toBe('[INFO]');
+      expect(labels.debug).toBe('[DEBUG]');
+    });
+  });
+
+  describe('getSymbols()', () => {
+    const originalEnv = process.env;
+    const originalPlatform = process.platform;
+
+    afterEach(() => {
+      process.env = originalEnv;
+      Object.defineProperty(process, 'platform', {
+        value: originalPlatform,
+      });
+    });
+
+    it('returns symbol-only labels when unicode is supported', () => {
+      Object.defineProperty(process, 'platform', { value: 'darwin' });
+      process.env = { ...originalEnv, TERM: 'xterm-256color' };
+      const symbols = getSymbols();
+      expect(symbols.success).toBe('✔');
+      expect(symbols.warning).toBe('⚠');
+      expect(symbols.error).toBe('✖');
+      expect(symbols.info).toBe('ℹ');
+      expect(symbols.debug).toBe('⚙');
+    });
+
+    it('returns ASCII labels when unicode is not supported', () => {
+      Object.defineProperty(process, 'platform', { value: 'linux' });
+      process.env = { ...originalEnv, TERM: 'linux' };
+      const symbols = getSymbols();
+      expect(symbols.success).toBe('[SUCCESS]');
+      expect(symbols.warning).toBe('[WARNING]');
+      expect(symbols.error).toBe('[ERROR]');
+      expect(symbols.info).toBe('[INFO]');
+      expect(symbols.debug).toBe('[DEBUG]');
     });
   });
 

lib/logger.ts

@@ -24,7 +24,15 @@ const UNICODE_LABELS: LogLabels = {
   warning: '⚠ WARNING',
   error: '✖ ERROR',
   info: 'ℹ INFO',
-  debug: 'DEBUG',
+  debug: '⚙ DEBUG',
+};
+
+const UNICODE_SYMBOLS: LogLabels = {
+  success: '✔',
+  warning: '⚠',
+  error: '✖',
+  info: 'ℹ',
+  debug: '⚙',
 };
 
 const ASCII_LABELS: LogLabels = {
@@ -39,14 +47,18 @@ export function getLabels(): LogLabels {
   return isUnicodeSupported() ? UNICODE_LABELS : ASCII_LABELS;
 }
 
+export function getSymbols(): LogLabels {
+  return isUnicodeSupported() ? UNICODE_SYMBOLS : ASCII_LABELS;
+}
+
 /**
  * Chalk styles for logger strings.
  */
 export const Styles = {
-  debug: chalk.reset.blue,
+  debug: chalk.reset.grey,
   log: chalk.reset.white,
   success: chalk.reset.green,
-  info: chalk.reset.white,
+  info: chalk.reset.blue,
   warn: chalk.reset.yellow,
   error: chalk.reset.red,
 };
@@ -82,7 +94,7 @@ export class Logger {
   }
   debug(...args: any[]) {
     const labels = getLabels();
-    console.debug(...stylize(labels.debug, Styles.log, args));
+    console.debug(...stylize(labels.debug, Styles.debug, args));
   }
   group(...args: any[]) {
     console.group(...args);