@milaboratories/pl-client 2.5.3 → 2.5.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@milaboratories/pl-client",
-  "version": "2.5.3",
+  "version": "2.5.5",
   "description": "New TS/JS client for Platform API",
   "types": "./dist/index.d.ts",
   "main": "./dist/index.js",
@@ -17,7 +17,7 @@
     "./src/**/*"
   ],
   "dependencies": {
-    "@grpc/grpc-js": "^1.12.0",
+    "@grpc/grpc-js": "^1.12.2",
     "@protobuf-ts/grpc-transport": "^2.9.4",
     "@protobuf-ts/runtime": "^2.9.4",
     "@protobuf-ts/runtime-rpc": "^2.9.4",
@@ -27,14 +27,14 @@
     "https-proxy-agent": "^7.0.5",
     "cacheable-lookup": "^6.1.0",
     "long": "^5.2.3",
-    "undici": "^6.19.8",
+    "undici": "^6.20.1",
     "utility-types": "^3.11.0",
-    "yaml": "^2.5.1",
+    "yaml": "^2.6.0",
     "@milaboratories/ts-helpers": "^1.1.0"
   },
   "devDependencies": {
     "typescript": "~5.5.4",
-    "vite": "^5.4.8",
+    "vite": "^5.4.10",
     "@types/node": "~20.16.10",
     "@protobuf-ts/plugin": "^2.9.4",
     "@types/http-proxy": "^1.17.15",

package/src/core/ll_client.ts

@@ -106,7 +106,7 @@ export class LLPlClient {
 
     const httpOptions: Client.Options = {
       // allowH2: true,
-      headersTimeout: 3e3,
+      headersTimeout: 5e3,
       keepAliveTimeout: 3e3,
       keepAliveMaxTimeout: 60e3,
       maxRedirections: 6,

package/src/core/transaction.ts

@@ -152,6 +152,8 @@ export class PlTransaction {
    * pending ops, to throw any pending errors. */
   private pendingVoidOps: Promise<void>[] = [];
 
+  private globalTxIdWasAwaited: boolean = false;
+
   constructor(
     private readonly ll: LLPlTransaction,
     public readonly name: string,
@@ -173,6 +175,13 @@ export class PlTransaction {
       },
       (r) => notEmpty(r.txOpen.tx?.id)
     );
+
+    // To avoid floating promise
+    this.globalTxId.catch((err) => {
+      if (!this.globalTxIdWasAwaited) {
+        console.warn(err);
+      }
+    });
   }
 
   private async drainAndAwaitPendingOps(): Promise<void> {
@@ -192,10 +201,12 @@ export class PlTransaction {
     // pushing operation packet to server
     const rawResponsePromise = this.ll.send(r, false);
 
-    await this.drainAndAwaitPendingOps();
-
-    // awaiting our result, and parsing the response
-    return parser(await rawResponsePromise);
+    try {
+      await this.drainAndAwaitPendingOps();
+    } finally {
+      // awaiting our result, and parsing the response
+      return parser(await rawResponsePromise);
+    }
   }
 
   private async sendMultiAndParse<Kind extends NonUndefined<ClientMessageRequest['oneofKind']>, T>(
@@ -205,10 +216,12 @@ export class PlTransaction {
     // pushing operation packet to server
     const rawResponsePromise = this.ll.send(r, true);
 
-    await this.drainAndAwaitPendingOps();
-
-    // awaiting our result, and parsing the response
-    return parser(await rawResponsePromise);
+    try {
+      await this.drainAndAwaitPendingOps();
+    } finally {
+      // awaiting our result, and parsing the response
+      return parser(await rawResponsePromise);
+    }
   }
 
   private async sendVoidSync<Kind extends NonUndefined<ClientMessageRequest['oneofKind']>, T>(
@@ -247,6 +260,7 @@ export class PlTransaction {
       await completeResult;
       await this.ll.await();
     } else {
+      // @TODO, also floating promises
       const commitResponse = this.sendSingleAndParse(
         { oneofKind: 'txCommit', txCommit: {} },
         (r) => r.txCommit.success
@@ -773,6 +787,7 @@ export class PlTransaction {
   //
 
   public async getGlobalTxId() {
+    this.globalTxIdWasAwaited = true;
     return await this.globalTxId;
   }