import-js/eslint-plugin-import

[import/default]: false negative when importing CJS module which uses dynamic import

Open

#2.769 aberto em 27 de abr. de 2023

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)JavaScript (1.540 forks)batch import
help wanted

Métricas do repositório

Stars
 (4.946 stars)
Métricas de merge de PR
 (Mesclagem média 138d 22h) (3 fundiu PRs em 30d)

Description

I found a weird problem with the import/default rule. See this minimal gist

I import two modules which are very similar, both being CJS with default export.

import ecmaScriptLiteralLoader from 'rdf-loader-code/ecmaScriptLiteral.js'
import ecmaScriptModuleLoader from 'rdf-loader-code/ecmaScriptModule.js'

The first is fine but the second triggers a rule

3:8  error  No default export found in imported module "rdf-loader-code/ecmaScriptModule.js"  import/default

Guia do colaborador

Direção de pesquisa
Investigue a regra `import/default` no repositório eslint plugin import. O bug ocorre quando um módulo CJS com uma exportação padrão também usa importações dinâmicas, causando um falso negativo. Revise a lógica da regra para lidar com módulos CJS, particularmente como ela determina exportações padrão. O gist de reprodução em https://gist.github.com/tpluscode/50d638600e4397a0bef4b3289cfb2925 fornece casos de teste mínimos. Procure por quaisquer comentários existentes na issue discutindo possíveis causas ou código relacionado.
Pilha de tecnologia
javascriptnodejs
Domain
toolingbackend
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.
3-5 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Node.jsESLintnpm
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60