eslint 8.39.0 → 8.40.0

package/lib/rules/computed-property-spacing.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent spacing inside computed property brackets",
             recommended: false,
-            url: "https://eslint.org/docs/rules/computed-property-spacing"
+            url: "https://eslint.org/docs/latest/rules/computed-property-spacing"
         },
 
         fixable: "whitespace",
@@ -49,7 +49,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const propertyNameMustBeSpaced = context.options[0] === "always"; // default is "never"
         const enforceForClassMembers = !context.options[1] || context.options[1].enforceForClassMembers;
 

package/lib/rules/consistent-return.js

@@ -48,7 +48,7 @@ module.exports = {
         docs: {
             description: "Require `return` statements to either always or never specify values",
             recommended: false,
-            url: "https://eslint.org/docs/rules/consistent-return"
+            url: "https://eslint.org/docs/latest/rules/consistent-return"
         },
 
         schema: [{
@@ -104,7 +104,7 @@ module.exports = {
             } else if (node.type === "ArrowFunctionExpression") {
 
                 // `=>` token
-                loc = context.getSourceCode().getTokenBefore(node.body, astUtils.isArrowToken).loc;
+                loc = context.sourceCode.getTokenBefore(node.body, astUtils.isArrowToken).loc;
             } else if (
                 node.parent.type === "MethodDefinition" ||
                 (node.parent.type === "Property" && node.parent.method)
@@ -115,7 +115,7 @@ module.exports = {
             } else {
 
                 // Function name or `function` keyword.
-                loc = (node.id || context.getSourceCode().getFirstToken(node)).loc;
+                loc = (node.id || context.sourceCode.getFirstToken(node)).loc;
             }
 
             if (!name) {

package/lib/rules/consistent-this.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent naming when capturing the current execution context",
             recommended: false,
-            url: "https://eslint.org/docs/rules/consistent-this"
+            url: "https://eslint.org/docs/latest/rules/consistent-this"
         },
 
         schema: {
@@ -36,7 +36,7 @@ module.exports = {
 
     create(context) {
         let aliases = [];
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         if (context.options.length === 0) {
             aliases.push("that");

package/lib/rules/constructor-super.js

@@ -124,7 +124,7 @@ module.exports = {
         docs: {
             description: "Require `super()` calls in constructors",
             recommended: true,
-            url: "https://eslint.org/docs/rules/constructor-super"
+            url: "https://eslint.org/docs/latest/rules/constructor-super"
         },
 
         schema: [],

package/lib/rules/curly.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent brace style for all control statements",
             recommended: false,
-            url: "https://eslint.org/docs/rules/curly"
+            url: "https://eslint.org/docs/latest/rules/curly"
         },
 
         schema: {
@@ -70,7 +70,7 @@ module.exports = {
         const multiOrNest = (context.options[0] === "multi-or-nest");
         const consistent = (context.options[1] === "consistent");
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/default-case-last.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Enforce default clauses in switch statements to be last",
             recommended: false,
-            url: "https://eslint.org/docs/rules/default-case-last"
+            url: "https://eslint.org/docs/latest/rules/default-case-last"
         },
 
         schema: [],

package/lib/rules/default-case.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Require `default` cases in `switch` statements",
             recommended: false,
-            url: "https://eslint.org/docs/rules/default-case"
+            url: "https://eslint.org/docs/latest/rules/default-case"
         },
 
         schema: [{
@@ -42,7 +42,7 @@ module.exports = {
             ? new RegExp(options.commentPattern, "u")
             : DEFAULT_COMMENT_PATTERN;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/default-param-last.js

@@ -13,7 +13,7 @@ module.exports = {
         docs: {
             description: "Enforce default parameters to be last",
             recommended: false,
-            url: "https://eslint.org/docs/rules/default-param-last"
+            url: "https://eslint.org/docs/latest/rules/default-param-last"
         },
 
         schema: [],

package/lib/rules/dot-location.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent newlines before and after dots",
             recommended: false,
-            url: "https://eslint.org/docs/rules/dot-location"
+            url: "https://eslint.org/docs/latest/rules/dot-location"
         },
 
         schema: [
@@ -43,7 +43,7 @@ module.exports = {
         // default to onObject if no preference is passed
         const onObject = config === "object" || !config;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Reports if the dot between object and property is on the correct location.

package/lib/rules/dot-notation.js

@@ -28,7 +28,7 @@ module.exports = {
         docs: {
             description: "Enforce dot notation whenever possible",
             recommended: false,
-            url: "https://eslint.org/docs/rules/dot-notation"
+            url: "https://eslint.org/docs/latest/rules/dot-notation"
         },
 
         schema: [
@@ -59,7 +59,7 @@ module.exports = {
     create(context) {
         const options = context.options[0] || {};
         const allowKeywords = options.allowKeywords === void 0 || options.allowKeywords;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         let allowPattern;
 

package/lib/rules/eol-last.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Require or disallow newline at the end of files",
             recommended: false,
-            url: "https://eslint.org/docs/rules/eol-last"
+            url: "https://eslint.org/docs/latest/rules/eol-last"
         },
 
         fixable: "whitespace",
@@ -40,7 +40,7 @@ module.exports = {
 
         return {
             Program: function checkBadEOF(node) {
-                const sourceCode = context.getSourceCode(),
+                const sourceCode = context.sourceCode,
                     src = sourceCode.getText(),
                     lastLine = sourceCode.lines[sourceCode.lines.length - 1],
                     location = {

package/lib/rules/eqeqeq.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Require the use of `===` and `!==`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/eqeqeq"
+            url: "https://eslint.org/docs/latest/rules/eqeqeq"
         },
 
         schema: {
@@ -68,7 +68,7 @@ module.exports = {
     create(context) {
         const config = context.options[0] || "always";
         const options = context.options[1] || {};
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         const nullOption = (config === "always")
             ? options.null || "always"

package/lib/rules/for-direction.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Enforce \"for\" loop update clause moving the counter in the right direction",
             recommended: true,
-            url: "https://eslint.org/docs/rules/for-direction"
+            url: "https://eslint.org/docs/latest/rules/for-direction"
         },
 
         fixable: null,

package/lib/rules/func-call-spacing.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Require or disallow spacing between function identifiers and their invocations",
             recommended: false,
-            url: "https://eslint.org/docs/rules/func-call-spacing"
+            url: "https://eslint.org/docs/latest/rules/func-call-spacing"
         },
 
         fixable: "whitespace",
@@ -73,7 +73,7 @@ module.exports = {
 
         const never = context.options[0] !== "always";
         const allowNewlines = !never && context.options[1] && context.options[1].allowNewlines;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const text = sourceCode.getText();
 
         /**

package/lib/rules/func-name-matching.js

@@ -76,7 +76,7 @@ module.exports = {
         docs: {
             description: "Require function names to match the name of the variable or property to which they are assigned",
             recommended: false,
-            url: "https://eslint.org/docs/rules/func-name-matching"
+            url: "https://eslint.org/docs/latest/rules/func-name-matching"
         },
 
         schema: {

package/lib/rules/func-names.js

@@ -32,7 +32,7 @@ module.exports = {
         docs: {
             description: "Require or disallow named `function` expressions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/func-names"
+            url: "https://eslint.org/docs/latest/rules/func-names"
         },
 
         schema: {
@@ -69,7 +69,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Returns the config option for the given node.

package/lib/rules/func-style.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Enforce the consistent use of either `function` declarations or expressions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/func-style"
+            url: "https://eslint.org/docs/latest/rules/func-style"
         },
 
         schema: [

package/lib/rules/function-call-argument-newline.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Enforce line breaks between arguments of a function call",
             recommended: false,
-            url: "https://eslint.org/docs/rules/function-call-argument-newline"
+            url: "https://eslint.org/docs/latest/rules/function-call-argument-newline"
         },
 
         fixable: "whitespace",
@@ -35,7 +35,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         const checkers = {
             unexpected: {

package/lib/rules/function-paren-newline.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent line breaks inside function parentheses",
             recommended: false,
-            url: "https://eslint.org/docs/rules/function-paren-newline"
+            url: "https://eslint.org/docs/latest/rules/function-paren-newline"
         },
 
         fixable: "whitespace",
@@ -57,7 +57,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const rawOption = context.options[0] || "multiline";
         const multilineOption = rawOption === "multiline";
         const multilineArgumentsOption = rawOption === "multiline-arguments";

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

@@ -33,7 +33,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent spacing around `*` operators in generator functions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/generator-star-spacing"
+            url: "https://eslint.org/docs/latest/rules/generator-star-spacing"
         },
 
         fixable: "whitespace",
@@ -102,7 +102,7 @@ module.exports = {
             };
         }(context.options[0] || {}));
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Checks if the given token is a star token or not.

package/lib/rules/getter-return.js

@@ -37,7 +37,7 @@ module.exports = {
         docs: {
             description: "Enforce `return` statements in getters",
             recommended: true,
-            url: "https://eslint.org/docs/rules/getter-return"
+            url: "https://eslint.org/docs/latest/rules/getter-return"
         },
 
         fixable: null,
@@ -64,7 +64,7 @@ module.exports = {
     create(context) {
 
         const options = context.options[0] || { allowImplicit: false };
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         let funcInfo = {
             upper: null,

package/lib/rules/global-require.js

@@ -61,7 +61,7 @@ module.exports = {
         docs: {
             description: "Require `require()` calls to be placed at top-level module scope",
             recommended: false,
-            url: "https://eslint.org/docs/rules/global-require"
+            url: "https://eslint.org/docs/latest/rules/global-require"
         },
 
         schema: [],
@@ -71,7 +71,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             CallExpression(node) {

package/lib/rules/grouped-accessor-pairs.js

@@ -98,7 +98,7 @@ module.exports = {
         docs: {
             description: "Require grouped accessor pairs in object literals and classes",
             recommended: false,
-            url: "https://eslint.org/docs/rules/grouped-accessor-pairs"
+            url: "https://eslint.org/docs/latest/rules/grouped-accessor-pairs"
         },
 
         schema: [
@@ -115,7 +115,7 @@ module.exports = {
 
     create(context) {
         const order = context.options[0] || "anyOrder";
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Reports the given accessor pair.

package/lib/rules/guard-for-in.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Require `for-in` loops to include an `if` statement",
             recommended: false,
-            url: "https://eslint.org/docs/rules/guard-for-in"
+            url: "https://eslint.org/docs/latest/rules/guard-for-in"
         },
 
         schema: [],

package/lib/rules/handle-callback-err.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Require error handling in callbacks",
             recommended: false,
-            url: "https://eslint.org/docs/rules/handle-callback-err"
+            url: "https://eslint.org/docs/latest/rules/handle-callback-err"
         },
 
         schema: [
@@ -38,7 +38,7 @@ module.exports = {
     create(context) {
 
         const errorArgument = context.options[0] || "err";
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Checks if the given argument should be interpreted as a regexp pattern.

package/lib/rules/id-blacklist.js

@@ -121,7 +121,7 @@ module.exports = {
         docs: {
             description: "Disallow specified identifiers",
             recommended: false,
-            url: "https://eslint.org/docs/rules/id-blacklist"
+            url: "https://eslint.org/docs/latest/rules/id-blacklist"
         },
 
         schema: {
@@ -140,7 +140,7 @@ module.exports = {
 
         const denyList = new Set(context.options);
         const reportedNodes = new Set();
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         let globalScope;
 

package/lib/rules/id-denylist.js

@@ -101,7 +101,7 @@ module.exports = {
         docs: {
             description: "Disallow specified identifiers",
             recommended: false,
-            url: "https://eslint.org/docs/rules/id-denylist"
+            url: "https://eslint.org/docs/latest/rules/id-denylist"
         },
 
         schema: {
@@ -121,7 +121,7 @@ module.exports = {
 
         const denyList = new Set(context.options);
         const reportedNodes = new Set();
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         let globalScope;
 

package/lib/rules/id-length.js

@@ -9,41 +9,8 @@
 //------------------------------------------------------------------------------
 // Requirements
 //------------------------------------------------------------------------------
-const GraphemeSplitter = require("grapheme-splitter");
 
-//------------------------------------------------------------------------------
-// Helpers
-//------------------------------------------------------------------------------
-
-/**
- * Checks if the string given as argument is ASCII or not.
- * @param {string} value A string that you want to know if it is ASCII or not.
- * @returns {boolean} `true` if `value` is ASCII string.
- */
-function isASCII(value) {
-    if (typeof value !== "string") {
-        return false;
-    }
-    return /^[\u0020-\u007f]*$/u.test(value);
-}
-
-/** @type {GraphemeSplitter | undefined} */
-let splitter;
-
-/**
- * Gets the length of the string. If the string is not in ASCII, counts graphemes.
- * @param {string} value A string that you want to get the length.
- * @returns {number} The length of `value`.
- */
-function getStringLength(value) {
-    if (isASCII(value)) {
-        return value.length;
-    }
-    if (!splitter) {
-        splitter = new GraphemeSplitter();
-    }
-    return splitter.countGraphemes(value);
-}
+const { getGraphemeCount } = require("../shared/string-utils");
 
 //------------------------------------------------------------------------------
 // Rule Definition
@@ -57,7 +24,7 @@ module.exports = {
         docs: {
             description: "Enforce minimum and maximum identifier lengths",
             recommended: false,
-            url: "https://eslint.org/docs/rules/id-length"
+            url: "https://eslint.org/docs/latest/rules/id-length"
         },
 
         schema: [
@@ -169,7 +136,7 @@ module.exports = {
                 const name = node.name;
                 const parent = node.parent;
 
-                const nameLength = getStringLength(name);
+                const nameLength = getGraphemeCount(name);
 
                 const isShort = nameLength < minLength;
                 const isLong = nameLength > maxLength;

package/lib/rules/id-match.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Require identifiers to match a specified regular expression",
             recommended: false,
-            url: "https://eslint.org/docs/rules/id-match"
+            url: "https://eslint.org/docs/latest/rules/id-match"
         },
 
         schema: [
@@ -67,7 +67,7 @@ module.exports = {
             onlyDeclarations = !!options.onlyDeclarations,
             ignoreDestructuring = !!options.ignoreDestructuring;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         let globalScope;
 
         //--------------------------------------------------------------------------

package/lib/rules/implicit-arrow-linebreak.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Enforce the location of arrow function bodies",
             recommended: false,
-            url: "https://eslint.org/docs/rules/implicit-arrow-linebreak"
+            url: "https://eslint.org/docs/latest/rules/implicit-arrow-linebreak"
         },
 
         fixable: "whitespace",
@@ -34,7 +34,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const option = context.options[0] || "beside";
 
         /**

package/lib/rules/indent-legacy.js

@@ -28,7 +28,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent indentation",
             recommended: false,
-            url: "https://eslint.org/docs/rules/indent-legacy"
+            url: "https://eslint.org/docs/latest/rules/indent-legacy"
         },
 
         deprecated: true,
@@ -206,7 +206,7 @@ module.exports = {
             ObjectExpression: 1
         };
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         if (context.options.length) {
             if (context.options[0] === "tab") {

package/lib/rules/indent.js

@@ -510,7 +510,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent indentation",
             recommended: false,
-            url: "https://eslint.org/docs/rules/indent"
+            url: "https://eslint.org/docs/latest/rules/indent"
         },
 
         fixable: "whitespace",
@@ -703,7 +703,7 @@ module.exports = {
             }
         }
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const tokenInfo = new TokenInfo(sourceCode);
         const offsets = new OffsetStorage(tokenInfo, indentSize, indentType === "space" ? " " : "\t");
         const parameterParens = new WeakSet();

package/lib/rules/init-declarations.js

@@ -50,7 +50,7 @@ module.exports = {
         docs: {
             description: "Require or disallow initialization in variable declarations",
             recommended: false,
-            url: "https://eslint.org/docs/rules/init-declarations"
+            url: "https://eslint.org/docs/latest/rules/init-declarations"
         },
 
         schema: {

package/lib/rules/jsx-quotes.js

@@ -44,7 +44,7 @@ module.exports = {
         docs: {
             description: "Enforce the consistent use of either double or single quotes in JSX attributes",
             recommended: false,
-            url: "https://eslint.org/docs/rules/jsx-quotes"
+            url: "https://eslint.org/docs/latest/rules/jsx-quotes"
         },
 
         fixable: "whitespace",

package/lib/rules/key-spacing.js

@@ -9,13 +9,7 @@
 //------------------------------------------------------------------------------
 
 const astUtils = require("./utils/ast-utils");
-const GraphemeSplitter = require("grapheme-splitter");
-
-const splitter = new GraphemeSplitter();
-
-//------------------------------------------------------------------------------
-// Helpers
-//------------------------------------------------------------------------------
+const { getGraphemeCount } = require("../shared/string-utils");
 
 /**
  * Checks whether a string contains a line terminator as defined in
@@ -144,7 +138,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent spacing between keys and values in object literal properties",
             recommended: false,
-            url: "https://eslint.org/docs/rules/key-spacing"
+            url: "https://eslint.org/docs/latest/rules/key-spacing"
         },
 
         fixable: "whitespace",
@@ -332,7 +326,7 @@ module.exports = {
             singleLineOptions = ruleOptions.singleLine,
             alignmentOptions = ruleOptions.align || null;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Determines if the given property is key-value property.
@@ -523,7 +517,7 @@ module.exports = {
             const startToken = sourceCode.getFirstToken(property);
             const endToken = getLastTokenBeforeColon(property.key);
 
-            return splitter.countGraphemes(sourceCode.getText().slice(startToken.range[0], endToken.range[1]));
+            return getGraphemeCount(sourceCode.getText().slice(startToken.range[0], endToken.range[1]));
         }
 
         /**