Dependencies: npm audit fix

bin/phantomas.js

@@ -13,8 +13,6 @@ const { getProgram, parseArgv } = require("./program"),
   phantomas = require(".."),
   debug = require("debug")("phantomas:cli");
 
-var url = "";
-
 // parse command line arguments
 let options = parseArgv(process.argv);
 
@@ -25,7 +23,7 @@ if (typeof options.url !== "string" && typeof options.config === "undefined") {
   process.exit(1);
 }
 
-url = options.url;
+const url = options.url;
 debug("url: %s", url);
 
 delete options.url;

bin/program.js

@@ -11,7 +11,7 @@ function getProgram() {
   program
     .name("phantomas")
     .description(
-      "Headless Chromium-based web performance metrics collector and monitoring tool"
+      "Headless Chromium-based web performance metrics collector and monitoring tool",
     )
     .version(phantomas.version)
     .usage("--url <url> [options]")
@@ -27,15 +27,15 @@ function getProgram() {
     .option("-d, --debug", "run phantomas in debug mode")
     .option(
       "--modules <modules>",
-      "run selected modules only [moduleOne],[moduleTwo],..."
+      "run selected modules only [moduleOne],[moduleTwo],...",
     )
     .option(
       "--include-dirs <dirs>",
-      "load modules from specified directories [dirOne],[dirTwo],..."
+      "load modules from specified directories [dirOne],[dirTwo],...",
     )
     .option(
       "--skip-modules <modules>",
-      "skip selected modules [moduleOne],[moduleTwo],..."
+      "skip selected modules [moduleOne],[moduleTwo],...",
     )
 
     // .option(
@@ -54,78 +54,78 @@ function getProgram() {
     // HTTP options
     .option(
       "--auth-user <user>",
-      "sets the user name used for HTTP authentication"
+      "sets the user name used for HTTP authentication",
     )
     .option(
       "--auth-pass <password>",
-      "sets the password used for HTTP authentication"
+      "sets the password used for HTTP authentication",
     )
     .option(
       "--cookie <cookies>",
-      'document.cookie formatted string for setting a single cookie (e.g. "bar=foo;domain=url")'
+      'document.cookie formatted string for setting a single cookie (e.g. "bar=foo;domain=url")',
     )
     .option(
       "--cookies-file <file>",
-      "specifies the file name to store the persistent Cookies"
+      "specifies the file name to store the persistent Cookies",
     )
     .option(
       "--local-storage <values>",
-      'ability to set a local storage, key-value pairs (e.g. "bar=foo;domain=url")'
+      'ability to set a local storage, key-value pairs (e.g. "bar=foo;domain=url")',
     )
     .option(
       "--session-storage <values>",
-      'ability to set a session storage, key-value pairs (e.g. "bar=foo;domain=url")'
+      'ability to set a session storage, key-value pairs (e.g. "bar=foo;domain=url")',
     )
     .option(
       "--ignore-ssl-errors",
-      "ignores SSL errors, such as expired or self-signed certificate errors"
+      "ignores SSL errors, such as expired or self-signed certificate errors",
     )
     .option(
       "--proxy <host:port>",
-      "specifies the proxy server to use (e.g. --proxy=192.168.1.42:8080)"
+      "specifies the proxy server to use (e.g. --proxy=192.168.1.42:8080)",
     )
     .option(
       "--proxy-auth <username:password>",
-      "specifies the authentication information for the proxy"
+      "specifies the authentication information for the proxy",
     )
     .option(
       "--proxy-type <type>",
-      "specifies the type of the proxy server [http|socks5|none]"
+      "specifies the type of the proxy server [http|socks5|none]",
     )
     .option(
       "--ssl-protocol <protocol>",
-      "sets the SSL protocol for secure connections [sslv3|sslv2|tlsv1|any]"
+      "sets the SSL protocol for secure connections [sslv3|sslv2|tlsv1|any]",
     )
 
     // Runtime options
     .option(
       "--allow-domain <domain>",
-      "allow requests to given domain(s) - aka whitelist [domain],[domain],..."
+      "allow requests to given domain(s) - aka whitelist [domain],[domain],...",
     )
     .option(
       "--block-domain <domain>",
-      "disallow requests to given domain(s) - aka blacklist [domain],[domain],..."
+      "disallow requests to given domain(s) - aka blacklist [domain],[domain],...",
     )
     .option(
       "--disable-js",
-      "disable JavaScript on the page that will be loaded"
+      "disable JavaScript on the page that will be loaded",
     )
     .option("--no-externals", "block requests to 3rd party domains")
     .option(
       "--post-load-delay <N>",
-      "wait X seconds before generating a report"
+      "wait X seconds before generating a report",
     )
     .option("--scroll", "scroll down the page when it's loaded")
     .option("--spy-eval", "report calls to eval()")
     .option("--stop-at-onload", "stop phantomas immediately after onload event")
     .option("--timeout <seconds>", "timeout for phantomas run", 15)
     .option(
       "--wait-for-event <event>",
-      "wait for a given phantomas event before generating a report"
+      "wait for a given phantomas event before generating a report",
     )
     .option(
       "--wait-for-selector <selector>",
-      "wait for an element matching given CSS selector before generating a report"
+      "wait for an element matching given CSS selector before generating a report",
     )
     .option("--scroll", "scroll down the page when it's loaded")
 
@@ -135,23 +135,23 @@ function getProgram() {
     .option("--colors", "forces ANSI colors even when output is piped")
     .option(
       "--film-strip",
-      "register film strip when page is loading (a comma separated list of milliseconds can be passed)"
+      "register film strip when page is loading (a comma separated list of milliseconds can be passed)",
     )
     .option(
       "--film-strip-dir <dir>",
-      "folder path to output film strip (default is ./filmstrip directory)"
+      "folder path to output film strip (default is ./filmstrip directory)",
     )
     .option("--har <file>", "save HAR to a given file")
     .option("--log <file>", "log to a given file")
     .option("--page-source", "save page source to file")
     .option(
       "--page-source-dir <dir>",
-      "folder path to output page source (default is ./html directory)"
+      "folder path to output page source (default is ./html directory)",
     )
     .option("--pretty", "render formatted JSON")
     .option(
       "--screenshot <file>",
-      "render the viewport to a given file once fully loaded"
+      "render the viewport to a given file once fully loaded",
     )
     .option("--full-page-screenshot", "enlarge the screenshot to full page")
     .option("-s, --silent", "don't write anything to the console");

bin/utils.js

@@ -1,4 +1,5 @@
-const decamelize = require("decamelize");
+// @ts-check
+const decamelize = require("decamelize").default;
 
 function decamelizeOptions(options) {
   // decamelize option names as returned by commander (see issue #863)

core/modules/requestsMonitor/requestsMonitor.js

@@ -153,7 +153,7 @@ function addContentType(headerValue, entry) {
 
     default:
       debug(
-        "Unknown content type found: " + value + " for <" + entry.url + ">"
+        "Unknown content type found: " + value + " for <" + entry.url + ">",
       );
   }
 
@@ -185,7 +185,7 @@ module.exports = function (phantomas) {
       // request data
       // https://github.com/GoogleChrome/puppeteer/blob/v1.11.0/docs/api.md#class-request
       phantomas.emit("send", request); // @desc request has been sent
-    }
+    },
   );
 
   phantomas.on(
@@ -213,7 +213,7 @@ module.exports = function (phantomas) {
       if (entry.transferedSize == 0) {
         entry.transferedSize = parseInt(
           entry.headers["content-length"] || "0",
-          10
+          10,
         );
       }
 
@@ -240,7 +240,7 @@ module.exports = function (phantomas) {
         // resp.timing is empty when handling data:image/gif;base64,R0lGODlhAQABAIABAAAAAP///yH5BAEAAAEALAAAAAABAAEAQAICTAEAOw%3D%3D
         assert(
           typeof resp.timing !== "undefined",
-          "resp.timing is empty when handling " + resp.url
+          "resp.timing is empty when handling " + resp.url,
         );
 
         // how long a given request stalled waiting for DNS, proxy, connection, SSL negotation, etc.
@@ -290,7 +290,7 @@ module.exports = function (phantomas) {
                 headerValue,
                 entry.bodySize / 1024,
                 entry.transferedSize / 1024,
-                entry.bodySize / entry.transferedSize
+                entry.bodySize / entry.transferedSize,
               );
             }
 
@@ -366,13 +366,13 @@ module.exports = function (phantomas) {
           "recv: HTTP %d <%s> [%s]",
           entry.status,
           entry.url,
-          entry.contentType
+          entry.contentType,
         );
         phantomas.emit("recv", entry, resp); // @desc response has been received
       }
 
       phantomas.log("Response metadata: %j", entry);
-    }
+    },
   );
 
   // completion of the last HTTP request
@@ -382,7 +382,7 @@ module.exports = function (phantomas) {
   phantomas.on("recv", (entry) =>
     phantomas.setMetric(
       "httpTrafficCompleted",
-      entry.recvEndTime - loadStartedTime
-    )
+      entry.recvEndTime - loadStartedTime,
+    ),
   );
 };

