`param: { foo: T['foo']; bar: T['bar']; }` を `Pick<T, 'foo' | 'bar'>`にしたい

[KisaragiEffective]

目が滑る。 例えばpartialUser: { id: MiUser['id'] }は頻出だがこれをpartialUser: Pick<MiUser, 'id'>と書けるとより意図が明確になる:

- partialUser: { id: MiUser['id'] }
+ partialUser: Pick<MiUser, 'id'>

{ id: MiUser['id']; host: MiUser['host']; } などのパターンでは更に明確になる:

- partialUser: { id: MiUser['id']; host: MiUser['host']; }
+ partialUser: Pick<MiUser, 'id' | 'host'>

[anatawa12]

pickしたいとも限らない気もする。

idがuser idであるって明記したいってのも結構ある。

partialであることが明確ならそうでいいと思うけどidだけだとそうとも限らない気がする

[KisaragiEffective]

これを書いてから考えてたけど字句的にpickかどうかを判別するのが不可能なのはそうだし、1つか2つなら書き換えのコストの割にしょうもないなという気がしてきた

[KisaragiEffective]

(?:\s*(\w+):\s*\w+\['\1'\][,;]?){3} を 74c93fcebe1b6fde489470e19808389d13f07a05 で探すと15件ヒットしたので、そこだけ書き換えたいかも

[tai-cha]

より深いネストとかが生まれることもあるので結局pickできないものが出てくると混ざって読みにくいものも出てくるため機械的なコーディングルールのレベルでどうこうという話でもない気がする