fix: set onlyRemoveTypeImports: false by default

sapphi-red · sapphi-red · commit 84b75a39731e · 2024-12-20T14:45:57.000+09:00
diff --git a/packages/vite/src/node/plugins/oxc.ts b/packages/vite/src/node/plugins/oxc.ts
@@ -158,7 +158,12 @@ export async function transformWithOxc(
             `preserveValueImports=${preserveValueImports} + importsNotUsedAsValues=${importsNotUsedAsValues} is not supported by oxc.` +
               'Please migrate to the new verbatimModuleSyntax option.',
           )
+          resolvedOptions.typescript ??= {}
+          resolvedOptions.typescript.onlyRemoveTypeImports = false
         }
+      } else {
+        resolvedOptions.typescript ??= {}
+        resolvedOptions.typescript.onlyRemoveTypeImports = false
       }
 
       const resolvedTsconfigTarget = resolveTsconfigTarget(

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,12 @@ export async function transformWithOxc(`
`158`	`158`	`preserveValueImports=${preserveValueImports} + importsNotUsedAsValues=${importsNotUsedAsValues} is not supported by oxc.` +
`159`	`159`	`'Please migrate to the new verbatimModuleSyntax option.',`
`160`	`160`	`)`
	`161`	`+ resolvedOptions.typescript ??= {}`
	`162`	`+ resolvedOptions.typescript.onlyRemoveTypeImports = false`
`161`	`163`	`}`
	`164`	`+ } else {`
	`165`	`+ resolvedOptions.typescript ??= {}`
	`166`	`+ resolvedOptions.typescript.onlyRemoveTypeImports = false`
`162`	`167`	`}`
`163`	`168`
`164`	`169`	`const resolvedTsconfigTarget = resolveTsconfigTarget(`