refactor(setup-init): relax checks on React imports (#1596)

refactor(setup-init): relax checks on React imports due to IDE formatters removing it if not necessary

- filter out `import React from "react"` from the AST when parsing current and new content

Author: SutuSebastian

.changeset/cute-breads-attack.md

@@ -0,0 +1,7 @@
+---
+"flowbite-react": patch
+---
+
+refactor(setup-init): relax checks on React imports due to IDE formatters removing it if not necessary
+
+- filter out `import React from "react"` from the AST when parsing current and new content

packages/ui/src/cli/commands/setup-init.ts

@@ -1,4 +1,5 @@
 import fs from "fs/promises";
+import type { namedTypes } from "ast-types";
 import { parse } from "recast";
 import { initFilePath, initJsxFilePath } from "../consts";
 import { compareNodes } from "../utils/compare-nodes";
@@ -49,8 +50,8 @@ ThemeInit.displayName = "ThemeInit";
     }
 
     if (currentContent) {
-      const currentAst = parse(currentContent);
-      const newAst = parse(content);
+      const currentAst = removeReactImport(parse(currentContent));
+      const newAst = removeReactImport(parse(content));
 
       if (!compareNodes(currentAst.program, newAst.program)) {
         console.log(`Updating ${targetPath} file...`);
@@ -69,3 +70,22 @@ ThemeInit.displayName = "ThemeInit";
     console.error(`Failed to update ${targetPath}:`, error);
   }
 }
+
+/**
+ * Removes `import React from "react"` from the AST
+ */
+function removeReactImport(ast: namedTypes.File): namedTypes.File {
+  if (ast?.program?.body) {
+    ast.program.body = ast.program.body.filter(
+      (node) =>
+        !(
+          node.type === "ImportDeclaration" &&
+          "value" in node.source &&
+          typeof node.source.value === "string" &&
+          node.source.value === "react" &&
+          node.specifiers?.[0]?.local?.name === "React"
+        ),
+    );
+  }
+  return ast;
+}