@percy/client 1.28.8-beta.6 → 1.28.9-beta.0

package/dist/client.js

@@ -3,7 +3,7 @@ import PercyEnv from '@percy/env';
 import { git } from '@percy/env/utils';
 import logger from '@percy/logger';
 import Pako from 'pako';
-import { pool, request, formatBytes, sha256hash, base64encode, getPackageJSON, waitForTimeout, validateTiles, tagsList } from './utils.js';
+import { pool, request, formatBytes, sha256hash, base64encode, getPackageJSON, waitForTimeout, validateTiles, formatLogErrors, tagsList } from './utils.js';
 
 // Default client API URL can be set with an env var for API development
 const {
@@ -543,8 +543,12 @@ export class PercyClient {
       retries -= 1;
     } while (retries > 0 && !success);
     if (!success) {
-      this.log.error('Uploading comparison tile failed');
-      return false;
+      let errMsg = 'Uploading comparison tile failed';
+
+      // Detecting error and logging fix for the same
+      // We are throwing this error as the comparison will be failed
+      // even if 1 tile gets failed
+      throw new Error(errMsg);
     }
     return true;
   }
@@ -609,6 +613,13 @@ export class PercyClient {
       data: body
     });
   }
+  async getErrorAnalysis(errors) {
+    const errorLogs = formatLogErrors(errors);
+    this.log.debug('Sending error logs for analysis');
+    return this.post('suggestions/from_logs', {
+      data: errorLogs
+    });
+  }
   mayBeLogUploadSize(contentSize) {
     if (contentSize >= 25 * 1024 * 1024) {
       this.log.error('Uploading resource above 25MB might fail the build...');

package/dist/proxy.js

@@ -155,8 +155,16 @@ export class ProxyHttpsAgent extends https.Agent {
     // start the proxy connection and setup listeners
     let socket = proxy.connect();
     let handleError = err => {
+      var _err$message, _err$message2;
       socket.destroy(err);
       logger('client:proxy').error(`Proxying request failed: ${err}`);
+
+      // We don't get statusCode here, relying on checking error message only
+      if (!!err.message && ((_err$message = err.message) !== null && _err$message !== void 0 && _err$message.includes('ECONNREFUSED') || (_err$message2 = err.message) !== null && _err$message2 !== void 0 && _err$message2.includes('EHOSTUNREACH'))) {
+        logger('client:proxy').warn('If needed, Please verify if your proxy credentials are correct');
+        logger('client:proxy').warn('Please check if your proxy is set correctly and reachable');
+      }
+      logger('client:proxy').warn('Please check network connection, proxy and ensure that following domains are whitelisted: github.com, percy.io, storage.googleapis.com. In case you are an enterprise customer make sure to whitelist "percy-enterprise.browserstack.com" as well.');
       callback(err);
     };
     let handleClose = () => handleError(new Error('Connection closed while sending request to upstream proxy'));

package/dist/utils.js

@@ -233,6 +233,26 @@ export function validateTiles(tiles) {
   }
   return true;
 }
+export function formatLogErrors(errorLogs) {
+  let errors = [];
+  if (typeof errorLogs === 'string') {
+    errors.push({
+      message: errorLogs
+    });
+  } else if (Array.isArray(errorLogs)) {
+    errors = errorLogs;
+  } else {
+    errors.push({
+      message: errorLogs
+    });
+    errors.push({
+      message: (errorLogs === null || errorLogs === void 0 ? void 0 : errorLogs.message) || ''
+    });
+  }
+  return {
+    logs: errors
+  };
+}
 
 // convert tags comma-separated-names to array of objects for POST request
 export function tagsList(tags) {
@@ -246,4 +266,4 @@ export function tagsList(tags) {
   }
   return tagsArr;
 }
-export { hostnameMatches, ProxyHttpAgent, ProxyHttpsAgent, proxyAgentFor } from './proxy.js';
+export { hostnameMatches, getProxy, ProxyHttpAgent, ProxyHttpsAgent, proxyAgentFor } from './proxy.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@percy/client",
-  "version": "1.28.8-beta.6",
+  "version": "1.28.9-beta.0",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -32,9 +32,9 @@
     "test:coverage": "yarn test --coverage"
   },
   "dependencies": {
-    "@percy/env": "1.28.8-beta.6",
-    "@percy/logger": "1.28.8-beta.6",
+    "@percy/env": "1.28.9-beta.0",
+    "@percy/logger": "1.28.9-beta.0",
     "pako": "^2.1.0"
   },
-  "gitHead": "33ebd1aded619784a02e27cb26dbd344c7131ee9"
+  "gitHead": "a1114f1e18518012f48756c9558a8e7895d2b3a9"
 }