@thegetty/quire-cli 1.0.0-rc.44 → 1.0.0-rc.46

package/src/lib/logger/debug.test.js

@@ -1,128 +0,0 @@
-import test from 'ava'
-import debug from 'debug'
-// Import to register formatter and formatArgs overrides
-import './debug.js'
-
-/**
- * Debug formatter and output tests
- *
- * Verifies the custom %O formatter produces indented multi-line output,
- * the %o formatter remains compact single-line, and the formatArgs
- * override prevents per-line namespace prefixing in colored output.
- */
-
-/**
- * Helper to call a debug formatter with a value
- *
- * Formatters are called with `this` bound to a debugger-like context.
- * We provide the minimal context the formatters expect.
- */
-function callFormatter(specifier, value) {
-  const formatter = debug.formatters[specifier]
-  const context = {
-    useColors: false,
-    inspectOpts: { colors: false },
-  }
-  return formatter.call(context, value)
-}
-
-// ─────────────────────────────────────────────────────────────────────────────
-// %O formatter (pretty-printed, indented)
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('%O output starts with a newline', (t) => {
-  const result = callFormatter('O', { a: 1 })
-  t.true(result.startsWith('\n'))
-})
-
-test('%O output lines are indented with 2 spaces', (t) => {
-  const result = callFormatter('O', { a: 1, b: 2 })
-  const lines = result.split('\n').slice(1) // skip leading empty line
-  for (const line of lines) {
-    t.true(line.startsWith('  '), `Line not indented: "${line}"`)
-  }
-})
-
-test('%O formats nested objects with indentation', (t) => {
-  const result = callFormatter('O', { outer: { inner: 'value' } })
-  t.true(result.includes('outer'))
-  t.true(result.includes('inner'))
-  // Every line after the leading newline should be indented
-  const lines = result.split('\n').slice(1)
-  for (const line of lines) {
-    t.true(line.startsWith('  '), `Line not indented: "${line}"`)
-  }
-})
-
-test('%O handles null without error', (t) => {
-  const result = callFormatter('O', null)
-  t.true(result.includes('null'))
-})
-
-test('%O handles undefined without error', (t) => {
-  const result = callFormatter('O', undefined)
-  t.true(result.includes('undefined'))
-})
-
-test('%O handles primitive values', (t) => {
-  t.true(callFormatter('O', 42).includes('42'))
-  t.true(callFormatter('O', 'hello').includes('hello'))
-  t.true(callFormatter('O', true).includes('true'))
-})
-
-test('%O handles arrays', (t) => {
-  const result = callFormatter('O', [1, 2, 3])
-  t.true(result.includes('1'))
-  t.true(result.includes('2'))
-  t.true(result.includes('3'))
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// %o formatter (compact single-line, built-in)
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('%o output is single-line', (t) => {
-  const result = callFormatter('o', { a: 1, b: 2, c: 3 })
-  t.false(result.includes('\n'))
-})
-
-test('%o does not start with a newline', (t) => {
-  const result = callFormatter('o', { a: 1 })
-  t.false(result.startsWith('\n'))
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// formatArgs override (prefix only on first line)
-// ─────────────────────────────────────────────────────────────────────────────
-
-/**
- * Helper to call formatArgs and capture the result
- *
- * Creates a minimal debugger-like context to test formatArgs behavior.
- * In color mode, the default debug formatArgs would split on newlines
- * and insert the prefix before every line. Our override prevents this.
- */
-function callFormatArgs(message, useColors = true) {
-  const args = [message]
-  const context = {
-    namespace: 'test',
-    useColors,
-    color: 5,
-    diff: 0,
-  }
-  debug.formatArgs.call(context, args)
-  return args[0]
-}
-
-test('formatArgs prefixes the first line in color mode', (t) => {
-  const result = callFormatArgs('hello')
-  t.true(result.includes('test'))
-  t.true(result.includes('hello'))
-})
-
-test('formatArgs does not duplicate prefix on multi-line output', (t) => {
-  const result = callFormatArgs('paths:\n  { a: 1 }\n  { b: 2 }')
-  // Namespace should appear exactly once
-  const matches = result.match(/test/g)
-  t.is(matches.length, 1)
-})

package/src/lib/logger/index.js

@@ -1,228 +0,0 @@
-import { format } from 'node:util'
-import chalk from 'chalk'
-import log from 'loglevel'
-import config from '#lib/conf/config.js'
-
-/**
- * Logger factory for Quire CLI
- *
- * Creates loggers with colored output, log level filtering, and configurable prefixes.
- * Configuration is read from the conf module for prefix style, colors, etc.
- *
- * @example
- * // Default singleton for simple usage
- * import { logger } from '#lib/logger/index.js'
- * logger.info('Building site...')
- * // Output: [quire] INFO  Building site...
- *
- * @example
- * // In tests - mock the factory
- * const mockLogger = {
- *   info: sandbox.stub(),
- *   error: sandbox.stub(),
- *   debug: sandbox.stub(),
- *   warn: sandbox.stub(),
- *   trace: sandbox.stub()
- * }
- * const module = await esmock('./module.js', {
- *   '#lib/logger/index.js': { logger: mockLogger }
- * })
- */
-
-/**
- * Log level constants matching loglevel library
- * @see https://github.com/pimterry/loglevel#documentation
- */
-export const LOG_LEVELS = {
-  trace: 0,
-  debug: 1,
-  info: 2,
-  warn: 3,
-  error: 4,
-  silent: 5
-}
-
-/**
- * Chalk style functions for prefix and level label
- */
-const lableStyleFns = {
-  trace: chalk.gray,
-  debug: chalk.yellow,
-  info: chalk.magenta,
-  warn: chalk.yellow.inverse,
-  error: chalk.redBright.inverse
-}
-
-/**
- * Chalk style functions for message text
- *
- * Only levels that need distinct message styling are listed.
- * Unlisted levels render message text without styling.
- */
-const messageStyleFns = {
-  error: chalk.red,
-}
-
-/**
- * Emoji map for prefix styles
- */
-const EMOJI_MAP = {
-  build: '🔨',
-  cli: '⚡',
-  conf: '⚙️',
-  epub: '📚',
-  pdf: '📄',
-  quire: '📖',
-}
-
-/**
- * Environment variable for log level configuration
- * Set by CLI startup from config, can be overridden by CLI flags
- */
-export const LOG_LEVEL_ENV_VAR = 'QUIRE_LOG_LEVEL'
-
-/**
- * Format the log prefix based on configuration style
- *
- * @param {string} style - Prefix style (bracket, emoji, plain, none)
- * @param {string} text - Prefix text
- * @returns {string} Formatted prefix
- */
-export function formatPrefix(style, text) {
-  switch (style) {
-    case 'bracket':
-      return `[${text}]`
-    case 'emoji':
-      return EMOJI_MAP[text] || '📖'
-    case 'plain':
-      return `${text}:`
-    case 'none':
-      return ''
-    default:
-      return `[${text}]`
-  }
-}
-
-/**
- * Resolve log level from various sources
- *
- * Priority order:
- * 1. Explicit level parameter (for createLogger calls with specific level)
- * 2. QUIRE_LOG_LEVEL environment variable (set by CLI from config/flags)
- * 3. Config file setting
- * 4. Default to 'info'
- *
- * @param {string|number} [level] - Explicit level override
- * @returns {number} Numeric log level
- */
-function resolveLevel(level) {
-  // If level is explicitly provided, use it
-  if (level !== undefined) {
-    if (typeof level === 'number') return level
-    return LOG_LEVELS[level] ?? LOG_LEVELS.info
-  }
-
-  // Check environment variable (set by CLI startup from config)
-  const envVar = process.env[LOG_LEVEL_ENV_VAR]
-  if (envVar) {
-    return LOG_LEVELS[envVar] ?? LOG_LEVELS.info
-  }
-
-  // Check config
-  const configLevel = config.get('logLevel')
-  if (configLevel && LOG_LEVELS[configLevel] !== undefined) {
-    return LOG_LEVELS[configLevel]
-  }
-
-  // Default to info
-  return LOG_LEVELS.info
-}
-
-/**
- * Create a logger instance
- *
- * @param {string} [name='quire'] - Logger name for loglevel instance
- * @param {string|number} [level] - Log level (trace, debug, info, warn, error, silent)
- * @returns {Object} Logger instance with trace, debug, info, warn, error, setLevel methods
- */
-export default function createLogger(name = 'quire', level) {
-  const loggerInstance = log.getLogger(name)
-  loggerInstance.setLevel(resolveLevel(level), false)
-
-  /**
-   * Create a log function for a specific level
-   */
-  const createLogFn = (type) => {
-    const logMethod = loggerInstance[type]
-    const styleLabel = lableStyleFns[type]
-    const styleMessage = messageStyleFns[type]
-
-    return (...args) => {
-      // Read config each time to support runtime changes
-      const prefix = config.get('logPrefix')
-      const prefixStyle = config.get('logPrefixStyle')
-      const showLevel = config.get('logShowLevel')
-      const useColor = config.get('logUseColor') && !process.env.NO_COLOR
-      const colorMessages = config.get('logColorMessages')
-
-      const parts = []
-
-      // Add prefix
-      const formattedPrefix = formatPrefix(prefixStyle, prefix)
-      if (formattedPrefix) {
-        if (useColor) {
-          parts.push(styleLabel(chalk.bold(formattedPrefix)))
-        } else {
-          parts.push(formattedPrefix)
-        }
-      }
-
-      // Add level label
-      if (showLevel) {
-        const levelLabel = type.toUpperCase().padEnd(5, ' ')
-        if (useColor) {
-          parts.push(styleLabel(levelLabel))
-        } else {
-          parts.push(levelLabel)
-        }
-      }
-
-      // Format message with printf-style substitution (%s, %d, %i, %o, %O, %j)
-      const message = format(...args)
-
-      logMethod(...parts, useColor && colorMessages && styleMessage ? styleMessage(message) : message)
-    }
-  }
-
-  return {
-    trace: createLogFn('trace'),
-    debug: createLogFn('debug'),
-    info: createLogFn('info'),
-    warn: createLogFn('warn'),
-    error: createLogFn('error'),
-
-    /**
-     * Change the log level at runtime
-     * @param {string|number} newLevel - New log level
-     */
-    setLevel(newLevel) {
-      const numericLevel = typeof newLevel === 'number'
-        ? newLevel
-        : LOG_LEVELS[newLevel] ?? LOG_LEVELS.info
-      loggerInstance.setLevel(numericLevel, false)
-    },
-
-    /**
-     * Get the current log level
-     * @returns {number} Current numeric log level
-     */
-    getLevel() {
-      return loggerInstance.getLevel()
-    }
-  }
-}
-
-/**
- * Default singleton logger for simple usage
- */
-export const logger = createLogger('quire')

package/src/lib/logger/index.spec.js

@@ -1,131 +0,0 @@
-import test from 'ava'
-import createLogger, { logger, LOG_LEVELS } from './index.js'
-
-/**
- * Logger Module Unit Tests
- *
- * Tests the logger factory exports and API surface.
- */
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Export tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('createLogger is exported as default', (t) => {
-  t.is(typeof createLogger, 'function')
-})
-
-test('logger singleton is exported', (t) => {
-  t.truthy(logger)
-  t.is(typeof logger.info, 'function')
-})
-
-test('LOG_LEVELS constants are exported', (t) => {
-  t.deepEqual(LOG_LEVELS, {
-    trace: 0,
-    debug: 1,
-    info: 2,
-    warn: 3,
-    error: 4,
-    silent: 5
-  })
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Factory function tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('createLogger returns logger with all methods', (t) => {
-  const log = createLogger('test')
-
-  t.is(typeof log.trace, 'function')
-  t.is(typeof log.debug, 'function')
-  t.is(typeof log.info, 'function')
-  t.is(typeof log.warn, 'function')
-  t.is(typeof log.error, 'function')
-  t.is(typeof log.setLevel, 'function')
-  t.is(typeof log.getLevel, 'function')
-})
-
-test('createLogger uses default prefix when none provided', (t) => {
-  const log = createLogger()
-  t.truthy(log)
-  t.is(typeof log.info, 'function')
-})
-
-test('createLogger accepts custom prefix', (t) => {
-  const log = createLogger('custom:prefix')
-  t.truthy(log)
-})
-
-test('createLogger accepts level as string', (t) => {
-  const log = createLogger('test', 'debug')
-  t.is(log.getLevel(), LOG_LEVELS.debug)
-})
-
-test('createLogger accepts level as number', (t) => {
-  const log = createLogger('test', 3)
-  t.is(log.getLevel(), 3)
-})
-
-test('createLogger defaults to info level', (t) => {
-  const log = createLogger('test')
-  t.is(log.getLevel(), LOG_LEVELS.info)
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// setLevel tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('setLevel changes log level with string', (t) => {
-  const log = createLogger('test', 'info')
-  t.is(log.getLevel(), LOG_LEVELS.info)
-
-  log.setLevel('debug')
-  t.is(log.getLevel(), LOG_LEVELS.debug)
-})
-
-test('setLevel changes log level with number', (t) => {
-  const log = createLogger('test', 'info')
-
-  log.setLevel(0)
-  t.is(log.getLevel(), 0)
-})
-
-test('setLevel defaults to info for invalid level', (t) => {
-  const log = createLogger('test', 'debug')
-
-  log.setLevel('invalid')
-  t.is(log.getLevel(), LOG_LEVELS.info)
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Singleton tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('logger singleton has all required methods', (t) => {
-  t.is(typeof logger.trace, 'function')
-  t.is(typeof logger.debug, 'function')
-  t.is(typeof logger.info, 'function')
-  t.is(typeof logger.warn, 'function')
-  t.is(typeof logger.error, 'function')
-  t.is(typeof logger.setLevel, 'function')
-  t.is(typeof logger.getLevel, 'function')
-})
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Multiple logger instances tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-test('createLogger creates independent instances', (t) => {
-  const log1 = createLogger('module:one', 'debug')
-  const log2 = createLogger('module:two', 'error')
-
-  t.is(log1.getLevel(), LOG_LEVELS.debug)
-  t.is(log2.getLevel(), LOG_LEVELS.error)
-
-  // Changing one shouldn't affect the other
-  log1.setLevel('warn')
-  t.is(log1.getLevel(), LOG_LEVELS.warn)
-  t.is(log2.getLevel(), LOG_LEVELS.error)
-})