import-js/eslint-plugin-import

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

Open

#3 136 ouverte le 4 janv. 2025

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (4 946 stars) (1 540 forks)batch import
bughelp wantedtypescript

Description

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

Guide contributeur

Stack technique
javascripttypescript
Domaine
toolingdeveloper experience
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Familiarity with TypeScriptUnderstanding of ESLint plugin structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
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.