fix: only retry the actual download

Also, further improve error messaging

lib/transifex.js

@@ -103,19 +103,34 @@ const getResourceLocation = async function (projectSlug, resourceSlug, localeCod
  * strings, if the local is the source language)
  */
 const txPull = async function (project, resource, locale, mode = 'default') {
-    const url = await getResourceLocation(project, resource, locale, mode);
     let buffer;
-    for (let i = 0; i < 5; i++) {
     try {
-            buffer = await download(url);
+        const url = await getResourceLocation(project, resource, locale, mode);
+        for (let i = 0; i < 5; i++) {
+            if (i > 0) {
+                console.log(`Retrying txPull download after ${i} failed attempt(s)`);
+            }
+            try {
+                buffer = await download(url); // might throw(?)
+                break;
+            } catch (e) {
+                console.error(e, {project, resource, locale, buffer});
+            }
+        }
+        if (!buffer) {
+            throw Error(`txPull download failed after 5 retries: ${url}`);
+        }
         buffer = buffer.toString();
         return JSON.parse(buffer);
     } catch (e) {
-            console.error(e, {project, resource, locale, buffer});
+        e.cause = {
-            console.log(`Retrying after ${i + 1} failed attempt(s)`);
+            project,
+            resource,
+            locale,
+            buffer
+        };
+        throw e;
     }
-    }
-    throw Error('failed to pull after 5 retries');
 };
 
 /**

scripts/tx-pull-www.mjs

@@ -47,30 +47,34 @@ const getLocaleData = async function (item) {
     const locale = item.locale;
     const resource = item.resource;
     let txLocale = localeMap[locale] || locale;
-    for (let i = 0; i < 5; i++) {
+
-        try {
     const translations = await txPull(PROJECT, resource, txLocale);
 
     const txOutdir = `${OUTPUT_DIR}/${PROJECT}.${resource}`;
-            mkdirp.sync(txOutdir);
     const fileName = `${txOutdir}/${locale}.json`;
 
+    try {
+        mkdirp.sync(txOutdir);
         await fs.writeFile(
             fileName,
             JSON.stringify(translations, null, 4)
         );
-            this.progress.increment();
+
         return {
-                resource: resource,
+            resource,
-                locale: locale,
+            locale,
-                file: fileName
+            fileName
         };
     } catch (e) {
-            console.error(e);
+        e.cause = {
-            console.log(`retrying after ${i + 1} attempt(s)`);
+            resource,
+            locale,
+            translations,
+            txOutdir,
+            fileName
+        };
+        throw e;
     }
-    }
-    throw Error('failed to pull translations after 5 retries');
 };
 
 const expandResourceFiles = (resources) => {
@@ -94,9 +98,9 @@ const pullTranslations = async function () {
     const progress = new ProgressLogger(allFiles.length);
 
     try {
-        await batchMap(allFiles, CONCURRENCY_LIMIT, item => {
+        await batchMap(allFiles, CONCURRENCY_LIMIT, async item => {
             try {
-                getLocaleData(item);
+                await getLocaleData(item);
             } finally {
                 progress.increment();
             }