Rename import to module/require (#70)

Author: trivikr

src/transforms/v2-to-v3/utils/getV2ClientNames.ts

@@ -10,7 +10,7 @@ export const getV2ClientNames = (
   source: Collection<any>,
   { v2DefaultModuleName, v2ClientModuleNames }: GetV2ClientNamesOptions
 ): Array<string> => {
-  const v2ClientNamesFromDefaultImport = source
+  const v2ClientNamesFromDefaultModule = source
     .find(j.NewExpression, {
       callee: {
         type: "MemberExpression",
@@ -23,10 +23,10 @@ export const getV2ClientNames = (
       (newExpression) => ((newExpression.callee as MemberExpression).property as Identifier).name
     );
 
-  // Merge v2ClientNamesFromDefaultImport with v2ClientImportNames with duplicates removed.
-  return v2ClientNamesFromDefaultImport.concat(
+  // Merge v2ClientNamesFromDefaultModule with v2ClientModuleNames with duplicates removed.
+  return v2ClientNamesFromDefaultModule.concat(
     v2ClientModuleNames.filter(
-      (v2ClientImportName) => v2ClientNamesFromDefaultImport.indexOf(v2ClientImportName) < 0
+      (v2ClientModuleName) => v2ClientNamesFromDefaultModule.indexOf(v2ClientModuleName) < 0
     )
   );
 };

src/transforms/v2-to-v3/utils/removeDefaultRequire.ts

@@ -5,13 +5,13 @@ import { getRequireVariableDeclaration } from "./getRequireVariableDeclaration";
 export const removeDefaultRequire = (
   j: JSCodeshift,
   source: Collection<any>,
-  defaultImportName: string
+  defaultRequireName: string
 ) => {
   getRequireVariableDeclaration(j, source, "aws-sdk")
     .filter(
       (nodePath) =>
         ((nodePath.value.declarations[0] as VariableDeclarator).id as Identifier).name ===
-        defaultImportName
+        defaultRequireName
     )
     .remove();
 };

src/transforms/v2-to-v3/utils/removeV2ClientRequire.ts

@@ -7,8 +7,8 @@ export const removeV2ClientRequire = (
   source: Collection<any>,
   v2ClientName: string
 ) => {
-  const importSourceName = `aws-sdk/clients/${v2ClientName.toLowerCase()}`;
-  getRequireVariableDeclaration(j, source, importSourceName)
+  const requireLiteralName = `aws-sdk/clients/${v2ClientName.toLowerCase()}`;
+  getRequireVariableDeclaration(j, source, requireLiteralName)
     .filter(
       (nodePath) =>
         ((nodePath.value.declarations[0] as VariableDeclarator).id as Identifier).name ===

src/transforms/v2-to-v3/utils/replaceClientCreation.ts

@@ -12,7 +12,7 @@ export const replaceClientCreation = (
   source: Collection<any>,
   { v2DefaultModuleName, v2ClientName, v3ClientName }: ReplaceClientCreationOptions
 ): void => {
-  // Replace clients created with default import.
+  // Replace clients created with default module.
   source
     .find(j.NewExpression, {
       callee: {
@@ -26,7 +26,7 @@ export const replaceClientCreation = (
       return node;
     });
 
-  // Replace clients created with client import.
+  // Replace clients created with client module.
   source
     .find(j.NewExpression, {
       callee: { type: "Identifier", name: v2ClientName },