@samsara-dev/appwright 0.9.7 → 0.9.8

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # appwright
 
+## 0.9.8
+
+### Patch Changes
+
+- eb8aea2: Cap video download retry backoff at 60s and add timestamps to all download logs
+
 ## 0.9.7
 
 ### Patch Changes

package/dist/providers/browserstack/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/providers/browserstack/index.ts"],"names":[],"mappings":"AAIA,OAAO,EACL,eAAe,EACf,cAAc,EAGf,MAAM,aAAa,CAAC;AACrB,OAAO,EAAE,WAAW,EAAE,MAAM,kBAAkB,CAAC;AAC/C,OAAO,EAAE,MAAM,EAAE,MAAM,cAAc,CAAC;AA0DtC,qBAAa,0BAA2B,YAAW,cAAc;IAC/D,OAAO,CAAC,cAAc,CAAC,CAA6B;IACpD,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,OAAO,CAAC,OAAO,CAA+B;gBAG5C,OAAO,EAAE,WAAW,CAAC,eAAe,CAAC,EACrC,WAAW,EAAE,MAAM,GAAG,SAAS;IAU3B,WAAW;IA0EX,SAAS,IAAI,OAAO,CAAC,MAAM,CAAC;IAMlC,OAAO,CAAC,cAAc;YASR,YAAY;YAkCZ,iBAAiB;YAKjB,yBAAyB;WAK1B,aAAa,CACxB,SAAS,EAAE,MAAM,EACjB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,GACf,OAAO,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,WAAW,EAAE,MAAM,CAAA;KAAE,GAAG,IAAI,CAAC;IA+FlD,eAAe,CAAC,OAAO,EAAE;QAC7B,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,IAAI,CAAC,EAAE,MAAM,CAAC;KACf;IA2BD,OAAO,CAAC,YAAY;CA4KrB"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/providers/browserstack/index.ts"],"names":[],"mappings":"AAIA,OAAO,EACL,eAAe,EACf,cAAc,EAGf,MAAM,aAAa,CAAC;AACrB,OAAO,EAAE,WAAW,EAAE,MAAM,kBAAkB,CAAC;AAC/C,OAAO,EAAE,MAAM,EAAE,MAAM,cAAc,CAAC;AA0DtC,qBAAa,0BAA2B,YAAW,cAAc;IAC/D,OAAO,CAAC,cAAc,CAAC,CAA6B;IACpD,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,OAAO,CAAC,OAAO,CAA+B;gBAG5C,OAAO,EAAE,WAAW,CAAC,eAAe,CAAC,EACrC,WAAW,EAAE,MAAM,GAAG,SAAS;IAU3B,WAAW;IA0EX,SAAS,IAAI,OAAO,CAAC,MAAM,CAAC;IAMlC,OAAO,CAAC,cAAc;YASR,YAAY;YAkCZ,iBAAiB;YAKjB,yBAAyB;WAK1B,aAAa,CACxB,SAAS,EAAE,MAAM,EACjB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,GACf,OAAO,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,WAAW,EAAE,MAAM,CAAA;KAAE,GAAG,IAAI,CAAC;IAsGlD,eAAe,CAAC,OAAO,EAAE;QAC7B,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,IAAI,CAAC,EAAE,MAAM,CAAC;KACf;IA2BD,OAAO,CAAC,YAAY;CA4KrB"}

package/dist/providers/browserstack/index.js

@@ -175,6 +175,7 @@ class BrowserStackDeviceProvider {
         //To catch the browserstack error in case all retries fails
         try {
             if (videoURL) {
+                logger_1.logger.log(`[${new Date().toISOString()}] Video download starting: ${videoURL}`);
                 await (0, async_retry_1.default)(async () => {
                     const response = await fetch(videoURL, {
                         method: "GET",
@@ -200,11 +201,10 @@ class BrowserStackDeviceProvider {
                 }, {
                     retries: 10,
                     minTimeout: 3_000,
+                    maxTimeout: 60_000,
                     onRetry: (err, i) => {
                         const message = err instanceof Error ? err.message : String(err);
-                        if (i > 5) {
-                            logger_1.logger.warn(`Retry attempt ${i} failed: ${message}`);
-                        }
+                        logger_1.logger.warn(`[${new Date().toISOString()}] Video download retry ${i}/10 failed: ${message}`);
                     },
                 });
                 return new Promise((resolve, reject) => {
@@ -212,7 +212,7 @@ class BrowserStackDeviceProvider {
                     fileStream.on("finish", () => {
                         try {
                             fs_1.default.renameSync(tempPathForWriting, pathToTestVideo);
-                            logger_1.logger.log(`Download finished and file closed: ${pathToTestVideo}`);
+                            logger_1.logger.log(`[${new Date().toISOString()}] Video download completed: ${pathToTestVideo}`);
                             resolve({ path: pathToTestVideo, contentType: "video/mp4" });
                         }
                         catch (err) {
@@ -233,7 +233,8 @@ class BrowserStackDeviceProvider {
             }
         }
         catch (e) {
-            logger_1.logger.log(`Error Downloading video: `, e);
+            const message = e instanceof Error ? e.message : String(e);
+            logger_1.logger.warn(`[${new Date().toISOString()}] Video download failed after all retries: ${message}. Test will complete without video attachment.`);
             return null;
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@samsara-dev/appwright",
-  "version": "0.9.7",
+  "version": "0.9.8",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"