123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238 |
- /**
- * @fileoverview Prevent missing displayName in a React component definition
- * @author Yannick Croissant
- */
- 'use strict';
- const Components = require('../util/Components');
- const astUtil = require('../util/ast');
- const docsUrl = require('../util/docsUrl');
- const propsUtil = require('../util/props');
- // ------------------------------------------------------------------------------
- // Rule Definition
- // ------------------------------------------------------------------------------
- module.exports = {
- meta: {
- docs: {
- description: 'Prevent missing displayName in a React component definition',
- category: 'Best Practices',
- recommended: true,
- url: docsUrl('display-name')
- },
- messages: {
- noDisplayName: 'Component definition is missing display name'
- },
- schema: [{
- type: 'object',
- properties: {
- ignoreTranspilerName: {
- type: 'boolean'
- }
- },
- additionalProperties: false
- }]
- },
- create: Components.detect((context, components, utils) => {
- const config = context.options[0] || {};
- const ignoreTranspilerName = config.ignoreTranspilerName || false;
- /**
- * Mark a prop type as declared
- * @param {ASTNode} node The AST node being checked.
- */
- function markDisplayNameAsDeclared(node) {
- components.set(node, {
- hasDisplayName: true
- });
- }
- /**
- * Reports missing display name for a given component
- * @param {Object} component The component to process
- */
- function reportMissingDisplayName(component) {
- context.report({
- node: component.node,
- messageId: 'noDisplayName'
- });
- }
- /**
- * Checks if the component have a name set by the transpiler
- * @param {ASTNode} node The AST node being checked.
- * @returns {Boolean} True if component has a name, false if not.
- */
- function hasTranspilerName(node) {
- const namedObjectAssignment = (
- node.type === 'ObjectExpression'
- && node.parent
- && node.parent.parent
- && node.parent.parent.type === 'AssignmentExpression'
- && (
- !node.parent.parent.left.object
- || node.parent.parent.left.object.name !== 'module'
- || node.parent.parent.left.property.name !== 'exports'
- )
- );
- const namedObjectDeclaration = (
- node.type === 'ObjectExpression'
- && node.parent
- && node.parent.parent
- && node.parent.parent.type === 'VariableDeclarator'
- );
- const namedClass = (
- (node.type === 'ClassDeclaration' || node.type === 'ClassExpression')
- && node.id
- && !!node.id.name
- );
- const namedFunctionDeclaration = (
- (node.type === 'FunctionDeclaration' || node.type === 'FunctionExpression')
- && node.id
- && !!node.id.name
- );
- const namedFunctionExpression = (
- astUtil.isFunctionLikeExpression(node)
- && node.parent
- && (node.parent.type === 'VariableDeclarator' || node.parent.method === true)
- && (!node.parent.parent || !utils.isES5Component(node.parent.parent))
- );
- if (
- namedObjectAssignment || namedObjectDeclaration
- || namedClass
- || namedFunctionDeclaration || namedFunctionExpression
- ) {
- return true;
- }
- return false;
- }
- // --------------------------------------------------------------------------
- // Public
- // --------------------------------------------------------------------------
- return {
- ClassProperty(node) {
- if (!propsUtil.isDisplayNameDeclaration(node)) {
- return;
- }
- markDisplayNameAsDeclared(node);
- },
- MemberExpression(node) {
- if (!propsUtil.isDisplayNameDeclaration(node.property)) {
- return;
- }
- const component = utils.getRelatedComponent(node);
- if (!component) {
- return;
- }
- markDisplayNameAsDeclared(component.node);
- },
- FunctionExpression(node) {
- if (ignoreTranspilerName || !hasTranspilerName(node)) {
- return;
- }
- if (components.get(node)) {
- markDisplayNameAsDeclared(node);
- }
- },
- FunctionDeclaration(node) {
- if (ignoreTranspilerName || !hasTranspilerName(node)) {
- return;
- }
- if (components.get(node)) {
- markDisplayNameAsDeclared(node);
- }
- },
- ArrowFunctionExpression(node) {
- if (ignoreTranspilerName || !hasTranspilerName(node)) {
- return;
- }
- if (components.get(node)) {
- markDisplayNameAsDeclared(node);
- }
- },
- MethodDefinition(node) {
- if (!propsUtil.isDisplayNameDeclaration(node.key)) {
- return;
- }
- markDisplayNameAsDeclared(node);
- },
- ClassExpression(node) {
- if (ignoreTranspilerName || !hasTranspilerName(node)) {
- return;
- }
- markDisplayNameAsDeclared(node);
- },
- ClassDeclaration(node) {
- if (ignoreTranspilerName || !hasTranspilerName(node)) {
- return;
- }
- markDisplayNameAsDeclared(node);
- },
- ObjectExpression(node) {
- if (ignoreTranspilerName || !hasTranspilerName(node)) {
- // Search for the displayName declaration
- node.properties.forEach((property) => {
- if (!property.key || !propsUtil.isDisplayNameDeclaration(property.key)) {
- return;
- }
- markDisplayNameAsDeclared(node);
- });
- return;
- }
- markDisplayNameAsDeclared(node);
- },
- CallExpression(node) {
- if (!utils.isPragmaComponentWrapper(node)) {
- return;
- }
- if (node.arguments.length > 0 && astUtil.isFunctionLikeExpression(node.arguments[0])) {
- // Skip over React.forwardRef declarations that are embeded within
- // a React.memo i.e. React.memo(React.forwardRef(/* ... */))
- // This means that we raise a single error for the call to React.memo
- // instead of one for React.memo and one for React.forwardRef
- const isWrappedInAnotherPragma = utils.getPragmaComponentWrapper(node);
- if (
- !isWrappedInAnotherPragma
- && (ignoreTranspilerName || !hasTranspilerName(node.arguments[0]))
- ) {
- return;
- }
- if (components.get(node)) {
- markDisplayNameAsDeclared(node);
- }
- }
- },
- 'Program:exit'() {
- const list = components.list();
- // Report missing display name for all components
- Object.keys(list).filter((component) => !list[component].hasDisplayName).forEach((component) => {
- reportMissingDisplayName(list[component]);
- });
- }
- };
- })
- };
|