@kubb/cli 4.32.4 → 4.33.1

package/src/loggers/clackLogger.ts

@@ -2,13 +2,13 @@ import { relative } from 'node:path'
 import process from 'node:process'
 import { styleText } from 'node:util'
 import * as clack from '@clack/prompts'
-import { defineLogger, LogLevel } from '@kubb/core'
-import { formatHrtime, formatMs } from '@kubb/core/utils'
-import { type NonZeroExitError, x } from 'tinyexec'
-import { formatMsWithColor } from '../utils/formatMsWithColor.ts'
+import { formatMs, formatMsWithColor, toCause } from '@internals/utils'
+import { defineLogger, logLevel as logLevelMap } from '@kubb/core'
 import { getIntro } from '../utils/getIntro.ts'
 import { getSummary } from '../utils/getSummary.ts'
+import { runHook } from '../utils/runHook.ts'
 import { ClackWritable } from '../utils/Writables.ts'
+import { buildProgressLine, formatCommandWithArgs, formatMessage } from './utils.ts'
 
 /**
  * Clack adapter for local TTY environments
@@ -17,7 +17,7 @@ import { ClackWritable } from '../utils/Writables.ts'
 export const clackLogger = defineLogger({
   name: 'clack',
   install(context, options) {
-    const logLevel = options?.logLevel || LogLevel.info
+    const logLevel = options?.logLevel ?? logLevelMap.info
     const state = {
       totalPlugins: 0,
       completedPlugins: 0,
@@ -50,44 +50,18 @@ export const clackLogger = defineLogger({
     }
 
     function showProgressStep() {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
-      const parts: string[] = []
-      const duration = formatHrtime(state.hrStart)
-
-      if (state.totalPlugins > 0) {
-        const pluginStr =
-          state.failedPlugins > 0
-            ? `Plugins ${styleText('green', state.completedPlugins.toString())}/${state.totalPlugins} ${styleText('red', `(${state.failedPlugins} failed)`)}`
-            : `Plugins ${styleText('green', state.completedPlugins.toString())}/${state.totalPlugins}`
-        parts.push(pluginStr)
-      }
-
-      if (state.totalFiles > 0) {
-        parts.push(`Files ${styleText('green', state.processedFiles.toString())}/${state.totalFiles}`)
-      }
-
-      if (parts.length > 0) {
-        parts.push(`${styleText('green', duration)} elapsed`)
-        clack.log.step(getMessage(parts.join(styleText('dim', ' | '))))
+      const line = buildProgressLine(state)
+      if (line) {
+        clack.log.step(getMessage(line))
       }
     }
 
     function getMessage(message: string): string {
-      if (logLevel >= LogLevel.verbose) {
-        const timestamp = new Date().toLocaleTimeString('en-US', {
-          hour12: false,
-          hour: '2-digit',
-          minute: '2-digit',
-          second: '2-digit',
-        })
-
-        return [styleText('dim', `[${timestamp}]`), message].join(' ')
-      }
-
-      return message
+      return formatMessage(message, logLevel)
     }
 
     function startSpinner(text?: string) {
@@ -101,7 +75,7 @@ export const clackLogger = defineLogger({
     }
 
     context.on('info', (message, info = '') => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -115,11 +89,11 @@ export const clackLogger = defineLogger({
     })
 
     context.on('success', (message, info = '') => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
-      const text = getMessage([styleText('blue', '✓'), message, logLevel >= LogLevel.info ? styleText('dim', info) : undefined].filter(Boolean).join(' '))
+      const text = getMessage([styleText('blue', '✓'), message, logLevel >= logLevelMap.info ? styleText('dim', info) : undefined].filter(Boolean).join(' '))
 
       if (state.isSpinning) {
         stopSpinner(text)
@@ -129,19 +103,19 @@ export const clackLogger = defineLogger({
     })
 
     context.on('warn', (message, info) => {
-      if (logLevel < LogLevel.warn) {
+      if (logLevel < logLevelMap.warn) {
         return
       }
 
       const text = getMessage(
-        [styleText('yellow', '⚠'), message, logLevel >= LogLevel.info && info ? styleText('dim', info) : undefined].filter(Boolean).join(' '),
+        [styleText('yellow', '⚠'), message, logLevel >= logLevelMap.info && info ? styleText('dim', info) : undefined].filter(Boolean).join(' '),
       )
 
       clack.log.warn(text)
     })
 
     context.on('error', (error) => {
-      const caused = error.cause as Error | undefined
+      const caused = toCause(error)
 
       const text = [styleText('red', '✗'), error.message].join(' ')
 
@@ -152,7 +126,7 @@ export const clackLogger = defineLogger({
       }
 
       // Show stack trace in debug mode (first 3 frames)
-      if (logLevel >= LogLevel.debug && error.stack) {
+      if (logLevel >= logLevelMap.debug && error.stack) {
         const frames = error.stack.split('\n').slice(1, 4)
         for (const frame of frames) {
           clack.log.message(getMessage(styleText('dim', frame.trim())))
@@ -170,7 +144,7 @@ export const clackLogger = defineLogger({
     })
 
     context.on('version:new', (version, latestVersion) => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -196,7 +170,7 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('config:start', () => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -207,7 +181,7 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('config:end', (_configs) => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -217,17 +191,18 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('generation:start', (config) => {
-      // Initialize progress tracking
-      state.totalPlugins = config.plugins?.length || 0
+      reset()
+
+      // Initialize progress tracking for this generation
+      state.totalPlugins = config.plugins?.length ?? 0
 
       const text = getMessage(['Generation started', config.name ? `for ${styleText('dim', config.name)}` : undefined].filter(Boolean).join(' '))
 
       clack.intro(text)
-      reset()
     })
 
     context.on('plugin:start', (plugin) => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -253,7 +228,7 @@ Run \`npm install -g @kubb/cli\` to update`,
 
       const active = state.activeProgress.get(plugin.name)
 
-      if (!active || logLevel === LogLevel.silent) {
+      if (!active || logLevel === logLevelMap.silent) {
         return
       }
 
@@ -280,7 +255,7 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('files:processing:start', (files) => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -302,7 +277,7 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('file:processing:update', ({ file, config }) => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -320,7 +295,7 @@ Run \`npm install -g @kubb/cli\` to update`,
       active.progressBar.advance(undefined, text)
     })
     context.on('files:processing:end', () => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -347,7 +322,7 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('format:start', () => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -357,7 +332,7 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('format:end', () => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -367,7 +342,7 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('lint:start', () => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -377,7 +352,7 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('lint:end', () => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
@@ -387,7 +362,7 @@ Run \`npm install -g @kubb/cli\` to update`,
     })
 
     context.on('hook:start', async ({ id, command, args }) => {
-      const commandWithArgs = args?.length ? `${command} ${args.join(' ')}` : command
+      const commandWithArgs = formatCommandWithArgs(command, args)
       const text = getMessage(`Hook ${styleText('dim', commandWithArgs)} started`)
 
       // Skip hook execution if no id is provided (e.g., during benchmarks or tests)
@@ -395,113 +370,50 @@ Run \`npm install -g @kubb/cli\` to update`,
         return
       }
 
-      if (logLevel <= LogLevel.silent) {
-        try {
-          const result = await x(command, [...(args ?? [])], {
-            nodeOptions: { detached: true },
-            throwOnError: true,
-          })
-
-          await context.emit('debug', {
-            date: new Date(),
-            logs: [result.stdout.trimEnd()],
-          })
-
-          await context.emit('hook:end', {
-            command,
-            args,
-            id,
-            success: true,
-            error: null,
-          })
-        } catch (err) {
-          const error = err as NonZeroExitError
-          const stderr = error.output?.stderr ?? ''
-          const stdout = error.output?.stdout ?? ''
-
-          await context.emit('debug', {
-            date: new Date(),
-            logs: [stdout, stderr].filter(Boolean),
-          })
-
-          if (stderr) {
-            console.error(stderr)
-          }
-          if (stdout) {
-            console.log(stdout)
-          }
-
-          const errorMessage = new Error(`Hook execute failed: ${commandWithArgs}`)
-
-          await context.emit('hook:end', {
-            command,
-            args,
-            id,
-            success: false,
-            error: errorMessage,
-          })
-          await context.emit('error', errorMessage)
-        }
-
+      if (logLevel <= logLevelMap.silent) {
+        await runHook({
+          id,
+          command,
+          args,
+          commandWithArgs,
+          context,
+          sink: {
+            onStderr: (s) => console.error(s),
+            onStdout: (s) => console.log(s),
+          },
+        })
         return
       }
 
       clack.intro(text)
 
       const logger = clack.taskLog({
-        title: getMessage(['Executing hook', logLevel >= LogLevel.info ? styleText('dim', commandWithArgs) : undefined].filter(Boolean).join(' ')),
+        title: getMessage(['Executing hook', logLevel >= logLevelMap.info ? styleText('dim', commandWithArgs) : undefined].filter(Boolean).join(' ')),
       })
 
       const writable = new ClackWritable(logger)
 
-      try {
-        const proc = x(command, [...(args ?? [])], {
-          nodeOptions: { detached: true },
-          throwOnError: true,
-        })
-
-        for await (const line of proc) {
-          writable.write(line)
-        }
-
-        const result = await proc
-
-        await context.emit('debug', {
-          date: new Date(),
-          logs: [result.stdout.trimEnd()],
-        })
-
-        await context.emit('hook:end', { command, args, id, success: true, error: null })
-      } catch (err) {
-        const error = err as NonZeroExitError
-        const stderr = error.output?.stderr ?? ''
-        const stdout = error.output?.stdout ?? ''
-
-        await context.emit('debug', {
-          date: new Date(),
-          logs: [stdout, stderr].filter(Boolean),
-        })
-
-        if (stderr) {
-          logger.error(stderr)
-        }
-        if (stdout) {
-          logger.message(stdout)
-        }
-
-        const errorMessage = new Error(`Hook execute failed: ${commandWithArgs}`)
-
-        await context.emit('hook:end', { command, args, id, success: false, error: errorMessage })
-        await context.emit('error', errorMessage)
-      }
+      await runHook({
+        id,
+        command,
+        args,
+        commandWithArgs,
+        context,
+        stream: true,
+        sink: {
+          onLine: (line) => writable.write(line),
+          onStderr: (s) => logger.error(s),
+          onStdout: (s) => logger.message(s),
+        },
+      })
     })
 
     context.on('hook:end', ({ command, args }) => {
-      if (logLevel <= LogLevel.silent) {
+      if (logLevel <= logLevelMap.silent) {
         return
       }
 
-      const commandWithArgs = args?.length ? `${command} ${args.join(' ')}` : command
+      const commandWithArgs = formatCommandWithArgs(command, args)
       const text = getMessage(`Hook ${styleText('dim', commandWithArgs)} successfully executed`)
 
       clack.outro(text)
@@ -514,29 +426,17 @@ Run \`npm install -g @kubb/cli\` to update`,
         config,
         status,
         hrStart,
-        pluginTimings: logLevel >= LogLevel.verbose ? pluginTimings : undefined,
+        pluginTimings: logLevel >= logLevelMap.verbose ? pluginTimings : undefined,
       })
       const title = config.name || ''
 
       summary.unshift('\n')
       summary.push('\n')
 
-      if (status === 'success') {
-        clack.box(summary.join('\n'), getMessage(title), {
-          width: 'auto',
-          formatBorder: (s: string) => styleText('green', s),
-          rounded: true,
-          withGuide: false,
-          contentAlign: 'left',
-          titleAlign: 'center',
-        })
-
-        return
-      }
-
+      const borderColor = status === 'success' ? 'green' : 'red'
       clack.box(summary.join('\n'), getMessage(title), {
         width: 'auto',
-        formatBorder: (s: string) => styleText('red', s),
+        formatBorder: (s: string) => styleText(borderColor, s),
         rounded: true,
         withGuide: false,
         contentAlign: 'left',

package/src/loggers/fileSystemLogger.ts

@@ -1,7 +1,7 @@
 import { relative, resolve } from 'node:path'
+import process from 'node:process'
+import { formatMs, write } from '@internals/utils'
 import { defineLogger } from '@kubb/core'
-import { write } from '@kubb/core/fs'
-import { formatMs } from '@kubb/core/utils'
 
 type CachedEvent = {
   date: Date
@@ -63,7 +63,6 @@ export const fileSystemLogger = defineLogger({
       state.cachedLogs.add({
         date: new Date(),
         logs: [`ℹ ${message} ${info}`],
-        fileName: undefined,
       })
     })
 
@@ -71,7 +70,6 @@ export const fileSystemLogger = defineLogger({
       state.cachedLogs.add({
         date: new Date(),
         logs: [`✓ ${message} ${info}`],
-        fileName: undefined,
       })
     })
 
@@ -79,7 +77,6 @@ export const fileSystemLogger = defineLogger({
       state.cachedLogs.add({
         date: new Date(),
         logs: [`⚠ ${message} ${info}`],
-        fileName: undefined,
       })
     })
 
@@ -87,7 +84,6 @@ export const fileSystemLogger = defineLogger({
       state.cachedLogs.add({
         date: new Date(),
         logs: [`✗ ${error.message}`, error.stack || 'unknown stack'],
-        fileName: undefined,
       })
     })
 
@@ -95,7 +91,6 @@ export const fileSystemLogger = defineLogger({
       state.cachedLogs.add({
         date: new Date(),
         logs: message.logs,
-        fileName: undefined,
       })
     })
 
@@ -103,7 +98,6 @@ export const fileSystemLogger = defineLogger({
       state.cachedLogs.add({
         date: new Date(),
         logs: [`Generating ${plugin.name}`],
-        fileName: undefined,
       })
     })
 
@@ -113,7 +107,6 @@ export const fileSystemLogger = defineLogger({
       state.cachedLogs.add({
         date: new Date(),
         logs: [success ? `${plugin.name} completed in ${durationStr}` : `${plugin.name} failed in ${durationStr}`],
-        fileName: undefined,
       })
     })
 
@@ -121,7 +114,6 @@ export const fileSystemLogger = defineLogger({
       state.cachedLogs.add({
         date: new Date(),
         logs: [`Start ${files.length} writing:`, ...files.map((file) => file.path)],
-        fileName: undefined,
       })
     })
 
@@ -134,10 +126,6 @@ export const fileSystemLogger = defineLogger({
       reset()
     })
 
-    context.on('lifecycle:end', async () => {
-      // lifecycle:end handler can be used for cleanup if needed in the future
-    })
-
     // Fallback: Write logs on process exit to handle crashes
     const exitHandler = () => {
       // Synchronous write on exit - best effort