tools: update ESLint to 5.16.0

Update ESLint to 5.16.0

PR-URL: https://github.com/nodejs/node/pull/27005
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>

tools/node_modules/eslint/README.md

@@ -193,12 +193,13 @@ According to our policy, any minor update may report more errors than the previo
 
 These folks keep the project moving and are resources for help.
 
+<!-- NOTE: This section is autogenerated. Do not manually edit.-->
+<!--teamstart-->
+
 ### Technical Steering Committee (TSC)
 
 The people who manage releases, review feature requests, and meet regularly to ensure ESLint is properly maintained.
 
-<!-- NOTE: This section is autogenerated. Do not manually edit.-->
-<!--tscstart-->
 <table><tbody><tr><td align="center" valign="top" width="11%">
 <a href="https://github.com/nzakas">
 <img src="https://github.com/nzakas.png?s=75" width="75" height="75"><br />
@@ -239,14 +240,15 @@ Kai Cataldo
 <img src="https://github.com/not-an-aardvark.png?s=75" width="75" height="75"><br />
 Teddy Katz
 </a>
-</td></tr></tbody></table><!--tscend-->
+</td></tr></tbody></table>
+
+
+
 
 ### Committers
 
 The people who review and fix bugs and help triage issues.
 
-<!-- NOTE: This section is autogenerated. Do not manually edit.-->
-<!--committersstart-->
 <table><tbody><tr><td align="center" valign="top" width="11%">
 <a href="https://github.com/aladdin-add">
 <img src="https://github.com/aladdin-add.png?s=75" width="75" height="75"><br />
@@ -257,7 +259,10 @@ The people who review and fix bugs and help triage issues.
 <img src="https://github.com/g-plane.png?s=75" width="75" height="75"><br />
 Pig Fang
 </a>
-</td></tr></tbody></table><!--committersend-->
+</td></tr></tbody></table>
+
+
+<!--teamend-->
 
 ## Sponsors
 

tools/node_modules/eslint/lib/cli.js

