eslint 8.38.0 → 8.40.0

package/README.md

@@ -16,7 +16,8 @@
 [Report Bugs](https://eslint.org/docs/latest/contribute/report-bugs) |
 [Code of Conduct](https://eslint.org/conduct) |
 [Twitter](https://twitter.com/geteslint) |
-[Discord](https://eslint.org/chat)
+[Discord](https://eslint.org/chat) |
+[Mastodon](https://fosstodon.org/@eslint)
 
 ESLint is a tool for identifying and reporting on patterns found in ECMAScript/JavaScript code. In many ways, it is similar to JSLint and JSHint with a few exceptions:
 
@@ -248,11 +249,6 @@ Bryan Mishkin
 <img src="https://github.com/fasttime.png?s=75" width="75" height="75"><br />
 Francesco Trotta
 </a>
-</td><td align="center" valign="top" width="11%">
-<a href="https://github.com/yeonjuan">
-<img src="https://github.com/yeonjuan.png?s=75" width="75" height="75"><br />
-YeonJuan
-</a>
 </td></tr></tbody></table>
 
 ### Website Team
@@ -286,7 +282,7 @@ The following companies, organizations, and individuals support ESLint's ongoing
 <!--sponsorsstart-->
 <h3>Platinum Sponsors</h3>
 <p><a href="#"><img src="https://images.opencollective.com/2021-frameworks-fund/logo.png" alt="Chrome Frameworks Fund" height="undefined"></a> <a href="https://automattic.com"><img src="https://images.opencollective.com/automattic/d0ef3e1/logo.png" alt="Automattic" height="undefined"></a></p><h3>Gold Sponsors</h3>
-<p><a href="https://ridicorp.com/career/"><img src="https://images.opencollective.com/ridi-corporation/175dcf3/logo.png" alt="RIDI" height="96"></a> <a href="https://engineering.salesforce.com"><img src="https://images.opencollective.com/salesforce/ca8f997/logo.png" alt="Salesforce" height="96"></a> <a href="https://www.airbnb.com/"><img src="https://images.opencollective.com/airbnb/d327d66/logo.png" alt="Airbnb" height="96"></a></p><h3>Silver Sponsors</h3>
+<p><a href="https://engineering.salesforce.com"><img src="https://images.opencollective.com/salesforce/ca8f997/logo.png" alt="Salesforce" height="96"></a> <a href="https://www.airbnb.com/"><img src="https://images.opencollective.com/airbnb/d327d66/logo.png" alt="Airbnb" height="96"></a></p><h3>Silver Sponsors</h3>
 <p><a href="https://sentry.io"><img src="https://avatars.githubusercontent.com/u/1396951?v=4" alt="Sentry" height="64"></a> <a href="https://liftoff.io/"><img src="https://images.opencollective.com/liftoff/5c4fa84/logo.png" alt="Liftoff" height="64"></a> <a href="https://americanexpress.io"><img src="https://avatars.githubusercontent.com/u/3853301?v=4" alt="American Express" height="64"></a></p><h3>Bronze Sponsors</h3>
 <p><a href="https://paydaysay.com/"><img src="https://images.opencollective.com/payday-say-organization/9cd2467/logo.png" alt="PayDay Say" height="32"></a> <a href="https://themeisle.com"><img src="https://images.opencollective.com/themeisle/d5592fe/logo.png" alt="ThemeIsle" height="32"></a> <a href="https://nx.dev"><img src="https://images.opencollective.com/nx/0efbe42/logo.png" alt="Nx (by Nrwl)" height="32"></a> <a href="https://www.crosswordsolver.org/anagram-solver/"><img src="https://images.opencollective.com/anagram-solver/2666271/logo.png" alt="Anagram Solver" height="32"></a> <a href="https://icons8.com"><img src="https://images.opencollective.com/icons8/7fa1641/logo.png" alt="Icons8: free icons, photos, illustrations, and music" height="32"></a> <a href="https://discord.com"><img src="https://images.opencollective.com/discordapp/f9645d9/logo.png" alt="Discord" height="32"></a> <a href="https://transloadit.com/"><img src="https://avatars.githubusercontent.com/u/125754?v=4" alt="Transloadit" height="32"></a> <a href="https://www.ignitionapp.com"><img src="https://avatars.githubusercontent.com/u/5753491?v=4" alt="Ignition" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774?v=4" alt="HeroCoders" height="32"></a> <a href="https://quickbookstoolhub.com"><img src="https://avatars.githubusercontent.com/u/95090305?u=e5bc398ef775c9ed19f955c675cdc1fb6abf01df&v=4" alt="QuickBooks Tool hub" height="32"></a></p>
 <!--sponsorsend-->

package/lib/config/flat-config-schema.js

@@ -126,32 +126,65 @@ function normalizeRuleOptions(ruleOptions) {
 // Assertions
 //-----------------------------------------------------------------------------
 
+/**
+ * The error type when a rule's options are configured with an invalid type.
+ */
+class InvalidRuleOptionsError extends Error {
+
+    /**
+     * @param {string} ruleId Rule name being configured.
+     * @param {any} value The invalid value.
+     */
+    constructor(ruleId, value) {
+        super(`Key "${ruleId}": Expected severity of "off", 0, "warn", 1, "error", or 2.`);
+        this.messageTemplate = "invalid-rule-options";
+        this.messageData = { ruleId, value };
+    }
+}
+
 /**
  * Validates that a value is a valid rule options entry.
+ * @param {string} ruleId Rule name being configured.
  * @param {any} value The value to check.
  * @returns {void}
- * @throws {TypeError} If the value isn't a valid rule options.
+ * @throws {InvalidRuleOptionsError} If the value isn't a valid rule options.
  */
-function assertIsRuleOptions(value) {
-
+function assertIsRuleOptions(ruleId, value) {
     if (typeof value !== "string" && typeof value !== "number" && !Array.isArray(value)) {
-        throw new TypeError("Expected a string, number, or array.");
+        throw new InvalidRuleOptionsError(ruleId, value);
+    }
+}
+
+/**
+ * The error type when a rule's severity is invalid.
+ */
+class InvalidRuleSeverityError extends Error {
+
+    /**
+     * @param {string} ruleId Rule name being configured.
+     * @param {any} value The invalid value.
+     */
+    constructor(ruleId, value) {
+        super(`Key "${ruleId}": Expected severity of "off", 0, "warn", 1, "error", or 2.`);
+        this.messageTemplate = "invalid-rule-severity";
+        this.messageData = { ruleId, value };
     }
 }
 
 /**
  * Validates that a value is valid rule severity.
+ * @param {string} ruleId Rule name being configured.
  * @param {any} value The value to check.
  * @returns {void}
- * @throws {TypeError} If the value isn't a valid rule severity.
+ * @throws {InvalidRuleSeverityError} If the value isn't a valid rule severity.
  */
-function assertIsRuleSeverity(value) {
+function assertIsRuleSeverity(ruleId, value) {
     const severity = typeof value === "string"
         ? ruleSeverities.get(value.toLowerCase())
         : ruleSeverities.get(value);
 
     if (typeof severity === "undefined") {
-        throw new TypeError("Expected severity of \"off\", 0, \"warn\", 1, \"error\", or 2.");
+        throw new InvalidRuleSeverityError(ruleId, value);
     }
 }
 
@@ -357,39 +390,28 @@ const rulesSchema = {
     validate(value) {
         assertIsObject(value);
 
-        let lastRuleId;
-
-        // Performance: One try-catch has less overhead than one per loop iteration
-        try {
-
-            /*
-             * We are not checking the rule schema here because there is no
-             * guarantee that the rule definition is present at this point. Instead
-             * we wait and check the rule schema during the finalization step
-             * of calculating a config.
-             */
-            for (const ruleId of Object.keys(value)) {
-
-                // avoid hairy edge case
-                if (ruleId === "__proto__") {
-                    continue;
-                }
+        /*
+         * We are not checking the rule schema here because there is no
+         * guarantee that the rule definition is present at this point. Instead
+         * we wait and check the rule schema during the finalization step
+         * of calculating a config.
+         */
+        for (const ruleId of Object.keys(value)) {
 
-                lastRuleId = ruleId;
+            // avoid hairy edge case
+            if (ruleId === "__proto__") {
+                continue;
+            }
 
-                const ruleOptions = value[ruleId];
+            const ruleOptions = value[ruleId];
 
-                assertIsRuleOptions(ruleOptions);
+            assertIsRuleOptions(ruleId, ruleOptions);
 
-                if (Array.isArray(ruleOptions)) {
-                    assertIsRuleSeverity(ruleOptions[0]);
-                } else {
-                    assertIsRuleSeverity(ruleOptions);
-                }
+            if (Array.isArray(ruleOptions)) {
+                assertIsRuleSeverity(ruleId, ruleOptions[0]);
+            } else {
+                assertIsRuleSeverity(ruleId, ruleOptions);
             }
-        } catch (error) {
-            error.message = `Key "${lastRuleId}": ${error.message}`;
-            throw error;
         }
     }
 };

package/lib/eslint/flat-eslint.js

@@ -320,6 +320,45 @@ async function loadFlatConfigFile(filePath) {
     return config;
 }
 
+/**
+ * Determines which config file to use. This is determined by seeing if an
+ * override config file was passed, and if so, using it; otherwise, as long
+ * as override config file is not explicitly set to `false`, it will search
+ * upwards from the cwd for a file named `eslint.config.js`.
+ * @param {import("./eslint").ESLintOptions} options The ESLint instance options.
+ * @returns {{configFilePath:string,basePath:string,error:boolean}} Location information for
+ *      the config file.
+ */
+async function locateConfigFileToUse({ configFile, cwd }) {
+
+    // determine where to load config file from
+    let configFilePath;
+    let basePath = cwd;
+    let error = false;
+
+    if (typeof configFile === "string") {
+        debug(`Override config file path is ${configFile}`);
+        configFilePath = path.resolve(cwd, configFile);
+    } else if (configFile !== false) {
+        debug("Searching for eslint.config.js");
+        configFilePath = await findFlatConfigFile(cwd);
+
+        if (configFilePath) {
+            basePath = path.resolve(path.dirname(configFilePath));
+        } else {
+            error = true;
+        }
+
+    }
+
+    return {
+        configFilePath,
+        basePath,
+        error
+    };
+
+}
+
 /**
  * Calculates the config array for this run based on inputs.
  * @param {FlatESLint} eslint The instance to create the config array for.
@@ -342,25 +381,13 @@ async function calculateConfigArray(eslint, {
         return slots.configs;
     }
 
-    // determine where to load config file from
-    let configFilePath;
-    let basePath = cwd;
+    const { configFilePath, basePath, error } = await locateConfigFileToUse({ configFile, cwd });
 
-    if (typeof configFile === "string") {
-        debug(`Override config file path is ${configFile}`);
-        configFilePath = path.resolve(cwd, configFile);
-    } else if (configFile !== false) {
-        debug("Searching for eslint.config.js");
-        configFilePath = await findFlatConfigFile(cwd);
-
-        if (!configFilePath) {
-            throw new Error("Could not find config file.");
-        }
-
-        basePath = path.resolve(path.dirname(configFilePath));
+    // config file is required to calculate config
+    if (error) {
+        throw new Error("Could not find config file.");
     }
 
-
     const configs = new FlatConfigArray(baseConfig || [], { basePath, shouldIgnore });
 
     // load config file
@@ -1160,6 +1187,19 @@ class FlatESLint {
         return configs.getConfig(absolutePath);
     }
 
+    /**
+     * Finds the config file being used by this instance based on the options
+     * passed to the constructor.
+     * @returns {string|undefined} The path to the config file being used or
+     *      `undefined` if no config file is being used.
+     */
+    async findConfigFile() {
+        const options = privateMembers.get(this).options;
+        const { configFilePath } = await locateConfigFileToUse(options);
+
+        return configFilePath;
+    }
+
     /**
      * Checks if a given path is ignored by ESLint.
      * @param {string} filePath The path of the file to check.

package/lib/linter/linter.js

@@ -592,7 +592,7 @@ function findEslintEnv(text) {
  * Convert "/path/to/<text>" to "<text>".
  * `CLIEngine#executeOnText()` method gives "/path/to/<text>" if the filename
  * was omitted because `configArray.extractConfig()` requires an absolute path.
- * But the linter should pass `<text>` to `RuleContext#getFilename()` in that
+ * But the linter should pass `<text>` to `RuleContext#filename` in that
  * case.
  * Also, code blocks can have their virtual filename. If the parent filename was
  * `<text>`, the virtual filename is `<text>/0_foo.js` or something like (i.e.,
@@ -857,38 +857,6 @@ function parse(text, languageOptions, filePath) {
     }
 }
 
-/**
- * Marks a variable as used in the current scope
- * @param {SourceCode} sourceCode The source code for the currently linted file.
- * @param {ASTNode} currentNode The node currently being traversed
- * @param {LanguageOptions} languageOptions The options used to parse this text
- * @param {string} name The name of the variable that should be marked as used.
- * @returns {boolean} True if the variable was found and marked as used, false if not.
- */
-function markVariableAsUsed(sourceCode, currentNode, languageOptions, name) {
-    const parserOptions = languageOptions.parserOptions;
-    const sourceType = languageOptions.sourceType;
-    const hasGlobalReturn =
-        (parserOptions.ecmaFeatures && parserOptions.ecmaFeatures.globalReturn) ||
-        sourceType === "commonjs";
-    const specialScope = hasGlobalReturn || sourceType === "module";
-    const currentScope = sourceCode.getScope(currentNode);
-
-    // Special Node.js scope means we need to start one level deeper
-    const initialScope = currentScope.type === "global" && specialScope ? currentScope.childScopes[0] : currentScope;
-
-    for (let scope = initialScope; scope; scope = scope.upper) {
-        const variable = scope.variables.find(scopeVar => scopeVar.name === name);
-
-        if (variable) {
-            variable.eslintUsed = true;
-            return true;
-        }
-    }
-
-    return false;
-}
-
 /**
  * Runs a rule, and gets its listeners
  * @param {Rule} rule A normalized rule with a `create` method
@@ -934,7 +902,7 @@ const BASE_TRAVERSAL_CONTEXT = Object.freeze(
         (contextInfo, methodName) =>
             Object.assign(contextInfo, {
                 [methodName](...args) {
-                    return this.getSourceCode()[DEPRECATED_SOURCECODE_PASSTHROUGHS[methodName]](...args);
+                    return this.sourceCode[DEPRECATED_SOURCECODE_PASSTHROUGHS[methodName]](...args);
                 }
             }),
         {}
@@ -983,11 +951,15 @@ function runRules(sourceCode, configuredRules, ruleMapper, parserName, languageO
                 getAncestors: () => sourceCode.getAncestors(currentNode),
                 getDeclaredVariables: node => sourceCode.getDeclaredVariables(node),
                 getCwd: () => cwd,
+                cwd,
                 getFilename: () => filename,
+                filename,
                 getPhysicalFilename: () => physicalFilename || filename,
+                physicalFilename: physicalFilename || filename,
                 getScope: () => sourceCode.getScope(currentNode),
                 getSourceCode: () => sourceCode,
-                markVariableAsUsed: name => markVariableAsUsed(sourceCode, currentNode, languageOptions, name),
+                sourceCode,
+                markVariableAsUsed: name => sourceCode.markVariableAsUsed(name, currentNode),
                 parserOptions: {
                     ...languageOptions.parserOptions
                 },

package/lib/rules/accessor-pairs.js

@@ -142,7 +142,7 @@ module.exports = {
         docs: {
             description: "Enforce getter and setter pairs in objects and classes",
             recommended: false,
-            url: "https://eslint.org/docs/rules/accessor-pairs"
+            url: "https://eslint.org/docs/latest/rules/accessor-pairs"
         },
 
         schema: [{
@@ -178,7 +178,7 @@ module.exports = {
         const checkGetWithoutSet = config.getWithoutSet === true;
         const checkSetWithoutGet = config.setWithoutGet !== false;
         const enforceForClassMembers = config.enforceForClassMembers !== false;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Reports the given node.

package/lib/rules/array-bracket-newline.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Enforce linebreaks after opening and before closing array brackets",
             recommended: false,
-            url: "https://eslint.org/docs/rules/array-bracket-newline"
+            url: "https://eslint.org/docs/latest/rules/array-bracket-newline"
         },
 
         fixable: "whitespace",
@@ -56,7 +56,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
 
         //----------------------------------------------------------------------

package/lib/rules/array-bracket-spacing.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent spacing inside array brackets",
             recommended: false,
-            url: "https://eslint.org/docs/rules/array-bracket-spacing"
+            url: "https://eslint.org/docs/latest/rules/array-bracket-spacing"
         },
 
         fixable: "whitespace",
@@ -53,7 +53,7 @@ module.exports = {
     },
     create(context) {
         const spaced = context.options[0] === "always",
-            sourceCode = context.getSourceCode();
+            sourceCode = context.sourceCode;
 
         /**
          * Determines whether an option is set, relative to the spacing option.

package/lib/rules/array-callback-return.js

@@ -141,7 +141,7 @@ module.exports = {
         docs: {
             description: "Enforce `return` statements in callbacks of array methods",
             recommended: false,
-            url: "https://eslint.org/docs/rules/array-callback-return"
+            url: "https://eslint.org/docs/latest/rules/array-callback-return"
         },
 
         schema: [
@@ -172,7 +172,7 @@ module.exports = {
     create(context) {
 
         const options = context.options[0] || { allowImplicit: false, checkForEach: false };
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         let funcInfo = {
             arrayMethodName: null,

package/lib/rules/array-element-newline.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Enforce line breaks after each array element",
             recommended: false,
-            url: "https://eslint.org/docs/rules/array-element-newline"
+            url: "https://eslint.org/docs/latest/rules/array-element-newline"
         },
 
         fixable: "whitespace",
@@ -79,7 +79,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //----------------------------------------------------------------------
         // Helpers

package/lib/rules/arrow-body-style.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Require braces around arrow function bodies",
             recommended: false,
-            url: "https://eslint.org/docs/rules/arrow-body-style"
+            url: "https://eslint.org/docs/latest/rules/arrow-body-style"
         },
 
         schema: {
@@ -74,7 +74,7 @@ module.exports = {
         const asNeeded = !options[0] || options[0] === "as-needed";
         const never = options[0] === "never";
         const requireReturnForObjectLiteral = options[1] && options[1].requireReturnForObjectLiteral;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         let funcInfo = null;
 
         /**

package/lib/rules/arrow-parens.js

@@ -35,7 +35,7 @@ module.exports = {
         docs: {
             description: "Require parentheses around arrow function arguments",
             recommended: false,
-            url: "https://eslint.org/docs/rules/arrow-parens"
+            url: "https://eslint.org/docs/latest/rules/arrow-parens"
         },
 
         fixable: "code",
@@ -69,7 +69,7 @@ module.exports = {
         const asNeeded = context.options[0] === "as-needed";
         const requireForBlockBody = asNeeded && context.options[1] && context.options[1].requireForBlockBody === true;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Finds opening paren of parameters for the given arrow function, if it exists.

package/lib/rules/arrow-spacing.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent spacing before and after the arrow in arrow functions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/arrow-spacing"
+            url: "https://eslint.org/docs/latest/rules/arrow-spacing"
         },
 
         fixable: "whitespace",
@@ -61,7 +61,7 @@ module.exports = {
         rule.before = rule.before !== false;
         rule.after = rule.after !== false;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Get tokens of arrow(`=>`) and before/after arrow.

package/lib/rules/block-scoped-var.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Enforce the use of variables within the scope they are defined",
             recommended: false,
-            url: "https://eslint.org/docs/rules/block-scoped-var"
+            url: "https://eslint.org/docs/latest/rules/block-scoped-var"
         },
 
         schema: [],
@@ -28,7 +28,7 @@ module.exports = {
 
     create(context) {
         let stack = [];
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Makes a block scope.

package/lib/rules/block-spacing.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Disallow or enforce spaces inside of blocks after opening block and before closing block",
             recommended: false,
-            url: "https://eslint.org/docs/rules/block-spacing"
+            url: "https://eslint.org/docs/latest/rules/block-spacing"
         },
 
         fixable: "whitespace",
@@ -37,7 +37,7 @@ module.exports = {
     create(context) {
         const always = (context.options[0] !== "never"),
             messageId = always ? "missing" : "extra",
-            sourceCode = context.getSourceCode();
+            sourceCode = context.sourceCode;
 
         /**
          * Gets the open brace token from a given node.

package/lib/rules/brace-style.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent brace style for blocks",
             recommended: false,
-            url: "https://eslint.org/docs/rules/brace-style"
+            url: "https://eslint.org/docs/latest/rules/brace-style"
         },
 
         schema: [
@@ -53,7 +53,7 @@ module.exports = {
     create(context) {
         const style = context.options[0] || "1tbs",
             params = context.options[1] || {},
-            sourceCode = context.getSourceCode();
+            sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/callback-return.js

@@ -21,7 +21,7 @@ module.exports = {
         docs: {
             description: "Require `return` statements after callbacks",
             recommended: false,
-            url: "https://eslint.org/docs/rules/callback-return"
+            url: "https://eslint.org/docs/latest/rules/callback-return"
         },
 
         schema: [{
@@ -37,7 +37,7 @@ module.exports = {
     create(context) {
 
         const callbacks = context.options[0] || ["callback", "cb", "next"],
-            sourceCode = context.getSourceCode();
+            sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/camelcase.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Enforce camelcase naming convention",
             recommended: false,
-            url: "https://eslint.org/docs/rules/camelcase"
+            url: "https://eslint.org/docs/latest/rules/camelcase"
         },
 
         schema: [
@@ -73,7 +73,7 @@ module.exports = {
         const ignoreImports = options.ignoreImports;
         const ignoreGlobals = options.ignoreGlobals;
         const allow = options.allow || [];
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/capitalized-comments.js

@@ -107,7 +107,7 @@ module.exports = {
         docs: {
             description: "Enforce or disallow capitalization of the first letter of a comment",
             recommended: false,
-            url: "https://eslint.org/docs/rules/capitalized-comments"
+            url: "https://eslint.org/docs/latest/rules/capitalized-comments"
         },
 
         fixable: "code",
@@ -139,7 +139,7 @@ module.exports = {
 
         const capitalize = context.options[0] || "always",
             normalizedOptions = getAllNormalizedOptions(context.options[1]),
-            sourceCode = context.getSourceCode();
+            sourceCode = context.sourceCode;
 
         createRegExpForIgnorePatterns(normalizedOptions);
 

package/lib/rules/class-methods-use-this.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Enforce that class methods utilize `this`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/class-methods-use-this"
+            url: "https://eslint.org/docs/latest/rules/class-methods-use-this"
         },
 
         schema: [{
@@ -133,7 +133,7 @@ module.exports = {
             if (isIncludedInstanceMethod(node.parent) && !methodUsesThis) {
                 context.report({
                     node,
-                    loc: astUtils.getFunctionHeadLoc(node, context.getSourceCode()),
+                    loc: astUtils.getFunctionHeadLoc(node, context.sourceCode),
                     messageId: "missingThis",
                     data: {
                         name: astUtils.getFunctionNameWithKind(node)

package/lib/rules/comma-dangle.js

@@ -78,7 +78,7 @@ module.exports = {
         docs: {
             description: "Require or disallow trailing commas",
             recommended: false,
-            url: "https://eslint.org/docs/rules/comma-dangle"
+            url: "https://eslint.org/docs/latest/rules/comma-dangle"
         },
 
         fixable: "code",
@@ -136,7 +136,7 @@ module.exports = {
     create(context) {
         const options = normalizeOptions(context.options[0], context.languageOptions.ecmaVersion);
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Gets the last item of the given node.

package/lib/rules/comma-spacing.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent spacing before and after commas",
             recommended: false,
-            url: "https://eslint.org/docs/rules/comma-spacing"
+            url: "https://eslint.org/docs/latest/rules/comma-spacing"
         },
 
         fixable: "whitespace",
@@ -48,7 +48,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const tokensAndComments = sourceCode.tokensAndComments;
 
         const options = {