[consistent-destructuring] incorrect fix for ts `in` type guard · sindresorhus/eslint-plugin-unicorn

(1 条评论) (1 个反应) (0 位负责人)JavaScript (468 个派生)user submission

bughelp wanted

仓库指标

星标: (5,022 个星标)
PR 合并指标: (平均合并 1天 16小时) (30 天内合并 399 个 PR)

描述

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

贡献者指南

研究方向: 检查 rules/consistent destructuring.js 中的 consistent destructuring 规则源码。该规则当前在使用 'in' 类型保护时错误地解构属性。修复应检测 'in' 运算符的右侧是否为解构模式的参数，并避免解构该属性。playground 链接提供了清晰的复现。同时检查该规则的测试，确保修复覆盖边缘情况。
技术栈: javascripttypescriptnodejs
领域: toolingdeveloper experience
议题类型: 缺陷
难度: 3
预计时间: 1-2 天
活动状态: 较久未更新
清晰度: 清晰
前置要求: ESLint rule developmentTypeScript type guardsAST
新手友好度: 50