util: improve styleText performance

The validation for when to colorize or not was happening in a loop.
Instead, it is now done once up front instead.

Author: BridgeAR

lib/util.js

@@ -123,7 +123,6 @@ function styleText(format, text, { validateStream = true, stream = process.stdou
   validateString(text, 'text');
   validateBoolean(validateStream, 'options.validateStream');
 
-  let skipColorize;
   if (validateStream) {
     if (
       !isReadableStream(stream) &&
@@ -134,9 +133,13 @@ function styleText(format, text, { validateStream = true, stream = process.stdou
     }
 
     // If the stream is falsy or should not be colorized, set skipColorize to true
-    skipColorize = !lazyUtilColors().shouldColorize(stream);
+    const skipColorize = !lazyUtilColors().shouldColorize(stream);
+    if (skipColorize) {
+      return text;
+    }
   }
 
+
   // If the format is not an array, convert it to an array
   const formatArray = ArrayIsArray(format) ? format : [format];
 
@@ -148,14 +151,9 @@ function styleText(format, text, { validateStream = true, stream = process.stdou
     if (formatCodes == null) {
       validateOneOf(key, 'format', ObjectKeys(inspect.colors));
     }
-    if (skipColorize) continue;
     ArrayPrototypePush(codes, formatCodes);
   }
 
-  if (skipColorize) {
-    return text;
-  }
-
   // Build opening codes
   let openCodes = '';
   for (let i = 0; i < codes.length; i++) {