better code organize

Author: minhna

Diff for: constants.ts

@@ -0,0 +1,5 @@
+const CONSTANTS = {
+  METEOR_ROOT_DIRECTORY: "/home/minhna/WORKS/Mike/settler/se2-admin",
+};
+
+export default CONSTANTS;

Diff for: transform-export-async-function.ts

@@ -7,33 +7,21 @@
  * 6. Convert them to await expressions
  */
 
-import fs from "fs";
-
 import {
   FileInfo,
   API,
   Options,
-  ASTPath,
-  CallExpression,
   ExportDefaultDeclaration,
   ExportNamedDeclaration,
-  AwaitExpression,
-  Collection,
 } from "jscodeshift";
+import CONSTANTS from "./constants";
 
 const tsParser = require("jscodeshift/parser/ts");
 
 const debug = require("debug")("transform:export-async-script");
 const debug2 = require("debug")("transform:print:export-async-script");
 
-import {
-  addAwaitKeyword,
-  convertAllCallExpressionToAsync,
-  findParentFunction,
-  setFunctionAsync,
-} from "./utils";
-
-const METEOR_ROOT_DIRECTORY = "/home/minhna/WORKS/Mike/settler/se2-admin";
+import { convertAllCallExpressionToAsync, getFileContent } from "./utils";
 
 module.exports = function (fileInfo: FileInfo, { j }: API, options: Options) {
   debug(
@@ -52,45 +40,11 @@ module.exports = function (fileInfo: FileInfo, { j }: API, options: Options) {
 
   const getRealImportSource = (source: string): string => {
     if (/^\//.test(source)) {
-      return METEOR_ROOT_DIRECTORY + source;
+      return CONSTANTS.METEOR_ROOT_DIRECTORY + source;
     }
     return getPathFromSource(fileInfo.path) + "/" + source.replace(/^\.\//, "");
   };
 
-  const getFileContent = (path: string): string | undefined => {
-    let fileContent: Buffer | null = null;
-
-    if (/(\.js|\.ts)$/.test(path)) {
-      try {
-        fileContent = fs.readFileSync(path);
-      } catch (e) {
-        debug("File was not found:", path);
-      }
-    } else {
-      try {
-        fileContent = fs.readFileSync(path + ".js");
-      } catch (e) {
-        try {
-          fileContent = fs.readFileSync(path + ".ts");
-        } catch (e2) {
-          // check for index file
-          try {
-            fileContent = fs.readFileSync(path + "/index.js");
-          } catch (e3) {
-            try {
-              fileContent = fs.readFileSync(path + "/index.ts");
-            } catch (e4) {
-              debug("File was not found");
-            }
-          }
-        }
-      }
-    }
-
-    // debug("content", fileContent.toString());
-    return fileContent?.toString();
-  };
-
   // function to read the export source file, then check for exported async function
   interface AnalyzeSourceParams {
     exportedFunction: string;

Diff for: transform-find-await-without-async.ts

@@ -4,7 +4,7 @@
  */
 
 import { FileInfo, API, Options } from "jscodeshift";
-import { findParentFunction, getFunctionLocation } from "./utils";
+import { findParentFunction } from "./utils";
 
 const debug = require("debug")("transform:find-await-without-async");
 const debug2 = require("debug")("transform:print:find-await-without-async");

Diff for: transform-fix-async-overly.ts

@@ -4,39 +4,9 @@
  * 3. If there wasn't any await expression, then remove async from function expression.
  */
 
-import {
-  FileInfo,
-  API,
-  Options,
-  ASTPath,
-  Collection,
-  MemberExpression,
-  CallExpression,
-  ExpressionStatement,
-  FunctionDeclaration,
-  ArrowFunctionExpression,
-  FunctionExpression,
-  Position,
-} from "jscodeshift";
-
-const methodsMapping = {
-  findOne: "findOneAsync",
-  insert: "insertAsync",
-  upsert: "upsertAsync",
-  update: "updateAsync",
-  remove: "removeAsync",
-  createIndex: "createIndexAsync",
-  dropIndex: "dropIndexAsync",
-  dropCollection: "dropCollectionAsync",
-  // methods on cursors
-  count: "countAsync",
-  fetch: "fetchAsync",
-  forEach: "forEachAsync",
-  map: "mapAsync",
-};
+import { FileInfo, API, Options, ASTPath } from "jscodeshift";
 
 import {
-  addAwaitKeyword,
   findParentFunction,
   getFunctionLocation,
   setFunctionNotAsync,

Diff for: transform-rename-functions.ts

@@ -9,13 +9,8 @@ import {
   API,
   Options,
   ASTPath,
-  Collection,
-  MemberExpression,
   CallExpression,
   ExpressionStatement,
-  FunctionDeclaration,
-  ArrowFunctionExpression,
-  FunctionExpression,
 } from "jscodeshift";
 
 import * as recast from "recast";
@@ -26,14 +21,6 @@ import {
   setFunctionAsync,
 } from "./utils";
 
-type MethodDescType = {
-  type: string;
-  objectName?: string;
-  propertyName?: string;
-  functionName?: string;
-  await?: boolean;
-};
-
 type MethodsMappingItemType = {
   from: string;
   to: string;

Diff for: transform-use-async-function.ts

@@ -7,34 +7,17 @@
 import fs from "fs";
 6;
 
-import {
-  FileInfo,
-  API,
-  Options,
-  ASTPath,
-  CallExpression,
-  ExportDefaultDeclaration,
-  ExportNamedDeclaration,
-  AwaitExpression,
-  Collection,
-} from "jscodeshift";
-
-const tsParser = require("jscodeshift/parser/ts");
+import { FileInfo, API, Options } from "jscodeshift";
 
 const debug = require("debug")("transform:use-async-function");
 const debug2 = require("debug")("transform:print:use-async-function");
 
 import {
-  addAwaitKeyword,
   convertAllCallExpressionToAsync,
   convertAllMemberExpressionCallToAsync,
-  findParentFunction,
   findParentObject,
-  setFunctionAsync,
 } from "./utils";
 
-const METEOR_ROOT_DIRECTORY = "/home/minhna/WORKS/Mike/settler/se2-admin";
-
 module.exports = function (fileInfo: FileInfo, { j }: API, options: Options) {
   debug(
     `\n**************************************************

Diff for: utils.ts

@@ -1,4 +1,5 @@
 import { ASTPath, CallExpression, JSCodeshift, Collection } from "jscodeshift";
+import fs from "fs";
 
 const debug = require("debug")("transform:utils");
 
@@ -185,3 +186,37 @@ export const getFunctionLocation = (p: ASTPath) => {
       debug("Unhandled function type:", p.value.type);
   }
 };
+
+export const getFileContent = (path: string): string | undefined => {
+  let fileContent: Buffer | null = null;
+
+  if (/(\.js|\.ts)$/.test(path)) {
+    try {
+      fileContent = fs.readFileSync(path);
+    } catch (e) {
+      debug("File was not found:", path);
+    }
+  } else {
+    try {
+      fileContent = fs.readFileSync(path + ".js");
+    } catch (e) {
+      try {
+        fileContent = fs.readFileSync(path + ".ts");
+      } catch (e2) {
+        // check for index file
+        try {
+          fileContent = fs.readFileSync(path + "/index.js");
+        } catch (e3) {
+          try {
+            fileContent = fs.readFileSync(path + "/index.ts");
+          } catch (e4) {
+            debug("File was not found");
+          }
+        }
+      }
+    }
+  }
+
+  // debug("content", fileContent.toString());
+  return fileContent?.toString();
+};