chore: drop support for Node.js 12.x (#132)

Author: trivikr

.changeset/large-seals-rush.md

@@ -0,0 +1,5 @@
+---
+"aws-sdk-js-codemod": minor
+---
+
+Drop support for Node.js 12.x

.github/workflows/build.yml

@@ -9,7 +9,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x]
+        node-version: [14.x, 16.x, 18.x]
 
     steps:
       - uses: actions/checkout@v2

package.json

@@ -44,10 +44,10 @@
   },
   "devDependencies": {
     "@changesets/cli": "2.21.0",
-    "@tsconfig/node12": "1.0.2",
+    "@tsconfig/node14": "1.0.3",
     "@types/jest": "27.4.1",
     "@types/jscodeshift": "0.11.5",
-    "@types/node": "16.11.26",
+    "@types/node": "14.18.33",
     "@typescript-eslint/eslint-plugin": "5.43.0",
     "@typescript-eslint/parser": "5.43.0",
     "eslint": "8.27.0",
@@ -60,7 +60,7 @@
     "typescript": "4.8.4"
   },
   "engines": {
-    "node": ">=12.0.0"
+    "node": ">=14.0.0"
   },
   "packageManager": "[email protected]",
   "lint-staged": {

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

@@ -1,4 +1,6 @@
 import { readdirSync, readFileSync } from "fs";
+// eslint-disable-next-line @typescript-eslint/ban-ts-comment
+// @ts-ignore: Could not find a declaration file for module 'jscodeshift/dist/testUtils'
 import { runInlineTest } from "jscodeshift/dist/testUtils";
 import { join } from "path";
 

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

@@ -17,7 +17,7 @@ export default function transformer(file: FileInfo, api: API) {
   const j = api.jscodeshift;
   const source = j(file.source);
 
-  const v2DefaultModuleName = getV2DefaultModuleName(j, source);
+  const v2DefaultModuleName = getV2DefaultModuleName(j, source) as string;
   const v2ServiceModuleNames = getV2ServiceModuleNames(j, source);
   if (!v2DefaultModuleName && v2ServiceModuleNames.length === 0) {
     return source.toSource();

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

@@ -17,8 +17,8 @@ export const addV3ClientImport = (
   if (existingImports.size()) {
     existingImports.forEach((nodePath) => {
       // Append to existing import if specifier not present.
-      if (!nodePath.value.specifiers.find((specifier) => specifier.local.name === v3ClientName)) {
-        nodePath.value.specifiers.push(j.importSpecifier(j.identifier(v3ClientName)));
+      if (!nodePath.value.specifiers?.find((specifier) => specifier.local?.name === v3ClientName)) {
+        nodePath.value.specifiers?.push(j.importSpecifier(j.identifier(v3ClientName)));
       }
     });
     return;

src/transforms/v2-to-v3/utils/config/CLIENT_NAMES_MAP.ts

@@ -1,6 +1,6 @@
 import { CLIENT_NAMES } from "./CLIENT_NAMES";
 
-export const CLIENT_NAMES_MAP = {
+export const CLIENT_NAMES_MAP: Record<string, string> = {
   ...CLIENT_NAMES.reduce((acc, name) => ({ ...acc, [name]: name }), {}),
   AugmentedAIRuntime: "SageMakerA2IRuntime",
   CUR: "CostAndUsageReportService",

src/transforms/v2-to-v3/utils/config/CLIENT_PACKAGE_NAMES_MAP.ts

@@ -1,6 +1,6 @@
 import { CLIENT_NAMES } from "./CLIENT_NAMES";
 
-export const CLIENT_PACKAGE_NAMES_MAP = {
+export const CLIENT_PACKAGE_NAMES_MAP: Record<string, string> = {
   ...CLIENT_NAMES.reduce(
     (acc, name) => ({
       ...acc,

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

@@ -9,4 +9,4 @@ export const getImportIdentifierName = (
     .find(j.ImportDeclaration, {
       source: { value: literalValue },
     })
-    .nodes()[0]?.specifiers[0]?.local.name;
+    .nodes()[0]?.specifiers?.[0]?.local?.name;

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

@@ -9,6 +9,6 @@ import { getV2ServiceModulePath } from "./getV2ServiceModulePath";
 export const getV2ServiceModuleNames = (j: JSCodeshift, source: Collection<unknown>): string[] =>
   CLIENT_NAMES.map((clientName) =>
     containsRequire(j, source)
-      ? getRequireIdentifierName(j, source, getV2ServiceModulePath(clientName))
-      : getImportIdentifierName(j, source, getV2ServiceModulePath(clientName))
+      ? (getRequireIdentifierName(j, source, getV2ServiceModulePath(clientName)) as string)
+      : (getImportIdentifierName(j, source, getV2ServiceModulePath(clientName)) as string)
   ).filter((v2ServiceModuleName) => v2ServiceModuleName !== undefined);