Add an empty param if value is not passed for optional params in callback (#806)

Author: trivikr

.changeset/odd-insects-learn.md

@@ -0,0 +1,5 @@
+---
+"aws-sdk-js-codemod": patch
+---
+
+Add an empty param if value is not passed for optional params in callback

src/transforms/v2-to-v3/__fixtures__/misc/api-callback.input.js

@@ -2,7 +2,7 @@ import AWS from "aws-sdk";
 
 const client = new AWS.DynamoDB();
 
-client.listTables({}, (err, data) => {
+client.listTables((err, data) => {
   if (err) console.log(err, err.stack);
   else console.log(data);
 });

src/transforms/v2-to-v3/apis/addEmptyObjectForUndefined.ts

@@ -0,0 +1,33 @@
+import { Collection, JSCodeshift } from "jscodeshift";
+
+import { ClientIdentifier } from "../types";
+
+// Adds an empty object, if undefined is passed for optional parameters.
+export const addEmptyObjectForUndefined = (
+  j: JSCodeshift,
+  source: Collection<unknown>,
+  clientIdentifiers: ClientIdentifier[]
+): void => {
+  for (const clientId of clientIdentifiers) {
+    // Remove .promise() from client API calls.
+    source
+      .find(j.CallExpression, {
+        callee: {
+          type: "MemberExpression",
+          object: clientId,
+        },
+      })
+      .filter((callExpression) => {
+        return (
+          callExpression.value.arguments.length === 1 &&
+          ["FunctionExpression", "ArrowFunctionExpression"].includes(
+            callExpression.value.arguments[0].type
+          )
+        );
+      })
+      .replaceWith((callExpression) => {
+        callExpression.value.arguments.unshift(j.objectExpression([]));
+        return callExpression.value;
+      });
+  }
+};

src/transforms/v2-to-v3/apis/index.ts

@@ -1,3 +1,4 @@
+export * from "./addEmptyObjectForUndefined";
 export * from "./addNotSupportedComments";
 export * from "./addNotSupportedClientComments";
 export * from "./getClientWaiterStates";

src/transforms/v2-to-v3/transformer.ts

@@ -10,6 +10,7 @@ import {
   getClientIdentifiersRecord,
   replaceAwsIdentity,
   replaceAwsError,
+  addEmptyObjectForUndefined,
 } from "./apis";
 import { replaceAwsUtilFunctions } from "./aws-util";
 import {
@@ -98,6 +99,7 @@ const transformer = async (file: FileInfo, api: API) => {
     }
 
     removePromiseCalls(j, source, clientIdentifiers);
+    addEmptyObjectForUndefined(j, source, clientIdentifiers);
 
     if (v2ClientName === S3) {
       replaceS3GetSignedUrlApi(j, source, clientIdentifiers);