installer: clean platform-cache in finally so failures don't leak

Previous version only deleted platform-cache at the very end of the
success path. If anything between platform install and launcher
profile update failed, the cache jar stuck around. Move the rm into
a finally block so the directory is always cleaned up.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

package.json

@@ -1,6 +1,6 @@
 {
   "name": "minecraft-music-quiz-installer",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "마인크래프트 음악퀴즈 간편설치기 + 관리 사이트",
   "main": "dist/installer/main.js",
   "scripts": {

src/installer-rp/ffmpeg.ts

@@ -3,7 +3,7 @@ import { promises as fs, createWriteStream, constants as fsConst } from 'node:fs
 import path from 'node:path'
 import https from 'node:https'
 import http from 'node:http'
-import { getMcCustomDir } from '../shared/paths.js'
+import { getMcCustomDir, getMcCustomInstallerDir } from '../shared/paths.js'
 import { loadComponentI18n } from '../shared/i18n.js'
 
 const { t } = loadComponentI18n('installer-rp')
@@ -13,10 +13,30 @@ const extractZip: (source: string, options: { dir: string }) => Promise<void> =
 
 /**
  * 리소스팩 간편설치기는 Windows .exe 로 배포되므로 ffmpeg.exe 한 종류만 사용.
- * 경로: %appdata%/.mc_custom/ffmpeg.exe
+ * 경로: %appdata%/.mc_custom/installer/ffmpeg.exe
  */
 export function getFfmpegExePath(): string {
-  return path.join(getMcCustomDir(), 'ffmpeg.exe')
+  return path.join(getMcCustomInstallerDir(), 'ffmpeg.exe')
+}
+
+/**
+ * 0.2.1 이전 버전이 `.mc_custom/ffmpeg.exe` 에 받아둔 파일이 있으면 새 위치로
+ * 옮긴다.
+ */
+async function migrateLegacyExe(target: string): Promise<void> {
+  const legacy = path.join(getMcCustomDir(), 'ffmpeg.exe')
+  if (legacy === target) return
+  try {
+    await fs.access(legacy, fsConst.F_OK)
+  } catch {
+    return
+  }
+  try {
+    await fs.mkdir(path.dirname(target), { recursive: true })
+    await fs.rename(legacy, target)
+  } catch {
+    try { await fs.unlink(legacy) } catch { /* noop */ }
+  }
 }
 
 /** BtbN/FFmpeg-Builds 의 win64-gpl 빌드. zip 내부에 bin/ffmpeg.exe 가 들어 있음. */
@@ -33,6 +53,7 @@ export async function ensureFfmpegExe(
   log?: (line: string) => void
 ): Promise<string> {
   const target = getFfmpegExePath()
+  await migrateLegacyExe(target)
   if (await canExecute(target)) {
     log?.(t('log.ffmpegExists', { path: target }))
     return target
@@ -40,7 +61,7 @@ export async function ensureFfmpegExe(
   if (installPromise) return installPromise
 
   installPromise = (async () => {
-    const dir = getMcCustomDir()
+    const dir = getMcCustomInstallerDir()
     const zipPath = path.join(dir, '.tmp_ffmpeg.zip')
     const extractDir = path.join(dir, '.tmp_ffmpeg')
     try {

src/installer-rp/ytdlp.ts

@@ -3,17 +3,38 @@ import { promises as fs, createWriteStream, constants as fsConst } from 'node:fs
 import path from 'node:path'
 import https from 'node:https'
 import http from 'node:http'
-import { getMcCustomDir } from '../shared/paths.js'
+import { getMcCustomDir, getMcCustomInstallerDir } from '../shared/paths.js'
 import { loadComponentI18n } from '../shared/i18n.js'
 
 const { t } = loadComponentI18n('installer-rp')
 
 /**
  * 리소스팩 간편설치기는 Windows .exe 로 배포되므로 yt-dlp.exe 한 종류만 사용.
- * 경로: %appdata%/.mc_custom/yt-dlp.exe
+ * 경로: %appdata%/.mc_custom/installer/yt-dlp.exe
  */
 export function getYtDlpExePath(): string {
-  return path.join(getMcCustomDir(), 'yt-dlp.exe')
+  return path.join(getMcCustomInstallerDir(), 'yt-dlp.exe')
+}
+
+/**
+ * 0.2.1 이전 버전이 `.mc_custom/yt-dlp.exe` 에 받아둔 파일이 있으면 새 위치로
+ * 옮긴다. 마인크래프트 게임 폴더 루트가 외부 도구 파일로 더럽혀지지 않도록.
+ */
+async function migrateLegacyExe(target: string): Promise<void> {
+  const legacy = path.join(getMcCustomDir(), 'yt-dlp.exe')
+  if (legacy === target) return
+  try {
+    await fs.access(legacy, fsConst.F_OK)
+  } catch {
+    return
+  }
+  try {
+    await fs.mkdir(path.dirname(target), { recursive: true })
+    await fs.rename(legacy, target)
+  } catch {
+    // 권한·드라이브 문제 등으로 실패하면 그냥 새로 받으면 되므로 무시.
+    try { await fs.unlink(legacy) } catch { /* noop */ }
+  }
 }
 
 const YT_DLP_DOWNLOAD_URL =
@@ -29,6 +50,7 @@ export async function ensureYtDlpExe(
   log?: (line: string) => void
 ): Promise<string> {
   const target = getYtDlpExePath()
+  await migrateLegacyExe(target)
   if (await canExecute(target)) {
     log?.(t('log.ytdlpExists', { path: target }))
     return target

src/installer/main.ts

@@ -1128,41 +1128,49 @@ ipcMain.handle('client:install', async (_event, payload: ClientInstallPayload) =
   await fsp.mkdir(path.join(customRoot, 'mods'), { recursive: true })
   await fsp.mkdir(path.join(customRoot, 'resourcepacks'), { recursive: true })
 
-  // 사용자가 기존 .minecraft 에 저장해둔 설정(options.txt, servers.dat 등)을
-  // .mc_custom 으로 가져온다. 이미 있는 파일은 보존.
-  await copyMinecraftUserSettings(customRoot)
+  try {
+    // 사용자가 기존 .minecraft 에 저장해둔 설정(options.txt, servers.dat 등)을
+    // .mc_custom 으로 가져온다. 이미 있는 파일은 보존.
+    await copyMinecraftUserSettings(customRoot)
 
-  if (payload.installPlatform && pack.pack.platform.type === 'fabric') {
-    await installFabricLoader(pack.pack, customRoot)
-  } else if (payload.installPlatform && pack.pack.platform.type !== 'vanilla' && pack.pack.platform.downloadUrl) {
-    const platformUrl = resolveManifestRelative(pack.pack.platform.downloadUrl, 'platforms')
-    const cacheDir = path.join(customRoot, 'platform-cache')
-    await fsp.mkdir(cacheDir, { recursive: true })
-    const installerPath = path.join(cacheDir, deriveFileName(platformUrl) || 'platform-installer.jar')
-    sendLog(t('log.platformDownload', { type: pack.pack.platform.type, url: platformUrl }))
-    await downloadFile(platformUrl, installerPath)
-    sendLog(t('log.platformSaved', { path: installerPath }))
-  } else if (!payload.installPlatform) {
-    sendLog(t('log.platformSkipped'))
+    if (payload.installPlatform && pack.pack.platform.type === 'fabric') {
+      await installFabricLoader(pack.pack, customRoot)
+    } else if (payload.installPlatform && pack.pack.platform.type !== 'vanilla' && pack.pack.platform.downloadUrl) {
+      const platformUrl = resolveManifestRelative(pack.pack.platform.downloadUrl, 'platforms')
+      const cacheDir = path.join(customRoot, 'platform-cache')
+      await fsp.mkdir(cacheDir, { recursive: true })
+      const installerPath = path.join(cacheDir, deriveFileName(platformUrl) || 'platform-installer.jar')
+      sendLog(t('log.platformDownload', { type: pack.pack.platform.type, url: platformUrl }))
+      await downloadFile(platformUrl, installerPath)
+      sendLog(t('log.platformSaved', { path: installerPath }))
+    } else if (!payload.installPlatform) {
+      sendLog(t('log.platformSkipped'))
+    }
+
+    await downloadModsFolder(pack.pack, customRoot)
+    await downloadResourcepackZip(pack.pack, customRoot)
+
+    if (payload.skipMap) {
+      // 참가자 모드: 이전 설치 흐름에서 설치러가 풀어둔 맵이 있다면 제거한다.
+      // 사용자가 직접 만든 월드는 마커에 포함되지 않으므로 그대로 보존된다.
+      await cleanupInstallerMap(customRoot)
+      sendLog(t('log.skipMapZip'))
+    } else {
+      await downloadMapZip(pack.pack, customRoot)
+    }
+
+    // 런처가 .mc_custom 을 gameDir 로 잡아도 assets/libraries/versions 를
+    // 찾을 수 있도록 .minecraft 의 해당 폴더로 junction 링크.
+    await linkMinecraftRuntimeDirs(customRoot)
+
+    await updateLauncherProfile(pack.pack, customRoot)
+  } finally {
+    // 설치가 끝나면(또는 실패해도) 더 이상 필요 없는 platform-cache(다운받은
+    // fabric/forge/neoforge installer jar 캐시)를 삭제한다. 다음 실행에서 다시
+    // 받으면 되고, 남겨두면 사용자 .mc_custom 폴더만 차지한다. 실패 경로에서도
+    // 정리되도록 finally 에 둔다.
+    await fsp.rm(path.join(customRoot, 'platform-cache'), { recursive: true, force: true }).catch(() => {})
   }
-
-  await downloadModsFolder(pack.pack, customRoot)
-  await downloadResourcepackZip(pack.pack, customRoot)
-
-  if (payload.skipMap) {
-    // 참가자 모드: 이전 설치 흐름에서 설치러가 풀어둔 맵이 있다면 제거한다.
-    // 사용자가 직접 만든 월드는 마커에 포함되지 않으므로 그대로 보존된다.
-    await cleanupInstallerMap(customRoot)
-    sendLog(t('log.skipMapZip'))
-  } else {
-    await downloadMapZip(pack.pack, customRoot)
-  }
-
-  // 런처가 .mc_custom 을 gameDir 로 잡아도 assets/libraries/versions 를
-  // 찾을 수 있도록 .minecraft 의 해당 폴더로 junction 링크.
-  await linkMinecraftRuntimeDirs(customRoot)
-
-  await updateLauncherProfile(pack.pack, customRoot)
 })
 
 interface FabricInstallerMeta {

src/shared/paths.ts

@@ -32,3 +32,13 @@ export function getAppDataDir(): string {
 export function getMcCustomDir(): string {
   return path.join(getAppDataDir(), '.mc_custom')
 }
+
+/**
+ * %appdata%/.mc_custom/installer — 설치기가 자체적으로 다운로드해 사용하는
+ * 외부 바이너리(yt-dlp.exe, ffmpeg.exe 등) 보관 위치. .mc_custom 루트가
+ * 마인크래프트 게임 폴더(`mods/`, `resourcepacks/`, `saves/` 등)와 섞이지
+ * 않도록 별도 하위 폴더에 둔다.
+ */
+export function getMcCustomInstallerDir(): string {
+  return path.join(getMcCustomDir(), 'installer')
+}