@milaboratories/pl-drivers 1.3.11 → 1.3.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@milaboratories/pl-drivers",
-  "version": "1.3.11",
+  "version": "1.3.13",
   "description": "Drivers and a low-level clients for log streaming, downloading and uploading files from and to pl",
   "types": "./dist/index.d.ts",
   "main": "./dist/index.js",
@@ -27,9 +27,9 @@
     "undici": "^6.20.1",
     "zod": "^3.23.8",
     "@milaboratories/ts-helpers": "^1.1.1",
+    "@milaboratories/pl-client": "^2.5.9",
     "@milaboratories/computable": "^2.2.1",
-    "@milaboratories/pl-client": "^2.5.8",
-    "@milaboratories/pl-tree": "^1.4.9",
+    "@milaboratories/pl-tree": "^1.4.10",
     "@milaboratories/pl-model-common": "^1.5.2"
   },
   "devDependencies": {

package/src/drivers/upload.ts

@@ -289,10 +289,28 @@ class ProgressUpdater {
   }
 
   public mustGetProgress(blobExists: boolean) {
+    // We provide a deep copy of progress,
+    // since we do not want to pass a mutable object
+    // to API, it led to bugs before.
+
+    // We do not use '...' cloning syntax
+    // for the compiler to fail here if we change API.
+    const progress: sdk.ImportProgress = {
+      done: this.progress.done,
+      isUpload: this.progress.isUpload,
+      isUploadSignMatch: this.progress.isUploadSignMatch,
+      lastError: this.progress.lastError
+    };
+    if (this.progress.status)
+      progress.status = {
+        progress: this.progress.status.progress,
+        bytesProcessed: this.progress.status.bytesProcessed,
+        bytesTotal: this.progress.status.bytesTotal
+      };
+
     if (blobExists) {
       this.setDone(blobExists);
-
-      return this.progress;
+      return progress;
     }
 
     if (this.uploadingTerminallyFailed) {
@@ -300,7 +318,7 @@ class ProgressUpdater {
       throw new Error(this.progress.lastError);
     }
 
-    return this.progress;
+    return progress;
   }
 
   public attach(w: Watcher, callerId: string) {