refactor: code

Author: alexander-akait

lib/Server.js

@@ -74,24 +74,6 @@ class Server {
     }, this);
   }
 
-  static getArguments(webpack, ...args) {
-    // TODO remove after drop webpack v4
-    if (!webpack.cli) {
-      return require('../bin/cli-flags');
-    }
-
-    return webpack.cli.getArguments(schema, ...args);
-  }
-
-  static processArguments(webpack, ...args) {
-    // TODO remove after drop webpack v4
-    if (!webpack.cli) {
-      return require('../bin/process-arguments')(...args);
-    }
-
-    return webpack.cli.processArguments(...args);
-  }
-
   applyDevServerPlugin() {
     const DevServerPlugin = require('./utils/DevServerPlugin');
 
@@ -1054,4 +1036,48 @@ class Server {
   }
 }
 
-module.exports = Server;
+const mergeExports = (obj, exports) => {
+  const descriptors = Object.getOwnPropertyDescriptors(exports);
+
+  for (const name of Object.keys(descriptors)) {
+    const descriptor = descriptors[name];
+
+    if (descriptor.get) {
+      const fn = descriptor.get;
+
+      Object.defineProperty(obj, name, {
+        configurable: false,
+        enumerable: true,
+        get: fn,
+      });
+    } else if (typeof descriptor.value === 'object') {
+      Object.defineProperty(obj, name, {
+        configurable: false,
+        enumerable: true,
+        writable: false,
+        value: mergeExports({}, descriptor.value),
+      });
+    } else {
+      throw new Error(
+        'Exposed values must be either a getter or an nested object'
+      );
+    }
+  }
+
+  return Object.freeze(obj);
+};
+
+module.exports = mergeExports(Server, {
+  get schema() {
+    return schema;
+  },
+  // TODO compatibility with webpack v4, remove it after drop
+  cli: {
+    get getArguments() {
+      return () => require('../bin/cli-flags');
+    },
+    get processArguments() {
+      return require('../bin/process-arguments');
+    },
+  },
+});