sindresorhus/eslint-plugin-unicorn

[consistent-destructuring] incorrect fix for ts `in` type guard

Open

#1 112 ouverte le 18 févr. 2021

Voir sur GitHub
 (1 commentaire) (1 réaction) (0 assignés)JavaScript (5 022 stars) (468 forks)user submission
bughelp wanted

Description

const { a }: { a: string } & ({ b: number } | { c: number } | {}) = params
const value = 'b' in params ? params.b : 'c' in params ? params.c : null

// incorrect fixed
const { a, b, c }: { a: string } & ({ b: number } | { c: number } | {}) = params
const value = 'b' in params ? b : 'c' in params ? c : null

Reference: https://www.typescriptlang.org/docs/handbook/advanced-types.html#type-guards-and-differentiating-types

The fixed codes is not runnable anymore. Playground

Guide contributeur

Stack technique
javascripttypescriptnodejs
Domaine
toolingdeveloper experience
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
ESLint rule developmentTypeScript type guardsAST
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50
Direction de recherche
Examine the consistent destructuring rule source in rules/consistent destructuring.js. The rule currently incorrectly destructures properties when using the 'in' type guard. The fix should detect when the right hand side of 'in' operator is a parameter of a destructuring pattern and avoid destructuring that property. The playground link provides a clear reproduction. Also check the rule's tests to ensure the fix covers edge cases.