eslint 8.18.0 → 8.21.0

package/lib/rules/prefer-numeric-literals.js

@@ -45,7 +45,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "disallow `parseInt()` and `Number.parseInt()` in favor of binary, octal, and hexadecimal literals",
+            description: "Disallow `parseInt()` and `Number.parseInt()` in favor of binary, octal, and hexadecimal literals",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-numeric-literals"
         },

package/lib/rules/prefer-object-has-own.js

@@ -50,7 +50,7 @@ module.exports = {
         type: "suggestion",
         docs: {
             description:
-                "disallow use of `Object.prototype.hasOwnProperty.call()` and prefer use of `Object.hasOwn()`",
+                "Disallow use of `Object.prototype.hasOwnProperty.call()` and prefer use of `Object.hasOwn()`",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-object-has-own"
         },

package/lib/rules/prefer-object-spread.js

@@ -247,7 +247,7 @@ module.exports = {
 
         docs: {
             description:
-                "disallow using Object.assign with an object literal as the first argument and prefer the use of object spread instead.",
+                "Disallow using Object.assign with an object literal as the first argument and prefer the use of object spread instead.",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-object-spread"
         },

package/lib/rules/prefer-promise-reject-errors.js

@@ -16,7 +16,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require using Error objects as Promise rejection reasons",
+            description: "Require using Error objects as Promise rejection reasons",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-promise-reject-errors"
         },

package/lib/rules/prefer-reflect.js

@@ -15,7 +15,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require `Reflect` methods where applicable",
+            description: "Require `Reflect` methods where applicable",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-reflect"
         },

package/lib/rules/prefer-regex-literals.js

@@ -123,7 +123,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "disallow use of the `RegExp` constructor in favor of regular expression literals",
+            description: "Disallow use of the `RegExp` constructor in favor of regular expression literals",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-regex-literals"
         },

package/lib/rules/prefer-rest-params.js

@@ -65,7 +65,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require rest parameters instead of `arguments`",
+            description: "Require rest parameters instead of `arguments`",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-rest-params"
         },

package/lib/rules/prefer-spread.js

@@ -49,7 +49,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require spread operators instead of `.apply()`",
+            description: "Require spread operators instead of `.apply()`",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-spread"
         },

package/lib/rules/prefer-template.js

@@ -128,7 +128,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require template literals instead of string concatenation",
+            description: "Require template literals instead of string concatenation",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-template"
         },

package/lib/rules/quote-props.js

@@ -22,7 +22,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require quotes around object literal property names",
+            description: "Require quotes around object literal property names",
             recommended: false,
             url: "https://eslint.org/docs/rules/quote-props"
         },

package/lib/rules/quotes.js

@@ -80,7 +80,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "enforce the consistent use of either backticks, double, or single quotes",
+            description: "Enforce the consistent use of either backticks, double, or single quotes",
             recommended: false,
             url: "https://eslint.org/docs/rules/quotes"
         },

package/lib/rules/radix.js

@@ -80,7 +80,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "enforce the consistent use of the radix argument when using `parseInt()`",
+            description: "Enforce the consistent use of the radix argument when using `parseInt()`",
             recommended: false,
             url: "https://eslint.org/docs/rules/radix"
         },

package/lib/rules/require-atomic-updates.js

@@ -171,7 +171,7 @@ module.exports = {
         type: "problem",
 
         docs: {
-            description: "disallow assignments that can lead to race conditions due to usage of `await` or `yield`",
+            description: "Disallow assignments that can lead to race conditions due to usage of `await` or `yield`",
             recommended: false,
             url: "https://eslint.org/docs/rules/require-atomic-updates"
         },

package/lib/rules/require-await.js

@@ -34,7 +34,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "disallow async functions which have no `await` expression",
+            description: "Disallow async functions which have no `await` expression",
             recommended: false,
             url: "https://eslint.org/docs/rules/require-await"
         },

package/lib/rules/require-jsdoc.js

@@ -11,7 +11,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require JSDoc comments",
+            description: "Require JSDoc comments",
             recommended: false,
             url: "https://eslint.org/docs/rules/require-jsdoc"
         },

package/lib/rules/require-unicode-regexp.js

@@ -26,7 +26,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "enforce the use of `u` flag on RegExp",
+            description: "Enforce the use of `u` flag on RegExp",
             recommended: false,
             url: "https://eslint.org/docs/rules/require-unicode-regexp"
         },

package/lib/rules/require-yield.js

@@ -15,7 +15,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require generator functions to contain `yield`",
+            description: "Require generator functions to contain `yield`",
             recommended: true,
             url: "https://eslint.org/docs/rules/require-yield"
         },

package/lib/rules/rest-spread-spacing.js

