import-js/eslint-plugin-import

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

Open

#3.130 geöffnet am 23. Dez. 2024

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (4.946 Stars) (1.540 Forks)batch import
bughelp wantedtypescript

Beschreibung

"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 🎄

Contributor Guide

Tech Stack
typescriptjavascript
Domain
developer experience
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Understanding of TypeScript import typesBasic knowledge of ESLint rules
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
The issue is about the `no duplicates` rule with `prefer inline` option incorrectly removing a side effect import when a type import from the same module exists. To fix, examine the rule's implementation in `src/rules/no duplicates.ts` (or similar) to understand how it handles `type` imports and side effect imports. The auto fix logic should preserve `import "xyz";` even when `import type { X }` is present. Look at existing tests for `prefer inline` to see how they handle edge cases and add a test case for unassigned imports. The issue reporter provided a clear failing example.