import-js/eslint-plugin-import

"Multiple exports of name 'xxx' import/export" error is thrown if types are exported

Open

#3,136 opened on 2025年1月4日

GitHub で見る
 (3 comments) (0 reactions) (0 assignees)JavaScript (4,946 stars) (1,540 forks)batch import
bughelp wantedtypescript

説明

We have the following two files in the package:

m.ts

export interface Foo {}

export function f(foo: Foo): void {
}

index.ts

export {f} from 'm';
export type * from 'm';

Actual result:

While linting this code the plugin prints the following error for index.ts:

Multiple exports of name 'f' import/export

Expected result:

I don't expect the error to be thrown for the above case as the second expression in index.ts exports types, so there should be no duplicates.

Plugin version: 2.31.0

コントリビューターガイド

技術スタック
javascripttypescript
領域
toolingdeveloper experience
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Familiarity with TypeScriptUnderstanding of ESLint plugin structure
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40
調査方針
Investigate the rule 'import/export' in the eslint plugin import source code. Check the implementation of how exports are compared, specifically the handling of `export type *`. The issue indicates that type only exports are not being filtered out when checking for duplicate exports. Likely the fix involves checking whether an export is a type export and excluding it from the duplicate check. Refer to the file that implements the rule, possibly `src/rules/export.js` or similar. Also consider linked issues or PRs that might address similar problems.
"Multiple exports of name 'xxx' import/export" error is thrown if types are exported · import-js/eslint-plugin-import#3136 | Good First Issue