Fix the verbose option printing colors when stderr is not a terminal

Author: sindresorhus

lib/verbose/default.js

@@ -1,11 +1,22 @@
+import process from 'node:process';
 import figures from 'figures';
 import {
-	gray,
-	bold,
-	redBright,
-	yellowBright,
+	gray as grayColor,
+	bold as boldColor,
+	redBright as redBrightColor,
+	yellowBright as yellowBrightColor,
 } from 'yoctocolors';
 
+// `yoctocolors` checks whether the terminal type supports colors, but not whether `stderr`, where this is printed, is a terminal.
+// Like `util.styleText()`, colors are not used when `stderr` is redirected, e.g. to a file, unless `FORCE_COLOR` is set.
+const hasColors = process.env.FORCE_COLOR !== undefined || process.stderr.isTTY === true;
+const identity = string => string;
+const getColor = color => hasColors ? color : identity;
+const gray = getColor(grayColor);
+const bold = getColor(boldColor);
+const redBright = getColor(redBrightColor);
+const yellowBright = getColor(yellowBrightColor);
+
 // Default when `verbose` is not a function
 export const defaultVerboseFunction = ({
 	type,
@@ -43,8 +54,6 @@ const ICONS = {
 	duration: getFinalIcon,
 };
 
-const identity = string => string;
-
 const COLORS = {
 	command: () => bold,
 	output: () => identity,

test/verbose/log.js

@@ -22,13 +22,15 @@ test('Logs on stderr not stdout, verbose "none", sync', () => testNoStdout('none
 test('Logs on stderr not stdout, verbose "short", sync', () => testNoStdout('short', true));
 test('Logs on stderr not stdout, verbose "full", sync', () => testNoStdout('full', true));
 
-const testColor = async (expectedResult, forceColor) => {
-	const {stderr} = await nestedSubprocess('noop.js', [foobarString], {verbose: 'short'}, {env: {FORCE_COLOR: forceColor, NO_COLOR: undefined}});
+const testColor = async (expectedResult, forceColor, term) => {
+	const {stderr} = await nestedSubprocess('noop.js', [foobarString], {verbose: 'short'}, {env: {FORCE_COLOR: forceColor, NO_COLOR: undefined, TERM: term}});
 	assert.equal(stderr !== stripVTControlCharacters(stderr), expectedResult);
 };
 
 test('Prints with colors if supported', () => testColor(true, '1'));
 test('Prints without colors if not supported', () => testColor(false, '0'));
+// `stderr` is not a terminal, e.g. when redirected to a file, even though the terminal type supports colors
+test('Prints without colors if stderr is not a terminal', () => testColor(false, undefined, 'xterm-256color'));
 
 test('Prints lines in order when interleaved with subprocess stderr', async () => {
 	const results = await Promise.all(Array.from({length: PARALLEL_COUNT}, () =>