chore: several minor updates

Author: chrisbbreuer

src/formatters/index.ts

@@ -6,7 +6,7 @@ export * from './json'
 export * from './text'
 export * from './types'
 
-export function createFormatter(format: 'json' | 'text' = 'text', options: { colors?: boolean } = {}): LogFormatter {
+export async function createFormatter(format: 'json' | 'text' = 'text', options: { colors?: boolean } = {}): Promise<LogFormatter> {
   switch (format) {
     case 'json':
       return new JsonFormatter()

src/formatters/text.ts

@@ -1,5 +1,4 @@
 import type { LogEntry } from '../types'
-// src/formatters/text.ts
 import type { LogFormatter } from './types'
 import * as colors from '../colors'
 

src/formatters/types.ts

@@ -1,5 +1,5 @@
 import type { LogEntry } from '../types'
 
 export interface LogFormatter {
-  format: (entry: LogEntry) => string
+  format: (entry: LogEntry) => Promise<string> | string
 }

src/index.ts

@@ -172,6 +172,7 @@ export class Logger {
       timestamp: new Date(),
       level,
       message,
+      name: this.name,
     }
   }
 

src/logger.ts

@@ -13,16 +13,21 @@ export interface LoggerOptions {
 }
 
 export class Logger {
-  private formatter: LogFormatter
-  private isServer: boolean
+  private formatter!: LogFormatter
+  private isServer!: boolean
 
   constructor(
     private readonly name: string,
     private options: LoggerOptions = {},
   ) {
-    this.formatter = createFormatter(
-      options.format || (process.env.NODE_ENV === 'production' ? 'json' : 'text'),
-      { colors: options.colors ?? true },
+    // We need to initialize these asynchronously
+    this.initialize()
+  }
+
+  private async initialize() {
+    this.formatter = await createFormatter(
+      this.options.format || (process.env.NODE_ENV === 'production' ? 'json' : 'text'),
+      { colors: this.options.colors ?? true },
     )
     this.isServer = await isServerProcess()
   }
@@ -113,15 +118,15 @@ export class Logger {
   }
 
   private formatMessage(message: any, args: any[] = []): string {
-    if (typeof message === 'string' && args.length > 0) {
+    if (typeof message === 'string' && args.length > 0)
       return format(message, ...args)
-    }
-    if (message instanceof Error) {
+
+    if (message instanceof Error)
       return message.stack || message.message
-    }
-    if (typeof message === 'object') {
+
+    if (typeof message === 'object')
       return JSON.stringify(message, null, 2)
-    }
+
     return String(message)
   }
 }

src/types.ts

@@ -14,6 +14,7 @@ export interface LogEntry {
   timestamp: Date
   level: LogLevel
   message: any
+  name: string
 }
 
 export type ColorFunction = (text: string) => void

test/server.test.ts

@@ -1,4 +1,5 @@
 import { afterEach, beforeEach, describe, expect, test } from 'bun:test'
+import { JsonFormatter, TextFormatter } from '../src/formatters'
 import { Logger } from '../src/index'
 import { configManager } from '../src/storage/config-manager'
 import { logManager } from '../src/storage/log-manager'
@@ -203,7 +204,7 @@ describe('CLI', () => {
 describe('Formatters', () => {
   const sampleEntry = {
     timestamp: new Date('2024-02-01T12:34:56.789Z'),
-    level: 'info',
+    level: 'info' as const,
     name: 'test',
     message: 'Hello world',
   }
@@ -219,9 +220,9 @@ describe('Formatters', () => {
       expect(output).toContain('INFO')
     })
 
-    test('formats log entry without colors', () => {
+    test('formats log entry without colors', async () => {
       const formatter = new TextFormatter(false)
-      const output = formatter.format(sampleEntry)
+      const output = await formatter.format(sampleEntry)
 
       expect(output).toBe('12:34:56:789 [test] INFO: Hello world')
     })
@@ -239,9 +240,9 @@ describe('Formatters', () => {
   })
 
   describe('JsonFormatter', () => {
-    test('formats log entry as JSON', () => {
+    test('formats log entry as JSON', async () => {
       const formatter = new JsonFormatter()
-      const output = formatter.format(sampleEntry)
+      const output = await formatter.format(sampleEntry)
       const parsed = JSON.parse(output)
 
       expect(parsed).toEqual({
@@ -253,24 +254,24 @@ describe('Formatters', () => {
       })
     })
 
-    test('includes metadata', () => {
+    test('includes metadata', async () => {
       const formatter = new JsonFormatter()
-      const output = formatter.format(sampleEntry)
+      const output = await formatter.format(sampleEntry)
       const parsed = JSON.parse(output)
 
       expect(parsed.metadata).toHaveProperty('pid')
       expect(parsed.metadata).toHaveProperty('hostname')
       expect(parsed.metadata).toHaveProperty('environment')
     })
 
-    test('handles object messages', () => {
+    test('handles object messages', async () => {
       const formatter = new JsonFormatter()
       const entry = {
         ...sampleEntry,
         message: { key: 'value' },
       }
-      const output = formatter.format(entry)
-      const parsed = JSON.parse(output)
+      const output = await formatter.format(entry)
+      const parsed = await JSON.parse(output)
 
       expect(parsed.message).toEqual({ key: 'value' })
     })

tsconfig.json

@@ -1,7 +1,7 @@
 {
   "compilerOptions": {
     "target": "esnext",
-    "lib": ["esnext"],
+    "lib": ["esnext", "DOM"],
     "moduleDetection": "force",
     "module": "esnext",
     "moduleResolution": "bundler",