@testingbot/cli 1.0.0 → 1.0.1

package/dist/cli.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAwBpC,QAAA,MAAM,OAAO,SAAgB,CAAC;AA6e9B,eAAe,OAAO,CAAC"}
+{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAwBpC,QAAA,MAAM,OAAO,SAAgB,CAAC;AAyf9B,eAAe,OAAO,CAAC"}

package/dist/cli.js

@@ -101,7 +101,11 @@ const espressoCommand = program
             apiSecret: args.apiSecret,
         });
         if (credentials === null) {
-            throw new Error('Please specify credentials via --api-key/--api-secret, TB_KEY/TB_SECRET environment variables, or ~/.testingbot file');
+            throw new Error('No TestingBot credentials found. Please authenticate using one of these methods:\n' +
+                '  1. Run "testingbot login" to authenticate via browser (recommended)\n' +
+                '  2. Use --api-key and --api-secret options\n' +
+                '  3. Set TB_KEY and TB_SECRET environment variables\n' +
+                '  4. Create ~/.testingbot file with content: key:secret');
         }
         const espresso = new espresso_1.default(credentials, options);
         const result = await espresso.run();
@@ -215,7 +219,11 @@ const maestroCommand = program
             apiSecret: args.apiSecret,
         });
         if (credentials === null) {
-            throw new Error('Please specify credentials via --api-key/--api-secret, TB_KEY/TB_SECRET environment variables, or ~/.testingbot file');
+            throw new Error('No TestingBot credentials found. Please authenticate using one of these methods:\n' +
+                '  1. Run "testingbot login" to authenticate via browser (recommended)\n' +
+                '  2. Use --api-key and --api-secret options\n' +
+                '  3. Set TB_KEY and TB_SECRET environment variables\n' +
+                '  4. Create ~/.testingbot file with content: key:secret');
         }
         const maestro = new maestro_1.default(credentials, options);
         const result = await maestro.run();
@@ -296,7 +304,11 @@ const xcuitestCommand = program
             apiSecret: args.apiSecret,
         });
         if (credentials === null) {
-            throw new Error('Please specify credentials via --api-key/--api-secret, TB_KEY/TB_SECRET environment variables, or ~/.testingbot file');
+            throw new Error('No TestingBot credentials found. Please authenticate using one of these methods:\n' +
+                '  1. Run "testingbot login" to authenticate via browser (recommended)\n' +
+                '  2. Use --api-key and --api-secret options\n' +
+                '  3. Set TB_KEY and TB_SECRET environment variables\n' +
+                '  4. Create ~/.testingbot file with content: key:secret');
         }
         const xcuitest = new xcuitest_1.default(credentials, options);
         const result = await xcuitest.run();

package/dist/providers/login.js

