fix: allow to disable stats color via --no-color

packages/webpack-cli/lib/bootstrap.js

@@ -4,6 +4,7 @@ const logger = require('./utils/logger');
 const { isCommandUsed } = require('./utils/arg-utils');
 const argParser = require('./utils/arg-parser');
 const leven = require('leven');
+const { options: coloretteOptions } = require('colorette');
 
 process.title = 'webpack-cli';
 
@@ -23,6 +24,11 @@ const runCLI = async (cliArgs) => {
         // If the unknown arg starts with a '-', it will be considered an unknown flag rather than an entry
         let entry;
 
+        // enable/disable colors
+        if (coloretteOptions.enabled && !parsedArgs.opts.color) {
+            coloretteOptions.enabled = parsedArgs.opts.color;
+        }
+
         if (parsedArgs.unknownArgs.length > 0) {
             entry = [];
 

packages/webpack-cli/lib/webpack-cli.js

@@ -195,6 +195,10 @@ class WebpackCLI {
         let options = this.compilerConfiguration;
         let outputOptions = this.outputConfiguration;
 
+        if (coloretteOptions.enabled && !args.color) {
+            coloretteOptions.enabled = args.color;
+        }
+
         const isRawOutput = typeof outputOptions.json === 'undefined';
 
         if (isRawOutput) {
@@ -267,12 +271,6 @@ class WebpackCLI {
 
         compiler = this.createCompiler(options, callback);
 
-        if (compiler && outputOptions.interactive) {
-            const interactive = require('./utils/interactive');
-
-            interactive(compiler, options, outputOptions);
-        }
-
         return Promise.resolve();
     }
 }

test/colors/colors.test.js

@@ -43,6 +43,25 @@ describe('colorts', () => {
         expect(exitCode).toBe(0);
     });
 
+    it('should disable colored output with --no-color', () => {
+        const { stderr, stdout, exitCode } = run(__dirname, ['--stats=verbose', '--no-color']);
+
+        expect(stderr).toBeFalsy();
+        const output = isWebpack5 ? 'successfully' : 'main.js';
+        expect(stdout).not.toContain(`\u001b[1m\u001b[32m${output}\u001b[39m\u001b[22m`);
+        expect(stdout).toContain(output);
+        expect(exitCode).toBe(0);
+    });
+
+    it('should work with the "stats" option and --color flags', () => {
+        const { stderr, stdout, exitCode } = run(__dirname, ['--stats=verbose', '--color']);
+
+        expect(stderr).toBeFalsy();
+        const output = isWebpack5 ? 'successfully' : 'main.js';
+        expect(stdout).toContain(`\u001b[1m\u001b[32m${output}\u001b[39m\u001b[22m`);
+        expect(exitCode).toBe(0);
+    });
+
     it('should work with the "stats" option from the configuration', () => {
         const { stderr, stdout, exitCode } = run(__dirname, ['--config=stats-string.webpack.config.js']);
 

test/json/json.test.js

@@ -2,6 +2,9 @@
 const { run } = require('../utils/test-utils');
 const { stat, readFile } = require('fs');
 const { resolve } = require('path');
+const { green } = require('colorette');
+
+const successMessage = 'stats are successfully stored as json to stats.json';
 
 describe('json flag', () => {
     it('should return valid json', () => {
@@ -16,7 +19,29 @@ describe('json flag', () => {
     it('should store json to a file', (done) => {
         const { stdout, exitCode } = run(__dirname, ['--json', 'stats.json']);
 
-        expect(stdout).toContain('stats are successfully stored as json to stats.json');
+        expect(stdout).toContain(`${green(successMessage)}`);
+        expect(exitCode).toBe(0);
+
+        stat(resolve(__dirname, './stats.json'), (err, stats) => {
+            expect(err).toBe(null);
+            expect(stats.isFile()).toBe(true);
+
+            readFile(resolve(__dirname, 'stats.json'), 'utf-8', (err, data) => {
+                expect(err).toBe(null);
+                expect(JSON.parse(data)['hash']).toBeTruthy();
+                expect(JSON.parse(data)['version']).toBeTruthy();
+                expect(JSON.parse(data)['time']).toBeTruthy();
+                expect(() => JSON.parse(data)).not.toThrow();
+                done();
+            });
+        });
+    });
+
+    it('should store json to a file and respect --no-color', (done) => {
+        const { stdout, exitCode } = run(__dirname, ['--json', 'stats.json', '--no-color']);
+
+        expect(stdout).not.toContain(`${green(successMessage)}`);
+        expect(stdout).toContain(successMessage);
         expect(exitCode).toBe(0);
 
         stat(resolve(__dirname, './stats.json'), (err, stats) => {

test/unknown/unknown.test.js

@@ -1,12 +1,22 @@
 const { run } = require('../utils/test-utils');
+const { red } = require('colorette');
 
 describe('unknown behaviour', () => {
     it('warns the user if an unknown flag is passed in', () => {
         const { stderr, exitCode } = run(__dirname, ['--unknown']);
         expect(stderr).toBeTruthy();
+        expect(stderr).toContain(`${red('Unknown argument: --unknown')}`);
+        expect(exitCode).toBe(2);
+    });
+
+    it('throws error for unknow flag and respect --no-color', () => {
+        const { stderr, exitCode } = run(__dirname, ['--unknown', '--no-color']);
+        expect(stderr).toBeTruthy();
+        expect(stderr).not.toContain(`${red('Unknown argument: --unknown')}`);
         expect(stderr).toContain('Unknown argument: --unknown');
         expect(exitCode).toBe(2);
     });
+
     it('suggests the closest match to an unknown flag', () => {
         const { stderr, stdout, exitCode } = run(__dirname, ['--entyr', './a.js']);
         expect(stderr).toContain('Unknown argument: --entyr');