You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
187 lines
6.0 KiB
187 lines
6.0 KiB
/**
|
|
* @fileoverview Rule to enforce that all class methods use 'this'.
|
|
* @author Patrick Williams
|
|
*/
|
|
|
|
"use strict";
|
|
|
|
//------------------------------------------------------------------------------
|
|
// Requirements
|
|
//------------------------------------------------------------------------------
|
|
|
|
const astUtils = require("./utils/ast-utils");
|
|
|
|
//------------------------------------------------------------------------------
|
|
// Rule Definition
|
|
//------------------------------------------------------------------------------
|
|
|
|
/** @type {import('../shared/types').Rule} */
|
|
module.exports = {
|
|
meta: {
|
|
type: "suggestion",
|
|
|
|
docs: {
|
|
description: "Enforce that class methods utilize `this`",
|
|
recommended: false,
|
|
url: "https://eslint.org/docs/rules/class-methods-use-this"
|
|
},
|
|
|
|
schema: [{
|
|
type: "object",
|
|
properties: {
|
|
exceptMethods: {
|
|
type: "array",
|
|
items: {
|
|
type: "string"
|
|
}
|
|
},
|
|
enforceForClassFields: {
|
|
type: "boolean",
|
|
default: true
|
|
}
|
|
},
|
|
additionalProperties: false
|
|
}],
|
|
|
|
messages: {
|
|
missingThis: "Expected 'this' to be used by class {{name}}."
|
|
}
|
|
},
|
|
create(context) {
|
|
const config = Object.assign({}, context.options[0]);
|
|
const enforceForClassFields = config.enforceForClassFields !== false;
|
|
const exceptMethods = new Set(config.exceptMethods || []);
|
|
|
|
const stack = [];
|
|
|
|
/**
|
|
* Push `this` used flag initialized with `false` onto the stack.
|
|
* @returns {void}
|
|
*/
|
|
function pushContext() {
|
|
stack.push(false);
|
|
}
|
|
|
|
/**
|
|
* Pop `this` used flag from the stack.
|
|
* @returns {boolean | undefined} `this` used flag
|
|
*/
|
|
function popContext() {
|
|
return stack.pop();
|
|
}
|
|
|
|
/**
|
|
* Initializes the current context to false and pushes it onto the stack.
|
|
* These booleans represent whether 'this' has been used in the context.
|
|
* @returns {void}
|
|
* @private
|
|
*/
|
|
function enterFunction() {
|
|
pushContext();
|
|
}
|
|
|
|
/**
|
|
* Check if the node is an instance method
|
|
* @param {ASTNode} node node to check
|
|
* @returns {boolean} True if its an instance method
|
|
* @private
|
|
*/
|
|
function isInstanceMethod(node) {
|
|
switch (node.type) {
|
|
case "MethodDefinition":
|
|
return !node.static && node.kind !== "constructor";
|
|
case "PropertyDefinition":
|
|
return !node.static && enforceForClassFields;
|
|
default:
|
|
return false;
|
|
}
|
|
}
|
|
|
|
/**
|
|
* Check if the node is an instance method not excluded by config
|
|
* @param {ASTNode} node node to check
|
|
* @returns {boolean} True if it is an instance method, and not excluded by config
|
|
* @private
|
|
*/
|
|
function isIncludedInstanceMethod(node) {
|
|
if (isInstanceMethod(node)) {
|
|
if (node.computed) {
|
|
return true;
|
|
}
|
|
|
|
const hashIfNeeded = node.key.type === "PrivateIdentifier" ? "#" : "";
|
|
const name = node.key.type === "Literal"
|
|
? astUtils.getStaticStringValue(node.key)
|
|
: (node.key.name || "");
|
|
|
|
return !exceptMethods.has(hashIfNeeded + name);
|
|
}
|
|
return false;
|
|
}
|
|
|
|
/**
|
|
* Checks if we are leaving a function that is a method, and reports if 'this' has not been used.
|
|
* Static methods and the constructor are exempt.
|
|
* Then pops the context off the stack.
|
|
* @param {ASTNode} node A function node that was entered.
|
|
* @returns {void}
|
|
* @private
|
|
*/
|
|
function exitFunction(node) {
|
|
const methodUsesThis = popContext();
|
|
|
|
if (isIncludedInstanceMethod(node.parent) && !methodUsesThis) {
|
|
context.report({
|
|
node,
|
|
loc: astUtils.getFunctionHeadLoc(node, context.getSourceCode()),
|
|
messageId: "missingThis",
|
|
data: {
|
|
name: astUtils.getFunctionNameWithKind(node)
|
|
}
|
|
});
|
|
}
|
|
}
|
|
|
|
/**
|
|
* Mark the current context as having used 'this'.
|
|
* @returns {void}
|
|
* @private
|
|
*/
|
|
function markThisUsed() {
|
|
if (stack.length) {
|
|
stack[stack.length - 1] = true;
|
|
}
|
|
}
|
|
|
|
return {
|
|
FunctionDeclaration: enterFunction,
|
|
"FunctionDeclaration:exit": exitFunction,
|
|
FunctionExpression: enterFunction,
|
|
"FunctionExpression:exit": exitFunction,
|
|
|
|
/*
|
|
* Class field value are implicit functions.
|
|
*/
|
|
"PropertyDefinition > *.key:exit": pushContext,
|
|
"PropertyDefinition:exit": popContext,
|
|
|
|
/*
|
|
* Class static blocks are implicit functions. They aren't required to use `this`,
|
|
* but we have to push context so that it captures any use of `this` in the static block
|
|
* separately from enclosing contexts, because static blocks have their own `this` and it
|
|
* shouldn't count as used `this` in enclosing contexts.
|
|
*/
|
|
StaticBlock: pushContext,
|
|
"StaticBlock:exit": popContext,
|
|
|
|
ThisExpression: markThisUsed,
|
|
Super: markThisUsed,
|
|
...(
|
|
enforceForClassFields && {
|
|
"PropertyDefinition > ArrowFunctionExpression.value": enterFunction,
|
|
"PropertyDefinition > ArrowFunctionExpression.value:exit": exitFunction
|
|
}
|
|
)
|
|
};
|
|
}
|
|
};
|
|
|