fix(rp): 종료 시 임시 파일 정리 보장 (실패→재시도 중 닫아도 삭제)

기존 window-all-closed 는 fsp.rm 을 fire-and-forget 로 부르고 곧바로 app.quit()
하여 삭제 완료 전에 프로세스가 죽었고, '재시도 중' 에는 yt-dlp/ffmpeg 자식이 파일을
잠가 삭제가 실패했다. before-quit 에서 종료를 미루고 자식을 먼저 종료해 잠금을 푼 뒤,
.temp 를 재시도 삭제(잠금 해제 대기)한 다음 실제 종료하도록 변경. 0.4.10→0.4.11.

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

docs/installer.md

@@ -28,10 +28,14 @@
 
 ### 3-2. JDK 확인
 
-- 각 음악퀴즈(pack)는 사이트 편집기에서 **권장 JDK 버전**(예: Java 25)을 지정합니다. 설치기는 이 권장 버전을 기준으로 동작합니다.
-- 탐색 순서: **① 권장 JDK 자동 설치 위치(`%APPDATA%\jdk\temurin-<권장>`) 를 가장 먼저** 확인 → 없으면 **② 컴퓨터의 JDK(`JAVA_HOME` / `JDK_HOME` / `C:\Program Files\Java`)** 를 확인.
-- 최신 마인크래프트 서버는 특정 자바 버전 이상을 요구합니다(예: 서버 번들러 class file 69.0 = Java 25). 그보다 낮은 자바는 있어도 `UnsupportedClassVersionError` 로 서버가 뜨지 않으므로, 자동 탐색은 **권장 버전 미만을 "없음"으로 처리**해 자동 설치로 유도합니다. 직접 선택/입력한 경로도 "다음" 에서 버전을 검증해 미만이면 진행을 막습니다.
-- **자동 설치** 버튼을 누르면 Adoptium Temurin(권장 버전) Windows x64 zip 을 받아 `%APPDATA%\jdk\temurin-<권장>\` 에 풀어 사용합니다.
+- 각 음악퀴즈(pack)는 사이트 편집기에서 **권장 JDK 버전**(예: Java 25)을 지정합니다. 편집기 목록은 Adoptium(Temurin)에서 실제 배포되는 버전을 불러오며, "자세히 보기"로 비-LTS·스냅샷까지 확인할 수 있습니다. 설치기는 이 권장 버전을 기준으로 동작합니다.
+- 탐색 순서:
+  1. **설치기 자동 설치 위치 `%APPDATA%\.mc_custom\jdk`**(구버전 `%APPDATA%\jdk` 호환)에 JDK 가 있으면 **가장 먼저 사용**합니다(권장 버전이면 그대로, 혹시 다른 버전이면 경고).
+  2. 없으면 **환경변수(`JAVA_HOME` / `JDK_HOME`)** 의 JDK 가 **권장 버전이면** 사용합니다.
+  3. 그래도 없으면 **기본 폴더 `C:\Program Files\Java`** 에서 **권장 버전**을 찾아 사용합니다.
+  4. 권장 버전을 어디서도 못 찾으면 **환경변수 자바(없으면 Program Files 자바)로 폴백**하고, "권장 버전과 달라 서버가 정상 실행되지 않을 수 있다"는 **경고**를 띄웁니다.
+- 직접 선택/입력한 경로도 "다음" 에서 검증해, 권장과 다르면 경고 후 동의(확인) 시에만 진행합니다(무조건 차단이 아니라 경고+동의). 아무 JDK 도 없으면 자동 설치로 유도합니다.
+- **자동 설치** 버튼을 누르면 Adoptium Temurin(권장 버전) Windows x64 zip 을 받아 **`%APPDATA%\.mc_custom\jdk\temurin-<권장>\`** 에 풀어 사용합니다. (`.mc_custom` 안에 두므로 "음악퀴즈 파일제거" 도구로 한 번에 정리됩니다. `MC_CUSTOM_DIR` 로 폴더명을 바꾼 경우 그 폴더 아래 `jdk`.)
 - 설치 중 같은 버튼이 "설치 취소" 로 바뀌고, 누르면 다운로드를 즉시 중단하고 부분 파일을 정리합니다.
 - 서버 zip 의 `run.bat` 이 시스템 PATH 의 `java` 를 그대로 쓰면 낡은 자바로 실행돼 실패할 수 있어, 설치기가 준비/선택한 JDK 의 `java` 를 쓰도록 `run.bat` 을 자동으로 수정합니다(자동 설치 JDK 는 `%APPDATA%` 전개형 경로라 한글 사용자명에도 안전).
 

package.json

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

src/installer-rp/main.ts

@@ -77,10 +77,10 @@ function pickMusicConcurrency(): number {
  */
 const MUSIC_START_STAGGER_MS = 2000
 
-/** 사진(썸네일) 다운로드 사이 최소 간격(ms). i.ytimg.com 429(rate limit) 를 유발하지 않도록 순차 요청을 살짝 벌린다. */
-const IMAGE_REQUEST_INTERVAL_MS = 800
-/** 음악(yt-dlp) 단계 직후엔 유튜브가 IP 를 일시 throttle 할 수 있어, 사진 단계 전 잠깐 쉬어 429 를 피한다. */
-const IMAGE_PHASE_COOLDOWN_MS = 4000
+/** 사진(썸네일) 동시 다운로드 상한. 순차 대신 병렬로 받아 속도를 높인다. 429 는 images.ts 의 지수 백오프 재시도가 흡수한다. */
+const IMAGE_CONCURRENCY_CAP = 8
+/** 음악(yt-dlp) 단계 직후 유튜브 IP throttle 이 남아 있을 수 있어, 사진 단계 전 아주 잠깐만 쉰다. */
+const IMAGE_PHASE_COOLDOWN_MS = 1000
 
 const sleep = (ms: number): Promise<void> => new Promise((r) => setTimeout(r, ms))
 
@@ -516,58 +516,72 @@ ipcMain.handle('rp:install:start', async (): Promise<{ resourcepackPath: string
       sendLog(t('log.imageCooldown', { secs: Math.round(IMAGE_PHASE_COOLDOWN_MS / 1000) }))
       await sleep(IMAGE_PHASE_COOLDOWN_MS)
     }
-    let imageRequests = 0
-    for (let i = 0; i < imageTotal; i++) {
-      throwIfCancelled()
-      const entry = pack.list.images[i]
-      const idx = i + 1
-      // 이전 시도에서 이미 정규화해둔 사진은 건너뛴다(이어받기).
-      const coverPath = path.join(paintingDir, coverFileName(idx))
-      if (await fileExists(coverPath)) {
-        sendLog(t('log.imageSkip', { idx }))
-        sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 100, status: 'done' })
-        continue
-      }
-      // 실제 네트워크 요청을 하는 사진들 사이에만 간격을 둔다(건너뛴 것 사이엔 대기 없음).
-      if (imageRequests > 0) await sleep(IMAGE_REQUEST_INTERVAL_MS + Math.floor(Math.random() * 300))
-      imageRequests++
-      sendLog(t('log.imageDownloading', { idx }))
-      sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 10, status: 'running' })
-      let buf: Buffer
-      try {
-        buf = await downloadImage(entry.url, (info) => {
-          const secs = Math.ceil(info.delayMs / 1000)
-          sendLog(t('log.imageRetry', { idx, attempt: info.attempt, code: info.code ?? '-', secs }))
-          sendProgress({
-            phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 10, status: 'running',
-            message: t('progress.imageRetry', { code: info.code ?? '-', secs })
+    // 여러 장을 동시에 받아 속도를 높인다(순차+딜레이 대신 워커 풀). 유튜브 429 는
+    // 각 다운로드의 지수 백오프 재시도(images.ts)가 흡수하고, 첫 하드 실패를 만나면
+    // 새 작업 배정을 멈춘 뒤 워커가 모두 끝나면 그 오류를 던진다.
+    const images = pack.list.images
+    let imageNext = 0
+    let imageError: Error | null = null
+    async function imageWorker(): Promise<void> {
+      while (true) {
+        if (state.cancelRequested || imageError) return
+        const i = imageNext++
+        if (i >= imageTotal) return
+        const idx = i + 1
+        const entry = images[i]
+        const coverPath = path.join(paintingDir, coverFileName(idx))
+        // 이전 시도에서 이미 정규화해둔 사진은 건너뛴다(이어받기).
+        if (await fileExists(coverPath)) {
+          sendLog(t('log.imageSkip', { idx }))
+          sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 100, status: 'done' })
+          continue
+        }
+        sendLog(t('log.imageDownloading', { idx }))
+        sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 10, status: 'running' })
+        let buf: Buffer
+        try {
+          buf = await downloadImage(entry.url, (info) => {
+            const secs = Math.ceil(info.delayMs / 1000)
+            sendLog(t('log.imageRetry', { idx, attempt: info.attempt, code: info.code ?? '-', secs }))
+            sendProgress({
+              phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 10, status: 'running',
+              message: t('progress.imageRetry', { code: info.code ?? '-', secs })
+            })
           })
-        })
-      } catch (err) {
-        // 부분 생성됐을 수 있는 커버 파일 제거(이어받기 시 완성본 오인 방지).
-        await fsp.rm(coverPath, { force: true }).catch(() => {})
-        const rawMsg = (err as Error).message
-        // 429(rate limit)는 유튜브가 IP 를 일시 차단한 것. 잠시 뒤 다시 시도하면
-        // 이미 받은 사진은 건너뛰고 이어받는다는 안내를 덧붙인다.
-        const msg = /429/.test(rawMsg) ? `${rawMsg} — ${t('errors.imageRateLimitHint')}` : rawMsg
-        sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 0, status: 'error', message: msg })
-        throw new Error(t('errors.imageDownloadFailed', { idx, message: msg }))
+        } catch (err) {
+          // 부분 생성됐을 수 있는 커버 파일 제거(이어받기 시 완성본 오인 방지).
+          await fsp.rm(coverPath, { force: true }).catch(() => {})
+          const rawMsg = (err as Error).message
+          // 429(rate limit)는 유튜브가 IP 를 일시 차단한 것. 잠시 뒤 다시 시도하면
+          // 이미 받은 사진은 건너뛰고 이어받는다는 안내를 덧붙인다.
+          const msg = /429/.test(rawMsg) ? `${rawMsg} — ${t('errors.imageRateLimitHint')}` : rawMsg
+          sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 0, status: 'error', message: msg })
+          if (!imageError) imageError = new Error(t('errors.imageDownloadFailed', { idx, message: msg }))
+          return
+        }
+        if (state.cancelRequested || imageError) return
+        sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 60, status: 'running' })
+        try {
+          await normalizeToCover(buf, coverPath)
+        } catch (err) {
+          // 변환 중 부분 생성된 PNG 제거(이어받기 시 완성본 오인 방지).
+          await fsp.rm(coverPath, { force: true }).catch(() => {})
+          sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 0, status: 'error', message: (err as Error).message })
+          if (!imageError) imageError = new Error(t('errors.imageNormalizeFailed', { idx, message: (err as Error).message }))
+          return
+        }
+        sendLog(t('log.imageDone', { idx, name: path.basename(coverPath) }))
+        sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 100, status: 'done' })
       }
-      throwIfCancelled()
-      sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 60, status: 'running' })
-      const outPath = coverPath
-      try {
-        await normalizeToCover(buf, outPath)
-      } catch (err) {
-        // 변환 중 부분 생성된 PNG 제거(이어받기 시 완성본 오인 방지).
-        await fsp.rm(coverPath, { force: true }).catch(() => {})
-        sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 0, status: 'error', message: (err as Error).message })
-        throw new Error(t('errors.imageNormalizeFailed', { idx, message: (err as Error).message }))
-      }
-      sendLog(t('log.imageDone', { idx, name: path.basename(outPath) }))
-      sendProgress({ phase: 'item', kind: 'image', index: idx, total: imageTotal, percent: 100, status: 'done' })
     }
 
+    const imageWorkerCount = Math.min(IMAGE_CONCURRENCY_CAP, Math.max(1, imageTotal), Math.max(concurrency, 4))
+    const imageWorkers: Promise<void>[] = []
+    for (let w = 0; w < imageWorkerCount; w++) imageWorkers.push(imageWorker())
+    await Promise.all(imageWorkers)
+    throwIfCancelled()
+    if (imageError) throw imageError
+
     // 2-4. 베이스 리소스팩 다운로드 (있을 때만)
     throwIfCancelled()
     let baseZipPath: string | undefined
@@ -688,8 +702,37 @@ app.whenReady().then(() => {
   })
 })
 
+// 종료 시 임시 파일(.temp) 정리. 예전엔 window-all-closed 에서 fsp.rm 을 fire-and-forget
+// 으로 호출하고 곧바로 app.quit() 해서, 삭제가 끝나기 전에 프로세스가 죽어 미리 받아둔
+// 내용이 남곤 했다. 또 '재시도 중' 이면 yt-dlp/ffmpeg 자식이 파일을 잠가 삭제가 실패한다.
+// → 자식을 먼저 죽여 잠금을 풀고, 삭제가 끝날 때까지 종료를 미룬 뒤 실제로 종료한다.
+let rpExitCleanupDone = false
+async function cleanupTempOnExit(): Promise<void> {
+  state.cancelRequested = true
+  for (const child of state.activeChildren) {
+    try { if (!child.killed) child.kill() } catch { /* noop */ }
+  }
+  const tempDir = path.join(getMcCustomDir(), '.temp')
+  // 자식 종료로 파일 잠금이 풀릴 시간을 주며 몇 번 재시도.
+  for (let attempt = 0; attempt < 3; attempt++) {
+    await new Promise((r) => setTimeout(r, attempt === 0 ? 200 : 400))
+    try {
+      await fsp.rm(tempDir, { recursive: true, force: true })
+      if (!fs.existsSync(tempDir)) return
+    } catch { /* 잠금 등 — 다음 시도 */ }
+  }
+}
+
+app.on('before-quit', (event) => {
+  if (rpExitCleanupDone) return
+  event.preventDefault()
+  void cleanupTempOnExit().finally(() => {
+    rpExitCleanupDone = true
+    app.quit()
+  })
+})
+
 app.on('window-all-closed', () => {
-  // 강제 종료 시에도 임시 파일은 정리.
-  fsp.rm(path.join(getMcCustomDir(), '.temp'), { recursive: true, force: true }).catch(() => {})
+  // app.quit() → before-quit 에서 임시 파일을 정리한 뒤 실제 종료.
   if (process.platform !== 'darwin') app.quit()
 })

src/installer/main.ts

@@ -334,42 +334,88 @@ function jdkHomesUnder(parent: string): string[] {
   return out
 }
 
-/**
- * 컴퓨터에 설치된 JDK 홈 목록을 우선순위대로 모은다(중복 제거).
- * 순서: ① 기본 자바 설치 위치 C:\Program Files\Java → ② 환경변수(JAVA_HOME/JDK_HOME)
- *       → ③ 설치기 자동 설치 위치(.mc_custom/jdk, 구버전 %APPDATA%/jdk 호환).
- */
-function collectJdkHomes(): string[] {
-  const homes: string[] = []
-  const push = (h: string): void => {
-    if (h && !homes.some((x) => x.toLowerCase() === h.toLowerCase())) homes.push(h)
+interface JdkEntry { home: string; major: number }
+
+/** 홈 목록 → {home, major} 목록(중복/버전0 제거). */
+function homesToEntries(homes: string[]): JdkEntry[] {
+  const javaName = process.platform === 'win32' ? 'java.exe' : 'java'
+  const seen = new Set<string>()
+  const out: JdkEntry[] = []
+  for (const home of homes) {
+    const key = home.toLowerCase()
+    if (seen.has(key)) continue
+    seen.add(key)
+    const major = getJavaMajor(path.join(home, 'bin', javaName))
+    if (major > 0) out.push({ home, major })
   }
-  for (const h of jdkHomesUnder('C:\\Program Files\\Java')) push(h)
-  for (const env of [process.env.JAVA_HOME, process.env.JDK_HOME]) {
-    if (env) push(resolveJdkHome(env))
-  }
-  for (const base of [installerJdkRoot(), path.join(getAppDataDir(), 'jdk')]) {
-    for (const h of jdkHomesUnder(base)) push(h)
-  }
-  return homes
+  return out
 }
 
-// 권장 JDK 를 우선 찾는다. C:\Program Files\Java → 환경변수 → 설치기 자동설치 위치를
-// 훑어, ① 권장 버전과 정확히 일치하는 JDK 가 있으면 그걸 쓰고(match=true),
-// ② 없으면 가장 높은 버전을 대신 고르되 "권장과 다름"(match=false)으로 표시해 렌더러가
-//    경고를 띄우게 한다. ③ 아무 JDK 도 없으면 not found → 자동 설치 유도.
+/** 설치기 자동 설치 위치(.mc_custom/jdk, 구버전 %APPDATA%/jdk 호환)의 JDK 들. */
+function mcCustomJdkEntries(): JdkEntry[] {
+  const homes: string[] = []
+  for (const base of [installerJdkRoot(), path.join(getAppDataDir(), 'jdk')]) {
+    homes.push(...jdkHomesUnder(base))
+  }
+  return homesToEntries(homes)
+}
+
+/** 환경변수(JAVA_HOME/JDK_HOME)에 등록된 JDK 들. */
+function envJdkEntries(): JdkEntry[] {
+  const homes: string[] = []
+  for (const env of [process.env.JAVA_HOME, process.env.JDK_HOME]) {
+    if (env) {
+      const h = resolveJdkHome(env)
+      if (h) homes.push(h)
+    }
+  }
+  return homesToEntries(homes)
+}
+
+/** 기본 자바 설치 위치 C:\Program Files\Java 아래의 JDK 들. */
+function programFilesJdkEntries(): JdkEntry[] {
+  return homesToEntries(jdkHomesUnder('C:\\Program Files\\Java'))
+}
+
+// JDK 탐색 우선순위(요청 사양):
+//   ① .mc_custom/jdk(설치기가 설치해둔 위치)가 있으면 먼저 사용 — 권장 버전이면 그대로,
+//      혹시 다른 버전이면 경고(match=false).
+//   ② 환경변수(JAVA_HOME/JDK_HOME) 가 권장 버전이면 사용.
+//   ③ 기본 폴더 C:\Program Files\Java 에 권장 버전이 있으면 사용.
+//   ④ 없으면 환경변수 자바로 폴백하고 "권장과 다름" 경고(match=false).
+//      (환경변수 자바도 없으면 Program Files 자바로 폴백, 그것도 없으면 not found → 자동 설치 유도.)
 ipcMain.handle('jdk:detect', async (_event, recommendedInput?: number) => {
   const required = normalizeRecommendedJdk(recommendedInput)
-  const javaName = process.platform === 'win32' ? 'java.exe' : 'java'
-  const found = collectJdkHomes()
-    .map((home) => ({ home, major: getJavaMajor(path.join(home, 'bin', javaName)) }))
-    .filter((x) => x.major > 0)
+  const recommended = (entries: JdkEntry[]): JdkEntry | undefined => entries.find((e) => e.major === required)
+  const best = (entries: JdkEntry[]): JdkEntry => entries.slice().sort((a, b) => b.major - a.major)[0]
 
-  const exact = found.find((x) => x.major === required)
-  if (exact) return { found: true, path: exact.home, major: required, match: true }
-  if (found.length > 0) {
-    const best = found.slice().sort((a, b) => b.major - a.major)[0]
-    return { found: true, path: best.home, major: best.major, match: false }
+  // ① .mc_custom/jdk 우선.
+  const mc = mcCustomJdkEntries()
+  if (mc.length > 0) {
+    const rec = recommended(mc)
+    if (rec) return { found: true, path: rec.home, major: required, match: true }
+    const b = best(mc)
+    return { found: true, path: b.home, major: b.major, match: false }
+  }
+
+  // ② 환경변수 JDK 가 권장 버전이면 사용.
+  const env = envJdkEntries()
+  const envRec = recommended(env)
+  if (envRec) return { found: true, path: envRec.home, major: required, match: true }
+
+  // ③ 기본 폴더(Program Files\Java)에 권장 버전이 있으면 사용.
+  const pf = programFilesJdkEntries()
+  const pfRec = recommended(pf)
+  if (pfRec) return { found: true, path: pfRec.home, major: required, match: true }
+
+  // ④ 폴백: 환경변수 자바 → (없으면) Program Files 자바 + 경고.
+  if (env.length > 0) {
+    const b = best(env)
+    return { found: true, path: b.home, major: b.major, match: false }
+  }
+  if (pf.length > 0) {
+    const b = best(pf)
+    return { found: true, path: b.home, major: b.major, match: false }
   }
   return { found: false, path: '', major: required, match: false }
 })

views/op/editor.ejs

@@ -175,16 +175,26 @@
         sel.value = String(current)
       }
 
+      function escHtml(s) {
+        return String(s).replace(/[&<>"']/g, function (c) {
+          return c === '&' ? '&amp;' : c === '<' ? '&lt;' : c === '>' ? '&gt;' : c === '"' ? '&quot;' : '&#39;'
+        })
+      }
+      function detailRow(label, names) {
+        return '<div><strong>' + escHtml(label) + ':</strong> ' +
+          names.slice(0, 6).map(escHtml).join(', ') + '</div>'
+      }
       function loadDetails(major) {
         details.hidden = false
         details.textContent = JDK.detailsLoading
         fetch('/op/jdk-versions/' + major)
           .then(function (r) { if (!r.ok) throw new Error('HTTP ' + r.status); return r.json() })
           .then(function (d) {
+            // release_name 은 외부 API 값이라 반드시 escape 후 삽입(관리 페이지 XSS 방지).
             var html = ''
-            if (d.ga && d.ga.length) html += '<div><strong>' + JDK.detailsGa + ':</strong> ' + d.ga.slice(0, 6).join(', ') + '</div>'
-            if (d.ea && d.ea.length) html += '<div><strong>' + JDK.detailsEa + ':</strong> ' + d.ea.slice(0, 6).join(', ') + '</div>'
-            details.innerHTML = html || JDK.detailsNone
+            if (d.ga && d.ga.length) html += detailRow(JDK.detailsGa, d.ga)
+            if (d.ea && d.ea.length) html += detailRow(JDK.detailsEa, d.ea)
+            details.innerHTML = html || escHtml(JDK.detailsNone)
           })
           .catch(function () { details.textContent = JDK.detailsNone })
       }