Fix detection of local components to not generate warning on for vari…

…able inside JSX files that follow React component naming (fixes #75)

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+- Fix detection of local components to not generate warning on for variable inside JSX files that follow React component naming 
+
 ## 0.4.16
 
 Fix CJS/ESM interop issue. Sorry everyone for the trouble.

src/only-export-components.test.ts

@@ -194,6 +194,10 @@ const valid = [
     code: "const MyComponent = () => {}; export default observer(MyComponent);",
     options: [{ customHOCs: ["observer"] }],
   },
+  {
+    name: "Local constant with component casing and non component function",
+    code: "const SomeConstant = 42; export function someUtility() { return SomeConstant }",
+  },
 ];
 
 const invalid = [

src/only-export-components.ts

@@ -99,15 +99,6 @@ export const onlyExportComponents: TSESLint.RuleModule<
         )[] = [];
         const reactContextExports: TSESTree.Identifier[] = [];
 
-        const handleLocalIdentifier = (
-          identifierNode: TSESTree.BindingName,
-        ) => {
-          if (identifierNode.type !== "Identifier") return;
-          if (reactComponentNameRE.test(identifierNode.name)) {
-            localComponents.push(identifierNode);
-          }
-        };
-
         const handleExportIdentifier = (
           identifierNode: TSESTree.BindingName | TSESTree.StringLiteral,
           isFunction?: boolean,
@@ -264,10 +255,18 @@ export const onlyExportComponents: TSESLint.RuleModule<
             }
           } else if (node.type === "VariableDeclaration") {
             for (const variable of node.declarations) {
-              handleLocalIdentifier(variable.id);
+              if (
+                variable.id.type === "Identifier" &&
+                reactComponentNameRE.test(variable.id.name) &&
+                canBeReactFunctionComponent(variable.init)
+              ) {
+                localComponents.push(variable.id);
+              }
             }
           } else if (node.type === "FunctionDeclaration") {
-            handleLocalIdentifier(node.id);
+            if (reactComponentNameRE.test(node.id.name)) {
+              localComponents.push(node.id);
+            }
           } else if (
             node.type === "ImportDeclaration" &&
             node.source.value === "react"