-
Notifications
You must be signed in to change notification settings - Fork 350
Open
Description
内容
welcomeIpcMainHandle.tsの不適切なフォールバックをassertに置き換えたい。
背景
PR #2866 でAIが書いたコードに、不適切なフォールバックが含まれている。フォールバックがあると異常な操作をしてしまう可能性がある。
修正項目
-
line 37のフォールバック
- targetにしてないのが割り当たる可能性がある
- assertNonNullable的なのが良い
-
lines 40-42
- Unreachableならunreachableにする
- Unreachableじゃない可能性があるならエラーハンドリングを書くかTODOコメント
-
lines 26-28
- ここもUnreachableかどうか確認が必要
-
注意: 他にも同様のフォールバックがあるかもしれないので、ファイル全体を確認する
ref: #2866 (comment)
ref: #2866 (comment)
ref: #2866 (comment)
Pros 良くなる点
- 不正な状態を早期に検出できる
- バグが発見しやすくなる
Cons 悪くなる点
(なし)
実現方法
フォールバックをassertNonNullableまたはunreachableに置き換える。
VOICEVOXのバージョン
mainブランチ
OSの種類/ディストリ/バージョン
- Windows
- macOS
- Linux
その他
(参考)変更対象ファイル:
src/backend/electron/welcomeIpcMainHandle.ts
Reactions are currently unavailable