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

Métriques du dépôt

Stars

5 022 étoiles (5 022 étoiles)

Métriques de merge PR

Merge moyen 4h 30m (Merge moyen 4h 30m)26 PRs mergées en 30 j (26 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche

Examinez le code source de la règle consistent destructuring dans rules/consistent destructuring.js. Actuellement, la règle déstructure incorrectement les propriétés lors de l'utilisation du garde de type 'in'. La correction doit détecter lorsque le côté droit de l'opérateur 'in' est un paramètre d'un motif de déstructuration et éviter de déstructurer cette propriété. Le lien du playground fournit une reproduction claire. Vérifiez également les tests de la règle pour vous assurer que la correction couvre les cas limites.

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 jours

Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.

Ancienne

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

Claire

Prérequis

ESLint rule developmentTypeScript type guardsAST

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

50