Welcome画面関連：小さな修正リスト #2927
Description
内容
変更量が小さい複数の修正のリストissueです。
１つ１つ変更してPRするのもよし、ある程度まとめるも良し。
修正項目
ファイル名・命名関連
-
ファイル名の誤字
src/domain/defaultEngine/latetDefaultEngine.tsをlatestDefaultEngine.tsにリネームする- ref: feat: Welcome画面を追加 #2866 (comment)
-
RemoteとLocalの命名がわかりにくい
- LatestとCurrentのほうがわかりやすいかもしれない
- ref: feat: Welcome画面を追加 #2866 (comment)
docstring・コメント関連
-
launch関数のdocstring
src/backend/electron/appStateController.ts:32- 関数が担当する責務範囲のdocstringを追加する
- launchするだけなら一旦launch専用の関数名で良さそう
- ref: feat: Welcome画面を追加 #2866 (comment)
-
ipc.tsの設計思想のdocstring
src/backend/electron/ipc.ts:21-30- 何を想定しているか書いておく（windowの切り替えに対応、windowが複数同時に起動していないこと等）
- ref: feat: Welcome画面を追加 #2866 (comment)
ロジック移動・整理
-
defaultEngineExistsの取得ロジック移動
src/backend/electron/appStateController.ts:47- AppState管理とは話が別なので、どこかに移動する
- ref: feat: Welcome画面を追加 #2866 (comment)
-
vueからbackend/electronをimportしている問題
src/welcome/components/App.vueからbackend/electron/engineAndVvppControllerを直接importしている- vueコンポーネントはbackend/electronを直接参照すべきではない
- ref: feat: Welcome画面を追加 #2866 (comment)
型安全性・コード品質
-
ipc.tsのas stringまみれ
src/backend/electron/ipc.ts:44- 型アサーションを減らし、型安全性を改善する
- ref: feat: Welcome画面を追加 #2866 (comment)
-
ipc.tsのunreachableの使用
src/backend/electron/ipc.ts:53-65- 型的にunreachableならunreachableかassertNonNullish的なのにすべき
- ref: feat: Welcome画面を追加 #2866 (comment)
-
ipc.tsのpushじゃなくてset
src/backend/electron/ipc.ts:45-47- setでも良いか検討する
- ref: feat: Welcome画面を追加 #2866 (comment)
-
BaseIpcDataの目的明確化
src/backend/electron/ipcType.ts:235- 何のための型かわからない、ipc.tsからしか使われていない
- 目的をはっきりさせて、1箇所からしか使わないならprivateにする
- ref: feat: Welcome画面を追加 #2866 (comment)
コンポーネント関連
-
EngineCardのpropsをひとまとめ
src/welcome/components/EngineCard.vue:78-85- 下4つは全部remoteInfoがあるとき限定
- ref: feat: Welcome画面を追加 #2866 (comment)
-
表示条件と表示内容の不一致
src/welcome/components/EngineCard.vue:124-128- 「パッケージが選ばれてない」＝「読み込み中」がわかりづらい
- remoteがundefinedのときに読み込み中になるのが自然
- selectedPackageInfo側はcomputed内でassertNonNullableする
- ref: feat: Welcome画面を追加 #2866 (comment)
preload.ts関連
-
return voidの不一致
src/welcome/preload.ts:94- Main側は
voidになっている、揃えるべき - ref: feat: Welcome画面を追加 #2866 (comment)
-
直接呼び出しと?.の不一致
src/welcome/preload.ts:63-67- Main側は直接呼び出し、welcome側は
?. - 揃えるべき
- ref: feat: Welcome画面を追加 #2866 (comment)
Pros 良くなる点
- コードの一貫性が向上する
- 誤字が修正される
- 型安全性が向上する
Cons 悪くなる点
（なし）
実現方法
各修正項目に従ってコードを修正する。
VOICEVOXのバージョン
mainブランチ
その他
（参考）変更対象ファイル:
src/domain/defaultEngine/latetDefaultEngine.tssrc/backend/electron/appStateController.tssrc/backend/electron/ipc.tssrc/backend/electron/ipcType.tssrc/welcome/components/App.vuesrc/welcome/components/EngineCard.vuesrc/welcome/preload.ts