[Feature]: Declare variables used in the code after import equals global variable

[trivikr]

Self-service

• I'd be willing to implement this feature

Problem

In #603, we added named imports for require/import.

Since import equals does not support named imports, we started using property access from global import as follows:

import AWS_S3 = require("@aws-sdk/client-s3");

const client = new AWS_S3.S3();

This is not scalable when it comes to maintenance, as the importType needs to be passed to each function, and need to compute default local name while replacing types or client instances.

• https://github.com/awslabs/aws-sdk-js-codemod/blob/a01e79747f6ab8f6aae0e08c5ee5aa62abf6bdb5/src/transforms/v2-to-v3/transformer.ts#L74-L95
• https://github.com/awslabs/aws-sdk-js-codemod/blob/a01e79747f6ab8f6aae0e08c5ee5aa62abf6bdb5/src/transforms/v2-to-v3/ts-type/getV3ClientType.ts#L19-L23

Solution

Use destructuring after the last import as follows:

import AWS_S3 = require("@aws-sdk/client-s3");
import S3 = AWS_S3.S3;

const client = new S3();

Alternatives

N/A

Additional context

No response

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.