232 lines
6.7 KiB
JavaScript
232 lines
6.7 KiB
JavaScript
|
|
/**
|
|||
|
|
* @fileoverview Enforce React components to have a shouldComponentUpdate method
|
|||
|
|
* @author Evgueni Naverniouk
|
|||
|
|
*/
|
|||
|
|
'use strict';
|
|||
|
|
|
|||
|
|
var has = require('has');
|
|||
|
|
var Components = require('../util/Components');
|
|||
|
|
|
|||
|
|
module.exports = {
|
|||
|
|
meta: {
|
|||
|
|
docs: {
|
|||
|
|
description: 'Enforce React components to have a shouldComponentUpdate method',
|
|||
|
|
category: 'Best Practices',
|
|||
|
|
recommended: false
|
|||
|
|
},
|
|||
|
|
|
|||
|
|
schema: [{
|
|||
|
|
type: 'object',
|
|||
|
|
properties: {
|
|||
|
|
allowDecorators: {
|
|||
|
|
type: 'array',
|
|||
|
|
items: {
|
|||
|
|
type: 'string'
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
},
|
|||
|
|
additionalProperties: false
|
|||
|
|
}]
|
|||
|
|
},
|
|||
|
|
|
|||
|
|
create: Components.detect(function (context, components, utils) {
|
|||
|
|
var MISSING_MESSAGE = 'Component is not optimized. Please add a shouldComponentUpdate method.';
|
|||
|
|
var configuration = context.options[0] || {};
|
|||
|
|
var allowDecorators = configuration.allowDecorators || [];
|
|||
|
|
|
|||
|
|
/**
|
|||
|
|
* Checks to see if our component is decorated by PureRenderMixin via reactMixin
|
|||
|
|
* @param {ASTNode} node The AST node being checked.
|
|||
|
|
* @returns {Boolean} True if node is decorated with a PureRenderMixin, false if not.
|
|||
|
|
*/
|
|||
|
|
var hasPureRenderDecorator = function (node) {
|
|||
|
|
if (node.decorators && node.decorators.length) {
|
|||
|
|
for (var i = 0, l = node.decorators.length; i < l; i++) {
|
|||
|
|
if (
|
|||
|
|
node.decorators[i].expression &&
|
|||
|
|
node.decorators[i].expression.callee &&
|
|||
|
|
node.decorators[i].expression.callee.object &&
|
|||
|
|
node.decorators[i].expression.callee.object.name === 'reactMixin' &&
|
|||
|
|
node.decorators[i].expression.callee.property &&
|
|||
|
|
node.decorators[i].expression.callee.property.name === 'decorate' &&
|
|||
|
|
node.decorators[i].expression.arguments &&
|
|||
|
|
node.decorators[i].expression.arguments.length &&
|
|||
|
|
node.decorators[i].expression.arguments[0].name === 'PureRenderMixin'
|
|||
|
|
) {
|
|||
|
|
return true;
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
|
|||
|
|
return false;
|
|||
|
|
};
|
|||
|
|
|
|||
|
|
/**
|
|||
|
|
* Checks to see if our component is custom decorated
|
|||
|
|
* @param {ASTNode} node The AST node being checked.
|
|||
|
|
* @returns {Boolean} True if node is decorated name with a custom decorated, false if not.
|
|||
|
|
*/
|
|||
|
|
var hasCustomDecorator = function (node) {
|
|||
|
|
var allowLength = allowDecorators.length;
|
|||
|
|
|
|||
|
|
if (allowLength && node.decorators && node.decorators.length) {
|
|||
|
|
for (var i = 0; i < allowLength; i++) {
|
|||
|
|
for (var j = 0, l = node.decorators.length; j < l; j++) {
|
|||
|
|
if (
|
|||
|
|
node.decorators[j].expression &&
|
|||
|
|
node.decorators[j].expression.name === allowDecorators[i]
|
|||
|
|
) {
|
|||
|
|
return true;
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
|
|||
|
|
return false;
|
|||
|
|
};
|
|||
|
|
|
|||
|
|
/**
|
|||
|
|
* Checks if we are declaring a shouldComponentUpdate method
|
|||
|
|
* @param {ASTNode} node The AST node being checked.
|
|||
|
|
* @returns {Boolean} True if we are declaring a shouldComponentUpdate method, false if not.
|
|||
|
|
*/
|
|||
|
|
var isSCUDeclarеd = function (node) {
|
|||
|
|
return Boolean(
|
|||
|
|
node &&
|
|||
|
|
node.name === 'shouldComponentUpdate'
|
|||
|
|
);
|
|||
|
|
};
|
|||
|
|
|
|||
|
|
/**
|
|||
|
|
* Checks if we are declaring a PureRenderMixin mixin
|
|||
|
|
* @param {ASTNode} node The AST node being checked.
|
|||
|
|
* @returns {Boolean} True if we are declaring a PureRenderMixin method, false if not.
|
|||
|
|
*/
|
|||
|
|
var isPureRenderDeclared = function (node) {
|
|||
|
|
var hasPR = false;
|
|||
|
|
if (node.value && node.value.elements) {
|
|||
|
|
for (var i = 0, l = node.value.elements.length; i < l; i++) {
|
|||
|
|
if (node.value.elements[i].name === 'PureRenderMixin') {
|
|||
|
|
hasPR = true;
|
|||
|
|
break;
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
|
|||
|
|
return Boolean(
|
|||
|
|
node &&
|
|||
|
|
node.key.name === 'mixins' &&
|
|||
|
|
hasPR
|
|||
|
|
);
|
|||
|
|
};
|
|||
|
|
|
|||
|
|
/**
|
|||
|
|
* Mark shouldComponentUpdate as declared
|
|||
|
|
* @param {ASTNode} node The AST node being checked.
|
|||
|
|
*/
|
|||
|
|
var markSCUAsDeclared = function (node) {
|
|||
|
|
components.set(node, {
|
|||
|
|
hasSCU: true
|
|||
|
|
});
|
|||
|
|
};
|
|||
|
|
|
|||
|
|
/**
|
|||
|
|
* Reports missing optimization for a given component
|
|||
|
|
* @param {Object} component The component to process
|
|||
|
|
*/
|
|||
|
|
var reportMissingOptimization = function (component) {
|
|||
|
|
context.report({
|
|||
|
|
node: component.node,
|
|||
|
|
message: MISSING_MESSAGE,
|
|||
|
|
data: {
|
|||
|
|
component: component.name
|
|||
|
|
}
|
|||
|
|
});
|
|||
|
|
};
|
|||
|
|
|
|||
|
|
/**
|
|||
|
|
* Checks if we are declaring function in class
|
|||
|
|
* @returns {Boolean} True if we are declaring function in class, false if not.
|
|||
|
|
*/
|
|||
|
|
var isFunctionInClass = function () {
|
|||
|
|
var blockNode;
|
|||
|
|
var scope = context.getScope();
|
|||
|
|
while (scope) {
|
|||
|
|
blockNode = scope.block;
|
|||
|
|
if (blockNode && blockNode.type === 'ClassDeclaration') {
|
|||
|
|
return true;
|
|||
|
|
}
|
|||
|
|
scope = scope.upper;
|
|||
|
|
}
|
|||
|
|
|
|||
|
|
return false;
|
|||
|
|
};
|
|||
|
|
|
|||
|
|
return {
|
|||
|
|
ArrowFunctionExpression: function (node) {
|
|||
|
|
// Stateless Functional Components cannot be optimized (yet)
|
|||
|
|
markSCUAsDeclared(node);
|
|||
|
|
},
|
|||
|
|
|
|||
|
|
ClassDeclaration: function (node) {
|
|||
|
|
if (!(hasPureRenderDecorator(node) || hasCustomDecorator(node) || utils.isPureComponent(node))) {
|
|||
|
|
return;
|
|||
|
|
}
|
|||
|
|
markSCUAsDeclared(node);
|
|||
|
|
},
|
|||
|
|
|
|||
|
|
FunctionDeclaration: function (node) {
|
|||
|
|
// Skip if the function is declared in the class
|
|||
|
|
if (isFunctionInClass()) {
|
|||
|
|
return;
|
|||
|
|
}
|
|||
|
|
// Stateless Functional Components cannot be optimized (yet)
|
|||
|
|
markSCUAsDeclared(node);
|
|||
|
|
},
|
|||
|
|
|
|||
|
|
FunctionExpression: function (node) {
|
|||
|
|
// Skip if the function is declared in the class
|
|||
|
|
if (isFunctionInClass()) {
|
|||
|
|
return;
|
|||
|
|
}
|
|||
|
|
// Stateless Functional Components cannot be optimized (yet)
|
|||
|
|
markSCUAsDeclared(node);
|
|||
|
|
},
|
|||
|
|
|
|||
|
|
MethodDefinition: function (node) {
|
|||
|
|
if (!isSCUDeclarеd(node.key)) {
|
|||
|
|
return;
|
|||
|
|
}
|
|||
|
|
markSCUAsDeclared(node);
|
|||
|
|
},
|
|||
|
|
|
|||
|
|
ObjectExpression: function (node) {
|
|||
|
|
// Search for the shouldComponentUpdate declaration
|
|||
|
|
for (var i = 0, l = node.properties.length; i < l; i++) {
|
|||
|
|
if (
|
|||
|
|
!node.properties[i].key || (
|
|||
|
|
!isSCUDeclarеd(node.properties[i].key) &&
|
|||
|
|
!isPureRenderDeclared(node.properties[i])
|
|||
|
|
)
|
|||
|
|
) {
|
|||
|
|
continue;
|
|||
|
|
}
|
|||
|
|
markSCUAsDeclared(node);
|
|||
|
|
}
|
|||
|
|
},
|
|||
|
|
|
|||
|
|
'Program:exit': function () {
|
|||
|
|
var list = components.list();
|
|||
|
|
|
|||
|
|
// Report missing shouldComponentUpdate for all components
|
|||
|
|
for (var component in list) {
|
|||
|
|
if (!has(list, component) || list[component].hasSCU) {
|
|||
|
|
continue;
|
|||
|
|
}
|
|||
|
|
reportMissingOptimization(list[component]);
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
};
|
|||
|
|
})
|
|||
|
|
};
|