fix: consider that props might not be used in the component

Author: gndelia

dist/index.js

@@ -26,6 +26,10 @@ exports.default = (fileInfo, { j }) => {
  const outerNewTypeAnnotation = extractPropsDefinitionFromReactFC(j, reactFcOrSfcNode);
  // build the new nodes
  const componentFunctionNode = n.node.init;
+ // if no params, it could be that the component is not actually using props, so nothing to do here
+ if (componentFunctionNode.params.length === 0) {
+ return;
+ }
  const firstParam = componentFunctionNode.params[0];
  let componentFunctionFirstParameter;
  // form of (props) =>

transform.test.ts

@@ -283,6 +283,18 @@ const testCases: TestCase[] = [
  }
  `,
  },
+ {
+ input: `
+ import React from 'react'
+
+ const NoPropsComponent: React.FC<UnusedProps> = () => <span>foo</span>
+ `,
+ output: `
+ import React from 'react'
+
+ const NoPropsComponent = () => <span>foo</span>
+ `,
+ },
 ]
 
 function escapeLineEndingsAndMultiWhiteSpaces(text: string | null | undefined) {

transform.ts

@@ -36,6 +36,10 @@ export default (fileInfo: FileInfo, { j }: API) => {
  const outerNewTypeAnnotation = extractPropsDefinitionFromReactFC(j, reactFcOrSfcNode)
  // build the new nodes
  const componentFunctionNode = n.node.init as ArrowFunctionExpression | FunctionExpression
+ // if no params, it could be that the component is not actually using props, so nothing to do here
+ if (componentFunctionNode.params.length === 0) {
+ return
+ }
  const firstParam = componentFunctionNode.params[0]
 
  let componentFunctionFirstParameter: Identifier | ObjectPattern | undefined