installer: port page = check-only (no auto-open), big open/notOpen message

요청 반영: 간편설치기 포트포워딩 페이지가 포트를 직접 열지 않고 "열려 있는지"만
확인한다.
- server:portForward 를 UPnP 자동 개방/재점검 없이 외부 도달 점검만 하도록 변경,
  결과를 open/notOpen 으로 반환(PortForwardResult status 타입도 변경).
- 열려 있으면 외부 접속 주소를 크게, 안 열려 있으면 "직접 포트포워딩 해주세요"
  를 크게 표시.
- 페이지 체크에서만 쓰이던 openPortViaUpnp/removeUpnpMapping 제거(고아 코드).
- 참고: 서버 기동 시 run.bat 의 UPnP 자동 개방 주입(injectUpnpToRunBat)은 이번엔
  유지. v0.3.19.

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

src/installer/main.ts

@@ -860,12 +860,7 @@ ipcMain.handle('server:portForward', async (_event, port: number): Promise<PortF
   const targetPort = Number.isFinite(port) && port > 0 ? port : 25565
   sendLog(t('log.portCheckStart', { port: targetPort }))
 
-  // 1차 점검 전에 우리가 이전 실행에서 만든 UPnP 매핑이 남아 있으면 먼저 제거한다.
-  // 이렇게 해야 "사용자 라우터 규칙이 활성화돼서 외부 접근이 가능한 상태" 와 "UPnP 매핑 덕분에 접근 가능한 상태" 가 구별된다.
-  // 사용자 규칙이 비활성/없으면 1차 점검은 false 가 되어 UPnP 시도 단계로 자연스럽게 넘어간다.
-  sendLog(t('log.upnpCleanup'))
-  await removeUpnpMapping(targetPort)
-
+  // 설치기는 포트를 직접 열지 않는다. 외부에서 이미 열려 있는지만 확인한다.
   // 외부 IP 확보: 공용 API → 실패 시 UPnP 게이트웨이의 외부 IP로 폴백.
   let externalIp = await detectExternalIpHttp()
   if (externalIp) {
@@ -877,9 +872,9 @@ ipcMain.handle('server:portForward', async (_event, port: number): Promise<PortF
     else sendLog(t('log.externalIpUpnpFail'))
   }
 
-  // 1차 점검: 외부에서 이미 접근 가능한지 (서버가 떠 있거나, 우리가 임시 리스너 띄워서 검증).
+  // 외부에서 포트가 닿는지 점검(서버가 떠 있거나, 임시 리스너로 검증).
   sendLog(t('log.probeStart'))
-  let probe = await probePortFromOutside(targetPort, externalIp)
+  const probe = await probePortFromOutside(targetPort, externalIp)
   if (!externalIp && probe.detectedIp) externalIp = probe.detectedIp
   const verdict = probe.reachable === true
     ? t('log.probeVerdictSuccess')
@@ -888,47 +883,9 @@ ipcMain.handle('server:portForward', async (_event, port: number): Promise<PortF
 
   if (probe.reachable === true) {
     sendLog(t('log.probePreForwarded', { addr: externalIp || t('log.ipUnknown'), port: targetPort }))
-    return { status: 'preForwarded', externalIp, port: targetPort }
+    return { status: 'open', externalIp, port: targetPort }
   }
-
-  // UPnP 시도.
-  sendLog(t('log.upnpTryOpen', { port: targetPort }))
-  try {
-    await openPortViaUpnp(targetPort)
-    sendLog(t('log.upnpReqOk'))
-  } catch (error) {
-    const msg = (error as Error).message || String(error)
-    sendLog(t('log.upnpTryFail', { message: msg }))
-    return {
-      status: 'upnpFailed',
-      externalIp,
-      port: targetPort,
-      message: t('log.upnpFailDetail', { message: msg })
-    }
-  }
-
-  // NAT 반영 지연을 고려해 최대 3회 재점검.
-  for (let attempt = 1; attempt <= 3; attempt++) {
-    await sleep(1500)
-    sendLog(t('log.upnpRecheck', { attempt }))
-    probe = await probePortFromOutside(targetPort, externalIp)
-    if (!externalIp && probe.detectedIp) externalIp = probe.detectedIp
-    if (probe.reachable === true) {
-      sendLog(t('log.upnpDone', { port: targetPort }))
-      await removeUpnpMapping(targetPort)
-      return { status: 'upnpOk', externalIp, port: targetPort }
-    }
-  }
-
-  // 테스트 목적으로 만든 매핑 정리. 실제 개방은 run.bat 이 담당.
-  sendLog(t('log.upnpCleanupTest'))
-  await removeUpnpMapping(targetPort)
-
-  const reason = probe.reachable === false
-    ? t('log.upnpFailReason1')
-    : t('log.upnpFailReason2', { detail: probe.detail })
-  sendLog(reason)
-  return { status: 'upnpFailed', externalIp, port: targetPort, message: reason }
+  return { status: 'notOpen', externalIp, port: targetPort }
 })
 
 async function detectExternalIpHttp(): Promise<string> {
@@ -1124,64 +1081,6 @@ function fetchIfconfigCoPortOnce(port: number): Promise<IfconfigPortResult> {
   })
 }
 
-function removeUpnpMapping(port: number): Promise<void> {
-  return new Promise((resolve) => {
-    let settled = false
-    const done = () => { if (!settled) { settled = true; resolve() } }
-    let client: ReturnType<typeof natUpnp.createClient> | null = null
-    try {
-      client = natUpnp.createClient()
-    } catch (err) {
-      sendLog(t('log.upnpClientFailRemove', { message: (err as Error).message }))
-      done()
-      return
-    }
-    const timer = setTimeout(() => {
-      try { client && client.close() } catch {}
-      sendLog(t('log.upnpRemoveTimeout'))
-      done()
-    }, 8000)
-    client.portUnmapping({ public: port, protocol: 'tcp' }, (err: Error | null) => {
-      clearTimeout(timer)
-      try { client && client.close() } catch {}
-      if (err) sendLog(t('log.upnpRemoveAttempt', { message: err.message }))
-      else sendLog(t('log.upnpRemoveDone', { port }))
-      done()
-    })
-  })
-}
-
-function openPortViaUpnp(port: number): Promise<void> {
-  return new Promise((resolve, reject) => {
-    let settled = false
-    const done = (err?: Error) => {
-      if (settled) return
-      settled = true
-      if (err) reject(err)
-      else resolve()
-    }
-    let client: ReturnType<typeof natUpnp.createClient> | null = null
-    try {
-      client = natUpnp.createClient()
-    } catch (err) {
-      done(err as Error)
-      return
-    }
-    const timer = setTimeout(() => {
-      try { client && client.close() } catch {}
-      done(new Error(t('errors.upnpTimeout')))
-    }, 15000)
-    client.portMapping(
-      { public: port, private: port, ttl: 0, description: 'MusicQuiz Server', protocol: 'tcp' },
-      (error: Error | null) => {
-        clearTimeout(timer)
-        try { client && client.close() } catch {}
-        done(error || undefined)
-      }
-    )
-  })
-}
-
 function sleep(ms: number): Promise<void> {
   return new Promise((resolve) => setTimeout(resolve, ms))
 }

src/installer/types.ts

@@ -36,7 +36,8 @@ export interface RamCheckResult {
 }
 
 export interface PortForwardResult {
-  status: 'preForwarded' | 'upnpOk' | 'upnpFailed'
+  // 설치기는 포트를 직접 열지 않고 "열려 있는지"만 확인한다.
+  status: 'open' | 'notOpen'
   externalIp?: string
   port: number
   message?: string