import-js/eslint-plugin-import

[no-duplicates] prefer-inline breaks unassigned and top-level type imports

Open

Aperta il 23 dic 2024

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (4946 star) (1540 fork)batch import
bughelp wantedtypescript

Descrizione

"import/no-duplicates": ["error", { "prefer-inline": true }],

Will detect an error in the following typescript code:

import type { X } from "xyz";

import "xyz";

// [...]

It will auto-fix by removing the import "xyz"; statement. This is not the desired behavior. I would expect it to stay as is.

Merry Christmas 🎄

Guida contributor

Tech stack
typescriptjavascript
Dominio
developer experience
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
half day
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Understanding of TypeScript import typesBasic knowledge of ESLint rules
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65
Direzione di ricerca
Il problema riguarda la regola `no duplicates` con l'opzione `prefer inline` che rimuove erroneamente un'importazione con effetti collaterali quando esiste un'importazione di tipo dallo stesso modulo. Per risolvere, esamina l'implementazione della regola in `src/rules/no duplicates.ts` (o simile) per capire come gestisce le importazioni `type` e quelle con effetti collaterali. La logica di auto correzione dovrebbe preservare `import "xyz";` anche quando è presente `import type { X }`. Osserva i test esistenti per `prefer inline` per vedere come gestiscono i casi limite e aggiungi un caso di test per le importazioni non assegnate. Il segnalatore del problema ha fornito un esempio chiaro di fallimento.