feat!: remove --define-process-env-node-env in favor of --config-node-env (#2923) (#4318)

Author: snitin315

packages/webpack-cli/src/webpack-cli.ts

@@ -972,20 +972,21 @@ class WebpackCLI implements IWebpackCLI {
           },
         ],
         multiple: false,
-        description: "Sets process.env.NODE_ENV to the specified value.",
+        description:
+          "Sets process.env.NODE_ENV to the specified value for access within the configuration.(Deprecated: Use '--config-node-env' instead)",
         helpLevel: "minimum",
       },
       {
-        name: "define-process-env-node-env",
+        name: "config-node-env",
         configs: [
           {
             type: "string",
           },
         ],
         multiple: false,
         description:
-          "Sets process.env.NODE_ENV to the specified value. (Currently an alias for `--node-env`).",
-        helpLevel: "verbose",
+          "Sets process.env.NODE_ENV to the specified value for access within the configuration.",
+        helpLevel: "minimum",
       },
 
       // Adding more plugins
@@ -2364,9 +2365,8 @@ class WebpackCLI implements IWebpackCLI {
     options: Partial<WebpackDevServerOptions>,
     callback?: Callback<[Error | undefined, WebpackCLIStats | undefined]>,
   ): Promise<WebpackCompiler> {
-    if (typeof options.defineProcessEnvNodeEnv === "string") {
-      // TODO: This should only set NODE_ENV for the runtime not for the config too. Change this during next breaking change.
-      process.env.NODE_ENV = options.defineProcessEnvNodeEnv;
+    if (typeof options.configNodeEnv === "string") {
+      process.env.NODE_ENV = options.configNodeEnv;
     } else if (typeof options.nodeEnv === "string") {
       process.env.NODE_ENV = options.nodeEnv;
     }

…process-env-node-env/auto-mode.config.js …uild/config-node-env/auto-mode.config.jstest/build/define-process-env-node-env/auto-mode.config.js renamed to test/build/config-node-env/auto-mode.config.js test/build/define-process-env-node-env/auto-mode.config.js renamed to test/build/config-node-env/auto-mode.config.js

@@ -2,34 +2,25 @@
 
 const { run } = require("../../utils/test-utils");
 
-describe("--define-process-env-node-env flag", () => {
+describe("--config-node-env flag", () => {
   it('should set "process.env.NODE_ENV" to "development"', async () => {
-    const { exitCode, stderr, stdout } = await run(__dirname, [
-      "--define-process-env-node-env",
-      "development",
-    ]);
+    const { exitCode, stderr, stdout } = await run(__dirname, ["--config-node-env", "development"]);
 
     expect(exitCode).toBe(0);
     expect(stderr).toBeFalsy();
     expect(stdout).toContain("mode: 'development'");
   });
 
   it('should set "process.env.NODE_ENV" to "production"', async () => {
-    const { exitCode, stderr, stdout } = await run(__dirname, [
-      "--define-process-env-node-env",
-      "production",
-    ]);
+    const { exitCode, stderr, stdout } = await run(__dirname, ["--config-node-env", "production"]);
 
     expect(exitCode).toBe(0);
     expect(stderr).toBeFalsy();
     expect(stdout).toContain("mode: 'production'");
   });
 
   it('should set "process.env.NODE_ENV" to "none"', async () => {
-    const { exitCode, stderr, stdout } = await run(__dirname, [
-      "--define-process-env-node-env",
-      "none",
-    ]);
+    const { exitCode, stderr, stdout } = await run(__dirname, ["--config-node-env", "none"]);
 
     expect(exitCode).toBe(0);
     expect(stderr).toBeFalsy();
@@ -38,7 +29,7 @@ describe("--define-process-env-node-env flag", () => {
 
   it('should set "process.env.NODE_ENV" and the "mode" option to "development"', async () => {
     const { exitCode, stderr, stdout } = await run(__dirname, [
-      "--define-process-env-node-env",
+      "--config-node-env",
       "development",
       "--config",
       "./auto-mode.config.js",
@@ -51,7 +42,7 @@ describe("--define-process-env-node-env flag", () => {
 
   it('should set "process.env.NODE_ENV" and the "mode" option to "production"', async () => {
     const { exitCode, stderr, stdout } = await run(__dirname, [
-      "--define-process-env-node-env",
+      "--config-node-env",
       "production",
       "--config",
       "./auto-mode.config.js",
@@ -64,7 +55,7 @@ describe("--define-process-env-node-env flag", () => {
 
   it('should set "process.env.NODE_ENV" and the "mode" option to "none"', async () => {
     const { exitCode, stderr, stdout } = await run(__dirname, [
-      "--define-process-env-node-env",
+      "--config-node-env",
       "none",
       "--config",
       "./auto-mode.config.js",

…-env/define-process-env-node-env.test.js …/config-node-env/config-node-env.test.jstest/build/define-process-env-node-env/define-process-env-node-env.test.js renamed to test/build/config-node-env/config-node-env.test.js test/build/define-process-env-node-env/define-process-env-node-env.test.js renamed to test/build/config-node-env/config-node-env.test.js

@@ -0,0 +1 @@
+console.log('--config-node-env test');