feat(transformer/typescript): remove verbatim_module_syntax option

[Dunqing]

Remove verbatim_module_syntax option, Because Babel's onlyRemoveTypeImports option same behavior with verbatim_module_syntax , You can see babel/babel#15493 (comment)

[Dunqing]

• feat(transformer/typescript): remove verbatim_module_syntax option #2796 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on Graphite

[Dunqing]

@rzvxa cc

[codspeed-hq]

CodSpeed Performance Report

Merging #2796 will not alter performance

_{Comparing 03-24-feat_transformer/typescript_remove_verbatim_module_syntax_option (ad18009) with main (813226b)}

Summary

✅ 34 untouched benchmarks

[rzvxa]

@Dunqing Well it is great, I think we should move on with this. It is always possible to add an alias to the cli so we can set this flag with either option.

[rzvxa]

I'll resolve the discussion on #2794, I commented there before seeing this PR.

[Dunqing]

It is always possible to add an alias to the client so we can set this flag with either option.

Sorry, I don't get your mean. Can you explain this?

[rzvxa]

It is always possible to add an alias to the client so we can set this flag with either option.

Sorry, I don't get your mean. Can you explain this?

Yeah sure, Sorry I didn't make my assumptions clear, Let me try to rephrase it.

Since verbatim_module_syntax and only_remove_type_imports are doing basically the same thing we can have an alias for them, So no matter which option is passed to the oxc we always set the only_remove_type_imports since it has an easier and more clear name in my opinion.

Remember that I'm not 100% sure these 2 are supposed to do the same thing since the verbatim_module_syntax can mean more things in the future.