refactor: update imports and add notes for Node.js-only functionality in BaseRestClient and webpack config

Author: JJ-Cro

src/types/websockets/ws-general.ts

@@ -1,5 +1,5 @@
 import { AxiosRequestConfig } from 'axios';
-import { ClientRequestArgs } from 'http';
+import type { ClientRequestArgs } from 'http';
 import WebSocket from 'isomorphic-ws';
 
 import { RestClientOptions } from '../../util/requestUtils';

src/util/BaseRestClient.ts

@@ -1,4 +1,6 @@
 import axios, { AxiosError, AxiosRequestConfig, Method } from 'axios';
+// NOTE: https.Agent is Node.js-only and not available in browser environments
+// Browser builds (via webpack) exclude this module - see webpack.config.js fallback settings
 import https from 'https';
 
 import { BinanceBaseUrlKey } from '../types/shared';
@@ -78,6 +80,9 @@ export default abstract class BaseRestClient {
     };
 
     // If enabled, configure a https agent with keepAlive enabled
+    // NOTE: This is Node.js-only functionality. In browser environments, this code is skipped
+    // as the 'https' module is excluded via webpack fallback configuration.
+    // Browser connection pooling is handled automatically by the browser itself.
     if (this.options.keepAlive) {
       // Extract existing https agent parameters, if provided, to prevent the keepAlive flag from overwriting an existing https agent completely
       const existingHttpsAgent = this.globalRequestOptions.httpsAgent as

webpack/webpack.config.js

@@ -22,7 +22,10 @@ function generateConfig(name) {
           path.resolve(__dirname, "../lib/util/browser-support.js"),
       },
       fallback: {
-        https: false,
+        // Node.js core modules not available in browsers
+        // The REST client's https.Agent (for keepAlive) is Node.js-only and won't work in browsers
+        "http": false,
+        "https": false,
       }
     },
     module: {