@@ -20,7 +20,7 @@ class Login {
             // Open browser to auth URL
             const authUrl = `${AUTH_URL}?port=${this.port}&identifier=testingbotctl`;
             logger_1.default.info('Opening browser for authentication...');
-            logger_1.default.info(`\nIf the browser does not open automatically, visit:\n\n    ${authUrl}\n`);
+            logger_1.default.info(`If the browser does not open automatically, visit:\n\n    ${authUrl}\n`);
             await this.openBrowser(authUrl);
             // Wait for callback (handled by server)
             const credentials = await this.waitForCallback();

package/dist/providers/maestro.d.ts

@@ -1,7 +1,7 @@
 import MaestroOptions from '../models/maestro_options';
 import Credentials from '../models/credentials';
 export interface MaestroRunAssets {
-    logs?: string[];
+    logs?: Record<string, string>;
     video?: string | false;
     screenshots?: string[];
 }

package/dist/providers/maestro.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"maestro.d.ts","sourceRoot":"","sources":["../../src/providers/maestro.ts"],"names":[],"mappings":"AAAA,OAAO,cAAiC,MAAM,2BAA2B,CAAC;AAE1E,OAAO,WAAW,MAAM,uBAAuB,CAAC;AAehD,MAAM,WAAW,gBAAgB;IAC/B,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,GAAG,KAAK,CAAC;IACvB,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;CACxB;AAED,MAAM,WAAW,cAAc;IAC7B,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,SAAS,GAAG,OAAO,GAAG,MAAM,GAAG,QAAQ,CAAC;IAChD,YAAY,EAAE;QACZ,UAAU,EAAE,MAAM,CAAC;QACnB,YAAY,EAAE,MAAM,CAAC;QACrB,OAAO,CAAC,EAAE,MAAM,CAAC;KAClB,CAAC;IACF,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAClC,MAAM,CAAC,EAAE,gBAAgB,CAAC;CAC3B;AAED,MAAM,WAAW,iBAAkB,SAAQ,cAAc;IACvD,SAAS,EAAE,OAAO,CAAC;IACnB,aAAa,EAAE,OAAO,CAAC;CACxB;AAED,MAAM,WAAW,qBAAqB;IACpC,IAAI,EAAE,cAAc,EAAE,CAAC;IACvB,OAAO,EAAE,OAAO,CAAC;IACjB,SAAS,EAAE,OAAO,CAAC;CACpB;AAED,MAAM,WAAW,aAAa;IAC5B,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,EAAE,cAAc,EAAE,CAAC;CACxB;AAED,MAAM,WAAW,oBAAoB;IACnC,EAAE,EAAE,MAAM,CAAC;IACX,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,CAAC,OAAO,OAAO,OAAO;IAC1B,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAwD;IAC5E,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAQ;IACzC,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAAO;IAEzC,OAAO,CAAC,WAAW,CAAc;IACjC,OAAO,CAAC,OAAO,CAAiB;IAChC,OAAO,CAAC,MAAM,CAAS;IAEvB,OAAO,CAAC,KAAK,CAAiC;IAC9C,OAAO,CAAC,gBAAgB,CAA4C;IACpE,OAAO,CAAC,YAAY,CAAgB;IACpC,OAAO,CAAC,cAAc,CAAS;IAC/B,OAAO,CAAC,aAAa,CAA6B;IAClD,OAAO,CAAC,MAAM,CAAuB;IACrC,OAAO,CAAC,YAAY,CAAuB;IAC3C,OAAO,CAAC,SAAS,CAAuB;gBAErB,WAAW,EAAE,WAAW,EAAE,OAAO,EAAE,cAAc;YAMtD,QAAQ;YAsDR,qBAAqB;IA8BnC;;OAEG;YACW,cAAc;IAOf,GAAG,IAAI,OAAO,CAAC,aAAa,CAAC;YAiE5B,SAAS;YA8BT,WAAW;YAmGX,aAAa;YAgDb,oBAAoB;YAiFpB,cAAc;YAiCd,QAAQ;YAsDR,SAAS;YAoBT,iBAAiB;IAiF/B,OAAO,CAAC,gBAAgB;IAsCxB,OAAO,CAAC,SAAS;IAIjB,OAAO,CAAC,iBAAiB;IASzB,OAAO,CAAC,aAAa;YAkBP,YAAY;YAwDZ,aAAa;YAoBb,oBAAoB;YAoBpB,YAAY;IAiB1B,OAAO,CAAC,uBAAuB;YAYjB,iBAAiB;YA8JjB,sBAAsB;IAiBpC,OAAO,CAAC,KAAK;IAIb,OAAO,CAAC,mBAAmB;IAqD3B,OAAO,CAAC,mBAAmB;IAQ3B,OAAO,CAAC,oBAAoB;IAO5B,OAAO,CAAC,cAAc;YAyBR,cAAc;YAgBd,OAAO;IA8BrB,OAAO,CAAC,qBAAqB;IAqC7B,OAAO,CAAC,0BAA0B;IAOlC,OAAO,CAAC,iBAAiB;IAczB,OAAO,CAAC,kBAAkB;CAa3B"}
+{"version":3,"file":"maestro.d.ts","sourceRoot":"","sources":["../../src/providers/maestro.ts"],"names":[],"mappings":"AAAA,OAAO,cAAiC,MAAM,2BAA2B,CAAC;AAE1E,OAAO,WAAW,MAAM,uBAAuB,CAAC;AAehD,MAAM,WAAW,gBAAgB;IAC/B,IAAI,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC9B,KAAK,CAAC,EAAE,MAAM,GAAG,KAAK,CAAC;IACvB,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;CACxB;AAED,MAAM,WAAW,cAAc;IAC7B,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,SAAS,GAAG,OAAO,GAAG,MAAM,GAAG,QAAQ,CAAC;IAChD,YAAY,EAAE;QACZ,UAAU,EAAE,MAAM,CAAC;QACnB,YAAY,EAAE,MAAM,CAAC;QACrB,OAAO,CAAC,EAAE,MAAM,CAAC;KAClB,CAAC;IACF,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAClC,MAAM,CAAC,EAAE,gBAAgB,CAAC;CAC3B;AAED,MAAM,WAAW,iBAAkB,SAAQ,cAAc;IACvD,SAAS,EAAE,OAAO,CAAC;IACnB,aAAa,EAAE,OAAO,CAAC;CACxB;AAED,MAAM,WAAW,qBAAqB;IACpC,IAAI,EAAE,cAAc,EAAE,CAAC;IACvB,OAAO,EAAE,OAAO,CAAC;IACjB,SAAS,EAAE,OAAO,CAAC;CACpB;AAED,MAAM,WAAW,aAAa;IAC5B,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,EAAE,cAAc,EAAE,CAAC;CACxB;AAED,MAAM,WAAW,oBAAoB;IACnC,EAAE,EAAE,MAAM,CAAC;IACX,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,CAAC,OAAO,OAAO,OAAO;IAC1B,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAwD;IAC5E,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAQ;IACzC,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAAO;IAEzC,OAAO,CAAC,WAAW,CAAc;IACjC,OAAO,CAAC,OAAO,CAAiB;IAChC,OAAO,CAAC,MAAM,CAAS;IAEvB,OAAO,CAAC,KAAK,CAAiC;IAC9C,OAAO,CAAC,gBAAgB,CAA4C;IACpE,OAAO,CAAC,YAAY,CAAgB;IACpC,OAAO,CAAC,cAAc,CAAS;IAC/B,OAAO,CAAC,aAAa,CAA6B;IAClD,OAAO,CAAC,MAAM,CAAuB;IACrC,OAAO,CAAC,YAAY,CAAuB;IAC3C,OAAO,CAAC,SAAS,CAAuB;gBAErB,WAAW,EAAE,WAAW,EAAE,OAAO,EAAE,cAAc;YAMtD,QAAQ;YAsDR,qBAAqB;IA8BnC;;OAEG;YACW,cAAc;IAOf,GAAG,IAAI,OAAO,CAAC,aAAa,CAAC;YAiE5B,SAAS;YA8BT,WAAW;YAmGX,aAAa;YAgDb,oBAAoB;YAiFpB,cAAc;YAiCd,QAAQ;YAsDR,SAAS;YAoBT,iBAAiB;IA8E/B,OAAO,CAAC,gBAAgB;IAgDxB,OAAO,CAAC,SAAS;IAIjB,OAAO,CAAC,iBAAiB;IASzB,OAAO,CAAC,aAAa;YAkBP,YAAY;YAwDZ,aAAa;YAoBb,oBAAoB;YAoBpB,YAAY;YA0DZ,uBAAuB;YAqBvB,iBAAiB;YA8JjB,sBAAsB;IAiBpC,OAAO,CAAC,KAAK;IAIb,OAAO,CAAC,mBAAmB;IAqD3B,OAAO,CAAC,mBAAmB;IAQ3B,OAAO,CAAC,oBAAoB;IAO5B,OAAO,CAAC,cAAc;YAyBR,cAAc;YAgBd,OAAO;IA8BrB,OAAO,CAAC,qBAAqB;IAqC7B,OAAO,CAAC,0BAA0B;IAOlC,OAAO,CAAC,iBAAiB;IAczB,OAAO,CAAC,kBAAkB;CAa3B"}

