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

Métricas do repositório

Stars

5.022 estrelas (5.022 estrelas)

Métricas de merge de PR

Mesclagem média 1d 16h (Mesclagem média 1d 16h)399 fundiu PRs em 30d (399 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa

Examine o código fonte da regra consistent destructuring em rules/consistent destructuring.js. Atualmente, a regra desestrutura incorretamente propriedades ao usar a guarda de tipo 'in'. A correção deve detectar quando o lado direito do operador 'in' é um parâmetro de um padrão de desestruturação e evitar desestruturar essa propriedade. O link do playground fornece uma reprodução clara. Verifique também os testes da regra para garantir que a correção cubra casos extremos.

Pilha de tecnologia

javascripttypescriptnodejs

Domain

toolingdeveloper experience

Tipo Issue

Bug

DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.

3

Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.

1-2 dias

Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.

Antigo

ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.

Claro

Prerequisites

ESLint rule developmentTypeScript type guardsAST

Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.

50