Skip to content

Commit a01e797

Browse files
authored
Remove check for require global identifier (#611)
1 parent 52b7c9c commit a01e797

File tree

2 files changed

+5
-18
lines changed

2 files changed

+5
-18
lines changed

.changeset/long-panthers-wonder.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"aws-sdk-js-codemod": patch
3+
---
4+
5+
Remove check for require global identifier

src/transforms/v2-to-v3/modules/requireModule/addClientNamedModule.ts

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,8 @@
11
import { Collection, JSCodeshift, ObjectPattern, ObjectProperty, Property } from "jscodeshift";
22

33
import { OBJECT_PROPERTY_TYPE_LIST } from "../../config";
4-
import { getDefaultLocalName } from "../../utils";
54
import { getRequireDeclarator } from "../getRequireDeclarator";
65
import { getRequireDeclarators } from "../getRequireDeclarators";
7-
import { getRequireDeclaratorsWithIdentifier } from "../getRequireDeclaratorsWithIdentifier";
86
import { getRequireProperty } from "../getRequireProperty";
97
import { objectPatternPropertyCompareFn } from "../objectPatternPropertyCompareFn";
108
import { ClientModulesOptions, ImportSpecifierOptions } from "../types";
@@ -18,7 +16,6 @@ export const addClientNamedModule = (
1816
options;
1917
const localName = options.localName ?? importedName;
2018

21-
const defaultLocalName = getDefaultLocalName(v2ClientLocalName);
2219
const clientObjectProperty = getRequireProperty(j, { importedName, localName });
2320
const existingRequires = getRequireDeclarators(j, source, v3ClientPackageName);
2421

@@ -46,21 +43,6 @@ export const addClientNamedModule = (
4643
return;
4744
}
4845

49-
const requireDeclaratorsWithIdentifier = getRequireDeclaratorsWithIdentifier(j, source, {
50-
identifierName: defaultLocalName,
51-
sourceValue: v3ClientPackageName,
52-
});
53-
54-
if (requireDeclaratorsWithIdentifier && requireDeclaratorsWithIdentifier.nodes().length > 0) {
55-
requireDeclaratorsWithIdentifier.at(0).insertAfter(
56-
j.variableDeclarator(j.objectPattern([clientObjectProperty]), {
57-
type: "Identifier",
58-
name: defaultLocalName,
59-
})
60-
);
61-
return;
62-
}
63-
6446
if (existingRequireProperties.length > 0) {
6547
const firstRequireProperties = (existingRequireProperties[0].id as ObjectPattern).properties;
6648
firstRequireProperties.push(clientObjectProperty);

0 commit comments

Comments
 (0)