import-js/eslint-plugin-import

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

Open

#3.136 aberto em 4 de jan. de 2025

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)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

Guia do colaborador

Pilha de tecnologia
javascripttypescript
Domain
toolingdeveloper experience
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Familiarity with TypeScriptUnderstanding of ESLint plugin structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40
Direção de pesquisa
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.