package/dist/providers/maestro.js

@@ -523,7 +523,6 @@ class Maestro {
                         const statusEmoji = run.success === 1 ? '✅' : '❌';
                         const statusText = run.success === 1 ? 'Test completed successfully' : 'Test failed';
                         console.log(`  ${statusEmoji} Run ${run.id} (${run.capabilities.deviceName}): ${statusText}`);
-                        console.log(`     View results: https://testingbot.com/members/maestro/${this.appId}/runs/${run.id}`);
                     }
                 }
                 const allSucceeded = status.runs.every((run) => run.success === 1);
@@ -544,7 +543,7 @@ class Maestro {
                     await this.fetchReports(status.runs);
                 }
                 // Download artifacts if requested
-                if (this.options.downloadArtifacts && this.options.artifactsOutputDir) {
+                if (this.options.downloadArtifacts) {
                     await this.downloadArtifacts(status.runs);
                 }
                 return {
@@ -569,6 +568,11 @@ class Maestro {
                 (prevStatus === 'WAITING' || prevStatus === 'READY')) {
                 this.clearLine();
             }
+            // Show URL when test starts running (transitions from WAITING to READY)
+            if (statusChanged && prevStatus === 'WAITING' && run.status === 'READY') {
+                console.log(`  🚀 Run ${run.id} (${run.capabilities.deviceName}): Test started`);
+                console.log(`     Watch this test in realtime: https://testingbot.com/members/maestro/${this.appId}/runs/${run.id}`);
+            }
             previousStatus.set(run.id, run.status);
             const statusInfo = this.getStatusInfo(run.status);
             if (run.status === 'WAITING' || run.status === 'READY') {
@@ -680,31 +684,73 @@ class Maestro {
         }
         throw new testingbot_error_1.default(`Timed out waiting for artifacts to sync for run ${runId}`);
     }
-    async downloadFile(url, filePath) {
-        try {
-            const response = await axios_1.default.get(url, {
-                responseType: 'arraybuffer',
-                headers: {
-                    'User-Agent': utils_1.default.getUserAgent(),
-                },
-            });
-            await node_fs_1.default.promises.writeFile(filePath, response.data);
+    async downloadFile(url, filePath, retries = 3) {
+        let lastError;
+        for (let attempt = 1; attempt <= retries; attempt++) {
+            try {
+                const response = await axios_1.default.get(url, {
+                    responseType: 'arraybuffer',
+                    timeout: 60000, // 60 second timeout for large files
+                });
+                await node_fs_1.default.promises.writeFile(filePath, response.data);
+                return;
+            }
+            catch (error) {
+                lastError = error;
+                // Don't retry on 4xx errors (client errors like 403, 404)
+                if (axios_1.default.isAxiosError(error) && error.response?.status) {
+                    const status = error.response.status;
+                    if (status >= 400 && status < 500) {
+                        break;
+                    }
+                }
+                // Wait before retrying (exponential backoff)
+                if (attempt < retries) {
+                    await this.sleep(1000 * attempt);
+                }
+            }
         }
-        catch (error) {
-            throw new testingbot_error_1.default(`Failed to download file from ${url}`, {
-                cause: error,
-            });
+        // Extract detailed error message
+        let errorDetail = '';
+        if (axios_1.default.isAxiosError(lastError)) {
+            if (lastError.response) {
+                errorDetail = `HTTP ${lastError.response.status}: ${lastError.response.statusText}`;
+            }
+            else if (lastError.code) {
+                errorDetail = lastError.code;
+            }
+            else if (lastError.message) {
+                errorDetail = lastError.message;
+            }
         }
+        else if (lastError instanceof Error) {
+            errorDetail = lastError.message;
+        }
+        else if (lastError) {
+            errorDetail = String(lastError);
+        }
+        throw new testingbot_error_1.default(`Failed to download file${errorDetail ? `: ${errorDetail}` : ''}`, {
+            cause: lastError,
+        });
     }
-    generateArtifactZipName() {
-        // Use --build option if provided, otherwise generate timestamp-based name
-        if (this.options.build) {
-            const sanitizedBuild = this.options.build.replace(/[^a-zA-Z0-9_-]/g, '_');
-            return `${sanitizedBuild}.zip`;
+    async generateArtifactZipName(outputDir) {
+        if (!this.options.build) {
+            // Generate unique name with timestamp
+            const timestamp = new Date().toISOString().replace(/[:.]/g, '-');
+            return `maestro_artifacts_${timestamp}.zip`;
+        }
+        const baseName = this.options.build.replace(/[^a-zA-Z0-9_-]/g, '_');
+        const fileName = `${baseName}.zip`;
+        const filePath = node_path_1.default.join(outputDir, fileName);
+        try {
+            await node_fs_1.default.promises.access(filePath);
+            // File exists, append timestamp
+            return `${baseName}_${Date.now()}.zip`;
+        }
+        catch {
+            // File doesn't exist, use base name
+            return fileName;
         }
-        // Generate unique name with timestamp
-        const timestamp = new Date().toISOString().replace(/[:.]/g, '-');
-        return `maestro_artifacts_${timestamp}.zip`;
     }
     async downloadArtifacts(runs) {
         if (!this.options.downloadArtifacts)
@@ -730,12 +776,12 @@ class Maestro {
                     const runDir = node_path_1.default.join(tempDir, `run_${run.id}`);
                     await node_fs_1.default.promises.mkdir(runDir, { recursive: true });
                     // Download logs
-                    if (runDetails.assets.logs && runDetails.assets.logs.length > 0) {
+                    if (runDetails.assets.logs &&
+                        Object.keys(runDetails.assets.logs).length > 0) {
                         const logsDir = node_path_1.default.join(runDir, 'logs');
                         await node_fs_1.default.promises.mkdir(logsDir, { recursive: true });
-                        for (let i = 0; i < runDetails.assets.logs.length; i++) {
-                            const logUrl = runDetails.assets.logs[i];
-                            const logFileName = node_path_1.default.basename(logUrl) || `log_${i}.txt`;
+                        for (const [logName, logUrl] of Object.entries(runDetails.assets.logs)) {
+                            const logFileName = `${logName}.txt`;
                             const logPath = node_path_1.default.join(logsDir, logFileName);
                             try {
                                 await this.downloadFile(logUrl, logPath);
@@ -753,7 +799,7 @@ class Maestro {
                         const videoDir = node_path_1.default.join(runDir, 'video');
                         await node_fs_1.default.promises.mkdir(videoDir, { recursive: true });
                         const videoUrl = runDetails.assets.video;
-                        const videoFileName = node_path_1.default.basename(videoUrl) || 'video.mp4';
+                        const videoFileName = 'video.mp4';
                         const videoPath = node_path_1.default.join(videoDir, videoFileName);
                         try {
                             await this.downloadFile(videoUrl, videoPath);
@@ -771,7 +817,7 @@ class Maestro {
                         await node_fs_1.default.promises.mkdir(screenshotsDir, { recursive: true });
                         for (let i = 0; i < runDetails.assets.screenshots.length; i++) {
                             const screenshotUrl = runDetails.assets.screenshots[i];
-                            const screenshotFileName = node_path_1.default.basename(screenshotUrl) || `screenshot_${i}.png`;
+                            const screenshotFileName = `screenshot_${i}.png`;
                             const screenshotPath = node_path_1.default.join(screenshotsDir, screenshotFileName);
                             try {
                                 await this.downloadFile(screenshotUrl, screenshotPath);
@@ -804,7 +850,7 @@ class Maestro {
                     logger_1.default.error(`Failed to download artifacts for run ${run.id}: ${error instanceof Error ? error.message : error}`);
                 }
             }
-            const zipFileName = this.generateArtifactZipName();
+            const zipFileName = await this.generateArtifactZipName(outputDir);
             const zipFilePath = node_path_1.default.join(outputDir, zipFileName);
             if (!this.options.quiet) {
                 logger_1.default.info(`Creating artifacts zip: ${zipFileName}`);

package/dist/upload.d.ts

@@ -11,11 +11,8 @@ export interface UploadResult {
     id: number;
 }
 export default class Upload {
-    private lastProgressPercent;
     upload(options: UploadOptions): Promise<UploadResult>;
+    private drawProgressBar;
     private validateFile;
-    private handleProgress;
-    private displayProgress;
-    private clearProgressLine;
 }
 //# sourceMappingURL=upload.d.ts.map

package/dist/upload.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"upload.d.ts","sourceRoot":"","sources":["../src/upload.ts"],"names":[],"mappings":"AAIA,OAAO,WAAW,MAAM,sBAAsB,CAAC;AAI/C,MAAM,MAAM,WAAW,GACnB,yCAAyC,GACzC,0BAA0B,GAC1B,iBAAiB,CAAC;AAEtB,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,MAAM,CAAC;IACjB,GAAG,EAAE,MAAM,CAAC;IACZ,WAAW,EAAE,WAAW,CAAC;IACzB,WAAW,EAAE,WAAW,CAAC;IACzB,YAAY,CAAC,EAAE,OAAO,CAAC;CACxB;AAED,MAAM,WAAW,YAAY;IAC3B,EAAE,EAAE,MAAM,CAAC;CACZ;AAED,MAAM,CAAC,OAAO,OAAO,MAAM;IACzB,OAAO,CAAC,mBAAmB,CAAa;IAE3B,MAAM,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,YAAY,CAAC;YA+DpD,YAAY;IAQ1B,OAAO,CAAC,cAAc;IAetB,OAAO,CAAC,eAAe;IAmBvB,OAAO,CAAC,iBAAiB;CAI1B"}
+{"version":3,"file":"upload.d.ts","sourceRoot":"","sources":["../src/upload.ts"],"names":[],"mappings":"AAKA,OAAO,WAAW,MAAM,sBAAsB,CAAC;AAI/C,MAAM,MAAM,WAAW,GACnB,yCAAyC,GACzC,0BAA0B,GAC1B,iBAAiB,CAAC;AAEtB,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,MAAM,CAAC;IACjB,GAAG,EAAE,MAAM,CAAC;IACZ,WAAW,EAAE,WAAW,CAAC;IACzB,WAAW,EAAE,WAAW,CAAC;IACzB,YAAY,CAAC,EAAE,OAAO,CAAC;CACxB;AAED,MAAM,WAAW,YAAY;IAC3B,EAAE,EAAE,MAAM,CAAC;CACZ;AAED,MAAM,CAAC,OAAO,OAAO,MAAM;IACZ,MAAM,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,YAAY,CAAC;IAuGlE,OAAO,CAAC,eAAe;YAgBT,YAAY;CAO3B"}

package/dist/upload.js

@@ -7,23 +7,47 @@ const axios_1 = __importDefault(require("axios"));
 const node_fs_1 = __importDefault(require("node:fs"));
 const node_path_1 = __importDefault(require("node:path"));
 const form_data_1 = __importDefault(require("form-data"));
+const progress_stream_1 = __importDefault(require("progress-stream"));
 const testingbot_error_1 = __importDefault(require("./models/testingbot_error"));
 const utils_1 = __importDefault(require("./utils"));
 class Upload {
-    lastProgressPercent = 0;
     async upload(options) {
-        const { filePath, url, credentials, contentType, showProgress = false, } = options;
+        const { filePath, url, credentials, showProgress = false, } = options;
         await this.validateFile(filePath);
         const fileName = node_path_1.default.basename(filePath);
         const fileStats = await node_fs_1.default.promises.stat(filePath);
+        const totalSize = fileStats.size;
+        const sizeMB = (totalSize / (1024 * 1024)).toFixed(2);
+        // Create progress tracker
+        const progressTracker = (0, progress_stream_1.default)({
+            length: totalSize,
+            time: 100, // Emit progress every 100ms
+        });
+        let lastPercent = 0;
+        if (showProgress) {
+            // Draw initial progress bar
+            this.drawProgressBar(fileName, sizeMB, 0);
+            progressTracker.on('progress', (prog) => {
+                const percent = Math.round(prog.percentage);
+                if (percent !== lastPercent) {
+                    lastPercent = percent;
+                    this.drawProgressBar(fileName, sizeMB, percent);
+                }
+            });
+        }
+        // Create file stream and pipe through progress tracker
         const fileStream = node_fs_1.default.createReadStream(filePath);
+        const trackedStream = fileStream.pipe(progressTracker);
         const formData = new form_data_1.default();
-        formData.append('file', fileStream);
+        formData.append('file', trackedStream, {
+            filename: fileName,
+            contentType: options.contentType,
+            knownLength: totalSize,
+        });
         try {
             const response = await axios_1.default.post(url, formData, {
                 headers: {
-                    'Content-Type': contentType,
-                    'Content-Disposition': `attachment; filename=${fileName}`,
+                    ...formData.getHeaders(),
                     'User-Agent': utils_1.default.getUserAgent(),
                 },
                 auth: {
@@ -32,34 +56,52 @@ class Upload {
                 },
                 maxContentLength: Infinity,
                 maxBodyLength: Infinity,
-                onUploadProgress: showProgress
-                    ? (progressEvent) => {
-                        this.handleProgress(progressEvent, fileStats.size, fileName);
-                    }
-                    : undefined,
+                maxRedirects: 0, // Recommended for stream uploads to avoid buffering
             });
             const result = response.data;
             if (result.id) {
                 if (showProgress) {
-                    this.clearProgressLine();
+                    this.drawProgressBar(fileName, sizeMB, 100);
+                    console.log('');
                 }
                 return { id: result.id };
             }
             else {
+                if (showProgress) {
+                    console.log(' Failed');
+                }
                 throw new testingbot_error_1.default(`Upload failed: ${result.error || 'Unknown error'}`);
             }
         }
         catch (error) {
+            if (showProgress) {
+                console.log(' Failed');
+            }
             if (error instanceof testingbot_error_1.default) {
                 throw error;
             }
             if (axios_1.default.isAxiosError(error)) {
+                if (error.response?.status === 401) {
+                    throw new testingbot_error_1.default('Invalid TestingBot credentials. Please check your API key and secret.\n' +
+                        'You can update your credentials by running "testingbot login" or by using:\n' +
+                        '  --api-key and --api-secret options\n' +
+                        '  TB_KEY and TB_SECRET environment variables\n' +
+                        '  ~/.testingbot file with content: key:secret');
+                }
                 const message = error.response?.data?.error || error.message;
                 throw new testingbot_error_1.default(`Upload failed: ${message}`);
             }
             throw new testingbot_error_1.default(`Upload failed: ${error instanceof Error ? error.message : 'Unknown error'}`);
         }
     }
+    drawProgressBar(fileName, sizeMB, percent) {
+        const barWidth = 30;
+        const filled = Math.round((barWidth * percent) / 100);
+        const empty = barWidth - filled;
+        const bar = '█'.repeat(filled) + '░'.repeat(empty);
+        const transferred = ((percent / 100) * parseFloat(sizeMB)).toFixed(2);
+        process.stdout.write(`\r  ${fileName}: [${bar}] ${percent}% (${transferred}/${sizeMB} MB)`);
+    }
     async validateFile(filePath) {
         try {
             await node_fs_1.default.promises.access(filePath, node_fs_1.default.constants.R_OK);
@@ -68,27 +110,5 @@ class Upload {
             throw new testingbot_error_1.default(`File not found or not readable: ${filePath}`);
         }
     }
-    handleProgress(progressEvent, totalSize, fileName) {
-        const loaded = progressEvent.loaded;
-        const total = progressEvent.total || totalSize;
-        const percent = Math.round((loaded / total) * 100);
-        if (percent !== this.lastProgressPercent) {
-            this.lastProgressPercent = percent;
-            this.displayProgress(fileName, percent, loaded, total);
-        }
-    }
-    displayProgress(fileName, percent, loaded, total) {
-        const barWidth = 30;
-        const filledWidth = Math.round((percent / 100) * barWidth);
-        const emptyWidth = barWidth - filledWidth;
-        const bar = '█'.repeat(filledWidth) + '░'.repeat(emptyWidth);
-        const loadedMB = (loaded / (1024 * 1024)).toFixed(2);
-        const totalMB = (total / (1024 * 1024)).toFixed(2);
-        process.stdout.write(`\r  ${fileName}: [${bar}] ${percent}% (${loadedMB}/${totalMB} MB)`);
-    }
-    clearProgressLine() {
-        process.stdout.write('\r' + ' '.repeat(80) + '\r');
-        this.lastProgressPercent = 0;
-    }
 }
 exports.default = Upload;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testingbot/cli",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "CLI tool to run Espresso, XCUITest, and Maestro tests on TestingBot's cloud infrastructure",
   "main": "dist/index.js",
   "bin": {
@@ -55,6 +55,7 @@
     "form-data": "^4.0.5",
     "glob": "^13.0.0",
     "js-yaml": "^4.1.1",
+    "progress-stream": "^2.0.0",
     "socket.io-client": "^4.8.1",
     "tracer": "^1.3.0"
   },
@@ -65,6 +66,7 @@
     "@types/jest": "^29.5.14",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^20.19.0",
+    "@types/progress-stream": "^2.0.5",
     "babel-jest": "^29.7.0",
     "eslint": "^9.39.1",
     "eslint-config-prettier": "^10.1.8",