rp-pack: fail-fast on base track/painting collision (was: silent skip)

Reviewer correctly flagged that the previous skip-on-collision behavior silently drops new quiz tracks when the base resourcepack already has the same track_NN key. That makes the install LOOK successful but breaks the quiz at runtime (datapack references the missing track). The new behavior throws a clear error explaining which key collided and what the user must do (remove the conflicting base entry, or use a different base). The base assets are still preserved (we never overwrite); we just refuse to build a broken pack. Removed the now-unused skip-summary log keys.
rp-pack: never overwrite base resourcepack sounds/paintings (v0.3.5)
2026-05-23 17:26:41 +09:00 · 2026-05-23 17:18:46 +09:00 · 2026-05-20 10:29:06 +09:00 · 2026-05-20 10:22:02 +09:00 · 2026-05-20 10:20:49 +09:00 · 2026-05-20 10:14:42 +09:00
17 changed files with 530 additions and 137 deletions
--- a/installer-rp/renderer.js
+++ b/installer-rp/renderer.js
@@ -141,14 +141,55 @@ function renderStep1() {
 }

 // 약관 동의 페이지: 1단계 직후, 2단계 설치 진입 전에 노출.
-// rp 인스톨러는 리소스팩·설치기 두 약관만 확인·동의하면 된다.
+// v0.3.4~ : 사이트의 visibility 토글에 따라 표시할 약관이 결정된다. 명시적으로 빈 목록(terms:[])
+// 정상 응답일 때만 단계를 건너뛰고, 네트워크/서버 오류는 차단 후 다시 시도 UI를 보여준다.
 function renderAgreement() {
  setActiveStep(1)
  clearPage()
-  var KINDS = [
-    { id: 'resourcepack', tab: tt('agreement.tabResourcepack') },
-    { id: 'installer-rp', tab: tt('agreement.tabInstaller') }
-  ]
+  var loadingSection = document.createElement('section')
+  loadingSection.className = 'page'
+  loadingSection.innerHTML = '<h2>' + escapeHtml(tt('agreement.heading')) + '</h2>' +
+    '<p class="formMessage">' + escapeHtml(tt('agreement.loading')) + '</p>'
+  pageHost.appendChild(loadingSection)
+
+  api.getTermsList().then(function (res) {
+    if (!res || !res.ok) {
+      showAgreementError((res && res.message) || 'unknown')
+      return
+    }
+    var terms = (res.terms || []).map(function (t) {
+      return { id: t.kind, tab: t.label }
+    })
+    if (terms.length === 0) {
+      renderStep2()
+      return
+    }
+    clearPage()
+    renderAgreementWithKinds(terms)
+  }).catch(function (err) {
+    showAgreementError(err && err.message ? err.message : 'unknown')
+  })
+}
+
+// 약관 목록을 못 받아왔을 때: 사용자에게 오류 + 다시 시도 옵션. 동의 없이 설치 단계로
+// 자동 진입하지 않도록 next 버튼을 두지 않는다.
+function showAgreementError(message) {
+  clearPage()
+  var section = document.createElement('section')
+  section.className = 'page'
+  section.innerHTML =
+    '<h2>' + escapeHtml(tt('agreement.heading')) + '</h2>' +
+    '<p class="formMessage error">' + escapeHtml(tt('agreement.listLoadFailed', { message: message })) + '</p>' +
+    '<div class="actionRow">' +
+    '<button class="secondaryBtn" id="back">' + escapeHtml(tt('common.back')) + '</button>' +
+    '<button class="primaryBtn" id="retry">' + escapeHtml(tt('agreement.retry')) + '</button>' +
+    '</div>'
+  pageHost.appendChild(section)
+  section.querySelector('#back').addEventListener('click', renderStep1)
+  section.querySelector('#retry').addEventListener('click', renderAgreement)
+}
+
+function renderAgreementWithKinds(KINDS) {
  var section = document.createElement('section')
  section.className = 'page'
  section.innerHTML =
--- a/installer/renderer.js
+++ b/installer/renderer.js
@@ -149,15 +149,58 @@ function renderStep1() {
 }

 // 약관 동의 페이지: 음악퀴즈 선택 직후, 싱글/멀티 선택(step2) 진입 전에 노출.
-// 메인 설치기는 맵·모드·설치기 세 약관을 모두 확인·동의해야 다음 단계로 갈 수 있다.
+// v0.3.4~ : 어떤 약관을 표시할지는 사이트(/manifest/terms/<pack>/index.json) 가
+// 결정. 메인 인스톨러용으로 표시 토글된 항목만 받아 탭을 만든다. 목록이 비어 있는 (terms:[])
+// 정상 응답일 때만 단계 자체를 건너뛴다. 네트워크 오류/404/서버 오류는 사용자가 약관 동의
+// 없이 설치로 넘어가는 것을 막기 위해 오류 화면 + 다시 시도 버튼으로 차단한다.
 function renderAgreement() {
  setActiveStep(1)
  clearPage()
-  var KINDS = [
-    { id: 'map', tab: tt('agreement.tabMap') },
-    { id: 'mod', tab: tt('agreement.tabMod') },
-    { id: 'installer', tab: tt('agreement.tabInstaller') }
-  ]
+  var loadingSection = document.createElement('section')
+  loadingSection.className = 'page'
+  loadingSection.innerHTML = '<h2>' + tt('agreement.heading') + '</h2>' +
+    '<p class="formMessage">' + tt('agreement.loading') + '</p>'
+  pageHost.appendChild(loadingSection)
+
+  installerApi.getTermsList().then(function (res) {
+    if (!res || !res.ok) {
+      showAgreementError((res && res.message) || 'unknown')
+      return
+    }
+    var terms = (res.terms || []).map(function (t) {
+      return { id: t.kind, tab: t.label }
+    })
+    if (terms.length === 0) {
+      // 명시적으로 표시 대상이 0개라고 서버가 알려준 정상 응답 → 약관 단계 스킵.
+      renderStep2()
+      return
+    }
+    clearPage()
+    renderAgreementWithKinds(terms)
+  }).catch(function (err) {
+    showAgreementError(err && err.message ? err.message : 'unknown')
+  })
+}
+
+// 약관 목록을 못 받아왔을 때: 사용자에게 오류 + 다시 시도/뒤로 가기 옵션을 보여준다.
+// 동의 없이 설치 단계로 넘어가지 않도록 next 버튼을 두지 않는다.
+function showAgreementError(message) {
+  clearPage()
+  var section = document.createElement('section')
+  section.className = 'page'
+  section.innerHTML =
+    '<h2>' + tt('agreement.heading') + '</h2>' +
+    '<p class="formMessage error">' + tt('agreement.listLoadFailed', { message: message }) + '</p>' +
+    '<div class="actionRow">' +
+    '<button class="secondaryBtn" id="back">' + tt('common.back') + '</button>' +
+    '<button class="primaryBtn" id="retry">' + tt('agreement.retry') + '</button>' +
+    '</div>'
+  pageHost.appendChild(section)
+  section.querySelector('#back').addEventListener('click', renderStep1)
+  section.querySelector('#retry').addEventListener('click', renderAgreement)
+}
+
+function renderAgreementWithKinds(KINDS) {
  var section = document.createElement('section')
  section.className = 'page'
  section.innerHTML =
@@ -165,7 +208,7 @@ function renderAgreement() {
    '<p class="formMessage">' + tt('agreement.intro') + '</p>' +
    '<div class="tabBar" id="agTabs">' +
    KINDS.map(function (k, i) {
-      return '<button type="button" class="tabBtn' + (i === 0 ? ' active' : '') + '" data-ag="' + k.id + '">' + k.tab + '</button>'
+      return '<button type="button" class="tabBtn' + (i === 0 ? ' active' : '') + '" data-ag="' + escapeHtml(k.id) + '">' + escapeHtml(k.tab) + '</button>'
    }).join('') +
    '</div>' +
    '<div class="agreementBody" id="agBody">' + tt('agreement.loading') + '</div>' +
@@ -917,6 +960,12 @@ function renderStep5() {
  })
 }

+function escapeHtml(s) {
+  return String(s).replace(/[&<>"']/g, function (c) {
+    return c === '&' ? '&amp;' : c === '<' ? '&lt;' : c === '>' ? '&gt;' : c === '"' ? '&quot;' : '&#39;'
+  })
+}
+
 // 시작 진입점: 사전을 먼저 받아서 정적 텍스트 갱신 후 첫 페이지 렌더.
 ;(async function () {
  try {
--- a/locales/installer-rp/ko-kr.json
+++ b/locales/installer-rp/ko-kr.json
@@ -38,7 +38,9 @@
    "tabInstaller": "리소스팩 설치기 약관",
    "loading": "약관을 불러오는 중...",
    "loadFailed": "약관 로드 실패: {{message}}",
-    "agreeAll": "위 모든 약관(리소스팩·설치기)에 동의합니다.",
+    "listLoadFailed": "약관 표시에 실패하여 설치를 진행할 수 없습니다.\n사유: {{message}}\n네트워크 상태를 확인하고 다시 시도하거나, 처음 단계로 돌아가 주세요.",
+    "retry": "다시 시도",
+    "agreeAll": "위 모든 약관에 동의합니다.",
    "agreeRequired": "약관에 동의해야 다음 단계로 진행할 수 있습니다.",
    "cancelling": "취소 중…"
  },
@@ -104,6 +106,8 @@
    "packFormatFallback": "pack_format = {{format}} (mcVersion \"{{version}}\" 매칭 실패, 최신 폴백)",
    "packFormatRange": "호환 범위 선언: pack_format {{min}} ~ {{max}} (supported_formats / min_format / max_format 모두 기록)",
    "soundsMerged": "기존 sounds.json 병합 ({{count}}개 항목)",
+    "tracksAdded": "음악 트랙 추가됨: {{count}}곡",
+    "paintingsAdded": "사진 텍스처 추가됨: {{count}}장",
    "ytdlpLine": "yt-dlp> {{line}}"
  },
  "progress": {
@@ -137,6 +141,8 @@
    "ytdlpInstallFailed": "yt-dlp.exe 자동 설치 실패: {{message}}",
    "ffmpegNotInZip": "zip 내부에서 ffmpeg.exe 를 찾을 수 없습니다.",
    "ffmpegVerifyFailed": "ffmpeg.exe 다운로드는 됐지만 실행 검증에 실패했습니다.",
-    "ffmpegInstallFailed": "ffmpeg.exe 자동 설치 실패: {{message}}"
+    "ffmpegInstallFailed": "ffmpeg.exe 자동 설치 실패: {{message}}",
+    "baseTrackCollision": "베이스 리소스팩에 같은 트랙 ID 가 이미 있어 설치를 중단합니다: {{trackId}}\n베이스 자산을 보존하면서 새 트랙을 같은 ID 로 추가할 수 없습니다. 베이스의 sounds.json 엔트리/sounds 폴더에서 충돌하는 항목을 제거하거나 다른 베이스를 사용하세요.",
+    "basePaintingCollision": "베이스 리소스팩에 같은 사진 파일이 이미 있어 설치를 중단합니다: {{name}}\n베이스의 painting 텍스처를 보존하면서 같은 파일명을 추가할 수 없습니다. 베이스에서 충돌하는 파일을 제거하거나 다른 베이스를 사용하세요."
  }
 }
--- a/locales/installer/ko-kr.json
+++ b/locales/installer/ko-kr.json
@@ -38,7 +38,9 @@
    "tabInstaller": "설치기 약관",
    "loading": "약관을 불러오는 중...",
    "loadFailed": "약관 로드 실패: {{message}}",
-    "agreeAll": "위 모든 약관(맵·모드·설치기)에 동의합니다.",
+    "listLoadFailed": "약관 표시에 실패하여 설치를 진행할 수 없습니다.\n사유: {{message}}\n네트워크 상태를 확인하고 다시 시도하거나, 처음 단계로 돌아가 주세요.",
+    "retry": "다시 시도",
+    "agreeAll": "위 모든 약관에 동의합니다.",
    "agreeRequired": "약관에 동의해야 다음 단계로 진행할 수 있습니다."
  },
  "step1": {
--- a/locales/server/ko-kr.json
+++ b/locales/server/ko-kr.json
@@ -160,7 +160,11 @@
    "slashQuote": "인용",
    "slashCode": "코드",
    "leaveConfirm": "저장하지 않은 변경사항이 있습니다.\n저장 없이 이 페이지를 떠나시겠습니까?",
-    "builtinBadge": "기본",
+    "visibilityHeading": "표시 대상 (중복 선택 가능)",
+    "visibilityInstaller": "설치기에 표시",
+    "visibilityInstallerRp": "리소스팩 설치기에 표시",
+    "visibilityInstallerShort": "설치기",
+    "visibilityInstallerRpShort": "리소스팩",
    "addHeading": "약관 추가",
    "kindLabel": "식별자",
    "kindPlaceholder": "예: privacy",
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "minecraft-music-quiz-installer",
-  "version": "0.3.3",
+  "version": "0.3.5",
  "description": "마인크래프트 음악퀴즈 간편설치기 + 관리 사이트",
  "main": "dist/installer/main.js",
  "scripts": {
--- a/public/termsEditor.js
+++ b/public/termsEditor.js
@@ -15,6 +15,8 @@
  var dirtyMark = document.getElementById('dirty-mark')
  var saveBtn = document.getElementById('saveBtn')
  var tabBtns = document.querySelectorAll('.tabBar .tabBtn')
+  var visInstaller = document.getElementById('visInstaller')
+  var visInstallerRp = document.getElementById('visInstallerRp')

  editor.value = INITIAL || ''
  var dirty = false
@@ -23,6 +25,10 @@
    dirtyMark.hidden = !v
  }

+  // 토글이 바뀌어도 dirty 표시. 저장 시 함께 전송된다.
+  if (visInstaller) visInstaller.addEventListener('change', function () { setDirty(true) })
+  if (visInstallerRp) visInstallerRp.addEventListener('change', function () { setDirty(true) })
+
  // ─── markdown 미리 보기용 미니 렌더러 ────────────────────────────────
  // 정식 markdown 파서는 아니지만, 본 편집기가 만들어 내는 형태(#, ##, ###,
  // - , 1. , > , ---, ``` , 토글 details) 정도는 충실히 처리한다.
@@ -162,10 +168,13 @@
  function save() {
    status.classList.remove('error')
    status.textContent = I18N.saving
+    var payload = { content: editor.value }
+    if (visInstaller) payload.showInInstaller = !!visInstaller.checked
+    if (visInstallerRp) payload.showInInstallerRp = !!visInstallerRp.checked
    fetch('/op/agreement/' + encodeURIComponent(PACK_KEY) + '/' + encodeURIComponent(TERM_KIND), {
      method: 'POST',
      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ content: editor.value })
+      body: JSON.stringify(payload)
    }).then(function (r) {
      return r.json().then(function (j) { return { ok: r.ok && j && j.ok !== false, body: j } })
    }).then(function (res) {
--- a/src/installer-rp/main.ts
+++ b/src/installer-rp/main.ts
@@ -252,12 +252,13 @@ ipcMain.handle('rp:packs:select', async (_event, packKey: string) => {
 ipcMain.handle('rp:i18n:dict', () => localeDict)

 // ── IPC: 약관 다운로드 ──────────────────────────────
-// 사이트가 /manifest/terms/<packKey>/<kind>.md 로 노출하는 md 파일을 그대로 받아 본문(string) 으로 반환.
-// rp 인스톨러에서는 'resourcepack' 과 'installer-rp' 두 종류만 실제로 사용하지만, 메인
-// 인스톨러와 동일한 화이트리스트를 둬서 사이트 컴포넌트 분류와 1:1 로 매칭되게 한다.
-const TERM_KIND_WHITELIST = new Set(['map', 'resourcepack', 'mod', 'installer', 'installer-rp'])
+// v0.3.4~ : 사이트에서 임의 kind 가 만들어질 수 있으니 5종 화이트리스트 대신
+// kind 형식만 검증한다. 어떤 약관을 rp 인스톨러에 보여줄지는 사이트의 visibility 토글이 결정.
+const TERM_KIND_RE = /^[a-z0-9][a-z0-9-]{0,31}$/
 ipcMain.handle('rp:terms:get', async (_event, kind: string) => {
-  if (!TERM_KIND_WHITELIST.has(kind)) return { ok: false, message: 'unknown term kind' }
+  if (typeof kind !== 'string' || !TERM_KIND_RE.test(kind)) {
+    return { ok: false, message: 'invalid term kind' }
+  }
  if (!state.selectedKey) return { ok: false, message: 'pack not selected' }
  try {
    const url = `${state.baseUrl}/manifest/terms/${encodeURIComponent(state.selectedKey)}/${encodeURIComponent(kind)}.md`
@@ -268,6 +269,31 @@ ipcMain.handle('rp:terms:get', async (_event, kind: string) => {
  }
 })

+// rp 인스톨러용 약관 목록. /manifest/terms/<packKey>/index.json 을 받아
+// showInInstallerRp=true 인 항목만 추려 반환. 비어 있으면 렌더러가 약관 단계를 건너뛴다.
+ipcMain.handle('rp:terms:list', async (): Promise<{ ok: boolean; terms?: Array<{ kind: string; label: string }>; message?: string }> => {
+  if (!state.selectedKey) return { ok: false, message: 'pack not selected' }
+  try {
+    const url = `${state.baseUrl}/manifest/terms/${encodeURIComponent(state.selectedKey)}/index.json`
+    const buf = await fetchBuffer(url)
+    const parsed = JSON.parse(buf.toString('utf8')) as { terms?: unknown }
+    const items = Array.isArray(parsed.terms) ? parsed.terms : []
+    const terms: Array<{ kind: string; label: string }> = []
+    for (const it of items) {
+      if (!it || typeof it !== 'object') continue
+      const entry = it as Record<string, unknown>
+      if (entry.showInInstallerRp !== true) continue
+      const kind = typeof entry.kind === 'string' ? entry.kind : ''
+      const label = typeof entry.label === 'string' ? entry.label : ''
+      if (!TERM_KIND_RE.test(kind) || label.length === 0) continue
+      terms.push({ kind, label })
+    }
+    return { ok: true, terms }
+  } catch (error) {
+    return { ok: false, message: (error as Error).message }
+  }
+})
+
 // ── IPC: 2단계 설치 ──────────────────────────────────
 ipcMain.handle('rp:install:start', async (): Promise<{ resourcepackPath: string }> => {
  if (!state.selectedKey) throw new Error(t('errors.selectPackFirst'))
--- a/src/installer-rp/pack.ts
+++ b/src/installer-rp/pack.ts
@@ -131,6 +131,10 @@ export async function buildResourcepackZip(opts: BuildResourcepackOptions): Prom
  opts.log?.(t('log.packFormatRange', { min: minFmt, max: maxFmt }))

  // 2) 음악 파일 복사 + sounds.json 생성/병합
+  // 핵심 정책: 베이스 리소스팩에 이미 있는 자산은 절대 덮어쓰지 않는다.
+  // - 베이스 sounds.json 의 엔트리는 그대로 보존하고, 우리 트랙은 그 위에 "추가" 만 한다.
+  // - 베이스 sounds/track_NN.ogg 가 이미 있으면 덮어쓰지 않고 건너뛴다.
+  // - 키나 파일명이 충돌하면 우리 트랙을 스킵하고 로그로 알린다.
  const musicFiles = (await fs.readdir(opts.musicDir))
    .filter((n) => n.toLowerCase().endsWith('.ogg'))
    .sort()
@@ -152,7 +156,19 @@ export async function buildResourcepackZip(opts: BuildResourcepackOptions): Prom
    // NN.ogg → track_NN.ogg 로 리네임해 패키지.
    const stem = path.basename(fname, path.extname(fname)) // "01"
    const trackId = `track_${stem}`
-    await fs.copyFile(path.join(opts.musicDir, fname), path.join(soundsDir, `${trackId}.ogg`))
+    const destFile = path.join(soundsDir, `${trackId}.ogg`)
+    // 베이스에 같은 trackId 의 엔트리/파일이 있으면 두 선택지 다 깨진다:
+    //   (a) 덮어쓰면 베이스의 기존 곡이 사라지고,
+    //   (b) 새 곡을 스킵하면 데이터팩이 가리키는 곡이 빠진 채로 설치된다.
+    // 안전하게 설치를 즉시 실패시키고 어떤 키가 충돌했는지 알린다.
+    let collides = soundsJson[trackId] !== undefined
+    if (!collides) {
+      try { await fs.access(destFile); collides = true } catch { /* 없음 → OK */ }
+    }
+    if (collides) {
+      throw new Error(t('errors.baseTrackCollision', { trackId }))
+    }
+    await fs.copyFile(path.join(opts.musicDir, fname), destFile)
    soundsJson[trackId] = {
      sounds: [
        { name: `${NAMESPACE}:${trackId}`, stream: true }
@@ -160,16 +176,25 @@ export async function buildResourcepackZip(opts: BuildResourcepackOptions): Prom
    }
  }
  await fs.writeFile(soundsJsonPath, JSON.stringify(soundsJson, null, 2) + '\n')
+  opts.log?.(t('log.tracksAdded', { count: musicFiles.length }))
  throwIfCancelled(cancel)

-  // 3) painting 텍스처 복사 (이미 cover_NN.png 형태). 같은 파일명은 덮어씀.
+  // 3) painting 텍스처 복사 (이미 cover_NN.png 형태).
+  // 음악과 동일한 정책: 베이스에 같은 파일명이 이미 있으면 설치를 실패시킨다.
  const paintingFiles = (await fs.readdir(opts.paintingDir))
    .filter((n) => n.toLowerCase().endsWith('.png'))
    .sort()
  for (const fname of paintingFiles) {
    throwIfCancelled(cancel)
-    await fs.copyFile(path.join(opts.paintingDir, fname), path.join(paintingOutDir, fname))
+    const destFile = path.join(paintingOutDir, fname)
+    let collides = false
+    try { await fs.access(destFile); collides = true } catch { /* 없음 → OK */ }
+    if (collides) {
+      throw new Error(t('errors.basePaintingCollision', { name: fname }))
    }
+    await fs.copyFile(path.join(opts.paintingDir, fname), destFile)
+  }
+  opts.log?.(t('log.paintingsAdded', { count: paintingFiles.length }))
  throwIfCancelled(cancel)

  // 4) zip 으로 묶기. 이 단계가 가장 길어서 별도로 cancel 폴링이 들어간다.
--- a/src/installer-rp/preload.ts
+++ b/src/installer-rp/preload.ts
@@ -12,10 +12,14 @@ const api = {
  selectPack: (packKey: string): Promise<void> =>
    ipcRenderer.invoke('rp:packs:select', packKey),

-  /** 약관(Markdown) 다운로드. kind: 'resourcepack' | 'installer-rp'. */
+  /** 약관(Markdown) 다운로드. v0.3.4~ : 임의 kind 허용 (사이트에서 설정). */
  getTerm: (kind: string): Promise<{ ok: boolean; content?: string; message?: string }> =>
    ipcRenderer.invoke('rp:terms:get', kind),

+  /** rp 인스톨러에 표시할 약관 목록 (사이트의 visibility 토글로 필터링). */
+  getTermsList: (): Promise<{ ok: boolean; terms?: Array<{ kind: string; label: string }>; message?: string }> =>
+    ipcRenderer.invoke('rp:terms:list'),
+
  /** 리소스팩 빌드/설치 시작. 완료 또는 취소될 때까지 resolve 되지 않을 수 있음. */
  startInstall: (): Promise<{ resourcepackPath: string }> =>
    ipcRenderer.invoke('rp:install:start'),
--- a/src/installer/main.ts
+++ b/src/installer/main.ts
@@ -155,11 +155,11 @@ ipcMain.handle('packs:load', async (_event, manifestUrlInput?: string): Promise<
 })

 // 약관(Markdown) 을 사이트(/manifest/terms/<packKey>/<kind>.md) 에서 받아와 그대로 돌려준다.
-// 화이트리스트로 5종 제한. pack 미선택 상태에서는 에러를 돌려준다. 네트워크 실패 시 에러 메시지가 그대로 화면에 노출된다.
-const TERM_KIND_WHITELIST = new Set(['map', 'resourcepack', 'mod', 'installer', 'installer-rp'])
+// v0.3.4~ : 사이트에서 임의 kind 등록 가능 → 하드코딩 5종 화이트리스트 대신 kind 형식만 검증.
+const TERM_KIND_RE = /^[a-z0-9][a-z0-9-]{0,31}$/
 ipcMain.handle('terms:get', async (_event, kind: string): Promise<{ ok: boolean; content?: string; message?: string }> => {
-  if (!TERM_KIND_WHITELIST.has(kind)) {
-    return { ok: false, message: 'unknown term kind' }
+  if (typeof kind !== 'string' || !TERM_KIND_RE.test(kind)) {
+    return { ok: false, message: 'invalid term kind' }
  }
  if (!state.selectedKey) {
    return { ok: false, message: 'pack not selected' }
@@ -173,6 +173,31 @@ ipcMain.handle('terms:get', async (_event, kind: string): Promise<{ ok: boolean;
  }
 })

+// 메인 인스톨러용 약관 목록. /manifest/terms/<packKey>/index.json 을 받아
+// showInInstaller=true 인 항목만 추려 반환. 비어 있으면 렌더러가 약관 단계를 건너뛴다.
+ipcMain.handle('terms:list', async (): Promise<{ ok: boolean; terms?: Array<{ kind: string; label: string }>; message?: string }> => {
+  if (!state.selectedKey) return { ok: false, message: 'pack not selected' }
+  try {
+    const url = `${state.baseUrl}/manifest/terms/${encodeURIComponent(state.selectedKey)}/index.json`
+    const buf = await fetchBuffer(url)
+    const parsed = JSON.parse(buf.toString('utf8')) as { terms?: unknown }
+    const items = Array.isArray(parsed.terms) ? parsed.terms : []
+    const terms: Array<{ kind: string; label: string }> = []
+    for (const it of items) {
+      if (!it || typeof it !== 'object') continue
+      const entry = it as Record<string, unknown>
+      if (entry.showInInstaller !== true) continue
+      const kind = typeof entry.kind === 'string' ? entry.kind : ''
+      const label = typeof entry.label === 'string' ? entry.label : ''
+      if (!TERM_KIND_RE.test(kind) || label.length === 0) continue
+      terms.push({ kind, label })
+    }
+    return { ok: true, terms }
+  } catch (error) {
+    return { ok: false, message: (error as Error).message }
+  }
+})
+
 ipcMain.handle('packs:select', async (_event, packKey: string) => {
  if (!state.packs.has(packKey)) {
    throw new Error(t('errors.packNotFound'))
--- a/src/installer/preload.ts
+++ b/src/installer/preload.ts
@@ -14,6 +14,9 @@ const api = {
  // 약관(Markdown) 다운로드
  getTerm: (kind: string): Promise<{ ok: boolean; content?: string; message?: string }> =>
    ipcRenderer.invoke('terms:get', kind),
+  // 메인 인스톨러용 약관 목록 (사이트의 visibility 토글에 따라 필터링됨)
+  getTermsList: (): Promise<{ ok: boolean; terms?: Array<{ kind: string; label: string }>; message?: string }> =>
+    ipcRenderer.invoke('terms:list'),

  // 3-1
  pickFolder: (): Promise<string | null> => ipcRenderer.invoke('dialog:pickFolder'),
--- a/src/server/app.ts
+++ b/src/server/app.ts
@@ -6,7 +6,9 @@ import {
  manifestRootPath, manifestDirPath, manifestTermsDirPath,
  fileDirPath, viewsDirPath, publicDirPath
 } from '../shared/paths.js'
-import { ensurePackTermsDir, isPublicTermsFile, loadPackDefinition } from '../shared/store.js'
+import {
+  ensurePackTermsDir, isPublicTermsFile, listTermsWithLabels, loadPackDefinition
+} from '../shared/store.js'
 import { loadEnv } from '../shared/env.js'
 import { t, localeDict } from './i18n.js'
 import { indexRouter } from './routes/index.js'
@@ -71,6 +73,29 @@ app.get('/manifest.json', (_req, res) => {
 // 요청하는 경우에도 작동하도록, 실제 pack 이면 ensurePackTermsDir 로 v0.3.1
 // 전역 .md 들을 시드 복사한 뒤 sendFile 한다. 임의 packKey 로 빈 폴더가
 // 생성되는 것은 loadPackDefinition 으로 차단.
+// 설치기가 자기에게 표시할 약관 목록을 받아갈 수 있도록 packKey 별 index.json.
+// 응답: [{ kind, label, showInInstaller, showInInstallerRp }]. v0.3.4~ builtin 개념이
+// 없어졌으므로 인스톨러는 이 목록을 받아 자기 인스톨러용(`showInInstaller` / `showInInstallerRp`)
+// 으로 필터링해서 탭을 만든다.
+app.get('/manifest/terms/:packKey/index.json', async (req, res, next) => {
+  try {
+    const { packKey } = req.params
+    if (!/^[a-zA-Z0-9_\-]+$/.test(packKey)) {
+      res.status(404).json({ terms: [] })
+      return
+    }
+    const pack = await loadPackDefinition(packKey)
+    if (!pack) {
+      res.status(404).json({ terms: [] })
+      return
+    }
+    const terms = await listTermsWithLabels(packKey)
+    res.json({ terms })
+  } catch (error) {
+    next(error)
+  }
+})
+
 app.get('/manifest/terms/:packKey/:fileName', async (req, res, next) => {
  try {
    const { packKey, fileName } = req.params
--- a/src/server/routes/op.ts
+++ b/src/server/routes/op.ts
@@ -5,9 +5,8 @@ import {
  createTerm,
  deletePackKeys,
  deleteTerm,
-  getTermLabel,
+  getTermEntry,
  importTerms,
-  isBuiltinTermKind,
  isTermKind,
  listPackKeys,
  listTermsWithLabels,
@@ -20,7 +19,8 @@ import {
  renamePack,
  sanitizePackKey,
  saveTerm,
-  savePackList
+  savePackList,
+  setTermVisibility
 } from '../../shared/store.js'
 import { fetchReleaseVersions } from '../../shared/mojang.js'
 import { fetchPlaylistEntries, fetchVideoMeta, YtDlpUnavailableError } from '../youtube.js'
@@ -306,8 +306,9 @@ opRouter.get('/op/datapack/:packName/images-zip', requireAuth, async (req, res,

 // ─── /op/agreement ─────────────────────────────────────────────────────
 // 약관(Markdown) 편집기. 음악퀴즈(pack) 단위로 따로 저장한다.
-// builtin 5종은 어느 pack 에서나 항상 존재하고 삭제 불가, 그 외 임의 kind 는
-// 사이트에서 추가/삭제 가능. 인스톨러는 /manifest/terms/<packKey>/<kind>.md 로 받아 표시한다.
+// 5종 기본 약관(map/mod/installer/resourcepack/installer-rp) 은 첫 접근 시 시드되지만
+// 사용자가 자유롭게 삭제/추가/표시 대상 변경할 수 있다 (v0.3.4~). 인스톨러는
+// /manifest/terms/<packKey>/index.json 으로 자신에게 표시할 약관 목록을 받는다.

 // /op/agreement → 음악퀴즈 선택(/op/list 와 동일한 카드 형식).
 opRouter.get('/op/agreement', requireAuth, async (req, res, next) => {
@@ -411,10 +412,6 @@ opRouter.post('/op/agreement/:packName/:kind/delete', requireAuth, async (req, r
      res.status(400).send(t('terms.invalidKind'))
      return
    }
-    if (isBuiltinTermKind(kind)) {
-      res.status(400).send(t('terms.cannotDeleteBuiltin'))
-      return
-    }
    await deleteTerm(packKey, kind)
    res.redirect(`/op/agreement/${packKey}`)
  } catch (error) {
@@ -435,14 +432,20 @@ opRouter.get('/op/agreement/:packName/:kind', requireAuth, async (req, res, next
      res.status(404).send(t('errors.unknown'))
      return
    }
+    const entry = await getTermEntry(packKey, kind)
+    if (!entry) {
+      res.status(404).send(t('errors.unknown'))
+      return
+    }
    const content = await loadTerm(packKey, kind)
-    const label = await getTermLabel(packKey, kind)
    res.render('op/termsEditor', {
      userId: req.session.userId,
      packKey,
      pack: definition,
      kind,
-      label,
+      label: entry.label,
+      showInInstaller: entry.showInInstaller,
+      showInInstallerRp: entry.showInInstallerRp,
      content
    })
  } catch (error) {
@@ -465,6 +468,17 @@ opRouter.post('/op/agreement/:packName/:kind', requireAuth, async (req, res, nex
    }
    const content = typeof req.body?.content === 'string' ? req.body.content : ''
    await saveTerm(packKey, kind, content)
+    // visibility 토글이 함께 전송되면 동시에 갱신. 두 값이 모두 false 면 어디에도
+    // 표시되지 않지만 사용자가 의도적으로 선택한 결과이므로 그대로 저장한다.
+    if (
+      typeof req.body?.showInInstaller === 'boolean'
+      || typeof req.body?.showInInstallerRp === 'boolean'
+    ) {
+      await setTermVisibility(packKey, kind, {
+        showInInstaller: req.body.showInInstaller === true,
+        showInInstallerRp: req.body.showInInstallerRp === true
+      })
+    }
    res.json({ ok: true })
  } catch (error) {
    next(error)
--- a/src/shared/store.ts
+++ b/src/shared/store.ts
@@ -318,24 +318,31 @@ export async function savePackList(packKey: string, list: PackList): Promise<voi
 // ─── Terms (Markdown 약관) ─────────────────────────────────────────────
 // 사이트와 인스톨러가 약관을 보여주기 위해 사용하는 markdown 파일.
 // - 음악퀴즈(pack)별로 독립 폴더(`manifest/terms/<packKey>/`) 에 저장한다.
-// - 5개 builtin 종류는 인스톨러가 직접 참조하므로 삭제할 수 없다.
-// - 그 외 임의 kind 는 사이트에서 추가/삭제 가능. 라벨은 `<packKey>/_meta.json` 에 저장.
+// - 각 약관(.md) 은 `_meta.json` 의 `terms.<kind>` 엔트리로 라벨/표시 대상이 관리된다.
+//   엔트리: { label, showInInstaller, showInInstallerRp }
+// - 모든 약관은 추가/삭제 가능. builtin 같은 보호 개념은 더 이상 없음 (v0.3.4~).
+//   인스톨러는 하드코딩 5종 대신 `index.json` 에서 자기 인스톨러용 약관 목록을 받는다.
+// - 첫 접근 시 5개 기본 약관(map/mod/installer + resourcepack/installer-rp) 을 시드.
 // - 파일명 규칙: `[a-z0-9][a-z0-9-]{0,31}\.md` (소문자/숫자/하이픈, 32자 이내).
 // - 레거시(전역) `manifest/terms/*.md` 파일이 남아 있으면 packKey 폴더 첫 접근 시 자동 시드.
 export type TermKind = string

-/** 인스톨러가 하드코딩으로 참조하는 builtin kind. 삭제 금지. */
-export const BUILTIN_TERM_KINDS = ['map', 'resourcepack', 'mod', 'installer', 'installer-rp'] as const
-export type BuiltinTermKind = typeof BUILTIN_TERM_KINDS[number]
-
-/** builtin 라벨. 사용자 정의 kind 는 _meta.json 에 저장된 라벨을 쓴다. */
-const BUILTIN_TERM_LABELS: Record<BuiltinTermKind, string> = {
-  'map': '맵 약관',
-  'resourcepack': '리소스팩 약관',
-  'mod': '모드 약관',
-  'installer': '설치기 약관',
-  'installer-rp': '리소스팩 설치기 약관'
-}
+/**
+ * 처음 pack 폴더를 만들 때 시드되는 기본 약관 5종 + 기본 표시 대상.
+ * 사용자는 이후 자유롭게 삭제하거나 표시 대상을 바꿀 수 있다.
+ */
+const DEFAULT_TERM_SEEDS: Array<{
+  kind: string
+  label: string
+  showInInstaller: boolean
+  showInInstallerRp: boolean
+}> = [
+  { kind: 'map',          label: '맵 약관',           showInInstaller: true,  showInInstallerRp: false },
+  { kind: 'mod',          label: '모드 약관',         showInInstaller: true,  showInInstallerRp: false },
+  { kind: 'installer',    label: '설치기 약관',       showInInstaller: true,  showInInstallerRp: false },
+  { kind: 'resourcepack', label: '리소스팩 약관',     showInInstaller: false, showInInstallerRp: true  },
+  { kind: 'installer-rp', label: '리소스팩 설치기 약관', showInInstaller: false, showInInstallerRp: true  }
+]

 const TERM_KIND_RE = /^[a-z0-9][a-z0-9-]{0,31}$/

@@ -343,13 +350,14 @@ export function isTermKind(value: unknown): value is TermKind {
  return typeof value === 'string' && TERM_KIND_RE.test(value)
 }

-export function isBuiltinTermKind(value: string): value is BuiltinTermKind {
-  return (BUILTIN_TERM_KINDS as readonly string[]).includes(value)
+export interface TermEntry {
+  label: string
+  showInInstaller: boolean
+  showInInstallerRp: boolean
 }

 interface TermsMeta {
-  /** 사용자 정의 kind 라벨. builtin 은 들어가지 않는다. */
-  customLabels: Record<string, string>
+  terms: Record<string, TermEntry>
 }

 const TERMS_META_FILE = '_meta.json'
@@ -375,28 +383,19 @@ function isValidPackKey(packKey: string): boolean {
 */
 export async function ensurePackTermsDir(packKey: string): Promise<string> {
  const dir = termsDirForPack(packKey)
+  let isNew = false
  try {
    await fsp.access(dir)
-    return dir
  } catch (error) {
    if ((error as NodeJS.ErrnoException).code !== 'ENOENT') throw error
-  }
+    isNew = true
    await fsp.mkdir(dir, { recursive: true })
-  // 레거시 전역 파일을 시드로 복사.
+    // 레거시(전역) .md 파일이 남아 있으면 그대로 복사 (.md 만, _meta.json 은 새 스키마로 새로 씀).
    try {
      const legacyEntries = await fsp.readdir(manifestTermsDirPath, { withFileTypes: true })
      for (const ent of legacyEntries) {
        if (!ent.isFile()) continue
        const name = ent.name
-      if (name === TERMS_META_FILE) {
-        try {
-          await fsp.copyFile(
-            path.join(manifestTermsDirPath, name),
-            path.join(dir, name)
-          )
-        } catch { /* ignore */ }
-        continue
-      }
        if (!name.toLowerCase().endsWith('.md')) continue
        const kind = name.slice(0, -3)
        if (!TERM_KIND_RE.test(kind)) continue
@@ -407,26 +406,125 @@ export async function ensurePackTermsDir(packKey: string): Promise<string> {
          )
        } catch { /* ignore */ }
      }
+    } catch (error2) {
+      if ((error2 as NodeJS.ErrnoException).code !== 'ENOENT') throw error2
+    }
+  }
+  // 폴더가 새로 만들어졌든 기존이든, _meta.json 이 없거나 구 스키마면 5종 기본 + .md 매칭으로 보완.
+  await ensureMetaInitialized(dir, isNew)
+  return dir
+}
+
+/**
+ * `_meta.json` 이 없으면 5종 기본 + 디스크 .md 매칭으로 새로 작성한다.
+ * 구 스키마(`customLabels`) 가 있으면 새 스키마(`terms`) 로 변환한다.
+ * 이미 새 스키마면 그대로 둔다 (사용자가 끈 visibility 가 다시 켜지지 않도록).
+ */
+async function ensureMetaInitialized(dir: string, dirWasJustCreated: boolean): Promise<void> {
+  const metaPath = path.join(dir, TERMS_META_FILE)
+  let parsed: unknown = null
+  try {
+    const raw = await fsp.readFile(metaPath, 'utf8')
+    parsed = JSON.parse(raw)
  } catch (error) {
    if ((error as NodeJS.ErrnoException).code !== 'ENOENT') throw error
  }
-  return dir
+
+  // 이미 새 스키마면 종료. 빠진 default kind 가 디스크에 있다면 그것만 보충.
+  if (parsed && typeof parsed === 'object' && (parsed as Record<string, unknown>).terms) {
+    const meta = parsed as { terms: Record<string, unknown> }
+    let changed = false
+    for (const seed of DEFAULT_TERM_SEEDS) {
+      if (meta.terms[seed.kind]) continue
+      // .md 가 실제로 디스크에 있을 때만 보충 (없는 약관까지 자동 부활시키지 않음).
+      try {
+        await fsp.access(path.join(dir, `${seed.kind}.md`))
+      } catch {
+        continue
+      }
+      meta.terms[seed.kind] = {
+        label: seed.label,
+        showInInstaller: seed.showInInstaller,
+        showInInstallerRp: seed.showInInstallerRp
+      }
+      changed = true
+    }
+    if (changed) {
+      await fsp.writeFile(metaPath, `${JSON.stringify(meta, null, 2)}\n`, 'utf8')
+    }
+    return
+  }
+
+  // 구 스키마 customLabels 만 있던 경우 → 새 스키마로 변환.
+  const oldCustomLabels: Record<string, string> = {}
+  if (parsed && typeof parsed === 'object' && (parsed as Record<string, unknown>).customLabels
+      && typeof (parsed as Record<string, unknown>).customLabels === 'object') {
+    for (const [k, v] of Object.entries((parsed as { customLabels: Record<string, unknown> }).customLabels)) {
+      if (typeof v === 'string' && TERM_KIND_RE.test(k)) oldCustomLabels[k] = v
+    }
+  }
+
+  const terms: Record<string, TermEntry> = {}
+  // 5종 기본: 디스크에 .md 가 있을 때만 추가 (없는 건 사용자가 의도적으로 지운 것일 수 있음).
+  // 다만 폴더가 막 생성된 경우는 5종을 무조건 시드 (legacy 시드가 비어 있어도).
+  for (const seed of DEFAULT_TERM_SEEDS) {
+    if (!dirWasJustCreated) {
+      try {
+        await fsp.access(path.join(dir, `${seed.kind}.md`))
+      } catch {
+        continue
+      }
+    } else {
+      // 폴더 새로 생성 케이스: .md 가 없으면 빈 파일 만들어 줌.
+      const filePath = path.join(dir, `${seed.kind}.md`)
+      try {
+        await fsp.access(filePath)
+      } catch {
+        await fsp.writeFile(filePath, `# ${seed.label}\n\n`, 'utf8')
+      }
+    }
+    terms[seed.kind] = {
+      label: seed.label,
+      showInInstaller: seed.showInInstaller,
+      showInInstallerRp: seed.showInInstallerRp
+    }
+  }
+  // 구 스키마의 사용자 정의 약관은 양쪽 인스톨러에 보이도록 기본값으로.
+  for (const [k, label] of Object.entries(oldCustomLabels)) {
+    if (terms[k]) continue
+    try {
+      await fsp.access(path.join(dir, `${k}.md`))
+    } catch {
+      continue
+    }
+    terms[k] = { label, showInInstaller: true, showInInstallerRp: true }
+  }
+  await fsp.writeFile(metaPath, `${JSON.stringify({ terms }, null, 2)}\n`, 'utf8')
 }

 async function loadTermsMeta(packKey: string): Promise<TermsMeta> {
  const dir = await ensurePackTermsDir(packKey)
  try {
    const raw = await fsp.readFile(path.join(dir, TERMS_META_FILE), 'utf8')
-    const parsed = JSON.parse(raw)
-    const customLabels: Record<string, string> = {}
-    if (parsed && typeof parsed === 'object' && parsed.customLabels && typeof parsed.customLabels === 'object') {
-      for (const [k, v] of Object.entries(parsed.customLabels as Record<string, unknown>)) {
-        if (typeof v === 'string' && TERM_KIND_RE.test(k)) customLabels[k] = v
+    const parsed = JSON.parse(raw) as unknown
+    const result: TermsMeta = { terms: {} }
+    if (parsed && typeof parsed === 'object' && (parsed as Record<string, unknown>).terms
+        && typeof (parsed as Record<string, unknown>).terms === 'object') {
+      for (const [k, v] of Object.entries((parsed as { terms: Record<string, unknown> }).terms)) {
+        if (!TERM_KIND_RE.test(k)) continue
+        if (!v || typeof v !== 'object') continue
+        const entry = v as Record<string, unknown>
+        const label = typeof entry.label === 'string' ? entry.label : k
+        result.terms[k] = {
+          label,
+          showInInstaller: entry.showInInstaller === true,
+          showInInstallerRp: entry.showInInstallerRp === true
        }
      }
-    return { customLabels }
+    }
+    return result
  } catch (error) {
-    if ((error as NodeJS.ErrnoException).code === 'ENOENT') return { customLabels: {} }
+    if ((error as NodeJS.ErrnoException).code === 'ENOENT') return { terms: {} }
    throw error
  }
 }
@@ -443,49 +541,83 @@ async function saveTermsMeta(packKey: string, meta: TermsMeta): Promise<void> {
 export interface TermItem {
  kind: string
  label: string
-  builtin: boolean
+  showInInstaller: boolean
+  showInInstallerRp: boolean
 }

 /**
- * 디스크의 .md 파일 + _meta.json 을 합쳐 약관 목록을 만든다.
- * - builtin 5종은 파일 존재 여부와 무관하게 항상 포함된다 (인스톨러가 fetch 하므로).
- * - 디스크에 있고 _meta.json 에 라벨이 있는 사용자 정의 kind 도 포함.
- * - builtin → 사용자 정의 순서, builtin 내부는 BUILTIN_TERM_KINDS 정의 순서를 유지.
+ * 디스크의 .md 파일과 매칭되면서 `_meta.json` 의 `terms` 에 등록된 약관 목록을 반환.
+ * 정렬: 5종 기본(DEFAULT_TERM_SEEDS 순서) → 그 외 사용자 정의 (kind 사전순).
 */
 export async function listTermsWithLabels(packKey: string): Promise<TermItem[]> {
  const dir = await ensurePackTermsDir(packKey)
  const meta = await loadTermsMeta(packKey)
-  const items: TermItem[] = []
-  for (const kind of BUILTIN_TERM_KINDS) {
-    items.push({ kind, label: BUILTIN_TERM_LABELS[kind], builtin: true })
-  }
-  // 디스크에 실제로 존재하는 사용자 정의 .md 파일만 노출.
  let onDisk: string[] = []
  try {
    onDisk = await fsp.readdir(dir)
  } catch (error) {
    if ((error as NodeJS.ErrnoException).code !== 'ENOENT') throw error
  }
-  const customKinds = new Set<string>()
+  const mdKinds = new Set<string>()
  for (const fname of onDisk) {
    if (!fname.toLowerCase().endsWith('.md')) continue
    const kind = fname.slice(0, -3)
    if (!TERM_KIND_RE.test(kind)) continue
-    if (isBuiltinTermKind(kind)) continue
-    customKinds.add(kind)
+    mdKinds.add(kind)
  }
-  // _meta.json 에 라벨이 등록된 것만 노출 (라벨 없는 orphan .md 는 무시).
-  for (const kind of Object.keys(meta.customLabels).sort((a, b) => a.localeCompare(b, 'ko'))) {
-    if (!customKinds.has(kind)) continue
-    items.push({ kind, label: meta.customLabels[kind], builtin: false })
+  const items: TermItem[] = []
+  const seen = new Set<string>()
+  // 1) 기본 시드 순서 우선.
+  for (const seed of DEFAULT_TERM_SEEDS) {
+    const entry = meta.terms[seed.kind]
+    if (!entry) continue
+    if (!mdKinds.has(seed.kind)) continue
+    items.push({
+      kind: seed.kind,
+      label: entry.label,
+      showInInstaller: entry.showInInstaller,
+      showInInstallerRp: entry.showInInstallerRp
+    })
+    seen.add(seed.kind)
+  }
+  // 2) 그 외 사용자 정의: 사전순.
+  const rest = Object.keys(meta.terms).filter((k) => !seen.has(k))
+  rest.sort((a, b) => a.localeCompare(b, 'ko'))
+  for (const kind of rest) {
+    if (!mdKinds.has(kind)) continue
+    const entry = meta.terms[kind]
+    items.push({
+      kind,
+      label: entry.label,
+      showInInstaller: entry.showInInstaller,
+      showInInstallerRp: entry.showInInstallerRp
+    })
  }
  return items
 }

 export async function getTermLabel(packKey: string, kind: string): Promise<string> {
-  if (isBuiltinTermKind(kind)) return BUILTIN_TERM_LABELS[kind]
  const meta = await loadTermsMeta(packKey)
-  return meta.customLabels[kind] ?? kind
+  return meta.terms[kind]?.label ?? kind
+}
+
+export async function getTermEntry(packKey: string, kind: string): Promise<TermEntry | null> {
+  const meta = await loadTermsMeta(packKey)
+  return meta.terms[kind] ?? null
+}
+
+export async function setTermVisibility(
+  packKey: string,
+  kind: string,
+  visibility: { showInInstaller: boolean; showInInstallerRp: boolean }
+): Promise<void> {
+  if (!isTermKind(kind)) throw new Error('invalid term kind')
+  const meta = await loadTermsMeta(packKey)
+  const entry = meta.terms[kind]
+  if (!entry) throw new Error('term not found')
+  entry.showInInstaller = !!visibility.showInInstaller
+  entry.showInInstallerRp = !!visibility.showInInstallerRp
+  await saveTermsMeta(packKey, meta)
 }

 export async function loadTerm(packKey: string, kind: TermKind): Promise<string> {
@@ -508,14 +640,17 @@ export async function saveTerm(packKey: string, kind: TermKind, markdown: string
  await fsp.writeFile(filePath, normalized.endsWith('\n') ? normalized : `${normalized}\n`, 'utf8')
 }

-/** 새로운 사용자 정의 약관 추가. kind 충돌/builtin 충돌은 예외. 빈 .md 파일을 만든다. */
+/**
+ * 새 약관 추가. kind 충돌은 예외. 빈 `.md` 파일을 만든다.
+ * v0.3.4~: builtin 보호 개념이 없어 임의 kind 를 추가/삭제할 수 있다. 다만
+ * `meta.terms` 에 이미 있는 kind 와 충돌하면 거부. 표시 대상 기본값은 양쪽 인스톨러 모두.
+ */
 export async function createTerm(packKey: string, kind: string, label: string): Promise<void> {
  if (!isTermKind(kind)) throw new Error('invalid term kind')
-  if (isBuiltinTermKind(kind)) throw new Error('builtin term kind cannot be created')
  const cleanLabel = label.trim()
  if (cleanLabel.length === 0 || cleanLabel.length > 50) throw new Error('invalid label length')
  const meta = await loadTermsMeta(packKey)
-  if (meta.customLabels[kind]) throw new Error('term kind already exists')
+  if (meta.terms[kind]) throw new Error('term kind already exists')
  const dir = await ensurePackTermsDir(packKey)
  const filePath = path.join(dir, `${kind}.md`)
  // 파일 충돌도 막는다 (수동 생성된 .md 가 있을 수 있음).
@@ -526,14 +661,19 @@ export async function createTerm(packKey: string, kind: string, label: string):
    if ((error as NodeJS.ErrnoException).code !== 'ENOENT') throw error
  }
  await fsp.writeFile(filePath, `# ${cleanLabel}\n\n`, 'utf8')
-  meta.customLabels[kind] = cleanLabel
+  // 기본 시드 kind 면 그 시드의 visibility 기본을 따르고, 그 외는 양쪽 인스톨러 모두 표시.
+  const seed = DEFAULT_TERM_SEEDS.find((s) => s.kind === kind)
+  meta.terms[kind] = {
+    label: cleanLabel,
+    showInInstaller: seed ? seed.showInInstaller : true,
+    showInInstallerRp: seed ? seed.showInInstallerRp : true
+  }
  await saveTermsMeta(packKey, meta)
 }

-/** 사용자 정의 약관 삭제. builtin 은 거부. */
+/** 약관 삭제. v0.3.4~: builtin 보호 없음 — 모든 kind 삭제 가능. */
 export async function deleteTerm(packKey: string, kind: string): Promise<void> {
  if (!isTermKind(kind)) throw new Error('invalid term kind')
-  if (isBuiltinTermKind(kind)) throw new Error('builtin term kind cannot be deleted')
  const dir = await ensurePackTermsDir(packKey)
  const filePath = path.join(dir, `${kind}.md`)
  try {
@@ -542,17 +682,17 @@ export async function deleteTerm(packKey: string, kind: string): Promise<void> {
    if ((error as NodeJS.ErrnoException).code !== 'ENOENT') throw error
  }
  const meta = await loadTermsMeta(packKey)
-  if (meta.customLabels[kind]) {
-    delete meta.customLabels[kind]
+  if (meta.terms[kind]) {
+    delete meta.terms[kind]
    await saveTermsMeta(packKey, meta)
  }
 }

 /**
 * 다른 음악퀴즈의 약관 전체를 현재 pack 으로 복사한다 (불러오기).
- * - source 의 모든 .md 와 _meta.json 을 target 에 덮어쓴다.
- * - target 에만 있던 사용자 정의 약관은 그대로 둔다 (source 에는 없으니 안 건드림).
- * - 동일한 kind 가 source 에도 있다면 source 값으로 덮어씀.
+ * - source 의 모든 .md 를 target 에 덮어쓴다.
+ * - target 에만 있던 약관 엔트리는 그대로 둔다 (source 에는 없으니 안 건드림).
+ * - 동일한 kind 가 source 에도 있다면 source 의 라벨/표시 대상으로 덮어씀.
 */
 export async function importTerms(targetPackKey: string, sourcePackKey: string): Promise<void> {
  if (!isValidPackKey(targetPackKey) || !isValidPackKey(sourcePackKey)) {
@@ -579,12 +719,12 @@ export async function importTerms(targetPackKey: string, sourcePackKey: string):
    await fsp.copyFile(path.join(sourceDir, name), path.join(targetDir, name))
  }

-  // 사용자 정의 라벨도 source 기준으로 머지 (덮어쓰기).
-  const mergedLabels: Record<string, string> = { ...targetMeta.customLabels }
-  for (const [k, v] of Object.entries(sourceMeta.customLabels)) {
-    mergedLabels[k] = v
+  // 약관 엔트리도 source 기준으로 머지 (덮어쓰기).
+  const mergedTerms: Record<string, TermEntry> = { ...targetMeta.terms }
+  for (const [k, v] of Object.entries(sourceMeta.terms)) {
+    mergedTerms[k] = { ...v }
  }
-  await saveTermsMeta(targetPackKey, { customLabels: mergedLabels })
+  await saveTermsMeta(targetPackKey, { terms: mergedTerms })
 }

 /**
--- a/views/op/terms-pack.ejs
+++ b/views/op/terms-pack.ejs
@@ -21,11 +21,19 @@
    .termsRow .termsRowLabel h2 { margin: 0; font-size: 16px; }
    .termsRow .termsRowSub { color: var(--text-muted); font-size: 12px; margin-top: 2px; }
    .termsRow .termsRowActions { display: flex; gap: 8px; align-items: center; }
-    .builtinBadge {
-      display: inline-block; padding: 2px 8px; border-radius: 999px;
-      background: rgba(255,255,255,0.08); color: var(--text-muted);
+    .visibilityBadges {
+      display: flex; gap: 6px; flex-wrap: wrap;
+    }
+    .visibilityBadge {
+      display: inline-flex; align-items: center; padding: 2px 8px; border-radius: 999px;
+      background: rgba(76, 175, 80, 0.15); color: #8ed68f;
+      border: 1px solid rgba(76, 175, 80, 0.35);
      font-size: 11px;
    }
+    .visibilityBadge.off {
+      background: rgba(255,255,255,0.05); color: var(--text-muted);
+      border-color: rgba(255,255,255,0.12);
+    }
    .termsSideBySide {
      display: grid; grid-template-columns: 1fr 1fr; gap: 16px; margin-top: 24px;
    }
@@ -76,21 +84,20 @@
          <a class="termsRowMain" href="/op/agreement/<%= packKey %>/<%= item.kind %>" style="text-decoration:none; color:inherit;">
            <div class="termsRowLabel">
              <h2><%= item.label %></h2>
-              <% if (item.builtin) { %>
-                <span class="builtinBadge"><%= t('terms.builtinBadge') %></span>
-              <% } %>
+              <span class="visibilityBadges">
+                <span class="visibilityBadge <%= item.showInInstaller ? '' : 'off' %>"><%= t('terms.visibilityInstallerShort') %></span>
+                <span class="visibilityBadge <%= item.showInInstallerRp ? '' : 'off' %>"><%= t('terms.visibilityInstallerRpShort') %></span>
+              </span>
            </div>
            <div class="termsRowSub"><%= item.kind %>.md</div>
          </a>
          <div class="termsRowActions">
            <a class="secondaryButton" href="/op/agreement/<%= packKey %>/<%= item.kind %>"><%= t('terms.edit') %></a>
-            <% if (!item.builtin) { %>
            <form method="post" action="/op/agreement/<%= packKey %>/<%= item.kind %>/delete"
                  onsubmit="return confirm('<%= t('terms.deleteConfirm', { label: item.label }).replace(/'/g, "\\'") %>');"
                  style="margin:0;">
              <button type="submit" class="dangerButton"><%= t('terms.deleteButton') %></button>
            </form>
-            <% } %>
          </div>
        </article>
      <% }) %>
--- a/views/op/termsEditor.ejs
+++ b/views/op/termsEditor.ejs
@@ -29,6 +29,19 @@
      <span class="statusText" id="status"></span>
    </div>

+    <!-- 표시 대상 토글: 어느 인스톨러에서 이 약관을 보여줄지 (중복 선택 가능). -->
+    <fieldset class="termsVisibility" style="margin-top:16px; padding:10px 14px; border:1px solid var(--border, #30363d); border-radius:8px;">
+      <legend style="padding:0 6px; font-size:12px; color:var(--text-muted);"><%= t('terms.visibilityHeading') %></legend>
+      <label style="display:inline-flex; align-items:center; gap:6px; margin-right:18px;">
+        <input type="checkbox" id="visInstaller" <%= showInInstaller ? 'checked' : '' %> />
+        <span><%= t('terms.visibilityInstaller') %></span>
+      </label>
+      <label style="display:inline-flex; align-items:center; gap:6px;">
+        <input type="checkbox" id="visInstallerRp" <%= showInInstallerRp ? 'checked' : '' %> />
+        <span><%= t('terms.visibilityInstallerRp') %></span>
+      </label>
+    </fieldset>
+
    <p class="muted" style="font-size:12px;"><%= t('terms.slashHint') %></p>

    <div id="editorWrap" class="termsEditorWrap">
Author	SHA1	Message	Date
claude-bot	5c13648f63	rp-pack: fail-fast on base track/painting collision (was: silent skip) Reviewer correctly flagged that the previous skip-on-collision behavior silently drops new quiz tracks when the base resourcepack already has the same track_NN key. That makes the install LOOK successful but breaks the quiz at runtime (datapack references the missing track). The new behavior throws a clear error explaining which key collided and what the user must do (remove the conflicting base entry, or use a different base). The base assets are still preserved (we never overwrite); we just refuse to build a broken pack. Removed the now-unused skip-summary log keys.	2026-05-23 17:26:41 +09:00
claude-bot	9efd4a696a	rp-pack: never overwrite base resourcepack sounds/paintings (v0.3.5) If the base resourcepack already has audio files under assets/musicquiz/sounds/ or entries in assets/musicquiz/sounds.json, the build now PRESERVES them and skips any new track that would collide. Same policy for painting textures: existing cover_*.png in the base are not overwritten by new ones. Per-track collision is logged so the user can see exactly what was preserved and what was skipped. Summary counts (added / skipped) are also logged. Requested by 사금향: "기존에 있는걸 삭제하거나 이상하게 엎어쓰지 말것" — preserve base assets unconditionally.	2026-05-23 17:18:46 +09:00
claude-bot	c580a50fd4	installer: escape agreement tab labels (XSS hardening) RP installer already escapes k.tab; main installer was injecting it raw. Add escapeHtml helper and apply to tab id/label so admin-supplied agreement labels can't break the HTML.	2026-05-20 10:29:06 +09:00
claude-bot	38df72e4f6	terms: phrase agreement-list failure as install failure (per request) User asked for "약관 표시 실패시 설치 실패로 처리". The block-on-failure path is already in place; this just sharpens the message so users see "설치를 진행할 수 없습니다" rather than a soft retry prompt. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-05-20 10:22:02 +09:00
claude-bot	6447b1cb78	terms: block install on terms list fetch failure (retry UI) Reviewer caught that v0.3.4 was bypassing the agreement step entirely on network/server errors, letting users install without ever seeing terms. Now only the explicit empty-list response (terms:[]) skips the step. Network errors, 404s, and IPC failures render an error page with Back/Retry buttons; no next button is exposed. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-05-20 10:20:49 +09:00
claude-bot	9ba5dc6b7b	terms: per-term installer visibility toggles + universal delete (v0.3.4) - _meta.json: customLabels -> terms.{label,showInInstaller,showInInstallerRp} - Drop builtin protection; any term kind can be deleted/added/toggled - New public route /manifest/terms/<pack>/index.json for installer term lists - Installers fetch terms:list dynamically; skip agreement step if list empty - Term editor: 2 visibility checkboxes (설치기 / 리소스팩 설치기), multi-select - Migration from old schema preserves custom labels (default: visible in both) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-05-20 10:14:42 +09:00