perf(util): optimise styleText

Author: AugustinMauroy

lib/util.js

@@ -119,6 +119,7 @@ function styleText(format, text, { validateStream = true, stream = process.stdou
   validateString(text, 'text');
   validateBoolean(validateStream, 'options.validateStream');
 
+  let shouldColorize = true;
   if (validateStream) {
     if (
       !isReadableStream(stream) &&
@@ -128,28 +129,30 @@ function styleText(format, text, { validateStream = true, stream = process.stdou
       throw new ERR_INVALID_ARG_TYPE('stream', ['ReadableStream', 'WritableStream', 'Stream'], stream);
     }
 
-    if (!lazyUtilColors().shouldColorize(stream)) {
-      return text;
-    };
+    // If the stream is falsy or should not be colorized, set skipColorize to true
+    shouldColorize = lazyUtilColors().shouldColorize(stream);
   }
 
   // If the format is not an array, convert it to an array
   const formatArray = ArrayIsArray(format) ? format : [format];
+  const { colors } = inspect;
 
-  let left = '';
-  let right = '';
-  for (const key of formatArray) {
-    const formatCodes = inspect.colors[key];
+  // We want to loop through the format array to check if the format is valid
+  // including if it's shouldn't be colorized
+  const { left, right } = formatArray.reduce((acc, key) => {
+    const formatCodes = colors[key];
     // If the format is not a valid style, throw an error
     if (formatCodes == null) {
-      validateOneOf(key, 'format', ObjectKeys(inspect.colors));
+      validateOneOf(key, 'format', ObjectKeys(colors));
     }
+    if(shouldColorize) {
+      acc.left += escapeStyleCode(formatCodes[0]);
+      acc.right = `${escapeStyleCode(formatCodes[1])}${acc.right}`;
+    }
+    return acc;
+  }, { left: '', right: '' });
 
-    left += escapeStyleCode(formatCodes[0]);
-    right = `${escapeStyleCode(formatCodes[1])}${right}`;
-  }
-
-  return `${left}${text}${right}`;
+  return shouldColorize ? `${left}${text}${right}` : text;
 }
 
 /**