sindresorhus/eslint-plugin-unicorn

Rule proposal: No computed object property existence check

Chiusa

#1096 aperta il 5 feb 2021

Vedi su GitHubGuida contributor
 (2 commenti) (2 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
help wantednew rule

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 4h 30m) (26 PR mergiate in 30 g)

Descrizione

If property name is a variable, this may leads to unexpected result.

Fail

if (foo[bar]) {}

if (bar in foo) {}

In these cases if bar is 'toString', 'constructor' etc, it will be true.

Pass

if (foo[bar] === true) {}

if (Object.prototype.hasOwnProperty.call(foo, bar)) {}

Problem: foo maybe an array, static check can't know.

Idea from https://github.com/prettier/prettier/pull/10256

Guida contributor

Direzione di ricerca
Esamina l'API delle regole personalizzate di ESLint, quindi implementa una regola che rilevi i controlli di esistenza delle proprietà calcolate con variabili. Usa i selettori AST per trovare MemberExpression (computed: true) dove la proprietà è un Identifier, e anche BinaryExpression con operatore 'in' dove il lato sinistro è un Identifier. Scrivi test con casi di fallimento e successo.
Tech stack
javascriptnodejs
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptESLintGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70