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

描述

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

贡献者指南

技术栈

javascripttypescriptnodejs

领域

toolingdeveloper experience

议题类型

bug

难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。

3

预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。

1-2 days

活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。

stale

清晰度议题是否清楚说明期望改动、验收标准和下一步。

clear

前置要求

ESLint rule developmentTypeScript type guardsAST

新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。

50

研究方向

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.