@@ -81,15 +81,23 @@ function translateOptions(cliOptions) {
  */
 function printResults(engine, results, format, outputFile) {
     let formatter;
+    let rules;
 
     try {
         formatter = engine.getFormatter(format);
+        rules = engine.getRules();
     } catch (e) {
         log.error(e.message);
         return false;
     }
 
-    const output = formatter(results);
+    const rulesMeta = {};
+
+    rules.forEach((rule, ruleId) => {
+        rulesMeta[ruleId] = rule.meta;
+    });
+
+    const output = formatter(results, { rulesMeta });
 
     if (output) {
         if (outputFile) {

tools/node_modules/eslint/lib/formatters/html-template-message.html

@@ -3,6 +3,6 @@
     <td class="clr-<%= severityNumber %>"><%= severityName %></td>
     <td><%- message %></td>
     <td>
-        <a href="https://eslint.org/docs/rules/<%= ruleId %>" target="_blank" rel="noopener noreferrer"><%= ruleId %></a>
+        <a href="<%= ruleUrl %>" target="_blank" rel="noopener noreferrer"><%= ruleId %></a>
     </td>
 </tr>

tools/node_modules/eslint/lib/formatters/html.js

@@ -62,9 +62,10 @@ function renderColor(totalErrors, totalWarnings) {
  * Get HTML (table rows) describing the messages.
  * @param {Array} messages Messages.
  * @param {int} parentIndex Index of the parent HTML row.
+ * @param {Object} rulesMeta Dictionary containing metadata for each rule executed by the analysis.
  * @returns {string} HTML (table rows) describing the messages.
  */
-function renderMessages(messages, parentIndex) {
+function renderMessages(messages, parentIndex, rulesMeta) {
 
     /**
      * Get HTML (table row) describing a message.
@@ -74,6 +75,13 @@ function renderMessages(messages, parentIndex) {
     return lodash.map(messages, message => {
         const lineNumber = message.line || 0;
         const columnNumber = message.column || 0;
+        let ruleUrl;
+
+        if (rulesMeta) {
+            const meta = rulesMeta[message.ruleId];
+
+            ruleUrl = lodash.get(meta, "docs.url", null);
+        }
 
         return messageTemplate({
             parentIndex,
@@ -82,33 +90,37 @@ function renderMessages(messages, parentIndex) {
             severityNumber: message.severity,
             severityName: message.severity === 1 ? "Warning" : "Error",
             message: message.message,
-            ruleId: message.ruleId
+            ruleId: message.ruleId,
+            ruleUrl
         });
     }).join("\n");
 }
 
 /**
  * @param {Array} results Test results.
+ * @param {Object} rulesMeta Dictionary containing metadata for each rule executed by the analysis.
  * @returns {string} HTML string describing the results.
  */
-function renderResults(results) {
+function renderResults(results, rulesMeta) {
     return lodash.map(results, (result, index) => resultTemplate({
         index,
         color: renderColor(result.errorCount, result.warningCount),
         filePath: result.filePath,
         summary: renderSummary(result.errorCount, result.warningCount)
 
-    }) + renderMessages(result.messages, index)).join("\n");
+    }) + renderMessages(result.messages, index, rulesMeta)).join("\n");
 }
 
 //------------------------------------------------------------------------------
 // Public Interface
 //------------------------------------------------------------------------------
 
-module.exports = function(results) {
+module.exports = function(results, data) {
     let totalErrors,
         totalWarnings;
 
+    const metaData = data ? data.rulesMeta : {};
+
     totalErrors = 0;
     totalWarnings = 0;
 
@@ -122,6 +134,6 @@ module.exports = function(results) {
         date: new Date(),
         reportColor: renderColor(totalErrors, totalWarnings),
         reportSummary: renderSummary(totalErrors, totalWarnings),
-        results: renderResults(results)
+        results: renderResults(results, metaData)
     });
 };

tools/node_modules/eslint/lib/formatters/json-with-metadata.js

@@ -0,0 +1,16 @@
+/**
+ * @fileoverview JSON reporter, including rules metadata
+ * @author Chris Meyer
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Public Interface
+//------------------------------------------------------------------------------
+
+module.exports = function(results, data) {
+    return JSON.stringify({
+        results,
+        metadata: data
+    });
+};

tools/node_modules/eslint/node_modules/js-yaml/dist/js-yaml.js

@@ -1,4 +1,4 @@
-/* js-yaml 3.12.2 https://github.com/nodeca/js-yaml */(function(f){if(typeof exports==="object"&&typeof module!=="undefined"){module.exports=f()}else if(typeof define==="function"&&define.amd){define([],f)}else{var g;if(typeof window!=="undefined"){g=window}else if(typeof global!=="undefined"){g=global}else if(typeof self!=="undefined"){g=self}else{g=this}g.jsyaml = f()}})(function(){var define,module,exports;return (function(){function r(e,n,t){function o(i,f){if(!n[i]){if(!e[i]){var c="function"==typeof require&&require;if(!f&&c)return c(i,!0);if(u)return u(i,!0);var a=new Error("Cannot find module '"+i+"'");throw a.code="MODULE_NOT_FOUND",a}var p=n[i]={exports:{}};e[i][0].call(p.exports,function(r){var n=e[i][1][r];return o(n||r)},p,p.exports,r,e,n,t)}return n[i].exports}for(var u="function"==typeof require&&require,i=0;i<t.length;i++)o(t[i]);return o}return r})()({1:[function(require,module,exports){
+/* js-yaml 3.13.0 https://github.com/nodeca/js-yaml */(function(f){if(typeof exports==="object"&&typeof module!=="undefined"){module.exports=f()}else if(typeof define==="function"&&define.amd){define([],f)}else{var g;if(typeof window!=="undefined"){g=window}else if(typeof global!=="undefined"){g=global}else if(typeof self!=="undefined"){g=self}else{g=this}g.jsyaml = f()}})(function(){var define,module,exports;return (function(){function r(e,n,t){function o(i,f){if(!n[i]){if(!e[i]){var c="function"==typeof require&&require;if(!f&&c)return c(i,!0);if(u)return u(i,!0);var a=new Error("Cannot find module '"+i+"'");throw a.code="MODULE_NOT_FOUND",a}var p=n[i]={exports:{}};e[i][0].call(p.exports,function(r){var n=e[i][1][r];return o(n||r)},p,p.exports,r,e,n,t)}return n[i].exports}for(var u="function"==typeof require&&require,i=0;i<t.length;i++)o(t[i]);return o}return r})()({1:[function(require,module,exports){
 'use strict';
 
 
@@ -1262,6 +1262,18 @@ function mergeMappings(state, destination, source, overridableKeys) {
 function storeMappingPair(state, _result, overridableKeys, keyTag, keyNode, valueNode, startLine, startPos) {
   var index, quantity;
 
+  // The output is a plain object here, so keys can only be strings.
+  // We need to convert keyNode to a string, but doing so can hang the process
+  // (deeply nested arrays that explode exponentially using aliases) or execute
+  // code via toString.
+  if (Array.isArray(keyNode)) {
+    for (index = 0, quantity = keyNode.length; index < quantity; index += 1) {
+      if (Array.isArray(keyNode[index])) {
+        throwError(state, 'nested arrays are not supported inside keys');
+      }
+    }
+  }
+
   keyNode = String(keyNode);
 
   if (_result === null) {

tools/node_modules/eslint/node_modules/js-yaml/lib/js-yaml/loader.js

@@ -285,6 +285,18 @@ function mergeMappings(state, destination, source, overridableKeys) {
 function storeMappingPair(state, _result, overridableKeys, keyTag, keyNode, valueNode, startLine, startPos) {
   var index, quantity;
 
+  // The output is a plain object here, so keys can only be strings.
+  // We need to convert keyNode to a string, but doing so can hang the process
+  // (deeply nested arrays that explode exponentially using aliases) or execute
+  // code via toString.
+  if (Array.isArray(keyNode)) {
+    for (index = 0, quantity = keyNode.length; index < quantity; index += 1) {
+      if (Array.isArray(keyNode[index])) {
+        throwError(state, 'nested arrays are not supported inside keys');
+      }
+    }
+  }
+
   keyNode = String(keyNode);
 
   if (_result === null) {

tools/node_modules/eslint/node_modules/js-yaml/package.json

@@ -66,5 +66,5 @@
   "scripts": {
     "test": "make test"
   },
-  "version": "3.12.2"
+  "version": "3.13.0"
 }

tools/node_modules/eslint/node_modules/parent-module/index.js

@@ -16,6 +16,10 @@ module.exports = filepath => {
 	for (const stack of stacks) {
 		const parentFilepath = stack.getFileName();
 
+		if (typeof parentFilepath !== 'string') {
+			continue;
+		}
+
 		if (parentFilepath === filepath) {
 			seenVal = true;
 			continue;

tools/node_modules/eslint/node_modules/parent-module/package.json

@@ -14,9 +14,9 @@
   "deprecated": false,
   "description": "Get the path of the parent module",
   "devDependencies": {
-    "ava": "^1.0.1",
+    "ava": "^1.4.1",
     "execa": "^1.0.0",
-    "xo": "^0.23.0"
+    "xo": "^0.24.0"
   },
   "engines": {
     "node": ">=6"
@@ -51,5 +51,5 @@
   "scripts": {
     "test": "xo && ava"
   },
-  "version": "1.0.0"
+  "version": "1.0.1"
 }

tools/node_modules/eslint/node_modules/parent-module/readme.md

@@ -8,7 +8,7 @@ Node.js exposes `module.parent`, but it only gives you the first cached parent,
 ## Install
 
 ```
-$ npm install --save parent-module
+$ npm install parent-module
 ```
 
 
@@ -21,12 +21,13 @@ const parentModule = require('parent-module');
 module.exports = () => {
 	console.log(parentModule());
 	//=> '/Users/sindresorhus/dev/unicorn/foo.js'
-}
+};
 ```
 
 ```js
 // foo.js
 const bar = require('./bar');
+
 bar();
 ```
 
@@ -57,7 +58,7 @@ const readPkgUp = require('read-pkg-up');
 const parentModule = require('parent-module');
 
 console.log(readPkgUp.sync({cwd: path.dirname(parentModule())}).pkg);
-//=> {name: 'chalk', version: '1.0.0', ...}
+//=> {name: 'chalk', version: '1.0.0', …}
 ```
 
 

tools/node_modules/eslint/node_modules/semver/README.md

@@ -20,6 +20,7 @@ semver.clean('  =v1.2.3   ') // '1.2.3'
 semver.satisfies('1.2.3', '1.x || >=2.5.0 || 5.0.0 - 7.2.3') // true
 semver.gt('1.2.3', '9.8.7') // false
 semver.lt('1.2.3', '9.8.7') // true
+semver.minVersion('>=1.0.0') // '1.0.0'
 semver.valid(semver.coerce('v2')) // '2.0.0'
 semver.valid(semver.coerce('42.6.7.9.3-alpha')) // '42.6.7'
 ```
@@ -29,7 +30,7 @@ As a command-line utility:
 ```
 $ semver -h
 
-A JavaScript implementation of the http://semver.org/ specification
+A JavaScript implementation of the https://semver.org/ specification
 Copyright Isaac Z. Schlueter
 
 Usage: semver [options] <version> [<version> [...]]
@@ -71,7 +72,7 @@ multiple versions to the utility will just sort them.
 ## Versions
 
 A "version" is described by the `v2.0.0` specification found at
-<http://semver.org/>.
+<https://semver.org/>.
 
 A leading `"="` or `"v"` character is stripped off and ignored.
 
@@ -137,6 +138,13 @@ the user is indicating that they are aware of the risk.  However, it
 is still not appropriate to assume that they have opted into taking a
 similar risk on the *next* set of prerelease versions.
 
+Note that this behavior can be suppressed (treating all prerelease
+versions as if they were normal versions, for the purpose of range
+matching) by setting the `includePrerelease` flag on the options
+object to any
+[functions](https://github.com/npm/node-semver#functions) that do
+range matching.
+
 #### Prerelease Identifiers
 
 The method `.inc` takes an additional `identifier` string argument that
@@ -325,6 +333,8 @@ strings that they parse.
 * `patch(v)`: Return the patch version number.
 * `intersects(r1, r2, loose)`: Return true if the two supplied ranges
   or comparators intersect.
+* `parse(v)`: Attempt to parse a string as a semantic version, returning either
+  a `SemVer` object or `null`.
 
 ### Comparison
 
@@ -361,6 +371,8 @@ strings that they parse.
   that satisfies the range, or `null` if none of them do.
 * `minSatisfying(versions, range)`: Return the lowest version in the list
   that satisfies the range, or `null` if none of them do.
+* `minVersion(range)`: Return the lowest version that can possibly match
+  the given range.
 * `gtr(version, range)`: Return `true` if version is greater than all the
   versions possible in the range.
 * `ltr(version, range)`: Return `true` if version is less than all the

tools/node_modules/eslint/node_modules/semver/bin/semver

@@ -4,20 +4,28 @@
 // any supplied version is valid and passes all tests.
 
 var argv = process.argv.slice(2)
-  , versions = []
+
-  , range = []
+var versions = []
-  , gt = []
+
-  , lt = []
+var range = []
-  , eq = []
+
-  , inc = null
+var inc = null
-  , version = require("../package.json").version
+
-  , loose = false
+var version = require('../package.json').version
-  , includePrerelease = false
+
-  , coerce = false
+var loose = false
-  , identifier = undefined
+
-  , semver = require("../semver")
+var includePrerelease = false
-  , reverse = false
+
-  , options = {}
+var coerce = false
+
+var identifier
+
+var semver = require('../semver')
+
+var reverse = false
+
+var options = {}
 
 main()
 
@@ -25,45 +33,45 @@ function main () {
   if (!argv.length) return help()
   while (argv.length) {
     var a = argv.shift()
-    var i = a.indexOf('=')
+    var indexOfEqualSign = a.indexOf('=')
-    if (i !== -1) {
+    if (indexOfEqualSign !== -1) {
-      a = a.slice(0, i)
+      a = a.slice(0, indexOfEqualSign)
-      argv.unshift(a.slice(i + 1))
+      argv.unshift(a.slice(indexOfEqualSign + 1))
     }
     switch (a) {
-      case "-rv": case "-rev": case "--rev": case "--reverse":
+      case '-rv': case '-rev': case '--rev': case '--reverse':
         reverse = true
         break
-      case "-l": case "--loose":
+      case '-l': case '--loose':
         loose = true
         break
-      case "-p": case "--include-prerelease":
+      case '-p': case '--include-prerelease':
         includePrerelease = true
         break
-      case "-v": case "--version":
+      case '-v': case '--version':
         versions.push(argv.shift())
         break
-      case "-i": case "--inc": case "--increment":
+      case '-i': case '--inc': case '--increment':
         switch (argv[0]) {
-          case "major": case "minor": case "patch": case "prerelease":
+          case 'major': case 'minor': case 'patch': case 'prerelease':
-          case "premajor": case "preminor": case "prepatch":
+          case 'premajor': case 'preminor': case 'prepatch':
             inc = argv.shift()
             break
           default:
-            inc = "patch"
+            inc = 'patch'
             break
         }
         break
-      case "--preid":
+      case '--preid':
         identifier = argv.shift()
         break
-      case "-r": case "--range":
+      case '-r': case '--range':
         range.push(argv.shift())
         break
-      case "-c": case "--coerce":
+      case '-c': case '--coerce':
         coerce = true
         break
-      case "-h": case "--help": case "-?":
+      case '-h': case '--help': case '-?':
         return help()
       default:
         versions.push(a)
@@ -74,15 +82,14 @@ function main () {
   var options = { loose: loose, includePrerelease: includePrerelease }
 
   versions = versions.map(function (v) {
-    return coerce ? (semver.coerce(v) || {version: v}).version : v
+    return coerce ? (semver.coerce(v) || { version: v }).version : v
   }).filter(function (v) {
     return semver.valid(v)
   })
   if (!versions.length) return fail()
-  if (inc && (versions.length !== 1 || range.length))
+  if (inc && (versions.length !== 1 || range.length)) { return failInc() }
-    return failInc()
 
-  for (var i = 0, l = range.length; i < l ; i ++) {
+  for (var i = 0, l = range.length; i < l; i++) {
     versions = versions.filter(function (v) {
       return semver.satisfies(v, range[i], options)
     })
@@ -92,62 +99,62 @@ function main () {
 }
 
 function failInc () {
-  console.error("--inc can only be used on a single version with no range")
+  console.error('--inc can only be used on a single version with no range')
   fail()
 }
 
 function fail () { process.exit(1) }
 
 function success () {
-  var compare = reverse ? "rcompare" : "compare"
+  var compare = reverse ? 'rcompare' : 'compare'
   versions.sort(function (a, b) {
     return semver[compare](a, b, options)
   }).map(function (v) {
     return semver.clean(v, options)
   }).map(function (v) {
     return inc ? semver.inc(v, inc, options, identifier) : v
-  }).forEach(function (v,i,_) { console.log(v) })
+  }).forEach(function (v, i, _) { console.log(v) })
 }
 
 function help () {
-  console.log(["SemVer " + version
+  console.log(['SemVer ' + version,
-              ,""
+    '',
-              ,"A JavaScript implementation of the http://semver.org/ specification"
+    'A JavaScript implementation of the https://semver.org/ specification',
-              ,"Copyright Isaac Z. Schlueter"
+    'Copyright Isaac Z. Schlueter',
-              ,""
+    '',
-              ,"Usage: semver [options] <version> [<version> [...]]"
+    'Usage: semver [options] <version> [<version> [...]]',
-              ,"Prints valid versions sorted by SemVer precedence"
+    'Prints valid versions sorted by SemVer precedence',
-              ,""
+    '',
-              ,"Options:"
+    'Options:',
-              ,"-r --range <range>"
+    '-r --range <range>',
-              ,"        Print versions that match the specified range."
+    '        Print versions that match the specified range.',
-              ,""
+    '',
-              ,"-i --increment [<level>]"
+    '-i --increment [<level>]',
-              ,"        Increment a version by the specified level.  Level can"
+    '        Increment a version by the specified level.  Level can',
-              ,"        be one of: major, minor, patch, premajor, preminor,"
+    '        be one of: major, minor, patch, premajor, preminor,',
-              ,"        prepatch, or prerelease.  Default level is 'patch'."
+    "        prepatch, or prerelease.  Default level is 'patch'.",
-              ,"        Only one version may be specified."
+    '        Only one version may be specified.',
-              ,""
+    '',
-              ,"--preid <identifier>"
+    '--preid <identifier>',
-              ,"        Identifier to be used to prefix premajor, preminor,"
+    '        Identifier to be used to prefix premajor, preminor,',
-              ,"        prepatch or prerelease version increments."
+    '        prepatch or prerelease version increments.',
-              ,""
+    '',
-              ,"-l --loose"
+    '-l --loose',
-              ,"        Interpret versions and ranges loosely"
+    '        Interpret versions and ranges loosely',
-              ,""
+    '',
-              ,"-p --include-prerelease"
+    '-p --include-prerelease',
-              ,"        Always include prerelease versions in range matching"
+    '        Always include prerelease versions in range matching',
-              ,""
+    '',
-              ,"-c --coerce"
+    '-c --coerce',
-              ,"        Coerce a string into SemVer if possible"
+    '        Coerce a string into SemVer if possible',
-              ,"        (does not imply --loose)"
+    '        (does not imply --loose)',
-              ,""
+    '',
-              ,"Program exits successfully if any valid version satisfies"
+    'Program exits successfully if any valid version satisfies',
-              ,"all supplied ranges, and prints all satisfying versions."
+    'all supplied ranges, and prints all satisfying versions.',
-              ,""
+    '',
-              ,"If no satisfying versions are found, then exits failure."
+    'If no satisfying versions are found, then exits failure.',
-              ,""
+    '',
-              ,"Versions are printed in ascending order, so supplying"
+    'Versions are printed in ascending order, so supplying',
-              ,"multiple versions to the utility will just sort them."
+    'multiple versions to the utility will just sort them.'
-              ].join("\n"))
+  ].join('\n'))
 }

tools/node_modules/eslint/node_modules/semver/package.json

@@ -9,7 +9,7 @@
   "deprecated": false,
   "description": "The semantic version parser used by npm.",
   "devDependencies": {
-    "tap": "^12.0.1"
+    "tap": "^13.0.0-rc.18"
   },
   "files": [
     "bin",
@@ -25,7 +25,13 @@
     "url": "git+https://github.com/npm/node-semver.git"
   },
   "scripts": {
-    "test": "tap test/*.js --cov -J"
+    "postpublish": "git push origin --all; git push origin --tags",
+    "postversion": "npm publish",
+    "preversion": "npm test",
+    "test": "tap"
   },
-  "version": "5.6.0"
+  "tap": {
+    "check-coverage": true
+  },
+  "version": "5.7.0"
 }

tools/node_modules/eslint/package.json

@@ -32,7 +32,7 @@
     "import-fresh": "^3.0.0",
     "imurmurhash": "^0.1.4",
     "inquirer": "^6.2.2",
-    "js-yaml": "^3.12.0",
+    "js-yaml": "^3.13.0",
     "json-stable-stringify-without-jsonify": "^1.0.1",
     "levn": "^0.3.0",
     "lodash": "^4.17.11",
@@ -71,7 +71,6 @@
     "eslint-release": "^1.2.0",
     "eslump": "^2.0.0",
     "esprima": "^4.0.1",
-    "istanbul": "^0.4.5",
     "jsdoc": "^3.5.5",
     "karma": "^3.1.4",
     "karma-chrome-launcher": "^2.2.0",
@@ -84,6 +83,7 @@
     "mocha": "^5.0.5",
     "mock-fs": "^4.8.0",
     "npm-license": "^0.3.3",
+    "nyc": "^13.3.0",
     "proxyquire": "^2.0.1",
     "puppeteer": "^1.12.2",
     "shelljs": "^0.8.2",
@@ -135,5 +135,5 @@
     "test": "node Makefile.js test",
     "webpack": "node Makefile.js webpack"
   },
-  "version": "5.15.3"
+  "version": "5.16.0"
 }