core/modules/timeToFirstByte/timeToFirstByte.js

@@ -29,7 +29,7 @@ module.exports = function (phantomas) {
       "Time to first byte: set to %d ms for request to <%s> (HTTP %d)",
       entry.timeToFirstByte,
       entry.url,
-      entry.status
+      entry.status,
     );
     phantomas.log("Time to last byte: set to %d ms", entry.timeToLastByte);
 

core/scope.js

@@ -69,7 +69,7 @@
     }
 
     function getBacktrace() {
-      var stack = [];
+      let stack;
 
       try {
         throw new Error("backtrace");
@@ -81,7 +81,7 @@
     }
 
     function getCaller(stepBack) {
-      var caller = false;
+      let caller;
 
       stepBack = stepBack || 0;
 
@@ -112,7 +112,7 @@
       try {
         origConsoleLog.call(
           console,
-          "log:" + stringify(Array.prototype.slice.call(arguments))
+          "log:" + stringify(Array.prototype.slice.call(arguments)),
         );
         // eslint-disable-next-line no-empty
       } catch (e) {}
@@ -192,7 +192,7 @@
       phantomas.log(
         "Spying " +
           (enabled ? "enabled" : "disabled") +
-          (reason ? " - " + reason : "")
+          (reason ? " - " + reason : ""),
       );
     }
 
@@ -208,7 +208,7 @@
       phantomas.log(
         'spy: attaching to "%s" function%s',
         fn,
-        reportResults ? " with results reporting" : ""
+        reportResults ? " with results reporting" : "",
       );
 
       obj[fn] = function () {
@@ -218,7 +218,7 @@
         if (enabled && typeof callback === "function") {
           callback.apply(
             this,
-            reportResults === true ? [results].concat(args) : args
+            reportResults === true ? [results].concat(args) : args,
           );
         }
 
@@ -266,8 +266,8 @@
    * Example: body.logged_out.vis-public.env-production > div > div
    */
   function getDOMPath(node, dontGoUpTheDom /* = false */) {
-    var path = [],
-      entry = "";
+    let path = [],
+      entry;
 
     if (node === window) {
       return "window";
@@ -303,8 +303,8 @@
             0,
             Array.prototype.indexOf.call(
               node.parentNode.children || node.parentNode.childNodes,
-              node
-            )
+              node,
+            ),
           ) +
           "]";
       }
@@ -329,6 +329,6 @@
   phantomas.log(
     "phantomas page scope initialized for <%s> (is an iframe: %s)",
     window.location.toString(),
-    window.parent !== window
+    window.parent !== window,
   );
 })(window);

