Exit-or-die — 메뉴 이미지 교체 스냅샷
1. 주요 변경사항
Assets/Images/image.png가 삭제되고Assets/Images/Menu.jpg가 추가됐다. 용량은 줄었지만 GUID가 변경되어 참조 안정성 검증이 필요하다.Assets/Scenes/Lobby.unity가Assets/Scenes/Menu.unity로 이름이 바뀌었고, Build Settings의 첫 씬 경로도 Menu로 갱신됐다.- 메뉴 UI 버튼 배치가 중앙 기준에서 좌측 기준으로 변경됐고,
Setting버튼이 새로 추가됐다. Test.unity에torchWall프리팹 인스턴스가 추가됐다. 메뉴 이미지 변경 커밋에 테스트 씬 배치 변경이 섞인 점은 변경 의도가 흐린 편이다.
2. 코드 품질 리뷰
이번 변경은 C# 코드보다 Unity 씬 직렬화 변경이 중심이다. 이미지 교체와 메뉴 씬 이름 변경 자체는 명확한 전진이다. 특히 PNG 약 1MB를 JPG 약 78KB로 바꾼 것은 메뉴 배경 용도라면 합리적인 최적화다. 다만 기존 image.png.meta를 Menu.jpg.meta로 rename하면서 GUID를 새로 바꿨기 때문에, 씬 외부에서 이전 GUID를 참조하던 UI나 프리팹이 있으면 Missing Sprite가 발생할 수 있다. 개선하려면 에셋 교체 후 t:Sprite cf9763... 같은 방식으로 프로젝트 전역 참조를 검사해야 한다.
가장 큰 문제는 새 Setting 버튼의 이벤트 연결이다. 오브젝트 이름과 텍스트는 설정 버튼인데 실행은 로비 목록 갱신이다.
m_Name: Setting
m_TargetAssemblyTypeName: Eod.Network.LobbyManager, Eod.Network
m_MethodName: RefreshLobbyList이 패턴은 UI가 늘어날수록 런타임에서만 발견되는 오동작을 만든다. 왜 문제냐면 Unity Inspector 이벤트는 컴파일 타임 검증이 약해서 이름과 동작이 불일치해도 빌드가 성공한다. 개선하려면 OpenSettingsPanel 같은 명시적 메서드를 만들고, 버튼명·표시 텍스트·이벤트 메서드를 같은 의도로 정렬해야 한다.
씬 이름 변경도 조심해야 한다. Lobby.unity.meta가 그대로 Menu.unity.meta로 이동된 것은 Build Settings GUID 관점에서는 안정적일 수 있으나, 코드에서 "Lobby" 문자열로 씬을 로드한다면 즉시 깨진다. 이번 diff에는 전역 검색 근거가 없으므로 해결됐다고 볼 수 없다.
3. 진행도 평가
이번 커밋은 메뉴 첫인상과 빌드 진입 씬을 정리한 작은 UI 전진이다. 그러나 이전 리포트의 핵심 리스크였던 던전 생성 규약, NGO 프리팹 구조, 네트워크 권한 문서화, 입력 정책, 문서 사양화는 해결 흔적이 없다. 진행도는 기능 완성보다 화면 자산 교체에 가까우므로 큰 폭으로 올릴 수 없다.
커밋 메시지 수정 : 메뉴 이미지 변경은 너무 좁다. 실제로는 씬 rename, Build Settings 변경, UI 버튼 추가, Test 씬 프리팹 추가까지 포함한다. 변경 단위를 분리하지 않으면 리뷰어가 의도하지 않은 씬 변경을 놓치기 쉽다.
4. 다음 권장사항
Menu.unity의Setting버튼 이벤트를 실제 설정 UI 동작으로 교체해야 한다.Lobby문자열 기반 씬 전환 코드가 남아 있는지 전역 검색하고, Menu rename 영향 범위를 검증해야 한다.ClientNetworkTransform과LobbyManager기준으로 네트워크 권한·예측·보정 정책을 문서화해야 한다.- 메뉴 UI를 16:9, 16:10, 21:9, 모바일 비율에서 캡처 검증해야 한다.
- 기획서와 기술설계서에 씬·입력·상태·결과를 테스트 가능한 표로 추가해야 한다.
5. 문서화 상태
design 점수는 4점이다. 컨셉과 방향성은 존재한다고 보지만, 이번 커밋에서 메뉴 구조나 플레이어 진입 흐름의 의사결정 근거가 문서로 보강되지 않았다.
technical 점수는 3점이다. 네트워크가 포함된 프로젝트인데 권한 모델, 씬 전환 흐름, 로비와 메뉴의 책임 분리가 문서화되지 않았다. 신규 팀원이 문서만 보고 안전하게 수정하기 어렵다.
spec 점수는 2점이다. 메뉴 버튼의 상태 전이, 입력 경로, 설정 화면 동작, 로비 갱신 조건 같은 검증 가능한 사양이 부족하다. _sample/docs 수준처럼 표와 체크리스트로 내려와야 테스트 자산이 된다.
6. Backlog
- 던전 시작계단 방향 규약과 바닥 교체 기준 불일치는 아직 남아 있다.
stairs_Lock계열의 중첩NetworkObject프리팹 위험은 해결 근거가 없다.- 네트워크 이동 권한과 클라이언트 예측·보정 정책 문서화가 여전히 필요하다.
Menu.unity의Setting버튼이RefreshLobbyList를 호출하는 오배선은 즉시 고쳐야 한다.- 문서는 여전히 컨셉 중심이며 테스트 가능한 수치·상태 사양이 부족하다.
Assets/_Recovery, 입력 시스템 이중화, 대량 예제 에셋,Deungeon폴더명 문제도 그대로 이월한다.