YJ — 턴제 동기화 수정 스냅샷

1. 주요 변경사항

• 서버에 TurnMachine 이 추가되고, 이동·드롭·타임아웃 처리에서 현재 턴 팀만 요청을 통과시키는 검사가 들어갔다.
• S_TurnOver 패킷에 Blue/Red 생명 값을 실어 클라이언트 UI와 로컬 생명 상태를 갱신하려는 흐름이 추가됐다.
• 접속 시 SessionManager 가 세션 순서로 팀을 배정하고 S_Login 으로 클라이언트에 팀 정보를 전달한다.
• 클라이언트 PlayerController 는 타임아웃을 직접 판정하지 않고 서버에 턴 종료를 요청하는 방향으로 일부 이동했다.

2. 코드 품질 리뷰

서버가 턴 권한을 검사하기 시작한 점은 명확한 전진이다. 이전에는 클라이언트가 턴 종료와 게임오버를 직접 판단했으나, 이번 변경으로 C_TurnOverHandler 에서 서버가 생명 차감과 게임오버를 결정한다. 멀티플레이 게임에서 “승패와 자원 감소는 서버가 결정한다”는 방향은 맞다.

다만 구현은 아직 중간 상태다. S_TurnOver 는 생명만 보내고 현재 턴은 보내지 않는다. 클라이언트는 수신 후 pc.EndTurn() 으로 로컬 턴을 넘긴다. 즉 서버 권위처럼 보이지만 실제 턴 상태는 여전히 클라이언트 추론에 의존한다. 패킷 유실, 중복 수신, 재접속, 지연 상황에서 다시 불일치가 생긴다. 개선하려면 서버 응답에 currentTurn, blueLife, redLife, 필요하면 turnSequence 를 포함하고 클라이언트는 그 값을 그대로 반영해야 한다.

pc.UpdateLifeUI(pkt.blueLife, pkt.redLife);
pc.EndTurn();

Assets/Scripts/Network/Session.cs 의 빈 Send 는 위험 신호다. 메서드 호출은 성공처럼 보이지만 실제로는 아무것도 전송하지 않는다. 이런 no-op 네트워크 코드는 런타임 디버깅을 매우 어렵게 만든다. 임시 스텁이면 컴파일 전용임을 명확히 분리하고, 실제 코드 경로에서는 예외를 던지거나 진짜 세션 구현으로 연결해야 한다.

3. 진행도 평가

이번 커밋은 “턴 동기화를 서버 중심으로 옮기려는 시도”라는 점에서 전진이다. 그러나 이동·드롭은 여전히 클라이언트 패킷을 그대로 브로드캐스트하고, 턴 상태도 완전한 서버 스냅샷으로 내려오지 않는다. 그래서 현재 진행도는 핵심 루프가 일부 붙은 30%대 초중반으로 보는 것이 타당하다.

가장 큰 장애물은 네트워크 권한 모델이 코드와 문서 양쪽에서 아직 확정되지 않았다는 점이다. 지금 구조로 기능을 계속 얹으면 버그가 “가끔 턴이 꼬임” 형태로만 보이게 된다. 이 단계에서 하루 정도를 투자해 패킷 방향, 상태 소유자, 재접속 정책을 고정하는 편이 이후 비용을 줄인다.

4. 다음 권장사항

• S_TurnOver 에 CurrentTurn 과 턴 시퀀스를 포함하고, 클라이언트 EndTurn() 추론을 제거한다.
• 이동·드롭 요청은 C_ 로 받고, 서버 검증 후 별도 S_ 결과 패킷만 브로드캐스트한다.
• 빈 Session.Send() 를 실제 전송 구현으로 연결하거나, 임시 스텁이면 런타임 경로에서 제거한다.
• 3명 이상 접속, 접속 해제, 재접속 시 팀 배정 정책을 명시하고 서버에서 방어한다.
• 기술 문서에 상태 소유자와 패킷 흐름을 표로 고정한 뒤 코드와 맞춘다.

5. 문서화 상태

design 점수는 3점이다. 컨셉 문서와 GDD 흔적은 있으나 placeholder 와 문법 깨짐 이슈가 남아 있어 팀원이 디자인 의도를 안정적으로 따라가기 어렵다.

technical 점수는 2점이다. 이번 커밋은 네트워크 권한 모델을 건드렸지만, 서버·클라이언트 상태 소유권, 패킷 방향, 실패 처리 기준이 문서로 남지 않았다. 신규 유입자가 코드를 읽지 않고 구조를 이해할 수 없는 상태다.

spec 점수는 2점이다. 턴 제한, 생명, 승리 조건, 팀 배정, 씬 흐름 같은 실제 테스트 기준이 사양서로 정리되지 않았다. 현재는 구현을 보고 규칙을 역추적해야 하므로 QA 체크리스트로 쓰기 어렵다.

6. Backlog

• 커스텀 네트워크 스택의 권한 모델·프로토콜 사양 부재는 계속 남아 있다.
• 클라이언트 로컬 TurnMachine 이 서버 상태를 완전히 수용하지 않아 지연·재접속 대응이 불안정하다.
• C_/S_ 패킷 명명과 실제 브로드캐스트 방향 불일치가 아직 해결되지 않았다.
• GDD/TDD placeholder, 단일 씬 구조, 포로 재배치 룰 출처 확인은 이번 커밋에서 변경 근거가 없다.
• concept.md 코드블록 미종결 문제도 수정 흔적이 없어 다음 문서 정리 때 함께 처리해야 한다.