fix(4.0.0-beta.1): fix backwards compatibility with req_id usage for ws subscribe/unsubscribe

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "bybit-api",
-  "version": "4.0.0-beta.0",
+  "version": "4.0.0-beta.1",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "bybit-api",
-      "version": "4.0.0-beta.0",
+      "version": "4.0.0-beta.1",
       "license": "MIT",
       "dependencies": {
         "axios": "^1.7.9",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "bybit-api",
-  "version": "4.0.0-beta.0",
+  "version": "4.0.0-beta.1",
   "description": "Complete & robust Node.js SDK for Bybit's REST APIs and WebSockets, with TypeScript & strong end to end tests.",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",

src/util/BaseRestClient.ts

@@ -4,7 +4,6 @@ import https from 'https';
 
 import {
   APIID,
-  REST_CLIENT_TYPE_ENUM,
   RestClientOptions,
   RestClientType,
   getRestBaseUrl,

src/websocket-client.ts

@@ -550,10 +550,18 @@ export class WebsocketClient extends BaseWebsocketClient<
 
     switch (market) {
       case 'all': {
+        const topics = requests.map((r) => r.topic);
+
+        // Previously used to track topics in a request. Keeping this for subscribe/unsubscribe requests, no need for incremental values
+        const req_id =
+          ['subscribe', 'unsubscribe'].includes(operation) && topics.length
+            ? topics.join('_')
+            : this.getNewRequestId();
+
         const wsEvent: WsRequestOperationBybit<WsTopic> = {
-          req_id: this.getNewRequestId(),
+          req_id: req_id,
           op: operation,
-          args: requests.map((r) => r.topic),
+          args: topics,
         };
 
         const midflightWsEvent: MidflightWsRequestEvent<