[Feature]: Use named imports in transformed code

[trivikr]

Self-service

• I'd be willing to implement this feature

Problem

If the input code contains only client initialization, the codemod uses named imports as follows:

Example input:

import AWS from "aws-sdk";
const client = new AWS.DynamoDB({ region: "us-west-2" });

Example output:

import { DynamoDB } from "@aws-sdk/client-dynamodb";
const client = new DynamoDB({ region: "us-west-2" });

However, if the input contains types, the codemod uses global import as follows:

Example input:

import AWS from "aws-sdk";
const input: AWS.DynamoDB.ListTablesInput = { Limit: 10 };

Example output:

import * as AWS_DynamoDB from "@aws-sdk/client-dynamodb";
const input: AWS_DynamoDB.ListTablesCommandInput = { Limit: 10 };

This has been done to avoid additional imports in the past, but the solution is not clean.
The users who ran codemod had requested cleaner solution in the past, like #593

Solution

Use named imports for types as follows:

Example input:

import AWS from "aws-sdk";
const input: AWS.DynamoDB.ListTablesInput = { Limit: 10 };

Example output:

import { ListTablesCommandInput } from "@aws-sdk/client-dynamodb";
const input: ListTablesCommandInput = { Limit: 10 };

Alternatives

Pass an option to the codemod to enable named import, similar to #589
This will not be straightforward to implement, and users might not discover it.

Additional context

No response

[trivikr]

The global import was introduced in the past to have as less changes in the code as possible. But as users have used the codemod, they prefer named imports.

Since import equals does not support named imports, that would be an exception to the standard.

[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.