chore: wip

Author: chrisbbreuer

build.ts

@@ -3,7 +3,13 @@ import { dts } from 'bun-plugin-dtsx'
 await Bun.build({
   entrypoints: ['src/index.ts'],
   outdir: './dist',
+  target: 'bun',
+  plugins: [dts()],
+})
+
+await Bun.build({
+  entrypoints: ['src/browser.ts'],
+  outdir: './dist',
   target: 'browser',
-  external: ['node:process'],
   plugins: [dts()],
 })

bun.lock

@@ -10,7 +10,7 @@
         "@types/bun": "^1.2.3",
         "bumpp": "^10.0.3",
         "bun-plugin-dtsx": "^0.21.9",
-        "bunfig": "^0.7.0",
+        "bunfig": "^0.8.0",
         "changelogen": "^0.5.7",
         "lint-staged": "^15.4.3",
         "simple-git-hooks": "^2.11.1",
@@ -687,7 +687,7 @@
 
     "bundle-name": ["[email protected]", "", { "dependencies": { "run-applescript": "^7.0.0" } }, "sha512-tjwM5exMg6BGRI+kNmTntNsvdZS1X8BFYS6tnJ2hdH0kVxM6/eVZ2xy+FqStSWvYmtfFMDLIxurorHwDKfDz5Q=="],
 
-    "bunfig": ["bunfig@0.7.0", "", {}, "sha512-q9F/r97pSepONQOnvvtU3WZ3kt+bhZYLbQsI2JnhqQVGuv3M/rP661QuK7fCDVmaGM5BOnd1H9qCBa4l2Y1fnA=="],
+    "bunfig": ["bunfig@0.8.0", "", { "bin": { "bunfig": "bin/cli.js" } }, "sha512-MFOLWu/PD5WcM8L09dnicwNtii+5rW5oMD5rEPOdbuSVikYGAgzmnMKzgEJpkIdhszDHS/EpQBZAJ7VQAXUwKg=="],
 
     "c12": ["[email protected]", "", { "dependencies": { "chokidar": "^4.0.1", "confbox": "^0.1.7", "defu": "^6.1.4", "dotenv": "^16.4.5", "giget": "^1.2.3", "jiti": "^2.3.0", "mlly": "^1.7.1", "ohash": "^1.1.4", "pathe": "^1.1.2", "perfect-debounce": "^1.0.0", "pkg-types": "^1.2.0", "rc9": "^2.1.2" }, "peerDependencies": { "magicast": "^0.3.5" }, "optionalPeers": ["magicast"] }, "sha512-Z4JgsKXHG37C6PYUtIxCfLJZvo6FyhHJoClwwb9ftUkLpPSkuYqn6Tr+vnaN8hymm0kIbcg6Ey3kv/Q71k5w/A=="],
 

package.json

@@ -67,7 +67,7 @@
     "@types/bun": "^1.2.3",
     "bumpp": "^10.0.3",
     "bun-plugin-dtsx": "^0.21.9",
-    "bunfig": "^0.7.0",
+    "bunfig": "^0.8.0",
     "changelogen": "^0.5.7",
     "lint-staged": "^15.4.3",
     "simple-git-hooks": "^2.11.1",

src/browser.ts

@@ -0,0 +1 @@
+export const wip = true

src/config.ts

@@ -1,17 +1,11 @@
 import type { ClarityConfig } from './types'
-import { loadConfig } from 'bunfig'
+import { homedir } from 'node:os'
+import { join } from 'node:path'
+import process from 'node:process'
+import { loadConfig as bunfigLoadConfig } from 'bunfig'
 
-const defaultRotationConfig: ClarityConfig['rotation'] = {
-  frequency: 'daily',
-  maxSize: 10 * 1024 * 1024,
-  maxFiles: 5,
-  compress: false,
-  rotateHour: 0,
-  rotateMinute: 0,
-  rotateDayOfWeek: 0,
-  rotateDayOfMonth: 1,
-  encrypt: false,
-}
+// Default log directory is in user's home directory
+const defaultLogDirectory = process.env.CLARITY_LOG_DIR || join(homedir(), '.clarity', 'logs')
 
 export const defaultConfig: ClarityConfig = {
   level: 'info',
@@ -21,14 +15,46 @@ export const defaultConfig: ClarityConfig = {
   format: 'text',
   maxLogSize: 10 * 1024 * 1024,
   logDatePattern: 'YYYY-MM-DD',
-  logDirectory: '',
-  rotation: defaultRotationConfig,
+  logDirectory: defaultLogDirectory,
+  rotation: {
+    frequency: 'daily',
+    maxSize: 10 * 1024 * 1024,
+    maxFiles: 5,
+    compress: false,
+    rotateHour: 0,
+    rotateMinute: 0,
+    rotateDayOfWeek: 0,
+    rotateDayOfMonth: 1,
+    encrypt: false,
+  },
   verbose: false,
 }
 
+// Wrapper around bunfig's loadConfig that handles verbose mode
+async function loadConfig() {
+  const isVerbose = process.env.CLARITY_VERBOSE === 'true' || defaultConfig.verbose
+
+  // Temporarily override console.error if not in verbose mode
+  const originalError = console.error
+  if (!isVerbose) {
+    console.error = () => {}
+  }
+
+  try {
+    return await bunfigLoadConfig({
+      name: 'clarity',
+      defaultConfig,
+      cwd: process.cwd(),
+      endpoint: '',
+      headers: {},
+    })
+  }
+  finally {
+    // Always restore console.error
+    console.error = originalError
+  }
+}
+
 // @ts-expect-error there is a current dtsx issue
 // eslint-disable-next-line antfu/no-top-level-await
-export const config: ClarityConfig = await loadConfig({
-  name: 'clarity',
-  defaultConfig,
-})
+export const config: ClarityConfig = await loadConfig()

src/logger.ts

@@ -83,12 +83,31 @@ export class Logger {
       delete configOptions.timestamp // Remove from config to avoid type error
     }
 
+    // Merge with default config
     this.config = {
       ...defaultConfig,
       ...configOptions,
       timestamp: hasTimestamp || defaultConfig.timestamp,
     }
 
+    // Ensure log directory exists
+    if (!this.config.logDirectory) {
+      this.config.logDirectory = defaultConfig.logDirectory
+    }
+
+    // Suppress config loading message if not in verbose mode
+    if (!this.config.verbose && !process.env.CLARITY_VERBOSE) {
+      const originalConsoleError = console.error
+      console.error = () => {}
+      try {
+        // Force config loading here to suppress messages
+        void loadConfig({ name: 'clarity', defaultConfig })
+      }
+      finally {
+        console.error = originalConsoleError
+      }
+    }
+
     this.timers = new Map()
     this.subLoggers = new Map()
     this.formatter = this.config.format === 'json'
@@ -738,7 +757,7 @@ export class Logger {
     }
   }
 
-  private async log(level: LogLevel, message: string, ...args: any[]): Promise<void> {
+  async log(level: LogLevel, message: string, ...args: any[]): Promise<void> {
     if (!this.shouldLog(level)) {
       return
     }