@@ -15,7 +15,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "enforce spacing between rest and spread operators and their expressions",
+            description: "Enforce spacing between rest and spread operators and their expressions",
             recommended: false,
             url: "https://eslint.org/docs/rules/rest-spread-spacing"
         },

package/lib/rules/semi-spacing.js

@@ -17,7 +17,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "enforce consistent spacing before and after semicolons",
+            description: "Enforce consistent spacing before and after semicolons",
             recommended: false,
             url: "https://eslint.org/docs/rules/semi-spacing"
         },

package/lib/rules/semi-style.js

@@ -73,7 +73,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "enforce location of semicolons",
+            description: "Enforce location of semicolons",
             recommended: false,
             url: "https://eslint.org/docs/rules/semi-style"
         },

package/lib/rules/semi.js

@@ -21,7 +21,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "require or disallow semicolons instead of ASI",
+            description: "Require or disallow semicolons instead of ASI",
             recommended: false,
             url: "https://eslint.org/docs/rules/semi"
         },

package/lib/rules/sort-imports.js

@@ -15,7 +15,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "enforce sorted import declarations within modules",
+            description: "Enforce sorted import declarations within modules",
             recommended: false,
             url: "https://eslint.org/docs/rules/sort-imports"
         },

package/lib/rules/sort-keys.js

@@ -81,7 +81,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require object keys to be sorted",
+            description: "Require object keys to be sorted",
             recommended: false,
             url: "https://eslint.org/docs/rules/sort-keys"
         },
@@ -105,6 +105,10 @@ module.exports = {
                         type: "integer",
                         minimum: 2,
                         default: 2
+                    },
+                    allowLineSeparatedGroups: {
+                        type: "boolean",
+                        default: false
                     }
                 },
                 additionalProperties: false
@@ -124,17 +128,21 @@ module.exports = {
         const insensitive = options && options.caseSensitive === false;
         const natural = options && options.natural;
         const minKeys = options && options.minKeys;
+        const allowLineSeparatedGroups = options && options.allowLineSeparatedGroups || false;
         const isValidOrder = isValidOrders[
             order + (insensitive ? "I" : "") + (natural ? "N" : "")
         ];
 
         // The stack to save the previous property's name for each object literals.
         let stack = null;
+        const sourceCode = context.getSourceCode();
 
         return {
             ObjectExpression(node) {
                 stack = {
                     upper: stack,
+                    prevNode: null,
+                    prevBlankLine: false,
                     prevName: null,
                     numKeys: node.properties.length
                 };
@@ -159,10 +167,45 @@ module.exports = {
                 const numKeys = stack.numKeys;
                 const thisName = getPropertyName(node);
 
+                // Get tokens between current node and previous node
+                const tokens = stack.prevNode && sourceCode
+                    .getTokensBetween(stack.prevNode, node, { includeComments: true });
+
+                let isBlankLineBetweenNodes = stack.prevBlankLine;
+
+                if (tokens) {
+
+                    // check blank line between tokens
+                    tokens.forEach((token, index) => {
+                        const previousToken = tokens[index - 1];
+
+                        if (previousToken && (token.loc.start.line - previousToken.loc.end.line > 1)) {
+                            isBlankLineBetweenNodes = true;
+                        }
+                    });
+
+                    // check blank line between the current node and the last token
+                    if (!isBlankLineBetweenNodes && (node.loc.start.line - tokens[tokens.length - 1].loc.end.line > 1)) {
+                        isBlankLineBetweenNodes = true;
+                    }
+
+                    // check blank line between the first token and the previous node
+                    if (!isBlankLineBetweenNodes && (tokens[0].loc.start.line - stack.prevNode.loc.end.line > 1)) {
+                        isBlankLineBetweenNodes = true;
+                    }
+                }
+
+                stack.prevNode = node;
+
                 if (thisName !== null) {
                     stack.prevName = thisName;
                 }
 
+                if (allowLineSeparatedGroups && isBlankLineBetweenNodes) {
+                    stack.prevBlankLine = thisName === null;
+                    return;
+                }
+
                 if (prevName === null || thisName === null || numKeys < minKeys) {
                     return;
                 }

package/lib/rules/sort-vars.js

@@ -15,7 +15,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require variables within the same declaration block to be sorted",
+            description: "Require variables within the same declaration block to be sorted",
             recommended: false,
             url: "https://eslint.org/docs/rules/sort-vars"
         },

package/lib/rules/space-before-blocks.js

@@ -40,7 +40,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "enforce consistent spacing before blocks",
+            description: "Enforce consistent spacing before blocks",
             recommended: false,
             url: "https://eslint.org/docs/rules/space-before-blocks"
         },

package/lib/rules/space-before-function-paren.js

@@ -20,7 +20,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "enforce consistent spacing before `function` definition opening parenthesis",
+            description: "Enforce consistent spacing before `function` definition opening parenthesis",
             recommended: false,
             url: "https://eslint.org/docs/rules/space-before-function-paren"
         },

package/lib/rules/space-in-parens.js

@@ -16,7 +16,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "enforce consistent spacing inside parentheses",
+            description: "Enforce consistent spacing inside parentheses",
             recommended: false,
             url: "https://eslint.org/docs/rules/space-in-parens"
         },

