[DownloadBuildArtifactsV0] Replace decompress-zip with extract-zip package (#19569)

* [DownloadBuildArtifactsV0] Replace decompress-zip with extract-zip package

* bump task version

* wrap in promise

Author: kirill-ivlev

Tasks/DownloadBuildArtifactsV0/DownloadHandlers/DownloadHandlerContainerZip.ts

@@ -4,6 +4,9 @@ import { FilesystemProvider, ZipProvider } from 'artifact-engine/Providers';
 import * as tl from 'azure-pipelines-task-lib/task';
 import * as path from 'path';
 import * as DecompressZip from 'decompress-zip';
+#if NODE20
+import * as extract from 'extract-zip'
+#endif
 
 /**
  * Handler for download artifact via build API
@@ -39,9 +42,15 @@ export class DownloadHandlerContainerZip extends DownloadHandler {
             }
 
             tl.debug(`Extracting ${this.zipLocation} to ${unzipLocation}`);
-
+#if NODE20
+            tl.debug(`Using extract-zip package for extracting archive`);
+            extract(this.zipLocation, { dir: unzipLocation }).then(() => {
+                resolve();
+            }).catch((error) => {
+                reject(error);
+            });
+#else
             const unzipper = new DecompressZip(this.zipLocation);
-
             unzipper.on('error', err => {
                 return reject(tl.loc('ExtractionFailed', err));
             });
@@ -54,7 +63,7 @@ export class DownloadHandlerContainerZip extends DownloadHandler {
             unzipper.extract({
                 path: unzipLocation
             });
-
+#endif
         });
 
         return unZipPromise;

Tasks/DownloadBuildArtifactsV0/_buildConfigs/Node20/package-lock.json

@@ -22,7 +22,8 @@
     "artifact-engine": "^1.4.0",
     "azure-devops-node-api": "^10.2.2",
     "azure-pipelines-task-lib": "^4.4.0",
-    "decompress-zip": "0.3.3"
+    "decompress-zip": "0.3.3",
+    "extract-zip": "2.0.1"
   },
   "devDependencies": {
     "typescript": "5.1.6"

Tasks/DownloadBuildArtifactsV0/_buildConfigs/Node20/package.json

@@ -9,7 +9,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 235,
+    "Minor": 236,
     "Patch": 0
   },
   "groups": [

Tasks/DownloadBuildArtifactsV0/task.json

@@ -9,7 +9,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 235,
+    "Minor": 236,
     "Patch": 0
   },
   "groups": [

Tasks/DownloadBuildArtifactsV0/task.loc.json

@@ -1,2 +1,2 @@
-Default|0.235.0
-Node20_229_3|0.235.1
+Default|0.236.0
+Node20_229_3|0.236.1

_generated/DownloadBuildArtifactsV0.versionmap.txt

@@ -39,9 +39,7 @@ export class DownloadHandlerContainerZip extends DownloadHandler {
             }
 
             tl.debug(`Extracting ${this.zipLocation} to ${unzipLocation}`);
-
             const unzipper = new DecompressZip(this.zipLocation);
-
             unzipper.on('error', err => {
                 return reject(tl.loc('ExtractionFailed', err));
             });
@@ -54,7 +52,6 @@ export class DownloadHandlerContainerZip extends DownloadHandler {
             unzipper.extract({
                 path: unzipLocation
             });
-
         });
 
         return unZipPromise;

_generated/DownloadBuildArtifactsV0/DownloadHandlers/DownloadHandlerContainerZip.ts

@@ -9,7 +9,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 235,
+    "Minor": 236,
     "Patch": 0
   },
   "groups": [
@@ -321,7 +321,7 @@
     }
   ],
   "_buildConfigMapping": {
-    "Default": "0.235.0",
-    "Node20_229_3": "0.235.1"
+    "Default": "0.236.0",
+    "Node20_229_3": "0.236.1"
   }
 }

_generated/DownloadBuildArtifactsV0/task.json

@@ -9,7 +9,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 0,
-    "Minor": 235,
+    "Minor": 236,
     "Patch": 0
   },
   "groups": [
@@ -321,7 +321,7 @@
     }
   ],
   "_buildConfigMapping": {
-    "Default": "0.235.0",
-    "Node20_229_3": "0.235.1"
+    "Default": "0.236.0",
+    "Node20_229_3": "0.236.1"
   }
 }

_generated/DownloadBuildArtifactsV0/task.loc.json

@@ -4,6 +4,7 @@ import { FilesystemProvider, ZipProvider } from 'artifact-engine/Providers';
 import * as tl from 'azure-pipelines-task-lib/task';
 import * as path from 'path';
 import * as DecompressZip from 'decompress-zip';
+import * as extract from 'extract-zip'
 
 /**
  * Handler for download artifact via build API
@@ -39,22 +40,12 @@ export class DownloadHandlerContainerZip extends DownloadHandler {
             }
 
             tl.debug(`Extracting ${this.zipLocation} to ${unzipLocation}`);
-
-            const unzipper = new DecompressZip(this.zipLocation);
-
-            unzipper.on('error', err => {
-                return reject(tl.loc('ExtractionFailed', err));
-            });
-
-            unzipper.on('extract', log => {
-                tl.debug(`Extracted ${this.zipLocation} to ${unzipLocation} successfully`);
-                return resolve();
-            });
-
-            unzipper.extract({
-                path: unzipLocation
+            tl.debug(`Using extract-zip package for extracting archive`);
+            extract(this.zipLocation, { dir: unzipLocation }).then(() => {
+                resolve();
+            }).catch((error) => {
+                reject(error);
             });
-
         });
 
         return unZipPromise;