Add annotations

Author: y-hsgw

lib/rules/boolean-prop-naming.js

@@ -22,6 +22,8 @@ const messages = {
   patternMismatch: 'Prop name `{{propName}}` doesn’t match rule `{{pattern}}`',
 };
 
+
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {

lib/rules/button-has-type.js

@@ -28,6 +28,7 @@ const messages = {
   forbiddenValue: '"{{value}}" is an invalid value for button type attribute',
 };
 
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {
@@ -149,9 +150,11 @@ module.exports = {
         }
 
         const props = node.arguments[1].properties;
-        const typeProp = props.find((prop) => prop.key && prop.key.name === 'type');
+        const typeProp = props.find(
+          (prop) => prop.type === "Property" && prop.key && prop.key.type === "PrivateIdentifier" && prop.key.name === 'type'
+        );
 
-        if (!typeProp) {
+        if (typeProp.type !== "Property") {
           reportMissing(node);
           return;
         }

lib/rules/checked-requires-onchange-or-readonly.js

@@ -24,7 +24,7 @@ const defaultOptions = {
 };
 
 /**
- * @param {string[]} properties
+ * @param {Array} properties
  * @param {string} keyName
  * @returns {Set<string>}
  */
@@ -41,6 +41,7 @@ function extractTargetProps(properties, keyName) {
   );
 }
 
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {

lib/rules/default-props-match-prop-types.js

@@ -21,6 +21,7 @@ const messages = {
   defaultHasNoType: 'defaultProp "{{name}}" has no corresponding propTypes declaration.',
 };
 
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {

lib/rules/destructuring-assignment.js

@@ -53,6 +53,7 @@ const messages = {
   destructureInSignature: 'Must destructure props in the function signature.',
 };
 
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {

lib/rules/display-name.js

@@ -27,6 +27,7 @@ const messages = {
   noContextDisplayName: 'Context definition is missing display name',
 };
 
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {

lib/rules/forbid-component-props.js

@@ -22,6 +22,7 @@ const messages = {
   propIsForbidden: 'Prop "{{prop}}" is forbidden on Components',
 };
 
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {

lib/rules/forbid-dom-props.js

@@ -37,6 +37,7 @@ const messages = {
   propIsForbidden: 'Prop "{{prop}}" is forbidden on DOM Nodes',
 };
 
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {

lib/rules/forbid-elements.js

@@ -19,6 +19,7 @@ const messages = {
   forbiddenElement_message: '<{{element}}> is forbidden, {{message}}',
 };
 
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {
@@ -103,13 +104,11 @@ module.exports = {
           return;
         }
 
-        const argType = argument.type;
-
-        if (argType === 'Identifier' && /^[A-Z_]/.test(argument.name)) {
+        if (argument.type === 'Identifier' && /^[A-Z_]/.test(argument.name)) {
           reportIfForbidden(argument.name, argument);
-        } else if (argType === 'Literal' && /^[a-z][^.]*$/.test(argument.value)) {
+        } else if (argument.type === 'Literal' && /^[a-z][^.]*$/.test(argument.value.toString())) {
           reportIfForbidden(argument.value, argument);
-        } else if (argType === 'MemberExpression') {
+        } else if (argument.type === 'MemberExpression') {
           reportIfForbidden(context.getSourceCode().getText(argument), argument);
         }
       },

lib/rules/forbid-foreign-prop-types.js

@@ -13,6 +13,7 @@ const messages = {
   forbiddenPropType: 'Using propTypes from another component is not safe because they may be removed in production builds',
 };
 
+/** @type { import('eslint').Rule.RuleModule } */
 module.exports = {
   meta: {
     docs: {
@@ -108,7 +109,7 @@ module.exports = {
             && !ast.isAssignmentLHS(node)
             && !isAllowedAssignment(node)
           )) || (
-            (node.property.type === 'Literal' || node.property.type === 'JSXText')
+            node.property.type === 'Literal'
             && node.property.value === 'propTypes'
             && !ast.isAssignmentLHS(node)
             && !isAllowedAssignment(node)
@@ -121,7 +122,9 @@ module.exports = {
       },
 
       ObjectPattern(node) {
-        const propTypesNode = node.properties.find((property) => property.type === 'Property' && property.key.name === 'propTypes');
+        const propTypesNode = node.properties.find(
+          (property) => property.type === 'Property' && property.key.type === "PrivateIdentifier" && property.key.name === 'propTypes'
+        );
 
         if (propTypesNode) {
           report(context, messages.forbiddenPropType, 'forbiddenPropType', {