VVPPアンインストール時とディレクトリ見つからない時のエラーハンドリング追加

src/background.ts

@@ -803,50 +803,67 @@ function openEngineDirectory(engineId: string) {
 
   const engineDirectory = engineInfo.path;
   if (engineDirectory == null) {
-    return;
+    throw new Error(`engineDirectory is null: engineId == ${engineId}`);
   }
 
   // Windows環境だとスラッシュ区切りのパスが動かない。
   // path.resolveはWindowsだけバックスラッシュ区切りにしてくれるため、path.resolveを挟む。
   shell.openPath(path.resolve(engineDirectory));
 }
 
+/**
+ * VVPPエンジンをインストールする。
+ */
 async function installVvppEngine(vvppPath: string) {
   try {
     await vvppManager.install(vvppPath);
     return true;
   } catch (e) {
     dialog.showErrorBox(
-      "読み込みエラー",
-      `${vvppPath} を読み込めませんでした。`
+      "インストールエラー",
+      `${vvppPath} をインストールできませんでした。`
     );
-    log.error(`Failed to load ${vvppPath}, ${e}`);
+    log.error(`Failed to install ${vvppPath}, ${e}`);
     return false;
   }
 }
 
+/**
+ * VVPPエンジンをアンインストールする。
+ * 関数を呼んだタイミングでアンインストール処理を途中まで行い、アプリ終了時に完遂する。
+ */
 async function uninstallVvppEngine(engineId: string) {
   const engineInfos = fetchEngineInfos();
   const engineInfo = engineInfos.find(
     (engineInfo) => engineInfo.uuid === engineId
   );
-  if (!engineInfo) {
-    throw new Error(`No such engineInfo registered: engineId == ${engineId}`);
-  }
+  try {
+    if (!engineInfo) {
+      throw new Error(`No such engineInfo registered: engineId == ${engineId}`);
+    }
 
-  if (engineInfo.type !== "vvpp") {
-    throw new Error(`engineInfo.type is not vvpp: engineId == ${engineId}`);
-  }
+    if (engineInfo.type !== "vvpp") {
+      throw new Error(`engineInfo.type is not vvpp: engineId == ${engineId}`);
+    }
 
-  const engineDirectory = engineInfo.path;
-  if (engineDirectory == null) {
-    throw new Error("engineDirectory == null");
-  }
+    const engineDirectory = engineInfo.path;
+    if (engineDirectory == null) {
+      throw new Error("engineDirectory == null");
+    }
 
-  // Windows環境だとエンジンを終了してから削除する必要がある。
-  // そのため、アプリの終了時に削除するようにする。
-  vvppManager.markWillDelete(engineId);
-  return true;
+    // Windows環境だとエンジンを終了してから削除する必要がある。
+    // そのため、アプリの終了時に削除するようにする。
+    vvppManager.markWillDelete(engineId);
+    return true;
+  } catch (e) {
+    const engineName = engineInfo?.name ?? engineId;
+    dialog.showErrorBox(
+      "アンインストールエラー",
+      `${engineName} をアンインストールできませんでした。`
+    );
+    log.error(`Failed to uninstall ${engineId}, ${e}`);
+    return false;
+  }
 }
 
 // ディレクトリがエンジンとして正しいかどうかを判定する

src/components/EngineManageDialog.vue

@@ -492,7 +492,6 @@ export default defineComponent({
           "addingEngine",
           store.dispatch("INSTALL_VVPP_ENGINE", vvppFilePath.value)
         );
-        uiLockedState.value = null;
         if (success) {
           requireRestart(
             "エンジンを追加しました。反映には再起動が必要です。今すぐ再起動しますか？"
@@ -531,15 +530,18 @@ export default defineComponent({
             );
             break;
           }
-          case "vvpp":
-            await lockUi(
+          case "vvpp": {
+            const success = await lockUi(
               "deletingEngine",
               store.dispatch("UNINSTALL_VVPP_ENGINE", selectedId.value)
             );
-            requireRestart(
-              "エンジンの削除には再起動が必要です。今すぐ再起動しますか？"
-            );
+            if (success) {
+              requireRestart(
+                "エンジンの削除には再起動が必要です。今すぐ再起動しますか？"
+              );
+            }
             break;
+          }
           default:
             throw new Error("assert engineInfos[selectedId.value].type");
         }