jsx-eslint/eslint-plugin-react

[Bug]: `hook-use-state` suggested fix should update all reference

Open

#3.810 geöffnet am 31. Aug. 2024

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.797 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (8.630 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Is there an existing issue for this?

  • I have searched the existing issues and my issue is unique
  • My issue appears in the command-line and not only in the text editor

Description Overview

The current fix suggested is to simply replace the variable, This isn't smart enough

It should update all references

https://github.com/user-attachments/assets/92187480-3b65-4667-a94b-66c8670de283

Expected Behavior

This is a custom react-hook-use-state rule I wrote. Before this, I didn’t know the existence of react/hook-use-state

it's smart enough

https://github.com/user-attachments/assets/196c5324-b1f8-41d7-8f5a-efd2c8342508

I have implemented almost all the functions of react/hook-use-state, but the logic is simpler

I checked the source code of hook-use-state, it is very complicated, and I can't modify it

I hope react/hook-use-state can also achieve this function by someone

eslint-plugin-react version

v7.35.0

eslint version

v9.9.1

node version

v20

Contributor Guide

Research-Richtung
Untersuche den Quellcode der `react/hook use state` Regel in eslint plugin react. Verstehe, wie die Regel mehrere useState Aufrufe erkennt und Korrekturvorschläge generiert. Die aktuelle Korrektur ersetzt nur die erste Variable. Um dies zu beheben, musst du alle Verweise auf die destrukturierten Variablen (z. B. `[count, setCount]`) verfolgen und aktualisieren, wenn die Korrektur den einzelnen Aufruf durch mehrere Aufrufe ersetzt. Analysiere die AST Struktur und die Fixer API. Implementiere eine Logik, um alle referenzierten Bezeichner zu sammeln und sicherzustellen, dass sie entsprechend aktualisiert werden.
Tech Stack
javascriptreact
Domain
frontend
Issue Type
Fehler
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.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptESLint plugin development
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
35