no-array-index-key not triggered when spreading props · jsx-eslint/eslint-plugin-react#1625

(2 commentaires) (0 réactions) (0 assignés)JavaScript (2 797 forks)batch import

enhancementhelp wanted

Métriques du dépôt

There's no warning or error when doing something like this:

people.map(({ name, email }, key) => {
  const props = {
    key,
    name,
    email,
  };

  return <Person {...props} />;
});

I realize this is a bit of an edge case, but maybe we can find a way to catch these uses as well.

Direction de recherche: Le problème concerne la règle `no array index key` qui ne se déclenche pas lorsque la clé est propagée via les props. Pour corriger cela, examinez l'implémentation de la règle dans `lib/rules/no array index key.js`. La règle vérifie actuellement la prop `key` directement sur les éléments JSX, mais ne détecte pas la propagation `{...props}`. Étudiez comment d'autres règles gèrent la propagation des props et envisagez d'ajouter un helper pour rechercher `key` dans les expressions d'objet utilisées dans les attributs de propagation JSX. Vérifiez également le fichier de test de la règle (`tests/lib/rules/no array index key.js`) pour les cas de test existants et ajoutez en de nouveaux couvrant les scénarios de propagation.
Stack technique: javascriptreact
Domaine: frontend
Type d'issue: Bug
Difficulté: 3
Temps estimé: 1-3 heures
Statut d'activité: Active
Clarté: Claire
Prérequis: JavaScriptReactESLint
Accessibilité débutant: 50