refactor(Server): remove redundant anonymous functions (#3750)

Author: ylemkimon

lib/Server.js

@@ -129,23 +129,23 @@ class Server {
   addAdditionalEntries(compiler) {
     const additionalEntries = [];
 
-    const isWebTarget = (compilerOptions) =>
-      compilerOptions.externalsPresets
-        ? compilerOptions.externalsPresets.web
-        : [
-            "web",
-            "webworker",
-            "electron-preload",
-            "electron-renderer",
-            "node-webkit",
-            // eslint-disable-next-line no-undefined
-            undefined,
-            null,
-          ].includes(compilerOptions.target);
+    const isWebTarget = compiler.options.externalsPresets
+      ? compiler.options.externalsPresets.web
+      : [
+          "web",
+          "webworker",
+          "electron-preload",
+          "electron-renderer",
+          "node-webkit",
+          // eslint-disable-next-line no-undefined
+          undefined,
+          null,
+        ].includes(compiler.options.target);
 
     // TODO maybe empty empty client
-    if (this.options.client && isWebTarget(compiler.options)) {
-      const getWebSocketURL = () => {
+    if (this.options.client && isWebTarget) {
+      let webSocketURL = "";
+      if (this.options.webSocketServer) {
         const searchParams = new URLSearchParams();
 
         /** @type {"ws:" | "wss:" | "http:" | "https:" | "auto:"} */
@@ -258,11 +258,8 @@ class Server {
           searchParams.set("logging", this.options.client.logging);
         }
 
-        return searchParams.toString();
-      };
-      const webSocketURL = this.options.webSocketServer
-        ? getWebSocketURL()
-        : "";
+        webSocketURL = searchParams.toString();
+      }
 
       additionalEntries.push(
         `${require.resolve("../client/index.js")}?${webSocketURL}`