Remove import statement if all of specifiers are type-only one

babel · Oct 16, 2021 · 6b79fd1 · 6b79fd1
1 parent ec6398e
commit 6b79fd1
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 3 deletions.
diff --git a/packages/babel-plugin-transform-typescript/src/index.ts b/packages/babel-plugin-transform-typescript/src/index.ts
@@ -288,7 +288,7 @@ export default declare((api, opts) => {
                 }
               }
 
-              if (!onlyRemoveTypeImports && isAllSpecifiersElided()) {
+              if (isAllSpecifiersElided()) {
                 stmt.remove();
               } else {
                 for (const importPath of importsToRemove) {

diff --git a/...bel-plugin-transform-typescript/test/fixtures/imports/only-remove-type-imports/output.mjs b/...bel-plugin-transform-typescript/test/fixtures/imports/only-remove-type-imports/output.mjs
@@ -5,6 +5,5 @@ import d, { d2 } from "d";
 import e, { e3 as e4 } from "e";
 import "f";
 import "g";
-import "k";
 import { L2 } from "l";
 ;
diff --git a/...plugin-transform-typescript/test/fixtures/imports/type-only-import-specifier-4/output.mjs b/...plugin-transform-typescript/test/fixtures/imports/type-only-import-specifier-4/output.mjs
@@ -1 +0,0 @@
-import "x";