transformer: improve the implementation of eliminating type-only exports

[Dunqing]

#3820 was added IdentifierReference to ExportSpecifier. This means we no longer need to collect type-only import binding names.

oxc/crates/oxc_transformer/src/typescript/annotations.rs

Lines 143 to 146 in 4456034

	if is_type || s.import_kind.is_type() {
	self.type_identifier_names.insert(s.local.name.clone());
	return false;
	}

We just need to check if the import binding has a type reference.

But, unfortunately, we're currently stuck due to issue #3826. Once it's fixed, I'll be able to start working on this.

[Dunqing]

https://oxc-project.github.io/oxc/playground/?code=3YCAAICjgICAgICAgIC0G8rnONK883rgbiRJi4tneyT%2FrTHJB6mX%2Ba%2BqVzpzEyct039%2FXqCAgA%3D%3D

We may also need a different flag for the type/value import

[overlookmotel]

I think we need to resolve the questions in #3799 before tackling this.

[Dunqing]

I think we need to resolve the questions in #3799 before tackling this.

Yes

[overlookmotel]

According to comment there, #3827 closes this. @Dunqing please re-open if that's not correct.