From 3b9a6cd5c93648b040e9205a0a0b64558da1221a Mon Sep 17 00:00:00 2001 From: Aviv Keller Date: Sat, 28 Sep 2024 09:50:39 -0400 Subject: [PATCH] Revert "console: colorize console error and warn" MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reverts commit a833c9e0bed85214884d6650fe344d5896c9e67a. PR-URL: https://github.com/nodejs/node/pull/54677 Reviewed-By: Joyee Cheung Reviewed-By: Michaël Zasso --- lib/internal/console/constructor.js | 25 ++++--------------- test/parallel/test-repl.js | 2 +- .../test-tty-color-support-warning-2.out | 4 +-- .../test-tty-color-support-warning.out | 4 +-- test/pseudo-tty/test-tty-color-support.out | 4 +-- 5 files changed, 12 insertions(+), 27 deletions(-) diff --git a/lib/internal/console/constructor.js b/lib/internal/console/constructor.js index 4a48dd4a0864e7..0b4cf5df67365e 100644 --- a/lib/internal/console/constructor.js +++ b/lib/internal/console/constructor.js @@ -8,7 +8,6 @@ const { ArrayIsArray, ArrayPrototypeForEach, ArrayPrototypePush, - ArrayPrototypeSome, ArrayPrototypeUnshift, Boolean, ErrorCaptureStackTrace, @@ -68,7 +67,6 @@ const { CHAR_LOWERCASE_N: kTraceInstant, CHAR_UPPERCASE_C: kTraceCount, } = require('internal/constants'); -const { styleText } = require('util'); const kCounts = Symbol('counts'); const kTraceConsoleCategory = 'node,node.console'; @@ -274,7 +272,7 @@ ObjectDefineProperties(Console.prototype, { [kWriteToConsole]: { __proto__: null, ...consolePropAttributes, - value: function(streamSymbol, string, color = '') { + value: function(streamSymbol, string) { const ignoreErrors = this._ignoreErrors; const groupIndent = this[kGroupIndent]; @@ -289,11 +287,6 @@ ObjectDefineProperties(Console.prototype, { } string = groupIndent + string; } - - if (color) { - string = styleText(color, string); - } - string += '\n'; if (ignoreErrors === false) return stream.write(string); @@ -384,15 +377,12 @@ const consoleMethods = { log(...args) { this[kWriteToConsole](kUseStdout, this[kFormatForStdout](args)); }, + + warn(...args) { - const color = (shouldColorize(args) && 'yellow') || ''; - this[kWriteToConsole](kUseStderr, this[kFormatForStderr](args), color); + this[kWriteToConsole](kUseStderr, this[kFormatForStderr](args)); }, - error(...args) { - const color = (shouldColorize(args) && 'red') || ''; - this[kWriteToConsole](kUseStderr, this[kFormatForStderr](args), color); - }, dir(object, options) { this[kWriteToConsole](kUseStdout, inspect(object, { @@ -685,12 +675,6 @@ const iterKey = '(iteration index)'; const isArray = (v) => ArrayIsArray(v) || isTypedArray(v) || isBuffer(v); -// TODO: remove string type check once the styleText supports objects -// Return true if all args are type string -const shouldColorize = (args) => { - return lazyUtilColors().hasColors && !ArrayPrototypeSome(args, (arg) => typeof arg !== 'string'); -}; - function noop() {} for (const method of ReflectOwnKeys(consoleMethods)) @@ -699,6 +683,7 @@ for (const method of ReflectOwnKeys(consoleMethods)) Console.prototype.debug = Console.prototype.log; Console.prototype.info = Console.prototype.log; Console.prototype.dirxml = Console.prototype.log; +Console.prototype.error = Console.prototype.warn; Console.prototype.groupCollapsed = Console.prototype.group; function initializeGlobalConsole(globalConsole) { diff --git a/test/parallel/test-repl.js b/test/parallel/test-repl.js index 22d1c19f1d58ab..1d59e213a38ce2 100644 --- a/test/parallel/test-repl.js +++ b/test/parallel/test-repl.js @@ -785,7 +785,6 @@ const errorTests = [ 'Object [console] {', ' log: [Function: log],', ' warn: [Function: warn],', - ' error: [Function: error],', ' dir: [Function: dir],', ' time: [Function: time],', ' timeEnd: [Function: timeEnd],', @@ -801,6 +800,7 @@ const errorTests = [ / {2}debug: \[Function: (debug|log)],/, / {2}info: \[Function: (info|log)],/, / {2}dirxml: \[Function: (dirxml|log)],/, + / {2}error: \[Function: (error|warn)],/, / {2}groupCollapsed: \[Function: (groupCollapsed|group)],/, / {2}Console: \[Function: Console],?/, ...process.features.inspector ? [ diff --git a/test/pseudo-tty/test-tty-color-support-warning-2.out b/test/pseudo-tty/test-tty-color-support-warning-2.out index 61a1baa0b54fb6..37b470a5f108f9 100644 --- a/test/pseudo-tty/test-tty-color-support-warning-2.out +++ b/test/pseudo-tty/test-tty-color-support-warning-2.out @@ -1,3 +1,3 @@ -*(node:*) Warning: The 'NODE_DISABLE_COLORS' env is ignored due to the 'FORCE_COLOR' env being set. -(Use `* --trace-warnings ...` to show where the warning was created)* +(node:*) Warning: The 'NODE_DISABLE_COLORS' env is ignored due to the 'FORCE_COLOR' env being set. +(Use `* --trace-warnings ...` to show where the warning was created) diff --git a/test/pseudo-tty/test-tty-color-support-warning.out b/test/pseudo-tty/test-tty-color-support-warning.out index 35ed65c96d29ad..b25d2e42cf7244 100644 --- a/test/pseudo-tty/test-tty-color-support-warning.out +++ b/test/pseudo-tty/test-tty-color-support-warning.out @@ -1,3 +1,3 @@ -*(node:*) Warning: The 'NODE_DISABLE_COLORS' and 'NO_COLOR' env is ignored due to the 'FORCE_COLOR' env being set. -(Use `* --trace-warnings ...` to show where the warning was created)* +(node:*) Warning: The 'NODE_DISABLE_COLORS' and 'NO_COLOR' env is ignored due to the 'FORCE_COLOR' env being set. +(Use `* --trace-warnings ...` to show where the warning was created) diff --git a/test/pseudo-tty/test-tty-color-support.out b/test/pseudo-tty/test-tty-color-support.out index 55b3986f73b2f0..df5831c555be19 100644 --- a/test/pseudo-tty/test-tty-color-support.out +++ b/test/pseudo-tty/test-tty-color-support.out @@ -1,2 +1,2 @@ -*(node:*) Warning: The 'NO_COLOR' env is ignored due to the 'FORCE_COLOR' env being set. -(Use `* --trace-warnings ...` to show where the warning was created)* +(node:*) Warning: The 'NO_COLOR' env is ignored due to the 'FORCE_COLOR' env being set. +(Use `* --trace-warnings ...` to show where the warning was created)