package/lib/rules/space-infix-ops.js

@@ -16,7 +16,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "require spacing around infix operators",
+            description: "Require spacing around infix operators",
             recommended: false,
             url: "https://eslint.org/docs/rules/space-infix-ops"
         },

package/lib/rules/space-unary-ops.js

@@ -20,7 +20,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "enforce consistent spacing before or after unary operators",
+            description: "Enforce consistent spacing before or after unary operators",
             recommended: false,
             url: "https://eslint.org/docs/rules/space-unary-ops"
         },

package/lib/rules/spaced-comment.js

@@ -152,7 +152,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "enforce consistent spacing after the `//` or `/*` in a comment",
+            description: "Enforce consistent spacing after the `//` or `/*` in a comment",
             recommended: false,
             url: "https://eslint.org/docs/rules/spaced-comment"
         },

package/lib/rules/strict.js

@@ -69,7 +69,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require or disallow strict mode directives",
+            description: "Require or disallow strict mode directives",
             recommended: false,
             url: "https://eslint.org/docs/rules/strict"
         },

package/lib/rules/switch-colon-spacing.js

@@ -21,7 +21,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "enforce spacing around colons of switch statements",
+            description: "Enforce spacing around colons of switch statements",
             recommended: false,
             url: "https://eslint.org/docs/rules/switch-colon-spacing"
         },

package/lib/rules/symbol-description.js

@@ -22,7 +22,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require symbol descriptions",
+            description: "Require symbol descriptions",
             recommended: false,
             url: "https://eslint.org/docs/rules/symbol-description"
         },

package/lib/rules/template-curly-spacing.js

@@ -21,7 +21,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "require or disallow spacing around embedded expressions of template strings",
+            description: "Require or disallow spacing around embedded expressions of template strings",
             recommended: false,
             url: "https://eslint.org/docs/rules/template-curly-spacing"
         },

package/lib/rules/template-tag-spacing.js

@@ -15,7 +15,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "require or disallow spacing between template tags and their literals",
+            description: "Require or disallow spacing between template tags and their literals",
             recommended: false,
             url: "https://eslint.org/docs/rules/template-tag-spacing"
         },

package/lib/rules/unicode-bom.js

@@ -14,7 +14,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "require or disallow Unicode byte order mark (BOM)",
+            description: "Require or disallow Unicode byte order mark (BOM)",
             recommended: false,
             url: "https://eslint.org/docs/rules/unicode-bom"
         },

package/lib/rules/use-isnan.js

@@ -37,7 +37,7 @@ module.exports = {
         type: "problem",
 
         docs: {
-            description: "require calls to `isNaN()` when checking for `NaN`",
+            description: "Require calls to `isNaN()` when checking for `NaN`",
             recommended: true,
             url: "https://eslint.org/docs/rules/use-isnan"
         },

package/lib/rules/valid-jsdoc.js

@@ -21,7 +21,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "enforce valid JSDoc comments",
+            description: "Enforce valid JSDoc comments",
             recommended: false,
             url: "https://eslint.org/docs/rules/valid-jsdoc"
         },

package/lib/rules/valid-typeof.js

@@ -14,7 +14,7 @@ module.exports = {
         type: "problem",
 
         docs: {
-            description: "enforce comparing `typeof` expressions against valid strings",
+            description: "Enforce comparing `typeof` expressions against valid strings",
             recommended: true,
             url: "https://eslint.org/docs/rules/valid-typeof"
         },

package/lib/rules/vars-on-top.js

@@ -15,7 +15,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "require `var` declarations be placed at the top of their containing scope",
+            description: "Require `var` declarations be placed at the top of their containing scope",
             recommended: false,
             url: "https://eslint.org/docs/rules/vars-on-top"
         },

package/lib/rules/wrap-iife.js

@@ -43,7 +43,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "require parentheses around immediate `function` invocations",
+            description: "Require parentheses around immediate `function` invocations",
             recommended: false,
             url: "https://eslint.org/docs/rules/wrap-iife"
         },

package/lib/rules/wrap-regex.js

