tools: remove eslint in tools

.eslintrc.js

@@ -8,9 +8,7 @@ NodePlugin.RULES_DIR = path.resolve(__dirname, 'tools', 'eslint-rules');
 
 const ModuleFindPath = Module._findPath;
 const hacks = [
-  'eslint-plugin-node-core',
-  'eslint-plugin-markdown',
-  'babel-eslint',
+  'eslint-plugin-node-core'
 ];
 Module._findPath = (request, paths, isMain) => {
   const r = ModuleFindPath(request, paths, isMain);

.gitignore

@@ -135,3 +135,5 @@ deps/v8/gypfiles/Release/
 deps/v8/third_party/eu-strip/
 
 .DS_Store
+tools/node_modules/*
+node_modules/*

Makefile

@@ -705,6 +705,7 @@ out/doc/api/assets/%: doc/api_assets/% out/doc/api/assets
 
 
 run-npm-ci = $(PWD)/$(NPM) ci
+run-npm-task = $(PWD)/$(NPM) run
 
 LINK_DATA = out/doc/apilinks.json
 gen-api = tools/doc/generate.js --node-version=$(FULLVERSION) \
@@ -1172,8 +1173,7 @@ lint-md: | tools/.mdlintstamp
 
 LINT_JS_TARGETS = .eslintrc.js benchmark doc lib test tools
 
-run-lint-js = tools/node_modules/eslint/bin/eslint.js --cache \
-	--report-unused-disable-directives --ext=.js,.mjs,.md $(LINT_JS_TARGETS)
+run-lint-js = $(call available-node,$(run-npm-ci)) && $(call available-node,$(run-npm-task) lint)
 run-lint-js-fix = $(run-lint-js) --fix
 
 .PHONY: lint-js-fix

package.json

@@ -0,0 +1,17 @@
+{
+  "name": "node-core",
+  "author": "The Amazing Node.js Contributors",
+  "version": "1.0.0",
+  "description": "Base tools to help manage node core",
+  "license": "MIT",
+  "scripts": {
+    "lint": "eslint --cache --report-unused-disable-directives --ext=.js,.mjs,.md .eslintrc.js benchmark doc lib test tools"
+  },
+  "dependencies": {
+    "babel-eslint": "8.2.1",
+    "eslint": "5.14.1",
+    "eslint-plugin-markdown": "1.0.0",
+    "glob": "7.1.3",
+    "js-yaml": "3.12.1"
+  }
+}

tools/doc/common.js

@@ -1,7 +1,6 @@
 'use strict';
 
-const yaml =
-  require(`${__dirname}/../node_modules/eslint/node_modules/js-yaml`);
+const yaml = require(`js-yaml`);
 
 function isYAMLBlock(text) {
   return /^<!-- YAML/.test(text);

tools/lint-js.js

@@ -11,7 +11,7 @@ const fs = require('fs');
 const totalCPUs = require('os').cpus().length;
 
 const CLIEngine = require('eslint').CLIEngine;
-const glob = require('eslint/node_modules/glob');
+const glob = require('glob');
 
 const cliOptions = {
   rulePaths: rulesDirs,

vcbuild.bat

@@ -626,9 +626,9 @@ goto exit
 :lint-js
 if defined lint_js_ci goto lint-js-ci
 if not defined lint_js goto lint-md-build
-if not exist tools\node_modules\eslint goto no-lint
 echo running lint-js
-%config%\node tools\node_modules\eslint\bin\eslint.js --cache --report-unused-disable-directives --rule "linebreak-style: 0" --ext=.js,.mjs,.md .eslintrc.js benchmark doc lib test tools
+%npm_exe% install
+%npm_exe% run lint
 goto lint-md-build
 
 :lint-js-ci