Skip to content

Commit

Permalink
revert: fix(logger): truncate log over 5000 characters long (#16581) (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
Henry-Hong authored Jul 22, 2024
1 parent df5ceb3 commit f4f488f
Showing 1 changed file with 10 additions and 28 deletions.
38 changes: 10 additions & 28 deletions packages/vite/src/node/logger.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import readline from 'node:readline'
import colors from 'picocolors'
import type { RollupError } from 'rollup'
import type { ResolvedServerUrls } from './server'
import { splitRE } from './utils'

export type LogType = 'error' | 'warn' | 'info'
export type LogLevel = LogType | 'silent'
Expand Down Expand Up @@ -64,8 +63,6 @@ function getTimeFormatter() {
return timeFormatter
}

const MAX_LOG_CHAR = 5000

export function createLogger(
level: LogLevel = 'info',
options: LoggerOptions = {},
Expand All @@ -81,32 +78,17 @@ export function createLogger(
allowClearScreen && process.stdout.isTTY && !process.env.CI
const clear = canClearScreen ? clearScreen : () => {}

function preventOverflow(msg: string) {
if (msg.length > MAX_LOG_CHAR) {
const shorten = msg.slice(0, MAX_LOG_CHAR)
const lines = msg.slice(MAX_LOG_CHAR).match(splitRE)?.length || 0

return `${shorten}\n... and ${lines} lines more`
}
return msg
}

function format(
type: LogType,
rawMsg: string,
options: LogErrorOptions = {},
) {
const msg = preventOverflow(rawMsg)
function format(type: LogType, msg: string, options: LogErrorOptions = {}) {
if (options.timestamp) {
const tag =
type === 'info'
? colors.cyan(colors.bold(prefix))
: type === 'warn'
? colors.yellow(colors.bold(prefix))
: colors.red(colors.bold(prefix))
return `${colors.dim(
getTimeFormatter().format(new Date()),
)} ${tag} ${msg}`
let tag = ''
if (type === 'info') {
tag = colors.cyan(colors.bold(prefix))
} else if (type === 'warn') {
tag = colors.yellow(colors.bold(prefix))
} else {
tag = colors.red(colors.bold(prefix))
}
return `${colors.dim(getTimeFormatter().format(new Date()))} ${tag} ${msg}`
} else {
return msg
}
Expand Down

0 comments on commit f4f488f

Please sign in to comment.