examples/config.json.example

@@ -1,24 +1,24 @@
 {
-	"url": "http://phantomjs.org",
-	"verbose": true,
-	"reporter": "tap:no-skip",
-	"block-domain": "google-analytics.com, amazonaws.com",
-	"cookies": [
-		{
-			"name": "mobile",
-			"value": "on",
-			"domain": ".phantomjs.org"
-		},
-		{
-			"name": "foo",
-			"value": "bar"
-		}
-	],
-	"asserts": {
-		"requests": 12,
-		"redirects": 0,
-		"domains": 2,
-		"timeToFirstByte": 600,
-		"httpTrafficCompleted": 1000
-	}
+  "url": "http://phantomjs.org",
+  "verbose": true,
+  "reporter": "tap:no-skip",
+  "block-domain": "google-analytics.com, amazonaws.com",
+  "cookies": [
+    {
+      "name": "mobile",
+      "value": "on",
+      "domain": ".phantomjs.org"
+    },
+    {
+      "name": "foo",
+      "value": "bar"
+    }
+  ],
+  "asserts": {
+    "requests": 12,
+    "redirects": 0,
+    "domains": 2,
+    "timeToFirstByte": 600,
+    "httpTrafficCompleted": 1000
+  }
 }

examples/index.js

@@ -41,7 +41,7 @@ promise.on("recv", (response) => {
     response.method,
     response.url,
     response.contentType,
-    response.httpVersion
+    response.httpVersion,
   );
 });