getTextAssetをbackendに実装せず、直接importする

[Hiroshiba]

内容

プライバシーポリシーの取得などは、現状electronのメインプロセスで読み込んだものをフロントエンドに渡しています。

window.backendからメインプロセスに問い合わせてるのがここで、

voicevox/src/backend/electron/preload.ts

Lines 26 to 30 in 41d6666

	getTextAsset: (textType) => {
	return ipcRendererInvokeProxy.GET_TEXT_ASSET(textType) as Promise<
	TextAsset[typeof textType]
	>;
	},

フロントからwindow.backendにリクエスト投げてるのがここです。

voicevox/src/store/index.ts

Lines 154 to 200 in 41d6666

	GET_HOW_TO_USE_TEXT: {
	async action() {
	return await window.backend.getTextAsset("HowToUse");
	},
	},
	GET_CONTACT_TEXT: {
	async action() {
	return await window.backend.getTextAsset("Contact");
	},
	},
	GET_Q_AND_A_TEXT: {
	async action() {
	return await window.backend.getTextAsset("QAndA");
	},
	},
	GET_POLICY_TEXT: {
	async action() {
	return await window.backend.getTextAsset("PrivacyPolicy");
	},
	},
	GET_OSS_LICENSES: {
	async action() {
	return await window.backend.getTextAsset("OssLicenses");
	},
	},
	GET_UPDATE_INFOS: {
	async action() {
	return await window.backend.getTextAsset("UpdateInfos");
	},
	},
	GET_OSS_COMMUNITY_INFOS: {
	async action() {
	return await window.backend.getTextAsset("OssCommunityInfos");
	},
	},
	GET_PRIVACY_POLICY_TEXT: {
	async action() {
	return await window.backend.getTextAsset("PrivacyPolicy");
	},
	},

最近viteに詳しくなってわかったのですが、こんなことせず普通にファイルをimportすれば手っ取り早いはずです。
window.backendからgetTextAssetを、ipc通信からGET_TEXT_ASSETを削除できるはず。

Pros 良くなる点

コードが減る

実現方法

importすればOKです。
ブラウザ版がちょうどそういう実装をしているので、これをstore/index.ts内に直接書けば良いはず！

voicevox/src/backend/browser/sandbox.ts

Lines 68 to 73 in 41d6666

	const fileName = AssetTextFileNames[textType];
	const v = await fetch(toStaticPath(fileName));
	if (textType === "OssLicenses" || textType === "UpdateInfos") {
	return v.json();
	}
	return v.text();

その他

結構簡単にできつつ、かつviteの使い方の勉強にもなるので初学者におすすめです。

[Hiroshiba]

@sabonerune さん辺りご興味ありませんか？
viteのちょっとした勉強になるのと、以前たしかこのあたりのリファクタリングに取り組んでくださったと思うので地の利があるかなーと思いメンションしてみました！

[sevenc-nanashi]

先読み：import.meta.globをeagerモードで動かしたほうが楽そうな気はちょっとします。
特定のフォルダ内におけばgetTextAssetsできる、みたいにできるはず？

[Hiroshiba]

たぶんこんな感じでimportできるはず！！（これはmock用のアセットを取得する例です）
https://github.com/VOICEVOX/voicevox/pull/2539/files?w=1