linter: import/export with a typescript file only export type

[markEHVN]

What version of Oxlint are you using?

0.11.1

What command did you run?

yarn oxlint -c .oxlint.json

What does your .oxlint.json config file look like?

{
  "$schema": "./node_modules/oxlint/configuration_schema.json",
  "categories": {
    "correctness": "allow"
  },
  "plugins": ["import"],
  "env": {
    "node": true
  },
  "globals": {},
  "settings": {},
  "rules": {
    "import/export": "error"
  }
}

What happened?

I caught an error when I try use import/export with an typescript only export type. ESLint plugin work well.

You can see source code at here

[pumano]

I try to check it.

UPD: reason is:

if files contains only export type or export interface that marked as no named export. Continue digging...

looks like problem in ( I may be wrong):

oxc/crates/oxc_semantic/src/module_record/builder.rs

Lines 273 to 281 in 1282221

	fn visit_export_named_declaration(&mut self, decl: &ExportNamedDeclaration) {
	if decl.export_kind.is_type() {
	return;
	}
	// ignore all TypeScript syntax as they overload
	if decl.is_typescript_syntax() {
	return;
	}

[Boshen]

I try to check it.

UPD: reason is:

if files contains only export type or export interface that marked as no named export. Continue digging...

looks like problem in ( I may be wrong):

oxc/crates/oxc_semantic/src/module_record/builder.rs

Lines 273 to 281 in 1282221

	fn visit_export_named_declaration(&mut self, decl: &ExportNamedDeclaration) {
	if decl.export_kind.is_type() {
	return;
	}
	// ignore all TypeScript syntax as they overload
	if decl.is_typescript_syntax() {
	return;
	}

Thank you for the investigation, this is the problem. We need to add type information the ModuleRecord 😢

[Boshen]

I think the best practice here is to use export type for proper tooling support.

I added support for export type * from 'mod' in #7676

by excluding type export syntax:

if star_export_entry.is_type {
  return;
}

Please forgive me for not checking for types, as tsc already does it, and it really complicates the import plugin.