refator: transformer util function

Author: developer-bandi

package/src/transformers/change-props-value/transformer.ts

@@ -1,6 +1,8 @@
 import { OptionsSchema } from "./optionsSchema";
 import getConvertedPath from "../../utils/getConvertedPath";
 import type { API, FileInfo } from "jscodeshift";
+import generateLiteralNode from "../../utils/jscodeshift/generateLiteralNode";
+
 function transformer(file: FileInfo, api: API, options: OptionsSchema) {
   const sourceCode = file.source;
   const jscodeshift = api.jscodeshift;
@@ -20,22 +22,6 @@ function transformer(file: FileInfo, api: API, options: OptionsSchema) {
     targetPath: componentSource,
   });
 
-  const getPropValueNode = () => {
-    if (typeof propsValue === "string") {
-      return jscodeshift.stringLiteral(propsValue);
-    }
-
-    if (typeof propsValue === "boolean") {
-      return jscodeshift.booleanLiteral(propsValue);
-    }
-
-    if (typeof propsValue === "number") {
-      return jscodeshift.numericLiteral(propsValue);
-    }
-
-    return jscodeshift.literal(propsValue);
-  };
-
   let convertedComponentName: null | string = null;
 
   jscodeshift(sourceCode)
@@ -107,7 +93,7 @@ function transformer(file: FileInfo, api: API, options: OptionsSchema) {
 
       if (attribute?.type === "JSXAttribute") {
         attribute.value = jscodeshift.jsxExpressionContainer(
-          getPropValueNode()
+          generateLiteralNode(propsValue)
         );
       }
     })

package/src/transformers/change-source/transformer.ts

@@ -7,25 +7,26 @@ function transformer(file: FileInfo, api: API, options: OptionsSchema) {
   const sourceCode = file.source;
   const jscodeshift = api.jscodeshift;
   const { sourceType = "absolute", fromSource, toSource } = options;
+  const root = jscodeshift(sourceCode);
 
   const convertedFromPath = getConvertedPath({
     type: sourceType,
     currentPath: file.path,
     targetPath: fromSource,
   });
+
   const convertedToPath = getConvertedPath({
     type: sourceType,
     currentPath: file.path,
     targetPath: toSource,
   });
 
-  const changedImportDeclarationSourceCode = jscodeshift(sourceCode)
+  root
     .find(jscodeshift.ImportDeclaration)
     .filter((node) => node.value.source.value === convertedFromPath)
-    .forEach((node) => (node.value.source.value = convertedToPath))
-    .toSource();
+    .forEach((node) => (node.value.source.value = convertedToPath));
 
-  return jscodeshift(changedImportDeclarationSourceCode)
+  root
     .find(jscodeshift.CallExpression, (node) => node.callee.type === "Import")
     .filter(
       (node) =>
@@ -36,8 +37,9 @@ function transformer(file: FileInfo, api: API, options: OptionsSchema) {
       (node) =>
         node.value.arguments[0].type === "StringLiteral" &&
         (node.value.arguments[0].value = options.toSource)
-    )
-    .toSource();
+    );
+
+  return root.toSource();
 }
 
 export default transformer;

package/src/transformers/remove-import/transformer.ts

@@ -7,14 +7,15 @@ function transformer(file: FileInfo, api: API, options: OptionsSchema) {
   const sourceCode = file.source;
   const jscodeshift = api.jscodeshift;
   const { sourceType = "absolute", source } = options;
+  const root = jscodeshift(sourceCode);
 
   const convertedToPath = getConvertedPath({
     type: sourceType,
     currentPath: file.path,
     targetPath: source,
   });
 
-  return jscodeshift(sourceCode)
+  return root
     .find(jscodeshift.ImportDeclaration)
     .filter(
       (node) =>

package/src/utils/jscodeshift/generateLiteralNode.ts

@@ -0,0 +1,21 @@
+import jscodeshift from "jscodeshift";
+
+function generateLiteralNode(
+  value: string | number | boolean | RegExp | bigint | null
+) {
+  if (typeof value === "string") {
+    return jscodeshift.stringLiteral(value);
+  }
+
+  if (typeof value === "boolean") {
+    return jscodeshift.booleanLiteral(value);
+  }
+
+  if (typeof value === "number") {
+    return jscodeshift.numericLiteral(value);
+  }
+
+  return jscodeshift.literal(value);
+}
+
+export default generateLiteralNode;

package/src/utils/jscodeshift/isImportDeclarationNoneSpecifier.ts

@@ -0,0 +1,10 @@
+import { ASTPath } from "jscodeshift";
+import { namedTypes } from "ast-types";
+
+function isImportDeclarationNoneSpecifier(
+  node: ASTPath<namedTypes.ImportDeclaration>
+) {
+  return !node.value.specifiers?.length;
+}
+
+export default isImportDeclarationNoneSpecifier;