import-js/eslint-plugin-import

[import/extensions] ".." and "." import paths are incorrectly flagged as requiring an extension, when either is al local package

Open

#2.844 geöffnet am 31. Juli 2023

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

Beschreibung

In #2778, we determined that this plugin incorrectly flags ".." and "." import paths, when those imports refer to a local package (with a package.json).

Contributor Guide

Tech Stack
javascriptnodejs
Domain
toolingdeveloper 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.
needs maintainer response
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Understanding of ESLint custom rulesNode.js module resolution
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40
Research-Richtung
The issue references #2778 where a similar case was discussed. To fix, examine the `import/extensions` rule implementation in the plugin, likely in `src/rules/extensions.js` or similar. Investigate how the rule determines whether a path is a local package by checking for a `package.json` in the directory. The fix should ensure that relative paths like '.' and '..' that resolve to a package (with a `package.json`) are not flagged as requiring an extension. Test cases should be added for this scenario.