12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788 |
- /**
- * @fileoverview Rule to enforce declarations in program or function body root.
- * @author Brandon Mills
- */
- "use strict";
- //------------------------------------------------------------------------------
- // Requirements
- //------------------------------------------------------------------------------
- const astUtils = require("./utils/ast-utils");
- //------------------------------------------------------------------------------
- // Rule Definition
- //------------------------------------------------------------------------------
- const validParent = new Set(["Program", "ExportNamedDeclaration", "ExportDefaultDeclaration"]);
- const validBlockStatementParent = new Set(["FunctionDeclaration", "FunctionExpression", "ArrowFunctionExpression"]);
- module.exports = {
- meta: {
- type: "problem",
- docs: {
- description: "disallow variable or `function` declarations in nested blocks",
- category: "Possible Errors",
- recommended: true,
- url: "https://eslint.org/docs/rules/no-inner-declarations"
- },
- schema: [
- {
- enum: ["functions", "both"]
- }
- ],
- messages: {
- moveDeclToRoot: "Move {{type}} declaration to {{body}} root."
- }
- },
- create(context) {
- /**
- * Ensure that a given node is at a program or function body's root.
- * @param {ASTNode} node Declaration node to check.
- * @returns {void}
- */
- function check(node) {
- const parent = node.parent;
- if (
- parent.type === "BlockStatement" && validBlockStatementParent.has(parent.parent.type)
- ) {
- return;
- }
- if (validParent.has(parent.type)) {
- return;
- }
- const upperFunction = astUtils.getUpperFunction(parent);
- context.report({
- node,
- messageId: "moveDeclToRoot",
- data: {
- type: (node.type === "FunctionDeclaration" ? "function" : "variable"),
- body: (upperFunction === null ? "program" : "function body")
- }
- });
- }
- return {
- FunctionDeclaration: check,
- VariableDeclaration(node) {
- if (context.options[0] === "both" && node.kind === "var") {
- check(node);
- }
- }
- };
- }
- };
|