feat: display used config path when logging level=log

packages/webpack-cli/__tests__/applyCLIPlugin.test.js

@@ -5,7 +5,7 @@ const applyCLIPlugin = new webpackCLI().applyCLIPlugin;
 
 describe('CLIPluginResolver', () => {
     it('should add CLI plugin to single compiler object', async () => {
-        const result = await applyCLIPlugin({ options: {} }, { hot: true, prefetch: true });
+        const result = await applyCLIPlugin({ options: {}, path: new WeakMap() }, { hot: true, prefetch: true });
         expect(result.options.plugins[0] instanceof CLIPlugin).toBeTruthy();
         expect(result.options.plugins[0].options).toEqual({
             configPath: undefined,
@@ -18,7 +18,7 @@ describe('CLIPluginResolver', () => {
     });
 
     it('should add CLI plugin to multi compiler object', async () => {
-        const result = await applyCLIPlugin({ options: [{}, {}] }, { hot: true, prefetch: true });
+        const result = await applyCLIPlugin({ options: [{}, {}], path: new WeakMap() }, { hot: true, prefetch: true });
         expect(result.options[0].plugins[0] instanceof CLIPlugin).toBeTruthy();
         expect(result.options[1].plugins[0] instanceof CLIPlugin).toBeTruthy();
         expect(result.options).toEqual([

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

@@ -39,10 +39,18 @@ class CLIPlugin {
 
     setupHelpfulOutput(compiler) {
         const pluginName = 'webpack-cli';
-        const getCompilationName = () => (compiler.name ? ` '${compiler.name}'` : '');
+        const getCompilationName = () => (compiler.name ? `'${compiler.name}'` : '');
+
+        const { configPath } = this.options;
 
         compiler.hooks.run.tap(pluginName, () => {
-            this.logger.log(`Compilation${getCompilationName()} starting...`);
+            const name = getCompilationName();
+
+            this.logger.log(`Compiler${name ? ` ${name}` : ''} starting...`);
+
+            if (configPath) {
+                this.logger.log(`Compiler${name ? ` ${name}` : ''} is using config: '${configPath}'`);
+            }
         });
 
         compiler.hooks.watchRun.tap(pluginName, (compiler) => {
@@ -52,7 +60,13 @@ class CLIPlugin {
                 this.logger.warn('You are using "bail" with "watch". "bail" will still exit webpack when the first error is found.');
             }
 
-            this.logger.log(`Compilation${getCompilationName()} starting...`);
+            const name = getCompilationName();
+
+            this.logger.log(`Compiler${name ? ` ${name}` : ''} starting...`);
+
+            if (configPath) {
+                this.logger.log(`Compiler${name ? ` ${name}` : ''} is using config: '${configPath}'`);
+            }
         });
 
         compiler.hooks.invalid.tap(pluginName, (filename, changeTime) => {
@@ -63,11 +77,13 @@ class CLIPlugin {
         });
 
         (compiler.webpack ? compiler.hooks.afterDone : compiler.hooks.done).tap(pluginName, () => {
-            this.logger.log(`Compilation${getCompilationName()} finished`);
+            const name = getCompilationName();
+
+            this.logger.log(`Compiler${name ? ` ${name}` : ''} finished`);
 
             process.nextTick(() => {
                 if (compiler.watchMode) {
-                    this.logger.log(`Compiler${getCompilationName()} is watching files for updates...`);
+                    this.logger.log(`Compiler${name ? `${name}` : ''} is watching files for updates...`);
                 }
             });
         });

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

@@ -1631,7 +1631,7 @@ class WebpackCLI {
 
             configOptions.plugins.unshift(
                 new CLIPlugin({
-                    configPath: config.path,
+                    configPath: config.path.get(configOptions),
                     helpfulOutput: !cliOptions.json,
                     hot: cliOptions.hot,
                     progress: cliOptions.progress,

test/build/basic/basic.test.js

@@ -1,5 +1,5 @@
 'use strict';
-
+const { resolve } = require('path');
 const { run } = require('../../utils/test-utils');
 
 describe('bundle command', () => {
@@ -140,4 +140,15 @@ describe('bundle command', () => {
         expect(stderr).toContain("Run 'webpack --help' to see available commands and options");
         expect(stdout).toBeFalsy();
     });
+
+    it('should log supplied config when logging level is log', () => {
+        const { exitCode, stderr, stdout } = run(__dirname, ['--config', './log.config.js']);
+        const configPath = resolve(__dirname, './log.config.js');
+
+        expect(exitCode).toBe(0);
+        expect(stderr).toContain('Compiler starting...');
+        expect(stderr).toContain(`Compiler is using config: '${configPath}'`);
+        expect(stderr).toContain('Compiler finished');
+        expect(stdout).toBeTruthy();
+    });
 });

test/build/basic/log.config.js

@@ -0,0 +1,6 @@
+module.exports = {
+    mode: 'development',
+    infrastructureLogging: {
+        level: 'log',
+    },
+};

test/core-flags/infrastructure-logging.test.js

@@ -23,8 +23,8 @@ describe('infrastructure logging related flag', () => {
         const { exitCode, stderr, stdout } = run(__dirname, ['--infrastructure-logging-level', 'log']);
 
         expect(exitCode).toBe(0);
-        expect(stderr).toContain("Compilation 'compiler' starting...");
-        expect(stderr).toContain("Compilation 'compiler' finished");
+        expect(stderr).toContain("Compiler 'compiler' starting...");
+        expect(stderr).toContain("Compiler 'compiler' finished");
         expect(stdout).toContain(`level: 'log'`);
     });
 });

test/json/json.test.js

@@ -111,8 +111,8 @@ describe('json', () => {
         const { exitCode, stderr, stdout } = run(__dirname, ['--json', '--config', 'logging.config.js']);
 
         expect(exitCode).toBe(0);
-        expect(stderr).toContain('Compilation starting');
-        expect(stderr).toContain('Compilation finished');
+        expect(stderr).toContain('Compiler starting...');
+        expect(stderr).toContain('Compiler finished');
         expect(() => JSON.parse(stdout)).not.toThrow();
         expect(JSON.parse(stdout)['hash']).toBeDefined();
     });
@@ -121,8 +121,8 @@ describe('json', () => {
         const { exitCode, stderr, stdout } = run(__dirname, ['--json', 'stats.json', '--config', 'logging.config.js']);
 
         expect(exitCode).toBe(0);
-        expect(stderr).toContain('Compilation starting');
-        expect(stderr).toContain('Compilation finished');
+        expect(stderr).toContain('Compiler starting...');
+        expect(stderr).toContain('Compiler finished');
         expect(stderr).toContain(successMessage);
         expect(stdout).toBeFalsy();
         expect(existsSync(resolve(__dirname, './stats.json'))).toBeTruthy();

test/serve/basic/log.config.js

@@ -0,0 +1,6 @@
+module.exports = {
+    mode: 'development',
+    infrastructureLogging: {
+        level: 'log',
+    },
+};

test/serve/basic/serve-basic.test.js

@@ -306,6 +306,16 @@ describe('basic serve usage', () => {
         expect(stdout.match(/HotModuleReplacementPlugin/g)).toBeNull();
     });
 
+    it('should log used supplied config with serve', async () => {
+        const { stderr, stdout } = await runServe(__dirname, ['--config', 'log.config.js', '--port', port]);
+        const configPath = path.resolve(__dirname, './log.config.js');
+
+        expect(stderr).toContain('Compiler starting...');
+        expect(stderr).toContain(`Compiler is using config: '${configPath}'`);
+        expect(stderr).toContain('Compiler finished');
+        expect(stdout).toBeTruthy();
+    });
+
     it("should log error on using '--watch' flag with serve", async () => {
         const { stdout, stderr } = await runServe(testPath, ['--watch']);
 

test/watch/basic/basic.test.js

@@ -159,6 +159,28 @@ describe('basic', () => {
         });
     });
 
+    it('should log supplied config with watch', (done) => {
+        const proc = runAndGetWatchProc(__dirname, ['watch', '--config', 'log.config.js']);
+        const configPath = resolve(__dirname, './log.config.js');
+
+        let stderr = '';
+
+        proc.stderr.on('data', (chunk) => {
+            const data = chunk.toString();
+
+            stderr += stripAnsi(data);
+
+            if (/Compiler finished/.test(data)) {
+                expect(stderr).toContain('Compiler starting...');
+                expect(stderr).toContain(`Compiler is using config: '${configPath}'`);
+                expect(stderr).toContain('Compiler finished');
+
+                proc.kill();
+                done();
+            }
+        });
+    });
+
     it('should recompile upon file change using the `command` option and the `--watch` option and log warning', async () => {
         const { exitCode, stderr, stdout } = await run(__dirname, ['watch', '--watch', '--mode', 'development']);
 

test/watch/basic/log.config.js

@@ -0,0 +1,6 @@
+module.exports = {
+    mode: 'development',
+    infrastructureLogging: {
+        level: 'log',
+    },
+};