@testingbot/cli 1.0.7 → 1.0.8

package/dist/upload.js

@@ -26,14 +26,34 @@ class Upload {
             length: totalSize,
             time: 100, // Emit progress every 100ms
         });
-        let lastPercent = 0;
+        const interactive = utils_1.default.isInteractive();
+        let lastPercent = -1;
+        let lastBucket = -1;
+        const BUCKET_SIZE = 10;
         if (showProgress) {
-            this.drawProgressBar(fileName, totalSize, 0);
+            if (interactive) {
+                this.drawProgressBar(fileName, totalSize, 0);
+            }
             progressTracker.on('progress', (prog) => {
                 const percent = Math.round(prog.percentage);
-                if (percent !== lastPercent) {
-                    lastPercent = percent;
-                    this.drawProgressBar(fileName, totalSize, percent);
+                if (interactive) {
+                    // Redraw whenever percent changes; speed/ETA still update because
+                    // progress-stream emits every 100ms.
+                    if (percent !== lastPercent) {
+                        lastPercent = percent;
+                        this.drawProgressBar(fileName, totalSize, percent, prog.speed, prog.eta);
+                    }
+                }
+                else {
+                    // Non-TTY: one line per 10% bucket, no ANSI, no \r. 100% is
+                    // reported by the success path.
+                    const bucket = Math.floor(percent / BUCKET_SIZE);
+                    if (bucket !== lastBucket && percent < 100) {
+                        lastBucket = bucket;
+                        const transferred = this.formatFileSize((percent / 100) * totalSize);
+                        const total = this.formatFileSize(totalSize);
+                        console.log(`  ${fileName}: ${percent}% (${transferred}/${total})`);
+                    }
                 }
             });
         }
@@ -68,21 +88,36 @@ class Upload {
             const result = response.data;
             if (result.id) {
                 if (showProgress) {
-                    this.drawProgressBar(fileName, totalSize, 100);
-                    console.log('');
+                    if (interactive) {
+                        this.drawProgressBar(fileName, totalSize, 100);
+                        console.log('');
+                    }
+                    else {
+                        console.log(`  ${fileName}: done (${this.formatFileSize(totalSize)})`);
+                    }
                 }
                 return { id: result.id };
             }
             else {
                 if (showProgress) {
-                    console.log(' Failed');
+                    if (interactive) {
+                        console.log(' Failed');
+                    }
+                    else {
+                        console.log(`  ${fileName}: failed`);
+                    }
                 }
                 throw new testingbot_error_1.default(`Upload failed: ${result.error || 'Unknown error'}`);
             }
         }
         catch (error) {
             if (showProgress) {
-                console.log(' Failed');
+                if (interactive) {
+                    console.log(' Failed');
+                }
+                else {
+                    console.log(`  ${fileName}: failed`);
+                }
             }
             if (error instanceof testingbot_error_1.default) {
                 throw error;
@@ -98,7 +133,7 @@ class Upload {
             throw new testingbot_error_1.default(`Upload failed: ${error instanceof Error ? error.message : 'Unknown error'}`, { cause: error instanceof Error ? error : undefined });
         }
     }
-    drawProgressBar(fileName, totalBytes, percent) {
+    drawProgressBar(fileName, totalBytes, percent, bytesPerSec, etaSeconds) {
         const barWidth = 30;
         const filled = Math.round((barWidth * percent) / 100);
         const empty = barWidth - filled;
@@ -106,14 +141,27 @@ class Upload {
         const transferredBytes = (percent / 100) * totalBytes;
         const transferred = this.formatFileSize(transferredBytes);
         const total = this.formatFileSize(totalBytes);
-        process.stdout.write(`\r  ${fileName}: [${bar}] ${percent}% (${transferred}/${total})`);
+        let suffix = '';
+        if (typeof bytesPerSec === 'number' &&
+            Number.isFinite(bytesPerSec) &&
+            bytesPerSec > 0) {
+            suffix += ` • ${this.formatFileSize(bytesPerSec)}/s`;
+        }
+        if (typeof etaSeconds === 'number' &&
+            Number.isFinite(etaSeconds) &&
+            etaSeconds > 0) {
+            suffix += ` • ETA ${this.formatDuration(etaSeconds)}`;
+        }
+        // `\x1b[K` clears from cursor to end of line so residue from a longer
+        // previous frame (e.g. "ETA 16s" → "ETA 1s") can't leak through.
+        process.stdout.write(`\r  ${fileName}: [${bar}] ${percent}% (${transferred}/${total})${suffix}\x1b[K`);
     }
     /**
      * Format file size in human-readable format (KB for small files, MB for larger)
      */
     formatFileSize(bytes) {
         if (bytes < 1024) {
-            return `${bytes} B`;
+            return `${Math.round(bytes)} B`;
         }
         else if (bytes < 1024 * 1024) {
             return `${(bytes / 1024).toFixed(1)} KB`;
@@ -122,6 +170,16 @@ class Upload {
             return `${(bytes / (1024 * 1024)).toFixed(2)} MB`;
         }
     }
+    /**
+     * Format seconds as compact duration: "12s" or "3m04s".
+     */
+    formatDuration(seconds) {
+        if (seconds < 60)
+            return `${Math.round(seconds)}s`;
+        const m = Math.floor(seconds / 60);
+        const s = Math.round(seconds % 60);
+        return `${m}m${s.toString().padStart(2, '0')}s`;
+    }
     async validateFile(filePath) {
         try {
             await node_fs_1.default.promises.access(filePath, node_fs_1.default.constants.R_OK);

package/dist/utils/connectivity.js

@@ -10,15 +10,14 @@ exports.formatConnectivityResults = formatConnectivityResults;
  */
 async function testEndpoint(url, description) {
     const startTime = Date.now();
+    const controller = new AbortController();
+    const timeoutId = setTimeout(() => controller.abort(), 3000);
     try {
-        const controller = new AbortController();
-        const timeoutId = setTimeout(() => controller.abort(), 3000);
         const response = await fetch(url, {
             method: 'HEAD',
             signal: controller.signal,
             redirect: 'manual',
         });
-        clearTimeout(timeoutId);
         const latencyMs = Date.now() - startTime;
         return {
             endpoint: `${description} (${url})`,
@@ -60,6 +59,9 @@ async function testEndpoint(url, description) {
             latencyMs,
         };
     }
+    finally {
+        clearTimeout(timeoutId);
+    }
 }
 /**
  * Check if the system has internet connectivity by testing against

package/dist/utils.d.ts

@@ -1,5 +1,11 @@
 declare const _default: {
     getUserAgent(): string;
+    /**
+     * True when stdout is attached to an interactive terminal. CI environments
+     * (CI=truthy) are always treated as non-interactive so progress animations
+     * don't spam log aggregators.
+     */
+    isInteractive(): boolean;
     getCurrentVersion(): string;
     /**
      * Compare two semver version strings

package/dist/utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../src/utils.ts"],"names":[],"mappings":";oBAQkB,MAAM;yBAID,MAAM;IAI3B;;;OAGG;wBACiB,MAAM,MAAM,MAAM,GAAG,MAAM;IAa/C;;OAEG;6BACsB,MAAM,GAAG,SAAS,GAAG,OAAO;IAWrD;;OAEG;+BACwB,MAAM,GAAG,SAAS,GAAG,OAAO;IAWvD;;OAEG;qCAC8B;QAC/B,UAAU,EAAE,OAAO,CAAC;QACpB,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,OAAO,CAAC,EAAE,MAAM,CAAC;QACjB,SAAS,EAAE,MAAM,CAAC;QAClB,UAAU,CAAC,EAAE,MAAM,CAAC;KACrB,GAAG,IAAI;IAyDR;;OAEG;kCAC2B,MAAM,GAAG,SAAS,GAAG,IAAI;;AA3HzD,wBAqJE"}
+{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../src/utils.ts"],"names":[],"mappings":";oBAQkB,MAAM;IAItB;;;;OAIG;qBACc,OAAO;yBAKH,MAAM;IAI3B;;;OAGG;wBACiB,MAAM,MAAM,MAAM,GAAG,MAAM;IAa/C;;OAEG;6BACsB,MAAM,GAAG,SAAS,GAAG,OAAO;IAWrD;;OAEG;+BACwB,MAAM,GAAG,SAAS,GAAG,OAAO;IAWvD;;OAEG;qCAC8B;QAC/B,UAAU,EAAE,OAAO,CAAC;QACpB,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,OAAO,CAAC,EAAE,MAAM,CAAC;QACjB,SAAS,EAAE,MAAM,CAAC;QAClB,UAAU,CAAC,EAAE,MAAM,CAAC;KACrB,GAAG,IAAI;IAyDR;;OAEG;kCAC2B,MAAM,GAAG,SAAS,GAAG,IAAI;;AArIzD,wBA+JE"}

package/dist/utils.js

@@ -12,6 +12,16 @@ exports.default = {
     getUserAgent() {
         return `TestingBot-CTL-${package_json_1.default.version}`;
     },
+    /**
+     * True when stdout is attached to an interactive terminal. CI environments
+     * (CI=truthy) are always treated as non-interactive so progress animations
+     * don't spam log aggregators.
+     */
+    isInteractive() {
+        if (process.env.CI)
+            return false;
+        return Boolean(process.stdout.isTTY);
+    },
     getCurrentVersion() {
         return package_json_1.default.version;
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testingbot/cli",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "description": "CLI tool to run Espresso, XCUITest and Maestro tests on TestingBot's cloud infrastructure",
   "main": "dist/index.js",
   "bin": {
@@ -16,10 +16,11 @@
   "scripts": {
     "lint": "prettier --check '**/*.{js,ts}' && eslint src/",
     "build": "tsc",
-    "clean": "rm -rf dist",
+    "clean": "node -e \"require('fs').rmSync('dist', {recursive: true, force: true})\"",
     "start": "node dist/index.js",
     "format": "prettier --write '**/*.{js,ts}'",
-    "test": "jest"
+    "test": "jest",
+    "test:coverage": "jest --coverage"
   },
   "keywords": [
     "testingbot",
@@ -57,6 +58,7 @@
     "picocolors": "^1.1.1",
     "progress-stream": "^2.0.0",
     "socket.io-client": "^4.8.1",
+    "testingbot-tunnel-launcher": "^1.1.18",
     "tracer": "^1.3.0"
   },
   "devDependencies": {