@@ -15,7 +15,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "require parenthesis around regex literals",
+            description: "Require parenthesis around regex literals",
             recommended: false,
             url: "https://eslint.org/docs/rules/wrap-regex"
         },

package/lib/rules/yield-star-spacing.js

@@ -15,7 +15,7 @@ module.exports = {
         type: "layout",
 
         docs: {
-            description: "require or disallow spacing around the `*` in `yield*` expressions",
+            description: "Require or disallow spacing around the `*` in `yield*` expressions",
             recommended: false,
             url: "https://eslint.org/docs/rules/yield-star-spacing"
         },

package/lib/rules/yoda.js

@@ -121,7 +121,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: 'require or disallow "Yoda" conditions',
+            description: 'Require or disallow "Yoda" conditions',
             recommended: false,
             url: "https://eslint.org/docs/rules/yoda"
         },

package/lib/unsupported-api.js

@@ -12,6 +12,8 @@
 //-----------------------------------------------------------------------------
 
 const { FileEnumerator } = require("./cli-engine/file-enumerator");
+const { FlatESLint } = require("./eslint/flat-eslint");
+const FlatRuleTester = require("./rule-tester/flat-rule-tester");
 
 //-----------------------------------------------------------------------------
 // Exports
@@ -19,5 +21,7 @@ const { FileEnumerator } = require("./cli-engine/file-enumerator");
 
 module.exports = {
     builtinRules: require("./rules"),
+    FlatESLint,
+    FlatRuleTester,
     FileEnumerator
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint",
-  "version": "8.18.0",
+  "version": "8.21.0",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "bin": {
@@ -55,7 +55,8 @@
   "bugs": "https://github.com/eslint/eslint/issues/",
   "dependencies": {
     "@eslint/eslintrc": "^1.3.0",
-    "@humanwhocodes/config-array": "^0.9.2",
+    "@humanwhocodes/config-array": "^0.10.4",
+    "@humanwhocodes/gitignore-to-minimatch": "^1.0.2",
     "ajv": "^6.10.0",
     "chalk": "^4.0.0",
     "cross-spawn": "^7.0.2",
@@ -65,14 +66,17 @@
     "eslint-scope": "^7.1.1",
     "eslint-utils": "^3.0.0",
     "eslint-visitor-keys": "^3.3.0",
-    "espree": "^9.3.2",
+    "espree": "^9.3.3",
     "esquery": "^1.4.0",
     "esutils": "^2.0.2",
     "fast-deep-equal": "^3.1.3",
     "file-entry-cache": "^6.0.1",
+    "find-up": "^5.0.0",
     "functional-red-black-tree": "^1.0.1",
     "glob-parent": "^6.0.1",
     "globals": "^13.15.0",
+    "globby": "^11.1.0",
+    "grapheme-splitter": "^1.0.4",
     "ignore": "^5.2.0",
     "import-fresh": "^3.0.0",
     "imurmurhash": "^0.1.4",
@@ -98,15 +102,14 @@
     "cheerio": "^0.22.0",
     "common-tags": "^1.8.0",
     "core-js": "^3.1.3",
-    "dateformat": "^4.5.1",
     "ejs": "^3.0.2",
     "eslint": "file:.",
     "eslint-config-eslint": "file:packages/eslint-config-eslint",
     "eslint-plugin-eslint-comments": "^3.2.0",
-    "eslint-plugin-eslint-plugin": "^4.2.0",
+    "eslint-plugin-eslint-plugin": "^4.4.0",
     "eslint-plugin-internal-rules": "file:tools/internal-rules",
-    "eslint-plugin-jsdoc": "^37.0.0",
-    "eslint-plugin-node": "^11.1.0",
+    "eslint-plugin-jsdoc": "^38.1.6",
+    "eslint-plugin-n": "^15.2.4",
     "eslint-plugin-unicorn": "^42.0.0",
     "eslint-release": "^3.2.0",
     "eslump": "^3.0.0",
@@ -124,8 +127,8 @@
     "karma-webpack": "^5.0.0",
     "lint-staged": "^11.0.0",
     "load-perf": "^0.2.0",
-    "markdownlint": "^0.24.0",
-    "markdownlint-cli": "^0.30.0",
+    "markdownlint": "^0.25.1",
+    "markdownlint-cli": "^0.31.1",
     "marked": "^4.0.8",
     "memfs": "^3.0.1",
     "metascraper": "^5.25.7",
@@ -142,7 +145,7 @@
     "pirates": "^4.0.5",
     "progress": "^2.0.3",
     "proxyquire": "^2.0.1",
-    "puppeteer": "^9.1.1",
+    "puppeteer": "^13.7.0",
     "recast": "^0.20.4",
     "regenerator-runtime": "^0.13.2",
     "semver": "^7.3.5",