eslint 8.38.0 → 8.40.0

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]));
         }
 
         /**

package/lib/rules/keyword-spacing.js

@@ -69,7 +69,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent spacing before and after keywords",
             recommended: false,
-            url: "https://eslint.org/docs/rules/keyword-spacing"
+            url: "https://eslint.org/docs/latest/rules/keyword-spacing"
         },
 
         fixable: "whitespace",
@@ -108,7 +108,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         const tokensToIgnore = new WeakSet();
 

package/lib/rules/line-comment-position.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Enforce position of line comments",
             recommended: false,
-            url: "https://eslint.org/docs/rules/line-comment-position"
+            url: "https://eslint.org/docs/latest/rules/line-comment-position"
         },
 
         schema: [
@@ -78,7 +78,7 @@ module.exports = {
         const defaultIgnoreRegExp = astUtils.COMMENTS_IGNORE_PATTERN;
         const fallThroughRegExp = /^\s*falls?\s?through/u;
         const customIgnoreRegExp = new RegExp(ignorePattern, "u");
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Public

package/lib/rules/linebreak-style.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent linebreak style",
             recommended: false,
-            url: "https://eslint.org/docs/rules/linebreak-style"
+            url: "https://eslint.org/docs/latest/rules/linebreak-style"
         },
 
         fixable: "whitespace",
@@ -40,7 +40,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/lines-around-comment.js

@@ -57,7 +57,7 @@ module.exports = {
         docs: {
             description: "Require empty lines around comments",
             recommended: false,
-            url: "https://eslint.org/docs/rules/lines-around-comment"
+            url: "https://eslint.org/docs/latest/rules/lines-around-comment"
         },
 
         fixable: "whitespace",
@@ -138,7 +138,7 @@ module.exports = {
 
         options.beforeBlockComment = typeof options.beforeBlockComment !== "undefined" ? options.beforeBlockComment : true;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         const lines = sourceCode.lines,
             numLines = lines.length + 1,

package/lib/rules/lines-around-directive.js

@@ -20,7 +20,7 @@ module.exports = {
         docs: {
             description: "Require or disallow newlines around directives",
             recommended: false,
-            url: "https://eslint.org/docs/rules/lines-around-directive"
+            url: "https://eslint.org/docs/latest/rules/lines-around-directive"
         },
 
         schema: [{
@@ -54,7 +54,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const config = context.options[0] || "always";
         const expectLineBefore = typeof config === "string" ? config : config.before;
         const expectLineAfter = typeof config === "string" ? config : config.after;

package/lib/rules/lines-between-class-members.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Require or disallow an empty line between class members",
             recommended: false,
-            url: "https://eslint.org/docs/rules/lines-between-class-members"
+            url: "https://eslint.org/docs/latest/rules/lines-between-class-members"
         },
 
         fixable: "whitespace",
@@ -55,7 +55,7 @@ module.exports = {
         options[0] = context.options[0] || "always";
         options[1] = context.options[1] || { exceptAfterSingleLine: false };
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Gets a pair of tokens that should be used to check lines between two class member nodes.

package/lib/rules/logical-assignment-operators.js

@@ -161,7 +161,7 @@ module.exports = {
         docs: {
             description: "Require or disallow logical assignment operator shorthand",
             recommended: false,
-            url: "https://eslint.org/docs/rules/logical-assignment-operators"
+            url: "https://eslint.org/docs/latest/rules/logical-assignment-operators"
         },
 
         schema: {
@@ -205,7 +205,7 @@ module.exports = {
     create(context) {
         const mode = context.options[0] === "never" ? "never" : "always";
         const checkIf = mode === "always" && context.options.length > 1 && context.options[1].enforceForIfStatements;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const isStrict = sourceCode.getScope(sourceCode.ast).isStrict;
 
         /**

package/lib/rules/max-classes-per-file.js

@@ -21,7 +21,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of classes per file",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-classes-per-file"
+            url: "https://eslint.org/docs/latest/rules/max-classes-per-file"
         },
 
         schema: [

package/lib/rules/max-depth.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum depth that blocks can be nested",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-depth"
+            url: "https://eslint.org/docs/latest/rules/max-depth"
         },
 
         schema: [

package/lib/rules/max-len.js

@@ -71,7 +71,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum line length",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-len"
+            url: "https://eslint.org/docs/latest/rules/max-len"
         },
 
         schema: [
@@ -97,7 +97,7 @@ module.exports = {
          */
         const URL_REGEXP = /[^:/?#]:\/\/[^?#]/u;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Computes the length of a line that may contain tabs. The width of each

package/lib/rules/max-lines-per-function.js

@@ -73,7 +73,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of lines of code in a function",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-lines-per-function"
+            url: "https://eslint.org/docs/latest/rules/max-lines-per-function"
         },
 
         schema: [
@@ -85,7 +85,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const lines = sourceCode.lines;
 
         const option = context.options[0];

package/lib/rules/max-lines.js

@@ -36,7 +36,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of lines per file",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-lines"
+            url: "https://eslint.org/docs/latest/rules/max-lines"
         },
 
         schema: [
@@ -87,7 +87,7 @@ module.exports = {
         const skipComments = option && option.skipComments;
         const skipBlankLines = option && option.skipBlankLines;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Returns whether or not a token is a comment node type

package/lib/rules/max-nested-callbacks.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum depth that callbacks can be nested",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-nested-callbacks"
+            url: "https://eslint.org/docs/latest/rules/max-nested-callbacks"
         },
 
         schema: [

package/lib/rules/max-params.js

@@ -24,7 +24,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of parameters in function definitions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-params"
+            url: "https://eslint.org/docs/latest/rules/max-params"
         },
 
         schema: [
@@ -57,7 +57,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const option = context.options[0];
         let numParams = 3;
 

package/lib/rules/max-statements-per-line.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of statements allowed per line",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-statements-per-line"
+            url: "https://eslint.org/docs/latest/rules/max-statements-per-line"
         },
 
         schema: [
@@ -45,7 +45,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode(),
+        const sourceCode = context.sourceCode,
             options = context.options[0] || {},
             maxStatementsPerLine = typeof options.max !== "undefined" ? options.max : 1;
 

package/lib/rules/max-statements.js

@@ -24,7 +24,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of statements allowed in function blocks",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-statements"
+            url: "https://eslint.org/docs/latest/rules/max-statements"
         },
 
         schema: [

package/lib/rules/multiline-comment-style.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Enforce a particular style for multiline comments",
             recommended: false,
-            url: "https://eslint.org/docs/rules/multiline-comment-style"
+            url: "https://eslint.org/docs/latest/rules/multiline-comment-style"
         },
 
         fixable: "whitespace",
@@ -65,7 +65,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const option = context.options[0] || "starred-block";
         const params = context.options[1] || {};
         const checkJSDoc = !!params.checkJSDoc;

package/lib/rules/multiline-ternary.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Enforce newlines between operands of ternary expressions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/multiline-ternary"
+            url: "https://eslint.org/docs/latest/rules/multiline-ternary"
         },
 
         schema: [
@@ -39,7 +39,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const option = context.options[0];
         const multiline = option !== "never";
         const allowSingleLine = option === "always-multiline";

package/lib/rules/new-cap.js

@@ -84,7 +84,7 @@ module.exports = {
         docs: {
             description: "Require constructor names to begin with a capital letter",
             recommended: false,
-            url: "https://eslint.org/docs/rules/new-cap"
+            url: "https://eslint.org/docs/latest/rules/new-cap"
         },
 
         schema: [
@@ -147,7 +147,7 @@ module.exports = {
 
         const listeners = {};
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/new-parens.js

@@ -27,7 +27,7 @@ module.exports = {
         docs: {
             description: "Enforce or disallow parentheses when invoking a constructor with no arguments",
             recommended: false,
-            url: "https://eslint.org/docs/rules/new-parens"
+            url: "https://eslint.org/docs/latest/rules/new-parens"
         },
 
         fixable: "code",
@@ -46,7 +46,7 @@ module.exports = {
         const options = context.options;
         const always = options[0] !== "never"; // Default is always
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             NewExpression(node) {

package/lib/rules/newline-after-var.js

@@ -24,7 +24,7 @@ module.exports = {
         docs: {
             description: "Require or disallow an empty line after variable declarations",
             recommended: false,
-            url: "https://eslint.org/docs/rules/newline-after-var"
+            url: "https://eslint.org/docs/latest/rules/newline-after-var"
         },
         schema: [
             {
@@ -43,7 +43,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         // Default `mode` to "always".
         const mode = context.options[0] === "never" ? "never" : "always";

package/lib/rules/newline-before-return.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Require an empty line before `return` statements",
             recommended: false,
-            url: "https://eslint.org/docs/rules/newline-before-return"
+            url: "https://eslint.org/docs/latest/rules/newline-before-return"
         },
 
         fixable: "whitespace",
@@ -31,7 +31,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/newline-per-chained-call.js

@@ -20,7 +20,7 @@ module.exports = {
         docs: {
             description: "Require a newline after each call in a method chain",
             recommended: false,
-            url: "https://eslint.org/docs/rules/newline-per-chained-call"
+            url: "https://eslint.org/docs/latest/rules/newline-per-chained-call"
         },
 
         fixable: "whitespace",
@@ -47,7 +47,7 @@ module.exports = {
         const options = context.options[0] || {},
             ignoreChainWithDepth = options.ignoreChainWithDepth || 2;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Get the prefix of a given MemberExpression node.

package/lib/rules/no-alert.js

@@ -90,7 +90,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `alert`, `confirm`, and `prompt`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-alert"
+            url: "https://eslint.org/docs/latest/rules/no-alert"
         },
 
         schema: [],
@@ -101,7 +101,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             CallExpression(node) {

package/lib/rules/no-array-constructor.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow `Array` constructors",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-array-constructor"
+            url: "https://eslint.org/docs/latest/rules/no-array-constructor"
         },
 
         schema: [],

package/lib/rules/no-async-promise-executor.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow using an async function as a Promise executor",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-async-promise-executor"
+            url: "https://eslint.org/docs/latest/rules/no-async-promise-executor"
         },
 
         fixable: null,
@@ -30,7 +30,7 @@ module.exports = {
         return {
             "NewExpression[callee.name='Promise'][arguments.0.async=true]"(node) {
                 context.report({
-                    node: context.getSourceCode().getFirstToken(node.arguments[0], token => token.value === "async"),
+                    node: context.sourceCode.getFirstToken(node.arguments[0], token => token.value === "async"),
                     messageId: "async"
                 });
             }

package/lib/rules/no-await-in-loop.js

@@ -61,7 +61,7 @@ module.exports = {
         docs: {
             description: "Disallow `await` inside of loops",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-await-in-loop"
+            url: "https://eslint.org/docs/latest/rules/no-await-in-loop"
         },
 
         schema: [],

package/lib/rules/no-bitwise.js

@@ -28,7 +28,7 @@ module.exports = {
         docs: {
             description: "Disallow bitwise operators",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-bitwise"
+            url: "https://eslint.org/docs/latest/rules/no-bitwise"
         },
 
         schema: [

package/lib/rules/no-buffer-constructor.js

@@ -21,7 +21,7 @@ module.exports = {
         docs: {
             description: "Disallow use of the `Buffer()` constructor",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-buffer-constructor"
+            url: "https://eslint.org/docs/latest/rules/no-buffer-constructor"
         },
 
         schema: [],

package/lib/rules/no-caller.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `arguments.caller` or `arguments.callee`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-caller"
+            url: "https://eslint.org/docs/latest/rules/no-caller"
         },
 
         schema: [],

package/lib/rules/no-case-declarations.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow lexical declarations in case clauses",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-case-declarations"
+            url: "https://eslint.org/docs/latest/rules/no-case-declarations"
         },
 
         schema: [],

package/lib/rules/no-catch-shadow.js

@@ -24,7 +24,7 @@ module.exports = {
         docs: {
             description: "Disallow `catch` clause parameters from shadowing variables in the outer scope",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-catch-shadow"
+            url: "https://eslint.org/docs/latest/rules/no-catch-shadow"
         },
 
         replacedBy: ["no-shadow"],
@@ -39,7 +39,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/no-class-assign.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Disallow reassigning class members",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-class-assign"
+            url: "https://eslint.org/docs/latest/rules/no-class-assign"
         },
 
         schema: [],
@@ -31,7 +31,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Finds and reports references that are non initializer and writable.

package/lib/rules/no-compare-neg-zero.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow comparing against -0",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-compare-neg-zero"
+            url: "https://eslint.org/docs/latest/rules/no-compare-neg-zero"
         },
 
         fixable: null,

package/lib/rules/no-cond-assign.js

@@ -36,7 +36,7 @@ module.exports = {
         docs: {
             description: "Disallow assignment operators in conditional expressions",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-cond-assign"
+            url: "https://eslint.org/docs/latest/rules/no-cond-assign"
         },
 
         schema: [
@@ -57,7 +57,7 @@ module.exports = {
 
         const prohibitAssign = (context.options[0] || "except-parens");
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Check whether an AST node is the test expression for a conditional statement.

package/lib/rules/no-confusing-arrow.js

@@ -33,7 +33,7 @@ module.exports = {
         docs: {
             description: "Disallow arrow functions where they could be confused with comparisons",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-confusing-arrow"
+            url: "https://eslint.org/docs/latest/rules/no-confusing-arrow"
         },
 
         fixable: "code",
@@ -56,7 +56,7 @@ module.exports = {
         const config = context.options[0] || {};
         const allowParens = config.allowParens || (config.allowParens === void 0);
         const onlyOneSimpleParam = config.onlyOneSimpleParam;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
 
         /**

package/lib/rules/no-console.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `console`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-console"
+            url: "https://eslint.org/docs/latest/rules/no-console"
         },
 
         schema: [
@@ -51,7 +51,7 @@ module.exports = {
     create(context) {
         const options = context.options[0] || {};
         const allowed = options.allow || [];
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Checks whether the given reference is 'console' or not.