fix: Preserve import {type T} from 'x' when onlyRemoveTypeImports:true (#15349)

liuxingbaoyu · web-flow · commit f9fbc25b5284 · 2023-01-15T10:26:19.000-05:00
fix
diff --git a/packages/babel-plugin-transform-typescript/src/index.ts b/packages/babel-plugin-transform-typescript/src/index.ts
@@ -348,7 +348,7 @@ export default declare((api, opts: Options) => {
                 }
               }
 
-              if (isAllSpecifiersElided()) {
+              if (isAllSpecifiersElided() && !onlyRemoveTypeImports) {
                 stmt.remove();
               } else {
                 for (const importPath of importsToRemove) {
diff --git a/packages/babel-plugin-transform-typescript/test/fixtures/imports/only-remove-type-imports/output.mjs b/packages/babel-plugin-transform-typescript/test/fixtures/imports/only-remove-type-imports/output.mjs
@@ -5,5 +5,6 @@ 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/packages/babel-plugin-transform-typescript/test/fixtures/imports/type-only-import-specifier-4/output.mjs b/packages/babel-plugin-transform-typescript/test/fixtures/imports/type-only-import-specifier-4/output.mjs
@@ -0,0 +1 @@
+import "x";

Original file line number	Diff line number	Diff line change
`@@ -348,7 +348,7 @@ export default declare((api, opts: Options) => {`
`348`	`348`	`}`
`349`	`349`	`}`
`350`	`350`
`351`		`- if (isAllSpecifiersElided()) {`
	`351`	`+ if (isAllSpecifiersElided() && !onlyRemoveTypeImports) {`
`352`	`352`	`stmt.remove();`
`353`	`353`	`} else {`
`354`	`354`	`for (const importPath of importsToRemove) {`