Fixes peggyjs#315. Check isTTY for all output before using colors.

hildjj · Jul 20, 2022 · b48a69f · b48a69f
1 parent e654186
commit b48a69f
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/bin/peggy-cli.js b/bin/peggy-cli.js
@@ -84,7 +84,6 @@ class PeggyCLI extends Command {
 
     /** @type {peggy.BuildOptionsBase} */
     this.argv = {};
-    this.colors = this.std.err.isTTY;
     /** @type {string?} */
     this.inputFile = null;
     /** @type {string?} */
@@ -386,7 +385,7 @@ class PeggyCLI extends Command {
 
   print(stream, ...args) {
     stream.write(util.formatWithOptions({
-      colors: this.colors,
+      colors: stream.isTTY,
       depth: Infinity,
       maxArrayLength: Infinity,
       maxStringLength: Infinity,
@@ -611,12 +610,7 @@ class PeggyCLI extends Command {
         opts.startRule = this.progOptions.startRule;
       }
       const results = exec.parse(this.testText, opts);
-      this.print(this.std.out, util.inspect(results, {
-        depth: Infinity,
-        colors: this.colors,
-        maxArrayLength: Infinity,
-        maxStringLength: Infinity,
-      }));
+      this.print(this.std.out, results);
     }
   }