[Build]: fix type-check errors

Author: golopot

lib/rules/destructuring-assignment.js

@@ -127,7 +127,7 @@ module.exports = {
           handleSFCUsage(node);
         }
         if (classComponent) {
-          handleClassUsage(node, classComponent);
+          handleClassUsage(node);
         }
       },
 

lib/rules/jsx-closing-bracket-location.js

@@ -198,6 +198,7 @@ module.exports = {
       }
       const openingLine = sourceCode.lines[opening.line - 1];
       const openingStartOfLine = {
+        // @ts-ignore
         column: /^\s*/.exec(openingLine)[0].length,
         line: opening.line
       };
@@ -274,7 +275,7 @@ module.exports = {
               case 'tag-aligned':
               case 'line-aligned':
                 return fixer.replaceTextRange([cachedLastAttributeEndPos, node.range[1]],
-                  `\n${getIndentation(tokens, expectedLocation, correctColumn)}${closingTag}`);
+                  `\n${getIndentation(tokens, expectedLocation, correctColumn || 0)}${closingTag}`);
               default:
                 return true;
             }

lib/rules/jsx-curly-brace-presence.js

@@ -110,7 +110,7 @@ module.exports = {
 
     /**
      * Report and fix an unnecessary curly brace violation on a node
-     * @param {ASTNode} node - The AST node with an unnecessary JSX expression
+     * @param {ASTNode} JSXExpressionNode - The AST node with an unnecessary JSX expression
      */
     function reportUnnecessaryCurly(JSXExpressionNode) {
       context.report({

lib/rules/jsx-curly-spacing.js

@@ -156,8 +156,8 @@ module.exports = {
     /**
      * Trims text of whitespace between two ranges
      * @param {Fixer} fixer - the eslint fixer object
-     * @param {Location} fromLoc - the start location
-     * @param {Location} toLoc - the end location
+     * @param {number} fromLoc - the start location
+     * @param {number} toLoc - the end location
      * @param {string} mode - either 'start' or 'end'
      * @param {string=} spacing - a spacing value that will optionally add a space to the removed text
      * @returns {Object|*|{range, text}}
@@ -183,6 +183,7 @@ module.exports = {
     * Reports that there shouldn't be a newline after the first token
     * @param {ASTNode} node - The node to report in the event of an error.
     * @param {Token} token - The token to use for the report.
+    * @param {string} spacing
     * @returns {void}
     */
     function reportNoBeginningNewline(node, token, spacing) {
@@ -201,6 +202,7 @@ module.exports = {
     * Reports that there shouldn't be a newline before the last token
     * @param {ASTNode} node - The node to report in the event of an error.
     * @param {Token} token - The token to use for the report.
+    * @param {string} spacing
     * @returns {void}
     */
     function reportNoEndingNewline(node, token, spacing) {

lib/rules/jsx-indent-props.js

@@ -59,6 +59,7 @@ module.exports = {
 
     const extraColumnStart = 0;
     let indentType = 'space';
+    /** @type {number|'first'} */
     let indentSize = 4;
 
     const sourceCode = context.getSourceCode();
@@ -126,7 +127,6 @@ module.exports = {
      * Check indent for nodes list
      * @param {ASTNode[]} nodes list of node objects
      * @param {Number} indent needed indent
-     * @param {Boolean} excludeCommas skip comma on start of line
      */
     function checkNodesIndent(nodes, indent) {
       nodes.forEach(node => {

lib/rules/jsx-indent.js

@@ -110,7 +110,7 @@ module.exports = {
      * @param {ASTNode} node Node violating the indent rule
      * @param {Number} needed Expected indentation character count
      * @param {Number} gotten Indentation character count in the actual node/code
-     * @param {Object} loc Error line and column location
+     * @param {Object} [loc] Error line and column location
      */
     function report(node, needed, gotten, loc) {
       const msgContext = {
@@ -141,8 +141,8 @@ module.exports = {
     /**
      * Get node indent
      * @param {ASTNode} node Node to examine
-     * @param {Boolean} byLastLine get indent of node's last line
-     * @param {Boolean} excludeCommas skip comma on start of line
+     * @param {Boolean} [byLastLine] get indent of node's last line
+     * @param {Boolean} [excludeCommas] skip comma on start of line
      * @return {Number} Indent
      */
     function getNodeIndent(node, byLastLine, excludeCommas) {
@@ -204,7 +204,7 @@ module.exports = {
      * Check indent for nodes list
      * @param {ASTNode} node The node to check
      * @param {Number} indent needed indent
-     * @param {Boolean} excludeCommas skip comma on start of line
+     * @param {Boolean} [excludeCommas] skip comma on start of line
      */
     function checkNodesIndent(node, indent, excludeCommas) {
       const nodeIndent = getNodeIndent(node, false, excludeCommas);

lib/rules/jsx-sort-props.js

@@ -75,7 +75,7 @@ function contextCompare(a, b, options) {
  * Create an array of arrays where each subarray is composed of attributes
  * that are considered sortable.
  * @param {Array<JSXSpreadAttribute|JSXAttribute>} attributes
- * @return {Array<Array<JSXAttribute>}
+ * @return {Array<Array<JSXAttribute>>}
  */
 function getGroupsOfSortableAttributes(attributes) {
   const sortableAttributeGroups = [];
@@ -156,7 +156,7 @@ const generateFixerFunction = (node, context, reservedList) => {
  * Checks if the `reservedFirst` option is valid
  * @param {Object} context The context of the rule
  * @param {Boolean|Array<String>} reservedFirst The `reservedFirst` option
- * @return {?Function} If an error is detected, a function to generate the error message, otherwise, `undefined`
+ * @return {Function|undefined} If an error is detected, a function to generate the error message, otherwise, `undefined`
  */
 // eslint-disable-next-line consistent-return
 function validateReservedFirstConfig(context, reservedFirst) {

lib/rules/no-danger-with-children.js

@@ -28,6 +28,7 @@ module.exports = {
      * Takes a ObjectExpression and returns the value of the prop if it has it
      * @param {object} node - ObjectExpression node
      * @param {string} propName - name of the prop to look for
+     * @param {any[]} seenProps
      */
     function findObjectProp(node, propName, seenProps) {
       if (!node.properties) {

lib/rules/no-unknown-property.js

@@ -157,8 +157,8 @@ function isTagName(node) {
 
 /**
  * Extracts the tag name for the JSXAttribute
- * @param {Object} node - JSXAttribute being tested.
- * @returns {String} tag name
+ * @param {JSXAttribute} node - JSXAttribute being tested.
+ * @returns {String|null} tag name
  */
 function getTagName(node) {
   if (node && node.parent && node.parent.name && node.parent.name) {
@@ -184,7 +184,7 @@ function tagNameHasDot(node) {
 /**
  * Get the standard name of the attribute.
  * @param {String} name - Name of the attribute.
- * @returns {String} The standard name of the attribute.
+ * @returns {String|undefined} The standard name of the attribute.
  */
 function getStandardName(name) {
   if (DOM_ATTRIBUTE_NAMES[name]) {
@@ -193,12 +193,9 @@ function getStandardName(name) {
   if (SVGDOM_ATTRIBUTE_NAMES[name]) {
     return SVGDOM_ATTRIBUTE_NAMES[name];
   }
-  let i;
-  const found = DOM_PROPERTY_NAMES.some((element, index) => {
-    i = index;
-    return element.toLowerCase() === name;
-  });
-  return found ? DOM_PROPERTY_NAMES[i] : null;
+
+  const found = DOM_PROPERTY_NAMES.find(domPropertyName => domPropertyName.toLowerCase() === name);
+  return found || null;
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-unused-prop-types.js

@@ -82,7 +82,7 @@ module.exports = {
     /**
      * Used to recursively loop through each declared prop type
      * @param {Object} component The component to process
-     * @param {Array} props List of props to validate
+     * @param {ASTNode[]|true} props List of props to validate
      */
     function reportUnusedPropType (component, props) {
       // Skip props that check instances

lib/rules/no-unused-state.js

@@ -101,7 +101,7 @@ module.exports = {
           parent &&
           parent.type === 'MethodDefinition' && (
             parent.static && parent.key.name === 'getDerivedStateFromProps' ||
-            classMethods.indexOf(parent.key.name !== -1)
+            classMethods.indexOf(parent.key.name) !== -1
           ) &&
           parent.value.type === 'FunctionExpression' &&
           parent.value.params[1] &&

lib/rules/prefer-stateless-function.js

@@ -137,7 +137,7 @@ module.exports = {
      * Checks whether the parameters of a constructor and the arguments of `super()`
      * have the same values or not.
      * @see ESLint no-useless-constructor rule
-     * @param {ASTNode} ctorParams - The parameters of a constructor to check.
+     * @param {ASTNode[]} ctorParams - The parameters of a constructor to check.
      * @param {ASTNode} superArgs - The arguments of `super()` to check.
      * @returns {boolean} `true` if those have the same values.
      */

lib/rules/static-property-placement.js

@@ -102,17 +102,17 @@ module.exports = {
 
     /**
      * Check if we should report this property node
-     * @param node
-     * @param expectedRule
+     * @param {ASTNode} node
+     * @param {string} expectedRule
      */
     function reportNodeIncorrectlyPositioned(node, expectedRule) {
       // Detect if this node is an expected property declaration adn return the property name
       const name = classProperties.find(propertyName => {
         if (propertiesToCheck[propertyName](node)) {
-          return propertyName;
+          return !!propertyName;
         }
 
-        return null;
+        return false;
       });
 
       // If name is set but the configured rule does not match expected then report error

lib/rules/style-prop-object.js

@@ -24,7 +24,7 @@ module.exports = {
 
   create: function(context) {
     /**
-     * @param {object} node An Identifier node
+     * @param {ASTNode} expression An Identifier node
      */
     function isNonNullaryLiteral(expression) {
       return expression.type === 'Literal' && expression.value !== null;

lib/types.d.ts

@@ -0,0 +1,13 @@
+import * as eslint from 'eslint';
+import * as estree from 'estree';
+
+declare global {
+  interface ASTNode extends estree.BaseNode {
+    [_: string]: any;
+  }
+  type Scope = eslint.Scope.Scope;
+  type Token = eslint.AST.Token;
+  type Fixer = eslint.Rule.RuleFixer;
+  type JSXAttribute = ASTNode;
+  type JSXSpreadAttribute = ASTNode;
+}

lib/util/Components.js

@@ -258,7 +258,7 @@ function componentRule(rule, context) {
     /**
      * Check if variable is destructured from pragma import
      *
-     * @param {variable} String The variable name to check
+     * @param {string} variable The variable name to check
      * @returns {Boolean} True if createElement is destructured from the pragma
      */
     isDestructuredFromPragmaImport: function(variable) {
@@ -351,7 +351,7 @@ function componentRule(rule, context) {
      * Check if the node is returning JSX
      *
      * @param {ASTNode} ASTnode The AST node being checked
-     * @param {Boolean} strict If true, in a ternary condition the node must return JSX in both cases
+     * @param {Boolean} [strict] If true, in a ternary condition the node must return JSX in both cases
      * @returns {Boolean} True if the node is returning JSX, false if not
      */
     isReturningJSX: function(ASTnode, strict) {
@@ -413,7 +413,7 @@ function componentRule(rule, context) {
      * Check if the node is returning JSX or null
      *
      * @param {ASTNode} ASTnode The AST node being checked
-     * @param {Boolean} strict If true, in a ternary condition the node must return JSX in both cases
+     * @param {Boolean} [strict] If true, in a ternary condition the node must return JSX in both cases
      * @returns {Boolean} True if the node is returning JSX or null, false if not
      */
     isReturningJSXOrNull(ASTNode, strict) {

lib/util/annotations.js

@@ -8,6 +8,7 @@
 /**
  * Checks if we are declaring a `props` argument with a flow type annotation.
  * @param {ASTNode} node The AST node being checked.
+ * @param {Object} context
  * @returns {Boolean} True if the node is a type annotated props declaration, false if not.
  */
 function isAnnotatedFunctionPropsDeclaration(node, context) {

lib/util/ast.js

@@ -6,7 +6,7 @@
 /**
  * Find a return statment in the current node
  *
- * @param {ASTNode} ASTnode The AST node being checked
+ * @param {ASTNode} node The AST node being checked
  */
 function findReturnStatement(node) {
   if (
@@ -114,7 +114,7 @@ function isNodeFirstInLine(context, node) {
 
 /**
  * Checks if the node is a function or arrow function expression.
- * @param {Object} context The node to check
+ * @param {ASTNode} node The node to check
  * @return {Boolean} true if it's a function-like expression
  */
 function isFunctionLikeExpression(node) {
@@ -123,7 +123,7 @@ function isFunctionLikeExpression(node) {
 
 /**
  * Checks if the node is a function.
- * @param {Object} context The node to check
+ * @param {ASTNode} node The node to check
  * @return {Boolean} true if it's a function
  */
 function isFunction(node) {
@@ -132,7 +132,7 @@ function isFunction(node) {
 
 /**
  * Checks if the node is an arrow function.
- * @param {Object} context The node to check
+ * @param {ASTNode} node The node to check
  * @return {Boolean} true if it's an arrow function
  */
 function isArrowFunction(node) {
@@ -141,7 +141,7 @@ function isArrowFunction(node) {
 
 /**
  * Checks if the node is a class.
- * @param {Object} context The node to check
+ * @param {ASTNode} node The node to check
  * @return {Boolean} true if it's a class
  */
 function isClass(node) {

lib/util/defaultProps.js

@@ -70,7 +70,7 @@ module.exports = function defaultPropsInstructions(context, components, utils) {
   /**
    * Adds defaultProps to the component passed in.
    * @param   {ASTNode}         component    The component to add the defaultProps to.
-   * @param   {String[]|String} defaultProps defaultProps to add to the component or the string "unresolved"
+   * @param   {Object[]|'unresolved'} defaultProps defaultProps to add to the component or the string "unresolved"
    *                                         if this component has defaultProps that can't be resolved.
    * @returns {void}
    */

lib/util/linkComponents.js

@@ -3,6 +3,7 @@
  */
 'use strict';
 
+/** @type {any[]}} */
 const DEFAULT_LINK_COMPONENTS = ['a'];
 const DEFAULT_LINK_ATTRIBUTE = 'href';