tools: move eslint from tools to tools/node_modules

This is required because we need to add the babel-eslint dependency
and it has to be able to resolve "eslint".
babel-eslint is required to support future ES features such as async
iterators and import.meta.

Refs: https://github.com/nodejs/node/pull/17755
PR-URL: https://github.com/nodejs/node/pull/17820
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>

1 files changed, 55 insertions, 0 deletions

diff --git a/tools/node_modules/eslint/lib/rules/no-implicit-globals.js b/tools/node_modules/eslint/lib/rules/no-implicit-globals.js
new file mode 100644
index 0000000000..f0962cdc7a
--- /dev/null
+++ b/tools/node_modules/eslint/lib/rules/no-implicit-globals.js
@@ -0,0 +1,55 @@
+/**
+ * @fileoverview Rule to check for implicit global variables and functions.
+ * @author Joshua Peek
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = {
+    meta: {
+        docs: {
+            description: "disallow variable and `function` declarations in the global scope",
+            category: "Best Practices",
+            recommended: false
+        },
+
+        schema: []
+    },
+
+    create(context) {
+        return {
+            Program() {
+                const scope = context.getScope();
+
+                scope.variables.forEach(variable => {
+                    if (variable.writeable) {
+                        return;
+                    }
+
+                    variable.defs.forEach(def => {
+                        if (def.type === "FunctionName" || (def.type === "Variable" && def.parent.kind === "var")) {
+                            context.report({ node: def.node, message: "Implicit global variable, assign as global property instead." });
+                        }
+                    });
+                });
+
+                scope.implicit.variables.forEach(variable => {
+                    const scopeVariable = scope.set.get(variable.name);
+
+                    if (scopeVariable && scopeVariable.writeable) {
+                        return;
+                    }
+
+                    variable.defs.forEach(def => {
+                        context.report({ node: def.node, message: "Implicit global variable, assign as global property instead." });
+                    });
+                });
+            }
+        };
+
+    }
+};