refactor: code

Author: alexander-akait

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

@@ -6,7 +6,6 @@ import {
   type MultiStatsOptions,
   type StatsOptions,
   type WebpackError,
-  cli,
   default as webpack,
 } from "webpack";
 import type webpackMerge from "webpack-merge";
@@ -104,9 +103,12 @@ class WebpackCLI implements IWebpackCLI {
     this.program = program;
     this.program.name("webpack");
     this.program.configureOutput({
-      writeErr: this.logger.error,
-      outputError: (str, write) =>
-        write(`Error: ${this.capitalizeFirstLetter(str.replace(/^error:/, "").trim())}`),
+      writeErr: (str) => {
+        this.logger.error(str);
+      },
+      outputError: (str, write) => {
+        write(`Error: ${this.capitalizeFirstLetter(str.replace(/^error:/, "").trim())}`);
+      },
     });
   }
 
@@ -135,11 +137,17 @@ class WebpackCLI implements IWebpackCLI {
   }
 
   createColors(useColor?: boolean): WebpackCLIColors {
-    if (typeof cli.createColors === "function") {
-      const { createColors, isColorSupported } = cli;
-      const shouldUseColor = useColor || isColorSupported();
+    try {
+      const { cli } = require("webpack");
 
-      return { ...createColors({ useColor: shouldUseColor }), isColorSupported: shouldUseColor };
+      if (typeof cli.createColors === "function") {
+        const { createColors, isColorSupported } = cli;
+        const shouldUseColor = useColor || isColorSupported();
+
+        return { ...createColors({ useColor: shouldUseColor }), isColorSupported: shouldUseColor };
+      }
+    } catch {
+      // Nothing
     }
 
     // TODO remove `colorette` and set [email protected] as the minimum supported version in the next major release
@@ -1332,16 +1340,14 @@ class WebpackCLI implements IWebpackCLI {
     });
 
     this.program.option("--color", "Enable colors on console.");
-    this.program.on("option:color", function color() {
-      // @ts-expect-error shadowing 'this' is intended
+    this.program.on("option:color", function color(this: WebpackCLICommand) {
       const { color } = this.opts();
 
       cli.isColorSupportChanged = color;
       cli.colors = cli.createColors(color);
     });
     this.program.option("--no-color", "Disable colors on console.");
-    this.program.on("option:no-color", function noColor() {
-      // @ts-expect-error shadowing 'this' is intended
+    this.program.on("option:no-color", function noColor(this: WebpackCLICommand) {
       const { color } = this.opts();
 
       cli.isColorSupportChanged = color;
@@ -1839,8 +1845,7 @@ class WebpackCLI implements IWebpackCLI {
           false,
           moduleType,
         );
-        // @ts-expect-error error type assertion
-      } catch (error: Error) {
+      } catch (error) {
         this.logger.error(`Failed to load '${configPath}' config`);
 
         if (this.isValidationError(error)) {
@@ -2337,8 +2342,10 @@ class WebpackCLI implements IWebpackCLI {
     return config;
   }
 
-  isValidationError(error: Error): error is WebpackError {
-    return error instanceof this.webpack.ValidationError || error.name === "ValidationError";
+  isValidationError(error: unknown): error is WebpackError {
+    return (
+      error instanceof this.webpack.ValidationError || (error as Error).name === "ValidationError"
+    );
   }
 
   async createCompiler(
@@ -2370,8 +2377,7 @@ class WebpackCLI implements IWebpackCLI {
             }
           : callback,
       )!;
-      // @ts-expect-error error type assertion
-    } catch (error: Error) {
+    } catch (error) {
       if (this.isValidationError(error)) {
         this.logger.error(error.message);
       } else {