refactor(useExportType): dedup code

[Conaclos]

Summary

Follow-up of #10561 that deduplicates code.

I added a new utility function split_separated_list in biome_analyze that allows splitting a list into two lists based on a function.
This allowed me to reduce some duplications in useExportType and useImportType.

I solved the duplication I talked in a comment by adding a new node union AnyJsExportSpecifier and by removing useless diagnostic details (the detail reported that an export with a type keyword was a type).

Test Plan

Expecting a green CI.

Docs

No user-facing changes.

[changeset-bot]

🦋 Changeset detected

Latest commit: 206c7fc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 13 packages

Name	Type
@biomejs/biome	Patch
@biomejs/cli-win32-x64	Patch
@biomejs/cli-win32-arm64	Patch
@biomejs/cli-darwin-x64	Patch
@biomejs/cli-darwin-arm64	Patch
@biomejs/cli-linux-x64	Patch
@biomejs/cli-linux-arm64	Patch
@biomejs/cli-linux-x64-musl	Patch
@biomejs/cli-linux-arm64-musl	Patch
@biomejs/wasm-web	Patch
@biomejs/wasm-bundler	Patch
@biomejs/wasm-nodejs	Patch
@biomejs/backend-jsonrpc	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

✅ Organic activity

No automation signals detected in the analyzed events.

View full analysis →

_{This is an automated analysis by AgentScan}

[Conaclos]

Using Separate instead of Extract is better because this matches the docs.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c8ba3085-1bce-4b52-8150-60d68f6e6bac

📥 Commits

Reviewing files that changed from the base of the PR and between 38d256c and 206c7fc.

⛔ Files ignored due to path filters (1)

• Cargo.lock is excluded by !**/*.lock and included by **

📒 Files selected for processing (6)

• .changeset/thick-bugs-push.md
• crates/biome_analyze/Cargo.toml
• crates/biome_analyze/src/utils.rs
• crates/biome_js_analyze/Cargo.toml
• crates/biome_js_analyze/src/lint/style/use_export_type.rs
• crates/biome_js_analyze/src/lint/style/use_import_type.rs

✅ Files skipped from review due to trivial changes (1)

• .changeset/thick-bugs-push.md

🚧 Files skipped from review as they are similar to previous changes (2)

• crates/biome_js_analyze/src/lint/style/use_import_type.rs
• crates/biome_js_analyze/src/lint/style/use_export_type.rs

---

Walkthrough

A generic split_separated_list function is added to biome_analyze::utils to partition AstSeparatedList values into two detached lists via a caller-provided closure, preserving trailing separators on the correct side.

use_export_type is refactored to use a new public AnyJsExportSpecifier union type covering both named and named-from specifiers, with a with_type_token helper. The ExportTypeFix enum's *2 variants are removed; AddInlineTypeQualifiers and SeparateTypes now carry Vec<AnyJsExportSpecifier>. Split helpers and action branches are rewritten using split_separated_list.

use_import_type's manual specifier split loop is replaced with a split_separated_list call.

Possibly related PRs

• biomejs/biome#10561: Directly overlaps with the use_export_type fix-state and named/named-from splitting refactor being continued here.
• biomejs/biome#10569: Touches the same SeparateTypes/SeparateTypes2 variants and split helpers that this PR consolidates.
• biomejs/biome#10530: Modifies use_import_type specifier classification logic that overlaps with this PR's split_separated_list adoption.

Suggested reviewers

• ematipico

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarises the main objective—refactoring the useExportType rule to deduplicate code—which aligns with the changeset's core purpose.
Description check	✅ Passed	The description clearly explains the refactoring's scope, including the new utility function, code deduplication across rules, node union addition, and diagnostic improvements.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch conaclos/useExportType-dedup-code

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codspeed-hq]

Merging this PR will not alter performance

⚡ 1 improved benchmark
❌ 1 regressed benchmark
✅ 247 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Benchmark	BASE	HEAD	Efficiency
❌	db_17847247775464589309.json[uncached]	13.5 ms	14.4 ms	-6.46%
⚡	uncached[simple_classes.txt]	219.3 µs	206.4 µs	+6.27%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.

---

_{Comparing conaclos/useExportType-dedup-code (206c7fc) with main (043fbb5)¹}

Footnotes

1. No successful run was found on main (a0f197e) during the generation of this report, so 043fbb5 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

[ematipico]

Remove Split and use either. The rest are nits, but I think the loop I flagged needs to be changed